fix(connectors): inspect all drive error reasons

Author: waleedlatif1

apps/sim/connectors/google-drive/google-drive-errors.ts

@@ -98,17 +98,22 @@ function classifyGoogleDriveError(
 
 export class GoogleDriveApiError extends Error {
   retryAfterMs?: number
+  readonly reasons: readonly string[]
+  readonly kind: GoogleDriveErrorKind
   readonly rateLimited: boolean
 
   constructor(
     readonly status: number,
-    readonly reasons: readonly string[],
-    readonly kind: GoogleDriveErrorKind
+    normalizedReasons: readonly string[]
   ) {
-    const reasonSuffix = reasons.length > 0 ? ` (${reasons.join(', ')})` : ''
+    const diagnosticReasons = normalizedReasons.slice(0, GOOGLE_ERROR_REASON_MAX_COUNT)
+    const reasonSuffix = diagnosticReasons.length > 0 ? ` (${diagnosticReasons.join(', ')})` : ''
     super(`Google Drive API request failed with HTTP ${status}${reasonSuffix}`)
     this.name = 'GoogleDriveApiError'
-    this.rateLimited = status === 429 || reasons.some((reason) => RATE_LIMIT_REASONS.has(reason))
+    this.reasons = diagnosticReasons
+    this.kind = classifyGoogleDriveError(status, normalizedReasons)
+    this.rateLimited =
+      status === 429 || normalizedReasons.some((reason) => RATE_LIMIT_REASONS.has(reason))
   }
 }
 
@@ -131,13 +136,8 @@ export async function readGoogleDriveApiError(response: Response): Promise<Googl
 
   const entries = parsedBody?.error?.errors ?? []
   const rawReasons = [...new Set(entries.flatMap((entry) => (entry.reason ? [entry.reason] : [])))]
-  const reasons = [...new Set(rawReasons.flatMap((reason) => normalizeReason(reason) ?? []))].slice(
-    0,
-    GOOGLE_ERROR_REASON_MAX_COUNT
-  )
-  return new GoogleDriveApiError(
-    response.status,
-    reasons,
-    classifyGoogleDriveError(response.status, rawReasons)
-  )
+  const normalizedReasons = [
+    ...new Set(rawReasons.flatMap((reason) => normalizeReason(reason) ?? [])),
+  ]
+  return new GoogleDriveApiError(response.status, normalizedReasons)
 }

apps/sim/connectors/google-drive/google-drive.test.ts

@@ -106,6 +106,27 @@ describe('Google Drive API error parsing', () => {
     expect(backendFailure.rateLimited).toBe(false)
   })
 
+  it('detects a structured rate limit beyond the bounded diagnostic reasons', async () => {
+    const reasons = [
+      ...Array.from({ length: 16 }, (_, index) => `providerReason${index}`),
+      'userRateLimitExceeded',
+    ]
+    const error = await readGoogleDriveApiError(
+      jsonResponse(
+        {
+          error: {
+            errors: reasons.map((reason) => ({ reason })),
+          },
+        },
+        403
+      )
+    )
+
+    expect(error.reasons).toEqual(reasons.slice(0, 16))
+    expect(error.kind).toBe('transient')
+    expect(error.rateLimited).toBe(true)
+  })
+
   it('omits provider messages from diagnostics', async () => {
     const message = `Authorization: Bearer private-token\ncontext ${'x'.repeat(700)}`
     const error = await readGoogleDriveApiError(