Skip to content

chore: sync workflow templates - #5188

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

chore: sync workflow templates#5188
stranske wants to merge 1 commit into
phase-3from
sync/workflows-da74423d2b57

Conversation

@stranske

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: ee8f0c1dbf1c49292dbf610869156a55ac0966d5
Template hash: da74423d2b57
Sync branch: sync/workflows-da74423d2b57
Consumer repo: stranske/Trend_Model_Project
Manifest: .github/sync-manifest.yml

Automated sync from stranske/Workflows
Template hash: da74423d2b57

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:03
@stranske stranske added the automated Automated sync from Workflows label Apr 27, 2026

@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: 7caf059319

ℹ️ 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 on lines +1041 to +1042
if (issueNumber && hasExplicitIssueSyncReference(pr)) {
return null;

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 Preserve non-issue override unless referenced issue matches

This new guard drops the explicit non-issue override whenever hasExplicitIssueSyncReference(pr) is true, but run() still uses issueNumber from extractIssueNumberFromPull (loaded from agents_pr_meta_keepalive.js), which prioritizes branch-derived issue-<n> values over body references. In a PR with an explicit non-issue marker plus a stale branch name (e.g., issue-99) and body text like Closes #123, this now re-enables issue-body sync and can pull content from the wrong issue (#99), overwriting PR metadata with unrelated scope/testing text.

Useful? React with 👍 / 👎.

@stranske

Copy link
Copy Markdown
Owner Author

Closing as superseded by the latest Workflows sync wave sync/workflows-12bc63cd403e after Workflows PR #1951 merged. Keeping the newest sync PR open for verification/check follow-up.

@stranske stranske closed this Apr 27, 2026
@stranske
stranske removed the request for review from Copilot April 27, 2026 04:25
@stranske
stranske deleted the sync/workflows-da74423d2b57 branch May 14, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated Automated sync from Workflows sync Automated sync from Workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant