instar 1.3.201 → 1.3.202

package/dist/commands/server.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"server.d.ts","sourceRoot":"","sources":["../../src/commands/server.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAwSH,UAAU,YAAY;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb;2DACuD;IACvD,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AA+wDD,wBAAsB,WAAW,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,IAAI,CAAC,CAk3PtE;AAED,wBAAsB,UAAU,CAAC,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAsDzE;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CAAC,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAuD5E"}
+{"version":3,"file":"server.d.ts","sourceRoot":"","sources":["../../src/commands/server.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAySH,UAAU,YAAY;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb;2DACuD;IACvD,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AA+wDD,wBAAsB,WAAW,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,IAAI,CAAC,CAi4PtE;AAED,wBAAsB,UAAU,CAAC,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAsDzE;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CAAC,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAuD5E"}

package/dist/commands/server.js

@@ -16,6 +16,7 @@ import pc from 'picocolors';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 import { loadConfig, ensureStateDir, detectTmuxPath } from '../core/Config.js';
 import { isNonFatalUncaught } from '../core/uncaughtExceptionPolicy.js';
+import { closeAllSqlite } from '../core/SqliteRegistry.js';
 import { SessionManager } from '../core/SessionManager.js';
 import { StateManager } from '../core/StateManager.js';
 import { StuckInputSentinel } from '../core/StuckInputSentinel.js';
@@ -9392,7 +9393,14 @@ export async function startServer(options) {
             // @silent-fallback-ok — upgrade guide check non-critical
         }
         // Graceful shutdown
+        let _shuttingDown = false;
         const shutdown = async () => {
+            // Re-entrancy guard: SIGINT+SIGTERM (or a restartDetected racing a signal)
+            // must not run the teardown twice. closeAllSqlite() is itself idempotent,
+            // but the resume-UUID save + sidecar flush should run once.
+            if (_shuttingDown)
+                return;
+            _shuttingDown = true;
             console.log('\nShutting down...');
             // Save resume UUIDs for ALL active topic-linked sessions before exit.
             // Without this, server restarts lose all resume mappings because:
@@ -9473,16 +9481,24 @@ export async function startServer(options) {
                 await telegram.stop();
             sessionManager.stopMonitoring();
             stuckInputSentinel.stop();
-            // Close SQLite databases before exit — prevents "mutex lock failed" crash
-            // when better-sqlite3 destructors fire during process teardown.
-            topicMemory?.close();
-            semanticMemory?.close();
-            // Integrated-Being v1 — flush stats sidecar (coalesces pending writes).
+            // Integrated-Being v1 — flush stats sidecar (coalesces pending writes)
+            // BEFORE closing SQLite, so no unflushed write is lost.
             try {
                 sharedStateLedger?.shutdown();
             }
             catch { /* best effort */ }
             await server.stop();
+            // Close EVERY registered SQLite handle LAST — after all writers (server,
+            // scheduler, sentinels, telegram) have stopped — to prevent the
+            // "mutex lock failed" SIGABRT when better-sqlite3 static destructors fire
+            // during process teardown. The structural registry (SqliteRegistry.ts)
+            // replaces the old hand-maintained topicMemory/semanticMemory close-list,
+            // which covered only 2 of the 15 long-lived stores.
+            try {
+                const closed = closeAllSqlite();
+                console.log(`[shutdown] closed ${closed} sqlite handle(s)`);
+            }
+            catch { /* best effort */ }
             process.exit(0);
         };
         process.on('SIGINT', shutdown);
@@ -9501,12 +9517,11 @@ export async function startServer(options) {
                 return; // Don't crash — the server is fine
             }
             console.error('[FATAL] Uncaught exception — closing databases before crash:', err.message);
+            // Close ALL registered SQLite handles (not just topic/semantic memory) so
+            // the crash exit doesn't compound into a "mutex lock failed" SIGABRT on top
+            // of the original error. closeAllSqlite() is best-effort + idempotent.
             try {
-                topicMemory?.close();
-            }
-            catch { /* best effort */ }
-            try {
-                semanticMemory?.close();
+                closeAllSqlite();
             }
             catch { /* best effort */ }
             process.exit(1);