Skip to content

fix(core): rpc simulateTransaction loadedAccountsDataSize is None (#475)#476

Merged
MicaiahReid merged 3 commits intosolana-foundation:mainfrom
pymongo:main
Jan 20, 2026
Merged

fix(core): rpc simulateTransaction loadedAccountsDataSize is None (#475)#476
MicaiahReid merged 3 commits intosolana-foundation:mainfrom
pymongo:main

Conversation

@pymongo
Copy link
Copy Markdown
Contributor

@pymongo pymongo commented Jan 14, 2026

Copy link
Copy Markdown
Collaborator

@MicaiahReid MicaiahReid left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, @pymongo! I have a few changes needed to get this over the finish line.

@MicaiahReid
Copy link
Copy Markdown
Collaborator

@pymongo do you think you'll be able to get these fixes in?

@pymongo
Copy link
Copy Markdown
Contributor Author

pymongo commented Jan 19, 2026

I am researching, fogo svm don't count program size as data size, but solana count program size as data size, how surfpool to compatiable both fogo and solana

@MicaiahReid
Copy link
Copy Markdown
Collaborator

I am researching, fogo svm don't count program size as data size, but solana count program size as data size, how surfpool to compatiable both fogo and solana

Nice good find. According to this SIMD, the program data account is included. However, the mint account is not. Are you able to get that change in, or do you need me to take over? We'd like to get this into our upcoming 1.0 release!

@pymongo
Copy link
Copy Markdown
Contributor Author

pymongo commented Jan 20, 2026

Sorry but I am busy rencently, no time to make the additional changes

You can resign/assigned to this issue/problem, thanks

@MicaiahReid MicaiahReid requested a review from lgalabru January 20, 2026 07:33
@MicaiahReid MicaiahReid merged commit 588583e into solana-foundation:main Jan 20, 2026
4 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.

simulateTransaction rpc response field loadedAccountsDataSize is null

2 participants