@parall/codex-agent 1.41.0 → 1.42.0

package/dist/config.d.ts

@@ -42,10 +42,18 @@ export declare function resolveCodexAgentConfig(env?: NodeJS.ProcessEnv): CodexA
  */
 export declare function normalizeSandbox(value: string): string;
 /**
- * Normalise an approval policy string to the kebab-case form the app-server
- * JSON-RPC enum expects (`never` / `on-request` / `on-failure` /
- * `unless-trusted`). Mirrors the sandbox enum — serde rename_all kebab-case
- * applies across app-server protocol enums.
+ * Normalise an approval policy string to the enum form the app-server
+ * JSON-RPC surface accepts. Empirically verified against both pinned CLI
+ * generations: 0.144.x accepts `untrusted` / `on-request` / `granular` /
+ * `never`; 0.133.0 additionally accepted `on-failure`. Two legacy inputs are
+ * translated rather than passed through, because the server rejects the whole
+ * thread/start with "unknown variant" otherwise:
+ *
+ * - `on-failure` → `on-request` (removed from the enum in 0.144; the closest
+ *   surviving escalation policy).
+ * - `unless-trusted` → `untrusted` (the wire constant was never
+ *   `unless-trusted` on either generation — accepting-but-forwarding it made
+ *   every thread/start fail for configs that set it).
  */
 export declare function normalizeApprovalPolicy(value: string): string;
 export declare function resolveWsUrl(apiUrl: string, explicitWsUrl?: string, swimlaneName?: string): string;

package/dist/config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":"AAIA,MAAM,MAAM,gBAAgB,GAAG;IAC7B,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB;;;;;;;OAOG;IACH,OAAO,EAAE,MAAM,CAAC;IAChB;;;;;OAKG;IACH,cAAc,EAAE,MAAM,CAAC;IACvB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB,CAAC;AAcF,wBAAgB,uBAAuB,CAAC,GAAG,GAAE,MAAM,CAAC,UAAwB,GAAG,gBAAgB,CAyC9F;AAED;;;;;;;;;;GAUG;AACH,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAUtD;AAED;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAW7D;AAED,wBAAgB,YAAY,CAC1B,MAAM,EAAE,MAAM,EACd,aAAa,CAAC,EAAE,MAAM,EACtB,YAAY,CAAC,EAAE,MAAM,GACpB,MAAM,CAMR;AAED,wBAAgB,oBAAoB,CAAC,WAAW,EAAE,MAAM,GAAG,MAAM,CAEhE;AAED,wBAAgB,8BAA8B,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,CAG3F;AAED,wBAAgB,yBAAyB,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,CAGtF;AAED;;;;;;GAMG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,CAE/D;AAED,wBAAgB,wBAAwB,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,CAGrF"}
+{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":"AAIA,MAAM,MAAM,gBAAgB,GAAG;IAC7B,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB;;;;;;;OAOG;IACH,OAAO,EAAE,MAAM,CAAC;IAChB;;;;;OAKG;IACH,cAAc,EAAE,MAAM,CAAC;IACvB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB,CAAC;AAcF,wBAAgB,uBAAuB,CAAC,GAAG,GAAE,MAAM,CAAC,UAAwB,GAAG,gBAAgB,CAyC9F;AAED;;;;;;;;;;GAUG;AACH,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAUtD;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,uBAAuB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAa7D;AAED,wBAAgB,YAAY,CAC1B,MAAM,EAAE,MAAM,EACd,aAAa,CAAC,EAAE,MAAM,EACtB,YAAY,CAAC,EAAE,MAAM,GACpB,MAAM,CAMR;AAED,wBAAgB,oBAAoB,CAAC,WAAW,EAAE,MAAM,GAAG,MAAM,CAEhE;AAED,wBAAgB,8BAA8B,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,CAG3F;AAED,wBAAgB,yBAAyB,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,CAGtF;AAED;;;;;;GAMG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,CAE/D;AAED,wBAAgB,wBAAwB,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,CAGrF"}

package/dist/config.js

@@ -69,20 +69,30 @@ export function normalizeSandbox(value) {
     return map[value.toLowerCase()] ?? value;
 }
 /**
- * Normalise an approval policy string to the kebab-case form the app-server
- * JSON-RPC enum expects (`never` / `on-request` / `on-failure` /
- * `unless-trusted`). Mirrors the sandbox enum — serde rename_all kebab-case
- * applies across app-server protocol enums.
+ * Normalise an approval policy string to the enum form the app-server
+ * JSON-RPC surface accepts. Empirically verified against both pinned CLI
+ * generations: 0.144.x accepts `untrusted` / `on-request` / `granular` /
+ * `never`; 0.133.0 additionally accepted `on-failure`. Two legacy inputs are
+ * translated rather than passed through, because the server rejects the whole
+ * thread/start with "unknown variant" otherwise:
+ *
+ * - `on-failure` → `on-request` (removed from the enum in 0.144; the closest
+ *   surviving escalation policy).
+ * - `unless-trusted` → `untrusted` (the wire constant was never
+ *   `unless-trusted` on either generation — accepting-but-forwarding it made
+ *   every thread/start fail for configs that set it).
  */
 export function normalizeApprovalPolicy(value) {
     const map = {
         never: 'never',
         'on-request': 'on-request',
         onrequest: 'on-request',
-        'unless-trusted': 'unless-trusted',
-        unlesstrusted: 'unless-trusted',
-        'on-failure': 'on-failure',
-        onfailure: 'on-failure',
+        untrusted: 'untrusted',
+        'unless-trusted': 'untrusted',
+        unlesstrusted: 'untrusted',
+        granular: 'granular',
+        'on-failure': 'on-request',
+        onfailure: 'on-request',
     };
     return map[value.toLowerCase()] ?? value;
 }

package/dist/dispatch.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"dispatch.d.ts","sourceRoot":"","sources":["../src/dispatch.ts"],"names":[],"mappings":"AASA,OAAO,KAAK,EACV,eAAe,EACf,eAAe,EACf,YAAY,EACZ,QAAQ,EACR,iBAAiB,EACjB,aAAa,EACb,YAAY,EACb,MAAM,oBAAoB,CAAC;AAC5B,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAC;AAIpD,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AAEhE,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;CAC7B,CAAC;AAqCF,qBAAa,qBAAsB,YAAW,eAAe;IA+B/C,OAAO,CAAC,QAAQ,CAAC,IAAI;IA9BjC,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,OAAO,CAAC,QAAQ,CAAS;IAEzB;;;;;;;OAOG;IACH,OAAO,CAAC,aAAa;gBAYQ,IAAI,EAAE,4BAA4B;IAE/D,YAAY,CAAC,MAAM,EAAE;QAAE,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QAAC,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,GAAG,IAAI;IAMhF,qBAAqB,CAAC,UAAU,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAqB/E,aAAa,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI;IAUvC,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;IA0O7C,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;IA+BhG,WAAW,CAAC,EAAE,IAAI,EAAE,EAAE,eAAe;IAIrC,OAAO,CAAC,cAAc;IAKhB,IAAI;YAiBI,aAAa;YA2Bb,OAAO;IAuFrB,OAAO,CAAC,qBAAqB;YAmCf,UAAU;IA2CxB,OAAO,CAAC,iBAAiB;CA2B1B"}
+{"version":3,"file":"dispatch.d.ts","sourceRoot":"","sources":["../src/dispatch.ts"],"names":[],"mappings":"AASA,OAAO,KAAK,EACV,eAAe,EACf,eAAe,EACf,YAAY,EACZ,QAAQ,EACR,iBAAiB,EACjB,aAAa,EACb,YAAY,EACb,MAAM,oBAAoB,CAAC;AAC5B,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAC;AAIpD,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AAEhE,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;CAC7B,CAAC;AAqCF,qBAAa,qBAAsB,YAAW,eAAe;IA+B/C,OAAO,CAAC,QAAQ,CAAC,IAAI;IA9BjC,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,OAAO,CAAC,QAAQ,CAAS;IAEzB;;;;;;;OAOG;IACH,OAAO,CAAC,aAAa;gBAYQ,IAAI,EAAE,4BAA4B;IAE/D,YAAY,CAAC,MAAM,EAAE;QAAE,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QAAC,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,GAAG,IAAI;IAMhF,qBAAqB,CAAC,UAAU,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAqB/E,aAAa,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI;IAUvC,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;IA6P7C,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;IAgChG,WAAW,CAAC,EAAE,IAAI,EAAE,EAAE,eAAe;IAIrC,OAAO,CAAC,cAAc;IAKhB,IAAI;YAiBI,aAAa;YA2Bb,OAAO;IAuFrB,OAAO,CAAC,qBAAqB;YAmCf,UAAU;IA6CxB,OAAO,CAAC,iBAAiB;CA2B1B"}

package/dist/dispatch.js

@@ -215,10 +215,32 @@ export class CodexAppServerAdapter {
             // this contract, we want a hard error surfaced to the operator, not a
             // silent fallback that hides real protocol drift.
             const turnInput = buildTurnInput(preparedBody, preparedImages);
-            const startTurn = (targetThreadId) => client.sendRequest('turn/start', {
-                threadId: targetThreadId,
-                input: turnInput,
-            });
+            // Per-turn overrides, first-class on both supported CLI generations
+            // (0.133 / 0.144) with "for this turn and subsequent turns" semantics.
+            // Sending the CURRENT config on every turn is what makes
+            // agent_config.update (model / thinking-effort hot updates) take
+            // effect on the next turn: thread-level config only applies at
+            // thread/start-resume, which an already-open thread never re-runs,
+            // and a prior turn's override would otherwise stay live for the rest
+            // of the subprocess lifetime.
+            //
+            // The fields are only sent when a value is configured: on the wire
+            // they are plain serde Options, so JSON null is indistinguishable
+            // from omission — there is no "reset to default" encoding. CLEARING
+            // a previously-set pin therefore keeps the old sticky value until
+            // the subprocess restarts (pod restart / redeploy / idle reap);
+            // an explicit restart-on-clear path is a tracked follow-up.
+            const startTurn = (targetThreadId) => {
+                const turnParams = {
+                    threadId: targetThreadId,
+                    input: turnInput,
+                };
+                if (this.opts.model)
+                    turnParams.model = this.opts.model;
+                if (this.opts.reasoningEffort)
+                    turnParams.effort = this.opts.reasoningEffort;
+                return client.sendRequest('turn/start', turnParams);
+            };
             // Start the turn. If the stored threadId is dead, we retry once with a fresh thread.
             let turnStartResult;
             try {
@@ -360,7 +382,8 @@ export class CodexAppServerAdapter {
             if (this.opts.model)
                 forkParams.model = this.opts.model;
             if (this.opts.reasoningEffort) {
-                forkParams.config = { modelReasoningEffort: this.opts.reasoningEffort };
+                // Raw config.toml key — see openThread for the snake_case rationale.
+                forkParams.config = { model_reasoning_effort: this.opts.reasoningEffort };
             }
             const result = await client.sendRequest('thread/fork', forkParams);
             const forkedThreadId = extractThreadId(result);
@@ -555,12 +578,14 @@ export class CodexAppServerAdapter {
         if (this.opts.model)
             commonParams.model = this.opts.model;
         if (this.opts.reasoningEffort) {
-            // The app-server JSON-RPC surface uses camelCase for overrides —
-            // `modelReasoningEffort` parallels `approvalPolicy` / `sandbox` on the
-            // top-level params. The corresponding config.toml key is
-            // `model_reasoning_effort` (snake_case), but the nested `config` on
-            // `thread/start` / `thread/resume` takes the camelCase form.
-            commonParams.config = { modelReasoningEffort: this.opts.reasoningEffort };
+            // The nested `config` object is raw config.toml overrides and keeps the
+            // TOML key names — `model_reasoning_effort` (snake_case) — unlike the
+            // typed top-level params (`approvalPolicy` / `sandbox`), which are
+            // camelCase. Empirically verified against both pinned CLI generations
+            // (0.133.0 and 0.144.1): the camelCase key is silently ignored
+            // (thread/start echoes reasoningEffort: null), the snake_case key is
+            // applied and echoed back.
+            commonParams.config = { model_reasoning_effort: this.opts.reasoningEffort };
         }
         const method = opts.resumeId ? 'thread/resume' : 'thread/start';
         const params = opts.resumeId

package/package.json

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

package/src/config.ts

@@ -112,20 +112,30 @@ export function normalizeSandbox(value: string): string {
 }
 
 /**
- * Normalise an approval policy string to the kebab-case form the app-server
- * JSON-RPC enum expects (`never` / `on-request` / `on-failure` /
- * `unless-trusted`). Mirrors the sandbox enum — serde rename_all kebab-case
- * applies across app-server protocol enums.
+ * Normalise an approval policy string to the enum form the app-server
+ * JSON-RPC surface accepts. Empirically verified against both pinned CLI
+ * generations: 0.144.x accepts `untrusted` / `on-request` / `granular` /
+ * `never`; 0.133.0 additionally accepted `on-failure`. Two legacy inputs are
+ * translated rather than passed through, because the server rejects the whole
+ * thread/start with "unknown variant" otherwise:
+ *
+ * - `on-failure` → `on-request` (removed from the enum in 0.144; the closest
+ *   surviving escalation policy).
+ * - `unless-trusted` → `untrusted` (the wire constant was never
+ *   `unless-trusted` on either generation — accepting-but-forwarding it made
+ *   every thread/start fail for configs that set it).
  */
 export function normalizeApprovalPolicy(value: string): string {
   const map: Record<string, string> = {
     never: 'never',
     'on-request': 'on-request',
     onrequest: 'on-request',
-    'unless-trusted': 'unless-trusted',
-    unlesstrusted: 'unless-trusted',
-    'on-failure': 'on-failure',
-    onfailure: 'on-failure',
+    untrusted: 'untrusted',
+    'unless-trusted': 'untrusted',
+    unlesstrusted: 'untrusted',
+    granular: 'granular',
+    'on-failure': 'on-request',
+    onfailure: 'on-request',
   };
   return map[value.toLowerCase()] ?? value;
 }

package/src/dispatch.ts

@@ -264,11 +264,30 @@ export class CodexAppServerAdapter implements DispatchAdapter {
       // this contract, we want a hard error surfaced to the operator, not a
       // silent fallback that hides real protocol drift.
       const turnInput = buildTurnInput(preparedBody, preparedImages);
-      const startTurn = (targetThreadId: string): Promise<unknown> =>
-        client.sendRequest('turn/start', {
+      // Per-turn overrides, first-class on both supported CLI generations
+      // (0.133 / 0.144) with "for this turn and subsequent turns" semantics.
+      // Sending the CURRENT config on every turn is what makes
+      // agent_config.update (model / thinking-effort hot updates) take
+      // effect on the next turn: thread-level config only applies at
+      // thread/start-resume, which an already-open thread never re-runs,
+      // and a prior turn's override would otherwise stay live for the rest
+      // of the subprocess lifetime.
+      //
+      // The fields are only sent when a value is configured: on the wire
+      // they are plain serde Options, so JSON null is indistinguishable
+      // from omission — there is no "reset to default" encoding. CLEARING
+      // a previously-set pin therefore keeps the old sticky value until
+      // the subprocess restarts (pod restart / redeploy / idle reap);
+      // an explicit restart-on-clear path is a tracked follow-up.
+      const startTurn = (targetThreadId: string): Promise<unknown> => {
+        const turnParams: Record<string, unknown> = {
           threadId: targetThreadId,
           input: turnInput,
-        });
+        };
+        if (this.opts.model) turnParams.model = this.opts.model;
+        if (this.opts.reasoningEffort) turnParams.effort = this.opts.reasoningEffort;
+        return client.sendRequest('turn/start', turnParams);
+      };
 
       // Start the turn. If the stored threadId is dead, we retry once with a fresh thread.
       let turnStartResult: unknown;
@@ -409,7 +428,8 @@ export class CodexAppServerAdapter implements DispatchAdapter {
       }
       if (this.opts.model) forkParams.model = this.opts.model;
       if (this.opts.reasoningEffort) {
-        forkParams.config = { modelReasoningEffort: this.opts.reasoningEffort };
+        // Raw config.toml key — see openThread for the snake_case rationale.
+        forkParams.config = { model_reasoning_effort: this.opts.reasoningEffort };
       }
       const result = await client.sendRequest('thread/fork', forkParams);
       const forkedThreadId = extractThreadId(result);
@@ -621,12 +641,14 @@ export class CodexAppServerAdapter implements DispatchAdapter {
     commonParams.sandbox = normalizeSandbox(this.opts.sandbox);
     if (this.opts.model) commonParams.model = this.opts.model;
     if (this.opts.reasoningEffort) {
-      // The app-server JSON-RPC surface uses camelCase for overrides —
-      // `modelReasoningEffort` parallels `approvalPolicy` / `sandbox` on the
-      // top-level params. The corresponding config.toml key is
-      // `model_reasoning_effort` (snake_case), but the nested `config` on
-      // `thread/start` / `thread/resume` takes the camelCase form.
-      commonParams.config = { modelReasoningEffort: this.opts.reasoningEffort };
+      // The nested `config` object is raw config.toml overrides and keeps the
+      // TOML key names — `model_reasoning_effort` (snake_case) — unlike the
+      // typed top-level params (`approvalPolicy` / `sandbox`), which are
+      // camelCase. Empirically verified against both pinned CLI generations
+      // (0.133.0 and 0.144.1): the camelCase key is silently ignored
+      // (thread/start echoes reasoningEffort: null), the snake_case key is
+      // applied and echoed back.
+      commonParams.config = { model_reasoning_effort: this.opts.reasoningEffort };
     }
 
     const method = opts.resumeId ? 'thread/resume' : 'thread/start';