fix(ci): bump CSHARP_CID after kit drift#36
Conversation
CSHARP_CID was pinned at the v1.2.0-era value. Recent kit additions (binaryCid + targetProofCid + sourceContractCid + EvidenceTerm landed across kits in PRs #14/#17/#18/#21) shifted what the C# self-contracts mint produces. CI captured the new value: old: 45d7cdbd...415c new: cec85197...c61e Quick pin bump to unblock CI. Structural follow-up tracked: extend the letter-envelope refactor (PR #30) to TS and C# so this constant disappears entirely. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ 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. Review rate limit: 0/1 reviews remaining, refill in 33 minutes and 25 seconds.Comment |
There was a problem hiding this comment.
Pull request overview
Updates the pinned C# self-contracts bundle CID in the top-level Makefile to match the current output after recent protocol-additive kit changes (e.g., binaryCid, sourceContractCid, targetProofCid, EvidenceTerm), unblocking the CI conformance gate.
Changes:
- Bump
CSHARP_CIDMakefile pin from the v1.2.0-era value to the newly minted CID.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
`CSHARP_CID` was pinned at v1.2.0-era value. Recent kit work (binaryCid + targetProofCid + sourceContractCid + EvidenceTerm in PRs #14/#17/#18/#21) shifted what mint produces.
Quick pin bump to unblock CI. Structural follow-up: extend letter-envelope refactor (PR #30) to TS and C# so this constant disappears.
Test plan