@forgecart/sdk 1.202607281310.0 → 1.202608041455.0

package/dist/client.generated.js

@@ -230,6 +230,16 @@ class BaseGraphQLClient {
         return new QueryObservable((subscriber) => {
             const client = this.getWsClient();
             let disposed = false;
+            // Delivery latch: once ANY sink callback has fired, the wire has
+            // spoken and a socket close can no longer mean "delivery unknown".
+            // Without it, a close event dispatched inside the delivery window
+            // (observed: between the error-result notify and the subscriber
+            // settling — CD runs 30845909288/30852033370/30855605921, where the
+            // server's NOT_VERIFIED answer was on the wire yet the consumer
+            // received WS_MUTATION_DELIVERY_INTERRUPTED, 3/3 deterministic)
+            // retro-fails an ALREADY-ANSWERED mutation. `settled` flips before
+            // any handler logic so every interleave is covered.
+            let settled = false;
             const offConnectionEvents = [];
             const cleanup = client.subscribe({
                 query: document,
@@ -237,6 +247,7 @@ class BaseGraphQLClient {
                 extensions: this.getPerOperationExtensions(),
             }, {
                 next: (result) => {
+                    settled = true;
                     if (disposed)
                         return;
                     if (result.errors && result.errors.length > 0) {
@@ -247,8 +258,12 @@ class BaseGraphQLClient {
                             })),
                         };
                         const error = new ClientError(response, result.errors[0].message);
-                        this.notifyErrorHandler(error);
+                        // Terminal delivery FIRST, side-channel notification after:
+                        // the notify hook runs arbitrary app code (toasts, logging,
+                        // change detection), and anything it lets re-enter must find
+                        // this operation already settled, never still interruptible.
                         subscriber.error(error);
+                        this.notifyErrorHandler(error);
                         return;
                     }
                     if (result.data) {
@@ -257,11 +272,13 @@ class BaseGraphQLClient {
                     }
                 },
                 error: (err) => {
+                    settled = true;
                     if (disposed)
                         return;
                     subscriber.error(this.toStreamError(err));
                 },
                 complete: () => {
+                    settled = true;
                     if (disposed)
                         return;
                     subscriber.complete();
@@ -271,7 +288,7 @@ class BaseGraphQLClient {
                 let armed = WS_CONNECTION_STATE.get(client)?.connected ?? false;
                 offConnectionEvents.push(client.on("connected", () => { armed = true; }));
                 offConnectionEvents.push(client.on("closed", () => {
-                    if (disposed || !armed)
+                    if (disposed || settled || !armed)
                         return;
                     const message = "WS_MUTATION_DELIVERY_INTERRUPTED";
                     // subscriber.error tears the subscription down synchronously,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forgecart/sdk",
-  "version": "1.202607281310.0",
+  "version": "1.202608041455.0",
   "description": "ForgeCart Client SDK",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",

package/src/client.generated.ts

@@ -334,6 +334,16 @@ export class BaseGraphQLClient {
     return new QueryObservable<T>((subscriber) => {
       const client = this.getWsClient();
       let disposed = false;
+      // Delivery latch: once ANY sink callback has fired, the wire has
+      // spoken and a socket close can no longer mean "delivery unknown".
+      // Without it, a close event dispatched inside the delivery window
+      // (observed: between the error-result notify and the subscriber
+      // settling — CD runs 30845909288/30852033370/30855605921, where the
+      // server's NOT_VERIFIED answer was on the wire yet the consumer
+      // received WS_MUTATION_DELIVERY_INTERRUPTED, 3/3 deterministic)
+      // retro-fails an ALREADY-ANSWERED mutation. `settled` flips before
+      // any handler logic so every interleave is covered.
+      let settled = false;
       const offConnectionEvents: (() => void)[] = [];
 
       const cleanup = client.subscribe<Record<string, unknown>>(
@@ -344,6 +354,7 @@ export class BaseGraphQLClient {
         },
         {
           next: (result) => {
+            settled = true;
             if (disposed) return;
             if (result.errors && result.errors.length > 0) {
               const response: GraphQLErrorResponse = {
@@ -353,8 +364,12 @@ export class BaseGraphQLClient {
                 })),
               };
               const error = new ClientError(response, result.errors[0].message);
-              this.notifyErrorHandler(error);
+              // Terminal delivery FIRST, side-channel notification after:
+              // the notify hook runs arbitrary app code (toasts, logging,
+              // change detection), and anything it lets re-enter must find
+              // this operation already settled, never still interruptible.
               subscriber.error(error);
+              this.notifyErrorHandler(error);
               return;
             }
             if (result.data) {
@@ -363,10 +378,12 @@ export class BaseGraphQLClient {
             }
           },
           error: (err) => {
+            settled = true;
             if (disposed) return;
             subscriber.error(this.toStreamError(err));
           },
           complete: () => {
+            settled = true;
             if (disposed) return;
             subscriber.complete();
           },
@@ -377,7 +394,7 @@ export class BaseGraphQLClient {
         let armed = WS_CONNECTION_STATE.get(client)?.connected ?? false;
         offConnectionEvents.push(client.on("connected", () => { armed = true; }));
         offConnectionEvents.push(client.on("closed", () => {
-          if (disposed || !armed) return;
+          if (disposed || settled || !armed) return;
           const message = "WS_MUTATION_DELIVERY_INTERRUPTED";
           // subscriber.error tears the subscription down synchronously,
           // disposing the pending graphql-ws operation BEFORE its retry