fix(opencode): scope project agents to their workspace#21532

Approvability

Verdict: Needs Human Approval

Macroscope's review found this PR requires human approval — This change adds workspace-specific OpenCode agent discovery and selection across server, web, and mobile, including changing which agent executes prompts and native commands. It also introduces workspace-dependent agent defaults and new discovery timeout behavior, making the runtime and product-default impact broader than a routine bug fix.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

No code changes detected at 5477dba. Prior analysis still applies.

^{You can add or adjust custom eligibility rules. Learn more.}

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Baseline: 7208335 · PR result: 5477dba · Source CI: success

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Usage-based review receipt

• Mode: Continue automatically
• Reviewed files: 10
• Charged: $2.50
• View usage details

_{Comment @coderabbitai help to get the list of available commands.}

This would be incredibly helpful 🙏🏻

Another report for the OpenCode 2 half of this PR.

On OpenCode 2 (@opencode/cli), the Agent select is missing from the model options for every model, including with only global agents and no project-local ones. So this isn't limited to project-local agents. The OpenCode 2 catalog path in status.ts only builds the Reasoning option, and the OpenCode 2 adapter's agentFor always returns build, or plan in plan mode. OpenCode 1.x shows the picker as expected.

I'd like this PR to land, because it covers both parts: discovering the agents and then using the selected one. I reproduced the same two gaps independently and wrote a replay test for the execution half: with the Agent option set to a custom primary agent, the OpenCode 2 session receives session.switchAgent for that agent, and with plan mode on it switches to plan regardless of the pick. Happy to share it if it's useful alongside your native execution rows.

Note that the PR currently shows as conflicting with main, which has moved a lot recently (the OpenCode 2 catalog loader in openCode2Catalog.ts now reads models and providers together).

OpenCode 2 half reproduced, with a focused patch

Environment: T3 Code from current main (5211acfe95), Linux web/server, OpenCode 2.2.0 over http://127.0.0.1:4096, no project-local agents.

Confirmed both v2 gaps in this PR's description:

1. openCode2ModelCapabilities in packages/provider-opencode/src/server/status.ts builds only the variant descriptor. No agent descriptor is emitted from agent.list, so the composer has no Agent control.
2. agentFor in packages/provider-opencode/src/server/v2/adapter.ts is fixed to plan in plan mode and build otherwise, so a selected agent is never applied.

I implemented a narrower version scoped only to OpenCode 2, in case the v2 half is useful independently of the workspace work. Patch and tests: https://gist.github.com/soares-daniel/308618f601c137d05b1d791adaa8aa68

What it does:

• loads client.agent.list({ location: { directory } }) and client.config.get(...), emits visible primary/all agents as the existing agent select descriptor, and marks the server's configured default_agent (falling back to build);
• reads the selected agent model option in agentFor, so the turn's session.switchAgent and the initial session rules use it;
• in the v2 adapter, keeps the selected agent's deny rules and explicit allows after T3's Full access allow-all, converting only ask to allow.

Verification at 080d0f147a:

• New status.test.ts case filters hidden and subagent agents, checks the option order, and checks the configured default.
• New v2/adapter.agent.test.ts covers selected-agent execution in default and plan modes, the no-selection fallback, and Full-access rule order.
• vp run check:changed passes format, lint, both knip runs, 10 package typechecks, and 47 tests across 3 files.
• With the provider reconnected, the Agent menu shows Orchestrator Default and Build; the default comes from default_agent in the server config. I did not run a paid provider turn, so this is picker plus focused-test evidence, not a completed live turn. A screenshot is available if useful.

This does not implement per-workspace agent discovery, so it would not fix the 1.x project-local case in #11871 by itself. I have not opened a competing PR.

Two gaps left at 5477dba

I rebased a v2-only version of this on dac8b11ddc to compare. The descriptor and agentFor here match what I had, so I did not open a separate PR. Two behaviors are still missing at 5477dba.

1. The configured default_agent is ignored

openCodeAgentOptionDescriptor picks Build:

OpenCode's config declares which agent should run by default in default_agent, and config.get reports it. Hardcoding Build means T3 preselects an agent the user's config does not ask for, and the picker disagrees with opencode on the same machine. Nothing else reads the config today.

Reading it next to the agents keeps the descriptor input unchanged apart from the default:

Then thread it through as openCodeAgentOptionDescriptor(agents, configuredDefault) and prefer it:

default_agent is an id, and the call site already normalizes to name: agent.id, so the lookup lines up.

2. The session's first rules are still built for build

agentFor now follows the selection for session.switchAgent, but the initial rules pass in v2/adapter.ts still hardcodes the agent:

A session opened with a selected agent therefore gets its first path rules computed for Build, before the switch corrects the agent. rulesFor takes the agent from its first argument, so using the same selector keeps the two in agreement:

Verification for both

Split patch, tested on dac8b11ddc:

• status.test.ts covers filtering hidden and subagent agents, option order, and the configured default (Build when the config has none).
• v2/adapter.agent.test.ts covers the selected agent in default and plan mode, and the no-selection fallback.
• vp run check:changed passes format, lint, knip, 10 package typechecks, and 46 tests across 3 files.
• Against a live OpenCode 2.2.0 server, the picker lists the configured primary agent first as Default, and Build second. I did not run a paid provider turn, so this is picker plus focused-test evidence.

I am not opening a competing PR for either point. Happy to fold these into this branch if you would rather have them as commits here.