Skip to content
This repository was archived by the owner on Sep 8, 2026. It is now read-only.

fix(solana-cli): use bump-aware ATA create CU budget - #391

Open
vihu wants to merge 2 commits into
mainfrom
rg/ata-helper
Open

fix(solana-cli): use bump-aware ATA create CU budget#391
vihu wants to merge 2 commits into
mainfrom
rg/ata-helper

Conversation

@vihu

@vihu vihu commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes malbeclabs/doublezero#3768

Use the shared Wallet::ata_address_and_create_compute_units helper in the publisher rewards distribute path so ATA creation compute budgets account for the actual ATA bump seed.

Changes

  • Replace direct get_associated_token_address usage in publisher_rewards/distribute.rs with the shared Wallet helper.
  • Carry the helper-derived create-ATA CU estimate into distribute transaction construction.
  • Remove the fixed 25_000 create-ATA CU term from the publisher rewards distribute budget.
  • Preserve existing destination ATA prefetching and known-existing ATA behavior.
  • Add unit coverage for the distribute CU budget helper.

@vihu
vihu requested a review from karl-dz June 18, 2026 18:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

solana-client-tools: helper for ATA address plus create-instruction CU cost

1 participant