fix(zerobounce): handle 200-status API errors and guard JSON parsing

Author: waleedlatif1

apps/sim/tools/zerobounce/get_credits.ts

@@ -28,14 +28,20 @@ export const getCreditsTool: ToolConfig<ZeroBounceGetCreditsParams, ZeroBounceGe
     },
 
     transformResponse: async (response: Response) => {
-      if (!response.ok) {
+      const data = await response.json().catch(() => ({}))
+      // ZeroBounce returns HTTP 200 with an `{ error }` envelope on auth failure,
+      // so detect API-level errors from the body, not just the HTTP status.
+      const errorMessage =
+        typeof data === 'object' && data !== null && typeof data.error === 'string'
+          ? data.error
+          : ''
+      if (!response.ok || errorMessage.length > 0) {
         return {
           success: false,
-          error: `ZeroBounce API error: ${response.status} ${response.statusText}`,
+          error: errorMessage || `ZeroBounce API error: ${response.status} ${response.statusText}`,
           output: { credits: 0 },
         }
       }
-      const data = await response.json()
       const credits = Number(data.Credits ?? 0)
       return {
         success: true,

apps/sim/tools/zerobounce/verify_email.ts

@@ -54,7 +54,7 @@ export const verifyEmailTool: ToolConfig<
         output: { email: '', status: '', deliverable: false },
       }
     }
-    const data = await response.json()
+    const data = await response.json().catch(() => ({}))
     if (data.error) {
       return {
         success: false,