Skip to content

fix(ci): add skill-drift workflow and block floating GHA tags in pre-commit#478

Closed
castrojo wants to merge 1 commit into
mainfrom
fix/ci-skill-drift-and-floating-tags
Closed

fix(ci): add skill-drift workflow and block floating GHA tags in pre-commit#478
castrojo wants to merge 1 commit into
mainfrom
fix/ci-skill-drift-and-floating-tags

Conversation

@castrojo
Copy link
Copy Markdown
Contributor

@castrojo castrojo commented Jun 4, 2026

Fixes #413 and #477.

Changes

skill-drift.yml

Adds the skill drift detection workflow to common. This workflow runs on PRs to detect when agent skills are out of date relative to code changes. Adapted from bluefin's version with code-paths updated for common's structure (system_files/, Containerfile) instead of build_files/.

.pre-commit-config.yaml

Adds a local pygrep hook that blocks floating GitHub Action tags (@main, @master, @latest, @v1, @v2, etc.) in workflow files at commit time. This complements Renovate's existing SHA-pinning management by preventing new floating tags from being introduced.

Assisted-by: Claude Sonnet 4.6 via GitHub Copilot CLI

…commit

- .github/workflows/skill-drift.yml: detect stale agent skills on PRs (closes #413)
- .pre-commit-config.yaml: add pygrep hook blocking @main/@v* floating tags (closes #477)

Assisted-by: Claude Sonnet 4.6 via GitHub Copilot CLI
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 4, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 4, 2026

Warning

Review limit reached

@castrojo, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 48 minutes and 49 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8fb5b51b-9e6a-4a4c-b25a-6e472a17b0b4

📥 Commits

Reviewing files that changed from the base of the PR and between 939437d and 3068a4b.

📒 Files selected for processing (2)
  • .github/workflows/skill-drift.yml
  • .pre-commit-config.yaml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ci-skill-drift-and-floating-tags

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@dosubot dosubot Bot added the kind/automation Automation and CI/CD label Jun 4, 2026
@castrojo castrojo enabled auto-merge June 4, 2026 17:38
@castrojo
Copy link
Copy Markdown
Contributor Author

castrojo commented Jun 4, 2026

Superseded by #480 after rebuilding the guardrails branch on current main. The repo already has the floating-action-tags hook, and the overlapping skill-drift/workflow work now has a single path forward there.

@castrojo castrojo closed this Jun 4, 2026
auto-merge was automatically disabled June 4, 2026 19:53

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/automation Automation and CI/CD size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(ci): skill-drift.yml missing from common and knuckle

1 participant