cool-workflow 0.2.1 → 0.2.3

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "cool-workflow",
   "description": "A workflow control plane and run-time you are able to check: it sends out jobs in TypeScript, makes certain of work against facts before it goes through, puts state into fixed records, orders jobs by time, runs jobs again and again, gets a group of agents to do their parts together, and talks MCP. It gives the doing of the work to outside agents — it never runs the models itself.",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "author": {
     "name": "COOLWHITE LLC"
   },

package/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "cool-workflow",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "A workflow control plane and run-time you are able to check: it sends out jobs in TypeScript, makes certain of work against facts before it goes through, puts state into fixed records, orders jobs by time, runs jobs again and again, gets a group of agents to do their parts together, and talks MCP. It gives the doing of the work to outside agents — it never runs the models itself.",
   "author": {
     "name": "COOLWHITE LLC"

package/apps/architecture-review/app.json

@@ -3,7 +3,7 @@
   "id": "architecture-review",
   "title": "Architecture Review",
   "summary": "Map a repository architecture, assess risks, verify important findings, and synthesize an evidence-backed verdict.",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "author": "COOLWHITE LLC",
   "inputs": [
     {

package/apps/architecture-review-fast/app.json

@@ -3,7 +3,7 @@
   "id": "architecture-review-fast",
   "title": "Architecture Review Fast",
   "summary": "Run a shorter architecture review with parallel map and assess phases for faster first results.",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "author": "COOLWHITE LLC",
   "inputs": [
     {

package/apps/end-to-end-golden-path/app.json

@@ -3,7 +3,7 @@
   "id": "end-to-end-golden-path",
   "title": "End-to-End Golden Path",
   "summary": "Deterministic one-worker workflow app for proving the CW integration chain.",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "author": "COOLWHITE LLC",
   "inputs": [
     {

package/apps/pr-review-fix-ci/app.json

@@ -3,7 +3,7 @@
   "id": "pr-review-fix-ci",
   "title": "PR Review Fix CI",
   "summary": "Review a pull request or branch, inspect CI failures, diagnose actionable issues, optionally patch, verify, and summarize with evidence.",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "author": "COOLWHITE LLC",
   "inputs": [
     {

package/apps/release-cut/app.json

@@ -3,7 +3,7 @@
   "id": "release-cut",
   "title": "Release Cut",
   "summary": "Prepare a release with checklist discipline: version checks, changelog, tests, packaging, release notes, and final verification.",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "author": "COOLWHITE LLC",
   "inputs": [
     {

package/apps/research-synthesis/app.json

@@ -3,7 +3,7 @@
   "id": "research-synthesis",
   "title": "Research Synthesis",
   "summary": "Split a research question into claims, investigate sources, cross-check evidence, verify claims, and synthesize a concise answer.",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "author": "COOLWHITE LLC",
   "inputs": [
     {

package/dist/cli/dispatch.js

@@ -27,7 +27,6 @@ const capability_table_1 = require("../core/capability-table");
 const parseargv_1 = require("./parseargv");
 Object.defineProperty(exports, "KNOWN_COMMANDS", { enumerable: true, get: function () { return parseargv_1.KNOWN_COMMANDS; } });
 const io_1 = require("./io");
-const workflow_app_loader_1 = require("../shell/workflow-app-loader");
 function firstPositional(args, index = 0) {
     return args.positionals[index];
 }
@@ -55,8 +54,14 @@ function renderCliResult(result, jsonMode, options) {
 /** Tries the capability table for `args`. Matches the row whose
  *  `cli.path` is `[command]` or `[command, positionals[0]]` (the only two
  *  path lengths any milestone-2 row uses); returns true when a table row
- *  handled the command. */
-function dispatchTable(args) {
+ *  handled the command. `await`ing the handler's result is a no-op for
+ *  the ~199 rows whose handler returns a plain CliHandlerResult (an
+ *  `await` on a non-Promise value just resolves on the next microtask,
+ *  invisible to any caller that itself awaits dispatch()) -- it only
+ *  matters for the live drive rows, which return a real Promise so their
+ *  round loop can actually stay interruptible (see shell/drive.ts's
+ *  driveAsync). */
+async function dispatchTable(args) {
     const candidates = [[args.command]];
     if (args.positionals.length > 0)
         candidates.push([args.command, args.positionals[0]]);
@@ -69,32 +74,19 @@ function dispatchTable(args) {
             positionals: args.positionals.slice(consumed),
             options: args.options,
         };
-        const result = row.cli.handler(cliArgs);
+        const result = await row.cli.handler(cliArgs);
         renderCliResult(result, row.cli.jsonMode, args.options);
         return true;
     }
     return false;
 }
-// MILESTONE 12 (workflow-apps) — `search` filters the SAME real app
-// discovery `cw app list`/`cw list` use (shell/workflow-app-loader.ts),
-// by id/title/summary, matching `listApps` in the old build.
-function formatSearchResults(keyword, results) {
-    if (results.length === 0) {
-        return `No workflows matched "${keyword}".\n  Tip: cw list for all available workflows.`;
-    }
-    const lines = [`${results.length} workflow${results.length === 1 ? "" : "s"} matching "${keyword}"`];
-    for (const r of results) {
-        lines.push(`  ${r.id} — ${r.title}`);
-        const cut = r.summary.length > 120 ? `${r.summary.slice(0, 119)}…` : r.summary;
-        lines.push(`    ${cut}`);
-    }
-    lines.push("");
-    lines.push("Use cw info <id> for full details.");
-    return lines.join("\n");
-}
 /** The milestone-1 carry-over switch. See file header: never extended
  *  again — each arm here is replaced by a capability-table row when its
- *  own build-order milestone lands, not edited in place. */
+ *  own build-order milestone lands, not edited in place. Only entry-level
+ *  concerns live here now (bare help, and the two verbs a real
+ *  capability-table row can never fully own): every other verb this
+ *  switch used to carry is now a table row (see the NOTEs below) and
+ *  dispatchTable() above always matches it first. */
 function dispatchLegacy(args) {
     switch (args.command) {
         case "": {
@@ -114,38 +106,15 @@ function dispatchLegacy(args) {
         // NOTE: "version" is not an arm here — it is a real capability-table
         // row (core/capability-table.ts) that dispatchTable() above always
         // matches first, per the Revision note's "table rows, never a new
-        // switch arm" rule. Same for "list", "status", and "sandbox list".
-        // `search` filters the real app discovery by title/summary/id (see
-        // note above); MILESTONE 12.
-        case "search": {
-            const keyword = args.positionals.join(" ");
-            if (!keyword.trim()) {
-                throw new Error('Missing search keyword.\n  Tip: cw search architecture to find workflows about architecture.');
-            }
-            const lower = keyword.toLowerCase();
-            const results = (0, workflow_app_loader_1.listWorkflowApps)()
-                .filter((a) => String(a.title).toLowerCase().includes(lower) ||
-                String(a.summary).toLowerCase().includes(lower) ||
-                String(a.id).toLowerCase().includes(lower))
-                .map((a) => ({ id: String(a.id), title: String(a.title), summary: String(a.summary) }));
-            if ((0, io_1.wantsJson)(args.options)) {
-                (0, io_1.printJson)(results);
-            }
-            else {
-                process.stdout.write(`${formatSearchResults(keyword, results)}\n`);
-            }
-            return;
-        }
+        // switch arm" rule. Same for "list", "status", "sandbox list", and
+        // "search" (a cli-only row, hiddenFromHelp, so `cw help search` keeps
+        // its existing "Unknown command" text).
         // NOTE: "plan" and "quickstart" are not arms here any more — both are
         // now real capability-table rows (core/capability-table.ts, milestone
         // 6+7) that dispatchTable() above always matches first.
-        // PLACEHOLDER (milestone 3/6, state kernel + pipeline) — real `next`
-        // loads run state and returns dispatchable tasks; this milestone only
-        // reproduces the io.required missing-run-id refusal.
-        case "next": {
-            const runId = (0, io_1.required)((0, io_1.optionalArg)(firstPositional(args)), "run id");
-            throw new Error(`next is not implemented in this milestone (runId=${runId})`);
-        }
+        // NOTE: "next" is not an arm here any more — it is a real
+        // capability-table row (core/capability-table.ts) that dispatchTable()
+        // above always matches first.
         // MILESTONE 8 — `ledger propose|review|verify|apply|list` are now
         // real capability-table rows (core/capability-table.ts) that
         // dispatchTable() above always matches first; this arm is reached
@@ -157,30 +126,9 @@ function dispatchLegacy(args) {
         case "ledger": {
             throw new Error("Usage: cw ledger propose|review|verify|apply|list [options]");
         }
-        // PLACEHOLDER (milestone 10, scheduling/gc) — real `gc verify` checks
-        // whether a run's disk footprint was actually reclaimed; a run that
-        // was never reclaimed is not a failure (exit 0), which is exactly the
-        // shape this stub reproduces for an unresolvable run id.
-        case "gc": {
-            const sub = firstPositional(args);
-            if (sub === "verify") {
-                const runId = (0, io_1.optionalArg)(firstPositional(args, 1));
-                const payload = {
-                    schemaVersion: 1,
-                    runId: runId ?? null,
-                    reclaimed: false,
-                    verified: false,
-                    tier: "live",
-                    capability: "re-runnable",
-                    chainLength: 0,
-                    checks: [{ name: "located", pass: false, code: "not-reclaimed", detail: "run source not found" }],
-                    nextAction: "node scripts/cw.js registry refresh --scope home",
-                };
-                (0, io_1.printJson)(payload);
-                return;
-            }
-            throw new Error(`gc ${sub ?? ""} is not implemented in this milestone`);
-        }
+        // NOTE: "gc" is not an arm here any more — gc.plan/gc.run/gc.verify
+        // are now real capability-table rows (core/capability-table.ts) that
+        // dispatchTable() above always matches first.
         // NOTE: "run" is not an arm here any more — run.drive.step/run.drive
         // are now real capability-table rows (core/capability-table.ts,
         // milestone 6+7) that dispatchTable() above always matches first
@@ -196,20 +144,10 @@ function dispatchLegacy(args) {
         // (core/capability-table.ts, milestone 9) that dispatchTable() above
         // always matches first, per the Revision note's "table rows, never a
         // new switch arm" rule.
-        // PLACEHOLDER (milestone 3/4, state kernel + contract-migration) —
-        // real `migration check`/`prove` resolve a run id or file target; the
-        // missing-target refusal is the only shape this milestone reproduces.
-        case "migration": {
-            const sub = firstPositional(args);
-            if (sub === "check" || sub === "prove") {
-                const target = (0, io_1.optionalArg)(firstPositional(args, 1));
-                if (!target) {
-                    throw new Error('Missing target (run-id or state/app file).\n  Tip: find run ids with "cw run list" or create one with "cw quickstart"');
-                }
-                throw new Error(`migration ${sub} is not implemented in this milestone`);
-            }
-            throw new Error(`migration ${sub ?? ""} is not implemented in this milestone`);
-        }
+        // NOTE: "migration" is not an arm here any more — migration.list/
+        // check/prove are now real capability-table rows
+        // (core/capability-table.ts) that dispatchTable() above always
+        // matches first.
         // NOTE: "report" is not an arm here any more — report/report.bundle/
         // report.verify-bundle are now real capability-table rows
         // (core/capability-table.ts, milestones 8/11) that dispatchTable()
@@ -229,8 +167,8 @@ function dispatchLegacy(args) {
  *
  *  Tries the capability table FIRST (real rows always win), then falls
  *  back to the milestone-1 legacy switch for verbs not yet migrated. */
-function dispatch(args) {
-    if (dispatchTable(args))
+async function dispatch(args) {
+    if (await dispatchTable(args))
         return;
     dispatchLegacy(args);
 }

package/dist/cli/entry.js

@@ -33,6 +33,9 @@ const io_1 = require("./io");
  *  comes from core/format/help.ts either way. */
 function printVersion() {
     const row = (0, capability_table_1.findCapability)("version");
+    // `version`'s own handler is a pure, synchronous projection (never one of
+    // the live drive rows) -- safe to read `.text` straight off it rather than
+    // route through the generic (possibly-async) dispatch path.
     const result = row?.cli?.handler({ positionals: [], options: {} });
     process.stdout.write(result?.text ?? "");
 }
@@ -88,6 +91,21 @@ async function runCli(argv = process.argv.slice(2)) {
         process.env.CW_NO_COLOR = "1";
     if (args.options.full)
         process.env.CW_OUTPUT = "full";
+    // --quiet is a documented CLI spelling of the existing CW_DRIVE_PROGRESS=0
+    // env var (shell/drive.ts's emitProgress) — it only silences the terse
+    // "[drive] " progress lines, not the end-of-run summary or any other
+    // Rule of Silence gate point (SPEC/reporting-ux.md's 3 gate points are
+    // each independent; --verbose/--full don't touch them either). It also
+    // does NOT reach two OTHER TTY-gated narration channels a user might
+    // reasonably expect it to quiet: live raw agent stderr streaming
+    // (shell/execution-backend/agent.ts's shouldStreamAgentStderr, gated by
+    // CW_AGENT_STREAM/CW_NO_STREAM/isTTY) and the local backend's
+    // "● Running…"/"✓ Done" lines (shell/execution-backend/local.ts, gated
+    // by isTTY with no env override at all). Folding those in is a larger,
+    // separate change — this flag's help text says "not agent output" on
+    // purpose so the scope is clear without reading this comment.
+    if (args.options.quiet)
+        process.env.CW_DRIVE_PROGRESS = "0";
     // `cw <verb> --help` / `-h` -> per-command help
     // (src/cli/command-surface.ts:80-83).
     if ((args.options.help || args.options.h) && args.command && !args.command.startsWith("-")) {
@@ -103,13 +121,35 @@ async function runCli(argv = process.argv.slice(2)) {
     else if (!args.command && typeof args.options.question === "string") {
         args.command = "quickstart";
     }
-    (0, dispatch_1.dispatch)(args);
+    await (0, dispatch_1.dispatch)(args);
+}
+/** Broken pipe (`cw ... --json | head`): when the reader at the other end
+ *  of a pipe goes away early, a write to stdout gives an async 'error'
+ *  event that no promise .catch can see — without a listener node comes
+ *  down hard with a stack trace. Raw writes are all over the CLI (help,
+ *  version, printJson, human text), so ONE process-level listener here
+ *  covers every write point: EPIPE says "the reader has all it needs",
+ *  so stop quietly with code 0. Any other stream error is thrown again
+ *  and still comes up as an uncaughtException, same as before. */
+function exitQuietOnEpipe(stream) {
+    stream.on("error", (error) => {
+        if (error && error.code === "EPIPE")
+            process.exit(0);
+        throw error;
+    });
 }
 /** Top-level run wrapper matching src/cli.ts's catch shape byte-for-byte:
  *  `cw: <message>\n` then, only when a hint matches, `  Try: <hint>\n` on
  *  stderr; `process.exitCode = 1` (never a hard `process.exit`). */
 function main(argv = process.argv.slice(2)) {
+    exitQuietOnEpipe(process.stdout);
+    exitQuietOnEpipe(process.stderr);
     return runCli(argv).catch((error) => {
+        // On some platforms a broken-pipe write throws EPIPE in the write call
+        // itself (not as a stream 'error' event) — same broken pipe, same quiet
+        // exit 0, never a `cw: write EPIPE` line.
+        if (error?.code === "EPIPE")
+            return;
         const message = error instanceof Error ? error.message : String(error);
         process.stderr.write(`${(0, term_1.bold)("cw:", process.stderr)} ${(0, term_1.red)(message, process.stderr)}\n`);
         const hint = recoveryHint(message);

package/dist/cli/io.js

@@ -16,6 +16,7 @@ exports.optionalArg = optionalArg;
 exports.printJson = printJson;
 exports.wantsJson = wantsJson;
 const help_1 = require("../core/format/help");
+const safe_json_1 = require("../core/format/safe-json");
 const term_1 = require("../shell/term");
 /** Bold ONLY the fixed "Cool Workflow" header line of `formatHelp()`'s
  *  plain text. */
@@ -34,9 +35,12 @@ function required(value, label) {
 function optionalArg(value) {
     return typeof value === "string" && value.trim() ? value.trim() : undefined;
 }
-/** Machine payload to stdout (stdout = data; never colored, never chrome). */
+/** Machine payload to stdout (stdout = data; never colored, never chrome).
+ *  Byte-capped via safeJsonStringify — an aggregate result too large to be
+ *  useful (or large enough to blow V8's string limit) prints a small
+ *  overflow notice instead of hundreds of MB. */
 function printJson(value) {
-    process.stdout.write(`${JSON.stringify(value, null, 2)}\n`);
+    process.stdout.write(`${(0, safe_json_1.safeJsonStringify)(value)}\n`);
 }
 /** True when the caller asked for JSON output (`--json` or `--format json`). */
 function wantsJson(options) {

package/dist/cli/parseargv.js

@@ -103,7 +103,10 @@ function parseArgv(argv) {
 /** src/orchestrator.ts:842-851 — every top-level command name, used for
  *  "did you mean". NOTE: this deliberately does NOT include "ledger" even
  *  though the dispatcher handles it and formatHelp lists it — a known,
- *  intentionally-preserved wart (see docs/rebuild/PLAN.md "Kept byte-for-byte"). */
+ *  intentionally-preserved wart (see docs/rebuild/PLAN.md "Kept byte-for-byte").
+ *  NOTE: "update" is gone from the old capture's list on purpose: no code
+ *  was behind the verb in this build, so having it here made `cw update`
+ *  say "Did you mean: update?" — a hint that points at itself. */
 exports.KNOWN_COMMANDS = new Set([
     "help", "list", "doctor", "info", "search", "man", "init", "quickstart",
     "plan", "status", "next", "dispatch", "result", "state", "commit", "report",
@@ -112,7 +115,8 @@ exports.KNOWN_COMMANDS = new Set([
     "run", "queue", "clones", "orphans", "history", "audit-run", "multi-agent",
     "topology", "summary", "blackboard", "coordinator", "metrics", "operator",
     "sched", "gc", "telemetry", "migration", "demo", "workbench", "approve",
-    "reject", "comment", "handoff", "graph", "eval", "version", "update", "fix",
+    "reject", "comment", "handoff", "graph", "eval", "version", "fix",
+    "completion",
 ]);
 /** Levenshtein edit distance between two strings. */
 function levenshtein(a, b) {