@rotorsoft/act-pg 1.10.0 → 1.10.2

package/dist/index.js

@@ -33,6 +33,7 @@ types.setTypeParser(
 var SAFE_IDENTIFIER = /^[a-zA-Z_][a-zA-Z0-9_]*$/;
 var PG_UNIQUE_VIOLATION = "23505";
 var NOTIFY_CHANNEL_PREFIX = "act_commit";
+var NOTIFY_MAX_PAYLOAD_BYTES = 8e3;
 function notify_channel(schema, table) {
   return `${NOTIFY_CHANNEL_PREFIX}_${schema}_${table}`;
 }
@@ -48,7 +49,35 @@ var DEFAULT_CONFIG = {
   password: "postgres",
   schema: "public",
   table: "events",
-  notify: false
+  notify: false,
+  // Opinionated pool defaults (#1119). node-postgres ships `max: 10`
+  // with no acquisition timeout and no statement timeout — a saturated
+  // pool makes every caller hang indefinitely instead of failing with
+  // a diagnosable error. Nearly every store method holds a client for
+  // a multi-statement transaction, so multi-lane drains plus API
+  // traffic can exhaust a 10-client pool quickly. All four values are
+  // plain `pg.PoolConfig` fields — caller config overrides any of them
+  // via the constructor spread.
+  //
+  // - `max: 20` — floor for the default lane's parallel handler budget
+  //   (streamLimit 10, each commit holds a client) plus API commits,
+  //   the optional LISTEN client, and headroom. Sizing rule in the
+  //   README: Σ per-lane streamLimit + API concurrency + notify + 2–4.
+  // - `connectionTimeoutMillis: 10_000` — fail acquisition fast (the
+  //   pg default of 0 waits forever); surfaces as StoreError via
+  //   `_client()` so operators see *which* operation starved.
+  // - `idleTimeoutMillis: 30_000` — keep idle clients warm across
+  //   drain cycles (cycleMs can exceed the pg default of 10s in
+  //   low-traffic deployments) without pinning connections for long.
+  // - `statement_timeout: 60_000` — per-statement (not per-transaction)
+  //   ceiling; every statement the store issues (claim CTE, seed DDL,
+  //   truncate, restore's per-event inserts) legitimately completes
+  //   orders of magnitude faster, so 60s only fires on a wedged server
+  //   or a lost lock instead of holding the client hostage.
+  max: 20,
+  connectionTimeoutMillis: 1e4,
+  idleTimeoutMillis: 3e4,
+  statement_timeout: 6e4
 };
 var PostgresStore = class {
   _pool;
@@ -119,6 +148,21 @@ var PostgresStore = class {
     }
     this._resolve_pii_key = this.config.pii_encryption ? makeKeyResolver(this.config.pii_encryption) : void 0;
   }
+  /**
+   * Acquire a pooled client, translating acquisition failures into
+   * {@link StoreError} with the calling operation as context. With the
+   * default `connectionTimeoutMillis`, a saturated pool fails here
+   * after 10s with `Store operation "<operation>" failed` (driver
+   * error preserved as `cause`) instead of hanging indefinitely.
+   * Every method that checks out a client routes through this helper.
+   */
+  async _client(operation) {
+    try {
+      return await this._pool.connect();
+    } catch (error) {
+      throw new StoreError(operation, { cause: error });
+    }
+  }
   /**
    * Dispose of the store and close all database connections.
    * Releases any active LISTEN client first so the pool can drain cleanly.
@@ -152,7 +196,7 @@ var PostgresStore = class {
    * @throws Error if seeding fails
    */
   async seed() {
-    const client = await this._pool.connect();
+    const client = await this._client("seed");
     try {
       await client.query("BEGIN");
       await client.query(
@@ -365,7 +409,7 @@ var PostgresStore = class {
    */
   async commit(stream, msgs, meta, expectedVersion) {
     if (msgs.length === 0) return [];
-    const client = await this._pool.connect();
+    const client = await this._client("commit");
     let version = -1;
     try {
       await client.query("BEGIN");
@@ -417,10 +461,11 @@ var PostgresStore = class {
           events: committed.map((c) => ({ id: c.id, name: c.name })),
           by: this._by
         });
-        await client.query(`SELECT pg_notify($1, $2)`, [
-          this._channel,
-          payload
-        ]);
+        if (Buffer.byteLength(payload, "utf8") < NOTIFY_MAX_PAYLOAD_BYTES)
+          await client.query(`SELECT pg_notify($1, $2)`, [
+            this._channel,
+            payload
+          ]);
       }
       await client.query("COMMIT");
       return committed;
@@ -447,7 +492,7 @@ var PostgresStore = class {
    * @returns Leased streams with metadata
    */
   async claim(lagging, leading, by, millis, lane) {
-    const client = await this._pool.connect();
+    const client = await this._client("claim");
     try {
       await client.query("BEGIN");
       const lane_clause = lane !== void 0 ? `AND s.lane = $5` : "";
@@ -529,7 +574,7 @@ var PostgresStore = class {
    * @returns subscribed count and current max watermark.
    */
   async subscribe(streams) {
-    const client = await this._pool.connect();
+    const client = await this._client("subscribe");
     try {
       await client.query("BEGIN");
       let subscribed = 0;
@@ -589,30 +634,31 @@ var PostgresStore = class {
    * @returns Acked leases.
    */
   async ack(leases) {
-    const client = await this._pool.connect();
+    const client = await this._client("ack");
     try {
       await client.query("BEGIN");
       const { rows } = await client.query(
         `
       WITH input AS (
         SELECT * FROM jsonb_to_recordset($1::jsonb)
-        AS x(stream text, by text, at int, lagging boolean)
+        AS x(stream text, by text, at int, lagging boolean, due bigint)
       )
       UPDATE ${this._fqs} AS s
       SET
-        at = i.at,
+        at = CASE WHEN i.due IS NULL THEN i.at ELSE s.at END,
         retry = -1,
         leased_by = NULL,
         leased_until = NULL,
-        deferred_at = NULL
+        deferred_at = CASE WHEN i.due IS NULL THEN NULL
+                           ELSE to_timestamp(i.due / 1000.0) END
       FROM input i
       WHERE s.stream = i.stream AND s.leased_by = i.by
-      RETURNING s.stream, s.source, s.at, i.by, s.retry, i.lagging, s.lane
+      RETURNING s.stream, s.source, s.at, i.by, s.retry, i.lagging, s.lane, i.due
       `,
         [JSON.stringify(leases)]
       );
       await client.query("COMMIT");
-      return rows.map((row) => ({
+      return rows.filter((row) => row.due === null).map((row) => ({
         stream: row.stream,
         source: row.source ?? void 0,
         at: row.at,
@@ -635,7 +681,7 @@ var PostgresStore = class {
    * @returns Blocked leases.
    */
   async block(leases) {
-    const client = await this._pool.connect();
+    const client = await this._client("block");
     try {
       await client.query("BEGIN");
       const { rows } = await client.query(
@@ -866,7 +912,7 @@ var PostgresStore = class {
     if (conditions.length) sql += " WHERE " + conditions.join(" AND ");
     values.push(limit);
     sql += ` ORDER BY stream LIMIT $${values.length}`;
-    const client = await this._pool.connect();
+    const client = await this._client("query_streams");
     try {
       const [streamsResult, maxResult] = await Promise.all([
         client.query(sql, values),
@@ -1096,7 +1142,7 @@ var PostgresStore = class {
    */
   async _subscribe_notifications(handler) {
     await this._teardown_listen();
-    const client = await this._pool.connect();
+    const client = await this._client("notify");
     const on_notification = (msg) => {
       if (msg.channel !== this._channel) return;
       if (!msg.payload) return;
@@ -1157,7 +1203,7 @@ var PostgresStore = class {
   async truncate(targets) {
     if (!targets.length) return /* @__PURE__ */ new Map();
     const streams = targets.map((t) => t.stream);
-    const client = await this._pool.connect();
+    const client = await this._client("truncate");
     try {
       await client.query("BEGIN");
       await client.query(`DELETE FROM ${this._fqs} WHERE stream = ANY($1)`, [
@@ -1208,7 +1254,7 @@ var PostgresStore = class {
    * from 1. `created` is preserved verbatim from the source.
    */
   async restore(driver) {
-    const client = await this._pool.connect();
+    const client = await this._client("restore");
     try {
       await client.query("BEGIN");
       await client.query(