@chllming/wave-orchestration 0.9.13 → 0.9.14

package/package.json

@@ -1,8 +1,9 @@
 {
   "name": "@chllming/wave-orchestration",
-  "version": "0.9.13",
+  "version": "0.9.14",
   "license": "MIT",
   "description": "Generic wave-based multi-agent orchestration for repository work.",
+  "packageManager": "pnpm@10.23.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/chllming/agent-wave-orchestrator.git"
@@ -31,12 +32,6 @@
     "wave-dashboard": "scripts/wave-dashboard.mjs",
     "wave-local-executor": "scripts/wave-local-executor.mjs"
   },
-  "devDependencies": {
-    "@mozilla/readability": "^0.6.0",
-    "jsdom": "^29.0.1",
-    "pdfjs-dist": "^5.5.207",
-    "vitest": "3.2.4"
-  },
   "scripts": {
     "context7:api-check": "bash scripts/context7-export-env.sh run bash scripts/context7-api-check.sh",
     "research:import-agent-context": "node scripts/research/import-agent-context-archive.mjs scripts/research/manifests/agent-context-expanded-2026-03-22.mjs",
@@ -50,5 +45,11 @@
     "wave:feedback": "node scripts/wave-human-feedback.mjs",
     "wave:launch": "node scripts/wave-launcher.mjs",
     "wave:local": "node scripts/wave-local-executor.mjs"
+  },
+  "devDependencies": {
+    "@mozilla/readability": "^0.6.0",
+    "jsdom": "^29.0.1",
+    "pdfjs-dist": "^5.5.207",
+    "vitest": "3.2.4"
   }
-}
+}

package/scripts/wave-orchestrator/agent-state.mjs

@@ -922,8 +922,28 @@ export function validateImplementationSummary(agent, summary) {
   };
 }
 
-export function validateDocumentationClosureSummary(agent, summary) {
+export function validateDocumentationClosureSummary(agent, summary, options = {}) {
   if (!summary?.docClosure) {
+    // When the agent had exit-code 0 but produced no structured output (e.g. credential
+    // broker collision, empty run), allow a graceful fallback instead of hard-failing
+    // the entire wave.  The caller (gate-engine) decides whether the surrounding
+    // integration/QA state justifies auto-closing documentation.
+    const emptyRun = !summary || (
+      !summary.proof && !summary.component && !summary.integration &&
+      !summary.verdict && !summary.docClosure && !summary.security &&
+      (summary.rawSignalCount === 0 || summary.rawSignalCount === undefined)
+    );
+    if (options.allowFallbackOnEmptyRun && emptyRun) {
+      return {
+        ok: false,
+        statusCode: "missing-doc-closure-empty-run",
+        detail: appendTerminationHint(
+          `Documentation steward ${agent?.agentId || "A9"} produced no output (empty run). Eligible for fallback auto-closure.`,
+          summary,
+        ),
+        eligibleForFallback: true,
+      };
+    }
     return {
       ok: false,
       statusCode: "missing-doc-closure",

package/scripts/wave-orchestrator/gate-engine.mjs

@@ -922,7 +922,29 @@ export function readWaveDocumentationGate(wave, agentRuns, options = {}) {
       mode,
       securityRolePromptPath: options.securityRolePromptPath,
     });
-  const validation = validateDocumentationClosureSummary(docRun.agent, summary);
+  const validation = validateDocumentationClosureSummary(docRun.agent, summary, {
+    allowFallbackOnEmptyRun: true,
+  });
+  // Fallback: when doc steward had an empty run but integration/QA passed,
+  // auto-close instead of blocking the wave.
+  if (!validation.ok && validation.eligibleForFallback) {
+    const integrationSummary = options.derivedState?.integrationSummary;
+    const integrationReady =
+      integrationSummary?.recommendation === "ready-for-doc-closure";
+    const contQaPassed =
+      options.derivedState?.ledger?.contQa?.verdict === "pass" ||
+      options.derivedState?.ledger?.contQa?.verdict === "PASS";
+    if (integrationReady || contQaPassed) {
+      return {
+        ok: true,
+        agentId: docRun.agent.agentId,
+        statusCode: "fallback-auto-closed",
+        detail: `Documentation steward ${docRun.agent.agentId} had an empty run. Auto-closed because ${integrationReady ? "integration is ready-for-doc-closure" : "cont-QA passed"}.`,
+        logPath: summary?.logPath || path.relative(REPO_ROOT, docRun.logPath),
+        docClosureState: "no-change",
+      };
+    }
+  }
   return {
     ok: validation.ok,
     agentId: docRun.agent.agentId,
@@ -1501,7 +1523,30 @@ export function readWaveDocumentationGatePure(wave, agentResults, options = {})
   }
   const summary = agentResults?.[documentationAgentId] || null;
   const agent = { agentId: documentationAgentId };
-  const validation = validateDocumentationClosureSummary(agent, summary);
+  const validation = validateDocumentationClosureSummary(agent, summary, {
+    allowFallbackOnEmptyRun: true,
+  });
+  // When the doc steward had an empty run (broker collision, no output) but
+  // integration is already ready-for-doc-closure, auto-close documentation
+  // instead of failing the entire wave.
+  if (!validation.ok && validation.eligibleForFallback) {
+    const integrationSummary = options.derivedState?.integrationSummary;
+    const integrationReady =
+      integrationSummary?.recommendation === "ready-for-doc-closure";
+    const contQaPassed =
+      options.derivedState?.ledger?.contQa?.verdict === "pass" ||
+      options.derivedState?.ledger?.contQa?.verdict === "PASS";
+    if (integrationReady || contQaPassed) {
+      return {
+        ok: true,
+        agentId: documentationAgentId,
+        statusCode: "fallback-auto-closed",
+        detail: `Documentation steward ${documentationAgentId} had an empty run. Auto-closed because ${integrationReady ? "integration is ready-for-doc-closure" : "cont-QA passed"}.`,
+        logPath: summary?.logPath || null,
+        docClosureState: "no-change",
+      };
+    }
+  }
   return { ok: validation.ok, agentId: documentationAgentId, statusCode: validation.statusCode,
     detail: validation.detail, logPath: summary?.logPath || null };
 }

package/scripts/wave-orchestrator/task-entity.mjs

@@ -905,10 +905,19 @@ export function evaluateOwnedSliceProven(task, agentResult, proofBundles = []) {
   }
 
   if (task.taskType === "documentation") {
-    const validation = validateDocumentationClosureSummary(agent, agentResult);
-    return validation.ok
-      ? { proven: true, reason: "Documentation closure satisfied" }
-      : { proven: false, reason: validation.detail || validation.statusCode };
+    const validation = validateDocumentationClosureSummary(agent, agentResult, {
+      allowFallbackOnEmptyRun: true,
+    });
+    if (validation.ok) {
+      return { proven: true, reason: "Documentation closure satisfied" };
+    }
+    // Allow fallback-eligible empty runs to pass at the task level;
+    // the gate engine will make the final call on whether surrounding
+    // state justifies auto-closure.
+    if (validation.eligibleForFallback) {
+      return { proven: true, reason: "Documentation closure fallback (empty run, deferred to gate)" };
+    }
+    return { proven: false, reason: validation.detail || validation.statusCode };
   }
 
   if (task.taskType === "security") {