Repository navigation
docs(hooks): clarify cross-file execution order - #12295
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe hook documentation now states that all matching hooks execute across loaded configuration files. It defines descending precedence, listed array order, and alphabetical ChangesHook execution order
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This documentation-only change clarifies hook ordering without changing product behavior or runtime code; no actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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 |
Greptile SummaryClarifies how hooks are combined and ordered across multiple configuration files.
Confidence Score: 5/5The documentation-only change appears safe to merge. The documented additive behavior, highest-to-lowest config precedence, reverse-alphabetical fragment example, and listed array ordering agree with the hook-loading behavior. Important Files Changed
Reviews (1): Last reviewed commit: "docs(hooks): clarify cross-file executio..." | Re-trigger Greptile |
Summary
conf.dexample and recommend arrays for order-dependent hooksContext: #12276
Validation
git diff --checkhk run --check --safe --format jsonscoped todocs/hooks.mdAI-assisted — Tool: Codex; model: OpenAI/GPT-5; version: unavailable.
Note
Low Risk
Documentation-only clarification of hook execution order; no code or behavior changes.
Overview
Documents that matching hooks from multiple config files are additive, not overridden, and run from highest- to lowest-precedence files (arrays keep listed order within a file).
Adds a
conf.dexample (c, thenb, thena) and recommends putting order-dependent hooks in one array.Reviewed by Cursor Bugbot for commit d3c2821. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by CodeRabbit