@mcoda/agents 0.1.129 → 0.1.131

package/CHANGELOG.md

@@ -1,7 +1,14 @@
 # Changelog
 
 ## Unreleased
-- None.
+- Accept every reasoning effort codex-cli accepts
+  (`minimal|low|medium|high|xhigh|max|ultra`) instead of only
+  `low|medium|high|xhigh`, so `max` and `ultra` reach
+  `-c model_reasoning_effort=...` instead of being dropped for the model
+  default. The gpt-5.1 ceiling now caps anything above `high` rather than only
+  `xhigh`, and an effort outside the enum is reported on stderr instead of
+  being ignored silently. The report names the effort the run falls back to,
+  which is the `high` cap on gpt-5.1 and the model default elsewhere.
 
 ## 0.1.75
 - Forward per-agent Codex CLI `reasoningEffort` configuration into `codex exec` so agents such as `codex55` can use `model_reasoning_effort=xhigh` without relying on process-wide environment variables.

package/dist/adapters/codex/CodexCliRunner.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"CodexCliRunner.d.ts","sourceRoot":"","sources":["../../../src/adapters/codex/CodexCliRunner.ts"],"names":[],"mappings":"AAu1BA,eAAO,MAAM,UAAU,GAAI,sBAAoB,KAAG;IAAE,EAAE,EAAE,OAAO,CAAC;IAAC,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;CA+BjG,CAAC;AAEF,eAAO,MAAM,YAAY,GACvB,QAAQ,MAAM,EACd,QAAQ,MAAM,EACd,eAAe,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EACtC,YAAY,MAAM,EAClB,kBAAkB,MAAM,KACvB,OAAO,CAAC;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAA;CAAE,CASzC,CAAC;AAEF,wBAAuB,kBAAkB,CACvC,MAAM,EAAE,MAAM,EACd,KAAK,CAAC,EAAE,MAAM,EACd,YAAY,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EACtC,SAAS,CAAC,EAAE,MAAM,EAClB,eAAe,CAAC,EAAE,MAAM,GACvB,cAAc,CAAC;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAA;CAAE,EAAE,IAAI,EAAE,OAAO,CAAC,CAoFhE"}
+{"version":3,"file":"CodexCliRunner.d.ts","sourceRoot":"","sources":["../../../src/adapters/codex/CodexCliRunner.ts"],"names":[],"mappings":"AAo4BA,eAAO,MAAM,UAAU,GAAI,sBAAoB,KAAG;IAAE,EAAE,EAAE,OAAO,CAAC;IAAC,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;CA+BjG,CAAC;AAEF,eAAO,MAAM,YAAY,GACvB,QAAQ,MAAM,EACd,QAAQ,MAAM,EACd,eAAe,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EACtC,YAAY,MAAM,EAClB,kBAAkB,MAAM,KACvB,OAAO,CAAC;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAA;CAAE,CASzC,CAAC;AAEF,wBAAuB,kBAAkB,CACvC,MAAM,EAAE,MAAM,EACd,KAAK,CAAC,EAAE,MAAM,EACd,YAAY,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EACtC,SAAS,CAAC,EAAE,MAAM,EAClB,eAAe,CAAC,EAAE,MAAM,GACvB,cAAc,CAAC;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAA;CAAE,EAAE,IAAI,EAAE,OAAO,CAAC,CAoFhE"}

package/dist/adapters/codex/CodexCliRunner.js

@@ -2,6 +2,7 @@ import { spawn, spawnSync } from "node:child_process";
 import { mkdtemp, rm, writeFile } from "node:fs/promises";
 import os from "node:os";
 import path from "node:path";
+import { CODEX_REASONING_EFFORTS, codexReasoningEffortRank, normalizeCodexReasoningEffort, } from "@mcoda/shared";
 const CODEX_MAX_BUFFER_BYTES = 10 * 1024 * 1024;
 const CODEX_REASONING_ENV = "MCODA_CODEX_REASONING_EFFORT";
 const CODEX_REASONING_ENV_FALLBACK = "CODEX_REASONING_EFFORT";
@@ -572,27 +573,54 @@ const createStreamFormatter = (model) => {
     };
     return { handleLine, end };
 };
+// An effort codex does not accept used to be dropped without a trace: the run
+// silently fell back to whatever the model would have used anyway while
+// `mcoda agent details` still showed the configured value. Say so instead, and
+// name the effort actually used, which is not always the model default -- on
+// gpt-5.1 the fallback below is `high`. Deduped per distinct (value, fallback)
+// pair so a long-running process does not repeat itself.
+const warnedReasoningEfforts = new Set();
+const warnUnsupportedReasoningEffort = (raw, fallback) => {
+    const key = `${raw}\u0000${fallback ?? ""}`;
+    if (warnedReasoningEfforts.has(key))
+        return;
+    warnedReasoningEfforts.add(key);
+    const used = fallback ? `${JSON.stringify(fallback)}` : "the model default";
+    process.stderr.write(`[mcoda] codex reasoning effort ${JSON.stringify(raw)} is not one of ` +
+        `${CODEX_REASONING_EFFORTS.join(", ")}; using ${used} instead.\n`);
+};
+// Returns the recognised effort, or the raw text under `rejected` so the caller
+// can report it once it knows which fallback the model will actually get.
 const normalizeReasoningEffort = (raw) => {
     if (!raw)
-        return undefined;
-    const normalized = raw.trim().toLowerCase();
+        return {};
+    const trimmed = raw.trim();
+    if (!trimmed)
+        return {};
+    const normalized = normalizeCodexReasoningEffort(trimmed);
     if (!normalized)
-        return undefined;
-    if (!["low", "medium", "high", "xhigh"].includes(normalized))
-        return undefined;
-    return normalized;
+        return { rejected: trimmed };
+    return { effort: normalized };
 };
+// gpt-5.1 tops out at `high`; anything stronger is rejected upstream, so cap it
+// rather than fail the run.
+const GPT_51_MAX_REASONING_EFFORT = "high";
 const resolveReasoningEffort = (model, effort) => {
-    const configured = normalizeReasoningEffort(effort ?? process.env[CODEX_REASONING_ENV] ?? process.env[CODEX_REASONING_ENV_FALLBACK]);
+    const { effort: configured, rejected } = normalizeReasoningEffort(effort ?? process.env[CODEX_REASONING_ENV] ?? process.env[CODEX_REASONING_ENV_FALLBACK]);
     const normalizedModel = (model ?? "").toLowerCase();
     const isGpt51 = normalizedModel.includes("gpt-5.1");
+    if (rejected) {
+        warnUnsupportedReasoningEffort(rejected, isGpt51 ? GPT_51_MAX_REASONING_EFFORT : undefined);
+    }
     if (configured) {
-        if (configured === "xhigh" && isGpt51)
-            return "high";
+        if (isGpt51 &&
+            codexReasoningEffortRank(configured) > codexReasoningEffortRank(GPT_51_MAX_REASONING_EFFORT)) {
+            return GPT_51_MAX_REASONING_EFFORT;
+        }
         return configured;
     }
     if (isGpt51)
-        return "high";
+        return GPT_51_MAX_REASONING_EFFORT;
     return undefined;
 };
 const extractAssistantText = (parsed) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mcoda/agents",
-  "version": "0.1.129",
+  "version": "0.1.131",
   "description": "Agent registry and capabilities for mcoda.",
   "type": "module",
   "main": "dist/index.js",
@@ -30,8 +30,8 @@
     "access": "public"
   },
   "dependencies": {
-    "@mcoda/db": "0.1.129",
-    "@mcoda/shared": "0.1.129"
+    "@mcoda/db": "0.1.131",
+    "@mcoda/shared": "0.1.131"
   },
   "scripts": {
     "build": "tsc -p tsconfig.json",