Skip to content

refactor(agent-modal): revamped UI/UX for the agent modal #1838

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Aug 12, 2025

Conversation

spoons-and-mirrors
Copy link
Contributor

@spoons-and-mirrors spoons-and-mirrors commented Aug 12, 2025

Summary

  • Refactored the TUI agent modal for more consistent UI with models modal
  • Improved user experience with Recent (max 5), Agents, and Subagents categories
  • In list coloring of agent names with their respective color
  • Exposing Subagents in the list so users can use them directly (resolves half of #1832)
  • Future proofing UX for users with many agents and subagents
image

Note

Plan and Build agents description could be added for consistency, I'll leave this for a later date

@spoons-and-mirrors spoons-and-mirrors changed the title refactor(agent-modal): revamped UI for the agent modal refactor(agent-modal): revamped UI/UX for the agent modal Aug 12, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found several violations of AGENTS.md guidelines:

  1. Line 319 in app.go: Variable name agentName violates "PREFER single word variable names where possible". Consider renaming to agent or name.

  2. Line 274 in app.go: Variable name agentName violates the single word variable naming guideline.

  3. Line 39 in app.go: Variable name agentUsage could be simplified to usage.

  4. Line 90 in state.go: Variable name agentName should be agent or name per guidelines.

  5. Line 331-337 in agents.go: The nested if-else structure could be simplified by removing unnecessary else statements as per "AVOID else statements".

These changes would improve code consistency with the project guidelines.

@thdxr thdxr merged commit 81583cd into sst:dev Aug 12, 2025
1 check passed
yokowasis pushed a commit to yokowasis/opencode-sst that referenced this pull request Aug 14, 2025
@spoons-and-mirrors spoons-and-mirrors deleted the feat/modal-agent-revamped branch August 14, 2025 19:04
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.

2 participants