@polderlabs/bizar 10.7.2 → 10.11.0

package/cli/install/index.test.mjs

@@ -1,61 +0,0 @@
-/**
- * cli/install/index.test.mjs
- *
- * Tests for cli/install/index.mjs — runInstaller orchestrator.
- */
-import { test, describe, beforeEach, afterEach } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, rmSync, existsSync, writeFileSync, mkdirSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-
-const ORIG_HOME = process.env.HOME;
-
-function freshHome() {
-  const home = mkdtempSync(join(tmpdir(), 'bizar-install-index-'));
-  process.env.HOME = home;
-  delete process.env.CLAUDE_CONFIG_DIR;
-  delete process.env.XDG_CONFIG_HOME;
-  return home;
-}
-
-function restoreHome() {
-  if (ORIG_HOME === undefined) delete process.env.HOME;
-  else process.env.HOME = ORIG_HOME;
-}
-
-describe('runInstaller()', () => {
-  let home;
-
-  beforeEach(() => { home = freshHome(); });
-  afterEach(() => {
-    restoreHome();
-    if (home) try { rmSync(home, { recursive: true, force: true }); } catch { /* ignore */ }
-  });
-
-  test('runInstaller({ dryRun: true }) does not write files', async () => {
-    const { runInstaller } = await import('./index.mjs');
-    const result = await runInstaller({ dryRun: true });
-    assert.equal(result.ok, true);
-  });
-
-  test('runInstaller({ quiet: true }) only prints location card', async () => {
-    const { runInstaller } = await import('./index.mjs');
-    const result = await runInstaller({ quiet: true });
-    assert.equal(result.ok, true);
-  });
-
-  test('runInstaller({ force: true }) runs without error', async () => {
-    const { runInstaller } = await import('./index.mjs');
-    const result = await runInstaller({ force: true, dryRun: true });
-    assert.equal(result.ok, true);
-  });
-
-  test('runInstaller({ mode: "update" }) runs without error', async () => {
-    const { runInstaller } = await import('./index.mjs');
-    const result = await runInstaller({ mode: 'update', dryRun: true });
-    assert.equal(result.ok, true);
-  });
-});
-
-console.log('  index.test.mjs loaded — run with: node --test cli/install/index.test.mjs');

package/cli/install/paths.test.mjs

@@ -1,77 +0,0 @@
-/**
- * cli/install/paths.test.mjs
- *
- * Tests for cli/install/paths.mjs
- */
-import { test, describe } from 'node:test';
-import assert from 'node:assert/strict';
-import { PATHS, printInstallLocations, resolveClaudeDir } from './paths.mjs';
-
-describe('PATHS', () => {
-  test('has all required keys', () => {
-    const required = [
-      'claudeDir', 'agentsDir', 'skillsDir', 'commandsDir',
-      'hooksDir', 'pluginsDir', 'settingsFile', 'bizarHome',
-      'servicePid', 'dashboardPid', 'dashboardPort', 'installMarker',
-    ];
-    for (const key of required) {
-      assert.ok(key in PATHS, `missing key: ${key}`);
-    }
-  });
-
-  test('PATHS values are non-empty strings', () => {
-    for (const [key, val] of Object.entries(PATHS)) {
-      assert.ok(typeof val === 'string' && val.length > 0, `${key} should be non-empty string, got: ${val}`);
-    }
-  });
-});
-
-describe('printInstallLocations()', () => {
-  test('does not throw', () => {
-    assert.doesNotThrow(() => printInstallLocations({ dryRun: false }));
-  });
-
-  test('output contains ~/.claude', () => {
-    let output = '';
-    const orig = console.log;
-    console.log = (...args) => { output += args.join(' ') + '\n'; };
-    try { printInstallLocations({}); } finally { console.log = orig; }
-    // Output mentions the Claude config dir — exact form depends on resolved HOME
-    assert.ok(output.includes('.claude') || output.includes('/.claude'), 'should mention .claude path');
-  });
-});
-
-describe('resolveClaudeDir()', () => {
-  test('uses CLAUDE_CONFIG_DIR env override', () => {
-    const orig = process.env.CLAUDE_CONFIG_DIR;
-    process.env.CLAUDE_CONFIG_DIR = '/tmp/my-claude-dir';
-    try {
-      const dir = resolveClaudeDir();
-      assert.equal(dir, '/tmp/my-claude-dir');
-    } finally {
-      if (orig === undefined) delete process.env.CLAUDE_CONFIG_DIR;
-      else process.env.CLAUDE_CONFIG_DIR = orig;
-    }
-  });
-});
-
-describe('idempotency', () => {
-  test('two consecutive calls produce same output', () => {
-    let out1 = '', out2 = '';
-    const orig = console.log;
-    const capture = () => { let s = ''; console.log = (...a) => { s += a.join(' ') + '\n'; }; return s; };
-    const restore = (s) => { console.log = orig; return s; };
-
-    const c1 = capture();
-    printInstallLocations({});
-    out1 = restore(c1);
-
-    const c2 = capture();
-    printInstallLocations({});
-    out2 = restore(c2);
-
-    assert.equal(out1, out2, 'consecutive calls should produce identical output');
-  });
-});
-
-console.log('  paths.test.mjs loaded — run with: node --test cli/install/paths.test.mjs');

package/cli/install/plugin.mjs

@@ -1,161 +0,0 @@
-/**
- * cli/install/plugin.mjs
- *
- * installPluginFromGlobal() — copied from cli/install.mjs (original ~220 lines).
- * Installs the Bizar plugin from the global npm package into ~/.claude/plugins/bizar/.
- */
-
-import chalk from 'chalk';
-import { existsSync, lstatSync, rmSync } from 'node:fs';
-import { join } from 'node:path';
-import { fileURLToPath } from 'node:url';
-import { resolveClaudeDir } from './paths.mjs';
-
-const __dirname = fileURLToPath(new URL('.', import.meta.url));
-
-/**
- * Install the Bizar MCP server from this package's own `plugins/bizar/`
- * directory into `~/.claude/plugins/bizar/`.
- *
- * Symlink guard (v3.12.2): if the dest is a symlink (e.g. created by
- * `bizar dev-link`), do NOT overwrite it. Pass `{ force: true }` to override.
- *
- * node_modules copy: the plugin imports `@polderlabs/bizar-sdk`. After copying
- * the plugin files, this function also copies the source `node_modules/` to the
- * deployed `node_modules/` so Bun can resolve the import.
- *
- * Returns `true` if the plugin was installed (or already in place),
- * `false` otherwise. Never throws.
- */
-export async function installPluginFromGlobal(opts = {}) {
-  const { mkdir, readdir, copyFile } = await import('node:fs/promises');
-
-  const destDir = join(resolveClaudeDir(), 'plugins', 'bizar');
-  let destIsSymlink = false;
-  try {
-    destIsSymlink = lstatSync(destDir).isSymbolicLink();
-  } catch {
-    // doesn't exist yet — fine
-  }
-
-  if (destIsSymlink) {
-    if (opts.silent) return true;
-    console.log(chalk.yellow('  ⚠ ~/.claude/plugins/bizar is a dev symlink — skipping copy.'));
-    console.log(chalk.dim('    Run `bizar dev-unlink` to restore the deployed copy, or pass --force to overwrite.'));
-    if (!opts.force) return false;
-    rmSync(destDir, { force: true });
-  }
-
-  let pluginPath;
-  if (opts.sourceDir) {
-    pluginPath = opts.sourceDir;
-  } else {
-    pluginPath = join(__dirname, '..', '..', 'plugins', 'bizar');
-  }
-  if (!existsSync(pluginPath)) {
-    console.log(chalk.red(`  ✗ Plugin source not found at ${pluginPath}`));
-    console.log(chalk.dim('    This package appears to be missing plugins/bizar/. Reinstall:'));
-    console.log(chalk.dim('      npm install -g @polderlabs/bizar --force'));
-    return false;
-  }
-
-  const distEntry = join(pluginPath, 'dist', 'index.js');
-  if (!existsSync(distEntry)) {
-    try {
-      const { buildPlugin } = await import('../provision.mjs');
-      const buildRes = await buildPlugin();
-      if (buildRes.ok && !buildRes.skipped) {
-        console.log(chalk.dim(`    auto-built plugin (${buildRes.message})`));
-      } else if (!buildRes.ok) {
-        console.log(chalk.yellow(`  ⚠ Plugin auto-build failed: ${buildRes.message}`));
-        console.log(chalk.dim('    Run `bizar install` from a checkout, or:'));
-        console.log(chalk.dim('      npx tsc -p plugins/bizar/tsconfig.json'));
-        return false;
-      }
-    } catch (err) {
-      console.log(chalk.yellow(`  ⚠ Plugin auto-build threw: ${err.message}`));
-      return false;
-    }
-  }
-
-  await mkdir(destDir, { recursive: true });
-
-  async function copyRecursive(srcDir, dstDir) {
-    const entries = await readdir(srcDir, { withFileTypes: true });
-    for (const entry of entries) {
-      const src = join(srcDir, entry.name);
-      const dst = join(dstDir, entry.name);
-      if (entry.isDirectory()) {
-        if (entry.name === 'node_modules' || entry.name === 'dist') continue;
-        await mkdir(dst, { recursive: true });
-        await copyRecursive(src, dst);
-      } else {
-        if (entry.name === '.DS_Store' || entry.name.endsWith('.log')) continue;
-        await copyFile(src, dst);
-      }
-    }
-  }
-
-  try {
-    await copyRecursive(pluginPath, destDir);
-    console.log(chalk.green(`  ✓ Bizar plugin installed from global package`));
-    console.log(chalk.dim(`    ${pluginPath} → ${destDir}`));
-  } catch (err) {
-    console.log(chalk.yellow(`  ⚠ Failed to copy Bizar plugin: ${err.message}`));
-    return false;
-  }
-
-  // node_modules copy — the plugin imports `@polderlabs/bizar-sdk`
-  try {
-    const srcNmDir = join(pluginPath, 'node_modules');
-    if (existsSync(srcNmDir)) {
-      const dstNmDir = join(destDir, 'node_modules');
-      let dstNmIsSymlink = false;
-      try {
-        dstNmIsSymlink = lstatSync(dstNmDir).isSymbolicLink();
-      } catch { /* doesn't exist — fine */ }
-      if (dstNmIsSymlink) {
-        console.log(chalk.yellow(`  ⚠ ${dstNmDir} is a symlink — refusing to copy node_modules (remove manually).`));
-      } else {
-        const srcEntries = await readdir(srcNmDir, { withFileTypes: true });
-        await mkdir(dstNmDir, { recursive: true });
-        async function copyNodeModules(srcDir, dstDir) {
-          const entries = await readdir(srcDir, { withFileTypes: true });
-          for (const entry of entries) {
-            const src = join(srcDir, entry.name);
-            const dst = join(dstDir, entry.name);
-            if (entry.isDirectory()) {
-              await mkdir(dst, { recursive: true });
-              await copyNodeModules(src, dst);
-            } else {
-              await copyFile(src, dst);
-            }
-          }
-        }
-        await copyNodeModules(srcNmDir, dstNmDir);
-        console.log(chalk.green(`  ✓ copied node_modules (${srcEntries.length} packages) to deployed plugin`));
-      }
-    }
-
-    // Wire runtime deps (zod, @anthropic-ai/claude-agent-sdk)
-    let pluginMain = null;
-    try {
-      const pkgPath = join(destDir, 'package.json');
-      const pkg = JSON.parse(await import('node:fs/promises').then(fs => fs.readFile(pkgPath, 'utf8')));
-      pluginMain = pkg.main;
-    } catch { /* missing or unreadable */ }
-    const needsWiring = typeof pluginMain === 'string' && pluginMain.endsWith('.ts');
-    if (needsWiring) {
-      try {
-        const { wirePluginRuntimeDeps } = await import('../plugin-runtime-deps.mjs');
-        await wirePluginRuntimeDeps(destDir);
-      } catch (err) {
-        console.log(chalk.dim(`  ℹ Could not auto-wire runtime deps: ${err.message} (run \`bizar doctor\` to diagnose)`));
-      }
-    }
-  } catch (err) {
-    console.log(chalk.yellow(`  ⚠ Failed to copy node_modules: ${err.message}`));
-  }
-
-  return true;
-}

package/cli/install/postinstall.mjs

@@ -1,137 +0,0 @@
-/**
- * cli/install/postinstall.mjs
- *
- * runPostInstall() — bootstrap after npm install. Sets up the Claude Code
- * config dir with the canonical Bizar agents + a starter settings.json, and
- * installs Semble / Skills CLI as sidecars.
- *
- * Source paths resolve via `import.meta.url` so they work both when the
- * package is extracted to `node_modules/@polderlabs/bizar/` (npm install)
- * and when invoked from a local checkout (`node cli/bin.mjs install`).
- *
- * v10.7.2 fix: source paths previously used `PATHS.bizarHome + '/../config/'`
- * which resolved to a nonexistent `~/.config/config/` (one level up from
- * `BIZAR_HOME`, which is a runtime state dir, not the package source). After
- * F-107 deleted the legacy `config/agents/` tree, the install couldn't find
- * any agent files and crashed with ENOENT.
- */
-
-import chalk from 'chalk';
-import { existsSync, mkdirSync, copyFileSync, readdirSync } from 'node:fs';
-import { execSync } from 'node:child_process';
-import { join, dirname } from 'node:path';
-import { fileURLToPath } from 'node:url';
-import { resolveClaudeDir, PATHS } from './paths.mjs';
-
-/** Package root — `node_modules/@polderlabs/bizar/` or the checkout root. */
-const PACKAGE_ROOT = join(dirname(fileURLToPath(import.meta.url)), '..', '..');
-
-/** Source paths, all relative to the package root. */
-const SRC_AGENTS_DIR    = join(PACKAGE_ROOT, '.claude', 'agents');
-const SRC_SETTINGS_FILE = join(PACKAGE_ROOT, '.claude', 'settings.json');
-
-/**
- * Discover every agent .md shipped in `.claude/agents/`. Replaces the prior
- * hardcoded AGENT_FILES list (which omitted kevin and brad and broke
- * whenever a new agent was added).
- */
-function listShippedAgents() {
-  if (!existsSync(SRC_AGENTS_DIR)) return [];
-  return readdirSync(SRC_AGENTS_DIR).filter(f => f.endsWith('.md'));
-}
-
-/**
- * Post-install bootstrap: settings.json template, agents, Semble, Skills CLI.
- * Called by the npm postinstall hook.
- */
-export async function runPostInstall() {
-  // 1. settings.json — copy template if user has none yet.
-  const destSettings = join(resolveClaudeDir(), 'settings.json');
-  if (!existsSync(destSettings)) {
-    if (existsSync(SRC_SETTINGS_FILE)) {
-      mkdirSync(resolveClaudeDir(), { recursive: true });
-      copyFileSync(SRC_SETTINGS_FILE, destSettings);
-      console.log('  ✓ settings.json bootstrapped from package template');
-    } else {
-      console.warn('  ⚠ settings.json template not found at', SRC_SETTINGS_FILE);
-    }
-  }
-
-  // 2. Make sure Claude config dir exists.
-  if (!existsSync(resolveClaudeDir())) {
-    mkdirSync(resolveClaudeDir(), { recursive: true });
-    console.log(`BizarHarness: created ${resolveClaudeDir()}/`);
-  }
-
-  // 3. Copy agent files. Skip ones the user already has (don't clobber).
-  mkdirSync(PATHS.agentsDir, { recursive: true });
-  const agents = listShippedAgents();
-  if (agents.length === 0) {
-    console.warn(`BizarHarness: no agent files found at ${SRC_AGENTS_DIR} — package is broken.`);
-  }
-  for (const file of agents) {
-    const src = join(SRC_AGENTS_DIR, file);
-    const dst = join(PATHS.agentsDir, file);
-    if (!existsSync(dst)) {
-      copyFileSync(src, dst);
-    }
-  }
-  console.log(`BizarHarness: ${agents.length} agent(s) installed.`);
-
-  // 4. Install Semble (code search MCP).
-  const seemeCmd = (cmd) => {
-    try {
-      execSync(`command -v ${cmd}`, { stdio: ['ignore', 'pipe', 'ignore'] });
-      return true;
-    } catch { return false; }
-  };
-  const semblePresent = seemeCmd('semble');
-  if (!semblePresent) {
-    console.log('BizarHarness: installing Semble (code search)...');
-    try {
-      const hasUv = seemeCmd('uv');
-      if (!hasUv) {
-        if (process.platform === 'win32') {
-          execSync('powershell -ExecutionPolicy ByPass -c "irm https://astral.sh/uv/install.ps1 | iex"', { stdio: 'pipe', timeout: 60000 });
-        } else {
-          execSync('curl -LsSf https://astral.sh/uv/install.sh | sh', { stdio: 'pipe', timeout: 60000 });
-        }
-      }
-      execSync('uv tool install "semble[mcp]"', { stdio: 'pipe', timeout: 60000 });
-      console.log('BizarHarness: Semble installed.');
-    } catch {
-      console.log('BizarHarness: Semble install failed. Run `uv tool install "semble[mcp]"` manually.');
-    }
-  } else {
-    console.log('BizarHarness: Semble ready.');
-  }
-
-  // 5. Install Skills CLI.
-  const skillsPresent = seemeCmd('skills');
-  if (!skillsPresent) {
-    console.log('BizarHarness: installing Skills CLI...');
-    try {
-      execSync('npm install -g skills', { stdio: 'pipe', timeout: 30000 });
-      console.log('BizarHarness: Skills CLI installed.');
-    } catch {
-      console.log('BizarHarness: Skills CLI install failed. Run `npm install -g skills` manually.');
-    }
-  } else {
-    console.log('BizarHarness: Skills CLI ready.');
-  }
-
-  // 6. Install core skill pack.
-  console.log('BizarHarness: installing core skills...');
-  try {
-    execSync('skills add vercel-labs/skills --all -y', { stdio: 'pipe', timeout: 60000 });
-    console.log('BizarHarness: core skills installed.');
-  } catch {
-    console.log('BizarHarness: core skill install skipped — run `skills add vercel-labs/skills --all -y` manually.');
-  }
-
-  // 7. Install Bizar plugin.
-  const { installPluginFromGlobal } = await import('./plugin.mjs');
-  await installPluginFromGlobal();
-
-  console.log('Run `bizar` for interactive setup.');
-}

package/cli/install/postinstall.test.mjs

@@ -1,98 +0,0 @@
-/**
- * cli/install/postinstall.test.mjs
- *
- * Verifies the postinstall bootstrap resolves source paths via the package
- * root, not via BIZAR_HOME (regression test for the v10.7.0 ENOENT crash:
- * `~/.config/config/agents/mike.md → ~/.claude/agents/mike.md`).
- */
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, mkdirSync, writeFileSync, rmSync, existsSync, readdirSync, copyFileSync, readFileSync } from 'node:fs';
-import { join, dirname, resolve } from 'node:path';
-import { tmpdir } from 'node:os';
-import { fileURLToPath } from 'node:url';
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const REPO_ROOT = resolve(__dirname, '..', '..');
-
-// Mirror the postinstall's PACKAGE_ROOT computation.
-const PACKAGE_ROOT_FROM_HERE = join(__dirname, '..', '..');
-const SRC_AGENTS_DIR = join(PACKAGE_ROOT_FROM_HERE, '.claude', 'agents');
-const SRC_SETTINGS_FILE = join(PACKAGE_ROOT_FROM_HERE, '.claude', 'settings.json');
-
-test('PACKAGE_ROOT resolves to the repo root (not BIZAR_HOME)', () => {
-  // The bug: source paths used PATHS.bizarHome + '/../config/agents/' which
-  // climbed one level up from BIZAR_HOME — a runtime state dir. The fix
-  // resolves via import.meta.url to the actual package root.
-  assert.ok(SRC_AGENTS_DIR.endsWith('/.claude/agents'), `got ${SRC_AGENTS_DIR}`);
-  assert.ok(SRC_SETTINGS_FILE.endsWith('/.claude/settings.json'), `got ${SRC_SETTINGS_FILE}`);
-});
-
-test('.claude/agents/ directory exists in the repo (canonical agent source)', () => {
-  assert.ok(existsSync(SRC_AGENTS_DIR), `${SRC_AGENTS_DIR} must exist`);
-  const files = readdirSync(SRC_AGENTS_DIR).filter(f => f.endsWith('.md'));
-  assert.ok(files.length >= 14, `expected >=14 agents, got ${files.length}`);
-});
-
-test('every shipped agent file resolves to a real file under .claude/agents/', () => {
-  const files = readdirSync(SRC_AGENTS_DIR).filter(f => f.endsWith('.md'));
-  for (const file of files) {
-    assert.ok(existsSync(join(SRC_AGENTS_DIR, file)), `${file} must exist on disk`);
-  }
-});
-
-test('.claude/settings.json exists in the repo (settings template)', () => {
-  assert.ok(existsSync(SRC_SETTINGS_FILE), `${SRC_SETTINGS_FILE} must exist`);
-});
-
-test('copy semantics: existing files in destination are not overwritten', () => {
-  // Build a fake source tree + destination tree.
-  const work = mkdtempSync(join(tmpdir(), 'bizar-postinstall-'));
-  try {
-    const srcDir = join(work, 'src');
-    const dstDir = join(work, 'dst');
-    mkdirSync(srcDir, { recursive: true });
-    mkdirSync(dstDir, { recursive: true });
-    writeFileSync(join(srcDir, 'a.md'), '# shipped');
-    writeFileSync(join(dstDir, 'a.md'), '# user-edited');
-
-    // Mirror postinstall's loop: skip when dst exists.
-    const files = ['a.md'];
-    for (const f of files) {
-      const dst = join(dstDir, f);
-      if (!existsSync(dst)) {
-        copyFileSync(join(srcDir, f), dst);
-      }
-    }
-    const content = readFileSync(join(dstDir, 'a.md'), 'utf8');
-    assert.match(content, /user-edited/, 'existing user file must be preserved');
-  } finally {
-    rmSync(work, { recursive: true, force: true });
-  }
-});
-
-test('copy semantics: missing files are copied from source', () => {
-  const work = mkdtempSync(join(tmpdir(), 'bizar-postinstall-'));
-  try {
-    const srcDir = join(work, 'src');
-    const dstDir = join(work, 'dst');
-    mkdirSync(srcDir, { recursive: true });
-    mkdirSync(dstDir, { recursive: true });
-    writeFileSync(join(srcDir, 'b.md'), '# shipped');
-
-    const files = ['b.md'];
-    for (const f of files) {
-      const dst = join(dstDir, f);
-      if (!existsSync(dst)) copyFileSync(join(srcDir, f), dst);
-    }
-    assert.match(readFileSync(join(dstDir, 'b.md'), 'utf8'), /shipped/);
-  } finally {
-    rmSync(work, { recursive: true, force: true });
-  }
-});
-
-test('regression: source path does NOT include "config/agents/" (legacy deleted in F-107)', () => {
-  // F-107 deleted config/agents/. Postinstall must not reference it.
-  assert.ok(!SRC_AGENTS_DIR.includes('/config/agents/'),
-    `BUG: SRC_AGENTS_DIR must not contain /config/agents/ (legacy tree), got ${SRC_AGENTS_DIR}`);
-});