fix(coda): retry field-setting PATCH updates on rate limits and server errors

Author: waleedlatif1

apps/docs/content/docs/integrations/coda.mdx

@@ -27,7 +27,7 @@ This integration uses a reusable Coda **API token** connection, not OAuth. In Co
 - **Pagination:** List operations return a `nextPageToken`. Pass it as **Page Token** to get the next page. When a Page Token is set, the original filters and limit are reused automatically. Coda can change page sizes at any time, so keep paging until no token comes back.
 - **Page content:** **Get Page Content** reads a page as plain-text lines, with element IDs you can target in **Update Page** or **Delete Page Content**. To delete content, enter element IDs, or turn on **Delete All Page Content** to clear the whole page. For the full page as Markdown or HTML, run **Export Page**, then call **Get Page Export Status** until `downloadLink` is present. The link expires shortly after it is issued.
 - **Permissions and plans:** Creating docs and pages or renaming a doc requires Doc Maker access in the workspace. Publishing needs a Coda maker profile. Hiding pages and custom domains need a paid Coda plan. Workspace members, role activity, and role changes need a workspace that belongs to an organization, and role changes need Admin access. Page analytics are only available for docs in Enterprise workspaces.
-- **Rate limits:** Coda limits requests per user, with tighter limits on writes and on listing docs. Sim automatically retries reads, updates, and deletes when Coda returns HTTP 429. Inserts and creates are not retried, so space out bulk writes.
+- **Rate limits:** Coda limits requests per user, with tighter limits on writes and on listing docs. Sim automatically retries reads, updates, and deletes when Coda returns HTTP 429 or a transient server error. Inserts and creates are not retried, so space out bulk writes.
 {/* MANUAL-CONTENT-END */}
 
 

apps/sim/tools/coda/coda.test.ts

@@ -16,7 +16,7 @@ import { codaUpdateAclSettingsTool } from '@/tools/coda/update_acl_settings'
 import { codaUpdatePageTool } from '@/tools/coda/update_page'
 import { codaUpdateRowTool } from '@/tools/coda/update_row'
 import { codaUpsertRowsTool } from '@/tools/coda/upsert_rows'
-import { buildCodaUrl, CODA_RETRY } from '@/tools/coda/utils'
+import { buildCodaUrl, CODA_FIELD_UPDATE_RETRY, CODA_RETRY } from '@/tools/coda/utils'
 import { codaWhoamiTool } from '@/tools/coda/whoami'
 import { ErrorExtractorId, extractErrorMessageWithId } from '@/tools/error-extractors'
 
@@ -340,14 +340,16 @@ describe('Coda tool registration', () => {
   })
 
   it.each(allTools)(
-    '%s retries idempotent calls and authenticates with the Coda credential',
+    '%s retries safely repeatable calls and authenticates with the Coda credential',
     (_, tool) => {
       const config = tool as {
-        request: { retry?: unknown }
+        request: { method: unknown; retry?: unknown }
         oauth?: unknown
         params: Record<string, unknown>
       }
-      expect(config.request.retry).toBe(CODA_RETRY)
+      expect(config.request.retry).toBe(
+        config.request.method === 'PATCH' ? CODA_FIELD_UPDATE_RETRY : CODA_RETRY
+      )
       expect(config.oauth).toEqual({ required: true, provider: 'coda' })
       expect(config.params.accessToken).toMatchObject({ required: true, visibility: 'hidden' })
     }

apps/sim/tools/coda/update_acl_settings.ts

@@ -2,7 +2,7 @@ import type { CodaAclSettingsParams, CodaAclSettingsResponse } from '@/tools/cod
 import {
   ACL_SETTINGS_OUTPUTS,
   buildCodaUrl,
-  CODA_RETRY,
+  CODA_FIELD_UPDATE_RETRY,
   codaAuthParams,
   codaDocPath,
   codaHeaders,
@@ -54,7 +54,7 @@ export const codaUpdateAclSettingsTool: ToolConfig<CodaAclSettingsParams, CodaAc
     request: {
       url: (params) => buildCodaUrl(codaDocPath(params.docId, 'acl', 'settings')),
       method: 'PATCH',
-      retry: CODA_RETRY,
+      retry: CODA_FIELD_UPDATE_RETRY,
       headers: (params) => codaHeaders(params.accessToken, true),
       body: (params) => {
         const body: Record<string, boolean> = {}

apps/sim/tools/coda/update_doc.ts

@@ -1,7 +1,7 @@
 import type { CodaDocIdResponse, CodaUpdateDocParams } from '@/tools/coda/types'
 import {
   buildCodaUrl,
-  CODA_RETRY,
+  CODA_FIELD_UPDATE_RETRY,
   codaAuthParams,
   codaDocPath,
   codaHeaders,
@@ -41,7 +41,7 @@ export const codaUpdateDocTool: ToolConfig<CodaUpdateDocParams, CodaDocIdRespons
   request: {
     url: (params) => buildCodaUrl(codaDocPath(params.docId)),
     method: 'PATCH',
-    retry: CODA_RETRY,
+    retry: CODA_FIELD_UPDATE_RETRY,
     headers: (params) => codaHeaders(params.accessToken, true),
     body: (params) => {
       const title = optionalTrimmed(params.title)

apps/sim/tools/coda/update_folder.ts

@@ -1,7 +1,7 @@
 import type { CodaFolderResponse, CodaUpdateFolderParams } from '@/tools/coda/types'
 import {
   buildCodaUrl,
-  CODA_RETRY,
+  CODA_FIELD_UPDATE_RETRY,
   codaAuthParams,
   codaHeaders,
   codaOAuth,
@@ -44,7 +44,7 @@ export const codaUpdateFolderTool: ToolConfig<CodaUpdateFolderParams, CodaFolder
   request: {
     url: (params) => buildCodaUrl(codaPath('folders', [params.folderId, 'folderId'])),
     method: 'PATCH',
-    retry: CODA_RETRY,
+    retry: CODA_FIELD_UPDATE_RETRY,
     headers: (params) => codaHeaders(params.accessToken, true),
     body: (params) => {
       const name = optionalTrimmed(params.name)

apps/sim/tools/coda/utils.ts

@@ -213,6 +213,15 @@ export const CODA_RETRY = {
   retryIdempotentOnly: true,
 } as const satisfies ToolRetryConfig
 
+/**
+ * Coda's PATCH endpoints set the supplied fields to fixed values, so repeating one is safe even
+ * though the executor does not treat PATCH as idempotent.
+ */
+export const CODA_FIELD_UPDATE_RETRY = {
+  ...CODA_RETRY,
+  retryIdempotentOnly: false,
+} as const satisfies ToolRetryConfig
+
 interface RawReference {
   id?: string
   name?: string