Skip to content

fix(desktop): resolve effective model and prompt from persona in display path#972

Merged
wpfleger96 merged 1 commit into
mainfrom
wpfleger/fix-persona-model-display
Jun 11, 2026
Merged

fix(desktop): resolve effective model and prompt from persona in display path#972
wpfleger96 merged 1 commit into
mainfrom
wpfleger/fix-persona-model-display

Conversation

@wpfleger96

Copy link
Copy Markdown
Collaborator

The Agents panel and ModelPicker displayed stale model/system_prompt values from ManagedAgentRecord — a creation-time snapshot that never updated when the linked persona changed. The spawn path already resolved the effective values from the persona; this aligns the display path to do the same.

Changes

  • build_managed_agent_summary() now accepts a &[PersonaRecord] slice and resolves the effective model and system_prompt from the linked persona (mirroring the spawn-time logic in resolve_effective_prompt_model_provider())
  • All callers in agents.rs, agent_models.rs, and agent_settings.rs load personas once per handler and pass the slice
  • get_agent_models now resolves the effective model for selected_model so the ModelPicker dropdown shows the correct selection
  • Made resolve_effective_prompt_model_provider pub(crate) for reuse in the display path

Behavior

  • When a persona is linked: summary shows the persona's current model and system_prompt
  • When no persona is linked: falls back to record.model / record.system_prompt (unchanged)
  • No changes to spawn-time behavior (spawn_agent_child untouched)

@wpfleger96 wpfleger96 force-pushed the wpfleger/fix-persona-model-display branch from 5f30b5c to 79461dd Compare June 11, 2026 15:37
…lay path

The Agents panel and ModelPicker displayed stale model/system_prompt
values from ManagedAgentRecord — a creation-time snapshot that never
updated when the linked persona changed. The spawn path already
resolved the effective values from the persona; this aligns the
display path to do the same.

Co-authored-by: Will Pfleger <pfleger.will@gmail.com>
Signed-off-by: Will Pfleger <pfleger.will@gmail.com>
@wpfleger96 wpfleger96 force-pushed the wpfleger/fix-persona-model-display branch from 79461dd to 07c8216 Compare June 11, 2026 17:27
@wpfleger96 wpfleger96 merged commit 6373813 into main Jun 11, 2026
14 checks passed
@wpfleger96 wpfleger96 deleted the wpfleger/fix-persona-model-display branch June 11, 2026 17:29
tlongwell-block pushed a commit that referenced this pull request Jun 12, 2026
Co-authored-by: npub1mprnacetjua2xx3p5eddmhxyk6wv929ymm5py8kd2xfxurxahspqqlgyta <d8473ee32b973aa31a21a65adddcc4b69cc2a8a4dee8121ecd51926e0cddbc02@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1mprnacetjua2xx3p5eddmhxyk6wv929ymm5py8kd2xfxurxahspqqlgyta <d8473ee32b973aa31a21a65adddcc4b69cc2a8a4dee8121ecd51926e0cddbc02@sprout-oss.stage.blox.sqprod.co>

* origin/main: (35 commits)
  feat(huddle): sentence-at-a-time voice-mode guidelines for lower TTS latency (#996)
  Shard desktop Playwright CI jobs (#992)
  chore(release): release version 0.3.18 (#995)
  Video Player Improvements  (#993)
  Improve first-run welcome setup (#970)
  fix(release): use legacy updater key secret (#991)
  Replace built-in personas with Fizz (#987)
  docs(buzz-acp): rewrite Communication Patterns for mention accuracy and threading clarity (#982)
  chore(justfile): build git-credential-nostr in dev and staging recipes (#980)
  Fix Buzz command migration for saved agents (#979)
  fix(desktop): resolve effective model and prompt from persona in display path (#972)
  docs: clean up remaining Buzz references (#977)
  chore(release): release version 0.3.17 (#976)
  fix(onboarding): skip onboarding when relay already has a profile (#973)
  docs: finish Buzz rename cleanup (#974)
  fix(desktop): let channel members bypass mention agent gate (#965)
  Rename desktop app to Buzz (#960)
  feat(desktop): open profile panel from MembersSidebar rows (#962)
  feat(desktop): per-event notification sounds and alert controls (#968)
  fix(desktop): make header chrome zoom-correct and tidy split-pane (#941)
  ...

# Conflicts:
#	crates/buzz-agent/README.md
#	crates/buzz-agent/src/config.rs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant