prostgles-server 4.2.605 → 4.2.607

package/lib/TableConfig/getTableTriggerQueries.ts

@@ -1,73 +1,117 @@
 import { as } from "pg-promise";
-import { md5 } from "prostgles-types/dist/md5";
 import type { DB } from "../initProstgles";
+import { asName } from "prostgles-types";
 import type { TableConfig } from "./TableConfigTypes";
+import {
+  AUDIT_HISTORY_PROTECTION_PREFIX,
+  getManagedTriggerName,
+  isManagedTriggerName,
+} from "./managedTriggerNames";
 
-const MANAGED_TRIGGER_PREFIX = "prostgles.tableConfig:";
-
-/** Reconcile only triggers owned by tableConfig; preserve manually created triggers. */
+/** Compare actual definitions so unchanged triggers do not cause schema refreshes. */
 export const getTableTriggerQueries = async (
   db: DB,
-  tableName: string,
+  tableIdent: string,
   tableConfig: TableConfig[string],
-  asName: (name: string) => string,
 ): Promise<string[]> => {
-  const tableIdent = asName(tableName);
-  const existing = await db.any<{ name: string; comment: string | null }>(
+  const existing = await db.any<{
+    name: string;
+    type: number;
+    enabled: string;
+    function_name: string;
+    query: string;
+    old_table: string | null;
+    new_table: string | null;
+    args: number;
+    unconditional: boolean;
+    visible_function: boolean;
+  }>(
     `
-    SELECT t.tgname AS name, obj_description(t.oid, 'pg_trigger') AS comment
-    FROM pg_trigger t
-    WHERE t.tgrelid = to_regclass($1) AND NOT t.tgisinternal
+    SELECT t.tgname AS name, t.tgtype AS type, t.tgenabled AS enabled,
+      p.proname AS function_name, p.prosrc AS query, t.tgoldtable AS old_table,
+      t.tgnewtable AS new_table, t.tgnargs AS args,
+      (t.tgqual IS NULL AND t.tgattr::text = '' AND t.tgconstraint = 0) AS unconditional,
+      p.oid = to_regprocedure(quote_ident(p.proname) || '()') AS visible_function
+    FROM pg_trigger t JOIN pg_proc p ON p.oid = t.tgfoid
+    WHERE t.tgrelid = to_regclass($1) AND NOT t.tgisinternal AND t.tgparentid = 0
   `,
     [tableIdent],
   );
   const queries: string[] = [];
   const desired = new Set<string>();
-  const isDropped = tableConfig.dropIfExists || tableConfig.dropIfExistsCascade;
+  const actionBits = { insert: 4, delete: 8, update: 16, truncate: 32 };
   for (const [functionName, trigger] of Object.entries(
     tableConfig.triggers ?? {},
   )) {
     const functionIdent = asName(functionName);
     let functionAdded = false;
     for (const action of trigger.actions) {
-      const name = `${functionName}_${action}`;
+      const name = getManagedTriggerName(functionName, action);
       desired.add(name);
+      // Replace the unprefixed names used by earlier tableConfig versions.
+      const legacyName = `${functionName}_${action}`;
+      if (
+        legacyName !== name &&
+        existing.some(
+          (t) => t.name === legacyName && t.function_name === functionName,
+        )
+      ) {
+        queries.push(`DROP TRIGGER ${asName(legacyName)} ON ${tableIdent};`);
+      }
       const triggerIdent = asName(name);
-      const transitionTables =
-        trigger.forEach === "row"
-          ? ""
-          : [
-              "REFERENCING",
-              action !== "delete" ? "NEW TABLE AS new_table" : "",
-              action !== "insert" ? "OLD TABLE AS old_table" : "",
-            ].join(" ");
-      const definition = `CREATE TRIGGER ${triggerIdent}
-        ${trigger.type} ${action} ON ${tableIdent}
-        ${transitionTables} FOR EACH ${trigger.forEach}
-        EXECUTE FUNCTION ${functionIdent}();`;
-      const comment = MANAGED_TRIGGER_PREFIX + md5(definition + trigger.query);
+      const hasTransitionTables =
+        trigger.forEach === "statement" &&
+        trigger.type === "after" &&
+        action !== "truncate";
+      const oldTable =
+        hasTransitionTables && action !== "insert" ? "old_table" : null;
+      const newTable =
+        hasTransitionTables && action !== "delete" ? "new_table" : null;
+      const transitionTables = hasTransitionTables
+        ? [
+            "REFERENCING",
+            newTable ? "NEW TABLE AS new_table" : "",
+            oldTable ? "OLD TABLE AS old_table" : "",
+          ].join(" ")
+        : "";
+      const type =
+        actionBits[action] |
+        (trigger.forEach === "row" ? 1 : 0) |
+        (trigger.type === "before"
+          ? 2
+          : trigger.type === "instead of"
+            ? 64
+            : 0);
       const previous = existing.find((entry) => entry.name === name);
-      if (!isDropped && previous?.comment === comment) continue;
+      if (
+        previous?.type === type &&
+        previous.enabled === "O" &&
+        previous.function_name === functionName &&
+        previous.query === trigger.query &&
+        previous.old_table === oldTable &&
+        previous.new_table === newTable &&
+        !previous.args &&
+        previous.unconditional &&
+        previous.visible_function
+      )
+        continue;
       if (!functionAdded) {
-        queries.push(`CREATE OR REPLACE FUNCTION ${functionIdent}()
-          RETURNS trigger LANGUAGE plpgsql AS ${as.text(trigger.query)};`);
+        queries.push(
+          `CREATE OR REPLACE FUNCTION ${functionIdent}() RETURNS trigger LANGUAGE plpgsql AS ${as.text(trigger.query)};`,
+        );
         functionAdded = true;
       }
       queries.push(`DROP TRIGGER IF EXISTS ${triggerIdent} ON ${tableIdent};`);
-      queries.push(definition);
-      queries.push(
-        `COMMENT ON TRIGGER ${triggerIdent} ON ${tableIdent} IS ${as.text(comment)};`,
-      );
+      queries.push(`CREATE TRIGGER ${triggerIdent} ${trigger.type} ${action} ON ${tableIdent}
+        ${transitionTables} FOR EACH ${trigger.forEach} EXECUTE FUNCTION ${functionIdent}();`);
     }
   }
-  if (!isDropped) {
-    for (const trigger of existing) {
-      if (
-        trigger.comment?.startsWith(MANAGED_TRIGGER_PREFIX) &&
-        !desired.has(trigger.name)
-      ) {
-        queries.push(`DROP TRIGGER ${asName(trigger.name)} ON ${tableIdent};`);
-      }
+  for (const trigger of existing) {
+    const managed = isManagedTriggerName(trigger.name);
+    // History remains append-only even after audit is disabled or moved.
+    const protectsHistory = trigger.name.startsWith(AUDIT_HISTORY_PROTECTION_PREFIX);
+    if (managed && !protectsHistory && !desired.has(trigger.name)) {
+      queries.push(`DROP TRIGGER ${asName(trigger.name)} ON ${tableIdent};`);
     }
   }
   return queries;

package/lib/TableConfig/initTableConfig.ts

@@ -9,10 +9,10 @@ import { fetchTableConstraints } from "./fetchTableConstraints";
 import { getConstraintDefinitionQueries } from "./getConstraintDefinitionQueries";
 import { getFutureTableSchema } from "./getFutureTableSchema";
 import { getTableColumnQueries } from "./getTableColumnQueries";
-import { getTableTriggerQueries } from "./getTableTriggerQueries";
 import { getIndexesQueries } from "./indexes/getIndexesQueries";
 import { runMigrations } from "./runMigrations";
 import type { TableConfigurator } from "./TableConfigurator";
+import { syncTableTriggers } from "./syncTableTriggers";
 
 export const initTableConfig = async function (this: TableConfigurator) {
   this.initialising = true;
@@ -234,8 +234,6 @@ export const initTableConfig = async function (this: TableConfigurator) {
 
     const indexQueries = await getIndexesQueries(this.db, tableName, tableConf);
     queries.push(...indexQueries);
-
-    queries.push(...(await getTableTriggerQueries(this.db, tableName, tableConf, asName)));
   }
 
   if (queries.length) {
@@ -268,7 +266,7 @@ export const initTableConfig = async function (this: TableConfigurator) {
   }
   this.initialising = false;
   // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
-  if (changedSchema && !failedQueries.length) {
+  if (changedSchema && !failedQueries.length && !this.prostgles.opts.audit) {
     if (!this.prevInitQueryHistory) {
       this.prevInitQueryHistory = queryHistory;
     } else if (this.prevInitQueryHistory.join() !== queryHistory.join()) {
@@ -284,7 +282,6 @@ export const initTableConfig = async function (this: TableConfigurator) {
   }
 
   await this.prostgles.rebuildDBO();
-  await this.setTableOnMounts();
-  /** Needed in case some onMounts change schema */
-  await this.prostgles.rebuildDBO();
+  await syncTableTriggers(this.prostgles);
+  if (this.prostgles.opts.audit) await this.prostgles.rebuildDBO();
 };

package/lib/TableConfig/managedTriggerNames.ts

@@ -0,0 +1,16 @@
+import { md5 } from "prostgles-types/dist/md5";
+
+// Persisted database names: creation and cleanup must use the same prefixes.
+// Keep generated names within PostgreSQL's 63-byte identifier limit.
+export const TABLE_CONFIG_TRIGGER_PREFIX = "prostgles_table_config_";
+export const AUDIT_TRIGGER_PREFIX = "prostgles_audit_";
+// History protection survives removing or changing the audit configuration.
+export const AUDIT_HISTORY_PROTECTION_PREFIX = AUDIT_TRIGGER_PREFIX + "guard_";
+export const AUDIT_TRUNCATE_PROTECTION_PREFIX = AUDIT_TRIGGER_PREFIX + "block_";
+export const AUDIT_WRITER_PREFIX = AUDIT_TRIGGER_PREFIX + "write_";
+
+export const isManagedTriggerName = (name: string) =>
+  name.startsWith(TABLE_CONFIG_TRIGGER_PREFIX) || name.startsWith(AUDIT_TRIGGER_PREFIX);
+
+export const getManagedTriggerName = (functionName: string, action: string) =>
+  `${functionName.startsWith(AUDIT_TRIGGER_PREFIX) ? functionName : TABLE_CONFIG_TRIGGER_PREFIX + md5(functionName)}_${action}`;

package/lib/TableConfig/runSQLFile.ts

@@ -3,7 +3,7 @@ import { getFileText, type Prostgles } from "../Prostgles";
 
 export const runSQLFile = async (prostgles: Prostgles) => {
   const {
-    db,
+    dbForSchema: db,
     opts: { sqlFilePath, onLog },
   } = prostgles;
   if (!sqlFilePath) return;

package/lib/TableConfig/syncTableTriggers.ts

@@ -0,0 +1,42 @@
+import { parseAuditConfig } from "../Audit/parseAuditConfig";
+import type { Prostgles } from "../Prostgles";
+import { EXCLUDE_QUERY_FROM_SCHEMA_WATCH_ID } from "../PubSubManager/PubSubManagerUtils";
+import { getTableTriggerQueries } from "./getTableTriggerQueries";
+
+/** Applies application and generated audit triggers together, after all tables exist. */
+export async function syncTableTriggers(prgl: Prostgles) {
+  const previousAudit = prgl.parsedAuditConfig;
+  const nextAudit =
+    prgl.opts.audit || previousAudit ? await parseAuditConfig(prgl) : undefined;
+  prgl.parsedAuditConfig = nextAudit;
+  try {
+    const config = prgl.mergedTableConfig.tableConfig ?? {};
+    const queries: string[] = [];
+    for (const name of Object.keys(config)) {
+      if (!prgl.dboBuilder.tables.some((t) => t.name === name)) {
+        throw new Error(
+          `Table config name must exactly match a schema table name: ${name}`,
+        );
+      }
+    }
+    for (const table of prgl.dboBuilder.tables) {
+      queries.push(
+        ...(await getTableTriggerQueries(
+          prgl.dbForSchema!,
+          table.escaped_identifier,
+          { triggers: config[table.name]?.triggers },
+        )),
+      );
+    }
+    if (queries.length) {
+      await prgl.dbForSchema!.tx((tx) =>
+        tx.none(
+          `/* ${EXCLUDE_QUERY_FROM_SCHEMA_WATCH_ID} */\n${queries.join("\n")}`,
+        ),
+      );
+    }
+  } catch (error) {
+    prgl.parsedAuditConfig = previousAudit;
+    throw error;
+  }
+}

package/lib/deferDbQueries.ts

@@ -0,0 +1,27 @@
+import type { DB } from "./initProstgles";
+
+/** Keep the original connection available for schema SQL; defer application calls. */
+export const deferDbQueries = (
+  db: DB,
+  getSchemaReady: () => Promise<void>,
+): DB =>
+  new Proxy(db, {
+    get(target, key, receiver) {
+      const value: unknown = Reflect.get(target, key, receiver);
+      // Connection metadata and lifecycle methods remain unchanged.
+      if (
+        !Object.hasOwn(target, key) ||
+        typeof value !== "function" ||
+        String(key).startsWith("$")
+      )
+        return value;
+      return async (...args: unknown[]) => {
+        let ready: Promise<void>;
+        do {
+          ready = getSchemaReady();
+          await ready;
+        } while (ready !== getSchemaReady());
+        return Reflect.apply(value, target, args) as unknown;
+      };
+    },
+  });

package/lib/index.ts

@@ -40,8 +40,11 @@ export type {
 } from "./initProstgles";
 export type { ProstglesInitOptions } from "./ProstglesTypes";
 export type * from "./TableConfig/TableConfigTypes";
+export type * from "./Audit/AuditTypes";
 export type * from "./TableHooks/TableHooks";
 export * from "./Auth/utils/upsertNamedExpressMiddleware";
 export type { RequestWithUser } from "./Auth/middleware/userContextMiddleware";
 export default prostgles;
 export type { FileTableRow } from "./StorageClient/getFileTableConfig";
+
+export { AUDIT_TABLE_COLUMN_DEFINITIONS, AUDIT_TABLE_COLUMNS, type AuditTableRow } from "./Audit/AuditTable";

package/lib/initProstgles.ts

@@ -1,3 +1,5 @@
+import { syncTableTriggers } from "./TableConfig/syncTableTriggers";
+import { deferDbQueries } from "./deferDbQueries";
 import type pgPromise from "pg-promise";
 import type pg from "pg-promise/typescript/pg-subset";
 import type { MaybePromise, SQLHandler, TableSchema } from "prostgles-types";
@@ -42,6 +44,7 @@ export type UpdatableOptions<
   | "restApi"
   | "tableHooks"
   | "tableConfig"
+  | "audit"
   | "schemaFilter"
   | "auth"
   | "publish"
@@ -209,7 +212,8 @@ export const initProstgles = async function (
       )
       .catch(() => {});
 
-    this.db = db;
+    this.dbForSchema = db;
+    this.db = deferDbQueries(db, () => this.schemaReady);
     this.pgp = pgp;
     this.isSuperUser = await getIsSuperUser(db);
   }
@@ -221,17 +225,24 @@ export const initProstgles = async function (
   try {
     await this.cleanupContext();
 
-    /* 2. Execute any SQL file if provided */
-    await runSQLFile(this);
-    this.preparingTableConfig = true;
-    try {
+    await this.tableConfigurator?.destroy();
+    await this.runSchemaQueries(async () => {
+      /* 2. Execute any SQL file if provided */
+      await runSQLFile(this);
+      this.preparingTableConfig = true;
+      try {
+        await this.rebuildDBO();
+        await this.initTableConfig(reason);
+      } finally {
+        this.preparingTableConfig = false;
+      }
+    });
+    await this.tableConfigurator?.setTableOnMounts();
+    // onMount may create additional targets; validate joins against the completed schema.
+    await this.runSchemaQueries(async () => {
       await this.rebuildDBO();
-      await this.initTableConfig(reason);
-    } finally {
-      this.preparingTableConfig = false;
-    }
-    // Validate all configured joins against the completed schema before publishing.
-    await this.rebuildDBO();
+      await syncTableTriggers(this);
+    });
     await this.createContext(reason);
     this.initRestApi();
 
@@ -267,7 +278,7 @@ export const initProstgles = async function (
         {
           sql: this.dboBuilder.sql,
           dbo: this.dbo!,
-          db: this.db,
+          db,
           tables: this.dboBuilder.tables,
           reason,
           context: this.context,
@@ -324,6 +335,7 @@ export const initProstgles = async function (
         await this.tableConfigurator?.destroy();
         this.dbo = undefined;
         this.db = undefined;
+        this.dbForSchema = undefined;
         await db.$pool.end();
         await this.adminClient?.end().catch(() => {});
         this.adminClient = undefined;

package/lib/updateConfiguration.ts

@@ -20,6 +20,7 @@ export const updateConfiguration = async (
     })
     .filter(isDefined);
   if (!optionsThatChanged.length) {
+    await prgl.schemaReady;
     console.warn("No options changed");
     return;
   }
@@ -27,12 +28,6 @@ export const updateConfiguration = async (
   if (includes(optionsThatChanged, "restApi")) {
     prgl.initRestApi();
   }
-  if (includes(optionsThatChanged, "tableConfig")) {
-    await prgl.initTableConfig({
-      type: "prgl.update",
-      newOpts,
-    });
-  }
   if (includes(optionsThatChanged, "schema")) {
     await prgl.refreshDBO();
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prostgles-server",
-  "version": "4.2.605",
+  "version": "4.2.607",
   "description": "",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/dist/TableConfig/getTableTriggerQueries.spec.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=getTableTriggerQueries.spec.d.ts.map

package/dist/TableConfig/getTableTriggerQueries.spec.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"getTableTriggerQueries.spec.d.ts","sourceRoot":"","sources":["../../lib/TableConfig/getTableTriggerQueries.spec.ts"],"names":[],"mappings":""}

package/dist/TableConfig/getTableTriggerQueries.spec.js

@@ -1,56 +0,0 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-const strict_1 = __importDefault(require("node:assert/strict"));
-const node_test_1 = __importDefault(require("node:test"));
-const prostgles_types_1 = require("prostgles-types");
-const getTableTriggerQueries_1 = require("./getTableTriggerQueries");
-void (0, node_test_1.default)("trigger configuration quotes names and bodies and preserves unmanaged triggers", async () => {
-    const db = {
-        any: () => Promise.resolve([
-            { name: "manual", comment: null },
-            { name: "old_insert", comment: "prostgles.tableConfig:old" },
-        ]),
-    };
-    const queries = await (0, getTableTriggerQueries_1.getTableTriggerQueries)(db, "Odd ' table", {
-        triggers: {
-            audit: {
-                type: "after",
-                actions: ["insert"],
-                forEach: "row",
-                query: "BEGIN RAISE NOTICE '$$'; RETURN NULL; END;",
-            },
-        },
-    }, prostgles_types_1.asName);
-    strict_1.default.ok(queries.some((query) => query.includes('ON "Odd \' table"')));
-    strict_1.default.ok(queries.some((query) => query.includes("AS 'BEGIN RAISE NOTICE ''$$''; RETURN NULL; END;';")));
-    strict_1.default.ok(queries.some((query) => query.includes('DROP TRIGGER "old_insert"')));
-    strict_1.default.ok(!queries.some((query) => query.includes('DROP TRIGGER "manual"')));
-});
-void (0, node_test_1.default)("unchanged triggers do not produce DDL; changed bodies replace them", async () => {
-    const trigger = {
-        type: "after",
-        actions: ["update"],
-        forEach: "row",
-        query: "BEGIN RETURN NULL; END;",
-    };
-    const config = {
-        triggers: { audit: { ...trigger, actions: [...trigger.actions] } },
-    };
-    const first = await (0, getTableTriggerQueries_1.getTableTriggerQueries)({ any: () => Promise.resolve([]) }, "records", config, prostgles_types_1.asName);
-    const commentQuery = first.find((query) => query.startsWith("COMMENT ON"));
-    const comment = commentQuery.split(" IS '")[1].slice(0, -2);
-    const db = {
-        any: () => Promise.resolve([{ name: "audit_update", comment }]),
-    };
-    strict_1.default.deepEqual(await (0, getTableTriggerQueries_1.getTableTriggerQueries)(db, "records", config, prostgles_types_1.asName), []);
-    config.triggers.audit.query =
-        "BEGIN RAISE NOTICE 'changed'; RETURN NULL; END;";
-    strict_1.default.ok((await (0, getTableTriggerQueries_1.getTableTriggerQueries)(db, "records", config, prostgles_types_1.asName)).some((query) => query.startsWith("CREATE OR REPLACE FUNCTION")));
-    strict_1.default.deepEqual(await (0, getTableTriggerQueries_1.getTableTriggerQueries)(db, "records", {}, prostgles_types_1.asName), [
-        'DROP TRIGGER "audit_update" ON "records";',
-    ]);
-});
-//# sourceMappingURL=getTableTriggerQueries.spec.js.map

package/dist/TableConfig/getTableTriggerQueries.spec.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"getTableTriggerQueries.spec.js","sourceRoot":"","sources":["../../lib/TableConfig/getTableTriggerQueries.spec.ts"],"names":[],"mappings":";;;;;AAAA,gEAAwC;AACxC,0DAA6B;AAC7B,qDAAyC;AAEzC,qEAAkE;AAElE,KAAK,IAAA,mBAAI,EAAC,gFAAgF,EAAE,KAAK,IAAI,EAAE;IACrG,MAAM,EAAE,GAAG;QACT,GAAG,EAAE,GAAG,EAAE,CACR,OAAO,CAAC,OAAO,CAAC;YACd,EAAE,IAAI,EAAE,QAAQ,EAAE,OAAO,EAAE,IAAI,EAAE;YACjC,EAAE,IAAI,EAAE,YAAY,EAAE,OAAO,EAAE,2BAA2B,EAAE;SAC7D,CAAC;KACY,CAAC;IACnB,MAAM,OAAO,GAAG,MAAM,IAAA,+CAAsB,EAC1C,EAAE,EACF,aAAa,EACb;QACE,QAAQ,EAAE;YACR,KAAK,EAAE;gBACL,IAAI,EAAE,OAAO;gBACb,OAAO,EAAE,CAAC,QAAQ,CAAC;gBACnB,OAAO,EAAE,KAAK;gBACd,KAAK,EAAE,4CAA4C;aACpD;SACF;KACF,EACD,wBAAM,CACP,CAAC;IACF,gBAAM,CAAC,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC;IACxE,gBAAM,CAAC,EAAE,CACP,OAAO,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CACrB,KAAK,CAAC,QAAQ,CAAC,oDAAoD,CAAC,CACrE,CACF,CAAC;IACF,gBAAM,CAAC,EAAE,CACP,OAAO,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,2BAA2B,CAAC,CAAC,CACrE,CAAC;IACF,gBAAM,CAAC,EAAE,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,uBAAuB,CAAC,CAAC,CAAC,CAAC;AAC/E,CAAC,CAAC,CAAC;AAEH,KAAK,IAAA,mBAAI,EAAC,oEAAoE,EAAE,KAAK,IAAI,EAAE;IACzF,MAAM,OAAO,GAAG;QACd,IAAI,EAAE,OAAO;QACb,OAAO,EAAE,CAAC,QAAQ,CAAC;QACnB,OAAO,EAAE,KAAK;QACd,KAAK,EAAE,yBAAmC;KAClC,CAAC;IACX,MAAM,MAAM,GAAG;QACb,QAAQ,EAAE,EAAE,KAAK,EAAE,EAAE,GAAG,OAAO,EAAE,OAAO,EAAE,CAAC,GAAG,OAAO,CAAC,OAAO,CAAC,EAAE,EAAE;KACnE,CAAC;IACF,MAAM,KAAK,GAAG,MAAM,IAAA,+CAAsB,EACxC,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC,EAAmB,EACnD,SAAS,EACT,MAAM,EACN,wBAAM,CACP,CAAC;IACF,MAAM,YAAY,GAAG,KAAK,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,UAAU,CAAC,YAAY,CAAC,CAAE,CAAC;IAC5E,MAAM,OAAO,GAAG,YAAY,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,CAAE,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;IAC7D,MAAM,EAAE,GAAG;QACT,GAAG,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,EAAE,IAAI,EAAE,cAAc,EAAE,OAAO,EAAE,CAAC,CAAC;KAC/C,CAAC;IACnB,gBAAM,CAAC,SAAS,CACd,MAAM,IAAA,+CAAsB,EAAC,EAAE,EAAE,SAAS,EAAE,MAAM,EAAE,wBAAM,CAAC,EAC3D,EAAE,CACH,CAAC;IACF,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,KAAK;QACzB,iDAAiD,CAAC;IACpD,gBAAM,CAAC,EAAE,CACP,CAAC,MAAM,IAAA,+CAAsB,EAAC,EAAE,EAAE,SAAS,EAAE,MAAM,EAAE,wBAAM,CAAC,CAAC,CAAC,IAAI,CAChE,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,UAAU,CAAC,4BAA4B,CAAC,CAC1D,CACF,CAAC;IACF,gBAAM,CAAC,SAAS,CAAC,MAAM,IAAA,+CAAsB,EAAC,EAAE,EAAE,SAAS,EAAE,EAAE,EAAE,wBAAM,CAAC,EAAE;QACxE,2CAA2C;KAC5C,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/lib/TableConfig/getTableTriggerQueries.spec.ts

@@ -1,77 +0,0 @@
-import assert from "node:assert/strict";
-import test from "node:test";
-import { asName } from "prostgles-types";
-import type { DB } from "../initProstgles";
-import { getTableTriggerQueries } from "./getTableTriggerQueries";
-
-void test("trigger configuration quotes names and bodies and preserves unmanaged triggers", async () => {
-  const db = {
-    any: () =>
-      Promise.resolve([
-        { name: "manual", comment: null },
-        { name: "old_insert", comment: "prostgles.tableConfig:old" },
-      ]),
-  } as unknown as DB;
-  const queries = await getTableTriggerQueries(
-    db,
-    "Odd ' table",
-    {
-      triggers: {
-        audit: {
-          type: "after",
-          actions: ["insert"],
-          forEach: "row",
-          query: "BEGIN RAISE NOTICE '$$'; RETURN NULL; END;",
-        },
-      },
-    },
-    asName,
-  );
-  assert.ok(queries.some((query) => query.includes('ON "Odd \' table"')));
-  assert.ok(
-    queries.some((query) =>
-      query.includes("AS 'BEGIN RAISE NOTICE ''$$''; RETURN NULL; END;';"),
-    ),
-  );
-  assert.ok(
-    queries.some((query) => query.includes('DROP TRIGGER "old_insert"')),
-  );
-  assert.ok(!queries.some((query) => query.includes('DROP TRIGGER "manual"')));
-});
-
-void test("unchanged triggers do not produce DDL; changed bodies replace them", async () => {
-  const trigger = {
-    type: "after",
-    actions: ["update"],
-    forEach: "row",
-    query: "BEGIN RETURN NULL; END;" as string,
-  } as const;
-  const config = {
-    triggers: { audit: { ...trigger, actions: [...trigger.actions] } },
-  };
-  const first = await getTableTriggerQueries(
-    { any: () => Promise.resolve([]) } as unknown as DB,
-    "records",
-    config,
-    asName,
-  );
-  const commentQuery = first.find((query) => query.startsWith("COMMENT ON"))!;
-  const comment = commentQuery.split(" IS '")[1]!.slice(0, -2);
-  const db = {
-    any: () => Promise.resolve([{ name: "audit_update", comment }]),
-  } as unknown as DB;
-  assert.deepEqual(
-    await getTableTriggerQueries(db, "records", config, asName),
-    [],
-  );
-  config.triggers.audit.query =
-    "BEGIN RAISE NOTICE 'changed'; RETURN NULL; END;";
-  assert.ok(
-    (await getTableTriggerQueries(db, "records", config, asName)).some(
-      (query) => query.startsWith("CREATE OR REPLACE FUNCTION"),
-    ),
-  );
-  assert.deepEqual(await getTableTriggerQueries(db, "records", {}, asName), [
-    'DROP TRIGGER "audit_update" ON "records";',
-  ]);
-});