@commonlyai/cli 0.1.163 → 0.1.165

package/package.json

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

package/src/commands/agent.js

@@ -1681,7 +1681,7 @@ export const performRun = ({
 
     if (adapter.requiresModel) {
       try {
-        requireSeatModel(seatEnvironment, adapter.name);
+        requireSeatModel(seatEnvironment, adapter.name, true);
       } catch (error) {
         if (error.code === 'seat_model_missing' && eventPodId && !missingModelNotices.has(eventPodId)) {
           try {

package/src/lib/adapters/claude.js

@@ -43,7 +43,7 @@
  * SIGKILL escalation is a post-v1 concern (ADR-005 §Spawning semantics).
  */
 
-import { requireSeatModel } from './seat-model.js';
+import { requireSeatModel, seatModelRequired } from './seat-model.js';
 import { spawn as childSpawn, spawnSync } from 'child_process';
 import { createHash, randomUUID } from 'crypto';
 import { accessSync, constants, realpathSync } from 'fs';
@@ -726,7 +726,7 @@ const prepareArgv = async (innerArgv, ctx) => {
 };
 
 export default {
-  requiresModel: true,
+  get requiresModel() { return seatModelRequired(); },
   name: 'claude',
   // Identity-bearing runtime tag written into AgentInstallation.config.runtime
   // by `commonly agent attach`. The CLI command stays `commonly agent attach
@@ -812,7 +812,7 @@ export default {
     // session-recovery retry, or a retry runs at a different effort than the
     // turn it replaces.
     const modelArgs = [
-      '--model', seatModel,
+      ...(seatModel ? ['--model', seatModel] : []),
       ...(ctx.environment?.effort ? ['--effort', String(ctx.environment.effort)] : []),
     ];
     const baseArgs = ['-p', fullPrompt, '--output-format', 'text', sessionFlag, sessionId, ...modelArgs];

package/src/lib/adapters/codex.js

@@ -37,7 +37,7 @@
  * SIGKILL escalation is a post-v1 concern (ADR-005 §Spawning semantics).
  */
 
-import { requireSeatModel } from './seat-model.js';
+import { requireSeatModel, seatModelRequired } from './seat-model.js';
 import { spawn as childSpawn, spawnSync } from 'child_process';
 import { createHash } from 'crypto';
 import { accessSync, constants } from 'fs';
@@ -617,7 +617,7 @@ const runCodex = ({ args, cwd, env, timeoutMs, credentials = [], spawnImpl = chi
 });
 
 export default {
-  requiresModel: true,
+  get requiresModel() { return seatModelRequired(); },
   name: 'codex',
   // See claude.js for the rationale — identity-bearing runtime tag persisted
   // to AgentInstallation.config.runtime (paired with host:'byo') so a CLI-

package/src/lib/adapters/dsh.js

@@ -248,7 +248,7 @@ export default {
   },
 
   async spawn(prompt, ctx = {}) {
-    const seatModel = requireSeatModel(ctx.environment, 'dsh');
+    const seatModel = requireSeatModel(ctx.environment, 'dsh', true);
     assertPrivateDsh(ctx.environment);
     if (!ctx.environment?.provider || !ctx.environment?.model) {
       throw new Error('dsh requires environment.provider and environment.model');

package/src/lib/adapters/goose.js

@@ -194,7 +194,7 @@ export default {
   },
 
   async spawn(prompt, ctx = {}) {
-    const seatModel = requireSeatModel(ctx.environment, 'goose');
+    const seatModel = requireSeatModel(ctx.environment, 'goose', true);
     const keyFile = await this.validateEnvironment(ctx.environment, ctx.cwd);
     const sourceEnv = ctx.env || process.env;
     const configured = ctx.environment?.provider;

package/src/lib/adapters/hermes.js

@@ -1,5 +1,5 @@
 /** Hermes 0.22.2 local CLI wrapper. Hermes owns provider I/O; this file only spawns it. */
-import { requireSeatModel } from './seat-model.js';
+import { requireSeatModel, seatModelRequired } from './seat-model.js';
 import { spawn as childSpawn, spawnSync } from 'child_process';
 import { createHash } from 'crypto';
 import { mkdir } from 'fs/promises';
@@ -138,7 +138,7 @@ const runHermes = ({ args, prompt, cwd, env, timeoutMs, spawnImpl, credentials }
 });
 
 export default {
-  requiresModel: true,
+  get requiresModel() { return seatModelRequired(); },
   name: 'hermes',
   runtimeType: 'hermes',
   supportsProvider: false,
@@ -180,7 +180,7 @@ export default {
     }
     const args = ['chat', '--oneshot', '--query-file', '-', '--format', 'stream-json',
       '--provider', 'openrouter', '--source', 'tool', '--no-restore-cwd'];
-    args.push('--model', seatModel);
+    if (seatModel) args.push('--model', seatModel);
     if (ctx.sessionId) args.push('--resume', ctx.sessionId);
     const fullPrompt = buildMemoryPreamble(prompt, ctx.memoryLongTerm, {
       freshSession: !ctx.sessionId,

package/src/lib/adapters/opencode.js

@@ -19,7 +19,7 @@
  * `text.part.text`, and token counts in `step_finish.part.tokens`.
  */
 
-import { requireSeatModel } from './seat-model.js';
+import { requireSeatModel, seatModelRequired } from './seat-model.js';
 import { spawn as childSpawn, spawnSync } from 'child_process';
 import { createHash } from 'crypto';
 import { realpathSync } from 'fs';
@@ -623,7 +623,7 @@ const describeOpenCodeVersionMismatch = (installedVersion) => {
 };
 
 export default {
-  requiresModel: true,
+  get requiresModel() { return seatModelRequired(); },
   name: 'opencode',
   runtimeType: 'opencode',
   supportsProvider: true,

package/src/lib/adapters/pi.js

@@ -410,7 +410,7 @@ export default {
   },
 
   async spawn(prompt, ctx = {}) {
-    const seatModel = requireSeatModel(ctx.environment, 'pi');
+    const seatModel = requireSeatModel(ctx.environment, 'pi', true);
     // pi has no enforced sandbox. claude.js and codex.js refuse a public-trust
     // seat they cannot confine; so does this adapter, and it refuses any declared
     // sandbox mode rather than run unconfined under a spec that promised one.

package/src/lib/adapters/qwen.js

@@ -232,7 +232,7 @@ export default {
   },
 
   async spawn(prompt, ctx = {}) {
-    const seatModel = requireSeatModel(ctx.environment, 'qwen');
+    const seatModel = requireSeatModel(ctx.environment, 'qwen', true);
     const sandbox = normalizeSandboxTrust(ctx.environment?.sandbox);
     const isPublic = sandbox?.trust === 'public';
     const mode = isPublic ? resolvePublicSandboxMode(sandbox) : null;

package/src/lib/adapters/seat-model.js

@@ -1,7 +1,10 @@
-/** A seat must choose its model independently of the operator's CLI defaults. */
-export const requireSeatModel = (environment, adapterName) => {
+/** Fleet runners opt in; BYO users retain their own CLI's model default. */
+export const seatModelRequired = () => process.env.COMMONLY_REQUIRE_SEAT_MODEL === '1';
+
+export const requireSeatModel = (environment, adapterName, required = seatModelRequired()) => {
   const model = environment?.model;
   if (typeof model !== 'string' || !model.trim()) {
+    if (!required) return undefined;
     const error = new Error(
       `${adapterName} requires environment.model; set the seat model with commonly agent config <name> --model <model> before running`,
     );