Skip to content

Conversation

@sporicle
Copy link
Contributor

@sporicle sporicle commented Dec 1, 2025

Summary by CodeRabbit

  • Chores
    • Updated Anchor framework dependency from 0.31.x to 0.32.1 across the project.
    • Updated ephemeral-vrf-sdk dependencies to versions 0.2.0 and 0.2.1.
    • Added new internal feature flags for enhanced stability and debugging capabilities.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 1, 2025

Walkthrough

Updates Anchor framework dependencies from 0.31.x to 0.32.1 across project configurations and both program Cargo.toml files. Adds three new feature flags (anchor-debug, custom-heap, custom-panic) to both Anchor programs. Updates ephemeral-vrf-sdk to different versions per program (0.2.1 and 0.2.0).

Changes

Cohort / File(s) Summary
Documentation and JavaScript Configuration
roll-dice/README.md, roll-dice/package.json
Updated Anchor version references from 0.31.0 to 0.32.1 in documentation and build steps. Updated @coral-xyz/anchor dependency from ^0.31.0 to fixed 0.32.1 version in package.json; reordered devDependencies entries with no semantic changes.
Rust Program Dependencies
roll-dice/programs/roll-dice-delegated/Cargo.toml, roll-dice/programs/roll-dice/Cargo.toml
Bumped anchor-lang from 0.31.1 to 0.32.1 in both programs. Updated ephemeral-vrf-sdk from 0.1.2 to 0.2.1 (delegated) and 0.2.0 (main program). Added three new feature flags to both: anchor-debug, custom-heap, custom-panic.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • Verify that Anchor 0.32.1 is compatible with existing codebase logic
  • Confirm ephemeral-vrf-sdk version discrepancy between programs (0.2.1 vs 0.2.0) is intentional
  • Check that new feature flags (anchor-debug, custom-heap, custom-panic) are appropriate for both programs and don't conflict with existing build configurations

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating the roll-dice project's dependencies to be compatible with Anchor 0.32.1, which is reflected across all modified files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch anchor32

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 93481c6 and 1c4b8e7.

⛔ Files ignored due to path filters (2)
  • roll-dice/Cargo.lock is excluded by !**/*.lock
  • roll-dice/yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (4)
  • roll-dice/README.md (2 hunks)
  • roll-dice/package.json (1 hunks)
  • roll-dice/programs/roll-dice-delegated/Cargo.toml (1 hunks)
  • roll-dice/programs/roll-dice/Cargo.toml (1 hunks)
🔇 Additional comments (8)
roll-dice/README.md (1)

19-19: LGTM

Documentation version references are properly updated to reflect the Anchor 0.32.1 upgrade.

Also applies to: 33-33

roll-dice/package.json (1)

8-8: Verify the intent of pinning @coral-xyz/anchor to an exact version.

0.32.1 is a patch to fix a couple issues that showed up in 0.32.0, most notably a race condition when deploying programs. Removing the caret changes the dependency from a flexible semver range (^0.31.0) to an exact version (0.32.1), which prevents automatic installation of patch releases like 0.32.2 or 0.32.3 that may contain security fixes or critical bug fixes.

Consider keeping the caret (^0.32.1) to allow patch releases unless pinning is required for reproducibility.

roll-dice/programs/roll-dice-delegated/Cargo.toml (3)

18-20: Clarify the purpose and usage of new feature flags.

Three feature flags (anchor-debug, custom-heap, custom-panic) have been added but are not activated anywhere in the [features] section. Verify that these are required for Anchor 0.32.1 compatibility and consider documenting their purpose or enabling them if needed. If they're optional, consider whether they should be added to the default feature set.


24-24: LGTM

anchor-lang dependency is properly updated to 0.32.1 with the existing feature flag retained.


26-26: Verify the intentional difference in ephemeral-vrf-sdk versions.

The delegated program uses ephemeral-vrf-sdk 0.2.1, while the main roll-dice program uses 0.2.0. Confirm this version difference is intentional and document the reason if it is (e.g., due to different feature requirements or SDK changes between the programs).

roll-dice/programs/roll-dice/Cargo.toml (3)

18-20: Clarify the purpose and usage of new feature flags.

Three feature flags (anchor-debug, custom-heap, custom-panic) have been added but are not activated anywhere in the [features] section. Verify that these are required for Anchor 0.32.1 compatibility and consider documenting their purpose or enabling them if needed. If they're optional, consider whether they should be added to the default feature set.


23-23: LGTM

anchor-lang dependency is properly updated to 0.32.1 with the existing feature flag retained.


25-25: Verify the intentional difference in ephemeral-vrf-sdk versions.

The main program uses ephemeral-vrf-sdk 0.2.0, while the delegated program uses 0.2.1. Confirm this version difference is intentional and document the reason if it is (e.g., due to different feature requirements or SDK changes between the programs).


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sporicle sporicle merged commit 5fe82fa into main Dec 1, 2025
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants