@polderlabs/bizar 6.0.2 → 6.2.0

package/cli/commands/validate.test.mjs

@@ -0,0 +1,279 @@
+/**
+ * cli/commands/validate.test.mjs
+ *
+ * v6.2.0 — Unit tests for the new `bizar validate` subcommand.
+ *
+ * We exercise `runValidate()` directly. CLINE_DIR is mocked via
+ * `process.env.CLINE_DIR`; a minimal-but-complete Bizar install is
+ * staged inside a tmpdir. We then progressively delete files to
+ * verify each check fails with a sensible message.
+ */
+import { test, describe, beforeEach, afterEach } from 'node:test';
+import assert from 'node:assert/strict';
+import { mkdtempSync, mkdirSync, writeFileSync, rmSync, existsSync, readFileSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+import { Writable } from 'node:stream';
+
+const { runValidate } = await import('./validate.mjs');
+
+const ORIG_CLINE_DIR = process.env.CLINE_DIR;
+const ORIG_HOME = process.env.HOME;
+
+let workDir;
+
+function freshWorkDir() {
+  const root = mkdtempSync(join(tmpdir(), 'bizar-validate-'));
+  process.env.CLINE_DIR = root;
+  return root;
+}
+
+function makeFakeClineInstall(root) {
+  writeFileSync(
+    join(root, 'cline.json'),
+    JSON.stringify({
+      $schema: 'https://docs.cline.bot/config.json',
+      plugin: [[
+        './plugins/bizar',
+        { loopThresholdWarn: 5, loopThresholdEscalate: 8, loopThresholdBlock: 12, loopWindowSize: 10 },
+      ]],
+      default_agent: 'odin',
+      permission: 'allow',
+      snapshot: false,
+      instructions: ['.cline/instructions/bizar-tools.md'],
+      provider: {
+        '9router': {
+          baseUrl: 'http://localhost:20128/v1',
+          apiKey: '${env:NINEROUTER_KEY}',
+          models: { 'minimax/MiniMax-M3': { reasoning: true } },
+        },
+        minimax: { models: {} },
+      },
+    }, null, 2),
+  );
+  const agentsDir = join(root, 'agents');
+  mkdirSync(agentsDir, { recursive: true });
+  for (const f of [
+    'odin.md', 'vor.md', 'frigg.md', 'quick.md',
+    'mimir.md', 'heimdall.md', 'hermod.md', 'thor.md', 'baldr.md',
+    'tyr.md', 'vidarr.md', 'forseti.md',
+    'semble-search.md', 'agent-browser.md',
+  ]) {
+    writeFileSync(join(agentsDir, f), '# fake agent');
+  }
+  writeFileSync(join(agentsDir, 'odin.yaml'), 'name: odin\ndescription: odin\n');
+  const cmdsDir = join(root, 'commands');
+  mkdirSync(cmdsDir, { recursive: true });
+  for (const f of [
+    'audit.md', 'bizar.md', 'explain.md', 'init.md', 'learn.md',
+    'plan.md', 'plow-through.md', 'pr-review.md', 'tailscale-serve.md',
+    'visual-plan.md',
+    'team.md', 'test.md', 'validate.md',
+  ]) {
+    writeFileSync(join(cmdsDir, f), '# fake command');
+  }
+  const skillsDir = join(root, 'skills');
+  mkdirSync(skillsDir, { recursive: true });
+  for (const skill of ['9router', 'bizar', 'obsidian']) {
+    mkdirSync(join(skillsDir, skill), { recursive: true });
+    writeFileSync(join(skillsDir, skill, 'SKILL.md'), '# fake skill');
+  }
+  const rulesDir = join(root, 'rules');
+  mkdirSync(rulesDir, { recursive: true });
+  for (const r of ['general.md', 'git.md', 'javascript.md', 'python.md', 'testing.md', 'thinking.md', 'uncertainty.md']) {
+    writeFileSync(join(rulesDir, r), '# fake rule');
+  }
+  const hooksDir = join(root, 'hooks');
+  mkdirSync(hooksDir, { recursive: true });
+  for (const h of ['pre-tool-use.md', 'post-tool-use.md', 'README.md']) {
+    writeFileSync(join(hooksDir, h), '# fake hook');
+  }
+  const pluginDir = join(root, 'plugins', 'bizar');
+  mkdirSync(pluginDir, { recursive: true });
+  writeFileSync(join(pluginDir, 'index.ts'), '// fake plugin entry — >100 bytes to pass size check\n'.repeat(10));
+  mkdirSync(join(pluginDir, 'src'), { recursive: true });
+  writeFileSync(
+    join(pluginDir, 'src', 'clineruntime.ts'),
+    'export class ClineRuntime { config = { enableAgentTeams: true }; }\n',
+  );
+  // node_modules with the required runtime deps
+  const nmDir = join(pluginDir, 'node_modules');
+  mkdirSync(nmDir, { recursive: true });
+  mkdirSync(join(nmDir, 'zod'), { recursive: true });
+  writeFileSync(join(nmDir, 'zod', 'package.json'), '{}');
+  mkdirSync(join(nmDir, '@cline'), { recursive: true });
+  for (const dep of ['@cline/sdk', '@cline/core', '@cline/shared']) {
+    mkdirSync(join(nmDir, dep), { recursive: true });
+    writeFileSync(join(nmDir, dep, 'package.json'), '{}');
+  }
+}
+
+beforeEach(() => {
+  workDir = freshWorkDir();
+  makeFakeClineInstall(workDir);
+});
+
+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_HOME === undefined) delete process.env.HOME;
+  else process.env.HOME = ORIG_HOME;
+});
+
+/**
+ * Capture stdout + exit code for a single runValidate() call.
+ * The test runner writes its own progress to stdout, so we swap
+ * process.stdout for a sink Writable during the call.
+ */
+async function captureRun(opts) {
+  const captured = [];
+  const sink = new Writable({
+    write(chunk, _enc, cb) {
+      captured.push(Buffer.from(chunk));
+      cb();
+    },
+  });
+  const origStdoutDescriptor = Object.getOwnPropertyDescriptor(process, 'stdout');
+  const origExit = process.exit;
+  let exitCode = null;
+  Object.defineProperty(process, 'stdout', { value: sink, configurable: true, writable: true });
+  process.exit = (code) => { exitCode = code; };
+  try {
+    await runValidate(opts);
+  } finally {
+    if (origStdoutDescriptor) Object.defineProperty(process, 'stdout', origStdoutDescriptor);
+    process.exit = origExit;
+  }
+  return { stdout: Buffer.concat(captured).toString('utf8'), exitCode };
+}
+
+describe('runValidate() — JSON output', () => {
+  test('returns structured JSON with passed/failed/results', async () => {
+    const { stdout, exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 0, 'should exit 0 on success');
+    const parsed = JSON.parse(stdout);
+    assert.equal(typeof parsed.passed, 'number');
+    assert.equal(typeof parsed.failed, 'number');
+    assert.ok(Array.isArray(parsed.results));
+    assert.ok(parsed.results.length >= 18, 'expected at least 18 checks');
+    for (const r of parsed.results) {
+      assert.ok(typeof r.name === 'string');
+      assert.ok(typeof r.ok === 'boolean');
+      assert.ok(typeof r.message === 'string');
+    }
+  });
+
+  test('fails on missing team command', async () => {
+    rmSync(join(workDir, 'commands', 'team.md'));
+    const { exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 1, 'should exit 1 on team-command-present failure');
+  });
+
+  test('fails on missing test command', async () => {
+    rmSync(join(workDir, 'commands', 'test.md'));
+    const { exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 1);
+  });
+
+  test('fails on missing validate command', async () => {
+    rmSync(join(workDir, 'commands', 'validate.md'));
+    const { exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 1);
+  });
+
+  test('fails on missing agent file', async () => {
+    rmSync(join(workDir, 'agents', 'mimir.md'));
+    const { exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 1);
+  });
+
+  test('fails when cline.json missing', async () => {
+    rmSync(join(workDir, 'cline.json'));
+    const { exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 1);
+  });
+
+  test('fails when enableAgentTeams is false in clineruntime.ts', async () => {
+    writeFileSync(
+      join(workDir, 'plugins', 'bizar', 'src', 'clineruntime.ts'),
+      'export class ClineRuntime { config = { enableAgentTeams: false }; }\n',
+    );
+    const { exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 1, 'should fail when enableAgentTeams is false');
+  });
+
+  test('fails when provider config is missing', async () => {
+    const cfg = JSON.parse(readFileSync(join(workDir, 'cline.json'), 'utf8'));
+    delete cfg.provider;
+    writeFileSync(join(workDir, 'cline.json'), JSON.stringify(cfg, null, 2));
+    const { exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 1);
+  });
+
+  test('passes when only 9router is configured (no minimax)', async () => {
+    const cfg = JSON.parse(readFileSync(join(workDir, 'cline.json'), 'utf8'));
+    delete cfg.provider.minimax;
+    writeFileSync(join(workDir, 'cline.json'), JSON.stringify(cfg, null, 2));
+    const { exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 0);
+  });
+
+  test('passes when only minimax is configured (no 9router)', async () => {
+    const cfg = JSON.parse(readFileSync(join(workDir, 'cline.json'), 'utf8'));
+    delete cfg.provider['9router'];
+    writeFileSync(join(workDir, 'cline.json'), JSON.stringify(cfg, null, 2));
+    const { exitCode } = await captureRun({ json: true });
+    assert.equal(exitCode, 0);
+  });
+
+  test('default mode is lenient on 9router-unreachable', async () => {
+    const { stdout, exitCode } = await captureRun({ json: true });
+    const parsed = JSON.parse(stdout);
+    const nineR = parsed.results.find((r) => r.name === '9router-reachable');
+    assert.ok(nineR, '9router-reachable check should be present');
+    if (!nineR.ok) {
+      // 9router is not running. Default mode must be lenient.
+      assert.equal(exitCode, 0, 'default mode should be lenient when 9router is down');
+    } else {
+      // 9router is running (some test envs have it). Just confirm the
+      // check is in the list and exit code is sane.
+      assert.equal(exitCode, 0, 'expected exit 0 when all critical checks pass');
+    }
+  });
+
+  test('--strict fails when 9router is unreachable', async () => {
+    // We can't easily kill 9router in a test, but we can prove that
+    // --strict mode would fail if 9router were down: set NINEROUTER_URL
+    // to a port nothing is listening on.
+    const origUrl = process.env.NINEROUTER_URL;
+    process.env.NINEROUTER_URL = 'http://127.0.0.1:1';  // nothing here
+    try {
+      const { exitCode } = await captureRun({ json: true, strict: true });
+      assert.equal(exitCode, 1, 'strict mode should fail when 9router is unreachable');
+    } finally {
+      if (origUrl === undefined) delete process.env.NINEROUTER_URL;
+      else process.env.NINEROUTER_URL = origUrl;
+    }
+  });
+
+  test('--only filters to a single check', async () => {
+    const { stdout, exitCode } = await captureRun({ json: true, only: 'team-command-present' });
+    assert.equal(exitCode, 0);
+    const parsed = JSON.parse(stdout);
+    assert.equal(parsed.results.length, 1);
+    assert.equal(parsed.results[0].name, 'team-command-present');
+    assert.equal(parsed.results[0].ok, true);
+  });
+
+  test('--only on a missing file fails with exit 1', async () => {
+    rmSync(join(workDir, 'commands', 'team.md'));
+    const { exitCode } = await captureRun({ json: true, only: 'team-command-present' });
+    assert.equal(exitCode, 1);
+  });
+
+  test('--only with an unknown name exits 2', async () => {
+    const { exitCode } = await captureRun({ json: true, only: 'this-does-not-exist' });
+    assert.equal(exitCode, 2);
+  });
+});

package/cli/install.mjs

@@ -8,9 +8,10 @@ import { fileURLToPath } from 'node:url';
 // `__dirname` is a CommonJS global. ESM modules don't have it. We define it
 // at module scope so all functions in this file can use it without each
 // having to recreate the polyfill. (v4.2.3 — previously only `runInstaller`
-// had it, which caused `promptAndInstallOptional` to crash with
-// `ERR_AMBIGUOUS_MODULE_SYNTAX` when the bootstrap path fired on a fresh
-// install.)
+// had it, which caused the legacy `promptAndInstallOptional` bootstrap
+// to crash with `ERR_AMBIGUOUS_MODULE_SYNTAX`. v6.1.0 — that helper
+// has been removed; `__dirname` is still used by `runPostInstall` for
+// the same reason.)
 const __dirname = dirname(fileURLToPath(import.meta.url));
 
 import { showBanner, showPantheon, sectionHeading } from './banner.mjs';
@@ -302,39 +303,6 @@ async function isPackageInstalled(name) {
   }
 }
 
-async function promptAndInstallOptional() {
-  // v4.4.5 — Plugin and dashboard are both shipped inside this package.
-  // No separate npm install step is required for either. The plugin
-  // copy happens via installPluginFromGlobal() in runInstaller(); the
-  // dashboard is loaded directly from this package's bizar-dash/src/.
-  // We just verify they're present here and warn loudly if not.
-  const pluginPath = join(__dirname, '..', 'plugins', 'bizar');
-  if (!existsSync(pluginPath)) {
-    console.error('');
-    console.error('  ✗ Plugin source not found at plugins/bizar/.');
-    console.error('    This package is missing the plugin source. Reinstall:');
-    console.error('      npm install -g @polderlabs/bizar --force');
-  } else {
-    console.log('  ✓ Plugin source present (plugins/bizar/)');
-  }
-
-  const dashDir = join(__dirname, '..', 'bizar-dash');
-  // v6.0.2 — Dashboard shipped as a resource directory since v4.0.0.
-  // It no longer carries its own `package.json` (it became a separate
-  // `@polderlabs/bizar-dash` npm package, then was collapsed back in
-  // for v6.x). Probe for the built artifact + server source instead.
-  const dashDistHtml = join(dashDir, 'dist', 'index.html');
-  const dashServerSrc = join(dashDir, 'src', 'server', 'api.mjs');
-  if (!existsSync(dashDistHtml) && !existsSync(dashServerSrc)) {
-    console.error('');
-    console.error('  ✗ Dashboard source not found at bizar-dash/.');
-    console.error('    This package is missing the dashboard source. Reinstall:');
-    console.error('      npm install -g @polderlabs/bizar --force');
-  } else {
-    console.log('  ✓ Dashboard source present (bizar-dash/)');
-  }
-}
-
 async function promptGraphifyInstall() {
   const { spawnSync, execSync } = await import('node:child_process');
 
@@ -511,10 +479,19 @@ async function promptGraphifyInstall() {
 }
 
 export async function runPostInstall() {
-  // Skip interactive prompts in CI / non-TTY environments
-  if (!process.env.BIZAR_SKIP_OPTIONAL_INSTALLS) {
-    await promptAndInstallOptional();
-  }
+  // v6.1.0 — Bizar is Cline-only. The legacy opencode-era
+  // `promptAndInstallOptional()` (plugin/dashboard presence probes)
+  // has been removed; `cli/provision.mjs:runProvision` covers the same
+  // surface via `runProvision({ mode: 'install' })`, which is the path
+  // used by `bizar install` and `bizar update`. This `runPostInstall`
+  // is now a thin Cline-only bootstrap that:
+  //   - copies `config/cline.json` template on first install
+  //   - runs `installCommandsBizar()` for the legacy `commands-bizar/` dir
+  //   - installs agents into `~/.cline/agents/`
+  //   - probes for headroom / semble / skills-cli
+  //   - installs Headroom via pip or npm
+  // The old OpenCode-era probes (`Plugin source present`,
+  // `Dashboard source present`) moved to `cli/doctor.mjs` as live checks.
   const { mkdirSync, copyFileSync, existsSync } = await import('node:fs');
   const { execSync } = await import('node:child_process');
 

package/cli/post-install-smoke.mjs

@@ -6,8 +6,10 @@
  * Verifies the installation is functional by checking:
  *   1. Memory vault exists and is git-initialized
  *   2. `bizar doctor` exits 0
- *   3. `bizar bg list` returns (even if empty)
- *   4. lightrag-server installed
+ *   3. Dashboard HTTP responds 200
+ *   4. WebSocket connects to dashboard
+ *   5. `bizar bg list` returns (even if empty)
+ *   6. lightrag-server installed
  *
  * Each check has a 30s timeout. Exits 0 if all pass, 1 otherwise.
  *
@@ -27,6 +29,7 @@ const DEFAULT_TIMEOUT_MS = 30_000;
 const HOME = homedir();
 const DEFAULT_BIZAR_HOME = join(HOME, '.config', 'bizar');
 const DEFAULT_MEMORY_VAULT = join(HOME, '.bizar_memory');
+const DEFAULT_DASHBOARD_PORT = 4097;
 
 /**
  * @param {object} opts
@@ -37,6 +40,7 @@ const DEFAULT_MEMORY_VAULT = join(HOME, '.bizar_memory');
 export async function runSmokeTest({ bizarHome, repoPath, timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
   const bh = bizarHome || DEFAULT_BIZAR_HOME;
   const mv = process.env.BIZAR_MEMORY_VAULT || DEFAULT_MEMORY_VAULT;
+  const port = parseInt(process.env.BIZAR_DASHBOARD_PORT || String(DEFAULT_DASHBOARD_PORT), 10) || DEFAULT_DASHBOARD_PORT;
   const checks = [];
 
   function check(name, fn) {
@@ -77,7 +81,96 @@ export async function runSmokeTest({ bizarHome, repoPath, timeoutMs = DEFAULT_TI
     }
   });
 
-  // ── 3. `bizar bg list` returns ────────────────────────────────────────────
+  // ── 3. Dashboard HTTP responds 200 (with retry) ─────────────────────────────────
+  check('dashboard HTTP', async () => {
+    const maxAttempts = 5;
+    const delayMs = 2000;
+    let lastErr = null;
+    for (let attempt = 1; attempt <= maxAttempts; attempt++) {
+      try {
+        const controller = new AbortController();
+        const timer = setTimeout(() => controller.abort(), timeoutMs);
+        try {
+          const res = await fetch(`http://127.0.0.1:${port}/`, {
+            signal: controller.signal,
+            method: 'GET',
+          });
+          clearTimeout(timer);
+          if (res.ok) {
+            if (attempt > 1) {
+              return { ok: true, message: `HTTP ${res.status} on port ${port} (tried ${attempt}x)` };
+            }
+            return { ok: true, message: `HTTP ${res.status} on port ${port}` };
+          }
+          lastErr = `HTTP ${res.status} on port ${port} (expected 200)`;
+        } catch (err) {
+          clearTimeout(timer);
+          lastErr = err.name === 'AbortError'
+            ? `dashboard not responding on port ${port} (timeout)`
+            : `dashboard unreachable on port ${port}: ${err.message}`;
+        }
+      } catch (err) {
+        lastErr = `fetch not available: ${err.message}`;
+      }
+      if (attempt < maxAttempts) {
+        await new Promise((r) => setTimeout(r, delayMs));
+      }
+    }
+    return { ok: false, message: lastErr || 'dashboard HTTP failed' };
+  });
+
+  // ── 4. WebSocket connects (with retry) ────────────────────────────────────────
+  check('dashboard WebSocket', async () => {
+    const maxAttempts = 5;
+    const delayMs = 2000;
+    let lastErr = null;
+    for (let attempt = 1; attempt <= maxAttempts; attempt++) {
+      try {
+        const { WebSocket } = await import('ws' in globalThis
+          ? { ws: globalThis.ws, WebSocket: globalThis.WebSocket }
+          : await import(`ws`).then(m => ({ ws: m, WebSocket: m.WebSocket || m.default }))
+        );
+        const url = `ws://127.0.0.1:${port}/ws`;
+        const controller = new AbortController();
+        const timer = setTimeout(() => controller.abort(), timeoutMs);
+        try {
+          const ws = new WebSocket(url);
+          await new Promise((resolve, reject) => {
+            ws.on('open', resolve);
+            ws.on('error', reject);
+            ws.on('close', () => reject(new Error('closed before open')));
+            controller.signal.addEventListener('abort', () => {
+              ws.close();
+              reject(new Error('timeout'));
+            });
+          });
+          clearTimeout(timer);
+          ws.close();
+          if (attempt > 1) {
+            return { ok: true, message: `WebSocket connected at ${url} (tried ${attempt}x)` };
+          }
+          return { ok: true, message: `WebSocket connected at ${url}` };
+        } catch (err) {
+          clearTimeout(timer);
+          lastErr = err.message === 'timeout'
+            ? `WebSocket timeout on ${url}`
+            : `WebSocket failed on ${url}: ${err.message}`;
+        }
+      } catch (err) {
+        // ws module not available in this environment — skip
+        if (err.code === 'MODULE_NOT_FOUND' || err.message?.includes('ws')) {
+          return { ok: true, message: 'ws module not available — skipping WebSocket check' };
+        }
+        lastErr = `WebSocket check failed: ${err.message}`;
+      }
+      if (attempt < maxAttempts) {
+        await new Promise((r) => setTimeout(r, delayMs));
+      }
+    }
+    return { ok: false, message: lastErr || 'WebSocket failed' };
+  });
+
+  // ── 5. `bizar bg list` returns ────────────────────────────────────────────
   check('background agents', () => {
     try {
       execSync('bizar bg list', {

package/cli/post-install-smoke.test.mjs

@@ -5,11 +5,9 @@
  *
  * Tests:
  *   1. lightrag-server check: file existence is verified
- *   2. lightrag: warns when not found in PATH or ~/.local/bin/
- *
- * Note: dashboard HTTP/WS smoke checks were removed in v6.0.0-beta.2
- * (the dashboard is optional and the checks were failing noisily on
- * systems that don't run it). See PROGRESS.md.
+ *   2. HTTP retry: retries on failure, succeeds when server eventually responds
+ *   3. WS retry: retries on failure, succeeds when server eventually responds
+ *   4. lightrag: warns when not found in PATH or ~/.local/bin/
  */
 import { test, describe, beforeEach, afterEach } from 'node:test';
 import assert from 'node:assert/strict';

package/cli/provision.mjs

@@ -860,8 +860,46 @@ export async function patchClineJson({ dryRun, force }) {
     (p) => Array.isArray(p) && typeof p[0] === 'string' && p[0].includes('plugins/bizar'),
   );
 
-  // Auto-add provider.minimax block if missing (v5.x — must happen
-  // even when the plugin entry already exists, so always evaluate).
+  // v6.2.0 — Multi-patch installer. On every install/update, ensure the
+  // full Bizar config surface is present in cline.json:
+  //
+  //   1. provider.9router (preferred gateway since v6.0.1) — add if
+  //      missing so the install is "complete" out of the box.
+  //   2. provider.minimax (legacy fallback) — kept for back-compat.
+  //   3. default_agent — set to "odin" if missing.
+  //   4. $schema — add if missing (helps editors validate).
+  //   5. instructions — point at .cline/instructions/bizar-tools.md if
+  //      missing, so Cline loads the tool reference on every session.
+  //   6. plugin entry — add if missing (the critical one; without this
+  //      the Bizar plugin never loads).
+  //   7. permissions — set to "allow" if missing.
+  //   8. snapshot — set to false if missing.
+  //
+  // All patches are additive and idempotent. We never overwrite a value
+  // the user has set; the `force` flag bypasses that protection for the
+  // plugin entry only.
+  let addedProvider = false;
+  let addedDefaultAgent = false;
+  let addedSchema = false;
+  let addedInstructions = false;
+  let addedPermissions = false;
+  let addedSnapshot = false;
+  let added9router = false;
+
+  if (!cfg.provider) cfg.provider = {};
+
+  // 1. provider.9router — preferred gateway.
+  if (!cfg.provider['9router'] && existsSync(join(REPO_ROOT, 'config', 'cline.json'))) {
+    try {
+      const tpl = JSON.parse(readFileSync(join(REPO_ROOT, 'config', 'cline.json'), 'utf8'));
+      if (tpl.provider && tpl.provider['9router']) {
+        cfg.provider['9router'] = tpl.provider['9router'];
+        added9router = true;
+      }
+    } catch { /* ignore template parse errors */ }
+  }
+
+  // 2. provider.minimax — legacy fallback.
   const DEFAULT_MINIMAX_BLOCK = {
     options: {
       baseURL: 'https://api.minimax.io/v1',
@@ -874,21 +912,59 @@ export async function patchClineJson({ dryRun, force }) {
       'MiniMax-M3-Reasoning': { name: 'MiniMax M3 Reasoning', interleaved: { field: 'reasoning_details' }, reasoning: true },
     },
   };
-  let addedProvider = false;
-  if (!cfg.provider) {
-    cfg.provider = {};
-  }
   if (!cfg.provider.minimax) {
     cfg.provider.minimax = DEFAULT_MINIMAX_BLOCK;
     addedProvider = true;
   }
 
-  if (hasEntry && !force && !addedProvider) {
+  // 3. default_agent
+  if (!cfg.default_agent) {
+    cfg.default_agent = 'odin';
+    addedDefaultAgent = true;
+  }
+
+  // 4. $schema
+  if (!cfg.$schema) {
+    cfg.$schema = 'https://docs.cline.bot/config.json';
+    addedSchema = true;
+  }
+
+  // 5. instructions — point at the bundled Bizar tools reference.
+  if (!cfg.instructions || (Array.isArray(cfg.instructions) && cfg.instructions.length === 0)) {
+    cfg.instructions = ['.cline/instructions/bizar-tools.md'];
+    addedInstructions = true;
+  }
+
+  // 6. permission
+  if (!cfg.permission) {
+    cfg.permission = 'allow';
+    addedPermissions = true;
+  }
+
+  // 7. snapshot
+  if (typeof cfg.snapshot !== 'boolean') {
+    cfg.snapshot = false;
+    addedSnapshot = true;
+  }
+
+  const anyAdded = addedProvider || added9router || addedDefaultAgent || addedSchema
+    || addedInstructions || addedPermissions || addedSnapshot;
+
+  if (hasEntry && !force && !anyAdded) {
     return { ok: true, message: 'cline.json already has Bizar plugin entry' };
   }
 
   if (dryRun) {
-    return { ok: true, message: `[dry-run] would patch cline.json with plugin entry${addedProvider ? ' + provider.minimax' : ''}` };
+    const added = [];
+    if (!hasEntry) added.push('plugin entry');
+    if (added9router) added.push('provider.9router');
+    if (addedProvider) added.push('provider.minimax');
+    if (addedDefaultAgent) added.push('default_agent');
+    if (addedSchema) added.push('$schema');
+    if (addedInstructions) added.push('instructions');
+    if (addedPermissions) added.push('permission');
+    if (addedSnapshot) added.push('snapshot');
+    return { ok: true, message: `[dry-run] would patch cline.json with: ${added.join(', ')}` };
   }
 
   if (!hasEntry) {
@@ -904,16 +980,26 @@ export async function patchClineJson({ dryRun, force }) {
       loopThresholdEscalate: 8,
       loopThresholdBlock: 12,
       loopWindowSize: 10,
+      clineruntimeMaxConsecutiveMistakes: 6,
     }]);
     cfg.plugin = plugins;
   }
 
   writeFileSync(cfgPath, JSON.stringify(cfg, null, 2) + '\n');
+  const added = [];
+  if (!hasEntry) added.push('plugin entry');
+  if (added9router) added.push('provider.9router');
+  if (addedProvider) added.push('provider.minimax');
+  if (addedDefaultAgent) added.push('default_agent');
+  if (addedSchema) added.push('$schema');
+  if (addedInstructions) added.push('instructions');
+  if (addedPermissions) added.push('permission');
+  if (addedSnapshot) added.push('snapshot');
   return {
     ok: true,
-    message: addedProvider
-      ? 'cline.json patched with provider.minimax (plugin entry was already present)'
-      : 'cline.json patched with Bizar plugin entry + provider.minimax',
+    message: added.length > 0
+      ? `cline.json patched: ${added.join(', ')}`
+      : 'cline.json already complete',
   };
 }