improve listing prompt, fix mint nft action

coinbase · Feb 5, 2025 · 6fab218 · 6fab218
1 parent 10e5a92
commit 6fab218
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 15 deletions.
diff --git a/typescript/agentkit/CHANGELOG.md b/typescript/agentkit/CHANGELOG.md
@@ -4,12 +4,14 @@
 
 ### Added
 
+- Added `openseaActionProvider` to list NFTs on OpenSea.
 - Added `alchemyTokenPricesActionProvider` to fetch token prices from Alchemy.
 - 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
 
+- Fixed mint function definition in ERC721_ABI.
 - Added account argument in call to estimateGas in CdpWalletProvider
 
 ## [0.1.1] - 2025-02-02

diff --git a/typescript/agentkit/src/action-providers/erc721/constants.ts b/typescript/agentkit/src/action-providers/erc721/constants.ts
@@ -1,9 +1,6 @@
 export const ERC721_ABI = [
   {
-    inputs: [
-      { internalType: "address", name: "to", type: "address" },
-      { internalType: "uint256", name: "tokenId", type: "uint256" },
-    ],
+    inputs: [{ internalType: "address", name: "to", type: "address" }],
     name: "mint",
     outputs: [],
     payable: false,

diff --git a/typescript/agentkit/src/action-providers/erc721/erc721ActionProvider.test.ts b/typescript/agentkit/src/action-providers/erc721/erc721ActionProvider.test.ts
@@ -40,7 +40,7 @@ describe("ERC721 Action Provider", () => {
         data: encodeFunctionData({
           abi: ERC721_ABI,
           functionName: "mint",
-          args: [MOCK_DESTINATION, 1n],
+          args: [MOCK_DESTINATION],
         }),
       });
 

diff --git a/typescript/agentkit/src/action-providers/erc721/erc721ActionProvider.ts b/typescript/agentkit/src/action-providers/erc721/erc721ActionProvider.ts
@@ -39,7 +39,7 @@ Do not use the contract address as the destination address. If you are unsure of
       const data = encodeFunctionData({
         abi: ERC721_ABI,
         functionName: "mint",
-        args: [args.destination as Hex, 1n],
+        args: [args.destination as Hex],
       });
 
       const hash = await walletProvider.sendTransaction({

diff --git a/typescript/agentkit/src/action-providers/opensea/openseaActionProvider.ts b/typescript/agentkit/src/action-providers/opensea/openseaActionProvider.ts
@@ -96,7 +96,7 @@ export class OpenseaActionProvider extends ActionProvider {
   @CreateAction({
     name: "list_nft",
     description: `
-This tool will list an NFT for sale on OpenSea. 
+This tool will list an NFT for sale on the OpenSea marketplace. 
 Currently only base-sepolia and base-mainnet are supported.
 
 It takes the following inputs:
@@ -106,13 +106,14 @@ It takes the following inputs:
 - expirationDays: (Optional) Number of days the listing should be active for (default: 90)
 
 Important notes:
-- The wallet must own the NFT to list it
-- Price is in ETH (e.g. 1.5 for 1.5 ETH) - this is what you will receive if the NFT is sold, it is not required to have this amount in your wallet
-- This is a gasless action - no ETH balance is required for listing the NFT
-- This will approve the whole NFT collection to be managed by OpenSea
-- Only supported on the following networks:
-  - Base Sepolia (ie, 'base-sepolia')
-  - Base Mainnet (ie, 'base', 'base-mainnet')
+- The wallet must own the NFT
+- Price is in ETH (e.g., 1.5 for 1.5 ETH). This is the amount the seller will receive if the NFT is sold. It is not required to have this amount in the wallet.
+- Listing the NFT requires approval for OpenSea to manage the entire NFT collection:  
+  - If the collection is not already approved, an onchain transaction is required, which will incur gas fees.  
+  - If already approved, listing is gasless and does not require any onchain transaction.  
+- Only the following networks are supported:
+  - Base Sepolia (base-sepolia)
+  - Base Mainnet (base, base-mainnet)
   `,
     schema: ListNftSchema,
   })

diff --git a/typescript/agentkit/src/wallet-providers/cdpWalletProvider.ts b/typescript/agentkit/src/wallet-providers/cdpWalletProvider.ts
@@ -274,7 +274,7 @@ export class CdpWalletProvider extends EvmWalletProvider {
     const feeData = await this.#publicClient!.estimateFeesPerGas();
 
     const gas = await this.#publicClient!.estimateGas({
-      account: this.#publicClient.account,
+      account: this.#address as `0x${string}`,
       to,
       value,
       data,