@magnusekdahl/parallix 1.3.0 → 1.3.1

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
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@magnusekdahl/parallix",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "AI mission workflow toolkit with a px CLI — local-first, human-in-the-loop multi-agent development",
   "license": "AGPL-3.0-or-later",
   "private": false,