Conversation
Automated sync from stranske/Workflows Template hash: cab249988e6a Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #841 | 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 from stranske/Workflows, adding bot-comment App auth coverage reporting to the existing weekly aggregation pipeline.
Changes:
- Adds a new bot-comment auth coverage preflight step to the weekly metrics workflow and uploads its JSON/MD outputs as artifacts.
- Extends weekly artifact selection to include bot-comment auth coverage artifact families.
- Introduces
.github/scripts/bot_comment_auth_coverage.jsto summarize bot-comment auth coverage across downloaded artifacts (warning-only by default, optional 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 honors its exit status in the final hard-block gate. |
.github/scripts/weekly_metrics_artifacts.js |
Adds bot-comment auth coverage artifact prefixes/families to the bounded artifact selection logic. |
.github/scripts/bot_comment_auth_coverage.js |
New summarizer producing JSON + markdown reports for bot-comment auth coverage and enforcement status. |
| name.startsWith('bot-comment-auth-coverage-'); | ||
| }); | ||
| return { | ||
| schema: cleanString(report.schema), |
There was a problem hiding this comment.
normalizeArtifactSelectionSummary() returns an empty schema when the artifact-selection report JSON omits schema (pass-path uses cleanString(report.schema) with no fallback), while the missing/parse-error paths fall back to workflows-weekly-metrics-artifact-selection/v1. For consistency (and to avoid emitting reports with schema: ""), use the same fallback in the pass-path (e.g., cleanString(report.schema) || 'workflows-weekly-metrics-artifact-selection/v1').
| 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