@magnusekdahl/parallix 1.2.1 → 1.3.1

package/index.js

@@ -1,209 +1,226 @@
 #!/usr/bin/env node
-
+"use strict";
 /**
  * VisualBoard Parallix Coordination CLI
  * ADR 0037 — Wave 1
  */
-
-const fs = require('fs');
-const path = require('path');
-const fmt = require('./lib/core/fmt');
-const { ensureStandaloneGitRepo } = require('./lib/core/product-config');
-const { loadStateMap } = require('./lib/core/state-map');
-
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.KNOWN_COMMANDS = void 0;
+exports.deriveAliases = deriveAliases;
+exports.resolveAlias = resolveAlias;
+exports.printAliases = printAliases;
+exports.buildSuggestionSuffix = buildSuggestionSuffix;
+exports.suggestCommand = suggestCommand;
+exports.levenshteinDistance = levenshteinDistance;
+exports.printUsage = printUsage;
+exports.main = main;
+const node_fs_1 = __importDefault(require("node:fs"));
+const node_path_1 = __importDefault(require("node:path"));
+const fmt = __importStar(require("./lib/core/fmt.js"));
+const product_config_js_1 = require("./lib/core/product-config.js");
+const state_map_js_1 = require("./lib/core/state-map.js");
 // Fixed virtual-state → canonical-command invariants for alias derivation.
 const STATE_COMMAND_MAP = {
-  ready: 'draft',
-  approved: 'integrate',
-  done: 'integrate',
+    ready: 'draft',
+    approved: 'integrate',
+    done: 'integrate',
 };
-
-const KNOWN_COMMANDS = [
-  'mission-start',
-  'verify-env',
-  'verify',
-  'setup',
-  'setup-review',
-  'draft',
-  'active',
-  'status',
-  'checkpoint',
-  'review',
-  'handoff',
-  'integrate',
-  'resolve-conflict',
-  'rebase',
-  'stats',
-  'aliases',
-  'config',
-  'diff',
+exports.KNOWN_COMMANDS = [
+    'mission-start',
+    'verify-env',
+    'verify',
+    'setup',
+    'setup-review',
+    'draft',
+    'active',
+    'status',
+    'checkpoint',
+    'review',
+    'handoff',
+    'integrate',
+    'resolve-conflict',
+    'rebase',
+    'stats',
+    'aliases',
+    'config',
+    'diff',
 ];
-
 const READ_ONLY_COMMANDS = new Set(['config']);
-
 function loadStateMapForAliases(options = {}) {
-  return loadStateMap(options);
+    return (0, state_map_js_1.loadStateMap)(options);
 }
-
 // Derives the command-alias table from state-map.json and fixed parallix invariants.
 // Users only maintain state-map.json; aliases update automatically.
 function deriveAliases(options = {}) {
-  const stateMap = loadStateMapForAliases(options);
-  const aliases = { ...STATE_COMMAND_MAP };
-  for (const [virtual, actual] of Object.entries(stateMap)) {
-    if (actual && actual !== virtual && STATE_COMMAND_MAP[virtual]) {
-      aliases[actual] = STATE_COMMAND_MAP[virtual];
+    const stateMap = loadStateMapForAliases(options);
+    const aliases = { ...STATE_COMMAND_MAP };
+    for (const [virtual, actual] of Object.entries(stateMap)) {
+        if (typeof actual === 'string' && actual !== virtual && STATE_COMMAND_MAP[virtual]) {
+            aliases[actual] = STATE_COMMAND_MAP[virtual];
+        }
     }
-  }
-  return aliases;
+    return aliases;
 }
-
 function resolveAlias(command, aliases = deriveAliases()) {
-  return Object.prototype.hasOwnProperty.call(aliases, command) ? aliases[command] : null;
+    return Object.prototype.hasOwnProperty.call(aliases, command) ? aliases[command] : null;
 }
-
 function printAliases(aliases, logFn = fmt.log.plain) {
-  const entries = Object.entries(aliases).sort(([a], [b]) => a.localeCompare(b));
-  if (entries.length === 0) {
-    logFn('No aliases configured.');
-    return;
-  }
-  logFn(fmt.bold('alias                         canonical'));
-  logFn(fmt.bold('---                           ---'));
-  for (const [alias, canonical] of entries) {
-    logFn(`${alias.padEnd(30)}${canonical}`);
-  }
+    const entries = Object.entries(aliases).sort(([a], [b]) => a.localeCompare(b));
+    if (entries.length === 0) {
+        logFn('No aliases configured.');
+        return;
+    }
+    logFn(fmt.bold('alias                         canonical'));
+    logFn(fmt.bold('---                           ---'));
+    for (const [alias, canonical] of entries) {
+        logFn(`${alias.padEnd(30)}${canonical}`);
+    }
 }
-
-async function main(args = process.argv.slice(2), {
-  existsSyncFn = fs.existsSync,
-  cwdFn = () => process.cwd(),
-  ensureStandaloneGitRepoFn = ensureStandaloneGitRepo,
-  requireFn = require,
-  printUsageFn = printUsage,
-  exitFn = process.exit,
-  errorFn = fmt.log.plainError,
-  logFn = fmt.log.plain,
-  loadAliasesFn = deriveAliases,
-} = {}) {
-  const command = args[0];
-
-  if (command === '--version' || command === '-v') {
-    const pkg = requireFn(path.join(__dirname, 'package.json'));
-    logFn(pkg.name + '@' + pkg.version);
-    exitFn(0);
-    return;
-  }
-
-  if (!command || command === 'help' || command === '--help' || command === '-h') {
-    printUsageFn();
-    exitFn(0);
-    return;
-  }
-
-  if (command === 'aliases') {
-    printAliases(loadAliasesFn({ rootDir: cwdFn() }), logFn);
-    return;
-  }
-
-  const libPath = path.join(__dirname, 'lib', 'commands', `${command}.js`);
-
-  // Special case: verify-env is an alias for mission-start diagnostic.
-  // active is now its own command (real agent-start path).
-  let targetLib = libPath;
-  if (command === 'verify-env') {
-    targetLib = path.join(__dirname, 'lib', 'commands', 'mission-start.js');
-  }
-
-  if (existsSyncFn(targetLib)) {
-    if (!READ_ONLY_COMMANDS.has(command)) {
-      const initResult = ensureStandaloneGitRepoFn(cwdFn());
-      if (initResult && initResult.failed) {
-        errorFn(fmt.status('FAIL', `Git init: ${initResult.message}`));
-        exitFn(1);
+async function main(args = process.argv.slice(2), options = {}) {
+    const { existsSyncFn = node_fs_1.default.existsSync, cwdFn = () => process.cwd(), ensureStandaloneGitRepoFn = product_config_js_1.ensureStandaloneGitRepo, requireFn = require, printUsageFn = printUsage, exitFn = process.exit, errorFn = fmt.log.plainError, logFn = fmt.log.plain, loadAliasesFn = deriveAliases, } = options;
+    const command = args[0];
+    if (command === '--version' || command === '-v') {
+        const pkg = requireFn(node_path_1.default.join(__dirname, 'package.json'));
+        logFn(pkg.name + '@' + pkg.version);
+        exitFn(0);
         return;
-      }
-      if (initResult && initResult.initialized) {
-        logFn(fmt.status('INFO', `Initialized git repository for standalone parallix in ${cwdFn()} (branch ${initResult.branch || 'main'}).`));
-      }
     }
-
-    const cmdModule = requireFn(targetLib);
-    if (typeof cmdModule === 'function') {
-      await cmdModule(args.slice(1), { command });
-    } else {
-      errorFn(fmt.status('FAIL', `Command module '${command}' does not export a function.`));
-      exitFn(1);
+    if (!command || command === 'help' || command === '--help' || command === '-h') {
+        printUsageFn();
+        exitFn(0);
+        return;
     }
-  } else {
-    const aliases = loadAliasesFn({ rootDir: cwdFn() });
-    const canonical = resolveAlias(command, aliases);
-    if (canonical) {
-      logFn(fmt.status('INFO', `Resolving alias ${command} → ${canonical}`));
-      await main([canonical, ...args.slice(1)], { existsSyncFn, cwdFn, ensureStandaloneGitRepoFn, requireFn, printUsageFn, exitFn, errorFn, logFn, loadAliasesFn });
-      return;
+    if (command === 'aliases') {
+        printAliases(loadAliasesFn({ rootDir: cwdFn() }), logFn);
+        return;
     }
-
-    errorFn(fmt.status('FAIL', `Unknown command: ${command}`));
-    const suggestion = suggestCommand(command);
-    if (suggestion) {
-      errorFn(fmt.status('INFO', `Did you mean: px ${suggestion}${buildSuggestionSuffix(suggestion)}`));
+    const libPath = node_path_1.default.join(__dirname, 'lib', 'commands', `${command}.js`);
+    // Special case: verify-env is an alias for mission-start diagnostic.
+    // active is now its own command (real agent-start path).
+    let targetLib = libPath;
+    if (command === 'verify-env') {
+        targetLib = node_path_1.default.join(__dirname, 'lib', 'commands', 'mission-start.js');
+    }
+    if (existsSyncFn(targetLib)) {
+        if (!READ_ONLY_COMMANDS.has(command)) {
+            const initResult = ensureStandaloneGitRepoFn(cwdFn());
+            if (initResult && initResult.failed) {
+                errorFn(fmt.status('FAIL', `Git init: ${initResult.message}`));
+                exitFn(1);
+                return;
+            }
+            if (initResult && initResult.initialized) {
+                logFn(fmt.status('INFO', `Initialized git repository for standalone parallix in ${cwdFn()} (branch ${initResult.branch || 'main'}).`));
+            }
+        }
+        const cmdModule = requireFn(targetLib);
+        if (typeof cmdModule === 'function') {
+            await cmdModule(args.slice(1), { command });
+        }
+        else {
+            errorFn(fmt.status('FAIL', `Command module '${command}' does not export a function.`));
+            exitFn(1);
+        }
+    }
+    else {
+        const aliases = loadAliasesFn({ rootDir: cwdFn() });
+        const canonical = resolveAlias(command, aliases);
+        if (canonical) {
+            logFn(fmt.status('INFO', `Resolving alias ${command} → ${canonical}`));
+            await main([canonical, ...args.slice(1)], { existsSyncFn, cwdFn, ensureStandaloneGitRepoFn, requireFn, printUsageFn, exitFn, errorFn, logFn, loadAliasesFn });
+            return;
+        }
+        errorFn(fmt.status('FAIL', `Unknown command: ${command}`));
+        const suggestion = suggestCommand(command);
+        if (suggestion) {
+            errorFn(fmt.status('INFO', `Did you mean: px ${suggestion}${buildSuggestionSuffix(suggestion)}`));
+        }
+        printUsageFn();
+        exitFn(1);
     }
-    printUsageFn();
-    exitFn(1);
-  }
 }
-
 function buildSuggestionSuffix(command) {
-  if (command === 'diff' || command === 'resolve-conflict') {
-    return ' <slug>';
-  }
-  if (command === 'checkpoint') {
-    return ' <slug> <cp-name> "<next-action>"';
-  }
-  return '';
+    if (command === 'diff' || command === 'resolve-conflict') {
+        return ' <slug>';
+    }
+    if (command === 'checkpoint') {
+        return ' <slug> <cp-name> "<next-action>"';
+    }
+    return '';
 }
-
 function suggestCommand(input) {
-  if (!input) return null;
-  const normalizedInput = input.toLowerCase();
-  let best = null;
-
-  for (const candidate of KNOWN_COMMANDS) {
-    const distance = levenshteinDistance(normalizedInput, candidate);
-    if (distance > 2) continue;
-    if (!best || distance < best.distance) {
-      best = { candidate, distance };
+    if (!input) {
+        return null;
     }
-  }
-
-  return best ? best.candidate : null;
+    const normalizedInput = input.toLowerCase();
+    let best = null;
+    for (const candidate of exports.KNOWN_COMMANDS) {
+        const distance = levenshteinDistance(normalizedInput, candidate);
+        if (distance > 2) {
+            continue;
+        }
+        if (!best || distance < best.distance) {
+            best = { candidate, distance };
+        }
+    }
+    return best ? best.candidate : null;
 }
-
 function levenshteinDistance(a, b) {
-  const rows = a.length + 1;
-  const cols = b.length + 1;
-  const dp = Array.from({ length: rows }, () => Array(cols).fill(0));
-
-  for (let i = 0; i < rows; i += 1) dp[i][0] = i;
-  for (let j = 0; j < cols; j += 1) dp[0][j] = j;
-
-  for (let i = 1; i < rows; i += 1) {
-    for (let j = 1; j < cols; j += 1) {
-      const cost = a[i - 1] === b[j - 1] ? 0 : 1;
-      dp[i][j] = Math.min(
-        dp[i - 1][j] + 1,
-        dp[i][j - 1] + 1,
-        dp[i - 1][j - 1] + cost
-      );
+    const rows = a.length + 1;
+    const cols = b.length + 1;
+    const dp = Array.from({ length: rows }, () => Array(cols).fill(0));
+    for (let i = 0; i < rows; i += 1) {
+        dp[i][0] = i;
     }
-  }
-
-  return dp[a.length][b.length];
+    for (let j = 0; j < cols; j += 1) {
+        dp[0][j] = j;
+    }
+    for (let i = 1; i < rows; i += 1) {
+        for (let j = 1; j < cols; j += 1) {
+            const cost = a[i - 1] === b[j - 1] ? 0 : 1;
+            dp[i][j] = Math.min(dp[i - 1][j] + 1, dp[i][j - 1] + 1, dp[i - 1][j - 1] + cost);
+        }
+    }
+    return dp[a.length][b.length];
 }
-
 function printUsage() {
-  fmt.log.plain(`
+    fmt.log.plain(`
 Usage: px <command> [args]
 
 ${fmt.bold('Core Commands:')}
@@ -239,19 +256,7 @@ ${fmt.bold('Notes:')}
   - No npm dependencies — requires Node.js built-ins only.
 `);
 }
-
-if (require.main === module) {
-  main();
+// Run main when executed directly (compiled to CJS, require.main === module applies)
+if (typeof require !== 'undefined' && require.main === module) {
+    main();
 }
-
-module.exports = {
-  KNOWN_COMMANDS,
-  main,
-  printUsage,
-  printAliases,
-  suggestCommand,
-  buildSuggestionSuffix,
-  levenshteinDistance,
-  deriveAliases,
-  resolveAlias,
-};

package/lib/agents/agents.js

@@ -51,6 +51,7 @@ exports.isInvalidAgentConfigError = isInvalidAgentConfigError;
 exports.updateAgentBlock = updateAgentBlock;
 exports.resolveBlocklistTargetPath = resolveBlocklistTargetPath;
 exports.resolveNoOutputWatchdogConfig = resolveNoOutputWatchdogConfig;
+exports.shouldPersistLaunchFailureBlock = shouldPersistLaunchFailureBlock;
 const node_fs_1 = __importDefault(require("node:fs"));
 const node_path_1 = __importDefault(require("node:path"));
 const node_child_process_1 = require("node:child_process");
@@ -107,6 +108,17 @@ const KNOWN_AGENT_NAMES = Object.freeze([
     'human'
 ]);
 exports.KNOWN_AGENT_NAMES = KNOWN_AGENT_NAMES;
+const NON_BLOCKING_LAUNCH_ERROR_PATTERNS = Object.freeze([
+    /\b(?:invalid|unknown|unsupported|unrecognized)\s+model\b/i,
+    /\bmodel\s+(?:identifier|id)\s+(?:is\s+)?invalid\b/i,
+    /\b(?:model\s+not\s+found|no\s+such\s+model)\b/i,
+    /\bunknown\s+option\b/i,
+    /\bauth(?:entication)?\b/i,
+    /\bunauthorized\b/i,
+    /\bforbidden\b/i,
+    /\bapi\s+key\b/i,
+    /\bread-only file system\b/i
+]);
 function workflowLauncherStatus(agent) {
     const resolver = RESOLVERS[agent];
     if (!resolver) {
@@ -148,6 +160,21 @@ function commandInPath(name) {
     });
     return result.status === 0 && result.stdout.trim().length > 0;
 }
+function shouldPersistLaunchFailureBlock(agent, result) {
+    if (!result || agent === 'custom') {
+        return false;
+    }
+    const combined = [
+        result.stderr || '',
+        result.stdout || '',
+        result.error?.message || '',
+        result.error?.code || ''
+    ].join('\n');
+    if (!combined.trim()) {
+        return true;
+    }
+    return !NON_BLOCKING_LAUNCH_ERROR_PATTERNS.some(pattern => pattern.test(combined));
+}
 function buildInvalidAgentConfigError(configPath, scope, originalError) {
     const location = node_path_1.default.resolve(configPath);
     const detail = originalError && originalError.message ? originalError.message : 'invalid JSON';
@@ -790,10 +817,10 @@ async function startAgent(step, opts = { prompt: '' }) {
             });
             tried.add(chosen || '');
             launched.add(chosen || '');
-            // Block non-custom agents on non-limit failures so selectAgent excludes them
-            // on the next retry iteration, and the review-loop fallback path can activate.
-            // custom (opencode/local AI) is excluded — exit 1 is a temporary local error.
-            if (chosen !== 'custom') {
+            // Block retry candidates only when the failure looks transient. Deterministic
+            // setup/config errors (invalid model id, auth failure, read-only HOME, etc.)
+            // should fall through to the next family without poisoning agents.local.json.
+            if (shouldPersistLaunchFailureBlock(chosen || '', result)) {
                 const blockUntil = (0, limit_hit_js_1.formatBlockUntil)(new Date(Date.now() + limit_hit_js_1.DEFAULT_FALLBACK_HOURS * 60 * 60 * 1000));
                 try {
                     const blockResult = updateAgentBlockFn(chosen || '', blockUntil);
@@ -803,6 +830,9 @@ async function startAgent(step, opts = { prompt: '' }) {
                     log(fmt.status('WARN', `Could not persist blocklist entry for ${fmt.agent(chosen || '')}: ${err.message}`));
                 }
             }
+            else {
+                log(fmt.status('INFO', `Skipping blocklist write for ${fmt.agent(chosen || '')}; launch failure looks like a deterministic config/setup error.`));
+            }
             chosen = undefined;
             continue;
         }

package/lib/agents/agents.ts

@@ -22,6 +22,14 @@ interface LauncherStatus {
   reason?: string;
 }
 
+interface LaunchResultLike {
+  stdout?: string;
+  stderr?: string;
+  status?: number | null;
+  signal?: string | null;
+  error?: {code?: string, message?: string} | null;
+}
+
 type AgentConfig = { blocklist?: {[key: string]: any}, steps?: {[key: string]: any} };
 
 interface StartAgentOptions {
@@ -100,6 +108,18 @@ const KNOWN_AGENT_NAMES = Object.freeze([
   'human'
 ]);
 
+const NON_BLOCKING_LAUNCH_ERROR_PATTERNS = Object.freeze([
+  /\b(?:invalid|unknown|unsupported|unrecognized)\s+model\b/i,
+  /\bmodel\s+(?:identifier|id)\s+(?:is\s+)?invalid\b/i,
+  /\b(?:model\s+not\s+found|no\s+such\s+model)\b/i,
+  /\bunknown\s+option\b/i,
+  /\bauth(?:entication)?\b/i,
+  /\bunauthorized\b/i,
+  /\bforbidden\b/i,
+  /\bapi\s+key\b/i,
+  /\bread-only file system\b/i
+]);
+
 function workflowLauncherStatus(agent: string): LauncherStatus {
   const resolver = RESOLVERS[agent];
   if (!resolver) {
@@ -146,6 +166,18 @@ function commandInPath(name: string) {
   return result.status === 0 && result.stdout.trim().length > 0;
 }
 
+function shouldPersistLaunchFailureBlock(agent: string, result: LaunchResultLike | null | undefined) {
+  if (!result || agent === 'custom') {return false;}
+  const combined = [
+    result.stderr || '',
+    result.stdout || '',
+    result.error?.message || '',
+    result.error?.code || ''
+  ].join('\n');
+  if (!combined.trim()) {return true;}
+  return !NON_BLOCKING_LAUNCH_ERROR_PATTERNS.some(pattern => pattern.test(combined));
+}
+
 function buildInvalidAgentConfigError(configPath: string, scope: string, originalError: {message?: string} | null) {
   const location = path.resolve(configPath);
   const detail = originalError && originalError.message ? originalError.message : 'invalid JSON';
@@ -869,10 +901,10 @@ async function startAgent(step: string, opts: StartAgentOptions = { prompt: '' }
       });
       tried.add(chosen || '');
       launched.add(chosen || '');
-      // Block non-custom agents on non-limit failures so selectAgent excludes them
-      // on the next retry iteration, and the review-loop fallback path can activate.
-      // custom (opencode/local AI) is excluded — exit 1 is a temporary local error.
-      if (chosen !== 'custom') {
+      // Block retry candidates only when the failure looks transient. Deterministic
+      // setup/config errors (invalid model id, auth failure, read-only HOME, etc.)
+      // should fall through to the next family without poisoning agents.local.json.
+      if (shouldPersistLaunchFailureBlock(chosen || '', result)) {
         const blockUntil = formatBlockUntil(new Date(Date.now() + DEFAULT_FALLBACK_HOURS * 60 * 60 * 1000));
         try {
           const blockResult = updateAgentBlockFn(chosen || '', blockUntil);
@@ -880,6 +912,8 @@ async function startAgent(step: string, opts: StartAgentOptions = { prompt: '' }
         } catch (err) {
           log(fmt.status('WARN', `Could not persist blocklist entry for ${fmt.agent(chosen || '')}: ${(err as any).message}`));
         }
+      } else {
+        log(fmt.status('INFO', `Skipping blocklist write for ${fmt.agent(chosen || '')}; launch failure looks like a deterministic config/setup error.`));
       }
       chosen = undefined;
       continue;
@@ -926,5 +960,6 @@ export {
   isInvalidAgentConfigError,
   updateAgentBlock,
   resolveBlocklistTargetPath,
-  resolveNoOutputWatchdogConfig
+  resolveNoOutputWatchdogConfig,
+  shouldPersistLaunchFailureBlock
 };