@plainconceptsplatform/workflows 1.3.1 → 1.5.0

package/loops/actions/cleanup-artifacts/action.yml

@@ -1,91 +1,91 @@
-# Managed by @plainconceptsplatform/workflows. Source: loops/actions/cleanup-artifacts/action.yml. Update with workflows update --force; consumer edits may be overwritten.
-name: Clean up artifacts
-description: Delete workflow artifacts older than the configured retention period.
-inputs:
-  token:
-    description: GitHub token with actions write access.
-    required: true
-  artifact-retention-days:
-    description: Retention period in days. Invalid or empty values use 3.
-    required: false
-    default: ''
-runs:
-  using: composite
-  steps:
-    - name: Delete expired artifacts
-      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
-      env:
-        ARTIFACT_RETENTION_DAYS: ${{ inputs.artifact-retention-days }}
-      with:
-        github-token: ${{ inputs.token }}
-        script: |
-          const parsedDays = Number.parseInt(process.env.ARTIFACT_RETENTION_DAYS, 10);
-          const days = Number.isInteger(parsedDays) && parsedDays > 0 ? parsedDays : 3;
-          const cutoff = new Date(Date.now() - days * 24 * 60 * 60 * 1000);
-
-          core.info(`Retention: ${days} days (cutoff ${cutoff.toISOString()})`);
-
-          const artifacts = await github.paginate(github.rest.actions.listArtifactsForRepo, {
-            ...context.repo,
-            per_page: 100,
-          });
-
-          const expired = artifacts.filter((artifact) => new Date(artifact.created_at) < cutoff);
-
-          let deleted = 0;
-          const failed = [];
-
-          for (const artifact of expired) {
-            // GitHub answers a delete with a 5xx now and then. Three tries, because the
-            // alternative is a red daily run over one artifact that tomorrow's run would
-            // have removed anyway.
-            let lastError;
-            for (let attempt = 1; attempt <= 3; attempt += 1) {
-              try {
-                await github.rest.actions.deleteArtifact({
-                  ...context.repo,
-                  artifact_id: artifact.id,
-                });
-
-                core.info(`Deleted ${artifact.name} (created ${artifact.created_at})`);
-                deleted += 1;
-                lastError = undefined;
-                break;
-              } catch (error) {
-                lastError = error;
-                if (attempt < 3) await new Promise((wait) => setTimeout(wait, attempt * 2000));
-              }
-            }
-
-            if (lastError) {
-              core.warning(
-                `Could not delete ${artifact.name} (${artifact.id}) after 3 tries: ${lastError.message}`,
-              );
-              failed.push({ name: artifact.name, status: lastError.status });
-            }
-          }
-
-          await core.summary
-            .addHeading('Artifact cleanup', 2)
-            .addList([
-              `Retention: ${days} days (cutoff ${cutoff.toISOString()})`,
-              `Deleted: ${deleted}`,
-              `Failed to delete: ${failed.length}`,
-              `Within retention: ${artifacts.length - expired.length}`,
-            ])
-            .write();
-
-          // Deleting nothing at all while expired artifacts exist is a permission or
-          // configuration problem, and has to be loud. One stubborn artifact among many is
-          // the API having a bad minute: tomorrow's run picks it up, and failing here only
-          // teaches people to ignore a red cleanup. A 4xx is never transient, so it fails
-          // whatever else succeeded.
-          const names = failed.map((entry) => entry.name).join(', ');
-          const structural = failed.some((entry) => entry.status && entry.status < 500);
-          if (failed.length > 0 && (deleted === 0 || structural)) {
-            core.setFailed(`${failed.length} artifact(s) could not be deleted: ${names}`);
-          } else if (failed.length > 0) {
-            core.warning(
-              `${failed.length} artifact(s) survived this run and will be retried tomorrow: ${names}`,
-            );
-          }
+# Managed by @plainconceptsplatform/workflows. Source: loops/actions/cleanup-artifacts/action.yml. Update with workflows update --force; consumer edits may be overwritten.
+name: Clean up artifacts
+description: Delete workflow artifacts older than the configured retention period.
+inputs:
+  token:
+    description: GitHub token with actions write access.
+    required: true
+  artifact-retention-days:
+    description: Retention period in days. Invalid or empty values use 3.
+    required: false
+    default: ''
+runs:
+  using: composite
+  steps:
+    - name: Delete expired artifacts
+      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
+      env:
+        ARTIFACT_RETENTION_DAYS: ${{ inputs.artifact-retention-days }}
+      with:
+        github-token: ${{ inputs.token }}
+        script: |
+          const parsedDays = Number.parseInt(process.env.ARTIFACT_RETENTION_DAYS, 10);
+          const days = Number.isInteger(parsedDays) && parsedDays > 0 ? parsedDays : 3;
+          const cutoff = new Date(Date.now() - days * 24 * 60 * 60 * 1000);
+
+          core.info(`Retention: ${days} days (cutoff ${cutoff.toISOString()})`);
+
+          const artifacts = await github.paginate(github.rest.actions.listArtifactsForRepo, {
+            ...context.repo,
+            per_page: 100,
+          });
+
+          const expired = artifacts.filter((artifact) => new Date(artifact.created_at) < cutoff);
+
+          let deleted = 0;
+          const failed = [];
+
+          for (const artifact of expired) {
+            // GitHub answers a delete with a 5xx now and then. Three tries, because the
+            // alternative is a red daily run over one artifact that tomorrow's run would
+            // have removed anyway.
+            let lastError;
+            for (let attempt = 1; attempt <= 3; attempt += 1) {
+              try {
+                await github.rest.actions.deleteArtifact({
+                  ...context.repo,
+                  artifact_id: artifact.id,
+                });
+
+                core.info(`Deleted ${artifact.name} (created ${artifact.created_at})`);
+                deleted += 1;
+                lastError = undefined;
+                break;
+              } catch (error) {
+                lastError = error;
+                if (attempt < 3) await new Promise((wait) => setTimeout(wait, attempt * 2000));
+              }
+            }
+
+            if (lastError) {
+              core.warning(
+                `Could not delete ${artifact.name} (${artifact.id}) after 3 tries: ${lastError.message}`,
+              );
+              failed.push({ name: artifact.name, status: lastError.status });
+            }
+          }
+
+          await core.summary
+            .addHeading('Artifact cleanup', 2)
+            .addList([
+              `Retention: ${days} days (cutoff ${cutoff.toISOString()})`,
+              `Deleted: ${deleted}`,
+              `Failed to delete: ${failed.length}`,
+              `Within retention: ${artifacts.length - expired.length}`,
+            ])
+            .write();
+
+          // Deleting nothing at all while expired artifacts exist is a permission or
+          // configuration problem, and has to be loud. One stubborn artifact among many is
+          // the API having a bad minute: tomorrow's run picks it up, and failing here only
+          // teaches people to ignore a red cleanup. A 4xx is never transient, so it fails
+          // whatever else succeeded.
+          const names = failed.map((entry) => entry.name).join(', ');
+          const structural = failed.some((entry) => entry.status && entry.status < 500);
+          if (failed.length > 0 && (deleted === 0 || structural)) {
+            core.setFailed(`${failed.length} artifact(s) could not be deleted: ${names}`);
+          } else if (failed.length > 0) {
+            core.warning(
+              `${failed.length} artifact(s) survived this run and will be retried tomorrow: ${names}`,
+            );
+          }

package/loops/actions/close-agent-issues/action.yml

@@ -1,43 +1,43 @@
-# Managed by @plainconceptsplatform/workflows. Source: loops/actions/close-agent-issues/action.yml. Update with workflows update --force; consumer edits may be overwritten.
-name: Close agent issues
-description: Close every close_issue 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: Close issues
-      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, 'close_issue');
-
-          let closed = 0;
-
-          for (const item of items) {
-            if (!item.item_number) {
-              core.warning(`close_issue item has no item_number, skipping: ${JSON.stringify(item)}`);
-              continue;
-            }
-
-            await github.rest.issues.update({
-              ...context.repo,
-              issue_number: Number(item.item_number),
-              state: 'closed',
-              state_reason: item.state_reason === 'not_planned' ? 'not_planned' : 'completed',
-            });
-
-            closed += 1;
-          }
-
-          core.info(`Closed ${closed} issue(s).`);
+# Managed by @plainconceptsplatform/workflows. Source: loops/actions/close-agent-issues/action.yml. Update with workflows update --force; consumer edits may be overwritten.
+name: Close agent issues
+description: Close every close_issue 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: Close issues
+      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, 'close_issue');
+
+          let closed = 0;
+
+          for (const item of items) {
+            if (!item.item_number) {
+              core.warning(`close_issue item has no item_number, skipping: ${JSON.stringify(item)}`);
+              continue;
+            }
+
+            await github.rest.issues.update({
+              ...context.repo,
+              issue_number: Number(item.item_number),
+              state: 'closed',
+              state_reason: item.state_reason === 'not_planned' ? 'not_planned' : 'completed',
+            });
+
+            closed += 1;
+          }
+
+          core.info(`Closed ${closed} issue(s).`);

package/loops/actions/create-agent-issues/action.yml

@@ -8,6 +8,12 @@ inputs:
   token:
     description: GitHub token with issues:write.
     required: true
+  parent-issue:
+    description: >-
+      The split parent, set only when the run is a split. Each created issue is stamped with the
+      split-parent marker and linked as a sub-issue of it, and temporary ids are resolved.
+    required: false
+    default: ''
 outputs:
   created-count:
     description: Number of issues created.
@@ -15,6 +21,9 @@ outputs:
   first-issue-number:
     description: Issue number of the first issue created, or empty.
     value: ${{ steps.create.outputs.first-issue-number }}
+  created-numbers:
+    description: Space-separated numbers of every issue created, in order.
+    value: ${{ steps.create.outputs.created-numbers }}
 runs:
   using: composite
   steps:
@@ -24,13 +33,23 @@ runs:
       env:
         AGENT_OUTPUT_LIB: ${{ github.action_path }}/../agent-output.cjs
         OUTPUT_FILE: ${{ inputs.output-file }}
+        PARENT_ISSUE: ${{ inputs.parent-issue }}
       with:
         github-token: ${{ inputs.token }}
         script: |
           const { readAgentItems } = require(process.env.AGENT_OUTPUT_LIB);
           const items = readAgentItems(process.env.OUTPUT_FILE, 'create_issue');
+          // Set only for a split. The agent writes `<!-- split-parent: N -->` in every child, but
+          // gh-aw's sanitizer strips HTML comments before agent_output.json is written, so the
+          // marker never reaches this file (Lyceum #45, run 38747143251: three markers in
+          // safeoutputs.jsonl, none in agent_output.json). gh-aw used to link the children as
+          // sub-issues itself; staged, nothing does, and the hand-over found no children. So the
+          // workflow stamps the marker and links the sub-issue here, from a value it owns.
+          const parent = /^[0-9]+$/.test(process.env.PARENT_ISSUE || '') ? Number(process.env.PARENT_ISSUE) : null;
+          const marker = parent ? `<!-- split-parent: ${parent} -->` : '';
 
           const created = [];
+          const temporaryIds = {};
 
           for (const item of items) {
             if (!item.title) {
@@ -38,15 +57,57 @@ runs:
               continue;
             }
 
+            let body = item.body ?? '';
+            if (parent && !body.includes(marker)) body = `${body}\n\n${marker}`;
+
             const { data } = await github.rest.issues.create({
               ...context.repo,
               title: item.title,
-              body: item.body ?? '',
+              body,
             });
 
-            created.push(data.number);
+            created.push({ number: data.number, body });
+            const tid = String(item.temporary_id ?? '').replace(/^#/, '');
+            if (tid) temporaryIds[tid] = data.number;
             core.info(`Created #${data.number}: ${item.title}`);
+
+            if (parent) {
+              try {
+                await github.request('POST /repos/{owner}/{repo}/issues/{issue_number}/sub_issues', {
+                  ...context.repo,
+                  issue_number: parent,
+                  sub_issue_id: data.id,
+                });
+                core.info(`Linked #${data.number} under #${parent}`);
+              } catch (error) {
+                // The marker above is the fallback the hand-over searches, so a failed link is
+                // degraded, not lost.
+                core.warning(`Could not link #${data.number} under #${parent}: ${error.message}`);
+              }
+            }
+          }
+
+          // The agent references siblings by the temporary id it gave them, because their numbers
+          // do not exist until now: "Blocked by #aw_struct_modules". gh-aw resolves these when it
+          // creates the issues; staged, they reached the issues unresolved. Rewrite them in the
+          // children and in the parent's checklist, which was updated before this step ran.
+          const resolve = text => text.replace(/#(aw_[A-Za-z0-9_]+)/g, (whole, id) => (temporaryIds[id] ? `#${temporaryIds[id]}` : whole));
+          if (Object.keys(temporaryIds).length > 0) {
+            for (const issue of created) {
+              const next = resolve(issue.body);
+              if (next !== issue.body) {
+                await github.rest.issues.update({ ...context.repo, issue_number: issue.number, body: next });
+              }
+            }
+            if (parent) {
+              const { data: current } = await github.rest.issues.get({ ...context.repo, issue_number: parent });
+              const next = resolve(current.body ?? '');
+              if (next !== (current.body ?? '')) {
+                await github.rest.issues.update({ ...context.repo, issue_number: parent, body: next });
+              }
+            }
           }
 
           core.setOutput('created-count', String(created.length));
-          core.setOutput('first-issue-number', created.length > 0 ? String(created[0]) : '');
+          core.setOutput('first-issue-number', created.length > 0 ? String(created[0].number) : '');
+          core.setOutput('created-numbers', created.map(issue => issue.number).join(' '));

package/loops/actions/create-issue-comment/action.yml

@@ -1,29 +1,29 @@
-# Managed by @plainconceptsplatform/workflows. Source: loops/actions/create-issue-comment/action.yml. Update with workflows update --force; consumer edits may be overwritten.
-name: Create issue comment
-description: Create a comment on 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
-  body:
-    description: Complete Markdown comment body.
-    required: true
-runs:
-  using: composite
-  steps:
-    - name: Create comment
-      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
-      env:
-        ISSUE_NUMBER: ${{ inputs.issue-number }}
-        BODY: ${{ inputs.body }}
-      with:
-        github-token: ${{ inputs.token }}
-        script: |
-          await github.rest.issues.createComment({
-            ...context.repo,
-            issue_number: Number(process.env.ISSUE_NUMBER),
-            body: process.env.BODY,
-          });
+# Managed by @plainconceptsplatform/workflows. Source: loops/actions/create-issue-comment/action.yml. Update with workflows update --force; consumer edits may be overwritten.
+name: Create issue comment
+description: Create a comment on 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
+  body:
+    description: Complete Markdown comment body.
+    required: true
+runs:
+  using: composite
+  steps:
+    - name: Create comment
+      uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0
+      env:
+        ISSUE_NUMBER: ${{ inputs.issue-number }}
+        BODY: ${{ inputs.body }}
+      with:
+        github-token: ${{ inputs.token }}
+        script: |
+          await github.rest.issues.createComment({
+            ...context.repo,
+            issue_number: Number(process.env.ISSUE_NUMBER),
+            body: process.env.BODY,
+          });

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

@@ -1,53 +1,53 @@
-# Managed by @plainconceptsplatform/workflows. Source: loops/actions/download-agent-output/action.yml. Update with workflows update --force; consumer edits may be overwritten.
-name: Download agent output
-description: Download the agent artifact and expose the output JSON and bundle paths.
-inputs:
-  artifact-name:
-    description: >-
-      Artifact name to download. gh-aw prefixes it with the value it derives from the
-      workflow's inputs, so a workflow_call worker must pass the activation job's
-      artifact_prefix output followed by "agent", not the bare name.
-    required: true
-outputs:
-  output-file:
-    description: Path to agent_output.json, or empty if not found.
-    value: ${{ steps.download.outputs.output-file }}
-  bundle-file:
-    description: Path to the git bundle, or empty if not found.
-    value: ${{ steps.download.outputs.bundle-file }}
-  item-count:
-    description: Number of items in the agent output.
-    value: ${{ steps.download.outputs.item-count }}
-runs:
-  using: composite
-  steps:
-    - name: Download agent artifact
-      uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8.0.1
-      with:
-        name: ${{ inputs.artifact-name }}
-        path: agent-artifact
-    - name: Locate output and bundle
-      id: download
-      shell: bash
-      env:
-        ARTIFACT_NAME: ${{ inputs.artifact-name }}
-      run: |
-        set -euo pipefail
-        OUTPUT_FILE=""
-        BUNDLE_FILE=""
-        ITEM_COUNT="0"
-
-        if [ -f "agent-artifact/agent_output.json" ]; then
-          OUTPUT_FILE="agent-artifact/agent_output.json"
-          ITEM_COUNT=$(jq '.items | length' "$OUTPUT_FILE")
-          BUNDLE_FILE=$(ls agent-artifact/aw-*.bundle 2>/dev/null | head -1 || true)
-        else
-          echo "::error::Artifact '$ARTIFACT_NAME' contained no agent_output.json. Nothing the agent proposed will be applied."
-          exit 1
-        fi
-
-        echo "Artifact '$ARTIFACT_NAME' carries $ITEM_COUNT item(s)."
-
-        echo "output-file=$OUTPUT_FILE" >> "$GITHUB_OUTPUT"
-        echo "bundle-file=$BUNDLE_FILE" >> "$GITHUB_OUTPUT"
-        echo "item-count=$ITEM_COUNT" >> "$GITHUB_OUTPUT"
+# Managed by @plainconceptsplatform/workflows. Source: loops/actions/download-agent-output/action.yml. Update with workflows update --force; consumer edits may be overwritten.
+name: Download agent output
+description: Download the agent artifact and expose the output JSON and bundle paths.
+inputs:
+  artifact-name:
+    description: >-
+      Artifact name to download. gh-aw prefixes it with the value it derives from the
+      workflow's inputs, so a workflow_call worker must pass the activation job's
+      artifact_prefix output followed by "agent", not the bare name.
+    required: true
+outputs:
+  output-file:
+    description: Path to agent_output.json, or empty if not found.
+    value: ${{ steps.download.outputs.output-file }}
+  bundle-file:
+    description: Path to the git bundle, or empty if not found.
+    value: ${{ steps.download.outputs.bundle-file }}
+  item-count:
+    description: Number of items in the agent output.
+    value: ${{ steps.download.outputs.item-count }}
+runs:
+  using: composite
+  steps:
+    - name: Download agent artifact
+      uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8.0.1
+      with:
+        name: ${{ inputs.artifact-name }}
+        path: agent-artifact
+    - name: Locate output and bundle
+      id: download
+      shell: bash
+      env:
+        ARTIFACT_NAME: ${{ inputs.artifact-name }}
+      run: |
+        set -euo pipefail
+        OUTPUT_FILE=""
+        BUNDLE_FILE=""
+        ITEM_COUNT="0"
+
+        if [ -f "agent-artifact/agent_output.json" ]; then
+          OUTPUT_FILE="agent-artifact/agent_output.json"
+          ITEM_COUNT=$(jq '.items | length' "$OUTPUT_FILE")
+          BUNDLE_FILE=$(ls agent-artifact/aw-*.bundle 2>/dev/null | head -1 || true)
+        else
+          echo "::error::Artifact '$ARTIFACT_NAME' contained no agent_output.json. Nothing the agent proposed will be applied."
+          exit 1
+        fi
+
+        echo "Artifact '$ARTIFACT_NAME' carries $ITEM_COUNT item(s)."
+
+        echo "output-file=$OUTPUT_FILE" >> "$GITHUB_OUTPUT"
+        echo "bundle-file=$BUNDLE_FILE" >> "$GITHUB_OUTPUT"
+        echo "item-count=$ITEM_COUNT" >> "$GITHUB_OUTPUT"