Release v0.17.1

Origin-SHA: 79f4a52d1d8639104c612776a2b17f839fb9a992

Author: CodySearsOS

CHANGELOG.md

@@ -1,5 +1,11 @@
 # @opensea/tool-sdk
 
+## 0.17.1
+
+### Patch Changes
+
+- Thread `--rpc-url` through to registry reads in CLI commands. Previously the flag only applied to writes; read paths (status, get, list) fell back to the default RPC endpoint.
+
 ## 0.17.0
 
 ### Minor Changes

examples/nft-appraisal-tool/public/chonk.png

@@ -95,6 +95,7 @@ export function buildHolderManifest(opts: BuildManifestOptions) {
     name: "nft-appraiser-chonks",
     description: `${APPRAISAL_DESCRIPTION} Discounted tier for CHONK holders on Base.`,
     endpoint: opts.endpoint,
+    image: "https://nft-appraisal-tool.vercel.app/chonk.png",
     inputs: APPRAISAL_INPUTS,
     outputs: appraisalJsonSchema,
     creatorAddress: opts.creator,

examples/nft-appraisal-tool/src/manifest.ts

@@ -24,6 +24,7 @@ export function buildManifest(opts: BuildManifestOptions) {
       "Finds wallets that hold both a specified ERC-20 token and NFTs from a collection. " +
       "Returns the overlap set with holdings data for each wallet. Free for holders.",
     endpoint: opts.endpoint,
+    image: "https://token-nft-overlap-tool.vercel.app/token-nft-overlap.png",
     inputs: {
       type: "object",
       properties: {

examples/token-nft-overlap-tool/public/token-nft-overlap.png

@@ -1,6 +1,6 @@
 {
   "name": "@opensea/tool-sdk",
-  "version": "0.17.0",
+  "version": "0.17.1",
   "type": "module",
   "description": "SDK and CLI for building ERC-8257 compliant AI agent tools",
   "repository": {

examples/token-nft-overlap-tool/src/manifest.ts

@@ -23,8 +23,13 @@ const mockGetToolConfig = vi.fn(async () => ({
 
 const mockTryHasAccess = vi.fn(async () => ({ ok: true, granted: true }))
 
+const registryCtorConfigs: Record<string, unknown>[] = []
+
 vi.mock("../lib/onchain/registry.js", () => ({
   ToolRegistryClient: class {
+    constructor(config: Record<string, unknown>) {
+      registryCtorConfigs.push(config)
+    }
     getToolConfig = mockGetToolConfig
     tryHasAccess = mockTryHasAccess
   },
@@ -47,6 +52,7 @@ afterEach(() => {
   mockGetToolConfig.mockClear()
   mockTryHasAccess.mockClear()
   mockReadContract.mockReset()
+  registryCtorConfigs.length = 0
 })
 
 describe("inspect command", () => {
@@ -75,6 +81,34 @@ describe("inspect command", () => {
     logSpy.mockRestore()
   })
 
+  it("threads --rpc-url through to the registry client", async () => {
+    vi.stubGlobal(
+      "fetch",
+      vi.fn(
+        async () =>
+          new Response(JSON.stringify(VALID_MANIFEST), { status: 200 }),
+      ),
+    )
+
+    const logSpy = vi.spyOn(console, "log").mockImplementation(() => {})
+
+    const { inspectCommand } = await import("../cli/commands/inspect.js")
+
+    await inspectCommand.parseAsync([
+      "node",
+      "inspect",
+      "--tool-id",
+      "1",
+      "--rpc-url",
+      "http://localhost:9999",
+    ])
+
+    expect(registryCtorConfigs).toHaveLength(1)
+    expect(registryCtorConfigs[0].rpcUrl).toBe("http://localhost:9999")
+
+    logSpy.mockRestore()
+  })
+
   it("reports MISMATCH when computed hash differs from onchain hash", async () => {
     mockGetToolConfig.mockResolvedValueOnce({
       creator: "0xabcdefabcdefabcdefabcdefabcdefabcdefabcd",

package.json

@@ -29,6 +29,25 @@ class ExitError extends Error {
   }
 }
 
+const registryCtorConfigs: Record<string, unknown>[] = []
+const mockRegisterTool = vi.fn(async () => ({
+  toolId: 7n,
+  txHash: "0xtxhash",
+}))
+
+vi.mock("../lib/onchain/registry.js", () => ({
+  ToolRegistryClient: class {
+    constructor(config: Record<string, unknown>) {
+      registryCtorConfigs.push(config)
+    }
+    registerTool = mockRegisterTool
+  },
+}))
+
+vi.mock("@opensea/wallet-adapters/viem", () => ({
+  walletAdapterToViemClient: vi.fn(async () => ({})),
+}))
+
 function mockFetch(manifest: object) {
   return vi.spyOn(globalThis, "fetch").mockResolvedValueOnce(
     new Response(JSON.stringify(manifest), {
@@ -58,6 +77,8 @@ describe("register creatorAddress validation", () => {
     exitSpy.mockRestore()
     errorSpy.mockRestore()
     logSpy.mockRestore()
+    registryCtorConfigs.length = 0
+    mockRegisterTool.mockClear()
     if (originalPrivateKey !== undefined) {
       process.env.PRIVATE_KEY = originalPrivateKey
     } else {
@@ -197,6 +218,8 @@ describe("register --access-predicate + --predicate-config (F4d)", () => {
     exitSpy.mockRestore()
     errorSpy.mockRestore()
     logSpy.mockRestore()
+    registryCtorConfigs.length = 0
+    mockRegisterTool.mockClear()
     if (originalPrivateKey !== undefined) {
       process.env.PRIVATE_KEY = originalPrivateKey
     } else {
@@ -332,4 +355,30 @@ describe("register --access-predicate + --predicate-config (F4d)", () => {
 
     fetchSpy.mockRestore()
   })
+
+  it("threads --rpc-url through to the registry client", async () => {
+    process.env.PRIVATE_KEY = TEST_PRIVATE_KEY
+    process.env.RPC_URL = "http://localhost:8545"
+    const fetchSpy = mockFetch(validManifest)
+
+    const { registerCommand } = await import("../cli/commands/register.js")
+
+    await registerCommand.parseAsync([
+      "node",
+      "register",
+      "--metadata",
+      "https://test.example.com/.well-known/ai-tools/test-tool.json",
+      "--rpc-url",
+      "http://localhost:9999",
+      "--yes",
+    ])
+
+    expect(mockRegisterTool).toHaveBeenCalled()
+    // The registry client waits for the tx receipt on its internal public
+    // client, so it must use the explicit RPC, never viem's chain default.
+    expect(registryCtorConfigs).toHaveLength(1)
+    expect(registryCtorConfigs[0].rpcUrl).toBe("http://localhost:9999")
+
+    fetchSpy.mockRestore()
+  })
 })

src/__tests__/inspect.test.ts

@@ -23,8 +23,13 @@ const mockGetToolConfig = vi.fn(async () => ({
   accessPredicate: "0x0000000000000000000000000000000000000000",
 }))
 
+const registryCtorConfigs: Record<string, unknown>[] = []
+
 vi.mock("../lib/onchain/registry.js", () => ({
   ToolRegistryClient: class {
+    constructor(config: Record<string, unknown>) {
+      registryCtorConfigs.push(config)
+    }
     updateToolMetadata = mockUpdateToolMetadata
     getToolConfig = mockGetToolConfig
   },
@@ -49,6 +54,7 @@ afterEach(() => {
   vi.unstubAllGlobals()
   mockUpdateToolMetadata.mockClear()
   mockGetToolConfig.mockClear()
+  registryCtorConfigs.length = 0
   delete process.env.PRIVATE_KEY
   delete process.env.RPC_URL
 })
@@ -119,4 +125,33 @@ describe("update-metadata command", () => {
 
     logSpy.mockRestore()
   })
+
+  it("threads --rpc-url through to both registry clients", async () => {
+    const logSpy = vi.spyOn(console, "log").mockImplementation(() => {})
+
+    const { updateMetadataCommand } = await import(
+      "../cli/commands/update-metadata.js"
+    )
+
+    await updateMetadataCommand.parseAsync([
+      "node",
+      "update-metadata",
+      "--tool-id",
+      "42",
+      "--metadata",
+      "https://example.com/manifest.json",
+      "--rpc-url",
+      "http://localhost:9999",
+      "--yes",
+    ])
+
+    // Both the ownership-check (read-only) client and the transaction
+    // client must use the explicit RPC, never viem's chain default.
+    expect(registryCtorConfigs).toHaveLength(2)
+    for (const config of registryCtorConfigs) {
+      expect(config.rpcUrl).toBe("http://localhost:9999")
+    }
+
+    logSpy.mockRestore()
+  })
 })

src/__tests__/register.test.ts

@@ -25,6 +25,7 @@ interface InspectOptions {
   toolId: string
   network: string
   checkAccess?: string
+  rpcUrl?: string
 }
 
 export const inspectCommand = new Command("inspect")
@@ -41,6 +42,7 @@ export const inspectCommand = new Command("inspect")
     "--check-access <address>",
     "Check whether an address has access to the tool",
   )
+  .option("--rpc-url <url>", "RPC endpoint for onchain reads")
   .action(async (options: InspectOptions) => {
     if (!options.toolId) {
       console.error(pc.red("Error: --tool-id is required"))
@@ -56,7 +58,7 @@ export const inspectCommand = new Command("inspect")
     }
 
     const chain = getChain(options.network)
-    const client = new ToolRegistryClient({ chain })
+    const client = new ToolRegistryClient({ chain, rpcUrl: options.rpcUrl })
 
     console.log(pc.cyan("Reading onchain tool config...\n"))
 
@@ -78,7 +80,7 @@ export const inspectCommand = new Command("inspect")
     if (config.accessPredicate !== zeroAddress) {
       const publicClient = createPublicClient({
         chain,
-        transport: http(),
+        transport: http(options.rpcUrl),
       })
 
       let predicateName: string | undefined