Skip to content

Fix fee options for undeployed wallets#998

Merged
taylanpince merged 1 commit intomasterfrom
fix-undeployed-fee-options
May 6, 2026
Merged

Fix fee options for undeployed wallets#998
taylanpince merged 1 commit intomasterfrom
fix-undeployed-fee-options

Conversation

@corbanbrook
Copy link
Copy Markdown
Contributor

Root cause: RpcRelayer.feeOptions was ignoring the caller’s to and always sending to =
wallet with raw encoded calls, which matches the failing request.

Changes made:

  • Added Wallet.buildFeeOptionsTransaction() in wallet-core to build the correct fee-
    options target/data.
  • For undeployed wallets, it targets the guest module and prefixes the wallet deploy
    call before wallet.execute.
  • Updated WDK transactions.define() to pass that built target/data into relayer fee
    options.
  • Updated dapp-client fee options to pass the already built signed transaction data.
  • Updated relayer fee-options API to accept optional prebuilt calldata and stop
    overwriting to.
  • Added focused tests for core transaction construction and relayer request body
    behavior.

@corbanbrook corbanbrook requested a review from a team May 6, 2026 14:09
@corbanbrook corbanbrook requested a review from a team as a code owner May 6, 2026 14:09
Comment thread packages/services/relayer/src/relayer/rpc-relayer/index.ts
@taylanpince taylanpince merged commit 1c23094 into master May 6, 2026
6 checks passed
@taylanpince taylanpince deleted the fix-undeployed-fee-options branch May 6, 2026 16:44
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.

3 participants