fix: include account in call to estimateGas (coinbase#253)

emiridbest · Feb 4, 2025 · 0dc1498 · 0dc1498
1 parent 8fe99da
commit 0dc1498
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/typescript/agentkit/CHANGELOG.md b/typescript/agentkit/CHANGELOG.md
@@ -8,6 +8,10 @@
 - Added `token_prices_by_symbol` action to fetch token prices by symbol.
 - Added `token_prices_by_address` action to fetch token prices by network and address pairs.
 
+### Fixed
+
+- Added account argument in call to estimateGas in CdpWalletProvider
+
 ## [0.1.1] - 2025-02-02
 
 ### Added

diff --git a/typescript/agentkit/src/wallet-providers/cdpWalletProvider.ts b/typescript/agentkit/src/wallet-providers/cdpWalletProvider.ts
@@ -274,6 +274,7 @@ export class CdpWalletProvider extends EvmWalletProvider {
     const feeData = await this.#publicClient!.estimateFeesPerGas();
 
     const gas = await this.#publicClient!.estimateGas({
+      account: this.#publicClient.account,
       to,
       value,
       data,
-Original file line number
+Diff line change
@@ Expand Up / @@ -274,6 +274,7 @@ export class CdpWalletProvider extends EvmWalletProvider { @@
         const feeData = await this.#publicClient!.estimateFeesPerGas();
         const gas = await this.#publicClient!.estimateGas({
+          account: this.#publicClient.account,
           to,
           value,
           data,
@@ Expand Down @@