@robosystems/client 0.3.11 → 0.3.12

package/artifacts/OperationClient.js

@@ -88,6 +88,20 @@ class OperationClient {
                     this.scheduleCleanup(operationId, 5000);
                     resolve(result);
                 });
+                // Handle transport-level failures after the stream was initially
+                // opened (server drop, network blip exhausting retries). Without
+                // this, max_retries_exceeded closes the SSE client but leaves the
+                // monitorOperation promise pending until options.timeout fires.
+                sseClient.on('max_retries_exceeded', (err) => {
+                    if (timeoutHandle)
+                        clearTimeout(timeoutHandle);
+                    result = {
+                        success: false,
+                        error: (err && (err.message || err.error)) || 'SSE connection lost (max retries exceeded)',
+                    };
+                    this.scheduleCleanup(operationId, 5000);
+                    resolve(result);
+                });
             })
                 .catch((error) => {
                 if (timeoutHandle)

package/artifacts/OperationClient.ts

@@ -136,6 +136,21 @@ export class OperationClient {
             this.scheduleCleanup(operationId, 5000)
             resolve(result)
           })
+
+          // Handle transport-level failures after the stream was initially
+          // opened (server drop, network blip exhausting retries). Without
+          // this, max_retries_exceeded closes the SSE client but leaves the
+          // monitorOperation promise pending until options.timeout fires.
+          sseClient.on('max_retries_exceeded', (err) => {
+            if (timeoutHandle) clearTimeout(timeoutHandle)
+            result = {
+              success: false,
+              error:
+                (err && (err.message || err.error)) || 'SSE connection lost (max retries exceeded)',
+            }
+            this.scheduleCleanup(operationId, 5000)
+            resolve(result)
+          })
         })
         .catch((error) => {
           if (timeoutHandle) clearTimeout(timeoutHandle)

package/artifacts/SSEClient.js

@@ -38,18 +38,36 @@ class SSEClient {
             this.eventSource = new EventSource(url, {
                 withCredentials: this.config.credentials === 'include',
             });
+            let opened = false;
             const connectionTimeout = setTimeout(() => {
-                reject(new Error('Connection timeout'));
-                this.close();
+                if (!opened) {
+                    reject(new Error('SSE connection timed out before opening'));
+                    this.close();
+                }
             }, 10000);
             this.eventSource.onopen = () => {
+                opened = true;
                 clearTimeout(connectionTimeout);
                 this.reconnectAttempts = 0;
                 this.emit('connected', null);
                 resolve();
             };
             this.eventSource.onerror = (error) => {
-                clearTimeout(connectionTimeout);
+                if (!opened) {
+                    // Failed before the stream ever opened. Browser EventSource doesn't
+                    // expose the HTTP status, but a CLOSED readyState at this point
+                    // means a non-retryable response (commonly 401/403/404). Reject
+                    // the initial promise so callers surface a clear error instead of
+                    // waiting out the 10s connectionTimeout or spinning retries that
+                    // can never satisfy this promise.
+                    clearTimeout(connectionTimeout);
+                    const nonRetryable = this.eventSource?.readyState === EventSource.CLOSED;
+                    this.close();
+                    reject(new Error(nonRetryable
+                        ? 'SSE connection failed before open (likely auth/not-found; EventSource hides the HTTP status)'
+                        : 'SSE connection error before open'));
+                    return;
+                }
                 if (!this.closed) {
                     this.handleError(error, operationId, fromSequence);
                 }

package/artifacts/SSEClient.ts

@@ -80,12 +80,17 @@ export class SSEClient {
         withCredentials: this.config.credentials === 'include',
       } as any)
 
+      let opened = false
+
       const connectionTimeout = setTimeout(() => {
-        reject(new Error('Connection timeout'))
-        this.close()
+        if (!opened) {
+          reject(new Error('SSE connection timed out before opening'))
+          this.close()
+        }
       }, 10000)
 
       this.eventSource.onopen = () => {
+        opened = true
         clearTimeout(connectionTimeout)
         this.reconnectAttempts = 0
         this.emit('connected', null)
@@ -93,7 +98,25 @@ export class SSEClient {
       }
 
       this.eventSource.onerror = (error) => {
-        clearTimeout(connectionTimeout)
+        if (!opened) {
+          // Failed before the stream ever opened. Browser EventSource doesn't
+          // expose the HTTP status, but a CLOSED readyState at this point
+          // means a non-retryable response (commonly 401/403/404). Reject
+          // the initial promise so callers surface a clear error instead of
+          // waiting out the 10s connectionTimeout or spinning retries that
+          // can never satisfy this promise.
+          clearTimeout(connectionTimeout)
+          const nonRetryable = this.eventSource?.readyState === EventSource.CLOSED
+          this.close()
+          reject(
+            new Error(
+              nonRetryable
+                ? 'SSE connection failed before open (likely auth/not-found; EventSource hides the HTTP status)'
+                : 'SSE connection error before open'
+            )
+          )
+          return
+        }
         if (!this.closed) {
           this.handleError(error, operationId, fromSequence)
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@robosystems/client",
-  "version": "0.3.11",
+  "version": "0.3.12",
   "description": "TypeScript client library for RoboSystems Financial Knowledge Graph API",
   "main": "index.js",
   "types": "index.d.ts",

package/sdk/types.gen.d.ts

@@ -1507,7 +1507,7 @@ export type CreateEventHandlerRequest = {
     /**
      * Match Metadata Expression
      *
-     * JSONPath-style equality map, e.g. {"metadata.category": "payroll"}
+     * JSONPath-style equality map against event.metadata, e.g. {"category": "payroll"} or {"metadata.category": "payroll"}
      */
     match_metadata_expression?: {
         [key: string]: unknown;

package/sdk/types.gen.ts

@@ -1552,7 +1552,7 @@ export type CreateEventHandlerRequest = {
     /**
      * Match Metadata Expression
      *
-     * JSONPath-style equality map, e.g. {"metadata.category": "payroll"}
+     * JSONPath-style equality map against event.metadata, e.g. {"category": "payroll"} or {"metadata.category": "payroll"}
      */
     match_metadata_expression?: {
         [key: string]: unknown;

package/types.gen.d.ts

@@ -1507,7 +1507,7 @@ export type CreateEventHandlerRequest = {
     /**
      * Match Metadata Expression
      *
-     * JSONPath-style equality map, e.g. {"metadata.category": "payroll"}
+     * JSONPath-style equality map against event.metadata, e.g. {"category": "payroll"} or {"metadata.category": "payroll"}
      */
     match_metadata_expression?: {
         [key: string]: unknown;

package/types.gen.ts

@@ -1552,7 +1552,7 @@ export type CreateEventHandlerRequest = {
     /**
      * Match Metadata Expression
      *
-     * JSONPath-style equality map, e.g. {"metadata.category": "payroll"}
+     * JSONPath-style equality map against event.metadata, e.g. {"category": "payroll"} or {"metadata.category": "payroll"}
      */
     match_metadata_expression?: {
         [key: string]: unknown;