cool-workflow 0.2.1 → 0.2.2

package/scripts/version-sync-check.js

@@ -20,7 +20,7 @@ const repoRoot = path.resolve(pluginRoot, "..", "..");
 // CI checks out HEAD, so HEAD === the tree it is releasing.
 //
 // Fallback to the filesystem only when we cannot read from HEAD: not a git work
-// tree, or the path is not tracked at HEAD (e.g. the gitignored package-lock).
+// tree, or the path is not tracked at HEAD.
 // node + git only — no ripgrep (CI portability rule).
 const insideGitWorkTree = (() => {
   const r = spawnSync("git", ["rev-parse", "--is-inside-work-tree"], { cwd: repoRoot, encoding: "utf8" });
@@ -36,7 +36,7 @@ function readReleaseSource(relativePath) {
       maxBuffer: 1024 * 1024 * 32
     });
     if (r.status === 0) return { text: r.stdout, exists: true, fromHead: true };
-    // Not tracked at HEAD — fall through to the working tree (e.g. package-lock).
+    // Not tracked at HEAD — fall through to the working tree.
   }
   const abs = path.join(repoRoot, relativePath);
   if (!fs.existsSync(abs)) return { text: null, exists: false, fromHead: false };
@@ -56,9 +56,14 @@ const canonicalApps = CANONICAL_APP_IDS;
 function main() {
   const checks = [];
   checkJson("plugins/cool-workflow/package.json", "version", VERSION, checks);
-  // package-lock.json is a gitignored install artifact (the documented install
-  // uses `npm install --no-package-lock`), so only validate it when present.
+  // package-lock.json is tracked now, but the documented install still works
+  // without it (`npm install --no-package-lock`), so only validate it when
+  // present. Check BOTH version fields: the top-level one and the root-package
+  // entry packages[""].version — the second one is the field npm keeps in step
+  // with package.json on `npm install`, and it is the one that went stale
+  // (0.1.97) through the v0.2.0/v0.2.1 cuts while only the first was checked.
   checkJsonIfPresent("plugins/cool-workflow/package-lock.json", "version", VERSION, checks);
+  checkNestedJsonIfPresent("plugins/cool-workflow/package-lock.json", ["packages", "", "version"], VERSION, checks);
   checkJson("plugins/cool-workflow/.codex-plugin/plugin.json", "version", VERSION, checks);
   checkJson("plugins/cool-workflow/server.json", "version", VERSION, checks);
   checkNestedJson("plugins/cool-workflow/server.json", ["packages", 0, "version"], VERSION, checks);
@@ -160,7 +165,7 @@ function main() {
   checkIncludes("plugins/cool-workflow/docs/run-retention-reclamation.7.md", VERSION, checks);
   checkIncludes("plugins/cool-workflow/docs/index.md", "run-retention-reclamation.7.md", checks);
   checkIncludes("plugins/cool-workflow/test/run-retention-reclamation-smoke.js", "run-retention-reclamation-smoke", checks);
-  checkIncludes("plugins/cool-workflow/src/core/capability-table.ts", "gc.plan", checks);
+  checkIncludes("plugins/cool-workflow/src/wiring/capability-table/scheduling-registry.ts", "gc.plan", checks);
   checkIncludes("plugins/cool-workflow/docs/durable-state-and-locking.7.md", "Durable State & Locking", checks);
   checkIncludes("plugins/cool-workflow/docs/durable-state-and-locking.7.md", VERSION, checks);
   checkIncludes("plugins/cool-workflow/docs/index.md", "durable-state-and-locking.7.md", checks);
@@ -168,33 +173,37 @@ function main() {
   checkIncludes("plugins/cool-workflow/src/shell/fs-atomic.ts", "withFileLock", checks);
   checkIncludes("plugins/cool-workflow/src/shell/drive.ts", "driveStep", checks);
   checkIncludes("plugins/cool-workflow/dist/shell/drive.js", "driveStep", checks);
-  checkIncludes("plugins/cool-workflow/src/core/capability-table.ts", "run.drive", checks);
+  checkIncludes("plugins/cool-workflow/src/wiring/capability-table/pipeline.ts", "run.drive", checks);
   checkIncludes("plugins/cool-workflow/src/shell/collaboration-io.ts", "deriveReviewState", checks);
   checkIncludes("plugins/cool-workflow/dist/shell/collaboration-io.js", "deriveReviewState", checks);
-  checkIncludes("plugins/cool-workflow/src/core/capability-table.ts", "review.status", checks);
+  checkIncludes("plugins/cool-workflow/src/wiring/capability-table/multi-agent.ts", "review.status", checks);
   checkIncludes("plugins/cool-workflow/src/shell/observability.ts", "deriveMetricsReport", checks);
   checkIncludes("plugins/cool-workflow/dist/shell/observability.js", "deriveMetricsReport", checks);
-  checkIncludes("plugins/cool-workflow/src/core/capability-table.ts", "metrics.show", checks);
+  checkIncludes("plugins/cool-workflow/src/wiring/capability-table/reporting.ts", "metrics.show", checks);
   checkIncludes("plugins/cool-workflow/manifest/pricing.policy.json", "schemaVersion", checks);
   checkIncludes("plugins/cool-workflow/src/shell/workbench.ts", "buildWorkbenchRunView", checks);
   checkIncludes("plugins/cool-workflow/dist/shell/workbench.js", "buildWorkbenchRunView", checks);
-  checkIncludes("plugins/cool-workflow/src/core/capability-table.ts", "workbench.view", checks);
+  checkIncludes("plugins/cool-workflow/src/wiring/capability-table/reporting.ts", "workbench.view", checks);
   checkIncludes("plugins/cool-workflow/src/shell/execution-backend/registry.ts", "ExecutionBackend", checks);
   checkIncludes("plugins/cool-workflow/dist/shell/execution-backend/registry.js", "ExecutionBackend", checks);
-  checkIncludes("plugins/cool-workflow/src/core/capability-table.ts", "backend.list", checks);
+  checkIncludes("plugins/cool-workflow/src/wiring/capability-table/exec-backend.ts", "backend.list", checks);
   checkIncludes("plugins/cool-workflow/src/shell/run-registry-io.ts", "RunRegistry", checks);
   checkIncludes("plugins/cool-workflow/dist/shell/run-registry-io.js", "RunRegistry", checks);
-  checkIncludes("plugins/cool-workflow/src/core/capability-table.ts", "registry.refresh", checks);
+  checkIncludes("plugins/cool-workflow/src/wiring/capability-table/scheduling-registry.ts", "registry.refresh", checks);
   checkIncludes("plugins/cool-workflow/package.json", "parity:check", checks);
   checkIncludes("plugins/cool-workflow/scripts/parity-check.js", "buildParityReport", checks);
   checkIncludes("plugins/cool-workflow/test/cli-mcp-parity-smoke.js", "cli-mcp-parity-smoke", checks);
-  checkIncludes("plugins/cool-workflow/src/core/capability-table.ts", "export const REGISTRY", checks);
+  checkIncludes("plugins/cool-workflow/src/wiring/capability-table/registry-core.ts", "export const REGISTRY", checks);
   checkIncludes("plugins/cool-workflow/src/shell/pipeline-cli.ts", "planSummary", checks);
   checkIncludes("plugins/cool-workflow/dist/core/capability-table.js", "REGISTRY", checks);
   checkIncludes("plugins/cool-workflow/docs/multi-agent-runtime-core.7.md", "Multi-Agent Runtime Core", checks);
   checkIncludes("plugins/cool-workflow/docs/dogfood-one-real-repo.7.md", "Dogfood One Real Repo", checks);
   checkIncludes("plugins/cool-workflow/docs/getting-started.md", "npm run release:check", checks);
   checkIncludes("plugins/cool-workflow/package.json", "eval:replay", checks);
+  checkIncludes("plugins/cool-workflow/docs/trust-audit-anchor.7.md", "Trust Audit Anchor", checks);
+  checkIncludes("plugins/cool-workflow/docs/trust-audit-anchor.7.md", VERSION, checks);
+  checkIncludes("plugins/cool-workflow/docs/index.md", "trust-audit-anchor.7.md", checks);
+  checkIncludes("plugins/cool-workflow/test/trust-audit-anchor-smoke.js", "trust-audit-anchor-smoke", checks);
   checkIncludes("plugins/cool-workflow/docs/release-and-migration.7.md", VERSION, checks);
   checkIncludes("CHANGELOG.md", `## ${VERSION}`, checks);
   checkIncludes("RELEASE.md", VERSION, checks);
@@ -227,6 +236,18 @@ function checkNestedJson(relativePath, keyPath, expected, checks) {
   checks.push({ path: relativePath, key: keyPath.join("."), value });
 }
 
+function checkNestedJsonIfPresent(relativePath, keyPath, expected, checks) {
+  const src = readReleaseSource(relativePath);
+  if (!src.exists) {
+    checks.push({ path: relativePath, key: keyPath.join("."), skipped: "absent" });
+    return;
+  }
+  let value = JSON.parse(src.text);
+  for (const key of keyPath) value = value?.[key];
+  assert.equal(value, expected, `${relativePath}.${keyPath.join(".")} must be ${expected}`);
+  checks.push({ path: relativePath, key: keyPath.join("."), value });
+}
+
 function checkJsonIfPresent(relativePath, key, expected, checks) {
   const src = readReleaseSource(relativePath);
   if (!src.exists) {

package/workflows/README.md

@@ -0,0 +1,19 @@
+# workflows/ — legacy compatibility surface (pinned, do not remove)
+
+`architecture-review.workflow.js` and `research-synthesis.workflow.js` are
+the old, pre-`apps/` workflow-file format. The real, current homes for
+these two workflows are `apps/architecture-review/` and
+`apps/research-synthesis/` — these files exist ONLY so an id from before
+the `apps/` format still resolves.
+
+This is deliberate duplication, not drift to clean up: each file here
+registers its OWN distinct id (`legacy-architecture-review`,
+`legacy-research-synthesis`), pinned by
+`v2/conformance/cases/multiagent-app-list.case.js` and
+`plugins/cool-workflow/test/workflow-app-framework-smoke.js`. `cw list` /
+`cw app list` show both the real app and its legacy id side by side
+(`src/shell/workflow-app-loader.ts` discovers both roots).
+
+Per this project's own POLA rule, removing or thinning either file would
+change `cw list`'s output bytes — that is a breaking change, only doable
+behind a major-version break, not a routine cleanup.