Skip to content

chore: sync workflow templates - #5187

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

chore: sync workflow templates#5187
stranske wants to merge 1 commit into
phase-3from
sync/workflows-54179617a70c

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

Automated sync from stranske/Workflows
Template hash: 54179617a70c

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 03:31
@stranske stranske added sync Automated sync from Workflows automated Automated sync from Workflows labels 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: 90f808e241

ℹ️ 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 +1488 to +1492
prNumber: pr.number,
comments,
sourceContext,
core,
});

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 Skip non-issue resolution text for issue-linked PRs

This new unconditional resolution path runs even when an issueNumber is already present, but resolveSourceContextRepairComment() uses buildSourceContextResolvedCommentBody, which states that no linked GitHub issue is required. In the issue-linked path, that produces contradictory guidance in the existing warning comment and can mislead contributors into removing issue references that the workflow is actively using for body sync. Please gate this call to non-issue contexts or use a separate resolved message for issue-backed PRs.

Useful? React with 👍 / 👎.

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

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 deleted the sync/workflows-54179617a70c 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.

2 participants