Skip to content

ENG-7639: Retarget drift caller to agents-md-drift - #144

Merged
nsportsman merged 1 commit into
mainfrom
eng-7639-agents-md-drift
Sep 5, 2026
Merged

ENG-7639: Retarget drift caller to agents-md-drift#144
nsportsman merged 1 commit into
mainfrom
eng-7639-agents-md-drift

Conversation

@nsportsman

Copy link
Copy Markdown
Collaborator

Summary

  • retarget the instruction-file drift caller to agents-md-drift.yml@defe609cc7ff3f7630a36928d620c29bcf3b7930 (v2.19.3)
  • rename the workflow to Instruction-File Drift Detection and concurrency instruction-drift-

Linear: ENG-7639

@nsportsman
nsportsman requested a review from a team as a code owner September 5, 2026 18:21
@nsportsman
nsportsman requested review from DS-KoolAid, EvanLeleux, MarioBartolome, aashish-sec, gitoso, krkt5 and peter-kwan and removed request for a team September 5, 2026 18:21
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions github-actions 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

Critical Issues

None.

Security

No security concerns flagged.

Suggestions

No suggestions.

No critical issues — LGTM pending human review.


Reviewed by Codex (gpt-5.5)

@github-actions github-actions 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.

Claude Review

Critical issues

None.

Security

  • The reusable workflow is correctly pinned to a full commit SHA (defe609cc7ff3f7630a36928d620c29bcf3b7930) rather than a mutable tag — good practice. Worth confirming this SHA actually corresponds to v2.19.3 in praetorian-inc/public-workflows, since the trailing version comment is trust-on-first-use and not enforced by GitHub.

Test coverage

No production *.go/*.ts/*.py changes — CI workflow only, no tests needed.

No critical issues — LGTM pending human review.

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 0d3e7f44-0fbe-4a6e-b203-3b48a53c654a

📥 Commits

Reviewing files that changed from the base of the PR and between 6cf0ea9 and 0cd5bf1.

📒 Files selected for processing (1)
  • .github/workflows/agents-md-drift.yml

Included review availability: Your plan provides up to 10 included reviews per hour; 1 remains after this review.


Walkthrough

The workflow now uses agents-md-drift.yml version v2.19.3 instead of claude-md-drift.yml version v2.16.15. The workflow name and concurrency group now use Instruction terminology. Pull-request triggers, cancellation behavior, permissions, and secret forwarding remain unchanged.

Merge Risk: ⚪ Minimal · up to 0cd5b

Instruction-file drift detection now uses the updated reusable workflow and terminology. No merge-blocking risk is currently identified.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch eng-7639-agents-md-drift

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions 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.

Gemini Review

No critical issues — LGTM pending human review.


Reviewed by Gemini (gemini-3.1-pro-preview)

@nsportsman
nsportsman merged commit ae5082e into main Sep 5, 2026
30 checks passed
@nsportsman
nsportsman deleted the eng-7639-agents-md-drift branch September 5, 2026 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant