Conversation
Automated sync from stranske/Workflows Template hash: cab249988e6a Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #565 | 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 by extending the weekly metrics aggregation workflow to include bot-comment App auth coverage reporting, and updating artifact selection to recognize the new metric artifact families.
Changes:
- Add a bot-comment auth coverage preflight step to the weekly metrics workflow, upload its summary artifacts, and enforce it via the existing hard-block gate.
- Extend weekly metrics artifact selection to include
bot-comment-auth-coverage-*artifact families. - Introduce a new script to summarize bot-comment auth coverage records into JSON + Markdown outputs.
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 bot-comment auth coverage preflight, uploads its outputs, and includes it in the final hard-block enforcement. |
| .github/scripts/weekly_metrics_artifacts.js | Recognizes/selects bot-comment auth coverage artifacts as part of weekly metrics collection. |
| .github/scripts/bot_comment_auth_coverage.js | New summarizer that scans downloaded artifacts for auth-coverage JSON records and produces a coverage report. |
| 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`); |
There was a problem hiding this comment.
main() writes the full pretty-printed JSON report to stdout in addition to writing it to BOT_COMMENT_AUTH_COVERAGE_JSON. This can make Actions logs noisy; consider printing only the markdown summary (or a 1-line status) to stdout and relying on the uploaded JSON artifact for the detailed payload.
| 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`); | |
| const markdown = formatBotCommentAuthCoverageMarkdown(report); | |
| fs.writeFileSync(options.output, `${JSON.stringify(report, null, 2)}\n`); | |
| fs.writeFileSync(options.markdown, markdown); | |
| process.stdout.write(markdown.endsWith('\n') ? markdown : `${markdown}\n`); |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml