feat: Model and Persona Badge Display Enhancements#220
Open
kmackett wants to merge 3 commits into
Open
Conversation
Backend (Phase 1): - Created PersonaLoader service to load agent personas from filesystem - Added IPC handlers: list-personas, get-persona, set-persona - Enhanced buildSystemPromptAppend() with persona injection (3-layer: global → project → persona) - Integrated PersonaLoader into AppServices and ClaudeCodeManager Frontend (Phase 2): - Created PersonaSelector component following ModelSelector pattern - Added persona TypeScript types and API methods - Integrated PersonaSelector into ClaudeInputWithImages toolbar - Fixed ArchiveProgress.tsx electron import issue Fixes: - Fixed agentsDirectory path resolution for development mode 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
When model is set to 'auto', show dual badges to differentiate between user's selection and Claude Code's automatic model choice. Changes: - Update RichOutputView.tsx badge display logic - Show 'auto' badge (default variant) + actual model badge (primary variant) - For explicit model selection, show single badge as before - Add database migrations for model_id and persona_name tracking Fixes stravu#219 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add database migrations and backend support for tracking model and persona metadata in conversation messages and prompt markers. Changes: - Add migrations for model_id and persona_name columns - Update database models and initialization - Add IPC handlers for conversation message retrieval - Integrate persona loading and session management - Update PromptNavigation to support metadata display Part of Epic 193 - Persona Management System 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
kmackett
requested review from
SaucyWrong,
ghinkle and
jordan-BAIC
as code owners
Collaborator
|
Looks like some of the linting failed, fix it and I will re run the workflow. I'll try to give it a proper review this weekend or Monday. Thanks for your contribution! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR implements enhanced badge display for model and persona information in conversation messages, providing better visibility into which model and persona were used for each interaction.
Key Features
1. Dual Badge Display for Auto Model Selection
When model is set to 'auto', displays both the user's selection and Claude Code's actual model choice:
This provides clear visual differentiation between:
2. Model and Persona Tracking Infrastructure
model_idandpersona_namecolumnsChanges Made
Frontend
Backend
007_add_prompt_marker_metadata.sql008_add_conversation_message_metadata.sqlTesting
✅ Tested with auto model selection - dual badges display correctly
✅ Tested with explicit model selection - single badge displays correctly
✅ Persona badges display alongside model badges
✅ Database migrations tested and working
Related Issues
Fixes #219
Screenshots
[User would add screenshot showing dual badge display here]
🤖 Generated with Claude Code
Co-Authored-By: Claude noreply@anthropic.com