chore: sync workflow templates - #618
Conversation
Automated sync from stranske/Workflows Template hash: 3df44990832a Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #618 | 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
Syncs this repository’s workflow/scripts template set with stranske/Workflows (source SHA da4862697…) to expand weekly metrics artifact handling and improve agent telemetry/coverage reporting.
Changes:
- Add weekly metrics artifact download manifest generation, plus richer aggregation outputs (Markdown + JSON contract).
- Extend terminal disposition + bot-comment auth coverage logic and emit additional terminal disposition telemetry artifacts.
- Refine keepalive loop task counting (actionable-only) and update PR meta preamble behavior for campaign issues.
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
scripts/aggregate_agent_metrics.py |
Adds artifact/source attribution, parse-error detail tracking, legacy JSON fallback, and emits a JSON summary contract alongside Markdown. |
CLAUDE.md |
Documents optional GitNexus usage as a local-only indexing/cache helper. |
AGENTS.md |
Mirrors optional GitNexus guidance for agent/coding-assistant context. |
.github/workflows/agents-weekly-metrics.yml |
Generates/records an artifact download manifest, sanitizes artifact dirs, and uploads JSON summary + new artifacts. |
.github/workflows/agents-verify-to-new-pr.yml |
Adds verifier follow-up ledger telemetry and label/agent resolution tweaks; updates action references. |
.github/workflows/agents-bot-comment-handler.yml |
Emits a wrapper terminal disposition artifact with explicit skip reasons and uploads it for coverage/telemetry. |
.github/workflows/agents-81-gate-followups.yml |
Tweaks jq invocation to compact/null-input mode for metrics JSON generation. |
.github/scripts/weekly_metrics_download_manifest.js |
New script to initialize/record/finalize a machine-readable artifact download/unzip manifest + Markdown table. |
.github/scripts/weekly_metrics_artifacts.js |
Adds priority-family status reporting and “latest candidate by family” metadata into the selection report/markdown. |
.github/scripts/terminal_disposition_coverage.js |
Adds verifier model compatibility checks, priority-family status reporting, and expands markdown/report outputs accordingly. |
.github/scripts/terminal_disposition.js |
Extends terminal disposition schema normalization and adds verifier follow-up ledger/policy normalization utilities. |
.github/scripts/keepalive_loop.js |
Counts only “actionable” checklist items (filters placeholders/status metrics) and suppresses some noisy comments. |
.github/scripts/coverage_monitor_summary.js |
New script that summarizes weekly coverage monitor reports into a single JSON/Markdown contract. |
.github/scripts/bot_comment_auth_coverage.js |
Improves organic evidence reporting, including wrapper-driven “reusable invocation expected” skip semantics. |
.github/scripts/agents_pr_meta_update_body.js |
Adjusts preamble to differentiate campaign issues vs normal “Closes #…” behavior and emits issue meta marker. |
.github/actions/setup-api-client/action.yml |
Adds a redacted setup/auth/dependency “setup_contract” output and minor dependency-check commentary. |
| details[index] = replace(existing, count=existing.count + detail.count) | ||
| return | ||
|
|
||
| details.append(replace(detail, line=None)) |
There was a problem hiding this comment.
_append_parse_error_detail() is meant to cap stored parse-error details, but when len(details) >= detail_limit and no existing aggregate entry matches, it appends a new detail anyway (with line=None), which can grow the list beyond the configured limit. Adjust the overflow behavior so the list never exceeds detail_limit (e.g., aggregate into an existing "overflow" bucket / last entry, or drop additional details after incrementing a counter).
| - 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 now uses floating action tags (e.g., actions/github-script@v9, actions/checkout@v6, actions/setup-python@v6, actions/upload-artifact@v7). Elsewhere in this repo, the same actions are typically pinned to full commit SHAs with an inline major-version comment (e.g., .github/workflows/agents-weekly-metrics.yml pins actions/checkout and actions/setup-python). Please re-pin these actions to SHAs to reduce supply-chain risk and keep behavior reproducible.
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Source SHA:
da4862697843b534523be3398900d8870719ab21Template hash:
3df44990832aSync branch:
sync/workflows-3df44990832aConsumer repo:
stranske/TemplateManifest:
.github/sync-manifest.yml