@plainconceptsplatform/workflows 0.28.25 → 1.0.1

package/dist/stack-defaults.js

@@ -31,31 +31,31 @@ export function generateOpencodeCi(baseContent, inspection) {
     if (inspection.stackHints.solutionFiles.length > 0) {
         const solutionPath = inspection.stackHints.solutionFiles[0].replaceAll("\\", "/");
         if (!result.includes("name: Cache NuGet packages")) {
-            const nugetCache = `  - 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 }}-
-
+            const nugetCache = `  - 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 }}-
+
 `;
             result = insertBeforeMarker(result, nugetCache, "  - name: Install workspace dependencies");
         }
         if (!result.includes("name: Restore .NET dependencies")) {
-            const restore = `  - name: Restore .NET dependencies
-    run: dotnet restore ${solutionPath}
-
+            const restore = `  - name: Restore .NET dependencies
+    run: dotnet restore ${solutionPath}
+
 `;
             result = insertBeforeMarker(result, restore, "  - name: Install workspace dependencies");
         }
     }
     if (inspection.stackHints.openSpec && !result.includes("name: Install OpenSpec CLI")) {
-        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");
     }
@@ -74,32 +74,24 @@ export function generateOpencodeConfig(baseContent, inspection) {
     try {
         const config = JSON.parse(baseContent);
         const hasDotnet = inspection.stackHints.solutionFiles.length > 0;
-        if (!hasDotnet && config.lsp !== undefined) {
-            delete config.lsp;
-        }
-        else if (hasDotnet && config.lsp === undefined) {
-            config.lsp = {
-                csharp: { disabled: true },
-                fsharp: { disabled: true },
-                razor: { disabled: true },
-            };
-        }
-        const agent = config.agent;
-        if (agent !== undefined) {
-            const agentEntry = agent["ci-workflow-agent"];
+        // OpenCode 2 names the agent map `agents` and the prompt `system`. V2 does
+        // not run language servers, so there is no lsp block to add or remove.
+        const agents = config.agents;
+        if (agents !== undefined) {
+            const agentEntry = agents["ci-workflow-agent"];
             if (agentEntry !== undefined) {
-                let prompt = agentEntry.prompt ?? "";
+                let system = agentEntry.system ?? "";
                 if (hasDotnet) {
-                    if (!prompt.includes(".NET guardrails")) {
-                        prompt += "\n\n# .NET guardrails\nFollow Clean Architecture layering: API → Application → Domain. Infrastructure implements Application ports. Application must not reference EF Core or ASP.NET. Use Central Package Management (Directory.Packages.props). Build in Release mode for CI.";
+                    if (!system.includes(".NET guardrails")) {
+                        system += "\n\n# .NET guardrails\nFollow Clean Architecture layering: API → Application → Domain. Infrastructure implements Application ports. Application must not reference EF Core or ASP.NET. Use Central Package Management (Directory.Packages.props). Build in Release mode for CI.";
                     }
                 }
                 else {
-                    if (!prompt.includes("Node/React rules")) {
-                        prompt += "\n\n# Node/React rules\nFollow Feature-Sliced Design import boundaries. Use pnpm, never npm or yarn. All user-facing text must be i18n messages. TypeScript strict mode.";
+                    if (!system.includes("Node/React rules")) {
+                        system += "\n\n# Node/React rules\nFollow Feature-Sliced Design import boundaries. Use pnpm, never npm or yarn. All user-facing text must be i18n messages. TypeScript strict mode.";
                     }
                 }
-                agentEntry.prompt = prompt;
+                agentEntry.system = system;
             }
         }
         return JSON.stringify(config, null, 2) + "\n";

package/loops/actions/verify-route-matrix/verify-route-matrix.sh

@@ -2198,15 +2198,15 @@ echo "── Runner pools ──────────────────
 # RunnerLandingZone while merge-gate and release stayed on agents-arc: the override reached the
 # workers installed at the time and never the ones added later. Nothing reported the split.
 #
-#   agent jobs of every worker except release  ->  agents-arc          (the Azure fleet in
-#                                                                       agentrunner-pro-rg-01,
-#                                                                       runner group `agentic`)
-#   gate and post-processing jobs              ->  RunnerLandingZone   (GitHub-hosted, no fleet
-#                                                                       capacity consumed by
-#                                                                       trivial checks)
-#   agent-release.md and work-router.yml       ->  RunnerLandingZone
-#   ubuntu-latest                              ->  GitHub's own runner, chosen by nobody, left
-#                                                  wherever the package already has it
+#   agent jobs of every worker                     ->  agents-arc          (the Azure fleet in
+#                                                                           agentrunner-pro-rg-01,
+#                                                                           runner group `agentic`)
+#   GitHub-hosted gate jobs (where a worker keeps them) ->  RunnerLandingZone
+#   ubuntu-latest                                  ->  GitHub's own runner, chosen by nobody, left
+#                                                      wherever the package already has it
+#
+# Since 0.28.17 every job runs on agents-arc, release and router included; the rule below
+# still accepts RunnerLandingZone for a stray gate job but requires agents-arc to be named.
 AGENT_POOL="agents-arc"
 PLUMBING_POOL="RunnerLandingZone"
 POOL_OK=1
@@ -2224,43 +2224,35 @@ for worker_md in "${WORKFLOWS_DIR}"/agent-*.md; do
   [ -f "$worker_md" ] || continue
   worker_name="$(basename "$worker_md" .md)"
   want="$AGENT_POOL"
-  [ "$worker_name" = "agent-release" ] && want="$PLUMBING_POOL"
   got="$(pools_named "$worker_md" | tr '\n' ' ' | sed 's/ $//')"
-  # Gate jobs and post-processing jobs run on RunnerLandingZone (GitHub-hosted) to avoid
-  # consuming self-hosted fleet capacity for trivial checks. The agent job and safe_outputs
-  # stay on agents-arc. So a worker may name both pools.
-  allowed_pools="${want} ${PLUMBING_POOL}"
-  if [ "$worker_name" = "agent-release" ]; then
-    if [ "$got" != "$want" ]; then
-      POOL_OK=0
-      echo "FAIL: ${worker_name} names runner pool(s) '${got}' but must name only '${want}'" >&2
-    fi
-  else
-    # Every pool named must be in the allowed set; agents-arc must be present (the agent job).
-    ok=1
-    for p in $got; do
-      case " $allowed_pools " in *" $p "*) ;; *) ok=0 ;; esac
-    done
-    if ! echo "$got" | grep -qw "$AGENT_POOL"; then
-      # visual-verify gets its agent runs-on from a shared import, not from the .md file,
-      # so the pool check only sees the subject gate job on RunnerLandingZone.
-      if [ "$worker_name" = "agent-visual-verify" ] && echo "$got" | grep -qw "$PLUMBING_POOL"; then
-        :
-      else
-        ok=0
-      fi
-    fi
-    if [ "$ok" -ne 1 ]; then
-      POOL_OK=0
-      echo "FAIL: ${worker_name} names runner pool(s) '${got}' but must name only '${AGENT_POOL}' and/or '${PLUMBING_POOL}' (with at least ${AGENT_POOL})" >&2
+  # The package owns which pool every job runs on, and since 0.28.17 every job in
+  # every worker runs on agents-arc, release included. RunnerLandingZone is still
+  # accepted for the occasional GitHub-hosted gate job, but the agent pool must be
+  # present so a worker cannot drift onto a pool no runner carries.
+  allowed_pools="${AGENT_POOL} ${PLUMBING_POOL}"
+  ok=1
+  for p in $got; do
+    case " $allowed_pools " in *" $p "*) ;; *) ok=0 ;; esac
+  done
+  if ! echo "$got" | grep -qw "$AGENT_POOL"; then
+    # visual-verify gets its agent runs-on from a shared import, not from the .md file,
+    # so the pool check only sees the subject gate job on RunnerLandingZone.
+    if [ "$worker_name" = "agent-visual-verify" ] && echo "$got" | grep -qw "$PLUMBING_POOL"; then
+      :
+    else
+      ok=0
     fi
   fi
+  if [ "$ok" -ne 1 ]; then
+    POOL_OK=0
+    echo "FAIL: ${worker_name} names runner pool(s) '${got}' but must name only '${AGENT_POOL}' and/or '${PLUMBING_POOL}' (with at least ${AGENT_POOL})" >&2
+  fi
 done
 
 router_pools="$(pools_named "$ROUTER_YML" | tr '\n' ' ' | sed 's/ $//')"
-if [ "$router_pools" != "$PLUMBING_POOL" ]; then
+if ! echo "$router_pools" | grep -qw "$AGENT_POOL"; then
   POOL_OK=0
-  echo "FAIL: the router names runner pool(s) '${router_pools}' but must name only '${PLUMBING_POOL}'" >&2
+  echo "FAIL: the router names runner pool(s) '${router_pools}' but must name only '${AGENT_POOL}' and/or '${PLUMBING_POOL}' (with at least ${AGENT_POOL})" >&2
 fi
 
 # And the pool must not be reintroduced as a per-consumer value: that is the mechanism that let