Conversation
Automated sync from stranske/Workflows Template hash: 72017d1bb480 Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #248 | 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 weekly metrics workflow templates to include bot-comment App auth coverage reporting alongside existing terminal disposition coverage, and updates artifact selection to include the new bot-comment coverage artifact families.
Changes:
- Adds a bot-comment auth coverage preflight step to the weekly metrics workflow and uploads its summary artifacts.
- Extends weekly metrics artifact selection to recognize bot-comment auth coverage wrapper/reusable artifact families.
- Introduces a new
.github/scripts/bot_comment_auth_coverage.jsscript to aggregate and report bot-comment auth coverage.
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 and includes resulting summary artifacts; consolidates hard-block honoring. |
.github/scripts/weekly_metrics_artifacts.js |
Adds bot-comment auth coverage artifact prefixes/families to selection logic. |
.github/scripts/bot_comment_auth_coverage.js |
New aggregation + reporting script for bot-comment auth coverage (JSON + markdown). |
| name.startsWith('bot-comment-auth-coverage-'); | ||
| }); | ||
| return { | ||
| schema: cleanString(report.schema), |
There was a problem hiding this comment.
In normalizeArtifactSelectionSummary(), the non-error path sets schema: cleanString(report.schema) without a fallback. If the artifact selection JSON is missing schema, this will produce an empty schema value (unlike terminal_disposition_coverage.js which always defaults to workflows-weekly-metrics-artifact-selection/v1). Consider defaulting the schema here as well to keep the report stable for downstream consumers and markdown output.
| schema: cleanString(report.schema), | |
| schema: cleanString(report.schema) || 'workflows-weekly-metrics-artifact-selection/v1', |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml