Skip to content

feat(acp): Add /agent slash command for agent switching#86

Closed
CSRessel wants to merge 1 commit into
devfrom
acp-agent-model-picker
Closed

feat(acp): Add /agent slash command for agent switching#86
CSRessel wants to merge 1 commit into
devfrom
acp-agent-model-picker

Conversation

@CSRessel
Copy link
Copy Markdown
Collaborator

@CSRessel CSRessel commented Dec 9, 2025

Summary

🤖 Generated with Nori

  • Add /agent slash command to switch between ACP agents (Mock, Gemini, Claude)
  • Implement pending selection pattern - agent only switches on next prompt submission
  • Disable /model command in ACP mode with informational message directing users to use /agent
  • Add comprehensive E2E tests for agent picker functionality

The pending selection design ensures subprocess stability - the existing agent subprocess is not dropped while the user navigates the picker during an active turn.

Test Plan

  • Unit tests for list_available_agents() registry function
  • E2E tests for agent picker popup
  • E2E tests for pending selection behavior
  • E2E tests for agent switch on prompt submission
  • E2E tests verifying subprocess stability during picker navigation
  • cargo fmt passes
  • cargo clippy passes

Share Nori with your team: https://www.npmjs.com/package/nori-ai

…d with [Nori](https://nori.ai)\n\nCo-Authored-By: Nori <contact@tilework.tech>

Add ability to switch between ACP agents in the TUI:

- Add list_available_agents() to codex-acp registry

- Add /agent slash command that shows available ACP agents

- Implement pending selection pattern - agent only switches on next prompt submission

- Disable /model command in ACP mode with info message

- Add E2E tests for agent picker functionality

The pending selection design ensures subprocess stability - the existing agent is not dropped while the user navigates the picker during an active turn.
@CSRessel
Copy link
Copy Markdown
Collaborator Author

CSRessel commented Dec 9, 2025

Code is heavily entangled with codex-tui module, and registry design is a little off target, closing in favor of #84

@CSRessel CSRessel closed this Dec 9, 2025
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