Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📜 Recent review details⏰ Context from checks skipped due to timeout. (3)
🧰 Additional context used📓 Path-based instructions (1)*⚙️ CodeRabbit configuration file
Files:
🧠 Learnings (1)📚 Learning: 2026-06-23T00:31:58.686ZApplied to files:
🔇 Additional comments (1)
WalkthroughUpdates the ChangesVendored dependency hash refresh
Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
How to use the Graphite Merge QueueAdd the label add-to-gt-merge-queue to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has required the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
e51584f to
2588b51
Compare
Merge activity
|

Motivation
Cargo.lockpinsevent-sorcerytotag=0.1.2(bumped in #178), but the flake'soutputHashesstill pinned the oldtag=0.1.1rev. That left the0.1.1entry dead and the live0.1.2git dep without a hash, so crane'svendorCargoDepsfell back tobuiltins.fetchGit(IFD) — the exact cross-repo footgunAGENTS.mdwarns about.nix flake checkemittedNo output hash provided for git+…event-sorcery.git?tag=0.1.2#8f5c…and only succeeded because0.1.2happens to be a reachable tag; the fetch is non-hermetic and fragile.Implements RAI-1126.
Solution
Point the
outputHasheskey at the0.1.2tag/rev now in the lockfile and pin itssha256(read from the hash-mismatch error).nix flake checknow builds the vendored dep hermetically with no missing-hash warning.Checks
By submitting this for review, I'm confirming I've done the following:
nix flake check, which now passes without the missing-hash warning)Summary by CodeRabbit