Repository navigation
fix(ci): cancel superseded skills resolution PR checks - #553
Conversation
Preserve independent main runs required for exact-commit activation evidence. Refs #534
Strix Security Review✅ No open security findings. PR summary: Adds a workflow-level `concurrency` block to the skills-resolution GitHub Actions workflow: superseded runs for the same pull request are cancelled, while main-branch (push) runs each keep their own distinct identity so the exact-commit activation evidence is preserved. Review summaryThe diff adds a GitHub Actions concurrency policy (5 lines) to the skills-resolution workflow so superseded pull-request runs are cancelled while main-branch runs stay independent. I traced the group expression and cancellation rule: PR groups key on Review 1 on this PR · Reviewed by Strix |
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 36 minutes. View limit details
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #553 +/- ##
=======================================
Coverage 61.05% 61.05%
=======================================
Files 1791 1791
Lines 105965 105965
=======================================
+ Hits 64692 64694 +2
+ Misses 32005 32003 -2
Partials 9268 9268 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
New commits to a skills-resolution PR could leave its older Go/Docker checks running because this workflow had no concurrency policy.
Replace superseded runs of the same PR. Keep main runs independent: the Skills catalog operations contract requires this workflow's exact reviewed commit, binary digest and dedicated container probe before activation. Any future manual event also gets an independent run identity.
Only five workflow lines change. Triggers, permissions, pinned tools, runner, timeout, test/build steps and the owned cleanup trap remain unchanged.
Validation: actionlint, strict TypeScript and 14 cases using the official GitHub expressions evaluator pass. Full payload AST and shell trap match the baseline. No extra local Go/native run or new dependencies were introduced for this YAML-only change; normal exact-head CI remains required before merge.
Refs #534.