dsh-plugin-codegraph-sqlite 0.6.3 → 0.6.4

package/lib/database.d.ts

@@ -61,6 +61,14 @@ export declare class GraphPool {
      * @returns the open, version-checked connection, current as of this call.
      */
     acquire(projectRoot: string): DatabaseSync;
+    /**
+     * Drop the cached connection for one root, so a re-index may replace its file.
+     *
+     * A connection kept past the query is what breaks a Windows re-index: the writer
+     * replaces the graph with rename(), and Windows refuses to rename over a file that
+     * is open, while the design above assumes POSIX unlink semantics.
+     */
+    forget(projectRoot: string): void;
     /** Close every open connection; further {@link acquire} calls fail as `CODEGRAPH_DISPOSED`. */
     close(): void;
     /** Close least-recently-used connections until the pool fits its capacity. */

package/lib/database.js

@@ -151,6 +151,27 @@ export class GraphPool {
         this.evictOverflow();
         return db;
     }
+    /**
+     * Drop the cached connection for one root, so a re-index may replace its file.
+     *
+     * A connection kept past the query is what breaks a Windows re-index: the writer
+     * replaces the graph with rename(), and Windows refuses to rename over a file that
+     * is open, while the design above assumes POSIX unlink semantics.
+     */
+    forget(projectRoot) {
+        const entry = this.open.get(projectRoot);
+        if (entry === undefined)
+            return;
+        // Tolerate a connection the caller already closed: the store closes it first and
+        // then forgets it, and node:sqlite throws "database is not open" on a second close.
+        try {
+            entry.db.close();
+        }
+        catch {
+            // already closed by the caller
+        }
+        this.open.delete(projectRoot);
+    }
     /** Close every open connection; further {@link acquire} calls fail as `CODEGRAPH_DISPOSED`. */
     close() {
         this.disposed = true;

package/lib/index.js

@@ -88,25 +88,39 @@ export function apply(ctx, config) {
  */
 function run(pool, config, request) {
     const db = pool.acquire(request.projectRoot);
-    switch (request.operation) {
-        case 'search':
-            return search(db, request);
-        case 'node':
-            return node(db, request);
-        case 'callers':
-        case 'callees':
-            return relations(db, request);
-        case 'impact':
-            return impact(db, request, (origin, depth) => walkImpact(db, origin, depth, config.maxTraversalNodes));
-        case 'trace':
-            return trace(db, request, (from, to, maxDepth, maxPaths) => walkTrace(db, from, to, maxDepth, maxPaths, config.maxTraversalNodes));
-        case 'files':
-            return files(db, request);
-        case 'status':
-            return status(db, request.projectRoot, config.maxStalenessChecks);
-        /* v8 ignore next -- exhaustive over the seam's closed operation union; unreachable. */
-        default:
-            return assertNever(request, 'codegraph-sqlite request');
+    try {
+        switch (request.operation) {
+            case 'search':
+                return search(db, request);
+            case 'node':
+                return node(db, request);
+            case 'callers':
+            case 'callees':
+                return relations(db, request);
+            case 'impact':
+                return impact(db, request, (origin, depth) => walkImpact(db, origin, depth, config.maxTraversalNodes));
+            case 'trace':
+                return trace(db, request, (from, to, maxDepth, maxPaths) => walkTrace(db, from, to, maxDepth, maxPaths, config.maxTraversalNodes));
+            case 'files':
+                return files(db, request);
+            case 'status':
+                return status(db, request.projectRoot, config.maxStalenessChecks);
+            /* v8 ignore next -- exhaustive over the seam's closed operation union; unreachable. */
+            default:
+                return assertNever(request, 'codegraph-sqlite request');
+        }
+    }
+    finally {
+        // Every operation above is synchronous, so the connection goes back the moment the
+        // answer exists. That is what fixes the Windows EPERM: the writer may only replace
+        // the file when nobody has it open.
+        try {
+            db.close();
+        }
+        catch {
+            // already closed
+        }
+        pool.forget(request.projectRoot);
     }
 }
 /** Reject a non-positive-integer config value at load, so misconfiguration fails loud. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin-codegraph-sqlite",
-  "version": "0.6.3",
+  "version": "0.6.4",
   "description": "Read-only SQLite store for the DeepSeek Harness code-graph seam — serves structural queries from the schema-v4 graph at .codegraph/codegraph.db, the same on-disk format the codegraph CLI writes",
   "keywords": [
     "dsh",
@@ -47,7 +47,7 @@
   },
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "dsh-plugin-codegraph-service": "^0.6.3"
+    "dsh-plugin-codegraph-service": "^0.6.4"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": ">=4.0.1",