@flowcore/data-pump 0.21.0 → 0.21.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.21.1](https://github.com/flowcore-io/data-pump/compare/v0.21.0...v0.21.1) (2026-05-10)
+
+
+### Bug Fixes
+
+* **notifier:** resolve waitWebSocket promise on WS error to prevent pull-loop wedge ([#72](https://github.com/flowcore-io/data-pump/issues/72)) ([d76ee0f](https://github.com/flowcore-io/data-pump/commit/d76ee0f0b5431f6a250a46948ad0c39693135c9c))
+
 ## [0.21.0](https://github.com/flowcore-io/data-pump/compare/v0.20.0...v0.21.0) (2026-04-24)
 
 

package/esm/data-pump/notifier.d.ts

@@ -1,3 +1,4 @@
+import { NotificationClient } from "../deps/jsr.io/@flowcore/sdk/1.79.0/src/mod.js";
 import { NatsConnectionManager } from "./nats-connection.js";
 import type { FlowcoreDataPumpAuth, FlowcoreDataPumpDataSource, FlowcoreLogger } from "./types.js";
 export interface FlowcoreNotifierOptions {
@@ -27,4 +28,15 @@ export declare class FlowcoreNotifier {
     private waitWebSocket;
     private webSocketAuth;
 }
+/**
+ * Test seam for stubbing the NotificationClient constructor. Production code
+ * routes through `_internals.createNotificationClient`; tests replace this
+ * property with a fake factory to drive the subject manually. Not part of the
+ * public API.
+ */
+type NotificationClientFactory = (...args: any[]) => NotificationClient;
+export declare const _internals: {
+    createNotificationClient: NotificationClientFactory;
+};
+export {};
 //# sourceMappingURL=notifier.d.ts.map

package/esm/data-pump/notifier.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"notifier.d.ts","sourceRoot":"","sources":["../../src/data-pump/notifier.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,qBAAqB,EAAE,MAAM,sBAAsB,CAAA;AAC5D,OAAO,KAAK,EAAE,oBAAoB,EAAE,0BAA0B,EAAE,cAAc,EAAE,MAAM,YAAY,CAAA;AAKlG,MAAM,WAAW,uBAAuB;IACtC,UAAU,EAAE,0BAA0B,CAAA;IACtC,IAAI,EAAE,oBAAoB,CAAA;IAC1B,WAAW,CAAC,EAAE,MAAM,EAAE,CAAA;IACtB,qBAAqB,CAAC,EAAE,qBAAqB,CAAA;IAC7C,gBAAgB,CAAC,EAAE,MAAM,CAAA;IACzB,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,MAAM,CAAC,EAAE,cAAc,CAAA;IACvB,UAAU,CAAC,EAAE,OAAO,CAAA;IACpB,aAAa,CAAC,EAAE,OAAO,CAAA;CACxB;AAED,qBAAa,gBAAgB;IASf,OAAO,CAAC,QAAQ,CAAC,OAAO;IARpC,OAAO,CAAC,UAAU,CAAoB;IACtC,OAAO,CAAC,WAAW,CAAC,CAAuB;IAC3C,OAAO,CAAC,OAAO,CAAC,CAA4B;IAC5C,OAAO,CAAC,kBAAkB,CAAC,CAAoB;IAC/C,OAAO,CAAC,aAAa,CAAC,CAAY;IAElC,OAAO,CAAC,KAAK,CAAC,CAAK;gBAEU,OAAO,EAAE,uBAAuB;IAetD,IAAI,CAAC,MAAM,CAAC,EAAE,WAAW;YASlB,UAAU;YAUV,QAAQ;IAoCtB,OAAO,CAAC,gBAAgB;YAMV,aAAa;IAmC3B,OAAO,CAAC,aAAa;CAiBtB"}
+{"version":3,"file":"notifier.d.ts","sourceRoot":"","sources":["../../src/data-pump/notifier.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,kBAAkB,EAA0B,MAAM,gDAAgD,CAAA;AAG3G,OAAO,EAAE,qBAAqB,EAAE,MAAM,sBAAsB,CAAA;AAC5D,OAAO,KAAK,EAAE,oBAAoB,EAAE,0BAA0B,EAAE,cAAc,EAAE,MAAM,YAAY,CAAA;AAKlG,MAAM,WAAW,uBAAuB;IACtC,UAAU,EAAE,0BAA0B,CAAA;IACtC,IAAI,EAAE,oBAAoB,CAAA;IAC1B,WAAW,CAAC,EAAE,MAAM,EAAE,CAAA;IACtB,qBAAqB,CAAC,EAAE,qBAAqB,CAAA;IAC7C,gBAAgB,CAAC,EAAE,MAAM,CAAA;IACzB,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,MAAM,CAAC,EAAE,cAAc,CAAA;IACvB,UAAU,CAAC,EAAE,OAAO,CAAA;IACpB,aAAa,CAAC,EAAE,OAAO,CAAA;CACxB;AAED,qBAAa,gBAAgB;IASf,OAAO,CAAC,QAAQ,CAAC,OAAO;IARpC,OAAO,CAAC,UAAU,CAAoB;IACtC,OAAO,CAAC,WAAW,CAAC,CAAuB;IAC3C,OAAO,CAAC,OAAO,CAAC,CAA4B;IAC5C,OAAO,CAAC,kBAAkB,CAAC,CAAoB;IAC/C,OAAO,CAAC,aAAa,CAAC,CAAY;IAElC,OAAO,CAAC,KAAK,CAAC,CAAK;gBAEU,OAAO,EAAE,uBAAuB;IAetD,IAAI,CAAC,MAAM,CAAC,EAAE,WAAW;YASlB,UAAU;YAUV,QAAQ;IAoCtB,OAAO,CAAC,gBAAgB;YAMV,aAAa;IA2C3B,OAAO,CAAC,aAAa;CAiBtB;AAED;;;;;GAKG;AAEH,KAAK,yBAAyB,GAAG,CAAC,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,kBAAkB,CAAA;AACvE,eAAO,MAAM,UAAU,EAAE;IAAE,wBAAwB,EAAE,yBAAyB,CAAA;CAG7E,CAAA"}

package/esm/data-pump/notifier.js

@@ -113,12 +113,18 @@ export class FlowcoreNotifier {
     }
     async waitWebSocket(signal) {
         this.options.logger?.debug("Waiting for web socket");
+        const promise = new Promise((resolve) => {
+            this.eventResolver = resolve;
+        });
         this.subject = new Subject();
         this.subject.subscribe({
             next: this.onWebSocketEvent.bind(this),
-            error: (error) => this.options.logger?.error("Notification stream error:", { error }),
+            error: (error) => {
+                this.options.logger?.error("Notification stream error:", { error });
+                this.eventResolver?.();
+            },
         });
-        this.notificationClient = new NotificationClient(this.subject, this.webSocketAuth(), {
+        this.notificationClient = _internals.createNotificationClient(this.subject, this.webSocketAuth(), {
             tenant: this.dataSource.tenant,
             dataCore: this.dataSource.dataCore,
             flowType: this.dataSource.flowType,
@@ -126,16 +132,18 @@ export class FlowcoreNotifier {
             logger: this.options.logger ?? noOpLogger,
             reconnectInterval: 1000,
         });
-        await this.notificationClient.connect();
-        const promise = new Promise((resolve) => {
-            this.eventResolver = resolve;
-        });
-        clearTimeout(this.timer);
-        this.timer = setTimeout(() => this.eventResolver?.(), this.options.timeoutMs ?? DEFAULT_TIMEOUT_MS);
-        signal?.addEventListener("abort", () => this.eventResolver?.());
-        await promise;
-        this.eventResolver = undefined;
-        this.notificationClient.disconnect();
+        try {
+            await this.notificationClient.connect();
+            clearTimeout(this.timer);
+            this.timer = setTimeout(() => this.eventResolver?.(), this.options.timeoutMs ?? DEFAULT_TIMEOUT_MS);
+            signal?.addEventListener("abort", () => this.eventResolver?.());
+            await promise;
+        }
+        finally {
+            clearTimeout(this.timer);
+            this.eventResolver = undefined;
+            this.notificationClient.disconnect();
+        }
     }
     webSocketAuth() {
         if ("apiKey" in this.options.auth) {
@@ -155,3 +163,6 @@ export class FlowcoreNotifier {
         };
     }
 }
+export const _internals = {
+    createNotificationClient: (...args) => new NotificationClient(...args),
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flowcore/data-pump",
-  "version": "0.21.0",
+  "version": "0.21.1",
   "description": "Flowcore Data Pump",
   "homepage": "https://github.com/flowcore-io/flowcore-data-pump#readme",
   "repository": {

package/script/data-pump/notifier.d.ts

@@ -1,3 +1,4 @@
+import { NotificationClient } from "../deps/jsr.io/@flowcore/sdk/1.79.0/src/mod.js";
 import { NatsConnectionManager } from "./nats-connection.js";
 import type { FlowcoreDataPumpAuth, FlowcoreDataPumpDataSource, FlowcoreLogger } from "./types.js";
 export interface FlowcoreNotifierOptions {
@@ -27,4 +28,15 @@ export declare class FlowcoreNotifier {
     private waitWebSocket;
     private webSocketAuth;
 }
+/**
+ * Test seam for stubbing the NotificationClient constructor. Production code
+ * routes through `_internals.createNotificationClient`; tests replace this
+ * property with a fake factory to drive the subject manually. Not part of the
+ * public API.
+ */
+type NotificationClientFactory = (...args: any[]) => NotificationClient;
+export declare const _internals: {
+    createNotificationClient: NotificationClientFactory;
+};
+export {};
 //# sourceMappingURL=notifier.d.ts.map

package/script/data-pump/notifier.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"notifier.d.ts","sourceRoot":"","sources":["../../src/data-pump/notifier.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,qBAAqB,EAAE,MAAM,sBAAsB,CAAA;AAC5D,OAAO,KAAK,EAAE,oBAAoB,EAAE,0BAA0B,EAAE,cAAc,EAAE,MAAM,YAAY,CAAA;AAKlG,MAAM,WAAW,uBAAuB;IACtC,UAAU,EAAE,0BAA0B,CAAA;IACtC,IAAI,EAAE,oBAAoB,CAAA;IAC1B,WAAW,CAAC,EAAE,MAAM,EAAE,CAAA;IACtB,qBAAqB,CAAC,EAAE,qBAAqB,CAAA;IAC7C,gBAAgB,CAAC,EAAE,MAAM,CAAA;IACzB,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,MAAM,CAAC,EAAE,cAAc,CAAA;IACvB,UAAU,CAAC,EAAE,OAAO,CAAA;IACpB,aAAa,CAAC,EAAE,OAAO,CAAA;CACxB;AAED,qBAAa,gBAAgB;IASf,OAAO,CAAC,QAAQ,CAAC,OAAO;IARpC,OAAO,CAAC,UAAU,CAAoB;IACtC,OAAO,CAAC,WAAW,CAAC,CAAuB;IAC3C,OAAO,CAAC,OAAO,CAAC,CAA4B;IAC5C,OAAO,CAAC,kBAAkB,CAAC,CAAoB;IAC/C,OAAO,CAAC,aAAa,CAAC,CAAY;IAElC,OAAO,CAAC,KAAK,CAAC,CAAK;gBAEU,OAAO,EAAE,uBAAuB;IAetD,IAAI,CAAC,MAAM,CAAC,EAAE,WAAW;YASlB,UAAU;YAUV,QAAQ;IAoCtB,OAAO,CAAC,gBAAgB;YAMV,aAAa;IAmC3B,OAAO,CAAC,aAAa;CAiBtB"}
+{"version":3,"file":"notifier.d.ts","sourceRoot":"","sources":["../../src/data-pump/notifier.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,kBAAkB,EAA0B,MAAM,gDAAgD,CAAA;AAG3G,OAAO,EAAE,qBAAqB,EAAE,MAAM,sBAAsB,CAAA;AAC5D,OAAO,KAAK,EAAE,oBAAoB,EAAE,0BAA0B,EAAE,cAAc,EAAE,MAAM,YAAY,CAAA;AAKlG,MAAM,WAAW,uBAAuB;IACtC,UAAU,EAAE,0BAA0B,CAAA;IACtC,IAAI,EAAE,oBAAoB,CAAA;IAC1B,WAAW,CAAC,EAAE,MAAM,EAAE,CAAA;IACtB,qBAAqB,CAAC,EAAE,qBAAqB,CAAA;IAC7C,gBAAgB,CAAC,EAAE,MAAM,CAAA;IACzB,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,MAAM,CAAC,EAAE,cAAc,CAAA;IACvB,UAAU,CAAC,EAAE,OAAO,CAAA;IACpB,aAAa,CAAC,EAAE,OAAO,CAAA;CACxB;AAED,qBAAa,gBAAgB;IASf,OAAO,CAAC,QAAQ,CAAC,OAAO;IARpC,OAAO,CAAC,UAAU,CAAoB;IACtC,OAAO,CAAC,WAAW,CAAC,CAAuB;IAC3C,OAAO,CAAC,OAAO,CAAC,CAA4B;IAC5C,OAAO,CAAC,kBAAkB,CAAC,CAAoB;IAC/C,OAAO,CAAC,aAAa,CAAC,CAAY;IAElC,OAAO,CAAC,KAAK,CAAC,CAAK;gBAEU,OAAO,EAAE,uBAAuB;IAetD,IAAI,CAAC,MAAM,CAAC,EAAE,WAAW;YASlB,UAAU;YAUV,QAAQ;IAoCtB,OAAO,CAAC,gBAAgB;YAMV,aAAa;IA2C3B,OAAO,CAAC,aAAa;CAiBtB;AAED;;;;;GAKG;AAEH,KAAK,yBAAyB,GAAG,CAAC,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,kBAAkB,CAAA;AACvE,eAAO,MAAM,UAAU,EAAE;IAAE,wBAAwB,EAAE,yBAAyB,CAAA;CAG7E,CAAA"}

package/script/data-pump/notifier.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.FlowcoreNotifier = void 0;
+exports._internals = exports.FlowcoreNotifier = void 0;
 const mod_js_1 = require("../deps/jsr.io/@flowcore/sdk/1.79.0/src/mod.js");
 const rxjs_1 = require("rxjs");
 const data_source_js_1 = require("./data-source.js");
@@ -116,12 +116,18 @@ class FlowcoreNotifier {
     }
     async waitWebSocket(signal) {
         this.options.logger?.debug("Waiting for web socket");
+        const promise = new Promise((resolve) => {
+            this.eventResolver = resolve;
+        });
         this.subject = new rxjs_1.Subject();
         this.subject.subscribe({
             next: this.onWebSocketEvent.bind(this),
-            error: (error) => this.options.logger?.error("Notification stream error:", { error }),
+            error: (error) => {
+                this.options.logger?.error("Notification stream error:", { error });
+                this.eventResolver?.();
+            },
         });
-        this.notificationClient = new mod_js_1.NotificationClient(this.subject, this.webSocketAuth(), {
+        this.notificationClient = exports._internals.createNotificationClient(this.subject, this.webSocketAuth(), {
             tenant: this.dataSource.tenant,
             dataCore: this.dataSource.dataCore,
             flowType: this.dataSource.flowType,
@@ -129,16 +135,18 @@ class FlowcoreNotifier {
             logger: this.options.logger ?? no_op_logger_js_1.noOpLogger,
             reconnectInterval: 1000,
         });
-        await this.notificationClient.connect();
-        const promise = new Promise((resolve) => {
-            this.eventResolver = resolve;
-        });
-        clearTimeout(this.timer);
-        this.timer = setTimeout(() => this.eventResolver?.(), this.options.timeoutMs ?? DEFAULT_TIMEOUT_MS);
-        signal?.addEventListener("abort", () => this.eventResolver?.());
-        await promise;
-        this.eventResolver = undefined;
-        this.notificationClient.disconnect();
+        try {
+            await this.notificationClient.connect();
+            clearTimeout(this.timer);
+            this.timer = setTimeout(() => this.eventResolver?.(), this.options.timeoutMs ?? DEFAULT_TIMEOUT_MS);
+            signal?.addEventListener("abort", () => this.eventResolver?.());
+            await promise;
+        }
+        finally {
+            clearTimeout(this.timer);
+            this.eventResolver = undefined;
+            this.notificationClient.disconnect();
+        }
     }
     webSocketAuth() {
         if ("apiKey" in this.options.auth) {
@@ -159,3 +167,6 @@ class FlowcoreNotifier {
     }
 }
 exports.FlowcoreNotifier = FlowcoreNotifier;
+exports._internals = {
+    createNotificationClient: (...args) => new mod_js_1.NotificationClient(...args),
+};