Conversation
Automated sync from stranske/Workflows Template hash: ff1c00361abc Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #895 | 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, updating the weekly metrics workflow and its supporting scripts to standardize Node setup and tighten artifact-selection / bot-comment auth coverage preflight behavior.
Changes:
- Update
agents-weekly-metricsworkflow to useactions/setup-nodeand refine coverage hard-block reporting. - Tighten weekly metrics artifact family classification for bot-comment auth coverage artifacts using explicit regex patterns.
- Expand bot-comment auth coverage preflight to better track input parsing/reading stats and refine hard-block eligibility behavior.
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 | Switches to explicit Node setup and adds wrapper allowed-auth env var; improves hard-block error annotation text. |
| .github/scripts/weekly_metrics_artifacts.js | Moves bot-comment auth coverage artifacts from prefix-based family detection to bounded regex-based detection. |
| .github/scripts/bot_comment_auth_coverage.js | Enforces stricter input file contract, adds richer summary stats, and refactors component policy overrides / enforcement logic. |
| return { | ||
| schema: 'workflows-bot-comment-auth-organic-evidence/v1', | ||
| required_events: requiredEvents, | ||
| required_components: components, | ||
| expected_mode: expectedMode === 'unknown' ? '' : expectedMode, | ||
| event_counts: eventCounts, | ||
| blockers: [], |
There was a problem hiding this comment.
In summarizeOrganicEvidence(), when organic checks are enabled (required_events/components configured) but records.length === 0, the function returns status 'no-data' with an empty blockers list. This makes enforcement.blockers omit any indication that required organic evidence is missing, unlike the partial-data path which emits missing-organic-* blockers. Consider emitting at least one blocker (e.g., 'organic-evidence-no-data') or populating missing-organic blockers even in the no-records case so the overall summary explains why organic evidence is failing/no-data.
| return { | |
| schema: 'workflows-bot-comment-auth-organic-evidence/v1', | |
| required_events: requiredEvents, | |
| required_components: components, | |
| expected_mode: expectedMode === 'unknown' ? '' : expectedMode, | |
| event_counts: eventCounts, | |
| blockers: [], | |
| const blockers = organicChecksDisabled ? [] : ['organic-evidence-no-data']; | |
| return { | |
| schema: 'workflows-bot-comment-auth-organic-evidence/v1', | |
| required_events: requiredEvents, | |
| required_components: components, | |
| expected_mode: expectedMode === 'unknown' ? '' : expectedMode, | |
| event_counts: eventCounts, | |
| blockers, |
|
Superseded by newer Workflows sync PR #920 (sync/workflows-daec38e5dbf4). Closing this duplicate sync lane so review and verification can focus on the latest template hash. |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml