@commonlyai/cli 0.1.28 → 0.1.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@commonlyai/cli",
-  "version": "0.1.28",
+  "version": "0.1.29",
   "license": "Apache-2.0",
   "description": "The Commonly CLI \u2014 connect agents, manage pods, iterate fast",
   "type": "module",

package/src/lib/adapters/claude.js

@@ -488,7 +488,15 @@ export default {
     // present in one but not the other means a retry silently runs a different
     // model than the turn it is replacing — the same drifting-copy shape that
     // has bitten this codebase repeatedly.
-    const modelArgs = ctx.environment?.model ? ['--model', String(ctx.environment.model)] : [];
+    // `effort` rides in the same array for the same reason: Sam's 2026-09-01
+    // order ("flip all fable agents into fable 5.1 with high or above
+    // effort") needs both facts to reach every spawn, including the
+    // session-recovery retry, or a retry runs at a different effort than the
+    // turn it replaces.
+    const modelArgs = [
+      ...(ctx.environment?.model ? ['--model', String(ctx.environment.model)] : []),
+      ...(ctx.environment?.effort ? ['--effort', String(ctx.environment.effort)] : []),
+    ];
     const baseArgs = ['-p', fullPrompt, '--output-format', 'text', sessionFlag, sessionId, ...modelArgs];
 
     if (ctx.environment && ctx.cwd) {

package/src/lib/environment.js

@@ -41,7 +41,7 @@ import { homedir } from 'os';
 // "persona and runtime are chosen separately" requires to mean anything for a
 // BYO seat, and what lets an identity card answer "what is this running".
 const ALLOWED_TOP_KEYS = new Set([
-  'version', 'workspace', 'sandbox', 'skills', 'mcp', 'model',
+  'version', 'workspace', 'sandbox', 'skills', 'mcp', 'model', 'effort',
 ]);
 const ALLOWED_SANDBOX_MODES = new Set([
   'none', 'workspace', 'read-only', 'bwrap', 'firejail', 'container', 'managed',
@@ -135,6 +135,16 @@ export const validateEnvironmentSpec = (spec) => {
     }
   }
 
+  // `effort` is the reasoning budget the claude adapter passes to `--effort`.
+  // Unlike `model` it IS a closed set — the CLI documents exactly these — so a
+  // typo fails here at attach time, not silently at the first spawn.
+  if (spec.effort !== undefined) {
+    const EFFORTS = ['low', 'medium', 'high', 'xhigh', 'max'];
+    if (typeof spec.effort !== 'string' || !EFFORTS.includes(spec.effort)) {
+      errors.push(`effort must be one of: ${EFFORTS.join(', ')}`);
+    }
+  }
+
   if (spec.workspace !== undefined) {
     if (typeof spec.workspace !== 'object' || spec.workspace === null) {
       errors.push('workspace must be an object');