fix(zapier): paginate the Record dropdown and label by primary field

The records dropdown ignored bundle.meta.page, so "Load more" couldn't reach
records past the first page, and it labelled every option with the raw id.
Page through the table via skip = page * 100, and use Teable's record `name`
(the primary field value) as the label, falling back to the id. Adds `skip`
support to listRecords; drops the now-unused isPrimary lookup.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: caoxing9

packages/zapier/src/lib/records.ts

@@ -30,6 +30,7 @@ interface ListRecordsOptions {
   tableId: string;
   viewId?: string;
   take?: number;
+  skip?: number;
 }
 
 // How many records each poll pulls. Teable can't sort by the system
@@ -43,9 +44,10 @@ const POLL_PAGE_SIZE = 600;
 const listRecords = async (
   z: ZObject,
   bundle: Bundle,
-  { tableId, viewId, take = POLL_PAGE_SIZE }: ListRecordsOptions,
+  { tableId, viewId, take = POLL_PAGE_SIZE, skip }: ListRecordsOptions,
 ): Promise<IRecord[]> => {
   const params: Record<string, unknown> = { take, fieldKeyType: 'name' };
+  if (skip) params.skip = skip;
   if (viewId) params.viewId = viewId;
   const response = await z.request<IRecordsVo>({
     url: apiUrl(bundle, `/table/${tableId}/record`),

packages/zapier/src/lib/types.ts

@@ -13,7 +13,6 @@ export interface TeableField {
   id: string;
   name: string;
   type: string;
-  isPrimary?: boolean;
   options?: {
     choices?: Array<{ name: string }>;
   };

packages/zapier/src/triggers/records.ts

@@ -1,26 +1,28 @@
 import type { ZObject, Bundle } from 'zapier-platform-core';
 import { listRecords } from '../lib/records';
-import { fetchFields } from '../lib/fields';
 import type { DropdownItem } from '../lib/types';
 
+// How many records each dropdown page loads. Zapier calls perform again (with an
+// incremented bundle.meta.page) when the user clicks "Load more".
+const PAGE_SIZE = 100;
+
 // Hidden trigger powering the "Record" dropdown on actions/searches that take a
-// recordId (Update, Delete, Find Record by ID). Depends on a chosen tableId;
-// labels each record by its primary field value, falling back to the record id.
+// recordId (Update, Delete, Find Record by ID). Depends on a chosen tableId and
+// pages through the table so large tables stay reachable. Each record is
+// labelled by its primary field value (Teable's `name`), falling back to the id.
 const perform = async (z: ZObject, bundle: Bundle): Promise<DropdownItem[]> => {
   const tableId = bundle.inputData.tableId as string | undefined;
   if (!tableId) return [];
-  const [records, fields] = await Promise.all([
-    listRecords(z, bundle, { tableId, take: 100 }),
-    fetchFields(z, bundle, tableId),
-  ]);
-  const primary = fields.find((f) => f.isPrimary);
-  return records.map((r) => {
-    const label = primary ? r.fields[primary.name] : undefined;
-    return {
-      id: r.id,
-      name: label != null && label !== '' ? String(label) : r.id,
-    };
+  const page = (bundle.meta && (bundle.meta.page as number)) || 0;
+  const records = await listRecords(z, bundle, {
+    tableId,
+    take: PAGE_SIZE,
+    skip: page * PAGE_SIZE,
   });
+  return records.map((r) => ({
+    id: r.id,
+    name: r.name && r.name !== '' ? r.name : r.id,
+  }));
 };
 
 export default {