Skip to content

fix(app): hide default-only agent picker#2229

Open
src-opn wants to merge 2 commits into
devfrom
agent-dropdown-conditional-show
Open

fix(app): hide default-only agent picker#2229
src-opn wants to merge 2 commits into
devfrom
agent-dropdown-conditional-show

Conversation

@src-opn

@src-opn src-opn commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • hide the composer agent picker when only the default OpenWork agent is available
  • filter the default OpenWork agent out of agent picker lists so non-default agents are the only selectable alternatives
  • refresh agent availability after engine reloads so newly added agents reveal the picker

Tests

  • pnpm --filter @openwork/app typecheck

Evidence

  • Not captured (code-only conditional UI change).

Review in cubic

@vercel

vercel Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview, Comment Jun 13, 2026 5:06pm
openwork-den Ready Ready Preview, Comment Jun 13, 2026 5:06pm
openwork-den-worker-proxy Ready Ready Preview, Comment Jun 13, 2026 5:06pm
openwork-landing Ready Ready Preview, Comment, Open in v0 Jun 13, 2026 5:06pm

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

1 issue found across 2 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

@src-opn

src-opn commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator Author

Addressed the issue identified by cubic: when the agent picker should no longer show, the composer now closes agentMenuOpen so the hidden picker cannot continue capturing keyboard shortcuts.

Verification: pnpm --filter @openwork/app typecheck

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