fix(world): dedupe functions in getWorldAbi result

packages/world/ts/common.ts

@@ -1,4 +1,4 @@
-import { Hex } from "viem";
+import { Hex, createTestClient, http, publicActions, walletActions } from "viem";
 
 export type WorldFunction = {
   readonly signature: string;
@@ -7,3 +7,19 @@ export type WorldFunction = {
   readonly systemFunctionSignature: string;
   readonly systemFunctionSelector: Hex;
 };
+
+export const anvilHost = "127.0.0.1";
+export const anvilPort = 8555;
+
+// ID of the current test worker. Used by the `@viem/anvil` proxy server.
+export const poolId = Number(process.env.VITEST_POOL_ID ?? 1);
+
+export const anvilRpcUrl = `http://${anvilHost}:${anvilPort}/${poolId}`;
+
+export const testClient = createTestClient({
+  mode: "anvil",
+  transport: http(anvilRpcUrl),
+  pollingInterval: 10,
+})
+  .extend(publicActions)
+  .extend(walletActions);

packages/world/ts/getWorldAbi.test.ts

@@ -0,0 +1,44 @@
+import { describe, expect, it, vi } from "vitest";
+import { getWorldAbi } from "./getWorldAbi";
+import { testClient } from "./common";
+
+vi.mock("./getFunctions", () => {
+  const mockGetFunctionsResult = [{ signature: "setNumber(bool)" }, { signature: "batchCall((bytes32,bytes)[])" }];
+  const getFunctions = vi.fn();
+  getFunctions.mockResolvedValue(mockGetFunctionsResult);
+
+  return {
+    getFunctions,
+  };
+});
+
+describe("World ABI", () => {
+  it("should concat base and world ABI", async () => {
+    const abi = await getWorldAbi({
+      client: testClient,
+      worldAddress: "0xbBbBBBBbbBBBbbbBbbBbbbbBBbBbbbbBbBbbBBbB",
+      fromBlock: 0n,
+      toBlock: 0n,
+    });
+
+    expect(abi).toContainEqual({
+      inputs: [
+        {
+          type: "bool",
+        },
+      ],
+      name: "setNumber",
+      outputs: [],
+      stateMutability: "nonpayable",
+      type: "function",
+    });
+
+    expect(abi).not.toContainEqual({
+      name: "batchCall",
+      type: "function",
+      stateMutability: "nonpayable",
+      inputs: [{ type: "tuple[]", components: [{ type: "bytes32" }, { type: "bytes" }] }],
+      outputs: [],
+    });
+  });
+});

packages/world/ts/getWorldAbi.ts

@@ -1,8 +1,12 @@
-import { Client, Abi, Address, getAddress } from "viem";
+import { Client, Abi, AbiItem, AbiFunction, Address, getAddress, toFunctionSelector } from "viem";
 import IBaseWorldAbi from "../out/IBaseWorld.sol/IBaseWorld.abi.json";
 import { functionSignatureToAbiItem } from "./functionSignatureToAbiItem";
 import { getFunctions } from "./getFunctions";
 
+function isAbiFunction(abiItem: AbiItem): abiItem is AbiFunction {
+  return abiItem.type === "function";
+}
+
 export async function getWorldAbi({
   client,
   worldAddress,
@@ -20,7 +24,10 @@ export async function getWorldAbi({
     fromBlock,
     toBlock,
   });
-  const worldFunctionsAbi = worldFunctions.map((func) => functionSignatureToAbiItem(func.signature));
+  const baseFunctionSelectors = (IBaseWorldAbi as Abi).filter(isAbiFunction).map(toFunctionSelector);
+  const worldFunctionsAbi = worldFunctions
+    .map((func) => functionSignatureToAbiItem(func.signature))
+    .filter((abiItem) => !baseFunctionSelectors.includes(toFunctionSelector(abiItem as AbiFunction)));
   const abi = [...IBaseWorldAbi, ...worldFunctionsAbi];
 
   return abi;