@plainconceptsplatform/workflows 0.4.34 → 0.6.1

package/dist/route-processing.test.js

@@ -1,197 +1,168 @@
 import { describe, expect, it } from "vitest";
 import { addRouteExclusion, excludedWorkerFiles, processRoutes, stripRouteFromClassifier, stripRouteFromRouter, } from "./route-processing.js";
 import { routeNames } from "./workflow-catalog.js";
-const ROUTER_YAML = `# header
-name: "All Work Router"
-
-on:
-  schedule:
-    - cron: "17 1 * * 1"
-    - cron: "43 3 * * *"
-    - cron: "0 6 * * *"
-    - cron: "*/5 * * * *"
-    - cron: "0 */2 * * *"
-    - cron: "29 7 * * *"
-
-  workflow_dispatch:
-    inputs:
-      operation:
-        description: "Operation to run"
-        required: true
-        type: choice
-        options:
-          - refine
-          - implement
-          - direct
-          - triage
-          - apply-review
-          - merge-gate
-          - audit
-          - propose
-          - audit-close
-          - cleanup-artifacts
-          - reconcile-bot-pr-runs
-          - stale-recovery
-          - validate
-
-jobs:
-  call-refine:
-    needs: [classify, authorize]
-    if: needs.classify.outputs.route == 'refine' && needs.authorize.outputs.trusted == 'true'
-    uses: ./.github/workflows/agent-refine.lock.yml
-    secrets:
-      OPENAI_API_KEY: \${{ secrets.OPENAI_API_KEY }}
-
-  call-implement:
-    needs: [classify, authorize]
-    if: needs.classify.outputs.route == 'implement' && needs.authorize.outputs.trusted == 'true'
-    uses: ./.github/workflows/agent-implement.lock.yml
-    secrets:
-      OPENAI_API_KEY: \${{ secrets.OPENAI_API_KEY }}
-
-  call-triage:
-    needs: [classify, authorize]
-    if: needs.classify.outputs.route == 'triage' && needs.authorize.outputs.is_outside_collaborator == 'true'
-    uses: ./.github/workflows/agent-triage.lock.yml
-    secrets:
-      OPENAI_API_KEY: \${{ secrets.OPENAI_API_KEY }}
-
-  call-audit:
-    needs: classify
-    if: needs.classify.outputs.route == 'audit'
-    uses: ./.github/workflows/agent-audit.lock.yml
-    secrets:
-      OPENAI_API_KEY: \${{ secrets.OPENAI_API_KEY }}
-
-  call-propose:
-    needs: classify
-    if: needs.classify.outputs.route == 'propose'
-    uses: ./.github/workflows/agent-propose.lock.yml
-    secrets:
-      OPENAI_API_KEY: \${{ secrets.OPENAI_API_KEY }}
-
-  audit-close:
-    needs: classify
-    if: needs.classify.outputs.route == 'audit-close'
-    runs-on: ubuntu-latest
+const ROUTER_YAML = `# header
+name: "All Work Router"
+
+on:
+  schedule:
+    - cron: "17 1 * * 1"
+    - cron: "43 3 * * *"
+    - cron: "0 6 * * *"
+    - cron: "*/5 * * * *"
+    - cron: "0 */2 * * *"
+
+  workflow_dispatch:
+    inputs:
+      operation:
+        description: "Operation to run"
+        required: true
+        type: choice
+        options:
+          - refine
+          - implement
+          - triage
+          - apply-review
+          - merge-gate
+          - audit
+          - audit-close
+          - cleanup-artifacts
+          - reconcile-bot-pr-runs
+          - stale-recovery
+          - validate
+
+jobs:
+  call-refine:
+    needs: [classify, authorize]
+    if: needs.classify.outputs.route == 'refine' && needs.authorize.outputs.trusted == 'true'
+    uses: ./.github/workflows/agent-refine.lock.yml
+    secrets:
+      OPENAI_API_KEY: \${{ secrets.OPENAI_API_KEY }}
+
+  call-implement:
+    needs: [classify, authorize]
+    if: needs.classify.outputs.route == 'implement' && needs.authorize.outputs.trusted == 'true'
+    uses: ./.github/workflows/agent-implement.lock.yml
+    secrets:
+      OPENAI_API_KEY: \${{ secrets.OPENAI_API_KEY }}
+
+  call-triage:
+    needs: [classify, authorize]
+    if: needs.classify.outputs.route == 'triage' && needs.authorize.outputs.is_outside_collaborator == 'true'
+    uses: ./.github/workflows/agent-triage.lock.yml
+    secrets:
+      OPENAI_API_KEY: \${{ secrets.OPENAI_API_KEY }}
+
+  call-audit:
+    needs: classify
+    if: needs.classify.outputs.route == 'audit'
+    uses: ./.github/workflows/agent-audit.lock.yml
+    secrets:
+      OPENAI_API_KEY: \${{ secrets.OPENAI_API_KEY }}
+
+  audit-close:
+    needs: classify
+    if: needs.classify.outputs.route == 'audit-close'
+    runs-on: ubuntu-latest
 `;
-const CLASSIFIER_SH = `#!/usr/bin/env bash
-set -euo pipefail
-
-readonly AUDIT_CRON="17 1 * * 1"
-readonly AUDIT_CLOSE_CRON="43 3 * * *"
-readonly CLEANUP_ARTIFACTS_CRON="0 6 * * *"
-readonly RECONCILE_BOT_PR_RUNS_CRON="*/5 * * * *"
-readonly STALE_RECOVERY_CRON="0 */2 * * *"
-readonly PROPOSE_CRON="29 7 * * *"
-
-classify_route() {
-  local route="none" error=""
-
-  case "\${EVENT:-}" in
-    schedule)
-      trigger_kind="scheduled"
-      case "\${SCHEDULE:-}" in
-        "\$AUDIT_CRON") route="audit" ;;
-        "\$AUDIT_CLOSE_CRON") route="audit-close" ;;
-        "\$CLEANUP_ARTIFACTS_CRON") route="cleanup-artifacts" ;;
-        "\$RECONCILE_BOT_PR_RUNS_CRON") route="reconcile-bot-pr-runs" ;;
-        "\$STALE_RECOVERY_CRON") route="stale-recovery" ;;
-        "\$PROPOSE_CRON") route="propose" ;;
-        *) error="no route for cron '\${SCHEDULE:-}'" ;;
-      esac
-      ;;
-
-    workflow_dispatch)
-      trigger_kind="manual"
-      case "\${OPERATION:-}" in
-        refine | implement | direct)
-          route="\${OPERATION}"
-          ;;
-        apply-review)
-          route="apply-review"
-          ;;
-        merge-gate)
-          route="merge-gate"
-          ;;
-        audit | propose)
-          route="\${OPERATION}"
-          trigger_kind="\${INPUT_TRIGGER_KIND:-manual}"
-          ;;
-        audit-close | cleanup-artifacts | reconcile-bot-pr-runs | stale-recovery | validate)
-          route="\${OPERATION}"
-          ;;
-        *)
-          error="unknown operation '\${OPERATION:-}'"
-          ;;
-      esac
-      ;;
-  esac
-
-  cat <<EOF
-route=\${route}
-error=\${error}
-EOF
-}
-
-if [ "\${BASH_SOURCE[0]}" = "\$0" ]; then
-  classify_route
-fi
+const CLASSIFIER_SH = `#!/usr/bin/env bash
+set -euo pipefail
+
+readonly AUDIT_CRON="17 1 * * 1"
+readonly AUDIT_CLOSE_CRON="43 3 * * *"
+readonly CLEANUP_ARTIFACTS_CRON="0 6 * * *"
+readonly RECONCILE_BOT_PR_RUNS_CRON="*/5 * * * *"
+readonly STALE_RECOVERY_CRON="0 */2 * * *"
+
+classify_route() {
+  local route="none" error=""
+
+  case "\${EVENT:-}" in
+    schedule)
+      trigger_kind="scheduled"
+      case "\${SCHEDULE:-}" in
+        "\$AUDIT_CRON") route="audit" ;;
+        "\$AUDIT_CLOSE_CRON") route="audit-close" ;;
+        "\$CLEANUP_ARTIFACTS_CRON") route="cleanup-artifacts" ;;
+        "\$RECONCILE_BOT_PR_RUNS_CRON") route="reconcile-bot-pr-runs" ;;
+        "\$STALE_RECOVERY_CRON") route="stale-recovery" ;;
+        *) error="no route for cron '\${SCHEDULE:-}'" ;;
+      esac
+      ;;
+
+    workflow_dispatch)
+      trigger_kind="manual"
+      case "\${OPERATION:-}" in
+        refine | implement)
+          route="\${OPERATION}"
+          ;;
+        apply-review)
+          route="apply-review"
+          ;;
+        merge-gate)
+          route="merge-gate"
+          ;;
+        audit)
+          route="\${OPERATION}"
+          trigger_kind="\${INPUT_TRIGGER_KIND:-manual}"
+          ;;
+        audit-close | cleanup-artifacts | reconcile-bot-pr-runs | stale-recovery | validate)
+          route="\${OPERATION}"
+          ;;
+        *)
+          error="unknown operation '\${OPERATION:-}'"
+          ;;
+      esac
+      ;;
+  esac
+
+  cat <<EOF
+route=\${route}
+error=\${error}
+EOF
+}
+
+if [ "\${BASH_SOURCE[0]}" = "\$0" ]; then
+  classify_route
+fi
 `;
-const MATRIX_SH = `#!/usr/bin/env bash
-set -euo pipefail
-
-echo "── Router wiring ─────────────────────────────────────────────────────────"
-for route in refine implement direct triage apply-review merge-gate audit propose bot-approve \\
-  audit-close cleanup-artifacts reconcile-bot-pr-runs stale-recovery validate; do
-  if grep -q "route == '\${route}'" "\$ROUTER_YML"; then
-    PASS=\$((PASS + 1))
-  else
-    FAIL=\$((FAIL + 1))
-    echo "FAIL: work-router.yml has no job for route '\${route}'" >&2
-  fi
-done
-
-while read -r operation; do
-  if grep -q "route == '\${operation}'" "\$ROUTER_YML"; then
-    PASS=\$((PASS + 1))
-  else
-    FAIL=\$((FAIL + 1))
-    echo "FAIL: dispatch operation '\${operation}' has no job in work-router.yml" >&2
-  fi
-done < <(sed -n '/^      operation:/,/^      issue-number:/p' "\$ROUTER_YML" | sed -n 's/^          - //p')
-
-echo
-if [ "\$FAIL" -eq 0 ]; then
-  echo "Route matrix: \${PASS} passed"
-else
-  echo "Route matrix: \${PASS} passed, \${FAIL} FAILED" >&2
-fi
-
-exit \$((FAIL > 0))
+const MATRIX_SH = `#!/usr/bin/env bash
+set -euo pipefail
+
+echo "── Router wiring ─────────────────────────────────────────────────────────"
+for route in refine implement triage apply-review merge-gate audit bot-approve \\
+  audit-close cleanup-artifacts reconcile-bot-pr-runs stale-recovery validate; do
+  if grep -q "route == '\${route}'" "\$ROUTER_YML"; then
+    PASS=\$((PASS + 1))
+  else
+    FAIL=\$((FAIL + 1))
+    echo "FAIL: work-router.yml has no job for route '\${route}'" >&2
+  fi
+done
+
+while read -r operation; do
+  if grep -q "route == '\${operation}'" "\$ROUTER_YML"; then
+    PASS=\$((PASS + 1))
+  else
+    FAIL=\$((FAIL + 1))
+    echo "FAIL: dispatch operation '\${operation}' has no job in work-router.yml" >&2
+  fi
+done < <(sed -n '/^      operation:/,/^      issue-number:/p' "\$ROUTER_YML" | sed -n 's/^          - //p')
+
+echo
+if [ "\$FAIL" -eq 0 ]; then
+  echo "Route matrix: \${PASS} passed"
+else
+  echo "Route matrix: \${PASS} passed, \${FAIL} FAILED" >&2
+fi
+
+exit \$((FAIL > 0))
 `;
 describe("stripRouteFromRouter", () => {
-    it("removes the propose cron entry", () => {
-        const result = stripRouteFromRouter(ROUTER_YAML, "propose");
-        expect(result).not.toContain('cron: "29 7 * * *"');
-        expect(result).toContain('cron: "17 1 * * 1"');
-    });
-    it("removes the call-propose job block", () => {
-        const result = stripRouteFromRouter(ROUTER_YAML, "propose");
-        expect(result).not.toContain("call-propose");
-        expect(result).toContain("call-refine");
-        expect(result).toContain("call-audit");
-    });
-    it("removes propose from the dispatch options", () => {
-        const result = stripRouteFromRouter(ROUTER_YAML, "propose");
-        expect(result).not.toMatch(/^\s+- propose$/m);
-        expect(result).toMatch(/^\s+- refine$/m);
-        expect(result).toMatch(/^\s+- audit$/m);
-    });
     it("removes the audit cron entry", () => {
         const result = stripRouteFromRouter(ROUTER_YAML, "audit");
         expect(result).not.toContain('cron: "17 1 * * 1"');
-        expect(result).toContain('cron: "29 7 * * *"');
+        expect(result).toContain('cron: "43 3 * * *"');
     });
     it("removes the call-audit job block", () => {
         const result = stripRouteFromRouter(ROUTER_YAML, "audit");
@@ -209,42 +180,27 @@ describe("stripRouteFromRouter", () => {
     });
 });
 describe("stripRouteFromClassifier", () => {
-    it("removes the PROPOSE_CRON constant", () => {
-        const result = stripRouteFromClassifier(CLASSIFIER_SH, "propose");
-        expect(result).not.toContain('readonly PROPOSE_CRON');
-        expect(result).toContain('readonly AUDIT_CRON');
-    });
-    it("removes the propose schedule case", () => {
-        const result = stripRouteFromClassifier(CLASSIFIER_SH, "propose");
-        expect(result).not.toContain('"$PROPOSE_CRON") route="propose"');
-        expect(result).toContain('"$AUDIT_CRON") route="audit"');
-    });
     it("removes the AUDIT_CRON constant", () => {
         const result = stripRouteFromClassifier(CLASSIFIER_SH, "audit");
         expect(result).not.toContain('readonly AUDIT_CRON');
-        expect(result).toContain('readonly PROPOSE_CRON');
+        expect(result).toContain('readonly AUDIT_CLOSE_CRON');
     });
     it("removes the audit schedule case", () => {
         const result = stripRouteFromClassifier(CLASSIFIER_SH, "audit");
         expect(result).not.toContain('"$AUDIT_CRON") route="audit"');
-        expect(result).toContain('"$PROPOSE_CRON") route="propose"');
-    });
-    it("removes propose from the dispatch case union", () => {
-        const result = stripRouteFromClassifier(CLASSIFIER_SH, "propose");
-        expect(result).not.toContain("audit | propose)");
-        expect(result).toContain("audit)");
+        expect(result).toContain('"$AUDIT_CLOSE_CRON") route="audit-close"');
     });
 });
 describe("addRouteExclusion", () => {
     it("adds an exclusion assertion for the route", () => {
-        const result = addRouteExclusion(MATRIX_SH, "propose");
-        expect(result).toContain("excluded route 'propose'");
-        expect(result).toContain("propose correctly excluded from work-router.yml");
+        const result = addRouteExclusion(MATRIX_SH, "audit");
+        expect(result).toContain("excluded route 'audit'");
+        expect(result).toContain("audit correctly excluded from work-router.yml");
     });
     it("does not add the exclusion twice", () => {
-        const once = addRouteExclusion(MATRIX_SH, "propose");
-        const twice = addRouteExclusion(once, "propose");
-        const matchCount = (twice.match(/excluded route 'propose'/g) ?? []).length;
+        const once = addRouteExclusion(MATRIX_SH, "audit");
+        const twice = addRouteExclusion(once, "audit");
+        const matchCount = (twice.match(/excluded route 'audit'/g) ?? []).length;
         expect(matchCount).toBe(1);
     });
 });
@@ -266,33 +222,13 @@ describe("processRoutes", () => {
         ]);
         const result = processRoutes(files, []);
         const router = result.get(".github/workflows/work-router.yml");
-        expect(router).not.toContain("call-propose");
         expect(router).not.toContain("call-audit");
-        expect(router).not.toContain("- propose");
         expect(router).not.toContain("- refine");
         const classifier = result.get(".github/actions/classify-route/classify-route.sh");
         expect(classifier).toBe(CLASSIFIER_SH);
         const matrix = result.get(".github/actions/verify-route-matrix/verify-route-matrix.sh");
         expect(matrix).toContain("Route matrix: selected routes valid");
-        expect(matrix).toContain("for route in refine implement direct triage apply-review merge-gate audit propose");
-    });
-    it("strips propose from all three files when unselected", () => {
-        const files = new Map([
-            [".github/workflows/work-router.yml", ROUTER_YAML],
-            [".github/actions/classify-route/classify-route.sh", CLASSIFIER_SH],
-            [".github/actions/verify-route-matrix/verify-route-matrix.sh", MATRIX_SH],
-        ]);
-        const selectedRoutes = routeNames.filter((r) => r !== "propose");
-        const result = processRoutes(files, selectedRoutes);
-        const router = result.get(".github/workflows/work-router.yml");
-        expect(router).not.toContain("call-propose");
-        expect(router).not.toContain('cron: "29 7 * * *"');
-        expect(router).not.toMatch(/^\s+- propose$/m);
-        const classifier = result.get(".github/actions/classify-route/classify-route.sh");
-        expect(classifier).toBe(CLASSIFIER_SH);
-        const matrix = result.get(".github/actions/verify-route-matrix/verify-route-matrix.sh");
-        expect(matrix).toContain("for route in refine implement direct triage apply-review merge-gate audit");
-        expect(matrix).toContain("for route in propose");
+        expect(matrix).toContain("for route in refine implement triage apply-review merge-gate audit");
     });
     it("strips audit from all three files when unselected", () => {
         const files = new Map([
@@ -320,7 +256,6 @@ describe("processRoutes", () => {
         expect(router).toContain("call-refine");
         expect(router).toContain("call-implement");
         expect(router).not.toContain("call-audit");
-        expect(router).not.toContain("call-propose");
     });
 });
 describe("excludedWorkerFiles", () => {
@@ -330,7 +265,6 @@ describe("excludedWorkerFiles", () => {
         expect(excluded.has("agent-refine.md")).toBe(false);
         expect(excluded.has("agent-implement.md")).toBe(false);
         expect(excluded.has("agent-audit.md")).toBe(true);
-        expect(excluded.has("agent-propose.md")).toBe(true);
     });
     it("returns an empty set when all routes are selected", () => {
         const excluded = excludedWorkerFiles([...routeNames]);
@@ -338,13 +272,12 @@ describe("excludedWorkerFiles", () => {
     });
     it("returns all worker files when no routes are selected", () => {
         const excluded = excludedWorkerFiles([]);
-        expect(excluded.size).toBe(8);
+        expect(excluded.size).toBe(6);
         expect(excluded.has("agent-refine.md")).toBe(true);
         expect(excluded.has("agent-implement.md")).toBe(true);
-        expect(excluded.has("agent-direct.md")).toBe(true);
+        expect(excluded.has("agent-triage.md")).toBe(true);
         expect(excluded.has("agent-apply-review.md")).toBe(true);
         expect(excluded.has("agent-merge-gate.md")).toBe(true);
         expect(excluded.has("agent-audit.md")).toBe(true);
-        expect(excluded.has("agent-propose.md")).toBe(true);
     });
 });

package/dist/stack-defaults.js

@@ -32,24 +32,24 @@ export function generateOpencodeCi(baseContent, inspection) {
     let result = baseContent;
     if (inspection.stackHints.solutionFiles.length > 0) {
         const solutionPath = inspection.stackHints.solutionFiles[0].replaceAll("\\", "/");
-        const nugetSteps = `  - name: Cache NuGet packages
-    uses: actions/cache@55cc8345863c7cc4c66a329aec7e433d2d1c52a9 # v6.1.0
-    with:
-      path: ~/.nuget/packages
-      key: nuget-\${{ runner.os }}-\${{ hashFiles('**/*.slnx', '**/Directory.Packages.props') }}
-      restore-keys: nuget-\${{ runner.os }}-
-
-  - name: Restore .NET dependencies
-    run: dotnet restore ${solutionPath}
+        const nugetSteps = `  - name: Cache NuGet packages
+    uses: actions/cache@55cc8345863c7cc4c66a329aec7e433d2d1c52a9 # v6.1.0
+    with:
+      path: ~/.nuget/packages
+      key: nuget-\${{ runner.os }}-\${{ hashFiles('**/*.slnx', '**/Directory.Packages.props') }}
+      restore-keys: nuget-\${{ runner.os }}-
+
+  - name: Restore .NET dependencies
+    run: dotnet restore ${solutionPath}
 `;
         result = insertBeforeMarker(result, nugetSteps, "  - name: Install workspace dependencies");
     }
     if (inspection.stackHints.openSpec) {
-        const openspecStep = `  - name: Install OpenSpec CLI
-    run: |
-      set -euo pipefail
-      npm install -g "@fission-ai/openspec@1.8.0"
-      openspec --version
+        const openspecStep = `  - name: Install OpenSpec CLI
+    run: |
+      set -euo pipefail
+      npm install -g "@fission-ai/openspec@1.8.0"
+      openspec --version
 `;
         result = insertBeforeMarker(result, openspecStep, "  - name: Install workspace dependencies");
     }