fix(server): restore OpenCode adapter test typecheck

[Yash-Singh1]

Problem

The latest main CI run fails Typecheck because watchCalls in the OpenCode 2 adapter tests refers to an unimported ProviderAdapterV2SessionRuntime. That unresolved type also causes downstream Effect context and event type errors. Lint already passes on that commit.

Change

Qualify the runtime type through the existing ProviderAdapter namespace import, matching the other helpers in the file.

Scope and approval

This is a focused fix for an obvious compiler error in a test helper. It changes only a type reference and qualifies for the small bug fix exception without a prior issue or discussion.

Verification

Validated in a temporary worktree with a frozen-lockfile install of current dependencies:

• vp run --filter t3 typecheck passes.
• vp test run apps/server/src/orchestration-v2/Adapters/OpenCode2AdapterV2.test.ts passes all 89 tests.
• vp lint --report-unused-disable-directives apps/server/src/orchestration-v2/Adapters/OpenCode2AdapterV2.test.ts passes.
• vp fmt --check apps/server/src/orchestration-v2/Adapters/OpenCode2AdapterV2.test.ts passes.

Model: gpt-6.1-sol. Harness: Codex in T3 Code.

[macroscopeapp]

Approvability

Verdict: Approved at d83f5ca

Macroscope's review found this PR approvable — This is a one-file test/typecheck correction confined to an ignored path, with no production runtime, product-default, schema, or static-analysis suppression impact.

Notes:

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

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

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Path: .coderabbit.config.ts
• Review profile: CHILL
• Plan: Team
• Run ID: e7bf19b5-1339-4671-ba1d-2a6cc2985223

---

📥 Commits

Reviewing files that changed from the base of the PR and between f7664fd and d83f5ca.

---

📒 Files selected for processing (1)

• apps/server/src/orchestration-v2/Adapters/OpenCode2AdapterV2.test.ts

---

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

---

📝 Walkthrough

Walkthrough

The watchCalls test helper now refers to the session runtime type through the ProviderAdapter namespace. The change does not alter runtime behavior.

Changes

Test type reference

Layer / File(s)	Summary
Update watchCalls type reference apps/server/src/orchestration-v2/Adapters/OpenCode2AdapterV2.test.ts	watchCalls now uses ProviderAdapter.ProviderAdapterV2SessionRuntime.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

---

Merge Risk: ⚪ Minimal · up to d83f5

The test type reference resolves to the exported interface, and runtime behavior is unchanged; no actionable merge risk remains.

Pre-merge checks | 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check		The title clearly identifies the server-side fix and the restored OpenCode adapter test typecheck. It is concise and uses the repository's conventional commit format.
Description check		The description includes all required sections: Problem, Change, Scope and approval, and Verification. It explains the compiler error, the focused fix, the approval exception, and the specific validat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

---

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

---

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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

---

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	5.0 KiB	5.0 KiB	0 B (0.0%)	6.8 KiB	✅
Codex	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Codex	Live turn WebSocket wire	1.2 KiB	1.2 KiB	0 B (0.0%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	20.9 KiB	20.9 KiB	0 B (0.0%)	29.3 KiB	✅
Codex	Live turn messages	2	2	0 (0.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	0 B (0.0%)	6.8 KiB	✅
Claude	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Claude	Live turn WebSocket wire	1.2 KiB	1.2 KiB	0 B (0.0%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.2 KiB	21.2 KiB	0 B (0.0%)	29.3 KiB	✅
Claude	Live turn messages	2	2	0 (0.0%)	8	✅

Baseline: ee7924f · PR result: d83f5ca · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 108.5 KiB
• Claude decoded thread snapshot: 108.8 KiB

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