Conversation
- add the merge command and session_command tool across core and OpenCode surfaces - switch navigator-driven workflows from dispatch tags to literal session_command tags - sync docs, config, schema, generated output, and tests for the new command flow
…ody/agent format - Replace slash command parsing with explicit command/body/agent structure - Update session_command tool to require command parameter - Add command attribute to all session_command tags in command docs - Update navigator agent docs for new session command format - Update all tests to use new API signatures - Update OpenCode adapter tool registration and execution - Update web documentation for session command changes - Add merge command documentation - Add session-command tool documentation
There was a problem hiding this comment.
★★★★★
This is a well-structured migration PR that cleanly replaces the <dispatch> mechanism with the <session_command> tool approach. The changes are consistent across all command documentation, configuration files, schema, tests, and generated output.
Key observations:
-
Complete migration: All
<dispatch>references have been replaced with<session_command>throughout the codebase - command docs, agent docs, AGENTS.md guidelines, compiled output, and tests. -
Correct tag format: All
<session_command>tags use the proper format with bothagentandcommandattributes (e.g.,<session_command agent="worker" command="branch">). -
New implementation: The
packages/core/tools/dispatch.tsintroduces a cleanresolveSessionCommandfunction that properly expands command templates and resolves agent routing. -
OpenCode adapter updates: The adapter correctly uses
client.session.promptAsyncto queue commands instead of the previous expansion approach. -
Tests validate the changes: All 42 tests pass (5 dispatch tests, 28 commands-config tests, 9 tool-registration tests). The only failing test (
pr_load.test.ts) is unrelated to this PR - it's an environmental issue withviewerLoginexpectations. -
Type checking passes: No TypeScript errors.
-
New merge command: The PR also adds a new
mergecommand that follows the established patterns.
Checklist verification:
- ✅ Updated AGENTS.md to reflect session_command usage
- ✅ Verified command authoring guidelines reference correct tags
- ✅ Reviewed navigator delegation patterns in command docs
- ✅ Confirmed session_command examples are accurate
- ✅ Verified all command docs use session_command consistently
- ✅ No dispatch tags remain in documentation
- ✅ Workflow subsections follow the new pattern
No material issues found. The migration is thorough and consistent.
Ticket
SKIPPED
Description
Updates command documentation to use
<session_command>tags instead of<dispatch>tags for navigator-style command delegation. This aligns the docs with the current implementation where session commands are used for same-session dispatch.Checklist
Documentation
Commands
Validation