test(tools): enforce that the two tool-id resolvers never diverge

`resolveToolId` now exists twice on purpose — `@/tools/utils` resolves against
the live registry (so a tool added before regeneration still resolves at
runtime), `@/tools/tool-ids` against the generated id list (so client code
resolves without importing 4,300 tools). Nothing structurally kept them in step;
a change to versioning logic in one would silently drift from the other.

`tool-metadata:check` now asserts they agree across every id, every stripped
base name, and an unknown — 4,404 probes — and only after the staleness check
passes, so a missing regeneration reports as staleness rather than as drift.
Verified it fails: breaking resolution for `gmail*` exits 1; restoring it passes.

It cannot live in a vitest suite. `vitest.setup.ts` globally mocks
`@/tools/registry` to an empty map, so `getTool` resolves nothing there — a
parity test written as a spec passes or fails for the wrong reason. Both facts
are recorded where the code is.

Both resolvers stay exported. An earlier pass here un-exported the `@/tools/utils`
one as dead; `tools/utils.server.ts` imports it through a multi-line import that
a grep missed, and `tsc` caught it. Its doc now says which resolver a caller
should reach for instead of leaving two identically-named functions unexplained.

Author: waleedlatif1

apps/sim/tools/utils.ts

@@ -58,6 +58,13 @@ export function getLatestVersionTools(
 /**
  * Resolves a tool name to its actual tool ID in the registry.
  * Handles both stripped names (e.g., 'notion_search') and versioned names (e.g., 'notion_search_v2').
+ *
+ * Server-side counterpart to `resolveToolId` in `@/tools/tool-ids`. Both exist
+ * deliberately: this one reads the live registry, so a tool added but not yet
+ * regenerated stays resolvable; that one resolves against the generated id list
+ * without pulling 4,300 tools into a client graph. Client code wants that one.
+ * `tool-metadata:check` asserts the two never diverge.
+ *
  * @param toolName The tool name to resolve (may or may not have version suffix)
  * @returns The actual tool ID in the registry, or the original name if not found
  */

apps/sim/tools/zoho/types.ts

@@ -0,0 +1,200 @@
+import type { ToolResponse } from '@/tools/types'
+
+/** Params shared by every Zoho tool: the OAuth token and the account's region. */
+export interface ZohoBaseParams {
+  accessToken: string
+  dataCenter?: string
+}
+
+/** Params shared by every Zoho Desk tool — Desk requires an `orgId` header. */
+export interface ZohoDeskBaseParams extends ZohoBaseParams {
+  orgId: string
+}
+
+/**
+ * Pagination block returned by CRM list/search endpoints.
+ * @see https://www.zoho.com/crm/developer/docs/api/v8/get-records.html
+ */
+export interface ZohoCrmPageInfo {
+  page: number | null
+  perPage: number | null
+  count: number | null
+  moreRecords: boolean
+}
+
+export const ZOHO_CRM_PAGE_INFO_OUTPUT = {
+  type: 'object' as const,
+  description: 'Pagination metadata returned by Zoho CRM',
+  properties: {
+    page: { type: 'number' as const, description: 'Current page number', optional: true },
+    perPage: { type: 'number' as const, description: 'Records requested per page', optional: true },
+    count: {
+      type: 'number' as const,
+      description: 'Number of records in this page',
+      optional: true,
+    },
+    moreRecords: { type: 'boolean' as const, description: 'Whether further pages are available' },
+  },
+}
+
+/** A CRM write result entry, as returned in the `data` array of insert/update/upsert. */
+export interface ZohoCrmWriteResult {
+  id: string | null
+  code: string | null
+  status: string | null
+  message: string | null
+}
+
+export const ZOHO_CRM_WRITE_RESULT_OUTPUT = {
+  type: 'array' as const,
+  description: 'Per-record results returned by Zoho CRM',
+  properties: {
+    id: { type: 'string' as const, description: 'Record ID', optional: true },
+    code: {
+      type: 'string' as const,
+      description: 'Zoho result code (e.g. SUCCESS)',
+      optional: true,
+    },
+    status: { type: 'string' as const, description: 'Result status', optional: true },
+    message: {
+      type: 'string' as const,
+      description: 'Human-readable result message',
+      optional: true,
+    },
+  },
+}
+
+export interface ZohoCrmGetRecordsParams extends ZohoBaseParams {
+  module: string
+  recordId?: string
+  fields?: string
+  page?: string | number
+  perPage?: string | number
+  sortBy?: string
+  sortOrder?: string
+}
+
+export interface ZohoCrmGetRecordsResponse extends ToolResponse {
+  output: {
+    records: Record<string, unknown>[]
+    record?: Record<string, unknown> | null
+    info: ZohoCrmPageInfo
+  }
+}
+
+export interface ZohoCrmCreateRecordsParams extends ZohoBaseParams {
+  module: string
+  records: unknown
+  trigger?: string
+}
+
+export interface ZohoCrmWriteResponse extends ToolResponse {
+  output: {
+    results: ZohoCrmWriteResult[]
+  }
+}
+
+export interface ZohoCrmUpdateRecordParams extends ZohoBaseParams {
+  module: string
+  recordId: string
+  record: unknown
+  trigger?: string
+}
+
+export interface ZohoCrmUpsertRecordsParams extends ZohoBaseParams {
+  module: string
+  records: unknown
+  duplicateCheckFields?: string
+  trigger?: string
+}
+
+export interface ZohoCrmDeleteRecordParams extends ZohoBaseParams {
+  module: string
+  recordId: string
+  wfTrigger?: string
+}
+
+export interface ZohoCrmSearchRecordsParams extends ZohoBaseParams {
+  module: string
+  criteria?: string
+  email?: string
+  phone?: string
+  word?: string
+  fields?: string
+  page?: string | number
+  perPage?: string | number
+}
+
+export interface ZohoCrmCoqlQueryParams extends ZohoBaseParams {
+  selectQuery: string
+}
+
+export interface ZohoCrmCoqlQueryResponse extends ToolResponse {
+  output: {
+    records: Record<string, unknown>[]
+    info: ZohoCrmPageInfo
+  }
+}
+
+export interface ZohoCrmGetModulesParams extends ZohoBaseParams {}
+
+export interface ZohoCrmGetModulesResponse extends ToolResponse {
+  output: {
+    modules: Array<{
+      apiName: string | null
+      moduleName: string | null
+      id: string | null
+      pluralLabel: string | null
+      singularLabel: string | null
+      creatable: boolean | null
+      editable: boolean | null
+      deletable: boolean | null
+    }>
+  }
+}
+
+export interface ZohoCrmGetFieldsParams extends ZohoBaseParams {
+  module: string
+}
+
+export interface ZohoCrmGetFieldsResponse extends ToolResponse {
+  output: {
+    fields: Array<{
+      apiName: string | null
+      displayLabel: string | null
+      dataType: string | null
+      id: string | null
+      required: boolean | null
+      readOnly: boolean | null
+      length: number | null
+    }>
+  }
+}
+
+export interface ZohoCrmGetUsersParams extends ZohoBaseParams {
+  type?: string
+  page?: string | number
+  perPage?: string | number
+}
+
+export interface ZohoCrmGetUsersResponse extends ToolResponse {
+  output: {
+    users: Record<string, unknown>[]
+    info: ZohoCrmPageInfo
+  }
+}
+
+export interface ZohoCrmAddNoteParams extends ZohoBaseParams {
+  parentId: string
+  seModule: string
+  noteTitle?: string
+  noteContent: string
+}
+
+export type ZohoCrmResponse =
+  | ZohoCrmGetRecordsResponse
+  | ZohoCrmWriteResponse
+  | ZohoCrmCoqlQueryResponse
+  | ZohoCrmGetModulesResponse
+  | ZohoCrmGetFieldsResponse
+  | ZohoCrmGetUsersResponse