Skip to content

feat: add support for verified reg actor in state decode params #5896

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
akaladarshi merged 17 commits into from
Aug 19, 2025
Merged

feat: add support for verified reg actor in state decode params #5896

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
221faf6
add fvm_actor_utils
akaladarshi Aug 4, 2025
31b6839
add support for conversion of forest signature from and to fvm_shared…
akaladarshi Aug 5, 2025
a5de37c
feat: add support for verified reg actor in state decode params API a…
akaladarshi Aug 5, 2025
ece9a2c
add test for verified reg state decode params api
akaladarshi Aug 5, 2025
4be146d
remove unused dependecy
akaladarshi Aug 5, 2025
233fe46
address comments
akaladarshi Aug 6, 2025
8e2a6f0
add more test files
akaladarshi Aug 6, 2025
bd4dba4
replace failing rpc snap file
akaladarshi Aug 6, 2025
53be23e
refactor crypto signature conversion
akaladarshi Aug 6, 2025
b40cd3f
fix formatting issue
akaladarshi Aug 14, 2025
a7c7a54
Merge remote-tracking branch 'origin/main' into akaladarshi/add-verif…
akaladarshi Aug 14, 2025
311ae33
refactor the signature implementation and fix linter issues
akaladarshi Aug 18, 2025
836220a
Merge remote-tracking branch 'origin/main' into akaladarshi/add-verif…
akaladarshi Aug 18, 2025
41a3e90
Merge branch 'main' into akaladarshi/add-verifiedreg-actor-decode-params
akaladarshi Aug 18, 2025
203523f
Merge branch 'main' into akaladarshi/add-verifiedreg-actor-decode-params
akaladarshi Aug 18, 2025
9c0deab
Merge branch 'main' into akaladarshi/add-verifiedreg-actor-decode-params
akaladarshi Aug 19, 2025
be91d34
Merge branch 'main' into akaladarshi/add-verifiedreg-actor-decode-params
akaladarshi Aug 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -224,6 +224,7 @@ zstd = "0.13"

# optional dependencies
console-subscriber = { version = "0.4", features = ["parking_lot"], optional = true }
fvm_actor_utils = "14.0.0"
Copy link
Copy Markdown
Member

@LesnyRumcajs LesnyRumcajs Aug 19, 2025

Choose a reason for hiding this comment

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

Can you please summarise why it is needed?

Copy link
Copy Markdown
Collaborator Author

@akaladarshi akaladarshi Aug 19, 2025

Choose a reason for hiding this comment

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

There is one method UniversalReceiverHook that uses fvm_actor_utils::receiver::UniversalReceiverParams type as param

forest/src/rpc/registry/actors/verified_reg.rs

Line 195 in 203523f

Method::UniversalReceiverHook,

so we have to implement the HasLotusJson trait for it :

forest/src/lotus_json/actor_states/methods/verified_reg_actor.rs

Line 502 in 203523f

impl HasLotusJson for fvm_actor_utils::receiver::UniversalReceiverParams {

Copy link
Copy Markdown
Member

@LesnyRumcajs LesnyRumcajs Aug 19, 2025

Choose a reason for hiding this comment

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

It probably should be a re-export in fil-actor-states, then. Not a blocker, obviously.

paste = "1"
tikv-jemallocator = { version = "0.6", optional = true }
tracing-chrome = { version = "0.7", optional = true }
Expand Down
2 changes: 2 additions & 0 deletions src/lotus_json/actor_states/methods/mod.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
// Copyright 2019-2025 ChainSafe Systems
// SPDX-License-Identifier: Apache-2.0, MIT

use super::*;
mod account_authenticate_params;
mod account_constructor_params;
Expand All @@ -14,3 +15,4 @@ mod miner_constructor_params;
mod multisig_actor;
mod power_actor;
mod reward_methods;
pub mod verified_reg_actor;
Loading
Loading