Conversation
Automated sync from stranske/Workflows Template hash: f8490e265811 Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #84 | 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 workflow to include bot-comment App auth coverage signals in the aggregated reporting and artifact selection.
Changes:
- Adds a “Bot-comment auth coverage preflight” step to the weekly metrics workflow and includes its outputs in uploaded artifacts.
- Extends weekly artifact selection to recognize and prioritize
bot-comment-auth-coverage-*artifact families. - Introduces a new Node script to aggregate bot-comment auth coverage JSON records into a summary JSON + markdown report.
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 preflight, appends results to step summary/weekly report, uploads new coverage artifacts, and enforces both coverage hard-blocks. |
.github/scripts/weekly_metrics_artifacts.js |
Adds bot-comment auth coverage artifact prefixes/families so they are discovered and prioritized during artifact selection. |
.github/scripts/bot_comment_auth_coverage.js |
New aggregator that scans artifacts, summarizes bot-comment auth coverage status/policy, and emits JSON + markdown. |
| const basename = path.basename(normalized); | ||
| if (!normalized.endsWith('.json')) return false; | ||
| return normalized.includes('/bot-comment-auth-coverage-wrapper-') || | ||
| normalized.includes('/bot-comment-auth-coverage-reusable-') || | ||
| basename === 'wrapper.json' || | ||
| basename === 'reusable.json'; |
There was a problem hiding this comment.
isPotentialAuthCoverageFile() matches any wrapper.json / reusable.json basename anywhere under the artifacts directory. Those names are generic and can easily collide with unrelated artifacts, which would inflate parse_errors (and potentially trigger warnings/failures) even when bot-comment auth coverage artifacts are fine. Consider tightening the matcher to only include files whose path/name includes bot-comment-auth-coverage- (or another unique prefix), and drop the generic basename checks (or scope them under a bot-comment-auth-coverage-* directory).
| const basename = path.basename(normalized); | |
| if (!normalized.endsWith('.json')) return false; | |
| return normalized.includes('/bot-comment-auth-coverage-wrapper-') || | |
| normalized.includes('/bot-comment-auth-coverage-reusable-') || | |
| basename === 'wrapper.json' || | |
| basename === 'reusable.json'; | |
| if (!normalized.endsWith('.json')) return false; | |
| if (!normalized.includes('bot-comment-auth-coverage-')) return false; | |
| return normalized.includes('/bot-comment-auth-coverage-wrapper-') || | |
| normalized.includes('/bot-comment-auth-coverage-reusable-'); |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml