Skip to content

feat: implement the forest state decode param API #5676

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.

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
merged 7 commits into from
Jun 17, 2025

Conversation

akaladarshi
Copy link
Collaborator

@akaladarshi akaladarshi commented May 29, 2025

Summary of changes

Changes introduced in this pull request:

  • This is a POC for the Filecoin.StateDecodeParams API
  • Adds lotus_json/actor_states/methods module which contains all the currently implemented method params for actor states
  • Right now it only supports
  • All Versions of - Miner actor params:
    • Constructor
    • ChangeWorkerAddress
  • V11, V12, V13, V15, V16 Account actor params:
    • Constructor
    • AuthenticateMessageExported
  • V15, V16 EVM actor params:
    • Constructor

Reference issue to close (if applicable)

Closes: #5724
Partially closes: #5722 #5721

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.

@akaladarshi akaladarshi changed the base branch from akaladarshi/support-all-actors-read-state-api to main June 3, 2025 09:48
@akaladarshi akaladarshi changed the base branch from main to akaladarshi/support-all-actors-read-state-api June 3, 2025 09:50
@akaladarshi akaladarshi force-pushed the akaladarshi/support-all-actors-read-state-api branch from 7a88763 to 9ea30b5 Compare June 11, 2025 14:58
Base automatically changed from akaladarshi/support-all-actors-read-state-api to main June 13, 2025 10:26
@akaladarshi akaladarshi force-pushed the akaladarshi/add-state-decode-params-api branch from 3c76ddc to 5fb24b8 Compare June 13, 2025 12:07
@akaladarshi akaladarshi added the RPC requires calibnet RPC checks to run on CI label Jun 13, 2025
@akaladarshi akaladarshi marked this pull request as ready for review June 13, 2025 15:38
@akaladarshi akaladarshi requested a review from a team as a code owner June 13, 2025 15:38
@akaladarshi akaladarshi requested review from hanabi1224 and elmattic and removed request for a team June 13, 2025 15:38
@akaladarshi akaladarshi force-pushed the akaladarshi/add-state-decode-params-api branch from 3b91dc5 to 2e39ad0 Compare June 13, 2025 15:49
hanabi1224
hanabi1224 previously approved these changes Jun 16, 2025
LesnyRumcajs
LesnyRumcajs previously approved these changes Jun 16, 2025
Copy link
Member

@LesnyRumcajs LesnyRumcajs left a comment

Choose a reason for hiding this comment

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

LGTM!

@akaladarshi akaladarshi dismissed stale reviews from LesnyRumcajs and hanabi1224 via c853797 June 16, 2025 14:05
@akaladarshi akaladarshi requested a review from LesnyRumcajs June 16, 2025 14:15
@akaladarshi akaladarshi requested a review from hanabi1224 June 16, 2025 15:17
@akaladarshi akaladarshi added this pull request to the merge queue Jun 17, 2025
Merged via the queue into main with commit 4307c7f Jun 17, 2025
34 checks passed
@akaladarshi akaladarshi deleted the akaladarshi/add-state-decode-params-api branch June 17, 2025 06:02
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.

Add support for the Miner actor state in Filecoin.StateDecodeParams API Add support for the Account actor state in Filecoin.StateDecodeParams API
3 participants