@commonlyai/cli 0.1.27 → 0.1.29

package/package.json

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

package/src/commands/agent.js

@@ -1001,8 +1001,9 @@ export const performRun = ({
       }
     }
 
+    let turnResult;
     try {
-      return await runTurn({
+      turnResult = await runTurn({
         event,
         eventPodId,
         prompt: peerFrame ? `${peerFrame}\n\n${prompt}` : prompt,
@@ -1011,8 +1012,20 @@ export const performRun = ({
         claimKeeper,
         trigger,
       });
+      return turnResult;
     } finally {
-      await claimKeeper?.release();
+      // A silent, normally completed human wake is an explicit decline, not
+      // a successful answer. Tell the kernel so it can hand the message to
+      // exactly one remaining original listener. Any posted reply, refusal
+      // with a reason, or thrown spawn retains completion/legacy semantics:
+      // re-offering those would duplicate a visible response or defeat normal
+      // at-least-once redelivery after an infrastructure failure.
+      const claimOutcome = event.type === 'message.posted'
+        && event.payload?.senderIsHuman === true
+        && turnResult?.outcome === 'no_action' && !turnResult?.reason
+        ? 'declined'
+        : (turnResult ? 'completed' : undefined);
+      await claimKeeper?.release(claimOutcome);
     }
   };
 

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/api.js

@@ -63,9 +63,10 @@ export const createClient = ({ instance = null, token = undefined } = {}) => {
     body: JSON.stringify(body),
   }).then((res) => handleResponse(res, session));
 
-  const del = (path) => fetch(`${baseUrl}${path}`, {
+  const del = (path, body) => fetch(`${baseUrl}${path}`, {
     method: 'DELETE',
     headers: headers(authToken),
+    ...(body === undefined ? {} : { body: JSON.stringify(body) }),
   }).then((res) => handleResponse(res, session));
 
   // Multipart upload via native FormData/Blob (Node 18+) — no runtime deps.

package/src/lib/enforcement.js

@@ -420,11 +420,15 @@ export const createClaimKeeper = (client, {
       if (timer && typeof timer.unref === 'function') timer.unref();
     },
 
-    async release() {
+    async release(outcome) {
       stopRenewal();
       if (!acquired || lost) return;
       try {
-        await client.del(path);
+        // Preserve the one-argument legacy call when there is no explicit
+        // outcome. Besides keeping old clients' DELETE shape intact, callers
+        // that assert their transport arguments must not see a synthetic
+        // `undefined` body. An explicit outcome is the new D6.1 contract.
+        await (outcome ? client.del(path, { outcome }) : client.del(path));
       } catch {
         // Best-effort: a miss just means the lease already expired.
       }

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');