Skip to content

Conversation

@snormore
Copy link
Contributor

Summary of Changes

  • Reject empty GetProgramAccounts responses instead of silently returning empty data

Testing Verification

  • Add test coverage for the change

Copy link
Contributor

@juan-malbeclabs juan-malbeclabs left a comment

Choose a reason for hiding this comment

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

LGTM

Add validation to return an error when GetProgramAccounts returns an
empty result. This prevents silent failures where an RPC issue could
return empty data, which would then be written as an empty snapshot
and tombstone all existing entities.
@snormore snormore force-pushed the snor/sdk-serviceability-empty-check branch from a6752b2 to 73f9749 Compare January 22, 2026 16:26
@snormore snormore marked this pull request as ready for review January 22, 2026 16:27
@snormore snormore merged commit 772faba into main Jan 23, 2026
21 checks passed
@snormore snormore deleted the snor/sdk-serviceability-empty-check branch January 23, 2026 19:33
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.

2 participants