@constructive-io/graphql-server 5.16.10 → 5.16.12

package/esm/server.js

@@ -225,24 +225,39 @@ class Server {
         }
         this.listenClient = client;
         this.listenRelease = release;
+        // Attached before any query is issued: an 'error' event on a Client with no
+        // listener is rethrown by Node and takes the whole process down.
+        client.on('error', (e) => {
+            this.dropListener(client, 'Error with database notify listener', e);
+        });
         client.on('notification', ({ channel, payload }) => {
             if (channel === 'schema:update' && payload) {
                 log.info('schema:update', payload);
                 this.flush(payload);
             }
         });
-        client.query('LISTEN "schema:update"');
-        client.on('error', (e) => {
-            if (this.shuttingDown) {
-                release();
-                return;
-            }
-            this.error('Error with database notify listener', e);
-            release();
-            this.addEventListener();
+        client.query('LISTEN "schema:update"').catch((e) => {
+            this.dropListener(client, 'Failed to LISTEN for schema:update', e);
         });
         this.log('connected and listening for changes...');
     }
+    dropListener(client, message, err) {
+        // Another path (shutdown, an earlier failure) already tore this client down.
+        if (this.listenClient !== client)
+            return;
+        const release = this.listenRelease;
+        this.listenClient = null;
+        this.listenRelease = null;
+        client.removeAllListeners('notification');
+        client.removeAllListeners('error');
+        if (this.shuttingDown) {
+            release?.();
+            return;
+        }
+        this.error(message, err);
+        release?.();
+        this.addEventListener();
+    }
     async removeEventListener() {
         if (!this.listenClient || !this.listenRelease) {
             return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@constructive-io/graphql-server",
-  "version": "5.16.10",
+  "version": "5.16.12",
   "author": "Constructive <developers@constructive.io>",
   "description": "Constructive GraphQL Server",
   "main": "index.js",
@@ -44,18 +44,18 @@
     "@agentic-kit/ollama": "2.13.0",
     "@constructive-io/csrf": "^0.27.0",
     "@constructive-io/errors": "^0.10.0",
-    "@constructive-io/express-context": "^0.24.4",
-    "@constructive-io/graphql-env": "^3.29.3",
-    "@constructive-io/graphql-types": "^3.28.3",
+    "@constructive-io/express-context": "^0.24.5",
+    "@constructive-io/graphql-env": "^3.29.4",
+    "@constructive-io/graphql-types": "^3.28.4",
     "@constructive-io/llm-env": "^0.13.0",
     "@constructive-io/query-builder": "^3.10.4",
     "@constructive-io/s3-utils": "^2.30.0",
     "@constructive-io/url-domains": "^2.29.0",
     "@graphile-contrib/pg-many-to-many": "2.0.0-rc.2",
-    "@pgpmjs/env": "^2.41.3",
+    "@pgpmjs/env": "^2.41.4",
     "@pgpmjs/logger": "^2.24.1",
-    "@pgpmjs/server-utils": "^3.25.4",
-    "@pgpmjs/types": "^2.50.3",
+    "@pgpmjs/server-utils": "^3.25.5",
+    "@pgpmjs/types": "^2.50.4",
     "cors": "^2.8.6",
     "deepmerge": "^4.3.1",
     "express": "^5.2.1",
@@ -64,17 +64,17 @@
     "grafserv": "1.0.1",
     "graphile-build": "5.1.1",
     "graphile-build-pg": "5.1.3",
-    "graphile-cache": "^4.10.6",
+    "graphile-cache": "^4.10.7",
     "graphile-config": "1.1.0",
-    "graphile-function-bindings": "^1.11.9",
-    "graphile-settings": "^6.15.0",
+    "graphile-function-bindings": "^1.11.10",
+    "graphile-settings": "^6.15.1",
     "graphile-utils": "5.0.3",
     "graphql": "16.13.0",
     "graphql-upload": "^13.0.0",
     "lru-cache": "^11.2.7",
     "pg": "^8.21.0",
-    "pg-cache": "^3.25.4",
-    "pg-env": "^1.29.1",
+    "pg-cache": "^3.25.5",
+    "pg-env": "^1.29.2",
     "pg-query-context": "^2.29.1",
     "pg-sql2": "5.0.1",
     "postgraphile": "5.1.4",
@@ -90,11 +90,11 @@
     "@types/request-ip": "^0.0.41",
     "@types/supertest": "^7.2.1",
     "cookie-parser": "^1.4.7",
-    "graphile-test": "5.11.8",
+    "graphile-test": "5.11.9",
     "makage": "^0.3.0",
     "nodemon": "^3.1.14",
     "supertest": "^7.2.2",
     "ts-node": "^10.9.2"
   },
-  "gitHead": "d0d796b7eb0d16c8c19405a49c0ca62ce3df117b"
+  "gitHead": "eacbe10ce544848c93a4c17fc8a8474d1a788a18"
 }

package/server.d.ts

@@ -39,6 +39,7 @@ declare class Server {
     getPool(): Pool;
     addEventListener(): void;
     listenForChanges(err: Error | null, client: PoolClient, release: () => void): void;
+    private dropListener;
     removeEventListener(): Promise<void>;
     close(opts?: {
         closeCaches?: boolean;

package/server.js

@@ -232,24 +232,39 @@ class Server {
         }
         this.listenClient = client;
         this.listenRelease = release;
+        // Attached before any query is issued: an 'error' event on a Client with no
+        // listener is rethrown by Node and takes the whole process down.
+        client.on('error', (e) => {
+            this.dropListener(client, 'Error with database notify listener', e);
+        });
         client.on('notification', ({ channel, payload }) => {
             if (channel === 'schema:update' && payload) {
                 log.info('schema:update', payload);
                 this.flush(payload);
             }
         });
-        client.query('LISTEN "schema:update"');
-        client.on('error', (e) => {
-            if (this.shuttingDown) {
-                release();
-                return;
-            }
-            this.error('Error with database notify listener', e);
-            release();
-            this.addEventListener();
+        client.query('LISTEN "schema:update"').catch((e) => {
+            this.dropListener(client, 'Failed to LISTEN for schema:update', e);
         });
         this.log('connected and listening for changes...');
     }
+    dropListener(client, message, err) {
+        // Another path (shutdown, an earlier failure) already tore this client down.
+        if (this.listenClient !== client)
+            return;
+        const release = this.listenRelease;
+        this.listenClient = null;
+        this.listenRelease = null;
+        client.removeAllListeners('notification');
+        client.removeAllListeners('error');
+        if (this.shuttingDown) {
+            release?.();
+            return;
+        }
+        this.error(message, err);
+        release?.();
+        this.addEventListener();
+    }
     async removeEventListener() {
         if (!this.listenClient || !this.listenRelease) {
             return;