[ranxianglei]

Fixes #918

What

The same bili plugin install opencode wrapper now serves OpenCode 1.x through the V1 .server() hook surface, delivering near-v2 effectiveness on the standard install path:

• config hook: rewrites every provider options.baseURL to <proxy>/bili/<upstream> by mutating the shared in-process config object (idempotent; unwraps stale wraps from other proxy origins) + compaction.auto: false
• chat.headers: stamps the plugin headers per request → plugin mode (no wire tool injection)
• tool: registers the full bili tool set natively with real zod shapes (JSON-Schema→zod conversion; host registry wraps with its own z.object — zod pinned to 4.1.8 matching the host)
• /acp command: same status panel as v2, via the shared opencode-acp-command.ts factory
• reuses the self-spawn/attach proxy bootstrap (bili: attaching to running proxy … observed across processes); degrades to plain proxy mode when zod is unresolvable (rewrite only, wire-injected tools)

Refactor

• src/agent/opencode-acp-command.ts (new): /acp hooks extracted from the launcher plugin — V1 .server and V2 setup now share one implementation
• src/agent/opencode.ts: launcher plugin consumes the shared factory
• zod becomes the package's only runtime dependency (external; only dist/agent/opencode-native.js imports it lazily — dist/index.js verified zod-free)

Verification

• npm run typecheck / npm run build clean; full npm test 1692 pass / 0 fail (11 new tests: schema→zod mapping incl. cross-instance z.object interop, provider-rewrite idempotency/unwrapping, hooks wiring, zod-unavailable degrade)
• Live E2E in container (real sglang upstream) via bili plugin install opencode + plain opencode on 1.14.46 and 1.18.31:
  • [bili-opencode-native] v1 active (proxy http://127.0.0.1:PORT) + rewrote 2 provider baseURL(s)
  • plugin mode on the wire: [plugin] [acp-usage] input=… cached=… (cache hit 98%)
  • native tool round-trip: [plugin] tool acp_status executed via plugin (378 chars)
  • /acp in the TUI command palette + full panel rendered into the session (idle + live variants)
  • tag-echo guard stripping model-emitted render tags (plugin passthrough)

Known V1 limitation (documented in README): providers without an explicit baseURL (SDK defaults) keep going direct — V1 has no request-level URL seam to catch them.

[ranxianglei]

🤖 Powered by ework · qwen3.8-27b

[bot] 🏷 收到。这是 [PR] 类唤醒,不重做实现——正在核对:拉取 2026-09-18_opencode-v1-native 分支、核对 diff 与 CI 状态、本地跑 typecheck/test/build 验证。结论稍后回帖。

[ranxianglei]

🤖 Powered by ework · qwen3.8-27b

Verification complete — all green. Details:

CI status (head 086e80a)

• ✅ e2e-fake-codex (real codex vs deterministic fake upstream, zero tokens) — passed (run #35248016415)
• ℹ️ ci.yml (typecheck + test + build, ubuntu/windows × node 22/24 + version-guard) did not run: it only triggers on PRs targeting master, and this PR targets feat/pi-native-plugin. Filled the gap locally:

Local verification (branch checked out, node 22)

• npm ci — clean; zod resolves to exactly 4.1.8
• npm run typecheck — clean
• npm test — 1694 tests: 1692 pass / 0 fail / 2 skipped, matches your claim; the three new suites (jsonSchemaToZodShape / rewriteV1Providers / createV1ServerHooks) all green
• npm run build — clean
• zod-free claim verified: dist/index.js contains zero references to zod; dist/agent/opencode-native.js has exactly one lazy import("zod"), kept external (tsup noExternal list unchanged — acp-kernel & co still inlined)

Diff audit (every referenced symbol cross-checked against feat/pi-native-plugin)

• wrapUpstream/unwrapUpstream (src/launcher.ts:205/:213), ACP_TOOLS_OPENAI (src/compress-tool.ts:42), forwardTool (src/agent/shared.ts:117) — all exist, signatures match
• V1 header stamps use the same names as V2 (x-bili-plugin, x-bili-plugin-conversation) → same plugin-mode session binding path
• version field untouched ✓ (non-release branch rule)
• Scope check: diff touches agent-side plugins + docs + package.json only — no server.ts / loop/ / adapter / preflight changes, so the mandatory codex E2E re-run for pipeline changes doesn't apply (and the e2e-fake job covered the codex path anyway)

Minor observation (non-blocking, nothing pushed): in jsonSchemaToZodShape, the array-case guard item === undefined is dead code — the recursive call never returns undefined. Purely cosmetic; left alone per the [PR] verify-only rule. Say the word and I'll squash it into a commit.

Merge planning note: when this lands toward master (directly, or once feat/pi-native-plugin merges), ci.yml's full matrix will run again on the master-targeting PR.

一句话:PR 代码核对与本地 typecheck / test(1692 通过)/ build 全部干净,dist/index.js 确认无 zod、原生插件懒加载外置,无阻塞问题,可以合并。