From 880a249bee19eb9f5003cfbb5192917e2ffecefb Mon Sep 17 00:00:00 2001 From: Mikers Date: Thu, 5 Feb 2026 10:47:16 -1000 Subject: [PATCH] fix(ui): use viem function names for overloaded getters --- packages/templates/next-export-ui/src/lib/app.ts | 5 ++--- .../next-export-ui/test-scaffold/tests/ui/smoke.mjs | 5 +++++ 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/packages/templates/next-export-ui/src/lib/app.ts b/packages/templates/next-export-ui/src/lib/app.ts index 9eb276f..493023c 100644 --- a/packages/templates/next-export-ui/src/lib/app.ts +++ b/packages/templates/next-export-ui/src/lib/app.ts @@ -15,9 +15,8 @@ export function fnListIds(collectionName: string): string { } export function fnGet(collectionName: string): string { - // The generated contract overloads getC(uint256,bool) and getC(uint256). - // Use the full signature to disambiguate for viem encoding/decoding. - return `get${collectionName}(uint256)`; + // viem expects the function name (not full signature) and resolves overloads from args. + return `get${collectionName}`; } export function fnCreate(collectionName: string): string { diff --git a/packages/templates/next-export-ui/test-scaffold/tests/ui/smoke.mjs b/packages/templates/next-export-ui/test-scaffold/tests/ui/smoke.mjs index 6e4fd34..0a4e216 100644 --- a/packages/templates/next-export-ui/test-scaffold/tests/ui/smoke.mjs +++ b/packages/templates/next-export-ui/test-scaffold/tests/ui/smoke.mjs @@ -26,6 +26,11 @@ async function assertRoute200(baseUrl, route) { const u = `${baseUrl}${route}`; const out = await fetchOrThrow(u); assert.equal(out.status, 200, `Expected ${u} to return 200, got ${out.status}`); + assert.equal( + out.text.includes('not found on ABI'), + false, + `Route ${u} rendered an ABI lookup error. Check generated UI function names vs ABI overload handling.` + ); } async function runLiveChecks(root, baseUrl, ths) {