chore: sync workflow templates - #613
Conversation
Automated sync from stranske/Workflows Template hash: 8a669b3c40c8 Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #613 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
Keepalive Work Log (click to expand)
|
There was a problem hiding this comment.
Pull request overview
Synchronizes workflow/templates from stranske/Workflows, expanding the weekly metrics pipeline and terminal-disposition telemetry so weekly reporting can capture artifact download status, verifier follow-up ledger records, and additional coverage/compatibility contracts.
Changes:
- Add an artifact download manifest (JSON + markdown) to weekly metrics collection, and emit a machine-readable weekly metrics summary JSON contract.
- Extend terminal-disposition + verifier telemetry (verifier follow-up ledger records, verifier model compatibility checks, richer selection status reporting).
- Refine keepalive checklist/task accounting to exclude placeholder/status-metric items from “actionable” task counts.
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
scripts/aggregate_agent_metrics.py |
Adds metric source attribution, parse error details, verifier follow-up ledger aggregation, and emits a JSON summary contract. |
.github/workflows/agents-weekly-metrics.yml |
Tracks artifact download/unzip outcomes via a manifest; uploads new JSON artifacts. |
.github/workflows/agents-verify-to-new-pr.yml |
Adds verifier follow-up ledger emission + labels, and updates multiple action references. |
.github/workflows/agents-bot-comment-handler.yml |
Adds skip-reason output and emits a wrapper terminal-disposition record artifact. |
.github/workflows/agents-81-gate-followups.yml |
Minor tweak to jq invocation for metrics JSON generation. |
.github/scripts/weekly_metrics_download_manifest.js |
New script to initialize/record/finalize artifact download manifest (JSON + markdown). |
.github/scripts/weekly_metrics_artifacts.js |
Adds “priority family” availability/selection status reporting to selection output. |
.github/scripts/terminal_disposition_coverage.js |
Adds verifier model compatibility checks and richer terminal artifact family status reporting. |
.github/scripts/terminal_disposition.js |
Adds verifier follow-up ledger + policy normalization helpers and extra terminal-disposition fields. |
.github/scripts/keepalive_loop.js |
Switches to actionable-only checkbox counting and filters out status/placeholder checklist items. |
.github/scripts/coverage_monitor_summary.js |
New weekly contract summarizing multiple coverage monitors into one status/next-action output. |
.github/scripts/bot_comment_auth_coverage.js |
Adds reusable-wrapper skip semantics for organic evidence and richer missing/skipped requirement details. |
.github/scripts/agents_pr_meta_update_body.js |
Adds issue metadata comment + “Closes” vs campaign-related wording based on source issue labels. |
.github/actions/setup-api-client/action.yml |
Adds a redacted machine-readable setup_contract output describing auth modes and dependency state. |
| - name: Checkout repository | ||
| if: steps.check-merged.outputs.merged == 'true' | ||
| uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6 | ||
| uses: actions/checkout@v6 |
There was a problem hiding this comment.
This changes actions/checkout from a pinned commit SHA to @v6. Repo workflows consistently pin actions to commit SHAs (see .github/workflows/agents-weekly-metrics.yml:29-31) for reproducibility/security; please pin this action to the same SHA style.
| uses: actions/checkout@v6 | |
| uses: actions/checkout@FULL_LENGTH_COMMIT_SHA_FOR_V6 |
| id: chain-check | ||
| if: steps.check-merged.outputs.merged == 'true' | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9 | ||
| uses: actions/github-script@v9 |
There was a problem hiding this comment.
Floating actions/github-script@v9 deviates from the repo’s pinned-SHA workflow convention (e.g. .github/workflows/agents-bot-comment-handler.yml:86). Please pin to a commit SHA with an inline # v9 comment.
| steps.chain-check.outputs.exceeded != 'true' && | ||
| steps.extract-verdict.outputs.needs_human == 'true' | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9 | ||
| uses: actions/github-script@v9 |
There was a problem hiding this comment.
Floating actions/github-script@v9 deviates from the repo’s pinned-SHA workflow convention (e.g. .github/workflows/agents-bot-comment-handler.yml:86). Please pin to a commit SHA with an inline # v9 comment.
| steps.extract-verdict.outputs.needs_human != 'true' && | ||
| steps.create-issue.outputs.issue_number != '' | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9 | ||
| uses: actions/github-script@v9 |
There was a problem hiding this comment.
Floating actions/github-script@v9 deviates from the repo’s pinned-SHA workflow convention (e.g. .github/workflows/agents-bot-comment-handler.yml:86). Please pin to a commit SHA with an inline # v9 comment.
| steps.chain-check.outputs.exceeded != 'true' && | ||
| steps.extract-verdict.outputs.needs_human != 'true' | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9 | ||
| uses: actions/github-script@v9 |
There was a problem hiding this comment.
Floating actions/github-script@v9 deviates from the repo’s pinned-SHA workflow convention (e.g. .github/workflows/agents-bot-comment-handler.yml:86). Please pin to a commit SHA with an inline # v9 comment.
| id: collect | ||
| if: steps.check-merged.outputs.merged == 'true' | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9 | ||
| uses: actions/github-script@v9 |
There was a problem hiding this comment.
Floating actions/github-script@v9 deviates from the repo’s pinned-SHA workflow convention (e.g. .github/workflows/agents-bot-comment-handler.yml:86). Please pin to a commit SHA with an inline # v9 comment.
| uses: actions/github-script@v9 | |
| uses: actions/github-script@<FULL_LENGTH_V9_COMMIT_SHA> # v9 |
| steps.chain-check.outputs.exceeded != 'true' && | ||
| steps.extract-verdict.outputs.needs_human != 'true' | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9 | ||
| uses: actions/github-script@v9 |
There was a problem hiding this comment.
Floating actions/github-script@v9 deviates from the repo’s pinned-SHA workflow convention (e.g. .github/workflows/agents-bot-comment-handler.yml:86). Please pin to a commit SHA with an inline # v9 comment.
| uses: actions/github-script@v9 | |
| uses: actions/github-script@<PINNED_V9_COMMIT_SHA> # v9 |
| steps.chain-check.outputs.exceeded != 'true' && | ||
| steps.extract-verdict.outputs.needs_human != 'true' | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9 | ||
| uses: actions/github-script@v9 |
There was a problem hiding this comment.
Floating actions/github-script@v9 deviates from the repo’s pinned-SHA workflow convention (e.g. .github/workflows/agents-bot-comment-handler.yml:86). Please pin to a commit SHA with an inline # v9 comment.
| - name: Remove trigger label | ||
| if: steps.check-merged.outputs.merged == 'true' | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9 | ||
| uses: actions/github-script@v9 |
There was a problem hiding this comment.
Floating actions/github-script@v9 deviates from the repo’s pinned-SHA workflow convention (e.g. .github/workflows/agents-bot-comment-handler.yml:86). Please pin to a commit SHA with an inline # v9 comment.
| - name: Check PR is merged | ||
| id: check-merged | ||
| uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9 | ||
| uses: actions/github-script@v9 |
There was a problem hiding this comment.
This workflow switches from pinned-action SHAs to a floating tag (actions/github-script@v9). In this repo, workflows typically pin third-party actions to a full commit SHA (e.g. actions/github-script@3a2844… # v9 in .github/workflows/agents-weekly-metrics.yml:314) to avoid supply-chain drift. Please revert to the pinned SHA form here.
| uses: actions/github-script@v9 | |
| uses: actions/github-script@3a2844bcfa5f61316c1b5c2c7f3d5a2a7b6e3f2d # v9 |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml