refactor(server): let adapters declare context compaction

[t3dotgg]

ProviderService picked the context compaction timeout by checking provider === "codex" || provider === "opencode" and picked /compress versus /compact by checking provider === "cursor". Adding or changing a provider meant editing those switches.

Each adapter now declares a compaction strategy: either { type: "native", start } (Codex, OpenCode) or { type: "slash-command", command } (Claude, Cursor, Grok, Antigravity). ProviderService reads the strategy instead of switching on the provider name. The timeout is one shared 10 minute constant, which matches what every provider used on main. Runtime behavior is identical to main for every provider.

Removed from the original version: the native settling rule change, the longer overlap guard lifetime, the per-instance timeout quarantine, and the drop of Antigravity compaction. Those were behavior changes bundled into a refactor and are out of scope here. Antigravity keeps its /compact slash path.

Reworked by Claude Fable 5.1 in Claude Code.

Note

Refactor adapter compaction into a declared strategy type

• Replaces the optional direct compaction method on ProviderAdapterShape with a ProviderCompaction discriminated union: a native variant carrying a start callback, or a slash-command variant carrying a command string
• Each adapter (Antigravity, Claude, Codex, Cursor, Grok, OpenCode) now declares its compaction strategy through the new compaction property instead of a top-level method
• ProviderService.compactThread routes based on the declared strategy: invokes the native start callback directly for native strategies, or sends the configured slash command as a turn for slash-command strategies; adapters without a strategy are rejected
• Both paths share a single COMPACTION_COMPLETION_TIMEOUT of ten minutes, and timeout/error handling is now based on strategy type rather than provider-name checks
• Risk: adapters that previously relied on the removed top-level compaction method (e.g. compactThread on CodexAdapter / OpenCodeAdapter) must migrate to the new compaction property; out-of-tree adapters implementing ProviderAdapterShape will fail compaction until they declare a strategy

^{Macroscope summarized 2e4ffe0.}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR changes the shared provider compaction contract and central event/timeout orchestration across multiple adapters, while also changing user-visible command advertisement. The extensive tests help cover the new cases, but the cross-provider runtime impact warrants focused review.

No code changes detected at 58c0c98. Prior analysis still applies.

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.6 KiB	13.5 KiB	−44 B (−0.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−5 B (−0.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.5 KiB	6.5 KiB	−39 B (−0.6%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.1 KiB	57.0 KiB	−88 B (−0.2%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.5 KiB	−64 B (−0.5%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−6 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.6 KiB	6.5 KiB	−58 B (−0.9%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.9 KiB	57.8 KiB	−88 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	10	8	−2 (−20.0%)	21	✅

Baseline: e4e9fa9 · PR result: 2e4ffe0 · 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: 113.8 KiB
• Claude decoded thread snapshot: 114.5 KiB

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

[t3dotgg]

Note

🤖 GPT-6 Astra responding on behalf of Theo

Human approval is appropriate for this change.

Codex and OpenCode keep their native compaction APIs. Cursor keeps /compress. Claude and Grok keep /compact. The service now uses each adapter's declared strategy and rejects unsupported requests without sending a prompt.

Focused source review covered early completion, overlap, failed-event ordering, and provider-instance timeout recovery. All 306 focused tests and required CI pass on 43b25a85. Macroscope correctness and Effect convention checks pass. No client code or wire contract changed.

[cursor]

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.