@polderlabs/bizar 6.2.2 → 6.2.3

package/cli/commands/setup-provider.test.mjs

@@ -1,11 +1,16 @@
 /**
  * cli/commands/setup-provider.test.mjs
  *
- * v6.2.2 — Unit tests for the setup-provider subcommand.
+ * v6.2.3 — Unit tests for the setup-provider subcommand.
  *
- * Exercises the pure helpers (`buildProviderBlock`, `parseFlags`)
- * and the filesystem mutations (`applyProviderBlock`, `removeProvider`)
- * with a mocked CLINE_DIR.
+ * v6.2.3 changed the storage backend from `~/.cline/cline.json` to
+ * `~/.cline/data/settings/providers.json` (the file the Cline CLI
+ * and kanban mode actually read). The default providerId changed
+ * from `9router` to `litellm` (a real entry in Cline's catalog).
+ *
+ * Exercises the pure helpers (`buildProviderBlock`, `parseFlags`,
+ * `OPENAI_COMPATIBLE_PROVIDERS`) and the filesystem mutations
+ * (`applyProviderBlock`, `removeProvider`) with a mocked CLINE_DIR.
  */
 import { test, describe, beforeEach, afterEach } from 'node:test';
 import assert from 'node:assert/strict';
@@ -19,27 +24,41 @@ const {
   removeProvider,
   listGatewayModels,
   parseFlags,
+  OPENAI_COMPATIBLE_PROVIDERS,
 } = await import('./setup-provider.mjs');
 
 const ORIG_CLINE_DIR = process.env.CLINE_DIR;
+const ORIG_DATA_DIR = process.env.CLINE_DATA_DIR;
+const ORIG_PROVIDER_PATH = process.env.CLINE_PROVIDER_SETTINGS_PATH;
 let workDir;
 
 function freshWorkDir() {
   workDir = mkdtempSync(join(tmpdir(), 'bizar-setup-provider-'));
+  // CLINE_DIR is the *base* dir (e.g. ~/.cline). The settings file
+  // lives at `${CLINE_DIR}/data/settings/providers.json`. Mocking
+  // CLINE_DIR is the easiest way to redirect the whole tree.
   process.env.CLINE_DIR = workDir;
+  // Clear overrides that could change the resolution path.
+  delete process.env.CLINE_DATA_DIR;
+  delete process.env.CLINE_PROVIDER_SETTINGS_PATH;
   return workDir;
 }
 
-function makeFakeClineJson(root, body = {}) {
+function settingsFile() {
+  return join(workDir, 'data', 'settings', 'providers.json');
+}
+
+function makeFakeSettings(root, body = {}) {
   const cfg = {
-    $schema: 'https://docs.cline.bot/config.json',
-    plugin: [],
-    default_agent: 'odin',
-    permission: 'allow',
-    snapshot: false,
+    version: 1,
+    providers: {},
+    lastUsedProvider: 'litellm',
     ...body,
   };
-  writeFileSync(join(root, 'cline.json'), JSON.stringify(cfg, null, 2));
+  const file = join(root, 'data', 'settings', 'providers.json');
+  mkdirSync(join(root, 'data', 'settings'), { recursive: true });
+  writeFileSync(file, JSON.stringify(cfg, null, 2));
+  return file;
 }
 
 beforeEach(() => {
@@ -50,137 +69,215 @@ afterEach(() => {
   if (workDir && existsSync(workDir)) rmSync(workDir, { recursive: true, force: true });
   if (ORIG_CLINE_DIR === undefined) delete process.env.CLINE_DIR;
   else process.env.CLINE_DIR = ORIG_CLINE_DIR;
+  if (ORIG_DATA_DIR === undefined) delete process.env.CLINE_DATA_DIR;
+  else process.env.CLINE_DATA_DIR = ORIG_DATA_DIR;
+  if (ORIG_PROVIDER_PATH === undefined) delete process.env.CLINE_PROVIDER_SETTINGS_PATH;
+  else process.env.CLINE_PROVIDER_SETTINGS_PATH = ORIG_PROVIDER_PATH;
+});
+
+describe('OPENAI_COMPATIBLE_PROVIDERS', () => {
+  test('includes litellm (the new v6.2.3 default)', () => {
+    assert.ok(OPENAI_COMPATIBLE_PROVIDERS.includes('litellm'),
+      'litellm must be the canonical openai-compatible providerId');
+  });
+  test('does NOT include the fake "openai-compatible" or "9router" IDs', () => {
+    assert.equal(OPENAI_COMPATIBLE_PROVIDERS.includes('openai-compatible'), false,
+      'openai-compatible is a family, not a valid providerId');
+    assert.equal(OPENAI_COMPATIBLE_PROVIDERS.includes('9router'), false,
+      '9router is not a built-in Cline providerId (would fail in kanban)');
+  });
 });
 
 describe('buildProviderBlock()', () => {
-  test('emits a baseUrl, apiKey, and bare modelIds', () => {
+  test('emits a Cline-settings-shaped block (version 1 + settings + updatedAt)', () => {
     const block = buildProviderBlock({
-      name: '9router',
+      name: 'litellm',
       gateway: 'http://localhost:20128/v1',
       apiKey: 'sk-test',
-      models: ['minimax/MiniMax-M3', 'minimax/MiniMax-M2.7'],
+      model: 'minimaxcustom/MiniMax-M3',
     });
-    assert.ok(block['9router']);
-    assert.equal(block['9router'].baseUrl, 'http://localhost:20128/v1');
-    assert.equal(block['9router'].apiKey, 'sk-test');
-    assert.deepEqual(Object.keys(block['9router'].models), ['minimax/MiniMax-M3', 'minimax/MiniMax-M2.7']);
+    assert.ok(block.settings);
+    assert.equal(block.settings.provider, 'litellm');
+    assert.equal(block.settings.model, 'minimaxcustom/MiniMax-M3');
+    assert.equal(block.settings.apiKey, 'sk-test');
+    assert.equal(block.settings.baseUrl, 'http://localhost:20128/v1');
+    assert.ok(block.settings.reasoning);
+    assert.equal(block.settings.reasoning.enabled, false);
+    assert.ok(block.updatedAt);
+    assert.match(block.updatedAt, /^\d{4}-\d{2}-\d{2}T/);
+    assert.equal(block.tokenSource, 'manual');
   });
 
   test('wraps env-var-style keys in ${env:...}', () => {
     const block = buildProviderBlock({
-      name: '9router',
+      name: 'litellm',
       gateway: 'http://localhost:20128/v1',
       apiKey: 'NINEROUTER_KEY',
-      models: ['minimax/MiniMax-M3'],
+      model: 'm1',
     });
-    assert.equal(block['9router'].apiKey, '${env:NINEROUTER_KEY}');
+    assert.equal(block.settings.apiKey, '${env:NINEROUTER_KEY}');
   });
 
-  test('keeps literal keys as-is when not env-var shaped', () => {
+  test('sets reasoning.enabled=true when reason=true', () => {
     const block = buildProviderBlock({
-      name: 'custom',
-      gateway: 'https://x.example/v1',
-      apiKey: 'sk-abc-123',
-      models: ['m1'],
+      name: 'litellm', gateway: 'http://x', apiKey: 'k', model: 'm', reason: true,
     });
-    assert.equal(block['custom'].apiKey, 'sk-abc-123');
+    assert.equal(block.settings.reasoning.enabled, true);
   });
 });
 
 describe('applyProviderBlock()', () => {
-  test('writes a new provider to ~/.cline/cline.json', () => {
-    makeFakeClineJson(workDir);
-    const block = buildProviderBlock({
-      name: '9router',
-      gateway: 'http://localhost:20128/v1',
-      apiKey: 'sk-test',
-      models: ['minimax/MiniMax-M3'],
+  test('writes to ${CLINE_DIR}/data/settings/providers.json (NOT ~/.cline/cline.json)', () => {
+    const body = buildProviderBlock({
+      name: 'litellm', gateway: 'http://localhost:20128/v1',
+      apiKey: 'sk-test', model: 'minimaxcustom/MiniMax-M3',
     });
-    const r = applyProviderBlock(block);
-    assert.equal(r.name, '9router');
-    const cfg = JSON.parse(readFileSync(join(workDir, 'cline.json'), 'utf8'));
-    assert.ok(cfg.provider['9router']);
-    assert.equal(cfg.provider['9router'].baseUrl, 'http://localhost:20128/v1');
+    const r = applyProviderBlock('litellm', body);
+    const expected = settingsFile();
+    assert.equal(r.path, expected, 'must write to the Cline settings file');
+    assert.ok(existsSync(expected));
   });
 
-  test('preserves other provider entries when adding a new one', () => {
-    makeFakeClineJson(workDir, {
-      provider: { existing: { baseUrl: 'https://a', apiKey: 'k', models: { m: {} } } },
+  test('bootstraps the settings file when it does not exist', () => {
+    // No makeFakeSettings call.
+    const body = buildProviderBlock({
+      name: 'litellm', gateway: 'http://x', apiKey: 'k', model: 'm',
     });
-    const block = buildProviderBlock({
-      name: '9router',
-      gateway: 'http://localhost:20128/v1',
-      apiKey: 'sk-test',
-      models: ['minimax/MiniMax-M3'],
-    });
-    applyProviderBlock(block);
-    const cfg = JSON.parse(readFileSync(join(workDir, 'cline.json'), 'utf8'));
-    assert.ok(cfg.provider.existing, 'existing provider preserved');
-    assert.ok(cfg.provider['9router'], 'new provider added');
+    const r = applyProviderBlock('litellm', body);
+    assert.ok(existsSync(r.path));
+    const cfg = JSON.parse(readFileSync(r.path, 'utf8'));
+    assert.equal(cfg.version, 1);
+    assert.equal(cfg.lastUsedProvider, 'litellm');
+    assert.ok(cfg.providers.litellm);
   });
 
-  test('overwrites an existing provider with the same name', () => {
-    makeFakeClineJson(workDir, {
-      provider: { '9router': { baseUrl: 'https://old', apiKey: 'old', models: {} } },
+  test('preserves other provider entries when adding a new one', () => {
+    makeFakeSettings(workDir, {
+      providers: { ollama: { settings: { provider: 'ollama', model: 'llama3' } } },
     });
-    const block = buildProviderBlock({
-      name: '9router',
-      gateway: 'http://localhost:20128/v1',
-      apiKey: 'new',
-      models: ['minimax/MiniMax-M3'],
+    const body = buildProviderBlock({
+      name: 'litellm', gateway: 'http://x', apiKey: 'k', model: 'm',
     });
-    applyProviderBlock(block);
-    const cfg = JSON.parse(readFileSync(join(workDir, 'cline.json'), 'utf8'));
-    assert.equal(cfg.provider['9router'].baseUrl, 'http://localhost:20128/v1');
-    assert.equal(cfg.provider['9router'].apiKey, 'new');
-    assert.ok(cfg.provider['9router'].models['minimax/MiniMax-M3']);
+    applyProviderBlock('litellm', body);
+    const cfg = JSON.parse(readFileSync(settingsFile(), 'utf8'));
+    assert.ok(cfg.providers.ollama, 'existing provider preserved');
+    assert.ok(cfg.providers.litellm, 'new provider added');
   });
 
-  test('throws when cline.json is missing', () => {
-    // No makeFakeClineJson call
-    const block = buildProviderBlock({
-      name: '9router',
-      gateway: 'http://localhost:20128/v1',
-      apiKey: 'sk-test',
-      models: ['minimax/MiniMax-M3'],
+  test('overwrites an existing provider with the same name', () => {
+    makeFakeSettings(workDir, {
+      providers: { litellm: { settings: { provider: 'litellm', baseUrl: 'https://old', model: 'old' } } },
     });
-    assert.throws(() => applyProviderBlock(block), /not found/);
+    const body = buildProviderBlock({
+      name: 'litellm', gateway: 'http://new', apiKey: 'new', model: 'new',
+    });
+    applyProviderBlock('litellm', body);
+    const cfg = JSON.parse(readFileSync(settingsFile(), 'utf8'));
+    assert.equal(cfg.providers.litellm.settings.baseUrl, 'http://new');
+    assert.equal(cfg.providers.litellm.settings.model, 'new');
   });
 
   test('creates a backup file', () => {
-    makeFakeClineJson(workDir, {
-      plugin: [['./plugins/bizar', {}]],
+    makeFakeSettings(workDir, {
+      providers: { x: { settings: { provider: 'x' } } },
     });
-    const block = buildProviderBlock({
-      name: '9router',
-      gateway: 'http://localhost:20128/v1',
-      apiKey: 'sk-test',
-      models: ['minimax/MiniMax-M3'],
+    const body = buildProviderBlock({
+      name: 'litellm', gateway: 'http://x', apiKey: 'k', model: 'm',
     });
-    const r = applyProviderBlock(block);
+    const r = applyProviderBlock('litellm', body);
     assert.ok(existsSync(r.backup), 'backup file created');
     const backup = JSON.parse(readFileSync(r.backup, 'utf8'));
-    assert.deepEqual(backup.plugin, [['./plugins/bizar', {}]]);
+    assert.ok(backup.providers.x, 'backup contains original entry');
+  });
+});
+
+describe('migrateLegacyOpenaiCompatible()', async () => {
+  const { migrateLegacyOpenaiCompatible } = await import('./setup-provider.mjs');
+
+  test('renames an openai-compatible entry to litellm (preserving baseUrl/apiKey/model)', () => {
+    makeFakeSettings(workDir, {
+      providers: {
+        'openai-compatible': {
+          settings: {
+            provider: 'openai-compatible',
+            baseUrl: 'http://localhost:20128/v1',
+            apiKey: 'sk-test',
+            model: 'minimaxcustom/MiniMax-M3',
+            reasoning: { enabled: false, effort: 'medium' },
+          },
+          updatedAt: '2026-07-09T21:20:00.000Z',
+          tokenSource: 'manual',
+        },
+      },
+      lastUsedProvider: 'openai-compatible',
+    });
+    const r = migrateLegacyOpenaiCompatible('litellm');
+    assert.equal(r.migrated, true);
+    assert.equal(r.from, 'openai-compatible');
+    assert.equal(r.to, 'litellm');
+    const cfg = JSON.parse(readFileSync(settingsFile(), 'utf8'));
+    assert.equal(cfg.providers['openai-compatible'], undefined, 'old key removed');
+    assert.ok(cfg.providers.litellm, 'new key added');
+    assert.equal(cfg.providers.litellm.settings.baseUrl, 'http://localhost:20128/v1', 'baseUrl preserved');
+    assert.equal(cfg.providers.litellm.settings.apiKey, 'sk-test', 'apiKey preserved');
+    assert.equal(cfg.providers.litellm.settings.model, 'minimaxcustom/MiniMax-M3', 'model preserved');
+    assert.equal(cfg.lastUsedProvider, 'litellm', 'active provider switched');
+  });
+
+  test('no-op when no legacy entry exists', () => {
+    makeFakeSettings(workDir, {
+      providers: { litellm: { settings: { provider: 'litellm' } } },
+    });
+    const r = migrateLegacyOpenaiCompatible('litellm');
+    assert.equal(r.migrated, false);
+    assert.equal(r.reason, 'no legacy entry');
+  });
+
+  test('no-op when settings file missing', () => {
+    // No makeFakeSettings call
+    const r = migrateLegacyOpenaiCompatible('litellm');
+    assert.equal(r.migrated, false);
+    assert.equal(r.reason, 'no settings file');
+  });
+
+  test('preserves lastUsedProvider when it pointed at a different entry', () => {
+    makeFakeSettings(workDir, {
+      providers: {
+        'openai-compatible': { settings: { provider: 'openai-compatible', baseUrl: 'http://a' } },
+        ollama: { settings: { provider: 'ollama' } },
+      },
+      lastUsedProvider: 'ollama',
+    });
+    migrateLegacyOpenaiCompatible('litellm');
+    const cfg = JSON.parse(readFileSync(settingsFile(), 'utf8'));
+    // Should NOT clobber the active provider if it wasn't the broken one
+    assert.equal(cfg.lastUsedProvider, 'ollama', 'active ollama preserved');
   });
 });
 
 describe('removeProvider()', () => {
   test('removes a named provider', () => {
-    makeFakeClineJson(workDir, {
-      provider: {
-        a: { baseUrl: 'https://a', apiKey: 'k', models: {} },
-        b: { baseUrl: 'https://b', apiKey: 'k', models: {} },
+    makeFakeSettings(workDir, {
+      providers: {
+        a: { settings: { provider: 'a' } },
+        b: { settings: { provider: 'b' } },
       },
     });
     const r = removeProvider('a');
     assert.equal(r.removed, true);
-    const cfg = JSON.parse(readFileSync(join(workDir, 'cline.json'), 'utf8'));
-    assert.equal(cfg.provider.a, undefined);
-    assert.ok(cfg.provider.b, 'untouched provider preserved');
+    const cfg = JSON.parse(readFileSync(settingsFile(), 'utf8'));
+    assert.equal(cfg.providers.a, undefined);
+    assert.ok(cfg.providers.b, 'untouched provider preserved');
+  });
+
+  test('returns removed=false when settings file is missing', () => {
+    const r = removeProvider('nope');
+    assert.equal(r.removed, false);
+    assert.equal(r.reason, 'no settings file');
   });
 
   test('returns removed=false for a missing provider', () => {
-    makeFakeClineJson(workDir);
+    makeFakeSettings(workDir);
     const r = removeProvider('nonexistent');
     assert.equal(r.removed, false);
     assert.equal(r.reason, 'not present');
@@ -188,23 +285,26 @@ describe('removeProvider()', () => {
 });
 
 describe('parseFlags()', () => {
-  test('parses --gateway, --key, --provider with space separator', () => {
-    const f = parseFlags(['--gateway', 'http://x', '--key', 'sk-x', '--provider', 'foo']);
+  test('parses --gateway, --key, --provider, --model with space separator', () => {
+    const f = parseFlags(['--gateway', 'http://x', '--key', 'sk-x', '--provider', 'litellm', '--model', 'm1']);
     assert.equal(f.gateway, 'http://x');
     assert.equal(f.key, 'sk-x');
-    assert.equal(f.provider, 'foo');
+    assert.equal(f.provider, 'litellm');
+    assert.equal(f.model, 'm1');
   });
 
-  test('parses --gateway=, --key=, --provider= equals form', () => {
-    const f = parseFlags(['--gateway=http://x', '--key=sk-x', '--provider=foo']);
+  test('parses equals form', () => {
+    const f = parseFlags(['--gateway=http://x', '--key=sk-x', '--provider=litellm', '--model=m1']);
     assert.equal(f.gateway, 'http://x');
     assert.equal(f.key, 'sk-x');
-    assert.equal(f.provider, 'foo');
+    assert.equal(f.provider, 'litellm');
+    assert.equal(f.model, 'm1');
   });
 
-  test('parses --list and --remove', () => {
-    const f = parseFlags(['--list']);
+  test('parses --list, --discover, --remove', () => {
+    const f = parseFlags(['--list', '--discover']);
     assert.equal(f.list, true);
+    assert.equal(f.discover, true);
     const g = parseFlags(['--remove', 'myprov']);
     assert.equal(g.remove, 'myprov');
   });

package/cli/commands/validate.mjs

@@ -65,6 +65,23 @@ const REQUIRED_HOOKS = [
 
 const REQUIRED_RUNTIME_DEPS = ['zod', '@cline/sdk', '@cline/core', '@cline/shared'];
 
+// v6.2.3 — Built-in Cline providerIds that are safe for any
+// "OpenAI-compatible endpoint" use case. The harness warns about
+// any providerId NOT in this set (since they fail in kanban mode).
+const OPENAI_COMPATIBLE_BUILTIN_IDS = new Set([
+  'litellm', 'cline', 'ollama', 'lmstudio', 'huggingface',
+  // Plus the first-class providers (also OpenAI-compatible):
+  'openrouter', 'deepseek', 'xai', 'together', 'fireworks',
+  'groq', 'cerebras', 'sambanova', 'nebius', 'baseten',
+  'requesty', 'vercel-ai-gateway', 'v0', 'aihubmix', 'hicap',
+  'nousResearch', 'huawei-cloud-maas', 'qwen', 'qwen-code',
+  'doubao', 'zai', 'zai-coding-plan', 'moonshot', 'wandb',
+  'xiaomi', 'kilo', 'oca', 'asksage', 'sapaicore',
+  // Non-OpenAI-compatible (kept so we don't false-positive):
+  'anthropic', 'google', 'bedrock', 'vertex', 'gemini',
+  'openai', 'openai-native', 'openai-codex',
+]);
+
 function check(name, fn) {
   return Promise.resolve()
     .then(fn)
@@ -284,6 +301,50 @@ const CHECKS = {
     return `user-configured providers: ${summary.join('; ')}`;
   },
 
+  'cline-settings-provider': async () => {
+    // v6.2.3 — The Cline CLI and kanban mode read from
+    // `~/.cline/data/settings/providers.json`. The harness config
+    // (`cline.json`) is a separate thing. This check inspects the
+    // actual settings file and:
+    //   1. Warns if the user has a legacy `openai-compatible` providerId
+    //      (from the v6.0.1 Cline auto-migration shim). Kanban mode
+    //      fails on this with "Unknown or disabled provider". Run
+    //      `bizar setup-provider` to auto-migrate.
+    //   2. Reports the active `lastUsedProvider` so the user can
+    //      see which provider Cline will actually use on the next
+    //      session.
+    const file = join(homedir(), '.cline', 'data', 'settings', 'providers.json');
+    if (!existsSync(file)) {
+      return 'no settings file at ~/.cline/data/settings/providers.json — run `bizar setup-provider` to add one';
+    }
+    const settings = readJsonSafe(file);
+    if (!settings || !settings.providers) {
+      return 'settings file is empty or invalid — run `bizar setup-provider`';
+    }
+    const names = Object.keys(settings.providers);
+    if (names.length === 0) {
+      throw new Error('no providers configured in Cline settings — run `bizar setup-provider`');
+    }
+    const issues = [];
+    if (names.includes('openai-compatible')) {
+      issues.push(`legacy 'openai-compatible' providerId present (broken in kanban mode) — run \`bizar setup-provider\` to auto-migrate to 'litellm'`);
+    }
+    // Check for any providerId that's not in Cline's built-in catalog.
+    for (const n of names) {
+      if (n === 'openai-compatible') continue; // handled above
+      if (!OPENAI_COMPATIBLE_BUILTIN_IDS.has(n)) {
+        issues.push(`'${n}' providerId is NOT in Cline's built-in catalog (likely broken in kanban mode)`);
+      }
+    }
+    const lastUsed = settings.lastUsedProvider || '(none)';
+    if (issues.length > 0) {
+      // Throw so the ⚠ marker shows up; the check is in LENIENT_CHECKS
+      // so this won't cause `bizar validate` to exit non-zero.
+      throw new Error(`WARN: ${issues.join('; ')}. lastUsedProvider=${lastUsed} (${names.join(', ')})`);
+    }
+    return `lastUsedProvider=${lastUsed}; providers: ${names.join(', ')}`;
+  },
+
   '9router-reachable': async () => {
     const url = process.env.NINEROUTER_URL || 'http://localhost:20128';
     const ac = new AbortController();
@@ -371,12 +432,14 @@ const CHECK_ORDER = [
   'default-agent-set',
   'instructions-loaded',
   'provider-config',
+  'cline-settings-provider',
   '9router-reachable',
 ];
 
 const LENIENT_CHECKS = new Set([
   '9router-reachable',
   'provider-config', // v6.2.2+ — installer no longer touches provider config; user must configure
+  'cline-settings-provider', // v6.2.3 — warns about legacy/fake providerIds; non-blocking
 ]);
 
 export function showValidateHelp() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polderlabs/bizar",
-  "version": "6.2.2",
+  "version": "6.2.3",
   "description": "Norse-pantheon multi-agent system for cline — 13 agents across 4 cost tiers with cost-aware routing, plans, and a configurable agent harness. v4 ships as a single npm package bundling the dashboard server, cline plugin, and typed SDK.",
   "type": "module",
   "bin": {
@@ -24,7 +24,7 @@
     "test:sdk": "node_modules/.bin/vitest run --root packages/sdk",
     "test:web": "cd bizar-dash && npx vitest run",
     "test:sdk:watch": "node_modules/.bin/vitest --root packages/sdk",
-    "test": "npm run typecheck && npm run test:sdk && node --test cli/install.test.mjs cli/provision.test.mjs cli/commands/validate.test.mjs cli/commands/setup-provider.test.mjs && bun test plugins/bizar/tests/loop.test.ts plugins/bizar/tests/block.test.ts plugins/bizar/tests/stall-think.test.ts plugins/bizar/tests/tools/bg-get-comments.test.ts plugins/bizar/tests/tools/bg-spawn-delegation.test.ts plugins/bizar/tests/tools/cline-runner.test.ts plugins/bizar/tests/settings.test.ts plugins/bizar/tests/commands.test.ts plugins/bizar/tests/commands-impl.test.ts plugins/bizar/tests/tools/plan-action.test.ts plugins/bizar/tests/tools/wait-for-feedback.test.ts plugins/bizar/tests/tools/read-glyph-feedback.test.ts plugins/bizar/tests/reasoning-clean.test.ts plugins/bizar/tests/key-rotation.test.ts && node bizar-dash/tests/smoke-v2.mjs && node --test bizar-dash/tests/path-safe.test.mjs bizar-dash/tests/tmux-wrap.test.mjs bizar-dash/tests/cline-sessions-detail.test.mjs bizar-dash/tests/cline-runner.test.mjs bizar-dash/tests/mod-instructions.node.test.mjs bizar-dash/tests/mod-upgrade.node.test.mjs bizar-dash/tests/graphify-mod-spawn.node.test.mjs bizar-dash/tests/no-agent-browser.node.test.mjs bizar-dash/tests/providers-store-backup-keys.node.test.mjs bizar-dash/tests/dashboard-ports.test.mjs bizar-dash/tests/submit-feedback.test.mjs bizar-dash/tests/yaml.test.mjs bizar-dash/tests/memory-store.test.mjs bizar-dash/tests/memory-schema.test.mjs bizar-dash/tests/memory-secrets.test.mjs bizar-dash/tests/memory-git.test.mjs bizar-dash/tests/memory-sync.test.mjs bizar-dash/tests/obsidian-back-compat.test.mjs bizar-dash/tests/memory-lightrag.test.mjs bizar-dash/tests/memory-config.test.mjs bizar-dash/tests/memory-cli.test.mjs bizar-dash/tests/memory-cli-readlistdelete.test.mjs bizar-dash/tests/memory-cli-setup.test.mjs bizar-dash/tests/memory-conflicts.test.mjs bizar-dash/tests/memory-namespace.test.mjs bizar-dash/tests/memory-path-safety.test.mjs bizar-dash/tests/memory-protocol-drift.test.mjs bizar-dash/tests/memory-roundtrip.test.mjs bizar-dash/tests/cli-bugfixes.test.mjs bizar-dash/tests/server-bugfixes.test.mjs bizar-dash/tests/frontend-bugfixes.test.mjs",
+    "test": "npm run typecheck && npm run test:sdk && node --test cli/install.test.mjs cli/provision.test.mjs cli/commands/validate.test.mjs cli/commands/setup-provider.test.mjs cli/commands/rca.test.mjs && bun test plugins/bizar/tests/loop.test.ts plugins/bizar/tests/block.test.ts plugins/bizar/tests/stall-think.test.ts plugins/bizar/tests/tools/bg-get-comments.test.ts plugins/bizar/tests/tools/bg-spawn-delegation.test.ts plugins/bizar/tests/tools/cline-runner.test.ts plugins/bizar/tests/settings.test.ts plugins/bizar/tests/commands.test.ts plugins/bizar/tests/commands-impl.test.ts plugins/bizar/tests/tools/plan-action.test.ts plugins/bizar/tests/tools/wait-for-feedback.test.ts plugins/bizar/tests/tools/read-glyph-feedback.test.ts plugins/bizar/tests/reasoning-clean.test.ts plugins/bizar/tests/key-rotation.test.ts && node bizar-dash/tests/smoke-v2.mjs && node --test bizar-dash/tests/path-safe.test.mjs bizar-dash/tests/tmux-wrap.test.mjs bizar-dash/tests/cline-sessions-detail.test.mjs bizar-dash/tests/cline-runner.test.mjs bizar-dash/tests/mod-instructions.node.test.mjs bizar-dash/tests/mod-upgrade.node.test.mjs bizar-dash/tests/graphify-mod-spawn.node.test.mjs bizar-dash/tests/no-agent-browser.node.test.mjs bizar-dash/tests/providers-store-backup-keys.node.test.mjs bizar-dash/tests/dashboard-ports.test.mjs bizar-dash/tests/submit-feedback.test.mjs bizar-dash/tests/yaml.test.mjs bizar-dash/tests/memory-store.test.mjs bizar-dash/tests/memory-schema.test.mjs bizar-dash/tests/memory-secrets.test.mjs bizar-dash/tests/memory-git.test.mjs bizar-dash/tests/memory-sync.test.mjs bizar-dash/tests/obsidian-back-compat.test.mjs bizar-dash/tests/memory-lightrag.test.mjs bizar-dash/tests/memory-config.test.mjs bizar-dash/tests/memory-cli.test.mjs bizar-dash/tests/memory-cli-readlistdelete.test.mjs bizar-dash/tests/memory-cli-setup.test.mjs bizar-dash/tests/memory-conflicts.test.mjs bizar-dash/tests/memory-namespace.test.mjs bizar-dash/tests/memory-path-safety.test.mjs bizar-dash/tests/memory-protocol-drift.test.mjs bizar-dash/tests/memory-roundtrip.test.mjs bizar-dash/tests/cli-bugfixes.test.mjs bizar-dash/tests/server-bugfixes.test.mjs bizar-dash/tests/frontend-bugfixes.test.mjs",
     "build": "npm run build:sdk && npm run build:dash",
     "prepublishOnly": "npm run build"
   },

package/plugins/bizar/package.json

@@ -1,14 +1,19 @@
 {
   "name": "@polderlabs/bizar-plugin",
-  "version": "6.2.2",
+  "version": "6.2.3",
   "description": "Bizar Norse-pantheon multi-agent plugin for Cline — 14 agents across 4 cost tiers with cost-aware routing and plans.",
   "type": "module",
   "main": "./index.ts",
   "cline": {
     "plugins": [
       {
-        "paths": ["./index.ts"],
-        "capabilities": ["tools", "hooks"]
+        "paths": [
+          "./index.ts"
+        ],
+        "capabilities": [
+          "tools",
+          "hooks"
+        ]
       }
     ]
   },
@@ -21,8 +26,14 @@
     "@cline/shared": "*"
   },
   "peerDependenciesMeta": {
-    "@cline/sdk": { "optional": true },
-    "@cline/core": { "optional": true },
-    "@cline/shared": { "optional": true }
+    "@cline/sdk": {
+      "optional": true
+    },
+    "@cline/core": {
+      "optional": true
+    },
+    "@cline/shared": {
+      "optional": true
+    }
   }
 }

package/plugins/bizar/src/clineruntime.ts

@@ -160,7 +160,7 @@ export class ClineRuntime {
       workspaceRoot: opts.workspaceRoot,
       cwd: opts.workspaceRoot,
       enableTools: true,
-      enableSpawnAgent: false,
+      enableSpawnAgent: true,
       enableAgentTeams: true,
       ...(execution ? { execution } : {}),
       ...(recovery ? { onConsecutiveMistakeLimitReached: recovery } : {}),

package/plugins/bizar/tests/clineruntime-config.test.ts

@@ -180,7 +180,7 @@ describe("ClineRuntime.startSession — agent teams plumbing (v6.2.0)", () => {
     expect(cfg.enableAgentTeams).toBe(true);
   });
 
-  test("enableTools is true, enableSpawnAgent is false", async () => {
+  test("enableTools is true, enableSpawnAgent is true (subagents + task tool)", async () => {
     const runtime = new ClineRuntime({ logger: stubLogger() });
     (runtime as unknown as { core: unknown }).core = makeFakeCore();
     await runtime.startSession({
@@ -191,7 +191,8 @@ describe("ClineRuntime.startSession — agent teams plumbing (v6.2.0)", () => {
     });
     const cfg = capturedConfig[0]!.config;
     expect(cfg.enableTools).toBe(true);
-    expect(cfg.enableSpawnAgent).toBe(false);
+    expect(cfg.enableSpawnAgent).toBe(true,
+      "enableSpawnAgent must be true so Odin can use the `task` tool and `use_subagents`");
   });
 
   test("all three boolean flags survive even when caller passes no opts", async () => {
@@ -206,7 +207,27 @@ describe("ClineRuntime.startSession — agent teams plumbing (v6.2.0)", () => {
     const cfg = capturedConfig[0]!.config;
     expect(cfg.enableAgentTeams).toBe(true);
     expect(cfg.enableTools).toBe(true);
-    expect(cfg.enableSpawnAgent).toBe(false);
+    expect(cfg.enableSpawnAgent).toBe(true,
+      "v6.2.3 — subagents + task tool must be available even without caller opts");
+  });
+});
+
+describe("ClineRuntime.startSession — subagents plumbing (v6.2.3)", () => {
+  test("enableSpawnAgent regression: must NEVER be false (v6.2.3 fix)", async () => {
+    // v6.2.3 flipped enableSpawnAgent from false → true. Before this
+    // change, Odin could not use the `task` tool or `use_subagents`,
+    // which silently broke subagent delegation. This test pins the fix.
+    const runtime = new ClineRuntime({ logger: stubLogger() });
+    (runtime as unknown as { core: unknown }).core = makeFakeCore();
+    await runtime.startSession({
+      providerId: "anthropic",
+      modelId: "claude-sonnet-4-6",
+      workspaceRoot: "/tmp",
+      prompt: "go",
+    });
+    const cfg = capturedConfig[0]!.config;
+    expect(cfg.enableSpawnAgent).not.toBe(false);
+    expect(cfg.enableSpawnAgent).toBe(true);
   });
 });