test(cli): isolate PTY route tests from indexing

[marius-kilocode]

The Darwin CLI profile runs these PTY route tests with full instance bootstrap. Each temporary project inherits Kilo's default indexing plugin, which fork-detaches indexing startup and can race PTY setup or consume native test resources. In the failing run, indexing initialization appeared immediately before the plugin-backed PTY test stalled.\n\nDisable codebase indexing for this test file, matching the existing instance-route auth test isolation. The suite still exercises external plugin loading, PTY environment precedence, credential stripping, websocket output, and cleanup, but no longer starts an unrelated indexing worker for each temporary project. The test retains its original 30-second deadline.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

lgtm — incremental review of the follow-up commit. The PTY test file now sets KILO_DISABLE_CODEBASE_INDEXING = "vscode-no-workspace" at module scope (properly wrapped in kilocode_change markers), which src/kilocode/indexing.ts:278 reads lazily at per-instance boot, so the assignment is effective before any temp project spins up. This matches the established isolation pattern in test/server/httpapi-exercise/environment.ts. The test timeout reverts to its original 30_000 positional argument (valid for effectIt.live), consistent with the updated PR description, and the strict inner 5-second output wait in takeUntil still fails fast on genuinely missing PTY output.

Files Reviewed (1 file)

• packages/opencode/test/server/httpapi-v2-pty.test.ts

Previous Review Summary (commit 48400d8)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 48400d8)

Status: No Issues Found | Recommendation: Merge

lgtm — single-line test-only timeout bump (30_000 → 60_000) for the plugin-backed PTY integration test. Verified the positional timeout argument is valid for effectIt.live (opts?: number | TestOptions passed through to bun:test), the kilocode_change marker is retained, and the updated comment accurately reflects the rationale (Darwin CI contention). The strict inner 5-second output wait is preserved, so the test still fails fast on genuinely missing PTY output.

Files Reviewed (1 file)

• packages/opencode/test/server/httpapi-v2-pty.test.ts

---

_{Reviewed by kimi-k3 · Input: 39K · Output: 4.3K · Cached: 261.1K}

_{Review guidance: REVIEW.md from base branch main}