feat(opencode): restore build and plan modes - #272
Conversation
|
The following comment was made by an LLM, it may be inaccurate: |
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 18a25d3ae4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
18a25d3 to
c191378
Compare
- add /mode as the Build, Plan, and Run product switch - route Build and Plan through native OpenCode instructions with Agent Swarm guidance - keep Run server-backed and enable native Plan handoff to Build - add terminal E2E coverage for mode visibility, routing, and plan approval
- Add /mode as the Build, Plan, and Run product switch - Keep Run server-backed while Build and Plan use native OpenCode mode routing - Add terminal E2E coverage for switching, native prompts, and Run return flow
- keep /mode switches local instead of writing global model config - route Build and Plan slash commands through native OpenCode paths - preserve Run bridge routing when visible native model state is present - add focused model, schema, and terminal TUI coverage
- isolate question key handling with the upstream OpenCode mode stack shape - keep Build and Plan native while preserving /modes docs and E2E coverage - hide the non-natural-language agency agent helper from command help
c191378 to
f072fe2
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. Nice work! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
- clarify Plan handoff docs as upstream OpenCode behavior - fix Plan prompt path assertion for Windows JSON escaping - preserve mixed-mode undo and redo routing by target turn - keep unsupported Build models out of Run routing - remove unrelated and duplicate Agent Builder test coverage
1 participant
Issue for this PR
N/A. Feature PR; issue check is skipped for
feat:titles.Type of change
What does this PR do?
Restores the Agent Swarm CLI product modes behind
/modes:/buildand/planare not added as commands.How did you verify your code works?
Local checks on the latest branch head:
npx --yes bun@1.3.13 run typechecknpx --yes bun@1.3.13 test test/cli/tui/local.test.ts --timeout 120000npx --yes bun@1.3.13 test --timeout 180000 --max-concurrency=1 ../../e2e/agent-swarm-tui/terminal-tui.test.tsnpx --yes bun@1.3.13 turbo test:cinpx --yes bun@1.3.13 run build --single --skip-installnpx --yes bun@1.3.13 /tmp/agent-builder-binary-qa.tsBinary QA proves
/modes -> Plan -> Yesswitches to Build, Build gets Agent Swarm Build instructions, Plan and Build make zero Agency server requests, and Run still reaches the Agency server.Screenshots / recordings
Terminal TUI behavior is covered by automated PTY E2E and binary QA.
Checklist