@constructive-io/graphql-server 5.23.0 → 5.24.0

package/esm/plugins/error-events-plugin.js

@@ -5,7 +5,6 @@ import { escapeIdentifier } from 'pg';
 import { withPgClient } from 'pg-query-context';
 import { normalizeError } from '../middleware/mask-error';
 const log = new Logger('error-events');
-export const GRAPHQL_ERROR_EVENT = 'graphql.error';
 const getExpressRequest = (requestContext) => requestContext?.expressv4?.req;
 /**
  * The first structured, public-classified registry code among the errors.
@@ -20,23 +19,31 @@ const refusalCode = (errors) => {
     }
     return undefined;
 };
-export const recordEventSql = (events) => `SELECT ${escapeIdentifier(events.privateSchemaName)}.${escapeIdentifier(events.recordEvent)}($1, $2::uuid, $3::jsonb)`;
+export const recordErrorSql = (events) => {
+    if (!events.recordError) {
+        throw new Error(`events module ${events.privateSchemaName} has no record_error function`);
+    }
+    return `SELECT ${escapeIdentifier(events.privateSchemaName)}.${escapeIdentifier(events.recordError)}($1, $2::uuid, $3::jsonb)`;
+};
 /**
- * Records `graphql.error` when an authenticated mutation is refused with a
- * structured registry code. The refusal rolled back the mutation's own
- * transaction, so the event is written afterwards in a fresh transaction under
- * the same request claims, via the tenant's events module `record_event`.
+ * Records a refused authenticated mutation as an event named after the error
+ * code raised by `errors.raise_error`. The refusal rolled back the mutation's
+ * own transaction, so the event is written afterwards in a fresh transaction
+ * under the same request claims, via the tenant's events module
+ * `record_error`, which classifies the code as an error that earns no ladder
+ * progress.
  *
- * The server carries no policy about what a code means: the database reads
- * `payload->>'code'` (e.g. PRINCIPAL_CHILD_WIDENS demoting a principal on the
- * trust ladder). Endpoints without an events module record nothing.
- * Unauthenticated requests are never recorded, so anonymous traffic cannot
- * drive writes. The client response is never altered.
+ * The server carries no policy: the code is the event, and the database decides
+ * what it means — a ladder's `revoked_by` names the code directly (e.g.
+ * PRINCIPAL_CHILD_WIDENS demoting a principal on the trust ladder). Endpoints
+ * without an events module record nothing. Unauthenticated requests are never
+ * recorded, so anonymous traffic cannot drive writes. The client response is
+ * never altered.
  */
 export const createErrorEventsPlugin = (pool) => ({
     name: 'ErrorEventsPlugin',
     version: '0.0.0',
-    description: 'Records graphql.error through the tenant events module when an authenticated mutation is refused.',
+    description: 'Records a refused authenticated mutation as its error code through the tenant events module.',
     grafast: {
         middleware: {
             async execute(next, event) {
@@ -60,14 +67,10 @@ export const createErrorEventsPlugin = (pool) => ({
                     if (!events || !pgSettings)
                         return result;
                     const operation = args.operationName ?? getOperationAST(args.document)?.name?.value ?? null;
-                    await withPgClient(pool, pgSettings, (client) => client.query(recordEventSql(events), [
-                        GRAPHQL_ERROR_EVENT,
-                        actorId,
-                        JSON.stringify({ code, operation })
-                    ]));
+                    await withPgClient(pool, pgSettings, (client) => client.query(recordErrorSql(events), [code, actorId, JSON.stringify({ operation })]));
                 }
                 catch (err) {
-                    log.error(`${label} failed to record ${GRAPHQL_ERROR_EVENT} (${code}) for ${actorId}: ${err instanceof Error ? err.message : String(err)}`);
+                    log.error(`${label} failed to record refusal ${code} for ${actorId}: ${err instanceof Error ? err.message : String(err)}`);
                 }
                 return result;
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@constructive-io/graphql-server",
-  "version": "5.23.0",
+  "version": "5.24.0",
   "author": "Constructive <developers@constructive.io>",
   "description": "Constructive GraphQL Server",
   "main": "index.js",
@@ -44,7 +44,7 @@
     "@agentic-kit/ollama": "2.14.0",
     "@constructive-io/csrf": "^0.29.1",
     "@constructive-io/errors": "^0.13.0",
-    "@constructive-io/express-context": "^0.29.0",
+    "@constructive-io/express-context": "^0.30.0",
     "@constructive-io/graphql-env": "^3.32.0",
     "@constructive-io/graphql-types": "^3.31.0",
     "@constructive-io/llm-env": "^0.14.1",
@@ -67,7 +67,7 @@
     "graphile-cache": "^4.12.4",
     "graphile-config": "1.1.0",
     "graphile-function-bindings": "^1.14.5",
-    "graphile-settings": "^6.21.6",
+    "graphile-settings": "^6.21.7",
     "graphile-utils": "5.0.3",
     "graphql": "16.13.0",
     "graphql-upload": "^13.0.0",
@@ -96,5 +96,5 @@
     "supertest": "^7.2.2",
     "ts-node": "^10.9.2"
   },
-  "gitHead": "86896399e80595be179738d459106cfbfbada659"
+  "gitHead": "10d3a11478d17a20c80da5a827f8e8da4facfc80"
 }

package/plugins/error-events-plugin.d.ts

@@ -2,18 +2,20 @@ import '../middleware/types';
 import type { EventsConfig } from '@constructive-io/express-context';
 import type { GraphileConfig } from 'graphile-config';
 import { type Pool } from 'pg';
-export declare const GRAPHQL_ERROR_EVENT = "graphql.error";
-export declare const recordEventSql: (events: EventsConfig) => string;
+export declare const recordErrorSql: (events: EventsConfig) => string;
 /**
- * Records `graphql.error` when an authenticated mutation is refused with a
- * structured registry code. The refusal rolled back the mutation's own
- * transaction, so the event is written afterwards in a fresh transaction under
- * the same request claims, via the tenant's events module `record_event`.
+ * Records a refused authenticated mutation as an event named after the error
+ * code raised by `errors.raise_error`. The refusal rolled back the mutation's
+ * own transaction, so the event is written afterwards in a fresh transaction
+ * under the same request claims, via the tenant's events module
+ * `record_error`, which classifies the code as an error that earns no ladder
+ * progress.
  *
- * The server carries no policy about what a code means: the database reads
- * `payload->>'code'` (e.g. PRINCIPAL_CHILD_WIDENS demoting a principal on the
- * trust ladder). Endpoints without an events module record nothing.
- * Unauthenticated requests are never recorded, so anonymous traffic cannot
- * drive writes. The client response is never altered.
+ * The server carries no policy: the code is the event, and the database decides
+ * what it means — a ladder's `revoked_by` names the code directly (e.g.
+ * PRINCIPAL_CHILD_WIDENS demoting a principal on the trust ladder). Endpoints
+ * without an events module record nothing. Unauthenticated requests are never
+ * recorded, so anonymous traffic cannot drive writes. The client response is
+ * never altered.
  */
 export declare const createErrorEventsPlugin: (pool: Pool) => GraphileConfig.Plugin;

package/plugins/error-events-plugin.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.createErrorEventsPlugin = exports.recordEventSql = exports.GRAPHQL_ERROR_EVENT = void 0;
+exports.createErrorEventsPlugin = exports.recordErrorSql = void 0;
 require("../middleware/types"); // for Request type
 const logger_1 = require("@pgpmjs/logger");
 const graphql_1 = require("graphql");
@@ -8,7 +8,6 @@ const pg_1 = require("pg");
 const pg_query_context_1 = require("pg-query-context");
 const mask_error_1 = require("../middleware/mask-error");
 const log = new logger_1.Logger('error-events');
-exports.GRAPHQL_ERROR_EVENT = 'graphql.error';
 const getExpressRequest = (requestContext) => requestContext?.expressv4?.req;
 /**
  * The first structured, public-classified registry code among the errors.
@@ -23,24 +22,32 @@ const refusalCode = (errors) => {
     }
     return undefined;
 };
-const recordEventSql = (events) => `SELECT ${(0, pg_1.escapeIdentifier)(events.privateSchemaName)}.${(0, pg_1.escapeIdentifier)(events.recordEvent)}($1, $2::uuid, $3::jsonb)`;
-exports.recordEventSql = recordEventSql;
+const recordErrorSql = (events) => {
+    if (!events.recordError) {
+        throw new Error(`events module ${events.privateSchemaName} has no record_error function`);
+    }
+    return `SELECT ${(0, pg_1.escapeIdentifier)(events.privateSchemaName)}.${(0, pg_1.escapeIdentifier)(events.recordError)}($1, $2::uuid, $3::jsonb)`;
+};
+exports.recordErrorSql = recordErrorSql;
 /**
- * Records `graphql.error` when an authenticated mutation is refused with a
- * structured registry code. The refusal rolled back the mutation's own
- * transaction, so the event is written afterwards in a fresh transaction under
- * the same request claims, via the tenant's events module `record_event`.
+ * Records a refused authenticated mutation as an event named after the error
+ * code raised by `errors.raise_error`. The refusal rolled back the mutation's
+ * own transaction, so the event is written afterwards in a fresh transaction
+ * under the same request claims, via the tenant's events module
+ * `record_error`, which classifies the code as an error that earns no ladder
+ * progress.
  *
- * The server carries no policy about what a code means: the database reads
- * `payload->>'code'` (e.g. PRINCIPAL_CHILD_WIDENS demoting a principal on the
- * trust ladder). Endpoints without an events module record nothing.
- * Unauthenticated requests are never recorded, so anonymous traffic cannot
- * drive writes. The client response is never altered.
+ * The server carries no policy: the code is the event, and the database decides
+ * what it means — a ladder's `revoked_by` names the code directly (e.g.
+ * PRINCIPAL_CHILD_WIDENS demoting a principal on the trust ladder). Endpoints
+ * without an events module record nothing. Unauthenticated requests are never
+ * recorded, so anonymous traffic cannot drive writes. The client response is
+ * never altered.
  */
 const createErrorEventsPlugin = (pool) => ({
     name: 'ErrorEventsPlugin',
     version: '0.0.0',
-    description: 'Records graphql.error through the tenant events module when an authenticated mutation is refused.',
+    description: 'Records a refused authenticated mutation as its error code through the tenant events module.',
     grafast: {
         middleware: {
             async execute(next, event) {
@@ -64,14 +71,10 @@ const createErrorEventsPlugin = (pool) => ({
                     if (!events || !pgSettings)
                         return result;
                     const operation = args.operationName ?? (0, graphql_1.getOperationAST)(args.document)?.name?.value ?? null;
-                    await (0, pg_query_context_1.withPgClient)(pool, pgSettings, (client) => client.query((0, exports.recordEventSql)(events), [
-                        exports.GRAPHQL_ERROR_EVENT,
-                        actorId,
-                        JSON.stringify({ code, operation })
-                    ]));
+                    await (0, pg_query_context_1.withPgClient)(pool, pgSettings, (client) => client.query((0, exports.recordErrorSql)(events), [code, actorId, JSON.stringify({ operation })]));
                 }
                 catch (err) {
-                    log.error(`${label} failed to record ${exports.GRAPHQL_ERROR_EVENT} (${code}) for ${actorId}: ${err instanceof Error ? err.message : String(err)}`);
+                    log.error(`${label} failed to record refusal ${code} for ${actorId}: ${err instanceof Error ? err.message : String(err)}`);
                 }
                 return result;
             }