@polderlabs/bizar 5.6.0-beta.12 → 5.6.0-beta.14

package/cli/provision.mjs

@@ -259,11 +259,40 @@ export function writeInstallMarker({ version, repoPath, serviceUnit }) {
  */
 export function detectState({ cwd = process.cwd() } = {}) {
   // ── npm-global package location ─────────────────────────────────────
+  // Try multiple strategies because `npm root -g` doesn't always
+  // respect NPM_CONFIG_PREFIX (notably when npm is installed by the
+  // system package manager on Debian/Ubuntu).
   let globalRoot = null;
+  const npmRootCandidates = [];
+
+  // 1. Honour the user's explicit env override first.
+  if (process.env.NPM_CONFIG_PREFIX) {
+    npmRootCandidates.push(join(process.env.NPM_CONFIG_PREFIX, 'lib', 'node_modules'));
+  }
+  // 2. Honour NPM_CONFIG_GLOBAL_PREFIX (set by `npm -g`).
+  if (process.env.NPM_CONFIG_GLOBAL_PREFIX) {
+    npmRootCandidates.push(join(process.env.NPM_CONFIG_GLOBAL_PREFIX, 'lib', 'node_modules'));
+  }
+  // 3. The classic `npm root -g` query (works on most distros).
   try {
-    globalRoot = execSync('npm root -g', { stdio: ['ignore', 'pipe', 'ignore'] })
+    const r = execSync('npm root -g', { stdio: ['ignore', 'pipe', 'ignore'] })
       .toString().trim();
+    if (r) npmRootCandidates.push(r);
   } catch { /* ignore */ }
+  // 4. Common distro defaults.
+  npmRootCandidates.push('/usr/local/lib/node_modules');
+  npmRootCandidates.push('/usr/lib/node_modules');
+
+  // Pick the first candidate that actually has @polderlabs/bizar.
+  for (const candidate of npmRootCandidates) {
+    if (existsSync(join(candidate, '@polderlabs', 'bizar'))) {
+      globalRoot = candidate;
+      break;
+    }
+  }
+  // Fall back to the first candidate (even if it doesn't have the
+  // package) so downstream code can produce a clean error message.
+  if (!globalRoot) globalRoot = npmRootCandidates[0] ?? null;
 
   const pkgRoot = globalRoot ? join(globalRoot, '@polderlabs', 'bizar') : null;
   const pkgVersion = globalRoot ? currentVersion(PKG_MAIN) : null;
@@ -515,7 +544,11 @@ export async function ensureNpmPackage(pkg, { mode, dryRun, force }) {
   // processes are reading files inside the npm-global install dir. We
   // can't replace those files atomically while they're open. The caller
   // is expected to have already killed them via ensureInstancesKilled().
-  const r = spawnSync('npm', ['install', '-g', `${pkg}@latest`], { stdio: 'inherit', timeout: 600000 });
+  const npmArgs = ['install', '-g', `${pkg}@latest`];
+  if (process.env.NPM_CONFIG_PREFIX) {
+    npmArgs.push('--prefix', process.env.NPM_CONFIG_PREFIX);
+  }
+  const r = spawnSync('npm', npmArgs, { stdio: 'inherit', timeout: 600000 });
   if (r.status === null && r.error?.code === 'ETIMEDOUT') {
     return { ok: false, message: `${pkg} install timed out after 10 minutes`, installed: current };
   }
@@ -543,7 +576,11 @@ export async function updateClineCli({ dryRun, force }) {
     return { ok: true, message: 'cline updated via `cline upgrade`' };
   }
   console.log(chalk.dim('  cline upgrade not available; falling back to npm'));
-  const r2 = spawnSync('npm', ['install', '-g', 'cline@latest'], { stdio: 'inherit', timeout: 600000 });
+  const npm2Args = ['install', '-g', 'cline@latest'];
+  if (process.env.NPM_CONFIG_PREFIX) {
+    npm2Args.push('--prefix', process.env.NPM_CONFIG_PREFIX);
+  }
+  const r2 = spawnSync('npm', npm2Args, { stdio: 'inherit', timeout: 600000 });
   if (r2.status === null && r2.error?.code === 'ETIMEDOUT') {
     return { ok: false, message: 'cline install timed out after 10 minutes' };
   }
@@ -583,6 +620,40 @@ export async function copyPluginToCline({ dryRun, force }) {
     };
   }
 
+  // v5.6.0-beta.14: require a package.json with a `cline` field before
+  // copying. Per https://docs.cline.bot/customization/plugins — without
+  // the `cline.plugins` manifest, Cline falls back to recursive
+  // auto-discovery and tries to load every .ts file in the plugin tree
+  // (including `src/tools/*.ts`, `tests/*.test.ts`) as a plugin module,
+  // which produces ~100 `Invalid plugin module` errors at startup.
+  // Refuse to copy a plugin that isn't actually a Cline plugin.
+  const pkgJsonPath = join(src, 'package.json');
+  if (!existsSync(pkgJsonPath)) {
+    return {
+      ok: false,
+      message:
+        `plugin source missing package.json at ${pkgJsonPath} — ` +
+        `rebuild @polderlabs/bizar (the plugin must ship a cline field)`,
+    };
+  }
+  try {
+    const pkg = JSON.parse(readFileSync(pkgJsonPath, 'utf8'));
+    if (!pkg.cline?.plugins?.length) {
+      return {
+        ok: false,
+        message:
+          `plugin package.json at ${pkgJsonPath} is missing the ` +
+          'cline.plugins manifest required by Cline — see ' +
+          'https://docs.cline.bot/customization/plugins',
+      };
+    }
+  } catch (err) {
+    return {
+      ok: false,
+      message: `plugin package.json at ${pkgJsonPath} is invalid JSON: ${err.message}`,
+    };
+  }
+
   if (state.plugin.symlink && !force) {
     return {
       ok: true,
@@ -626,7 +697,19 @@ export async function copyPluginToCline({ dryRun, force }) {
     // Filter against RELATIVE path segments, not absolute substrings. This
     // is the v5.6.0-beta.12 fix: the old filter excluded the src dir itself
     // because npm-global paths contain `node_modules`.
-    const skipDirs = new Set(['node_modules', 'dist', '.DS_Store']);
+    //
+    // v5.6.0-beta.14: also skip `tests/` and `scripts/` (test fixtures &
+    // helper scripts aren't part of the runtime plugin) and `coverage/`
+    // (jest artifacts). The runtime only needs `index.ts`, `src/`,
+    // `package.json`, and the static docs.
+    const skipDirs = new Set([
+      'node_modules',
+      'dist',
+      'tests',
+      'scripts',
+      'coverage',
+      '.DS_Store',
+    ]);
     await cp(src, dest, {
       recursive: true,
       filter: (p) => {
@@ -683,9 +766,9 @@ function pluginContentMatches(src, dest) {
     if (!sEntries) return false;
     // src should not have files missing from dest
     for (const e of sEntries) {
-      if (skipDirs.has(e)) continue;
-      const sFull = join(s, e);
-      const dFull = join(d, e);
+      if (skipDirs.has(e.name)) continue;
+      const sFull = join(s, e.name);
+      const dFull = join(d, e.name);
       try {
         const sSt = statSync(sFull);
         if (sSt.isDirectory()) {
@@ -780,7 +863,14 @@ export async function patchClineJson({ dryRun, force }) {
   }
 
   if (!hasEntry) {
-    plugins.push(['./plugins/bizar/index.ts', {
+    // v5.6.0-beta.14: point at the plugin DIRECTORY (not a file inside
+    // it). Cline reads `package.json#cline.plugins` from there and loads
+    // the entry point declared in the manifest. Pointing at `index.ts`
+    // directly used to work, but with the new auto-discovery rules (see
+    // https://docs.cline.bot/customization/plugins) it can cause every
+    // .ts file under the plugin tree to be loaded as a separate plugin
+    // module — producing ~100 `Invalid plugin module` errors at startup.
+    plugins.push(['./plugins/bizar', {
       loopThresholdWarn: 5,
       loopThresholdEscalate: 8,
       loopThresholdBlock: 12,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polderlabs/bizar",
-  "version": "5.6.0-beta.12",
+  "version": "5.6.0-beta.14",
   "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": {

package/packages/sdk/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polderlabs/bizar-sdk",
-  "version": "0.2.0-beta.12",
+  "version": "0.2.0-beta.14",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",

package/plugins/bizar/package.json

@@ -0,0 +1,25 @@
+{
+  "name": "@polderlabs/bizar-plugin",
+  "version": "5.6.0-beta.14",
+  "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"]
+      }
+    ]
+  },
+  "peerDependencies": {
+    "@cline/sdk": "*",
+    "@cline/core": "*",
+    "@cline/shared": "*"
+  },
+  "peerDependenciesMeta": {
+    "@cline/sdk": { "optional": true },
+    "@cline/core": { "optional": true },
+    "@cline/shared": { "optional": true }
+  }
+}