chore: more delegate_enum to reduce repetitive code

[hanabi1224]

Summary of changes

Changes introduced in this pull request:

Reference issue to close (if applicable)

Closes

Other information and links

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.

Summary by CodeRabbit

• Refactor
  • Simplified internal type handling and conversions across core modules
  • Consolidated version-specific state management through unified delegation patterns

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

[coderabbitai]

Walkthrough

This PR removes unnecessary .into() type conversions and explicit type casting throughout the codebase where values are already compatible with expected types. Changes span interpreter modules, JSON serialization, RPC methods, state management utilities, and actor implementations, primarily replacing conversion calls with direct value usage and introducing delegation macros for enum variant conversions.

Changes

Cohort / File(s)	Summary
Interpreter worker value handling src/interpreter/fvm2.rs, src/interpreter/fvm3.rs, src/interpreter/fvm4.rs	Removed .into() conversion on worker value when passing to resolve_to_key_addr, using worker directly instead of converting
JSON serialization for miner info src/lotus_json/miner_info.rs	Removed .into() conversions for Address fields (owner, worker, beneficiary) and SectorSize in Lotus JSON conversions; assignments now use raw types directly
RPC F3 power table construction src/rpc/methods/f3.rs	Removed .into() coercion on miner_info.worker in five code branches (V8–V11); worker now passed directly to id_power_worker_mappings
State RPC methods src/rpc/methods/state.rs	Removed .into() conversions in address lookups, collateral calculations, and circulation supply handling; replaced with direct value usage throughout state query methods
Deal proposal API conversion src/rpc/types/deal_impl.rs	Updated From<DealProposal> to assign piece_size, client, provider, and collateral fields directly without .into() conversions
RPC types and imports src/rpc/types/mod.rs	Consolidated imports under crate::shim::actors; updated MiningBaseInfo.sector_size type from fvm_shared2::sector::SectorSize to imported SectorSize
Market actor state management src/shim/actors/builtin/market/mod.rs	Introduced delegated_enum(impl_conversions) on BalanceTable and DealProposals enums; replaced per-version From impls with centralized delegation macros for automatic conversions
Miner actor state expansion src/shim/actors/builtin/miner/mod.rs	Expanded State enum to include versions V9–V17 with delegated_enum(impl_conversions) annotation; replaced explicit pattern matching with delegate_state! macro usage for info and fee_debt methods
Power actor delegation refactoring src/shim/actors/builtin/power/mod.rs	Expanded State enum to versions V8–V17 with delegation support; changed ADDRESS constant type from fvm_shared2::address::Address to imported Address; replaced per-variant match logic with delegate_state! macro calls
Reward state conversion simplification src/shim/actors/builtin/reward/mod.rs	Replaced token/power conversion wrappers with direct .into() calls across all versions (V8–V17) for reward calculations and collateral bounds
Executor event types delegation src/shim/executor.rs	Converted ApplyRet, Receipt, Entry, ActorEvent, and StampedEvent from boxed variants to unboxed types; added delegated_enum(impl_conversions) to enable automatic conversions and replaced pattern matching with delegation macros
PaddedPieceSize conversions src/shim/piece.rs	Added From<PaddedPieceSize> for u64 and From<PaddedPieceSize> for BigInt conversion impls
Circulating supply calculations src/state_manager/circulating_supply.rs	Removed .into() conversions on total_locked(), into_total_storage_power_reward(), and into_total_locked() return values
State manager worker resolution src/state_manager/mod.rs	Removed .into() when passing info.worker() to resolve_to_key_addr; changed address table lookups to pass addresses directly
Sector size handling src/state_manager/utils.rs	Removed .into() conversion when passing info.sector_size() to RegisteredSealProof::from_sector_size
Migration test updates src/state_migration/nv17/miner.rs	Updated test calls to use address constants directly instead of converting with .into() for get_actor and set_actor calls

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

• refactor(actor_state): allow easy version upgrades #6087 — Overlaps with Lotus JSON deserialization refactoring and actor state construction changes using default_latest_version helpers
• fix: reduce repetitive code in enum match blocks #6453 — Related to address conversion usage removal around resolve_to_key_addr call sites

Suggested reviewers

• LesnyRumcajs
• akaladarshi

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 24.49% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'chore: more delegate_enum to reduce repetitive code' accurately describes the main objective of the PR, which involves expanding the use of the delegated_enum macro across multiple files (executor, miner, market, power, and reward modules) to reduce boilerplate code.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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

[codecov]

Codecov Report

❌ Patch coverage is 29.86111% with 303 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.76%. Comparing base (237583f) to head (7c759b4).
⚠️ Report is 5 commits behind head on main.

Files with missing lines	Patch %	Lines
src/shim/actors/builtin/miner/mod.rs	19.20%	97 Missing and 4 partials ⚠️
src/shim/actors/builtin/reward/mod.rs	17.27%	91 Missing ⚠️
src/shim/actors/builtin/market/mod.rs	11.84%	63 Missing and 4 partials ⚠️
src/shim/actors/builtin/power/mod.rs	26.92%	17 Missing and 2 partials ⚠️
src/shim/executor.rs	65.00%	4 Missing and 3 partials ⚠️
src/rpc/methods/f3.rs	0.00%	4 Missing ⚠️
src/rpc/methods/state.rs	84.00%	2 Missing and 2 partials ⚠️
src/shim/piece.rs	50.00%	3 Missing ⚠️
src/state_manager/mod.rs	25.00%	0 Missing and 3 partials ⚠️
src/interpreter/fvm2.rs	0.00%	1 Missing ⚠️
... and 3 more

Additional details and impacted files

Files with missing lines	Coverage Δ
src/lotus_json/miner_info.rs	97.50% <100.00%> (+4.31%)	⬆️
src/rpc/types/deal_impl.rs	100.00% <100.00%> (ø)
src/rpc/types/mod.rs	88.46% <ø> (ø)
src/state_manager/utils.rs	79.63% <100.00%> (ø)
src/state_migration/nv17/miner.rs	81.61% <100.00%> (-0.09%)	⬇️
src/interpreter/fvm2.rs	44.68% <0.00%> (ø)
src/interpreter/fvm3.rs	40.60% <0.00%> (ø)
src/interpreter/fvm4.rs	42.18% <0.00%> (ø)
src/state_manager/circulating_supply.rs	79.05% <75.00%> (-0.09%)	⬇️
src/shim/piece.rs	58.82% <50.00%> (-1.18%)	⬇️
... and 8 more

... and 16 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 237583f...7c759b4. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[LesnyRumcajs]

nice