node: use loaded addresses for Solana ALT resolution #4649 - #4950
node: use loaded addresses for Solana ALT resolution #4649#4950ayushmishra2005 wants to merge 1 commit into
Conversation
|
Thanks @djb15 for pointing me to the contribution policy. I understand and won’t open further PRs without coordinating first. I’m actively interested in contributing to Wormhole, particularly around Solana protocol/watcher work. If there’s an appropriate issue or area where an external contributor would be useful, I’d appreciate any guidance. |
|
Thanks for understanding. Glad to hear you're interested in contributing. Generally we aim to avoid changing Watcher related components where possible since they're high risk areas of the codebase. One area we can improve in that's low risk is test coverage. If you're interested in Solana specifically you could consider adding test coverage there. Specifically I would suggest looking at basing any changes off of #4924 which makes some small refactors to the Solana watcher to aid in writing effective tests. |
|
Thanks, that’s helpful. |
Fixes #4649.
Summary
For versioned Solana transactions, use
TransactionMeta.LoadedAddressesto populate lookup-table account keys when the metadata contains the complete set of loaded addresses.This avoids fetching historical ALT accounts when the resolved addresses are already available in the transaction metadata, so transactions can still be processed after an ALT has been closed.
If
LoadedAddressesis missing or incomplete, the watcher falls back to the existing ALT RPC lookup path.Changes