FeeOptions do not sign calls

Author: ScreamingHawk

packages/wallet/dapp-client/src/ChainSessionManager.ts

@@ -714,11 +714,12 @@ export class ChainSessionManager {
 
   /**
    * Fetches fee options for a set of transactions.
+   * @param wallet The wallet address to use for the fee options.
    * @param calls The transactions to estimate fees for.
    * @returns A promise that resolves with an array of fee options.
    * @throws {FeeOptionError} If fetching fee options fails.
    */
-  async getFeeOptions(calls: Transaction[]): Promise<Relayer.FeeOption[]> {
+  async getFeeOptions(wallet: Address.Address, calls: Transaction[]): Promise<Relayer.FeeOption[]> {
     const callsToSend = calls.map((tx) => ({
       to: tx.to,
       value: tx.value,
@@ -729,8 +730,7 @@ export class ChainSessionManager {
       behaviorOnError: tx.behaviorOnError ?? ('revert' as const),
     }))
     try {
-      const signedCall = await this._buildAndSignCalls(callsToSend)
-      const feeOptions = await this.relayer.feeOptions(signedCall.to, this.chainId, callsToSend)
+      const feeOptions = await this.relayer.feeOptions(wallet, this.chainId, callsToSend)
       return feeOptions.options
     } catch (err) {
       throw new FeeOptionError(`Failed to get fee options: ${err instanceof Error ? err.message : String(err)}`)

packages/wallet/dapp-client/src/DappClient.ts

@@ -536,11 +536,11 @@ export class DappClient {
    * }
    */
   async getFeeOptions(chainId: number, transactions: Transaction[]): Promise<Relayer.FeeOption[]> {
-    if (!this.isInitialized) throw new InitializationError('Not initialized')
+    if (!this.isInitialized || !this.walletAddress) throw new InitializationError('Not initialized')
     const chainSessionManager = this.getChainSessionManager(chainId)
     if (!chainSessionManager.isInitialized)
       throw new InitializationError(`ChainSessionManager for chain ${chainId} is not initialized.`)
-    return await chainSessionManager.getFeeOptions(transactions)
+    return await chainSessionManager.getFeeOptions(this.walletAddress, transactions)
   }
 
   /**