@the-open-engine/zeroshot 6.35.1 → 6.35.2

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "@the-open-engine/zeroshot",
-  "version": "6.35.1",
+  "version": "6.35.2",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@the-open-engine/zeroshot",
-      "version": "6.35.1",
+      "version": "6.35.2",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@the-open-engine/zeroshot",
-  "version": "6.35.1",
+  "version": "6.35.2",
   "description": "Independent executor–verifier orchestration for software changes.",
   "main": "src/orchestrator.js",
   "bin": {

package/src/agent/agent-lifecycle.js

@@ -892,7 +892,6 @@ ${'='.repeat(80)}`);
     unsupportedCapability,
     structuredOutputInvalid,
   });
-
   // Publish error to message bus for visibility in logs
   agent._publish({
     topic: 'AGENT_ERROR',
@@ -905,6 +904,7 @@ ${'='.repeat(80)}`);
         hookFailure: error?.hookFailure === true,
         restartExhausted: error?.restartExhausted === true,
         terminationExhausted: error?.terminationExhausted === true,
+        retryBudgetExhausted: true,
         hookRetries: error?.hookRetries ?? undefined,
         originalHookError: error?.originalHookError ?? undefined,
         agent: agent.id,

package/src/agent/critical-agent-policy.js

@@ -0,0 +1,17 @@
+// @ts-nocheck
+
+// Validators publish a rejection when their task exhausts retries. Every other named role can
+// execute provider work, including user-defined and orchestrator roles, so exhaustion must
+// terminalize the cluster by default instead of requiring this policy to know each future role.
+const NON_TERMINAL_AGENT_ROLES = new Set(['validator']);
+
+function isCriticalAgent(agent) {
+  if (agent?.id === 'consensus-coordinator') return true;
+  return (
+    typeof agent?.role === 'string' &&
+    agent.role.length > 0 &&
+    !NON_TERMINAL_AGENT_ROLES.has(agent.role)
+  );
+}
+
+module.exports = { isCriticalAgent };

package/src/agent/provider-terminal-failure.js

@@ -1,6 +1,7 @@
 // @ts-nocheck
 
 const { getProvider } = require('../providers');
+const { isCriticalAgent } = require('./critical-agent-policy');
 const { extractCliFailure } = require('./output-extraction');
 
 function categoryForProviderFailure(error, classification) {
@@ -120,10 +121,7 @@ function publishCriticalFailure({
     unsupportedCapability ||
     error?.vertexModelError ||
     error?.terminationExhausted;
-  const critical =
-    agent.role === 'implementation' ||
-    agent.role === 'coordinator' ||
-    agent.id === 'consensus-coordinator';
+  const critical = isCriticalAgent(agent);
   if (!critical || specific) return worker;
 
   agent._publish({

package/src/orchestrator.js

@@ -60,6 +60,7 @@ const {
   normalizeProviderSession,
   restoreAgentProviderSession,
 } = require('./agent/provider-session');
+const { isCriticalAgent } = require('./agent/critical-agent-policy');
 const {
   commandProofsToQualityGates,
   mergeCommandProofs,
@@ -1582,6 +1583,7 @@ class Orchestrator {
       const attempts = message.content?.data?.attempts || 1;
       const hookFailure = message.content?.data?.hookFailure === true;
       const restartExhausted = message.content?.data?.restartExhausted === true;
+      const retryBudgetExhausted = message.content?.data?.retryBudgetExhausted === true;
       const durableClusterFailure = this._findCurrentRunClusterFailure(
         messageBus,
         clusterId,
@@ -1590,14 +1592,14 @@ class Orchestrator {
 
       await this._saveClusters();
 
-      const shouldStopForRole =
-        agentRole === 'implementation' ||
-        agentRole === 'coordinator' ||
-        message.sender === 'consensus-coordinator';
+      const shouldStopForRole = isCriticalAgent({
+        id: message.sender,
+        role: agentRole,
+      });
       const shouldStop =
         !durableClusterFailure &&
         shouldStopForRole &&
-        (hookFailure || restartExhausted || attempts >= 3);
+        (hookFailure || restartExhausted || retryBudgetExhausted);
 
       if (shouldStop) {
         this._log(`\n${'='.repeat(80)}`);
@@ -1607,6 +1609,7 @@ class Orchestrator {
           `${message.sender} (${agentRole || 'unknown role'}) failed` +
             (hookFailure ? ` (hookFailure=true)` : ``) +
             (restartExhausted ? ` (restartExhausted=true)` : ``) +
+            (retryBudgetExhausted ? ` (retryBudgetExhausted=true)` : ``) +
             ` after ${attempts} attempts`
         );
         this._log(`Error: ${message.content?.data?.error || 'unknown'}`);