Conversation
Automated sync from stranske/Workflows Template hash: 2da5149cf7d4 Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #254 | 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:
- 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
.github/scripts/bot_comment_auth_coverage.jsto summarize coverage from downloaded artifacts and optionally hard-block.
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 and includes its outputs in uploaded weekly summary artifacts. |
.github/scripts/weekly_metrics_artifacts.js |
Recognizes bot-comment auth coverage artifacts so they can be selected/downloaded for the weekly aggregation job. |
.github/scripts/bot_comment_auth_coverage.js |
New script that scans downloaded artifacts, summarizes auth mode coverage, and supports warning-only vs hard-block enforcement. |
|
|
||
| function componentCoverageStatus(blockers, policy, latest) { | ||
| if (blockers.length === 0) return 'pass'; | ||
| if (!latest && policy.missing_record_severity === 'no-data') return 'no-data'; |
There was a problem hiding this comment.
componentCoverageStatus() returns no-data whenever the latest record is missing and missing_record_severity is no-data, even if there are other blockers (e.g., invalid_*_expected-auth-mode). This can misclassify a configuration problem as no-data and hide actionable warnings in the per-component table. Consider returning warning when blockers include anything other than the missing-record blocker, even when latest is null.
| if (!latest && policy.missing_record_severity === 'no-data') return 'no-data'; | |
| if (!latest && policy.missing_record_severity === 'no-data') { | |
| const hasOnlyMissingBlockers = blockers.every(isComponentMissingBlocker); | |
| if (hasOnlyMissingBlockers) return 'no-data'; | |
| } |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml