fix: bot notes

Author: shrugs

.changeset/dirty-snakes-knock.md

@@ -2,4 +2,4 @@
 "ensapi": minor
 ---
 
-Omnigraph API (BREAKING): Removed `ENSv2Domain.canonicalId` in favor of accessing `ENSv2Domain.tokenId` directly.
+Omnigraph API (BREAKING): Removed `ENSv2Domain.canonicalId`.

apps/ensapi/src/omnigraph-api/lib/find-domains/find-domains-resolver-helpers.test.ts

@@ -1,7 +1,6 @@
 import { describe, expect, it, vi } from "vitest";
 
 vi.mock("@/config", () => ({ default: { namespace: "mainnet" } }));
-vi.mock("@/omnigraph-api/lib/find-domains/find-domains-by-labelhash-path", () => ({}));
 
 import { isEffectiveDesc } from "./find-domains-resolver-helpers";
 

packages/enssdk/src/omnigraph/graphql.ts

@@ -25,7 +25,8 @@ export const graphql = initGraphQLTada<{
     // NOTE: graphql clients don't really do deserialization of scalars like bigint, so instead we
     // just helpfully type the string as 'a stringified bigint'
     BigInt: `${bigint}`;
-    // NOTE: keep these scalar types in sync with the scalars in apps/ensapi/src/omnigraph-api/builder.ts
+    // NOTE: keep these semantic scalar types in sync with the scalars in apps/ensapi/src/omnigraph-api/builder.ts
+    // (i.e. excluding the BigInt scalar, which we handle above)
     Address: Address;
     Hex: Hex;
     ChainId: ChainId;