@plainconceptsplatform/workflows 1.3.1 → 1.5.0

package/dist/catalog-installation.d.ts

@@ -30,6 +30,8 @@ export interface CatalogInstallResult {
     readonly baselines: readonly BaselineStatus[];
     readonly upToDate: boolean;
     readonly dryRun: boolean;
+    /** Problems the update cannot fix on its own, for a person to act on. */
+    readonly warnings: readonly string[];
 }
 export interface CatalogInstallOptions {
     /** Also overwrite files whose ownership header was removed, and changed templates. */

package/dist/catalog-installation.js

@@ -7,7 +7,7 @@ import { promisify } from "node:util";
 import { parse as parseYaml } from "yaml";
 import { catalogTemplates, mandatoryFiles, retiredWorkflowFiles, routeNames, templateNames, workflowRoutes } from "./workflow-catalog.js";
 import { processRoutes, excludedWorkerFiles } from "./route-processing.js";
-import { generateOpencodeCi, generateOpencodeConfig, generateStackDefaults, injectStackEnv } from "./stack-defaults.js";
+import { generateOpencodeCi, generateOpencodeConfig, generateStackDefaults, injectStackEnv, shippedVerifyDefaults } from "./stack-defaults.js";
 import { mergeRouter, mergeWorker } from "./worker-env.js";
 import { fetchBaseline, hasOwnershipHeader, installedVersion, packageVersion, stampVersion } from "./package-baseline.js";
 const execFileAsync = promisify(execFile);
@@ -133,6 +133,7 @@ export async function installCatalog(repositoryPath, options = {}) {
         baselines,
         upToDate: updates.length === 0 && removals.length === 0,
         dryRun: options.dryRun ?? false,
+        warnings: await verifyScriptWarnings(repositoryPath, updates, existing),
     };
     if (options.dryRun)
         return result;
@@ -158,12 +159,22 @@ export async function installCatalog(repositoryPath, options = {}) {
 // the empty string as the consumer's deliberate choice. All four consuming repositories were in
 // that state, so every merge-gate repair pushed a fix with the verification block interpolating
 // to nothing. An empty value is the absence of a choice, so an update fills it.
+//
+// So is a value the package wrote itself. The implement worker shipped a dotnet-only line, and
+// every full-stack consumer kept it as if someone had chosen it, so the bot never built the web
+// app before opening a pull request. A consumer holding a shipped default gets the current one:
+// its own value is blanked here, which the env merge then reads as no choice.
 function applyStackDefaults(files, inspection, existing) {
     const defaults = generateStackDefaults(inspection);
     const result = new Map(files);
     for (const [target, content] of result) {
-        if (isWorker(target) && (!existing.has(target) || hasEmptyVerifyCommands(existing.get(target)))) {
+        const current = existing.get(target);
+        if (isWorker(target) && (current === undefined || hasEmptyVerifyCommands(current))) {
+            result.set(target, injectStackEnv(content, defaults));
+        }
+        else if (isWorker(target) && hasShippedVerifyCommands(current, defaults.verifyCommands)) {
             result.set(target, injectStackEnv(content, defaults));
+            existing.set(target, current.replace(VERIFY_LINE, '  VERIFY_COMMANDS: ""'));
         }
         else if (target.endsWith("opencode-ci.md")) {
             result.set(target, generateOpencodeCi(content, inspection));
@@ -174,11 +185,42 @@ function applyStackDefaults(files, inspection, existing) {
     }
     return result;
 }
+const VERIFY_LINE = /^ {2}VERIFY_COMMANDS: *(.*?) *$/m;
 // A worker whose VERIFY_COMMANDS is present but empty. Anything else, including a value the
 // consumer wrote, is left alone.
 function hasEmptyVerifyCommands(current) {
     return typeof current === "string" && /^ {2}VERIFY_COMMANDS: *(""|'')? *$/m.test(current);
 }
+function verifyCommandsOf(content) {
+    const raw = VERIFY_LINE.exec(content)?.[1];
+    return raw?.replace(/^(["'])(.*)\1$/, "$2");
+}
+function hasShippedVerifyCommands(current, replacement) {
+    if (current === undefined)
+        return false;
+    const value = verifyCommandsOf(current);
+    return value !== undefined && value !== replacement && shippedVerifyDefaults.has(value);
+}
+// `pnpm verify` is only a command if the repository defines it. The update still writes it, since
+// it is the right default, and says what is missing instead of leaving the bot to find out.
+async function verifyScriptWarnings(repositoryPath, updates, existing) {
+    const written = new Map(existing);
+    for (const update of updates)
+        written.set(update.target, update.content);
+    const workers = [...written].filter(([target, content]) => isWorker(target) && /\bpnpm verify\b/.test(verifyCommandsOf(content) ?? ""));
+    if (workers.length === 0)
+        return [];
+    let scripts = {};
+    try {
+        scripts = JSON.parse(await readFile(join(repositoryPath, "package.json"), "utf8")).scripts ?? {};
+    }
+    catch {
+        scripts = {};
+    }
+    if (typeof scripts.verify === "string")
+        return [];
+    return [`VERIFY_COMMANDS runs \`pnpm verify\` in ${workers.map(([target]) => target).sort().join(", ")}, but package.json has no "verify" script. Add one that runs what CI blocks on.`];
+}
 // Directories that belong wholly to the package. A file here that carries our ownership header
 // and is no longer in the package was deleted upstream, and without this it would sit in every
 // consumer forever: that is how `stale-recovery` and `update-changelog` outlived the code that

package/dist/index.js

@@ -139,6 +139,8 @@ export async function run(arguments_, repositoryPath = process.cwd()) {
             console.error(`Template conflicts found. Re-run with --force to overwrite consumer-owned copies:\n${allConflicts.join("\n")}`);
             return 1;
         }
+        for (const warning of catalog?.warnings ?? [])
+            console.error(`warning: ${warning}`);
         console.log(JSON.stringify({
             command,
             dryRun,
@@ -177,6 +179,7 @@ function summarize(result) {
         baselines: result.baselines,
         changes: result.changes.filter((change) => change.status !== "unchanged"),
         unchanged: result.changes.filter((change) => change.status === "unchanged").length,
+        ...(result.warnings.length > 0 ? { warnings: result.warnings } : {}),
     };
 }
 // A fixed baseline for every recorded version, instead of asking npm. Rolling out a version that

package/dist/stack-defaults.d.ts

@@ -6,6 +6,7 @@ export interface StackDefaults {
     readonly hasNodeOnly: boolean;
 }
 export declare function generateStackDefaults(inspection: RepositoryInspection): StackDefaults;
+export declare const shippedVerifyDefaults: ReadonlySet<string>;
 export declare function injectStackEnv(content: string, defaults: StackDefaults): string;
 export declare function generateOpencodeCi(baseContent: string, inspection: RepositoryInspection): string;
 export declare function generateOpencodeConfig(baseContent: string, inspection: RepositoryInspection): string;

package/dist/stack-defaults.js

@@ -4,8 +4,12 @@ export function generateStackDefaults(inspection) {
     const hasNodeOnly = hasNode && !hasDotnet;
     let verifyCommands;
     let repoRulesBase;
+    // A full-stack repository verifies with its own `pnpm verify`, which runs what its CI blocks on.
+    // The command this replaced was prose the agent had to interpret, and the implement worker never
+    // got it at all: it kept the package's dotnet-only line, so bot pull requests shipped with the
+    // web build broken (Pliny-Bot #639).
     if (hasDotnet && hasNode) {
-        verifyCommands = ".NET: dotnet restore && dotnet build -c Release && dotnet test | Web: pnpm lint && pnpm test && pnpm build";
+        verifyCommands = "pnpm verify";
         repoRulesBase = "Full-stack .NET + React/Next.js repository. Follow Clean Architecture layering: API → Application → Domain. Infrastructure implements Application ports. Do not reference EF Core or ASP.NET from Application. Frontend communicates exclusively via HTTP endpoints. Run both .NET and frontend verification.";
     }
     else if (hasDotnet) {
@@ -18,6 +22,14 @@ export function generateStackDefaults(inspection) {
     }
     return { verifyCommands, repoRulesBase, hasDotnet, hasNodeOnly };
 }
+// Every VERIFY_COMMANDS value the package itself has ever written. A consumer holding one of these
+// never chose it, so an update may replace it with the current default for the stack.
+export const shippedVerifyDefaults = new Set([
+    ".NET: dotnet restore && dotnet build -c Release && dotnet test | Web: pnpm lint && pnpm test && pnpm build",
+    "dotnet restore && dotnet build -c Release --no-restore && dotnet test",
+    "dotnet restore && dotnet build -c Release --no-restore && dotnet test -c Release --no-build",
+    "pnpm verify",
+]);
 // Only a worker that prints a verification block declares VERIFY_COMMANDS. Adding the key to
 // the others made byte-identical workers read as changed on every update, so a worker without
 // the key is left without it.

package/dist/worker-env.js

@@ -139,9 +139,8 @@ function engineEndpoint(content) {
 // triage, refine, implement, apply-review and audit on RunnerLandingZone while its merge-gate
 // and release stayed on agents-arc -- the pool was applied to the workers installed at the time
 // and never to the ones added later, and nothing reported the split. The pool is package-owned
-// now: the agent jobs belong on the Azure fleet (agents-arc, runner group `agentic`), and the
-// router and release worker name RunnerLandingZone. The route matrix asserts both, so a repo
-// cannot drift back without a red run. Per the standing rule for this package, anything a
+// now: every job, router and release included, belongs on the Azure fleet (agents-arc, runner
+// group `agentic`). The route matrix asserts it, so a repo cannot drift back without a red run. Per the standing rule for this package, anything a
 // consumer really must vary belongs in `env:` at the top of the file, not in `runs-on`.
 // GitHub evaluates no expression in a `workflow_run.workflows:` list or in a `cron:`, so the two
 // router values that are also needed there cannot be read from `env:` at those two lines. The

package/loops/actions/add-issue-labels/action.yml

@@ -1,50 +1,50 @@
-# Managed by @plainconceptsplatform/workflows. Source: loops/actions/add-issue-labels/action.yml. Update with workflows update --force; consumer edits may be overwritten.
-name: Add issue labels
-description: Add one or more labels to an issue or pull request.
-inputs:
-  token:
-    description: GitHub token used by github-script.
-    required: true
-  issue-number:
-    description: Issue or pull request number.
-    required: true
-  labels:
-    description: Labels to add, one per line. A comma-separated list is accepted as well.
-    required: true
-runs:
-  using: composite
-  steps:
-    - name: Add labels
-      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
-      env:
-        ISSUE_NUMBER: ${{ inputs.issue-number }}
-        LABELS: ${{ inputs.labels }}
-      with:
-        github-token: ${{ inputs.token }}
-        script: |
-          const labels = process.env.LABELS.split(/\r?\n|,/).map((label) => label.trim()).filter(Boolean);
-          if (labels.length === 0) {
-            core.notice('No labels to add.');
-            return;
-          }
-
-          // Create what does not exist yet, so shipping a new label in the package does not
-          // require a person to go and make it in every repository first. Its sibling
-          // remove-issue-labels has swallowed 404 for a long time; this one had no guard at all,
-          // so a label the package introduced would fail the step in every consumer until
-          // somebody created it by hand. 422 is "already exists", which is the happy path on
-          // every run after the first.
-          for (const name of labels) {
-            try {
-              await github.rest.issues.createLabel({ ...context.repo, name, color: 'ededed' });
-              core.notice(`Created the ${name} label.`);
-            } catch (error) {
-              if (error.status !== 422) throw error;
-            }
-          }
-
-          await github.rest.issues.addLabels({
-            ...context.repo,
-            issue_number: Number(process.env.ISSUE_NUMBER),
-            labels,
-          });
+# Managed by @plainconceptsplatform/workflows. Source: loops/actions/add-issue-labels/action.yml. Update with workflows update --force; consumer edits may be overwritten.
+name: Add issue labels
+description: Add one or more labels to an issue or pull request.
+inputs:
+  token:
+    description: GitHub token used by github-script.
+    required: true
+  issue-number:
+    description: Issue or pull request number.
+    required: true
+  labels:
+    description: Labels to add, one per line. A comma-separated list is accepted as well.
+    required: true
+runs:
+  using: composite
+  steps:
+    - name: Add labels
+      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
+      env:
+        ISSUE_NUMBER: ${{ inputs.issue-number }}
+        LABELS: ${{ inputs.labels }}
+      with:
+        github-token: ${{ inputs.token }}
+        script: |
+          const labels = process.env.LABELS.split(/\r?\n|,/).map((label) => label.trim()).filter(Boolean);
+          if (labels.length === 0) {
+            core.notice('No labels to add.');
+            return;
+          }
+
+          // Create what does not exist yet, so shipping a new label in the package does not
+          // require a person to go and make it in every repository first. Its sibling
+          // remove-issue-labels has swallowed 404 for a long time; this one had no guard at all,
+          // so a label the package introduced would fail the step in every consumer until
+          // somebody created it by hand. 422 is "already exists", which is the happy path on
+          // every run after the first.
+          for (const name of labels) {
+            try {
+              await github.rest.issues.createLabel({ ...context.repo, name, color: 'ededed' });
+              core.notice(`Created the ${name} label.`);
+            } catch (error) {
+              if (error.status !== 422) throw error;
+            }
+          }
+
+          await github.rest.issues.addLabels({
+            ...context.repo,
+            issue_number: Number(process.env.ISSUE_NUMBER),
+            labels,
+          });

package/loops/actions/agent-output.cjs

@@ -1,17 +1,17 @@
-// Managed by @plainconceptsplatform/workflows. Source: loops/actions/agent-output.cjs. Update with `workflows update --force`; consumer edits may be overwritten.
-const fs = require('node:fs');
-
-// Every apply-agent-* action reads the same artifact the same way: a missing file means the
-// agent produced nothing, which is a normal outcome rather than a failure.
-function readAgentItems(outputFile, type) {
-  if (!outputFile || !fs.existsSync(outputFile)) {
-    return [];
-  }
-
-  const parsed = JSON.parse(fs.readFileSync(outputFile, 'utf8'));
-  const items = Array.isArray(parsed.items) ? parsed.items : [];
-
-  return type ? items.filter((item) => item.type === type) : items;
-}
-
-module.exports = { readAgentItems };
+// Managed by @plainconceptsplatform/workflows. Source: loops/actions/agent-output.cjs. Update with `workflows update --force`; consumer edits may be overwritten.
+const fs = require('node:fs');
+
+// Every apply-agent-* action reads the same artifact the same way: a missing file means the
+// agent produced nothing, which is a normal outcome rather than a failure.
+function readAgentItems(outputFile, type) {
+  if (!outputFile || !fs.existsSync(outputFile)) {
+    return [];
+  }
+
+  const parsed = JSON.parse(fs.readFileSync(outputFile, 'utf8'));
+  const items = Array.isArray(parsed.items) ? parsed.items : [];
+
+  return type ? items.filter((item) => item.type === type) : items;
+}
+
+module.exports = { readAgentItems };

package/loops/actions/apply-agent-bundle/action.yml

@@ -1,24 +1,24 @@
-# Managed by @plainconceptsplatform/workflows. Source: loops/actions/apply-agent-bundle/action.yml. Update with workflows update --force; consumer edits may be overwritten.
-name: Apply agent bundle
-description: Fast-forward an intended branch to the single commit exported by a verified git bundle.
-inputs:
-  bundle-file:
-    description: Path to the git bundle from the agent artifact.
-    required: true
-  target-branch:
-    description: Branch that must fast-forward to the bundle commit.
-    required: true
-  base-branch:
-    description: Existing branch used only when target-branch does not exist.
-    required: false
-    default: ''
-runs:
-  using: composite
-  steps:
-    - name: Fast-forward branch from bundle
-      shell: bash
-      env:
-        BUNDLE_FILE: ${{ inputs.bundle-file }}
-        TARGET_BRANCH: ${{ inputs.target-branch }}
-        BASE_BRANCH: ${{ inputs.base-branch }}
-      run: bash "$GITHUB_ACTION_PATH/apply-bundle.sh" "$BUNDLE_FILE" "$TARGET_BRANCH" "$BASE_BRANCH"
+# Managed by @plainconceptsplatform/workflows. Source: loops/actions/apply-agent-bundle/action.yml. Update with workflows update --force; consumer edits may be overwritten.
+name: Apply agent bundle
+description: Fast-forward an intended branch to the single commit exported by a verified git bundle.
+inputs:
+  bundle-file:
+    description: Path to the git bundle from the agent artifact.
+    required: true
+  target-branch:
+    description: Branch that must fast-forward to the bundle commit.
+    required: true
+  base-branch:
+    description: Existing branch used only when target-branch does not exist.
+    required: false
+    default: ''
+runs:
+  using: composite
+  steps:
+    - name: Fast-forward branch from bundle
+      shell: bash
+      env:
+        BUNDLE_FILE: ${{ inputs.bundle-file }}
+        TARGET_BRANCH: ${{ inputs.target-branch }}
+        BASE_BRANCH: ${{ inputs.base-branch }}
+      run: bash "$GITHUB_ACTION_PATH/apply-bundle.sh" "$BUNDLE_FILE" "$TARGET_BRANCH" "$BASE_BRANCH"

package/loops/actions/apply-agent-comments/action.yml

@@ -1,42 +1,42 @@
-# Managed by @plainconceptsplatform/workflows. Source: loops/actions/apply-agent-comments/action.yml. Update with workflows update --force; consumer edits may be overwritten.
-name: Apply agent comments
-description: Post every add_comment item from agent_output.json.
-inputs:
-  output-file:
-    description: Path to agent_output.json.
-    required: true
-  token:
-    description: GitHub token with issues:write and pull-requests:write.
-    required: true
-runs:
-  using: composite
-  steps:
-    - name: Post comments
-      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
-      env:
-        AGENT_OUTPUT_LIB: ${{ github.action_path }}/../agent-output.cjs
-        OUTPUT_FILE: ${{ inputs.output-file }}
-      with:
-        github-token: ${{ inputs.token }}
-        script: |
-          const { readAgentItems } = require(process.env.AGENT_OUTPUT_LIB);
-          const items = readAgentItems(process.env.OUTPUT_FILE, 'add_comment');
-
-          let posted = 0;
-
-          for (const item of items) {
-            if (!item.item_number) {
-              core.warning(`add_comment item has no item_number, skipping: ${JSON.stringify(item)}`);
-              continue;
-            }
-
-            await github.rest.issues.createComment({
-              ...context.repo,
-              issue_number: Number(item.item_number),
-              body: item.body,
-            });
-
-            posted += 1;
-          }
-
-          core.info(`Posted ${posted} comment(s).`);
+# Managed by @plainconceptsplatform/workflows. Source: loops/actions/apply-agent-comments/action.yml. Update with workflows update --force; consumer edits may be overwritten.
+name: Apply agent comments
+description: Post every add_comment item from agent_output.json.
+inputs:
+  output-file:
+    description: Path to agent_output.json.
+    required: true
+  token:
+    description: GitHub token with issues:write and pull-requests:write.
+    required: true
+runs:
+  using: composite
+  steps:
+    - name: Post comments
+      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
+      env:
+        AGENT_OUTPUT_LIB: ${{ github.action_path }}/../agent-output.cjs
+        OUTPUT_FILE: ${{ inputs.output-file }}
+      with:
+        github-token: ${{ inputs.token }}
+        script: |
+          const { readAgentItems } = require(process.env.AGENT_OUTPUT_LIB);
+          const items = readAgentItems(process.env.OUTPUT_FILE, 'add_comment');
+
+          let posted = 0;
+
+          for (const item of items) {
+            if (!item.item_number) {
+              core.warning(`add_comment item has no item_number, skipping: ${JSON.stringify(item)}`);
+              continue;
+            }
+
+            await github.rest.issues.createComment({
+              ...context.repo,
+              issue_number: Number(item.item_number),
+              body: item.body,
+            });
+
+            posted += 1;
+          }
+
+          core.info(`Posted ${posted} comment(s).`);

package/loops/actions/apply-agent-labels/action.yml

@@ -1,55 +1,55 @@
-# Managed by @plainconceptsplatform/workflows. Source: loops/actions/apply-agent-labels/action.yml. Update with workflows update --force; consumer edits may be overwritten.
-name: Apply agent labels
-description: Apply every add_labels and remove_labels item from agent_output.json.
-inputs:
-  output-file:
-    description: Path to agent_output.json.
-    required: true
-  token:
-    description: GitHub token with issues:write.
-    required: true
-runs:
-  using: composite
-  steps:
-    - name: Apply label changes
-      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
-      env:
-        AGENT_OUTPUT_LIB: ${{ github.action_path }}/../agent-output.cjs
-        OUTPUT_FILE: ${{ inputs.output-file }}
-      with:
-        github-token: ${{ inputs.token }}
-        script: |
-          const { readAgentItems } = require(process.env.AGENT_OUTPUT_LIB);
-          const items = readAgentItems(process.env.OUTPUT_FILE)
-            .filter((item) => item.type === 'add_labels' || item.type === 'remove_labels');
-
-          let added = 0;
-          let removed = 0;
-
-          for (const item of items) {
-            const labels = (item.labels ?? []).map((label) => label.name).filter(Boolean);
-
-            if (!item.item_number || labels.length === 0) {
-              core.warning(`${item.type} item is not actionable, skipping: ${JSON.stringify(item)}`);
-              continue;
-            }
-
-            const issue_number = Number(item.item_number);
-
-            if (item.type === 'add_labels') {
-              await github.rest.issues.addLabels({ ...context.repo, issue_number, labels });
-              added += labels.length;
-              continue;
-            }
-
-            for (const name of labels) {
-              try {
-                await github.rest.issues.removeLabel({ ...context.repo, issue_number, name });
-                removed += 1;
-              } catch (error) {
-                if (error.status !== 404) throw error;
-              }
-            }
-          }
-
-          core.info(`Added ${added} label(s), removed ${removed}.`);
+# Managed by @plainconceptsplatform/workflows. Source: loops/actions/apply-agent-labels/action.yml. Update with workflows update --force; consumer edits may be overwritten.
+name: Apply agent labels
+description: Apply every add_labels and remove_labels item from agent_output.json.
+inputs:
+  output-file:
+    description: Path to agent_output.json.
+    required: true
+  token:
+    description: GitHub token with issues:write.
+    required: true
+runs:
+  using: composite
+  steps:
+    - name: Apply label changes
+      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
+      env:
+        AGENT_OUTPUT_LIB: ${{ github.action_path }}/../agent-output.cjs
+        OUTPUT_FILE: ${{ inputs.output-file }}
+      with:
+        github-token: ${{ inputs.token }}
+        script: |
+          const { readAgentItems } = require(process.env.AGENT_OUTPUT_LIB);
+          const items = readAgentItems(process.env.OUTPUT_FILE)
+            .filter((item) => item.type === 'add_labels' || item.type === 'remove_labels');
+
+          let added = 0;
+          let removed = 0;
+
+          for (const item of items) {
+            const labels = (item.labels ?? []).map((label) => label.name).filter(Boolean);
+
+            if (!item.item_number || labels.length === 0) {
+              core.warning(`${item.type} item is not actionable, skipping: ${JSON.stringify(item)}`);
+              continue;
+            }
+
+            const issue_number = Number(item.item_number);
+
+            if (item.type === 'add_labels') {
+              await github.rest.issues.addLabels({ ...context.repo, issue_number, labels });
+              added += labels.length;
+              continue;
+            }
+
+            for (const name of labels) {
+              try {
+                await github.rest.issues.removeLabel({ ...context.repo, issue_number, name });
+                removed += 1;
+              } catch (error) {
+                if (error.status !== 404) throw error;
+              }
+            }
+          }
+
+          core.info(`Added ${added} label(s), removed ${removed}.`);

package/loops/actions/apply-agent-output/action.yml

@@ -39,6 +39,10 @@ inputs:
     description: Issue an update_issue item targets when it names none.
     required: false
     default: ''
+  parent-issue:
+    description: The split parent; see create-agent-issues. Empty for anything but a split.
+    required: false
+    default: ''
 outputs:
   item-count:
     description: Number of items the agent produced.
@@ -92,6 +96,7 @@ runs:
       with:
         output-file: ${{ steps.output.outputs.output-file }}
         token: ${{ inputs.token }}
+        parent-issue: ${{ inputs.parent-issue }}
 
     - name: Post agent comments
       if: steps.output.outputs.item-count != '0'