@malloydata/db-duckdb 0.0.388 → 0.0.390

package/dist/duckdb_connection.d.ts

@@ -64,10 +64,24 @@ export declare class DuckDBConnection extends DuckDBCommon {
     close(): Promise<void>;
     idle(): Promise<void>;
     /**
-     * Remove this connection from the shared `activeDBs` entry. When the
-     * last connection sharing the entry is removed, close the underlying
-     * `DuckDBInstance` and drop the entry — releasing DuckDB's per-process
-     * file lock.
+     * Drop our entry from the shared `activeDBs` bookkeeping. If we were
+     * the last sharer, close the underlying `DuckDBInstance`.
+     *
+     * NOTE: this does NOT fully release the OS file lock when other
+     * processes try to open the same path. `duckdb_close` (= `closeSync`)
+     * is a refcount-decrement on `shared_ptr<DatabaseInstance>`; the
+     * `fcntl(F_SETLK)` lock on the storage manager's `UnixFileHandle`
+     * survives until the last `shared_ptr` ref is gone, and live C++
+     * `Connection` objects keep that refcount above zero via their
+     * `ClientContext`. To force the lock release we'd need to also
+     * `disconnectSync()` the per-connection node-api Connection — but
+     * doing so unconditionally (as 0.0.389 did) crashes the language
+     * server because malloy's translate/persistence layer retains
+     * weak_ptrs to the destroyed C++ Connection. See the commented-out
+     * code below and PR #2793 / the revert PR for context.
+     *
+     * Proper fix needs to coordinate disconnect with invalidation of
+     * whatever caches survive idle (manifest reader is the main suspect).
      */
     private detachInstance;
     /**

package/dist/duckdb_connection.js

@@ -295,12 +295,36 @@ class DuckDBConnection extends duckdb_common_1.DuckDBCommon {
         // re-acquire the lock immediately and defeat the point of idling.
     }
     /**
-     * Remove this connection from the shared `activeDBs` entry. When the
-     * last connection sharing the entry is removed, close the underlying
-     * `DuckDBInstance` and drop the entry — releasing DuckDB's per-process
-     * file lock.
+     * Drop our entry from the shared `activeDBs` bookkeeping. If we were
+     * the last sharer, close the underlying `DuckDBInstance`.
+     *
+     * NOTE: this does NOT fully release the OS file lock when other
+     * processes try to open the same path. `duckdb_close` (= `closeSync`)
+     * is a refcount-decrement on `shared_ptr<DatabaseInstance>`; the
+     * `fcntl(F_SETLK)` lock on the storage manager's `UnixFileHandle`
+     * survives until the last `shared_ptr` ref is gone, and live C++
+     * `Connection` objects keep that refcount above zero via their
+     * `ClientContext`. To force the lock release we'd need to also
+     * `disconnectSync()` the per-connection node-api Connection — but
+     * doing so unconditionally (as 0.0.389 did) crashes the language
+     * server because malloy's translate/persistence layer retains
+     * weak_ptrs to the destroyed C++ Connection. See the commented-out
+     * code below and PR #2793 / the revert PR for context.
+     *
+     * Proper fix needs to coordinate disconnect with invalidation of
+     * whatever caches survive idle (manifest reader is the main suspect).
      */
     detachInstance() {
+        // [PR #2793, reverted] Adding `this.connection?.disconnectSync()`
+        // here releases the OS file lock — but it also destroys C++ state
+        // that malloy-internal caches (manifest reader, etc.) hold
+        // weak_ptrs to. Result: SIGSEGV in the language server during
+        // translate, or `bad_weak_ptr` exceptions surfacing as model
+        // problems on next op. Left here as a marker for the proper fix.
+        //
+        // if (this.connection) {
+        //   this.connection.disconnectSync();
+        // }
         const activeDB = DuckDBConnection.activeDBs[this.shareKey];
         if (activeDB) {
             activeDB.connections = activeDB.connections.filter(connection => connection !== this.connection);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@malloydata/db-duckdb",
-  "version": "0.0.388",
+  "version": "0.0.390",
   "license": "MIT",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -60,7 +60,7 @@
   "dependencies": {
     "@duckdb/duckdb-wasm": "1.33.1-dev13.0",
     "@duckdb/node-api": "1.4.4-r.1",
-    "@malloydata/malloy": "0.0.388",
+    "@malloydata/malloy": "0.0.390",
     "@motherduck/wasm-client": "^0.6.6",
     "apache-arrow": "^17.0.0",
     "web-worker": "^1.3.0"