Skip to content

chore: bump FVM#6931

Merged
LesnyRumcajs merged 3 commits intomainfrom
bump-fvm
Apr 20, 2026
Merged

chore: bump FVM#6931
LesnyRumcajs merged 3 commits intomainfrom
bump-fvm

Conversation

@LesnyRumcajs
Copy link
Copy Markdown
Member

@LesnyRumcajs LesnyRumcajs commented Apr 16, 2026

Summary of changes

Changes introduced in this pull request:

  • bumps FVM and bunch of other dependent crates
  • final cargo update

Reference issue to close (if applicable)

Closes #6918

Other information and links

Not strictly required to go ahead with NV28, so not super hot.

Change checklist

  • I have performed a self-review of my own code,
  • I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
  • I have added tests that prove my fix is effective or that my feature works (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Outside contributions

  • I have read and agree to the CONTRIBUTING document.
  • I have read and agree to the AI Policy document. I understand that failure to comply with the guidelines will lead to rejection of the pull request.

Summary by CodeRabbit

  • Chores
    • Updated Filecoin actor state crates to version 26.0.0.
    • Updated FVM and related crates (fvm2, fvm3, fvm4) to their latest minor versions.
    • Updated multihash-codetable dependency to version 0.2.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 16, 2026

Walkthrough

This PR updates multiple Filecoin and FVM dependency versions in Cargo.toml: multihash-codetable from 0.1 to 0.2, Filecoin actor state crates from 25.0.0 to 26.0.0, FVM crates across all versions (fvm2, fvm3, fvm4) with minor version increments, FVM shared crates to matching versions, and fvm_actor_utils from 14 to 15. Removes nv28-dev feature from fvm4.

Changes

Cohort / File(s) Summary
Dependency Versions
Cargo.toml
Updated multihash-codetable from 0.1 to 0.2; bumped Filecoin actor state crates (fil_actor_*_state, fil_actors_shared) from 25.0.0 to 26.0.0; upgraded FVM crates (fvm2, fvm3, fvm4) with minor version increments; updated FVM shared crates (fvm_shared2/3/4) to matching versions; bumped fvm_actor_utils to 15; removed nv28-dev feature from fvm4 while retaining verify-signature feature.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

Release

Suggested reviewers

  • hanabi1224
  • elmattic
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'chore: bump FVM' is vague and generic, using only a minimal description that lacks specificity about what FVM-related changes are being made. Consider a more descriptive title such as 'chore: bump FVM and actor crates to latest versions' to provide clearer context about the scope of dependency updates.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The pull request successfully upgrades ref-fvm and multiple dependent crates (FVM2/3/4, actor state crates, shared crates), achieving the primary objective from issue #6918.
Out of Scope Changes check ✅ Passed All changes are directly related to upgrading FVM and dependent crates as specified in issue #6918; no unrelated modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bump-fvm
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch bump-fvm

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

@LesnyRumcajs LesnyRumcajs force-pushed the bump-fvm branch 4 times, most recently from 29cec3d to 6ae32b6 Compare April 20, 2026 10:14
@LesnyRumcajs LesnyRumcajs marked this pull request as ready for review April 20, 2026 10:25
@LesnyRumcajs LesnyRumcajs requested a review from a team as a code owner April 20, 2026 10:25
@LesnyRumcajs LesnyRumcajs requested review from akaladarshi and hanabi1224 and removed request for a team April 20, 2026 10:25
@hanabi1224 hanabi1224 added the RPC requires calibnet RPC checks to run on CI label Apr 20, 2026
@LesnyRumcajs LesnyRumcajs enabled auto-merge April 20, 2026 10:25
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@Cargo.toml`:
- Around line 79-95: The NV28 migration was added but not wired into execution:
add a Height::NV28 variant to the Height enum and a NetworkVersion::V28 variant
to NetworkVersion, then register the nv28 migration module by inserting its
migration entry into each network's get_migrations() vector (at least for the
network(s) you intend to activate) so run_migration in src/state_migration::nv28
is actually invoked (remove the #[allow(unused)] once wired); also ensure an
actor bundle for the next actor version is published/registered so networks can
reach the new bundle version, or alternatively revert the fil_actor_* crate
version bumps back to the previous bundle until NV28 is fully integrated.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 52b66978-0e71-42d1-8da8-7068e32aed5a

📥 Commits

Reviewing files that changed from the base of the PR and between 919b53e and e7d4d38.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • Cargo.toml

Comment thread Cargo.toml
@LesnyRumcajs LesnyRumcajs added this pull request to the merge queue Apr 20, 2026
Merged via the queue into main with commit 4d64391 Apr 20, 2026
40 of 64 checks passed
@LesnyRumcajs LesnyRumcajs deleted the bump-fvm branch April 20, 2026 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RPC requires calibnet RPC checks to run on CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[NV28] ref-fvm upgrade

2 participants