@parall/codex-agent 1.51.0 → 1.52.1

package/dist/dispatch.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"dispatch.d.ts","sourceRoot":"","sources":["../src/dispatch.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EACV,eAAe,EACf,eAAe,EACf,YAAY,EACZ,QAAQ,EACR,iBAAiB,EACjB,aAAa,EACb,YAAY,EACb,MAAM,oBAAoB,CAAC;AAa5B,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAC;AAEpD,OAAO,EAEL,KAAK,eAAe,EAErB,MAAM,2BAA2B,CAAC;AAGnC,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AAGhE,KAAK,4BAA4B,GAAG,IAAI,CACtC,gBAAgB,EACd,gBAAgB,GAChB,UAAU,GACV,WAAW,GACX,OAAO,GACP,iBAAiB,GACjB,SAAS,GACT,cAAc,CACjB,GAAG;IACF,cAAc,EAAE,mBAAmB,CAAC;IACpC,GAAG,CAAC,EAAE,aAAa,CAAC;IACpB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B;;;;;;OAMG;IACH,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B;;;;;;;;;;;;;;;;;OAiBG;IACH,qBAAqB,CAAC,EAAE,MAAM,CAAC;IAC/B,gFAAgF;IAChF,4BAA4B,CAAC,EAAE,MAAM,CAAC;IACtC,8EAA8E;IAC9E,4BAA4B,CAAC,EAAE,MAAM,CAAC;CACvC,CAAC;AAEF;;;;;;;;;;;;;GAaG;AACH,qBAAa,qBAAsB,YAAW,eAAe;IA8C/C,OAAO,CAAC,QAAQ,CAAC,IAAI;IA7CjC,OAAO,CAAC,MAAM,CAAmC;IACjD,OAAO,CAAC,IAAI,CAA+C;IAC3D,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,YAAY,CAA8B;IAClD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAA+B;IAC3D,OAAO,CAAC,QAAQ,CAAC,aAAa,CAA6B;IAC3D,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAA6B;IAC/D,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAqB;IACtD,0EAA0E;IAC1E,OAAO,CAAC,QAAQ,CAAC,oBAAoB,CAAqB;IAC1D;;;;;;;;OAQG;IACH,OAAO,CAAC,mBAAmB,CAAS;IACpC,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAA8B;IAC/D,OAAO,CAAC,QAAQ,CAAC,qBAAqB,CAAkC;IACxE,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,8BAA8B,CAAK;IAE3C;;;;;;;OAOG;IACH,OAAO,CAAC,aAAa;gBAYQ,IAAI,EAAE,4BAA4B;IAS/D,6EAA6E;IAC7E,kBAAkB,CAAC,GAAG,EAAE,eAAe,GAAG,MAAM,IAAI;IAKpD;;;;;OAKG;IACH,OAAO,CAAC,kBAAkB;IAU1B,YAAY,CAAC,MAAM,EAAE;QACnB,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QACtB,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QAChC,qBAAqB,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;KACvC,GAAG,IAAI;IAQF,qBAAqB,CAAC,UAAU,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAqB/E,aAAa,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI;IAyBvC,oBAAoB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO;IAI1C,QAAQ,CAAC,EACd,KAAK,EACL,YAAY,EACZ,UAAU,EACV,OAAO,GACR,EAAE,YAAY,GAAG,aAAa,CAAC,YAAY,CAAC;IA+S7C;;;;;;OAMG;YACW,kBAAkB;IAoGhC,cAAc,CAAC,WAAW,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS;IAMjD,WAAW,CAAC,EAAE,UAAU,EAAE,gBAAgB,EAAE,EAAE,QAAQ,GAAG,OAAO,CAAC,iBAAiB,GAAG,IAAI,CAAC;IA6ChG,WAAW,CAAC,EAAE,IAAI,EAAE,EAAE,eAAe;IAIrC,OAAO,CAAC,cAAc;IAKtB;;;;;;;;;;;;OAYG;IACH,qBAAqB,IAAI,IAAI;IAI7B,OAAO,CAAC,gBAAgB,CAAS;IAEjC;;;;OAIG;IACH,OAAO,CAAC,iBAAiB,CAAK;YAEhB,mBAAmB;IAS3B,IAAI;YA2BI,aAAa;YA4Bb,OAAO;IA8HrB,OAAO,CAAC,qBAAqB;YAsCf,UAAU;IAqDxB,OAAO,CAAC,iBAAiB;CAsD1B"}
+{"version":3,"file":"dispatch.d.ts","sourceRoot":"","sources":["../src/dispatch.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EACV,eAAe,EACf,eAAe,EACf,YAAY,EACZ,QAAQ,EACR,iBAAiB,EACjB,aAAa,EACb,YAAY,EACb,MAAM,oBAAoB,CAAC;AAa5B,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAC;AAEpD,OAAO,EAEL,KAAK,eAAe,EAErB,MAAM,2BAA2B,CAAC;AAGnC,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AAGhE,KAAK,4BAA4B,GAAG,IAAI,CACtC,gBAAgB,EACd,gBAAgB,GAChB,UAAU,GACV,WAAW,GACX,OAAO,GACP,iBAAiB,GACjB,SAAS,GACT,cAAc,CACjB,GAAG;IACF,cAAc,EAAE,mBAAmB,CAAC;IACpC,GAAG,CAAC,EAAE,aAAa,CAAC;IACpB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B;;;;;;OAMG;IACH,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B;;;;;;;;;;;;;;;;;OAiBG;IACH,qBAAqB,CAAC,EAAE,MAAM,CAAC;IAC/B,gFAAgF;IAChF,4BAA4B,CAAC,EAAE,MAAM,CAAC;IACtC,8EAA8E;IAC9E,4BAA4B,CAAC,EAAE,MAAM,CAAC;CACvC,CAAC;AAEF;;;;;;;;;;;;;GAaG;AACH,qBAAa,qBAAsB,YAAW,eAAe;IA8C/C,OAAO,CAAC,QAAQ,CAAC,IAAI;IA7CjC,OAAO,CAAC,MAAM,CAAmC;IACjD,OAAO,CAAC,IAAI,CAA+C;IAC3D,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,YAAY,CAA8B;IAClD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAA+B;IAC3D,OAAO,CAAC,QAAQ,CAAC,aAAa,CAA6B;IAC3D,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAA6B;IAC/D,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAqB;IACtD,0EAA0E;IAC1E,OAAO,CAAC,QAAQ,CAAC,oBAAoB,CAAqB;IAC1D;;;;;;;;OAQG;IACH,OAAO,CAAC,mBAAmB,CAAS;IACpC,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAA8B;IAC/D,OAAO,CAAC,QAAQ,CAAC,qBAAqB,CAAkC;IACxE,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,8BAA8B,CAAK;IAE3C;;;;;;;OAOG;IACH,OAAO,CAAC,aAAa;gBAYQ,IAAI,EAAE,4BAA4B;IAS/D,6EAA6E;IAC7E,kBAAkB,CAAC,GAAG,EAAE,eAAe,GAAG,MAAM,IAAI;IAKpD;;;;;OAKG;IACH,OAAO,CAAC,kBAAkB;IAU1B,YAAY,CAAC,MAAM,EAAE;QACnB,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QACtB,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QAChC,qBAAqB,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;KACvC,GAAG,IAAI;IAQF,qBAAqB,CAAC,UAAU,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAqB/E,aAAa,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI;IAyBvC,oBAAoB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO;IAI1C,QAAQ,CAAC,EACd,KAAK,EACL,YAAY,EACZ,UAAU,EACV,OAAO,GACR,EAAE,YAAY,GAAG,aAAa,CAAC,YAAY,CAAC;IAoT7C;;;;;;OAMG;YACW,kBAAkB;IAoGhC,cAAc,CAAC,WAAW,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS;IAMjD,WAAW,CAAC,EAAE,UAAU,EAAE,gBAAgB,EAAE,EAAE,QAAQ,GAAG,OAAO,CAAC,iBAAiB,GAAG,IAAI,CAAC;IA6ChG,WAAW,CAAC,EAAE,IAAI,EAAE,EAAE,eAAe;IAIrC,OAAO,CAAC,cAAc;IAKtB;;;;;;;;;;;;OAYG;IACH,qBAAqB,IAAI,IAAI;IAI7B,OAAO,CAAC,gBAAgB,CAAS;IAEjC;;;;OAIG;IACH,OAAO,CAAC,iBAAiB,CAAK;YAEhB,mBAAmB;IAS3B,IAAI;YA2BI,aAAa;YA4Bb,OAAO;IA8HrB,OAAO,CAAC,qBAAqB;YAsCf,UAAU;IAqDxB,OAAO,CAAC,iBAAiB;CAsD1B"}

package/dist/dispatch.js

@@ -436,6 +436,11 @@ export class CodexAppServerAdapter {
                     yield { ...runtimeEvent, project: false, groupKey };
                     continue;
                 }
+                if (runtimeEvent.type === 'turn_outcome') {
+                    // Turn-boundary summary — no groupKey, passed through as-is.
+                    yield runtimeEvent;
+                    continue;
+                }
                 yield { ...runtimeEvent, groupKey };
             }
         }

package/dist/event-mapping.d.ts

@@ -19,6 +19,13 @@ import type { RuntimeEvent } from '@parall/agent-core';
  */
 export declare class EventMapper {
     private readonly toolCallStart;
+    /**
+     * Latest thread/tokenUsage/updated snapshot for this turn. On the pinned
+     * app-server (0.144.x) `turn/completed.turn` carries NO usage field —
+     * token accounting arrives through this separate notification
+     * (`tokenUsage.last`), so the mapper buffers it for the turn boundary.
+     */
+    private lastTokenUsage;
     /**
      * Map a single server notification to zero or more RuntimeEvents.
      * Return `null` for notifications that have no chat surface.

package/dist/event-mapping.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"event-mapping.d.ts","sourceRoot":"","sources":["../src/event-mapping.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAEvD;;;;;;;;;;;;;;;;;GAiBG;AACH,qBAAa,WAAW;IACtB,OAAO,CAAC,QAAQ,CAAC,aAAa,CAA6B;IAE3D;;;OAGG;IACH,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,GAAG,YAAY,EAAE;IA8CpD,OAAO,CAAC,OAAO;IA4Jf;;;;;OAKG;IACH,OAAO,CAAC,eAAe;CAaxB"}
+{"version":3,"file":"event-mapping.d.ts","sourceRoot":"","sources":["../src/event-mapping.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAA+B,MAAM,oBAAoB,CAAC;AAEpF;;;;;;;;;;;;;;;;;GAiBG;AACH,qBAAa,WAAW;IACtB,OAAO,CAAC,QAAQ,CAAC,aAAa,CAA6B;IAC3D;;;;;OAKG;IACH,OAAO,CAAC,cAAc,CAAsC;IAE5D;;;OAGG;IACH,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,GAAG,YAAY,EAAE;IA8DpD,OAAO,CAAC,OAAO;IA4Jf;;;;;OAKG;IACH,OAAO,CAAC,eAAe;CAaxB"}

package/dist/event-mapping.js

@@ -18,6 +18,13 @@
  */
 export class EventMapper {
     toolCallStart = new Map();
+    /**
+     * Latest thread/tokenUsage/updated snapshot for this turn. On the pinned
+     * app-server (0.144.x) `turn/completed.turn` carries NO usage field —
+     * token accounting arrives through this separate notification
+     * (`tokenUsage.last`), so the mapper buffers it for the turn boundary.
+     */
+    lastTokenUsage;
     /**
      * Map a single server notification to zero or more RuntimeEvents.
      * Return `null` for notifications that have no chat surface.
@@ -45,6 +52,15 @@ export class EventMapper {
                 events.push(...this.mapItem(item, 'completed'));
                 break;
             }
+            case 'thread/tokenUsage/updated': {
+                // Buffered for the turn boundary — see the field comment.
+                const tokenUsage = p.tokenUsage;
+                const last = tokenUsage?.last;
+                if (last && typeof last === 'object') {
+                    this.lastTokenUsage = last;
+                }
+                break;
+            }
             case 'turn/completed': {
                 const turn = p.turn;
                 const status = turn ? asString(turn.status) : undefined;
@@ -53,6 +69,12 @@ export class EventMapper {
                     const message = asString(error?.message) ?? 'Codex turn failed';
                     events.push({ type: 'error', message });
                 }
+                // Turn-boundary classification (agent-turn-outcome-design.md §4.2):
+                // structured codexErrorInfo first, message families second, generic
+                // api_error otherwise. Usage rides along on both outcomes. Pushed
+                // after the error event so the gateway folds them in wire order.
+                events.push(buildCodexTurnOutcome(status, error, turn?.usage ?? this.lastTokenUsage));
+                this.lastTokenUsage = undefined;
                 break;
             }
             case 'error': {
@@ -236,6 +258,124 @@ function asString(value) {
     const trimmed = value.trim();
     return trimmed.length > 0 ? trimmed : undefined;
 }
+const CODEX_LIMIT_TEXT = /usage limit|rate limit|quota exceeded|plan limit/i;
+const CODEX_AUTH_TEXT = /unauthorized|not logged in|invalid api key|authentication/i;
+const CODEX_CONTEXT_TEXT = /context (window|length)|prompt is too long|request too large/i;
+/**
+ * Extract the structured discriminator from a failed turn's error. On the
+ * pinned app-server (0.144.x) this is `error.codexErrorInfo` — a Rust enum
+ * serialized either as a bare string ("usageLimitExceeded") or as a
+ * single-key object ({"httpError": {"status": 429}}). Returns the variant
+ * name plus any HTTP status found inside the variant payload. `error.code`
+ * is kept as a secondary read for other/newer protocol shapes.
+ */
+function codexErrorDiscriminator(error) {
+    const info = error?.codexErrorInfo;
+    if (typeof info === 'string')
+        return { variant: info };
+    if (info && typeof info === 'object') {
+        const keys = Object.keys(info);
+        if (keys.length > 0) {
+            const variant = keys[0];
+            const payload = info[variant];
+            let status;
+            if (payload && typeof payload === 'object') {
+                // Pinned protocol carries camelCase `httpStatusCode`; `status` is
+                // kept as a compatibility fallback only (review round 3 P1).
+                const rec = payload;
+                const s = rec.httpStatusCode ?? rec.status;
+                if (typeof s === 'number' && Number.isFinite(s))
+                    status = s;
+            }
+            return { variant, status };
+        }
+    }
+    return {};
+}
+/**
+ * Classify a codex turn/completed frame into a TurnOutcomeEvent. The
+ * structured `codexErrorInfo` variant is the primary discriminator
+ * (usageLimitExceeded / contextWindowExceeded / unauthorized / HTTP-status
+ * variants), `error.code` and message families only refine, so unknown
+ * wording degrades to `api_error` — never to a wrong deferral. codex does
+ * not report a reset time, so usage_limit carries no retryAt and the server
+ * applies its default deferral window.
+ */
+function buildCodexTurnOutcome(status, error, usageRaw) {
+    const usage = extractCodexUsage(usageRaw);
+    if (status !== 'failed') {
+        return { type: 'turn_outcome', outcome: 'ok', ...(usage ? { usage } : {}) };
+    }
+    const message = asString(error?.message) ?? '';
+    const { variant, status: httpStatus } = codexErrorDiscriminator(error);
+    const code = typeof error?.code === 'number' ? error.code : (asString(error?.code) ?? undefined);
+    const codeStr = httpStatus !== undefined ? String(httpStatus) : code === undefined ? '' : String(code);
+    const variantLower = (variant ?? '').toLowerCase();
+    // Structured discriminators (variant + HTTP status) rule first; message text
+    // only refines when neither can classify. Mixing text into the same OR as
+    // the variant let a `usageLimitExceeded` frame whose message happened to
+    // quote "authentication"/"rate limit" misclassify as auth and lose the
+    // deferred redrive — the exact read-no-reply regression this PR prevents.
+    let outcome = 'api_error';
+    if (variantLower.includes('unauthorized') || codeStr === '401' || codeStr === '403') {
+        outcome = 'auth';
+    }
+    else if (variantLower.includes('contextwindow') || codeStr === '413') {
+        outcome = 'context_overflow';
+    }
+    else if (variantLower.includes('usagelimit') ||
+        variantLower.includes('ratelimit') ||
+        codeStr === '429') {
+        outcome = 'usage_limit';
+    }
+    else if (CODEX_AUTH_TEXT.test(message)) {
+        outcome = 'auth';
+    }
+    else if (CODEX_CONTEXT_TEXT.test(message)) {
+        outcome = 'context_overflow';
+    }
+    else if (CODEX_LIMIT_TEXT.test(message)) {
+        outcome = 'usage_limit';
+    }
+    const raw = {};
+    if (variant)
+        raw.codex_error_info = variant;
+    if (httpStatus !== undefined)
+        raw.status = httpStatus;
+    if (code !== undefined)
+        raw.code = code;
+    return {
+        type: 'turn_outcome',
+        outcome,
+        ...(message ? { detail: message.slice(0, 500) } : {}),
+        ...(usage ? { usage } : {}),
+        ...(Object.keys(raw).length > 0 ? { raw } : {}),
+    };
+}
+/** codex app-server usage fields are camelCase; tolerate snake_case too. */
+function extractCodexUsage(raw) {
+    if (!raw || typeof raw !== 'object')
+        return undefined;
+    const r = raw;
+    const num = (a, b) => {
+        if (typeof a === 'number' && Number.isFinite(a))
+            return a;
+        if (typeof b === 'number' && Number.isFinite(b))
+            return b;
+        return undefined;
+    };
+    const usage = {};
+    const input = num(r.inputTokens, r.input_tokens);
+    if (input !== undefined)
+        usage.inputTokens = input;
+    const output = num(r.outputTokens, r.output_tokens);
+    if (output !== undefined)
+        usage.outputTokens = output;
+    const cached = num(r.cachedInputTokens, r.cached_input_tokens);
+    if (cached !== undefined)
+        usage.cacheReadTokens = cached;
+    return Object.keys(usage).length > 0 ? usage : undefined;
+}
 function joinReasoningText(item) {
     const summary = Array.isArray(item.summary)
         ? item.summary.filter((x) => typeof x === 'string')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@parall/codex-agent",
-  "version": "1.51.0",
+  "version": "1.52.1",
   "description": "Codex CLI bridge runtime for self-hosted Parall agents",
   "license": "MIT",
   "repository": {
@@ -25,9 +25,9 @@
     "src"
   ],
   "dependencies": {
-    "@parall/agent-core": "1.51.0",
-    "@parall/cli": "1.51.0",
-    "@parall/sdk": "1.51.0"
+    "@parall/agent-core": "1.52.1",
+    "@parall/cli": "1.52.1",
+    "@parall/sdk": "1.52.1"
   },
   "devDependencies": {
     "@types/node": "^22.0.0",

package/src/dispatch.ts

@@ -531,6 +531,11 @@ export class CodexAppServerAdapter implements DispatchAdapter {
           yield { ...runtimeEvent, project: false, groupKey };
           continue;
         }
+        if (runtimeEvent.type === 'turn_outcome') {
+          // Turn-boundary summary — no groupKey, passed through as-is.
+          yield runtimeEvent;
+          continue;
+        }
         yield { ...runtimeEvent, groupKey };
       }
     } finally {

package/src/event-mapping.ts

@@ -1,4 +1,4 @@
-import type { RuntimeEvent } from '@parall/agent-core';
+import type { RuntimeEvent, TurnOutcomeEvent, TurnUsage } from '@parall/agent-core';
 
 /**
  * Translation layer between `codex app-server` JSON-RPC notifications and
@@ -20,6 +20,13 @@ import type { RuntimeEvent } from '@parall/agent-core';
  */
 export class EventMapper {
   private readonly toolCallStart = new Map<string, number>();
+  /**
+   * Latest thread/tokenUsage/updated snapshot for this turn. On the pinned
+   * app-server (0.144.x) `turn/completed.turn` carries NO usage field —
+   * token accounting arrives through this separate notification
+   * (`tokenUsage.last`), so the mapper buffers it for the turn boundary.
+   */
+  private lastTokenUsage: Record<string, unknown> | undefined;
 
   /**
    * Map a single server notification to zero or more RuntimeEvents.
@@ -50,6 +57,16 @@ export class EventMapper {
         break;
       }
 
+      case 'thread/tokenUsage/updated': {
+        // Buffered for the turn boundary — see the field comment.
+        const tokenUsage = p.tokenUsage as Record<string, unknown> | undefined;
+        const last = tokenUsage?.last;
+        if (last && typeof last === 'object') {
+          this.lastTokenUsage = last as Record<string, unknown>;
+        }
+        break;
+      }
+
       case 'turn/completed': {
         const turn = p.turn as Record<string, unknown> | undefined;
         const status = turn ? asString(turn.status) : undefined;
@@ -58,6 +75,12 @@ export class EventMapper {
           const message = asString(error?.message) ?? 'Codex turn failed';
           events.push({ type: 'error', message });
         }
+        // Turn-boundary classification (agent-turn-outcome-design.md §4.2):
+        // structured codexErrorInfo first, message families second, generic
+        // api_error otherwise. Usage rides along on both outcomes. Pushed
+        // after the error event so the gateway folds them in wire order.
+        events.push(buildCodexTurnOutcome(status, error, turn?.usage ?? this.lastTokenUsage));
+        this.lastTokenUsage = undefined;
         break;
       }
 
@@ -254,6 +277,122 @@ function asString(value: unknown): string | undefined {
   return trimmed.length > 0 ? trimmed : undefined;
 }
 
+const CODEX_LIMIT_TEXT = /usage limit|rate limit|quota exceeded|plan limit/i;
+const CODEX_AUTH_TEXT = /unauthorized|not logged in|invalid api key|authentication/i;
+const CODEX_CONTEXT_TEXT = /context (window|length)|prompt is too long|request too large/i;
+
+/**
+ * Extract the structured discriminator from a failed turn's error. On the
+ * pinned app-server (0.144.x) this is `error.codexErrorInfo` — a Rust enum
+ * serialized either as a bare string ("usageLimitExceeded") or as a
+ * single-key object ({"httpError": {"status": 429}}). Returns the variant
+ * name plus any HTTP status found inside the variant payload. `error.code`
+ * is kept as a secondary read for other/newer protocol shapes.
+ */
+function codexErrorDiscriminator(error: Record<string, unknown> | undefined): {
+  variant?: string;
+  status?: number;
+} {
+  const info = error?.codexErrorInfo;
+  if (typeof info === 'string') return { variant: info };
+  if (info && typeof info === 'object') {
+    const keys = Object.keys(info);
+    if (keys.length > 0) {
+      const variant = keys[0];
+      const payload = (info as Record<string, unknown>)[variant];
+      let status: number | undefined;
+      if (payload && typeof payload === 'object') {
+        // Pinned protocol carries camelCase `httpStatusCode`; `status` is
+        // kept as a compatibility fallback only (review round 3 P1).
+        const rec = payload as Record<string, unknown>;
+        const s = rec.httpStatusCode ?? rec.status;
+        if (typeof s === 'number' && Number.isFinite(s)) status = s;
+      }
+      return { variant, status };
+    }
+  }
+  return {};
+}
+
+/**
+ * Classify a codex turn/completed frame into a TurnOutcomeEvent. The
+ * structured `codexErrorInfo` variant is the primary discriminator
+ * (usageLimitExceeded / contextWindowExceeded / unauthorized / HTTP-status
+ * variants), `error.code` and message families only refine, so unknown
+ * wording degrades to `api_error` — never to a wrong deferral. codex does
+ * not report a reset time, so usage_limit carries no retryAt and the server
+ * applies its default deferral window.
+ */
+function buildCodexTurnOutcome(
+  status: string | undefined,
+  error: Record<string, unknown> | undefined,
+  usageRaw: unknown,
+): TurnOutcomeEvent {
+  const usage = extractCodexUsage(usageRaw);
+  if (status !== 'failed') {
+    return { type: 'turn_outcome', outcome: 'ok', ...(usage ? { usage } : {}) };
+  }
+  const message = asString(error?.message) ?? '';
+  const { variant, status: httpStatus } = codexErrorDiscriminator(error);
+  const code = typeof error?.code === 'number' ? error.code : (asString(error?.code) ?? undefined);
+  const codeStr =
+    httpStatus !== undefined ? String(httpStatus) : code === undefined ? '' : String(code);
+  const variantLower = (variant ?? '').toLowerCase();
+  // Structured discriminators (variant + HTTP status) rule first; message text
+  // only refines when neither can classify. Mixing text into the same OR as
+  // the variant let a `usageLimitExceeded` frame whose message happened to
+  // quote "authentication"/"rate limit" misclassify as auth and lose the
+  // deferred redrive — the exact read-no-reply regression this PR prevents.
+  let outcome: TurnOutcomeEvent['outcome'] = 'api_error';
+  if (variantLower.includes('unauthorized') || codeStr === '401' || codeStr === '403') {
+    outcome = 'auth';
+  } else if (variantLower.includes('contextwindow') || codeStr === '413') {
+    outcome = 'context_overflow';
+  } else if (
+    variantLower.includes('usagelimit') ||
+    variantLower.includes('ratelimit') ||
+    codeStr === '429'
+  ) {
+    outcome = 'usage_limit';
+  } else if (CODEX_AUTH_TEXT.test(message)) {
+    outcome = 'auth';
+  } else if (CODEX_CONTEXT_TEXT.test(message)) {
+    outcome = 'context_overflow';
+  } else if (CODEX_LIMIT_TEXT.test(message)) {
+    outcome = 'usage_limit';
+  }
+  const raw: Record<string, string | number | boolean | null> = {};
+  if (variant) raw.codex_error_info = variant;
+  if (httpStatus !== undefined) raw.status = httpStatus;
+  if (code !== undefined) raw.code = code;
+  return {
+    type: 'turn_outcome',
+    outcome,
+    ...(message ? { detail: message.slice(0, 500) } : {}),
+    ...(usage ? { usage } : {}),
+    ...(Object.keys(raw).length > 0 ? { raw } : {}),
+  };
+}
+
+/** codex app-server usage fields are camelCase; tolerate snake_case too. */
+function extractCodexUsage(raw: unknown): TurnUsage | undefined {
+  if (!raw || typeof raw !== 'object') return undefined;
+  const r = raw as Record<string, unknown>;
+  const num = (a: unknown, b: unknown): number | undefined => {
+    if (typeof a === 'number' && Number.isFinite(a)) return a;
+    if (typeof b === 'number' && Number.isFinite(b)) return b;
+    return undefined;
+  };
+  const usage: TurnUsage = {};
+  const input = num(r.inputTokens, r.input_tokens);
+  if (input !== undefined) usage.inputTokens = input;
+  const output = num(r.outputTokens, r.output_tokens);
+  if (output !== undefined) usage.outputTokens = output;
+  const cached = num(r.cachedInputTokens, r.cached_input_tokens);
+  if (cached !== undefined) usage.cacheReadTokens = cached;
+  return Object.keys(usage).length > 0 ? usage : undefined;
+}
+
 function joinReasoningText(item: Record<string, unknown>): string {
   const summary = Array.isArray(item.summary)
     ? item.summary.filter((x) => typeof x === 'string')