Migrate rain-interpreter-interface 0.1.0 -> rainlang-interface 0.2.3 - #71
Conversation
The interface repo/package was renamed (rain.interpreter.interface -> rainlang.interface); legacy rain-interpreter-interface is frozen and all new publishes go to rainlang-interface. - foundry.toml: rain-interpreter-interface 0.1.0 -> rainlang-interface 0.2.3; declare the flat transitive set rainlang-interface's src imports (@OpenZeppelin-Contracts 5.6.1, rain-lib-hash 0.1.0, rain-math-binary 0.1.4, rain-sol-codegen 0.1.0, rain-solmem 0.1.3). - Rewrite all src import paths to rainlang-interface-0.2.3/src/... - Regenerate soldeer.lock via forge soldeer update; drop the stale rain-interpreter-interface-0.1.0 remapping. - Delete stale foundry.lock pinning the pre-soldeer git submodule lib/rain.interpreter.interface (silences the forge build warning). - Update CLAUDE.md dependencies section from submodule-era description to soldeer. No interface changes absorbed: the consumed files are byte-identical between the two packages. Refs rainlanguage/rainlang.interface#107 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
rainlang-interface-0.2.3 is shorter than rain-interpreter-interface-0.1.0, so three previously wrapped single-symbol import blocks now fit on one line. Refs rainlanguage/rainlang.interface#107 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 58 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (9)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe PR migrates dependency configuration from ChangesRainlang dependency migration
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This change updates the dependency name and remappings without changing the repository’s interface behavior; no actionable merge-blocking risk remains after normal checks and review. Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 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 |
The static gate fails at 4162 bytes against the 4096 cap, 66 over. The multi-line forge-lint suppression bullet is standard tool syntax, not repo knowledge; cutting it (and the denofmt reflow) lands under the cap. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
@coderabbitai assess this PR size classification for the totality of the PR with the following criterias and report it in your comment: S/M/L PR Classification Guidelines:This guide helps classify merged pull requests by effort and complexity rather than just line count. The goal is to assess the difficulty and scope of changes after they have been completed. Small (S)Characteristics:
Review Effort: Would have taken 5-10 minutes Examples:
Medium (M)Characteristics:
Review Effort: Would have taken 15-30 minutes Examples:
Large (L)Characteristics:
Review Effort: Would have taken 45+ minutes Examples:
Additional Factors to ConsiderWhen deciding between sizes, also consider:
Notes:
|
|
Tip For best results, initiate chat on the files or code changes. SIZE=M You are interacting with an AI system. |
Refs rainlanguage/rainlang.interface#107
Migrates the soldeer dependency
rain-interpreter-interface0.1.0 ->rainlang-interface0.2.3, following the interface repo/package rename (legacyrain-interpreter-interfaceis frozen; all new publishes go torainlang-interface).Changes
foundry.toml:rain-interpreter-interface = "0.1.0"->rainlang-interface = "0.2.3".src/files rewritten fromrain-interpreter-interface-0.1.0/src/...torainlang-interface-0.2.3/src/.... Every imported file and symbol exists unchanged in 0.2.3.[dependencies]now also declares whatrainlang-interface-0.2.3/srcitself imports:@openzeppelin-contracts5.6.1,rain-lib-hash0.1.0,rain-math-binary0.1.4,rain-sol-codegen0.1.0,rain-solmem0.1.3 (forge-std1.16.1 already declared).soldeer.lockregenerated viaforge soldeer update; stalerain-interpreter-interface-0.1.0/remapping removed fromremappings.txt.foundry.lock, which still pinned the pre-soldeerlib/rain.interpreter.interfacegit submodule and made everyforge buildwarnDependency 'lib/rain.interpreter.interface' not found at expected path(sibling soldeer repos have nofoundry.lock).CLAUDE.mddependencies section updated from the stale git-submodule description to the soldeer reality.forge fmtcollapsed three single-symbol import blocks that fit within the line width now that the package name is shorter.Interface changes absorbed
None. Diffing
rain-interpreter-interface0.1.0 againstrainlang-interface0.2.3 package src: identical exceptsrc/lib/parse/LibParseMeta.solandsrc/lib/codegen/LibGenParseMeta.sol, which only bump theirrain-math-binaryimport path 0.1.1 -> 0.1.4. Neither file is imported by this repo. All symbols this repo consumes (IInterpreterCallerV4,EvaluableV4,SignedContextV1,IInterpreterV4,IInterpreterStoreV3,LibEvaluable, and the deprecated v1/v2 caller/deployer interfaces) are byte-identical.QA
forge testcompiles and reports no tests, and this change is a dependency rename with zero behavioral surface.forge buildgreen against the renamed package, plus a byte-level diff of the old (0.1.0) vs new (0.2.3) package src proving the consumed interfaces are identical.🤖 Generated with Claude Code
Summary by CodeRabbit
Maintenance
Documentation
Compatibility