Conversation
Automated sync from stranske/Workflows Template hash: 9d62510f22c9 Changes synced from sync-manifest.yml
stranske
added
automated
Collaborator
Collaborator
🤖 Keepalive Loop StatusPR #897 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
Collaborator
Keepalive Work Log (click to expand)
|
Contributor
There was a problem hiding this comment.
Pull request overview
Syncs workflow templates from stranske/Workflows to keep weekly metrics aggregation and its preflight scripts aligned with the upstream artifact-selection and bot-comment auth coverage contracts.
Changes:
- Update
agents-weekly-metricsworkflow to explicitly set up Node 20 and pass wrapper allowed auth modes into the bot-comment auth coverage preflight. - Tighten weekly metrics artifact family classification for bot-comment auth coverage artifacts using bounded regex contracts.
- Harden
bot_comment_auth_coverage.jsparsing/collection (read/parse error accounting, stricter artifact dir/file contract checks) and refine hard-block eligibility logic/reporting.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/agents-weekly-metrics.yml | Adds explicit Node setup and new env wiring for bot-comment auth coverage; improves hard-block error annotation. |
| .github/scripts/weekly_metrics_artifacts.js | Uses patterned artifact family matching for bot-comment auth coverage artifacts to enforce a bounded selection contract. |
| .github/scripts/bot_comment_auth_coverage.js | Introduces artifact/file contracts and more robust summarization + enforcement eligibility logic for the preflight report. |
Owner
Author
|
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. |
This branch was previously deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml