fix(google-docs): show error if agents api fails to process [INTEG-3419] (#10451)

Author: primeinteger

apps/google-docs/src/hooks/useGeneratePreview.ts

@@ -57,7 +57,7 @@ const parseAgentResponse = (data: any): AgentResponse => {
     } catch {
       // We do not throw an error here because we want to continue parsing the other parts
       // and other parts may already be valid and correctly parsed
-      console.warn('[parseAgentResponse] Failed to parse part:', { part, text: part.text });
+      console.warn('Failed to parse part:', { part, text: part.text });
     }
   }
 
@@ -110,14 +110,15 @@ const callGoogleDocsAgent = async (
   }
 
   const data = await response.text();
+
+  let parsedData: any;
   try {
-    const parsedData = JSON.parse(data);
-    return parseAgentResponse(parsedData);
-  } catch (error) {
-    console.error('[callGoogleDocsAgent] Failed to parse response:', error);
-    // Graceful fall back state so app doesn't crash but we still know there was an error
-    return { entries: [], assets: [] };
+    parsedData = JSON.parse(data);
+  } catch {
+    throw new Error('Failed to parse google docs agent response as JSON');
   }
+
+  return parseAgentResponse(parsedData);
 };
 
 interface UseGeneratePreviewResult {
@@ -201,6 +202,10 @@ export const useGeneratePreview = ({
         setPreviewEntries(previewEntriesWithTitles);
         setAssets(agentAssets);
       } catch (err) {
+        console.error(
+          'Error generating preview:',
+          err instanceof Error ? err.message : String(err)
+        );
         setError(err instanceof Error ? err : new Error(String(err)));
       } finally {
         setIsSubmitting(false);

apps/google-docs/src/locations/Page/components/mainpage/OAuthConnector.tsx

@@ -52,10 +52,6 @@ export const OAuthConnector = ({
 
     for (let attempt = 1; attempt <= maxRetries; attempt++) {
       try {
-        console.log(
-          `Checking Google OAuth connection status (attempt ${attempt}/${maxRetries})...`
-        );
-
         const { connected, token } = await callAppActionWithResult<CheckStatusResponse>(
           sdk,
           'checkGdocOauthTokenStatus',
@@ -64,19 +60,13 @@ export const OAuthConnector = ({
 
         // Assuming the response contains a connected field
         const isConnected = connected === true;
-        console.log(`Google OAuth connection status (attempt ${attempt}):`, isConnected);
         onOauthTokenChange(token);
 
         // If we have an expected status and it matches, or if we don't have an expected status, accept the result
         if (expectedStatus === undefined || isConnected === expectedStatus) {
           onOAuthConnectedChange(isConnected);
-          console.log(`Status check resolved to expected value: ${isConnected}`);
           break;
         } else {
-          console.log(
-            `Status mismatch. Expected: ${expectedStatus}, Got: ${isConnected}. Retrying...`
-          );
-
           // If this is the last attempt, accept the current result anyway
           if (attempt === maxRetries) {
             console.log(`Max retries reached. Accepting current status: ${isConnected}`);
@@ -86,7 +76,6 @@ export const OAuthConnector = ({
 
           // Wait before retrying (exponential backoff: 500ms, 1000ms, 1500ms, etc.)
           const waitTime = 500 * attempt;
-          console.log(`Waiting ${waitTime}ms before retry...`);
           await delay(waitTime);
         }
       } catch (error) {
@@ -101,13 +90,11 @@ export const OAuthConnector = ({
 
         // Wait before retrying on error
         const waitTime = 500 * attempt;
-        console.log(`Waiting ${waitTime}ms before retry after error...`);
         await delay(waitTime);
       }
     }
 
     setLoadingState(OAuthLoadingState.IDLE);
-    console.log(`Status check polling completed. Final status: ${isOAuthConnected}`);
   };
 
   const messageHandler = async (event: MessageEvent) => {