fix: fallback to string when response is not JSONable. (#2097)

Co-authored-by: eden <[email protected]>
Co-authored-by: Deep Singhvi <[email protected]>

Author: 3 people

packages/fern-docs/ui/src/playground/endpoint/PlaygroundEndpoint.tsx

@@ -144,7 +144,6 @@ export const PlaygroundEndpoint = ({
             break;
           }
           result += decoder.decode(value);
-          console.log(result);
           setResponse(
             loaded({
               type: "stream",

packages/fern-docs/ui/src/playground/fetch-utils/executeProxyRest.ts

@@ -48,34 +48,62 @@ export async function executeProxyRest(
     res.headers.get("Content-Type")?.toLowerCase()?.includes("application/json")
   ) {
     const startTime = Date.now();
-    const json = await res.json();
-    const endTime = Date.now();
+    try {
+      const text = await res.text();
+      const endTime = Date.now();
 
-    const fallbackTime =
-      Number(res.headers.get("X-Fern-Proxy-Origin-Latency") ?? 0) +
-      endTime -
-      startTime;
+      const fallbackTime =
+        Number(res.headers.get("X-Fern-Proxy-Origin-Latency") ?? 0) +
+        endTime -
+        startTime;
 
-    return {
-      type: "json",
-      response: {
-        headers: responseHeaders,
-        ok: res.ok,
-        redirected: res.redirected,
-        status: res.status,
-        statusText: res.statusText,
-        type: res.type,
-        url: res.url,
-        body: json,
-      },
-      contentType: res.headers.get("Content-Type") ?? "application/json",
-      time: Number(
-        res.headers.get("X-Fern-Proxy-Response-Time") ?? fallbackTime
-      ),
-      size:
-        res.headers.get("Content-Length") ??
-        String(new TextEncoder().encode(JSON.stringify(json)).length),
-    };
+      try {
+        const json = JSON.parse(text);
+        return {
+          type: "json",
+          response: {
+            headers: responseHeaders,
+            ok: res.ok,
+            redirected: res.redirected,
+            status: res.status,
+            statusText: res.statusText,
+            type: res.type,
+            url: res.url,
+            body: json,
+          },
+          contentType: res.headers.get("Content-Type") ?? "application/json",
+          time: Number(
+            res.headers.get("X-Fern-Proxy-Response-Time") ?? fallbackTime
+          ),
+          size:
+            res.headers.get("Content-Length") ??
+            String(new TextEncoder().encode(text).length),
+        };
+      } catch {
+        return {
+          type: "string",
+          response: {
+            headers: responseHeaders,
+            ok: res.ok,
+            redirected: res.redirected,
+            status: res.status,
+            statusText: res.statusText,
+            type: res.type,
+            url: res.url,
+            body: text,
+          },
+          contentType: res.headers.get("Content-Type") ?? "text/plain",
+          time: Number(
+            res.headers.get("X-Fern-Proxy-Response-Time") ?? fallbackTime
+          ),
+          size:
+            res.headers.get("Content-Length") ??
+            String(new TextEncoder().encode(text).length),
+        };
+      }
+    } catch () {
+      throw new Error("Failed to read response body");
+    }
   }
 
   const startTime = Date.now();

packages/fern-docs/ui/src/playground/types/playgroundResponse.ts

@@ -10,6 +10,14 @@ export declare namespace PlaygroundResponse {
     time: number;
   }
 
+  export interface String {
+    type: "string";
+    response: ProxyResponse.SerializableBody;
+    contentType: string;
+    time: number;
+    size: string | null;
+  }
+
   export interface Json {
     type: "json";
     response: ProxyResponse.SerializableBody;
@@ -30,4 +38,5 @@ export declare namespace PlaygroundResponse {
 export type PlaygroundResponse =
   | PlaygroundResponse.Stream
   | PlaygroundResponse.Json
+  | PlaygroundResponse.String
   | PlaygroundResponse.File;

packages/fern-docs/ui/src/playground/utils/oauth.ts

@@ -88,6 +88,9 @@ export const oAuthClientCredentialReferencedEndpointLoginFlow = async ({
         setDisplayFailedLogin && setDisplayFailedLogin(true);
       }
     },
+    string: () => {
+      setDisplayFailedLogin && setDisplayFailedLogin(true);
+    },
     file: () => {
       setDisplayFailedLogin && setDisplayFailedLogin(true);
     },