@addai/node 0.27.0 → 0.27.1

package/dist/kimi-spawn.js

@@ -68,17 +68,37 @@ const child_process_1 = require("child_process");
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const kimi_binary_1 = require("./kimi-binary");
+const harness_registry_1 = require("./harness-registry");
 const win_1 = require("./win");
 const events_1 = require("./events");
 const think_split_1 = require("./think-split");
 const mcp_headers_1 = require("./mcp-headers");
-/** kimi forwards this string to the provider verbatim — the override exists
- *  precisely to bypass the model's declared support_efforts — so a level kimi
- *  has never heard of reaches the API as-is. It retired 'max' (it migrates a
- *  stored 'max' to 'high' on load) and never had our 'xhigh', so both collapse
- *  to the highest level it does know. */
+/**
+ * The effort level to send kimi, or null to send nothing.
+ *
+ * KIMI_MODEL_THINKING_EFFORT is a real kimi env var, but it exists precisely to
+ * BYPASS the model's declared support_efforts — kimi forwards the string to the
+ * provider verbatim and lets the API judge it. Setting it unconditionally is
+ * therefore not a safe default: an entity on medium effort got
+ *
+ *   provider.api_error: 400 Invalid request Error
+ *
+ * four seconds into the run, every run, because the model behind kimi takes no
+ * thinking effort at all.
+ *
+ * The harness registry already records which levels a harness supports, and
+ * kimi's list is empty — so the registry decides, and an empty list means the
+ * variable is never set. If kimi gains effort support, updating the registry is
+ * enough; nothing here needs to change.
+ */
 function kimiEffort(level) {
-    return level === 'max' || level === 'xhigh' ? 'high' : level;
+    const supported = (0, harness_registry_1.harness)('kimi')?.efforts ?? [];
+    if (supported.length === 0)
+        return null;
+    // 'max' was retired by kimi (it migrates a stored 'max' to 'high') and it
+    // never had our 'xhigh', so both collapse to the highest level it declares.
+    const wanted = level === 'max' || level === 'xhigh' ? 'high' : level;
+    return supported.includes(wanted) ? wanted : null;
 }
 /**
  * Write the entity's MCP servers where kimi will actually read them:
@@ -339,13 +359,12 @@ function spawnKimi(input) {
     // No HOME override: the real $HOME flows through so kimi's own auth +
     // shelled-out tools (git/pip/node) can find their config files. The
     // explicit --mcp-config-file already prevents host mcp.json leakage.
+    const effortValue = input.effortLevel ? kimiEffort(input.effortLevel) : null;
     const childEnv = {
         ...process.env,
         TERM: 'dumb',
-        // kimi's own effort override. It bypasses the model's support_efforts
-        // list, and kimi migrates a stored 'max' to 'high' — so normalise here
-        // rather than send a level the provider may reject.
-        ...(input.effortLevel ? { KIMI_MODEL_THINKING_EFFORT: kimiEffort(input.effortLevel) } : {}),
+        // Effort, only when the registry says kimi can take one — see kimiEffort.
+        ...(effortValue ? { KIMI_MODEL_THINKING_EFFORT: effortValue } : {}),
     };
     const listeners = [];
     const emit = (e) => { for (const l of listeners)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@addai/node",
-  "version": "0.27.0",
+  "version": "0.27.1",
   "description": "Daemon that pairs a machine with your +Ai account and runs Claude / Codex / Kimi / Gemini agents on its behalf. Reachable via Supabase from Vault, Entity Studio, or any other +Ai surface.",
   "license": "MIT",
   "keywords": [