V2 event provider adapters

[juliusmarminge]

Implement v2 canonical event emission for Claude, Codex, and Cursor provider adapters to standardize event structure and enable the new Cursor agent.

---

  

Note

Add Cursor ACP provider adapter and adopt v2 canonical runtime events across server adapters and ingestion

Introduce a new cursor adapter wired into ProviderAdapterRegistryLive and server composition; migrate adapters, tests, and ingestion to v2 canonical runtime events (content.delta, item.*, request.*, payload-based turn states); extend provider runtime contracts with new event types, payloads, and ID types; update stale session recovery to drop resumeThreadId; broaden test harnesses to accept generic event records.

📍Where to Start

Start with the Cursor adapter implementation in makeCursorAdapterLive in CursorAdapter.ts, then review the event mapping overhaul in CodexAdapter.ts and the contract updates in providerRuntime.ts.

📊 Macroscope summarized 14865dc. 10 files reviewed, 47 issues evaluated, 1 issue filtered, 1 comment posted

🗂️ Filtered Issues

apps/server/src/orchestration/Layers/CheckpointReactor.ts — 0 comments posted, 2 evaluated, 1 filtered

• line 65: The newly added turnStateFromCompletion function strictly validates the event status against a hardcoded list ("completed", "failed", "interrupted", "cancelled"). If the provider runtime emits any other failure status (e.g., "error", "timeout", "aborted") or uses a different casing (e.g., "FAILED"), the function returns undefined. This undefined value is then passed to checkpointStatusFromRuntime (line 340), which falls back to the default case returning "ready". Consequently, unhandled failure modes are silently converted into successful ("ready") checkpoints, potentially corrupting the orchestration state with invalid checkpoints. [ Out of scope ]

[cursor]

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch cursor/v2-event-provider-adapters-068b

---

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

[macroscopeapp]

🟡 Medium Layers/CodexAdapter.ts:856

Using firstNonEmptyString here trims the delta text, corrupting indentation and whitespace. Consider using asString instead to preserve the raw content.

-    const delta =
-      firstNonEmptyString(
-        event.textDelta,
-        payload?.delta,
-        payload?.text,
-        payload?.outputDelta,
-        payload?.summaryTextDelta,
-        payload?.textDelta,
-      ) ?? "";
+    const delta =
+      asString(event.textDelta) ??
+      asString(payload?.delta) ??
+      asString(payload?.text) ??
+      asString(payload?.outputDelta) ??
+      asString(payload?.summaryTextDelta) ??
+      asString(payload?.textDelta) ??
+      "";

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file apps/server/src/provider/Layers/CodexAdapter.ts around lines 856-864:

Using `firstNonEmptyString` here trims the delta text, corrupting indentation and whitespace. Consider using `asString` instead to preserve the raw content.

Evidence trail:
CodexAdapter.ts lines 111-120: `firstNonEmptyString` function definition shows it returns `value.trim()`. CodexAdapter.ts lines 95-97: `asString` function returns raw string without trimming. CodexAdapter.ts lines 850-865: Context shows `firstNonEmptyString` is used for delta events including `item/commandExecution/outputDelta` and `item/fileChange/outputDelta` where whitespace preservation is critical.