Skip to content

chore: sync workflow templates - #5189

Closed
stranske wants to merge 1 commit into
phase-3from
sync/workflows-12bc63cd403e
Closed

chore: sync workflow templates#5189
stranske wants to merge 1 commit into
phase-3from
sync/workflows-12bc63cd403e

Conversation

@stranske

@stranske stranske commented Apr 27, 2026

Copy link
Copy Markdown
Owner

Sync Summary

Files Updated

  • aggregate_agent_metrics.py: Aggregates downloaded weekly agent metrics - required by agents-weekly-metrics.yml
  • source_context.js: Classifies PR workflow source context for issue, local, automation, sync, Dependabot, review follow-up, and direct GitHub work
  • coverage_monitor_summary.js: Machine-readable weekly coverage monitor checkpoint
  • weekly_metrics_artifacts.js: Bounded weekly metrics artifact selection contract
  • agents_pr_meta_keepalive.js: PR metadata handling for keepalive
  • agents_pr_meta_update_body.js: Updates PR body with agent metadata

Files Skipped

  • pr-00-gate.yml: File exists and sync_mode is create_only
  • ci.yml: File exists and sync_mode is create_only
  • dependabot.yml: File exists and sync_mode is create_only
  • AGENTS.md: Repo keeps historical Agents.md casing to avoid case-only path conflicts
  • llm_slots.json: None

Review Checklist

  • CI passes with updated workflows
  • No repo-specific customizations were overwritten

Source: stranske/Workflows
Source SHA: cfa51b59de2b1bb7cf15b4702794da43115bef26
Template hash: 12bc63cd403e
Sync branch: sync/workflows-12bc63cd403e
Consumer repo: stranske/Trend_Model_Project
Manifest: .github/sync-manifest.yml

@github-actions
chore: sync workflow templates from Workflows repo
15b0f40 
Automated sync from stranske/Workflows
Template hash: 12bc63cd403e

Changes synced from sync-manifest.yml
@stranske stranske added sync Automated sync from Workflows automated Automated sync from Workflows labels Apr 27, 2026
Copilot AI review requested due to automatic review settings April 27, 2026 04:22
chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Apr 27, 2026
View reviewed changes

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 15b0f40681

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/scripts/agents_pr_meta_update_body.js
Comment on lines +1497 to +1501
await resolveSourceContextRepairComment({
github,
owner,
repo,
prNumber: pr.number,

@chatgpt-codex-connector chatgpt-codex-connector Bot Apr 27, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Defer source-context warning resolution until issue fetch succeeds

This call resolves the <!-- missing-issue-warning --> comment before confirming that issueNumber can actually be read from GitHub. If the PR points to a stale/invalid issue (or issues.get fails), the workflow still rewrites the warning to “Workflow source detected” and then fails later, leaving a misleading “resolved” comment even though synchronization did not succeed. Resolve the warning only after a successful issue fetch.

Useful? React with 👍 / 👎.

Comment thread .github/scripts/agents_pr_meta_update_body.js
Comment on lines +1033 to +1035
/\b(?:close[sd]?|closing|fix(?:e[sd])?|fixing|resolve[sd]?|resolving|address(?:e[sd])?|addressing)\s*[:#-]?\s*#([0-9]+)\b/gi,
/\b(?:(?:relate[sd]?\s+to|references?)\s+(?:issue\s+)?|(?:source|github|linked)\s+issue\s*)[:#-]?\s*#([0-9]+)\b/gi,
];

@chatgpt-codex-connector chatgpt-codex-connector Bot Apr 27, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Recognize Refs #123 in explicit issue-sync detection

extractExplicitIssueSyncNumbers omits the common refs keyword, but extractIssueNumberFromPull still treats Refs #123 as a valid issue reference. With a non-issue workflow marker present, this mismatch causes resolveNonIssueWorkflowSourceContextForBodySync to incorrectly keep skipping issue-backed body sync for PRs that use Refs #.... Add refs? (and keep parity with issue-number extraction rules) to avoid inconsistent behavior.

Useful? React with 👍 / 👎.

Copilot AI reviewed Apr 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@stranske

stranske commented Apr 27, 2026

Copy link
Copy Markdown
Owner Author

Closing this superseded Workflows sync PR without deleting the branch. A newer post-#1952 sync wave is open on sync/workflows-20d0e241bca2; continue with that PR after the #1952 verifier is terminal.

@stranske stranske closed this Apr 27, 2026
@agents-workflows-bot agents-workflows-bot Bot mentioned this pull request Apr 27, 2026
Open
@stranske
stranske deleted the sync/workflows-12bc63cd403e branch May 14, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments

Assignees

No one assigned

Labels

automated Automated sync from Workflows sync Automated sync from Workflows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@stranske