@mikro-orm/pglite 7.1.5-dev.3 → 7.1.5-dev.5

package/PgliteConnection.js

@@ -40,14 +40,16 @@ export class PgliteConnection extends AbstractSqlConnection {
         const dataDir = options.dataDir ?? dbName;
         const parsers = { ...createPostgreSqlTypeParsers(s => array.parse(s)), ...options.parsers };
         this.#ownsPglite = !userPglite;
-        // Skip `pglite.close()` (called by kysely's `PGliteDriver.destroy()`) when:
-        //   - the user owns the instance (closing it is their responsibility), or
-        //   - we hold an in-memory DB whose data would otherwise be lost on
-        //     reconnect (e.g. test harnesses that drive `orm.close()` + `orm.connect()`).
-        // For file/IDB-backed instances we let kysely close so file handles / IDB
-        // connections are released; we drop our cached promise in `close()` below
-        // so the next dialect-construction rebuilds the PGlite instance.
-        this.#neuterClose = !this.#ownsPglite || isInMemoryDataDir(dataDir);
+        // Skip `pglite.close()` (called by kysely's `PGliteDriver.destroy()`) only
+        // when the user owns the instance — closing it is their responsibility, and
+        // patching `close` on it directly would orphan it (its real `close` would
+        // stay neutered after `orm.close()`). Instances we own (in-memory or
+        // file/IDB-backed) get really closed so the underlying WASM / file handle /
+        // IDB resources are released; we drop our cached promise in `close()` below
+        // so the next dialect-construction rebuilds the PGlite instance. In-memory
+        // data does not survive `orm.close()`, consistent with other in-memory
+        // drivers (e.g. SQLite `:memory:`).
+        this.#neuterClose = !this.#ownsPglite;
         if (!this.#pglite) {
             if (userPglite) {
                 this.#pglite = typeof userPglite === 'function' ? Promise.resolve(userPglite()) : Promise.resolve(userPglite);
@@ -88,9 +90,9 @@ export class PgliteConnection extends AbstractSqlConnection {
     }
     async close(force) {
         await super.close(force);
-        // Persistent backings (file/IDB) we own get really closed by kysely — drop
-        // the cached promise so a subsequent reconnect rebuilds a fresh instance
-        // (which then re-reads the persisted data from disk / IDB).
+        // Instances we own get really closed by kysely — drop the cached promise so a
+        // subsequent reconnect rebuilds a fresh instance (file/IDB backings re-read the
+        // persisted data; in-memory starts empty, matching other in-memory drivers).
         if (this.#ownsPglite && !this.#neuterClose) {
             this.#pglite = undefined;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/pglite",
-  "version": "7.1.5-dev.3",
+  "version": "7.1.5-dev.5",
   "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",
@@ -49,7 +49,7 @@
   },
   "dependencies": {
     "@electric-sql/pglite": "0.5.1",
-    "@mikro-orm/sql": "7.1.5-dev.3",
+    "@mikro-orm/sql": "7.1.5-dev.5",
     "kysely": "0.29.2",
     "postgres-array": "3.0.4",
     "postgres-date": "2.1.0",
@@ -59,7 +59,7 @@
     "@mikro-orm/core": "^7.1.4"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.1.5-dev.3"
+    "@mikro-orm/core": "7.1.5-dev.5"
   },
   "engines": {
     "node": ">= 22.17.0"