@robosystems/client 0.2.24 → 0.2.26

package/sdk-extensions/QueryClient.js

@@ -28,6 +28,11 @@ class QueryClient {
             ...(options.mode === 'stream' ? { parseAs: 'stream' } : {}),
         };
         const response = await (0, sdk_gen_1.executeCypherQuery)(data);
+        // Check for errors in the response (network errors, etc.)
+        if ('error' in response && response.error) {
+            const error = response.error;
+            throw error instanceof Error ? error : new Error(String(error));
+        }
         // Check if this is a raw stream response (when parseAs: 'stream')
         if (options.mode === 'stream' && response.response) {
             const contentType = response.response.headers.get('content-type') || '';
@@ -81,6 +86,8 @@ class QueryClient {
         let columns = null;
         let totalRows = 0;
         let executionTimeMs = 0;
+        const parseErrors = [];
+        let lineNumber = 0;
         // Use streaming reader to avoid "body already read" error
         const reader = response.body?.getReader();
         if (!reader) {
@@ -97,6 +104,7 @@ class QueryClient {
                 const lines = buffer.split('\n');
                 buffer = lines.pop() || '';
                 for (const line of lines) {
+                    lineNumber++;
                     if (!line.trim())
                         continue;
                     try {
@@ -120,12 +128,16 @@ class QueryClient {
                         }
                     }
                     catch (error) {
-                        console.error('Failed to parse NDJSON line:', error);
+                        parseErrors.push({
+                            line: lineNumber,
+                            error: error instanceof Error ? error.message : String(error),
+                        });
                     }
                 }
             }
             // Parse any remaining buffer
             if (buffer.trim()) {
+                lineNumber++;
                 try {
                     const chunk = JSON.parse(buffer);
                     if (columns === null && chunk.columns) {
@@ -144,13 +156,22 @@ class QueryClient {
                     }
                 }
                 catch (error) {
-                    console.error('Failed to parse final NDJSON line:', error);
+                    parseErrors.push({
+                        line: lineNumber,
+                        error: error instanceof Error ? error.message : String(error),
+                    });
                 }
             }
         }
         catch (error) {
             throw new Error(`NDJSON stream reading error: ${error}`);
         }
+        // Report parse errors if any occurred
+        if (parseErrors.length > 0) {
+            const errorDetails = parseErrors.slice(0, 3).map((e) => `line ${e.line}: ${e.error}`);
+            const moreErrors = parseErrors.length > 3 ? ` (and ${parseErrors.length - 3} more)` : '';
+            throw new Error(`NDJSON parsing failed for ${parseErrors.length} line(s): ${errorDetails.join('; ')}${moreErrors}`);
+        }
         // Return aggregated result
         return {
             data: allData,

package/sdk-extensions/QueryClient.test.ts

@@ -25,9 +25,10 @@ describe('QueryClient', () => {
       headers: { 'X-API-Key': 'test-key' },
     })
 
-    // Mock global fetch
+    // Mock global fetch (also set globalThis for SDK client compatibility)
     mockFetch = vi.fn()
     global.fetch = mockFetch
+    globalThis.fetch = mockFetch
 
     // Reset all mocks
     vi.clearAllMocks()

package/sdk-extensions/QueryClient.ts

@@ -81,6 +81,12 @@ export class QueryClient {
 
     const response = await executeCypherQuery(data)
 
+    // Check for errors in the response (network errors, etc.)
+    if ('error' in response && response.error) {
+      const error = response.error as Error
+      throw error instanceof Error ? error : new Error(String(error))
+    }
+
     // Check if this is a raw stream response (when parseAs: 'stream')
     if (options.mode === 'stream' && response.response) {
       const contentType = response.response.headers.get('content-type') || ''
@@ -142,6 +148,8 @@ export class QueryClient {
     let columns: string[] | null = null
     let totalRows = 0
     let executionTimeMs = 0
+    const parseErrors: { line: number; error: string }[] = []
+    let lineNumber = 0
 
     // Use streaming reader to avoid "body already read" error
     const reader = response.body?.getReader()
@@ -162,6 +170,7 @@ export class QueryClient {
         buffer = lines.pop() || ''
 
         for (const line of lines) {
+          lineNumber++
           if (!line.trim()) continue
 
           try {
@@ -186,13 +195,17 @@ export class QueryClient {
               executionTimeMs = Math.max(executionTimeMs, chunk.execution_time_ms)
             }
           } catch (error) {
-            console.error('Failed to parse NDJSON line:', error)
+            parseErrors.push({
+              line: lineNumber,
+              error: error instanceof Error ? error.message : String(error),
+            })
           }
         }
       }
 
       // Parse any remaining buffer
       if (buffer.trim()) {
+        lineNumber++
         try {
           const chunk = JSON.parse(buffer)
           if (columns === null && chunk.columns) {
@@ -209,13 +222,25 @@ export class QueryClient {
             executionTimeMs = Math.max(executionTimeMs, chunk.execution_time_ms)
           }
         } catch (error) {
-          console.error('Failed to parse final NDJSON line:', error)
+          parseErrors.push({
+            line: lineNumber,
+            error: error instanceof Error ? error.message : String(error),
+          })
         }
       }
     } catch (error) {
       throw new Error(`NDJSON stream reading error: ${error}`)
     }
 
+    // Report parse errors if any occurred
+    if (parseErrors.length > 0) {
+      const errorDetails = parseErrors.slice(0, 3).map((e) => `line ${e.line}: ${e.error}`)
+      const moreErrors = parseErrors.length > 3 ? ` (and ${parseErrors.length - 3} more)` : ''
+      throw new Error(
+        `NDJSON parsing failed for ${parseErrors.length} line(s): ${errorDetails.join('; ')}${moreErrors}`
+      )
+    }
+
     // Return aggregated result
     return {
       data: allData,