chore(passport): improve passport post to relayer error message

Author: nattb8

packages/passport/sdk/src/zkEvm/relayerClient.test.ts

@@ -44,6 +44,8 @@ describe('relayerClient', () => {
       const data = '0x123';
 
       (global.fetch as jest.Mock).mockResolvedValue({
+        ok: true,
+        text: () => Promise.resolve(JSON.stringify({ result: transactionHash })),
         json: () => ({
           result: transactionHash,
         }),
@@ -82,6 +84,8 @@ describe('relayerClient', () => {
       };
 
       (global.fetch as jest.Mock).mockResolvedValue({
+        ok: true,
+        text: () => Promise.resolve(JSON.stringify({ result: relayerTransaction })),
         json: () => ({
           result: relayerTransaction,
         }),
@@ -118,6 +122,8 @@ describe('relayerClient', () => {
       }];
 
       (global.fetch as jest.Mock).mockResolvedValue({
+        ok: true,
+        text: () => Promise.resolve(JSON.stringify({ result: feeOptions })),
         json: () => ({
           result: feeOptions,
         }),
@@ -152,6 +158,8 @@ describe('relayerClient', () => {
       const relayerSignature = '0x123';
 
       (global.fetch as jest.Mock).mockResolvedValue({
+        ok: true,
+        text: () => Promise.resolve(JSON.stringify({ result: relayerSignature })),
         json: () => ({
           result: relayerSignature,
         }),
@@ -186,6 +194,8 @@ describe('relayerClient', () => {
       const relayerSignature = '0x123';
 
       (global.fetch as jest.Mock).mockResolvedValue({
+        ok: true,
+        text: () => Promise.resolve(JSON.stringify({ result: relayerSignature })),
         json: () => ({
           result: relayerSignature,
         }),

packages/passport/sdk/src/zkEvm/relayerClient.ts

@@ -102,6 +102,12 @@ export class RelayerClient {
     this.authManager = authManager;
   }
 
+  private static getResponsePreview(text: string): string {
+    return text.length > 100
+      ? `${text.substring(0, 50)}...${text.substring(text.length - 50)}`
+      : text;
+  }
+
   private async postToRelayer<T>(request: RelayerTransactionRequest): Promise<T> {
     const body: RelayerTransactionRequest & JsonRpc = {
       id: 1,
@@ -120,7 +126,20 @@ export class RelayerClient {
       body: JSON.stringify(body),
     });
 
-    const jsonResponse = await response.json();
+    if (!response.ok) {
+      const responseText = await response.text();
+      const preview = RelayerClient.getResponsePreview(responseText);
+      throw new Error(`Relayer HTTP error: ${response.status} ${response.statusText}. Content: "${preview}"`);
+    }
+
+    const responseText = await response.text();
+    let jsonResponse;
+    try {
+      jsonResponse = JSON.parse(responseText);
+    } catch (parseError) {
+      const preview = RelayerClient.getResponsePreview(responseText);
+      throw new Error(`Relayer JSON parse error: ${parseError instanceof Error ? parseError.message : 'Unknown error'}. Content: "${preview}"`);
+    }
     if (jsonResponse.error) {
       throw jsonResponse.error;
     }