@mikro-orm/postgresql 7.2.6-dev.17 → 7.2.6-dev.19

package/PostgreSqlConnection.d.ts

@@ -16,8 +16,8 @@ export declare class PostgreSqlConnection extends AbstractSqlConnection {
      * from a client while that client is checked out, deliberately leaving errors to whoever holds it,
      * and it re-emits errors of idle clients on the pool itself. The query in flight still rejects on
      * its own. An idle client is already discarded by the pool by the time we see it, a checked-out one
-     * is released here, as kysely's `ControlledTransaction` only releases after a successful
-     * commit/rollback (kysely-org/kysely#1990), which a dead connection never gets.
+     * is released here, as kysely's `ControlledTransaction` only releases after a commit or a rollback,
+     * which a transaction left open on a dead connection may never get.
      */
     private handlePoolErrors;
     /** Applies the active EM's session context (or resets it) on every pooled connection acquire. */

package/PostgreSqlConnection.js

@@ -1,15 +1,36 @@
-import { Pool, TypeOverrides } from 'pg';
+import { Client, Pool, TypeOverrides } from 'pg';
 import Cursor from 'pg-cursor';
 import { CompiledQuery, PostgresDialect } from 'kysely';
 import array from 'postgres-array';
 import { AbstractSqlConnection, createPostgreSqlTypeParsers, Utils } from '@mikro-orm/sql';
+/**
+ * PostgreSQL answers `COMMIT` of an aborted transaction with the command tag `ROLLBACK` instead of an error,
+ * and kysely drops the tag, so the failed commit is rejected here, like any other commit the server refused.
+ */
+class PostgreSqlClient extends Client {
+    query(...args) {
+        const res = super.query.apply(this, args);
+        // kysely sends a top-level commit as exactly `commit`, a callback-style call returns nothing
+        if (args[0] !== 'commit' || !res) {
+            return res;
+        }
+        return res.then((result) => {
+            if (result.command === 'ROLLBACK') {
+                throw Object.assign(new Error('current transaction is aborted, COMMIT was answered with ROLLBACK'), {
+                    code: '25P02',
+                });
+            }
+            return result;
+        });
+    }
+}
 /** PostgreSQL database connection using the `pg` driver. */
 export class PostgreSqlConnection extends AbstractSqlConnection {
     #pool;
     createKyselyDialect(overrides) {
         const { onPoolCreated, ...poolOverrides } = (overrides ?? {});
         const options = this.mapOptions(poolOverrides);
-        const pool = new Pool(options);
+        const pool = new Pool({ Client: PostgreSqlClient, ...options });
         this.#pool = pool;
         this.handlePoolErrors(pool);
         void onPoolCreated?.(pool);
@@ -36,8 +57,8 @@ export class PostgreSqlConnection extends AbstractSqlConnection {
      * from a client while that client is checked out, deliberately leaving errors to whoever holds it,
      * and it re-emits errors of idle clients on the pool itself. The query in flight still rejects on
      * its own. An idle client is already discarded by the pool by the time we see it, a checked-out one
-     * is released here, as kysely's `ControlledTransaction` only releases after a successful
-     * commit/rollback (kysely-org/kysely#1990), which a dead connection never gets.
+     * is released here, as kysely's `ControlledTransaction` only releases after a commit or a rollback,
+     * which a transaction left open on a dead connection may never get.
      */
     handlePoolErrors(pool) {
         const checkedOut = new WeakSet();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/postgresql",
-  "version": "7.2.6-dev.17",
+  "version": "7.2.6-dev.19",
   "description": "TypeScript ORM for Node.js based on Data Mapper, Unit of Work and Identity Map patterns. Supports MongoDB, MySQL, PostgreSQL and SQLite databases as well as usage with vanilla JavaScript.",
   "keywords": [
     "data-mapper",
@@ -47,7 +47,7 @@
     "copy": "node ../../scripts/copy.mjs"
   },
   "dependencies": {
-    "@mikro-orm/sql": "7.2.6-dev.17",
+    "@mikro-orm/sql": "7.2.6-dev.19",
     "kysely": "0.29.7",
     "pg": "8.23.1",
     "pg-cursor": "2.22.1",
@@ -59,7 +59,7 @@
     "@mikro-orm/core": "^7.2.5"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.2.6-dev.17"
+    "@mikro-orm/core": "7.2.6-dev.19"
   },
   "engines": {
     "node": ">= 22.17.0"