Conversation
Automated sync from stranske/Workflows Template hash: 72017d1bb480 Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #843 | 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 agent metrics workflow to include a new “bot-comment app auth coverage” preflight and ensuring the artifact selector recognizes the new metrics artifacts.
Changes:
- Add a bot-comment auth coverage preflight step to the weekly metrics workflow and upload its summary artifacts.
- Extend weekly metrics artifact selection to include bot-comment auth coverage artifact families.
- Introduce a new Node script to summarize bot-comment auth coverage JSON records into JSON/Markdown reports.
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 |
Runs the new bot-comment auth coverage preflight, appends its markdown to summaries, and uploads the new artifacts; gates hard-block enforcement with a combined step. |
.github/scripts/weekly_metrics_artifacts.js |
Recognizes and prioritizes new bot-comment-auth-coverage-* artifact families during selection. |
.github/scripts/bot_comment_auth_coverage.js |
New summarizer producing coverage status + markdown table for wrapper/reusable bot-comment auth modes. |
| fs.writeFileSync(options.output, `${JSON.stringify(report, null, 2)}\n`); | ||
| fs.writeFileSync(options.markdown, formatBotCommentAuthCoverageMarkdown(report)); | ||
| process.stdout.write(`${JSON.stringify(report, null, 2)}\n`); | ||
| return report.status === 'fail' ? 1 : 0; |
There was a problem hiding this comment.
main() always writes the full JSON report to stdout. Since the report includes input_files (potentially many paths) this can bloat Actions logs and make failures harder to read or hit log limits. Consider printing the markdown summary (like terminal_disposition_coverage.js), or only emitting the JSON to stdout behind a debug flag / emitting a compact summary instead.
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml