Skip to content

feat(solana): add gasValue flag to GMP script#1366

Open
Deiadara wants to merge 2 commits into
mainfrom
feat/solana-call-contract-gas
Open

feat(solana): add gasValue flag to GMP script#1366
Deiadara wants to merge 2 commits into
mainfrom
feat/solana-call-contract-gas

Conversation

@Deiadara

@Deiadara Deiadara commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

why

GMP Solana Scripts were missing auto gas payment

how


Note

Medium Risk
Adds an optional PayGas instruction and changes CallContract transaction construction to bundle multiple instructions in one transaction, which could affect how transactions are serialized/signed and how gas is attributed/refunded.

Overview
Adds --gas-value and optional --refund-address to the CallContract CLI so GMP calls can atomically pay cross-chain gas in the same Solana transaction.

Updates build_transaction/call_contract to emit a two-instruction bundle (PayGas via solana_axelar_gas_service + gateway CallContract) keyed by a keccak payload_hash, with refund defaulting to the fee payer.

Reviewed by Cursor Bugbot for commit 3634e78. Bugbot is set up for automated code reviews on this repo. Configure here.

@Deiadara Deiadara requested a review from a team as a code owner April 9, 2026 14:28

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d07dfdc. Configure here.

Comment thread solana/src/gateway.rs
@h4writer h4writer changed the title feat(solana): add gasValue flag to GMP sctipt feat(solana): add gasValue flag to GMP script May 5, 2026
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.

1 participant