chore: sync workflow templates - #305
Closed
stranske wants to merge 1 commit into
Closed
Conversation
Automated sync from stranske/Workflows Template hash: c91ca2f0d673 Changes synced from sync-manifest.yml
stranske
added
sync
stranske
added
sync
There was a problem hiding this comment.
Pull request overview
Syncs updated workflow support scripts from the shared Workflows templates, expanding weekly metrics aggregation and improving the robustness/shape of generated metrics and artifact download manifests.
Changes:
- Add support for the
codex-cli-freshnessartifact family and metric summary/contract output. - Expand weekly metrics download manifest selection details and normalize per-artifact result shape before updates.
- Centralize optional-field normalization for terminal disposition records and align verifier model “requires model metadata” evaluation logic.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
scripts/aggregate_agent_metrics.py |
Recognizes and summarizes workflows-codex-cli-freshness/v1 records; adds verifier CLI version counting. |
.github/scripts/weekly_metrics_download_manifest.js |
Adds richer selection details, normalizes artifact result objects before updating, exports helper. |
.github/scripts/weekly_metrics_artifacts.js |
Adds codex-cli-freshness to exact/prefix matching and priority families. |
.github/scripts/terminal_disposition_coverage.js |
Updates verifier model “requires codex model” logic to treat non-evaluate (including empty) as requiring model metadata. |
.github/scripts/terminal_disposition.js |
Adds normalizeOptionalValue helper, adds llm_cli_version, and reuses normalization in multiple record builders. |
Owner
Author
|
Closing as superseded by merged sync PR #324 (sync/workflows-3e5a27aaceee). No branch deletion performed. |
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
Source SHA:
8ffac0178658825b0e14803a8ce7193bf6118f89Template hash:
c91ca2f0d673Sync branch:
sync/workflows-c91ca2f0d673Consumer repo:
stranske/Inv-Man-IntakeManifest:
.github/sync-manifest.yml