@constructive-io/knative-job-worker 0.7.15 → 0.8.0

package/CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [0.8.0](https://github.com/constructive-io/jobs/compare/@constructive-io/knative-job-worker@0.7.16...@constructive-io/knative-job-worker@0.8.0) (2026-01-18)
+
+**Note:** Version bump only for package @constructive-io/knative-job-worker
+
+## [0.7.16](https://github.com/constructive-io/jobs/compare/@constructive-io/knative-job-worker@0.7.15...@constructive-io/knative-job-worker@0.7.16) (2026-01-18)
+
+**Note:** Version bump only for package @constructive-io/knative-job-worker
+
 ## [0.7.15](https://github.com/constructive-io/jobs/compare/@constructive-io/knative-job-worker@0.7.14...@constructive-io/knative-job-worker@0.7.15) (2026-01-14)
 
 **Note:** Version bump only for package @constructive-io/knative-job-worker

package/dist/index.d.ts

@@ -1,5 +1,5 @@
 import type { PgClientLike } from '@constructive-io/job-utils';
-import type { Pool } from 'pg';
+import type { Pool, PoolClient } from 'pg';
 export interface JobRow {
     id: number | string;
     task_identifier: string;
@@ -13,6 +13,9 @@ export default class Worker {
     doNextTimer?: NodeJS.Timeout;
     pgPool: Pool;
     _initialized?: boolean;
+    listenClient?: PoolClient;
+    listenRelease?: () => void;
+    stopped?: boolean;
     constructor({ tasks, idleDelay, pgPool, workerId }: {
         tasks: string[];
         idleDelay?: number;
@@ -37,5 +40,6 @@ export default class Worker {
     doWork(job: JobRow): Promise<void>;
     doNext(client: PgClientLike): Promise<void>;
     listen(): void;
+    stop(): Promise<void>;
 }
 export { Worker };

package/dist/index.js

@@ -49,6 +49,9 @@ class Worker {
     doNextTimer;
     pgPool;
     _initialized;
+    listenClient;
+    listenRelease;
+    stopped;
     constructor({ tasks, idleDelay = 15000, pgPool = job_pg_1.default.getPool(), workerId = 'worker-0' }) {
         /*
          * idleDelay: This is how long to wait between polling for jobs.
@@ -114,6 +117,8 @@ class Worker {
         });
     }
     async doNext(client) {
+        if (this.stopped)
+            return;
         if (!this._initialized) {
             return await this.initialize(client);
         }
@@ -130,7 +135,9 @@ class Worker {
                     : this.supportedTaskNames
             }));
             if (!job || !job.id) {
-                this.doNextTimer = setTimeout(() => this.doNext(client), this.idleDelay);
+                if (!this.stopped) {
+                    this.doNextTimer = setTimeout(() => this.doNext(client), this.idleDelay);
+                }
                 return;
             }
             const start = process.hrtime();
@@ -155,13 +162,20 @@ class Worker {
             catch (fatalError) {
                 await this.handleFatalError(client, { err, fatalError, jobId });
             }
-            return this.doNext(client);
+            if (!this.stopped) {
+                return this.doNext(client);
+            }
+            return;
         }
         catch (err) {
-            this.doNextTimer = setTimeout(() => this.doNext(client), this.idleDelay);
+            if (!this.stopped) {
+                this.doNextTimer = setTimeout(() => this.doNext(client), this.idleDelay);
+            }
         }
     }
     listen() {
+        if (this.stopped)
+            return;
         const listenForChanges = (err, client, release) => {
             if (err) {
                 log.error('Error connecting with notify listener', err);
@@ -170,9 +184,17 @@ class Worker {
                 }
                 // Try again in 5 seconds
                 // should this really be done in the node process?
-                setTimeout(this.listen, 5000);
+                if (!this.stopped) {
+                    setTimeout(this.listen, 5000);
+                }
+                return;
+            }
+            if (this.stopped) {
+                release();
                 return;
             }
+            this.listenClient = client;
+            this.listenRelease = release;
             client.on('notification', () => {
                 if (this.doNextTimer) {
                     // Must be idle, do something!
@@ -181,18 +203,46 @@ class Worker {
             });
             client.query('LISTEN "jobs:insert"');
             client.on('error', (e) => {
+                if (this.stopped) {
+                    release();
+                    return;
+                }
                 log.error('Error with database notify listener', e);
                 if (e instanceof Error && e.stack) {
                     log.debug(e.stack);
                 }
                 release();
-                this.listen();
+                if (!this.stopped) {
+                    this.listen();
+                }
             });
             log.info(`${this.workerId} connected and looking for jobs...`);
             this.doNext(client);
         };
         this.pgPool.connect(listenForChanges);
     }
+    async stop() {
+        this.stopped = true;
+        if (this.doNextTimer) {
+            clearTimeout(this.doNextTimer);
+            this.doNextTimer = undefined;
+        }
+        const client = this.listenClient;
+        const release = this.listenRelease;
+        this.listenClient = undefined;
+        this.listenRelease = undefined;
+        if (client && release) {
+            client.removeAllListeners('notification');
+            client.removeAllListeners('error');
+            try {
+                await client.query('UNLISTEN "jobs:insert"');
+            }
+            catch {
+                // Ignore listener cleanup errors during shutdown.
+            }
+            release();
+        }
+    }
 }
 exports.default = Worker;
 exports.Worker = Worker;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@constructive-io/knative-job-worker",
-  "version": "0.7.15",
+  "version": "0.8.0",
   "description": "knative job worker",
   "author": "Constructive <developers@constructive.io>",
   "homepage": "https://github.com/constructive-io/jobs/tree/master/packages/knative-job-worker#readme",
@@ -32,14 +32,14 @@
     "url": "https://github.com/constructive-io/jobs/issues"
   },
   "dependencies": {
-    "@constructive-io/job-pg": "^0.3.20",
-    "@constructive-io/job-utils": "^0.5.15",
-    "@pgpmjs/logger": "^1.3.7",
+    "@constructive-io/job-pg": "^0.4.0",
+    "@constructive-io/job-utils": "^0.6.0",
+    "@pgpmjs/logger": "^1.4.0",
     "pg": "8.16.3",
     "request": "2.88.2"
   },
   "devDependencies": {
-    "pgsql-test": "^2.24.15"
+    "pgsql-test": "^2.25.0"
   },
-  "gitHead": "b9719cd48487af70e6e065510197c9d2d6b331b6"
+  "gitHead": "481b3a50b4eec2da6b376c4cd1868065e1e28edb"
 }

package/src/index.ts

@@ -21,6 +21,9 @@ export default class Worker {
   doNextTimer?: NodeJS.Timeout;
   pgPool: Pool;
   _initialized?: boolean;
+  listenClient?: PoolClient;
+  listenRelease?: () => void;
+  stopped?: boolean;
 
   constructor({
     tasks,
@@ -118,6 +121,7 @@ export default class Worker {
     });
   }
   async doNext(client: PgClientLike): Promise<void> {
+    if (this.stopped) return;
     if (!this._initialized) {
       return await this.initialize(client);
     }
@@ -136,10 +140,12 @@ export default class Worker {
       })) as JobRow | undefined;
 
       if (!job || !job.id) {
-        this.doNextTimer = setTimeout(
-          () => this.doNext(client),
-          this.idleDelay
-        );
+        if (!this.stopped) {
+          this.doNextTimer = setTimeout(
+            () => this.doNext(client),
+            this.idleDelay
+          );
+        }
         return;
       }
       const start = process.hrtime();
@@ -164,12 +170,21 @@ export default class Worker {
       } catch (fatalError: unknown) {
         await this.handleFatalError(client, { err, fatalError, jobId });
       }
-      return this.doNext(client);
+      if (!this.stopped) {
+        return this.doNext(client);
+      }
+      return;
     } catch (err: unknown) {
-      this.doNextTimer = setTimeout(() => this.doNext(client), this.idleDelay);
+      if (!this.stopped) {
+        this.doNextTimer = setTimeout(
+          () => this.doNext(client),
+          this.idleDelay
+        );
+      }
     }
   }
   listen() {
+    if (this.stopped) return;
     const listenForChanges = (
       err: Error | null,
       client: PoolClient,
@@ -182,9 +197,17 @@ export default class Worker {
         }
         // Try again in 5 seconds
         // should this really be done in the node process?
-        setTimeout(this.listen, 5000);
+        if (!this.stopped) {
+          setTimeout(this.listen, 5000);
+        }
         return;
       }
+      if (this.stopped) {
+        release();
+        return;
+      }
+      this.listenClient = client;
+      this.listenRelease = release;
       client.on('notification', () => {
         if (this.doNextTimer) {
           // Must be idle, do something!
@@ -193,18 +216,47 @@ export default class Worker {
       });
       client.query('LISTEN "jobs:insert"');
       client.on('error', (e: unknown) => {
+        if (this.stopped) {
+          release();
+          return;
+        }
         log.error('Error with database notify listener', e);
         if (e instanceof Error && e.stack) {
           log.debug(e.stack);
         }
         release();
-        this.listen();
+        if (!this.stopped) {
+          this.listen();
+        }
       });
       log.info(`${this.workerId} connected and looking for jobs...`);
       this.doNext(client);
     };
     this.pgPool.connect(listenForChanges);
   }
+
+  async stop(): Promise<void> {
+    this.stopped = true;
+    if (this.doNextTimer) {
+      clearTimeout(this.doNextTimer);
+      this.doNextTimer = undefined;
+    }
+    const client = this.listenClient;
+    const release = this.listenRelease;
+    this.listenClient = undefined;
+    this.listenRelease = undefined;
+
+    if (client && release) {
+      client.removeAllListeners('notification');
+      client.removeAllListeners('error');
+      try {
+        await client.query('UNLISTEN "jobs:insert"');
+      } catch {
+        // Ignore listener cleanup errors during shutdown.
+      }
+      release();
+    }
+  }
 }
 
 export { Worker };