Conversation
Automated sync from stranske/Workflows Template hash: 43b0c81e73e6 Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #845 | 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 workflow templates from stranske/Workflows, extending the weekly metrics workflow to include a new bot-comment App auth coverage preflight and ensuring the artifact selector recognizes the new artifact families.
Changes:
- Extend
agents-weekly-metrics.ymlto run a newbot_comment_auth_coverage.jspreflight and upload its JSON/MD outputs. - Update weekly artifact selection to include/prioritize
bot-comment-auth-coverage-*artifact families. - Add
bot_comment_auth_coverage.jsscript to summarize bot-comment auth coverage (JSON + markdown) and optionally hard-block based on repo vars.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
.github/workflows/agents-weekly-metrics.yml |
Adds bot-comment auth coverage preflight step, uploads new summary artifacts, and enforces combined hard-block status. |
.github/scripts/weekly_metrics_artifacts.js |
Includes new bot-comment auth coverage artifact prefixes/families in weekly selection and priority list. |
.github/scripts/bot_comment_auth_coverage.js |
New script to scan downloaded artifacts, summarize auth-mode coverage, emit JSON/MD, and return fail status when hard-block mode is active. |
| if [ -f bot-comment-auth-coverage-summary.md ]; then | ||
| cat bot-comment-auth-coverage-summary.md >> "$GITHUB_STEP_SUMMARY" | ||
| if [ -f agent-weekly-metrics.md ]; then | ||
| { | ||
| printf '\n' | ||
| cat bot-comment-auth-coverage-summary.md |
There was a problem hiding this comment.
The step summary/append logic checks for a hard-coded markdown filename (bot-comment-auth-coverage-summary.md) even though the output path is already configurable via BOT_COMMENT_AUTH_COVERAGE_MD. This will break the summary output if the env var is changed later; use the env var (or a local variable derived from it) for the file existence check and cat operations instead of duplicating the literal filename.
| if [ -f bot-comment-auth-coverage-summary.md ]; then | |
| cat bot-comment-auth-coverage-summary.md >> "$GITHUB_STEP_SUMMARY" | |
| if [ -f agent-weekly-metrics.md ]; then | |
| { | |
| printf '\n' | |
| cat bot-comment-auth-coverage-summary.md | |
| bot_comment_auth_coverage_md="${BOT_COMMENT_AUTH_COVERAGE_MD}" | |
| if [ -f "${bot_comment_auth_coverage_md}" ]; then | |
| cat "${bot_comment_auth_coverage_md}" >> "$GITHUB_STEP_SUMMARY" | |
| if [ -f agent-weekly-metrics.md ]; then | |
| { | |
| printf '\n' | |
| cat "${bot_comment_auth_coverage_md}" |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml