cool-workflow 0.2.0 → 0.2.1

package/scripts/agents/claude-p-agent.js

@@ -34,7 +34,7 @@ const { spawn, spawnSync } = require("node:child_process");
 // wrappers instead of carrying a private copy. A drifted inline copy (ASCII
 // hyphens silently became em-dashes here) meant claude was sent a different
 // instruction text than the other providers for the same contract.
-const { buildPrompt, createRenderer, persistStderr, toolLabel, summarizeToolResult } = require("./agent-adapter-core");
+const { buildPrompt, createRenderer, persistStderr, toolLabel, summarizeToolResult, buildFailureDetail } = require("./agent-adapter-core");
 
 const inputPath = process.argv[2];
 const resultPath = process.argv[3];
@@ -179,9 +179,14 @@ child.on("close", (code) => {
   render.finishLive(); // stop the spinner + restore the cursor BEFORE any further output
   render.writeTranscript(transcriptPath); // full narration + tool I/O always saved
   if (code !== 0) {
-    const detail = childStderr.trim() || `claude exited ${code === null ? "(timeout/killed)" : code}`;
+    const stderrText = childStderr.trim();
+    // childStderr is the OS-level stream; claude's real failure reason (auth,
+    // rate limit, relay error) is often reported as a stream-json `result`
+    // event on STDOUT instead, already parsed into `resultText` above — fold
+    // it in here so a silent-stderr failure still leaves a readable reason.
+    const detail = buildFailureDetail({ label: "claude", code, childStderr: stderrText, partialText: resultText });
     persistStderr(resultPath, detail);
-    if (childStderr.trim()) process.stderr.write(`${childStderr.trim()}\n`);
+    if (stderrText) process.stderr.write(`${stderrText}\n`);
     process.stderr.write(`claude exited ${code === null ? "(timeout/killed)" : code}\n`);
     process.exit(code === null ? 1 : code);
   }

package/scripts/agents/gemini-agent.js

@@ -17,6 +17,7 @@
 const path = require("node:path");
 const { spawn } = require("node:child_process");
 const {
+  buildFailureDetail,
   buildPrompt,
   createRenderer,
   emitReport,
@@ -94,7 +95,10 @@ child.on("close", (code) => {
   render.finishLive();
   render.writeTranscript(transcriptPath);
   if (code !== 0) {
-    const detail = childStderr.trim() || `gemini exited ${code === null ? "(timeout/killed)" : code}`;
+    // gemini's real failure reason is often only in the NDJSON text/result
+    // fragments already parsed into `state`, not in raw OS-level stderr.
+    const partial = state.finalResult || state.textFragments.join("\n\n");
+    const detail = buildFailureDetail({ label: "gemini", code, childStderr: childStderr.trim(), partialText: partial });
     persistStderr(resultPath, detail);
     process.stderr.write(`${detail}\n`);
     process.exit(code === null ? 1 : code);

package/scripts/agents/opencode-agent.js

@@ -22,6 +22,7 @@
 const path = require("node:path");
 const { spawn } = require("node:child_process");
 const {
+  buildFailureDetail,
   buildPrompt,
   createRenderer,
   emitReport,
@@ -147,7 +148,10 @@ child.on("close", (code) => {
   render.finishLive();
   render.writeTranscript(transcriptPath);
   if (code !== 0) {
-    const detail = childStderr.trim() || `opencode exited ${code === null ? "(timeout/killed)" : code}`;
+    // opencode's real failure reason is often only in the JSONL text/result
+    // fragments already parsed into `state`, not in raw OS-level stderr.
+    const partial = state.finalResult || state.lastMessageText || state.textFragments.join("\n\n");
+    const detail = buildFailureDetail({ label: "opencode", code, childStderr: childStderr.trim(), partialText: partial });
     persistStderr(resultPath, detail);
     process.stderr.write(`${detail}\n`);
     process.exit(code === null ? 1 : code);

package/scripts/canonical-apps.js

@@ -83,7 +83,7 @@ const canonicalApps = [
       "--source",
       "plugins/cool-workflow/docs/workflow-app-framework.7.md",
       "--scope",
-      "Cool Workflow v0.2.0",
+      "Cool Workflow v0.2.1",
       "--freshness",
       "as of release preparation"
     ]
@@ -117,14 +117,14 @@ function main() {
     assert.ok(summary, `${app.id} must appear in app list`);
     assert.equal(summary.sourceKind, "app-directory");
     assert.equal(summary.legacy, false);
-    assert.equal(summary.version, "0.2.0");
+    assert.equal(summary.version, "0.2.1");
 
     const validation = runJson(["app", "validate", manifestPath]);
     assert.equal(validation.valid, true, `${app.id} manifest must validate`);
 
     const shown = runJson(["app", "show", app.id]);
     assert.equal(shown.app.id, app.id);
-    assert.equal(shown.app.version, "0.2.0");
+    assert.equal(shown.app.version, "0.2.1");
     assert.ok(shown.app.metadata.canonical, `${app.id} must be marked canonical`);
     assert.ok(shown.app.sandboxProfiles.length > 0, `${app.id} must declare sandbox profiles`);
     assertTaskIdsUnique(shown);
@@ -135,7 +135,7 @@ function main() {
     const plan = runJson(["plan", app.id, ...app.args(workspace)]);
     const state = JSON.parse(fs.readFileSync(plan.statePath, "utf8"));
     assert.equal(state.workflow.app.id, app.id);
-    assert.equal(state.workflow.app.version, "0.2.0");
+    assert.equal(state.workflow.app.version, "0.2.1");
     assert.equal(state.workflow.app.metadata.canonical, true);
     assert.ok(state.tasks.some((task) => task.requiresEvidence), `${app.id} plan must include evidence gates`);
     assert.ok(state.tasks.every((task) => task.sandboxProfileId), `${app.id} plan must include sandbox hints`);

package/scripts/dogfood-release.js

@@ -6,7 +6,7 @@ const fs = require("node:fs");
 const path = require("node:path");
 const { CoolWorkflowRunner } = require("../dist/shell/orchestrator.js");
 
-const TARGET_VERSION = "0.2.0";
+const TARGET_VERSION = "0.2.1";
 const PREVIOUS_VERSION = "0.1.31";
 const pluginRoot = path.resolve(__dirname, "..");
 const repoRoot = path.resolve(pluginRoot, "..", "..");

package/scripts/golden-path.js

@@ -33,7 +33,7 @@ function main() {
     const appValidation = runJson(["app", "validate", "end-to-end-golden-path"], pluginRoot);
     assert.equal(appValidation.valid, true);
     assert.equal(appValidation.summary.id, "end-to-end-golden-path");
-    assert.equal(appValidation.summary.version, "0.2.0");
+    assert.equal(appValidation.summary.version, "0.2.1");
 
     const plan = runJson(
       [
@@ -42,7 +42,7 @@ function main() {
         "--repo",
         tmp,
         "--question",
-        "Prove the deterministic v0.2.0 end-to-end golden path."
+        "Prove the deterministic v0.2.1 end-to-end golden path."
       ],
       pluginRoot
     );
@@ -52,7 +52,7 @@ function main() {
 
     let state = readJson(plan.statePath);
     assert.equal(state.workflow.app.id, "end-to-end-golden-path");
-    assert.equal(state.workflow.app.version, "0.2.0");
+    assert.equal(state.workflow.app.version, "0.2.1");
     assert.equal(state.loopStage, "interpret");
 
     const dispatch = runJson(["dispatch", plan.runId, "--limit", "1", "--sandbox", "readonly"], tmp);
@@ -195,7 +195,7 @@ function main() {
     assert.equal(reportPath, plan.reportPath);
     assert.ok(fs.existsSync(reportPath));
     const report = fs.readFileSync(reportPath, "utf8");
-    assert.match(report, /Workflow App: end-to-end-golden-path@0\.2\.0/);
+    assert.match(report, /Workflow App: end-to-end-golden-path@0\.2\.1/);
     assert.match(report, /## Candidates/);
     assert.match(report, /## Trust Audit/);
     assert.match(report, /## Acceptance Rationale/);