Fix review findings: beforeExit handler, process-exit docs

Domain expert review found TS MCP had no safety net for event loop
drain or editor disconnect without signal. Added beforeExit + exit
handlers with idempotency guard (_hasWrapped flag). Updated
CLAUDE.md.snippet to mention process-exit consolidation.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: phillipclapham

examples/CLAUDE.md.snippet

@@ -37,6 +37,9 @@ like sleep for memory. It prunes dormant nodes to the audit trail, graduates
 frequently-used knowledge through temporal tiers (current → developing →
 proven → foundation), and saves. Without regular wraps, knowledge
 accumulates as noise instead of maturing. Each wrap makes the graph smarter.
+NOTE: A final consolidation also runs automatically on process exit (editor
+close, signal), but explicit wraps produce the best results because you can
+review and store final reasoning before consolidation happens.
 
 Queries keep knowledge alive — every `query_tensions`, `query_blocked`, etc.
 touches the returned nodes, driving graduation. Knowledge that keeps coming

src/mcp-server.ts

@@ -146,14 +146,19 @@ const memory = isDemo ? seedDemoMemory(memoryPath) : Memory.loadOrCreate(memoryP
 
 // Graceful shutdown: attempt a full session wrap (consolidation) on exit.
 // Falls back to save-only if wrap throws (crash safety > data loss).
+let _hasWrapped = false;
 function gracefulShutdown() {
+  if (_hasWrapped) return;
+  _hasWrapped = true;
   try { memory.sessionWrap(); } catch {
     try { memory.save(); } catch { /* best effort */ }
   }
-  process.exit(0);
 }
-process.on('SIGTERM', gracefulShutdown);
-process.on('SIGINT', gracefulShutdown);
+process.on('SIGTERM', () => { gracefulShutdown(); process.exit(0); });
+process.on('SIGINT', () => { gracefulShutdown(); process.exit(0); });
+// Safety net for natural event loop drain or editor disconnect without signal
+process.on('beforeExit', gracefulShutdown);
+process.on('exit', gracefulShutdown);
 
 // Helper: validate required args exist
 function requireArgs(args: Record<string, unknown> | undefined, ...keys: string[]): string | null {