Conversation
Automated sync from stranske/Workflows Template hash: 8c7e6f62db72 Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #247 | 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 aggregation workflow to include bot-comment auth coverage reporting and artifact selection support.
Changes:
- Add a bot-comment auth coverage preflight step to the weekly metrics workflow and publish its summary/artifacts.
- Extend weekly metrics artifact selection to recognize/prioritize bot-comment auth coverage artifact families.
- Introduce
.github/scripts/bot_comment_auth_coverage.jsto compute and render the bot-comment auth coverage summary.
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 |
Checks out the new script, runs a new preflight step, uploads new summary artifacts, and gates on both coverage exit statuses. |
.github/scripts/weekly_metrics_artifacts.js |
Adds artifact family prefixes and priority families for bot-comment auth coverage artifacts. |
.github/scripts/bot_comment_auth_coverage.js |
New script to scan downloaded artifacts for auth coverage records and emit JSON/Markdown summaries with optional hard-block behavior. |
| BOT_COMMENT_AUTH_COVERAGE_MD: bot-comment-auth-coverage-summary.md | ||
| BOT_COMMENT_AUTH_COVERAGE_MODE: ${{ vars.BOT_COMMENT_AUTH_COVERAGE_MODE || 'warning-only' }} | ||
| BOT_COMMENT_AUTH_HARD_BLOCK_APPROVED: ${{ vars.BOT_COMMENT_AUTH_HARD_BLOCK_APPROVED || 'false' }} | ||
| BOT_COMMENT_WRAPPER_EXPECTED_AUTH_MODE: ${{ vars.BOT_COMMENT_WRAPPER_EXPECTED_AUTH_MODE || 'client-id' }} |
There was a problem hiding this comment.
The workflow passes BOT_COMMENT_WRAPPER_EXPECTED_AUTH_MODE / BOT_COMMENT_REUSABLE_ALLOWED_AUTH_MODES, but it never exports BOT_COMMENT_WRAPPER_ALLOWED_AUTH_MODES. Since .github/scripts/bot_comment_auth_coverage.js reads process.env.BOT_COMMENT_WRAPPER_ALLOWED_AUTH_MODES for wrapper policy configuration, any repo/org variable with that name won’t take effect here. Consider adding it to this env: block (from vars.BOT_COMMENT_WRAPPER_ALLOWED_AUTH_MODES) or removing that unused configuration path from the script to avoid a confusing/partially-configurable setup.
| BOT_COMMENT_WRAPPER_EXPECTED_AUTH_MODE: ${{ vars.BOT_COMMENT_WRAPPER_EXPECTED_AUTH_MODE || 'client-id' }} | |
| BOT_COMMENT_WRAPPER_EXPECTED_AUTH_MODE: ${{ vars.BOT_COMMENT_WRAPPER_EXPECTED_AUTH_MODE || 'client-id' }} | |
| BOT_COMMENT_WRAPPER_ALLOWED_AUTH_MODES: ${{ vars.BOT_COMMENT_WRAPPER_ALLOWED_AUTH_MODES }} |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml