@hyperfixi/patterns-reference 2.7.1 → 2.8.0

package/dist/api/index.js

@@ -1,6 +1,7 @@
 'use strict';
 
 var Database = require('better-sqlite3');
+var fs = require('fs');
 var url = require('url');
 var path = require('path');
 
@@ -16,10 +17,31 @@ var __dirname$1 = path.dirname(__filename2);
 var DEFAULT_DB_PATH = process.env.LSP_DB_PATH || process.env.HYPERSCRIPT_LSP_DB || path.join(__dirname$1, "../data/patterns.db");
 var dbInstance = null;
 var currentDbPath = null;
+var openFileId = null;
+var lastIdentityCheckAt = 0;
+var IDENTITY_TTL_MS = 1e3;
+function fileIdentity(path) {
+  try {
+    const s = fs.statSync(path);
+    return `${s.dev}:${s.ino}:${s.size}:${s.mtimeMs}`;
+  } catch {
+    return null;
+  }
+}
 function getDatabase(options = {}) {
   const dbPath = options.dbPath ?? DEFAULT_DB_PATH;
   if (dbInstance !== null && currentDbPath === dbPath) {
-    return dbInstance;
+    const now = Date.now();
+    if (now - lastIdentityCheckAt < IDENTITY_TTL_MS) {
+      return dbInstance;
+    }
+    lastIdentityCheckAt = now;
+    const identity = fileIdentity(dbPath);
+    if (identity === null || identity === openFileId) {
+      return dbInstance;
+    }
+    dbInstance.close();
+    dbInstance = null;
   }
   if (dbInstance !== null) {
     dbInstance.close();
@@ -29,6 +51,8 @@ function getDatabase(options = {}) {
   });
   dbInstance.pragma("foreign_keys = ON");
   currentDbPath = dbPath;
+  openFileId = fileIdentity(dbPath);
+  lastIdentityCheckAt = Date.now();
   return dbInstance;
 }
 

package/dist/api/index.mjs

@@ -1,4 +1,5 @@
 import Database from 'better-sqlite3';
+import { statSync } from 'fs';
 import { fileURLToPath } from 'url';
 import { dirname, join } from 'path';
 
@@ -8,10 +9,31 @@ var __dirname$1 = dirname(__filename$1);
 var DEFAULT_DB_PATH = process.env.LSP_DB_PATH || process.env.HYPERSCRIPT_LSP_DB || join(__dirname$1, "../data/patterns.db");
 var dbInstance = null;
 var currentDbPath = null;
+var openFileId = null;
+var lastIdentityCheckAt = 0;
+var IDENTITY_TTL_MS = 1e3;
+function fileIdentity(path) {
+  try {
+    const s = statSync(path);
+    return `${s.dev}:${s.ino}:${s.size}:${s.mtimeMs}`;
+  } catch {
+    return null;
+  }
+}
 function getDatabase(options = {}) {
   const dbPath = options.dbPath ?? DEFAULT_DB_PATH;
   if (dbInstance !== null && currentDbPath === dbPath) {
-    return dbInstance;
+    const now = Date.now();
+    if (now - lastIdentityCheckAt < IDENTITY_TTL_MS) {
+      return dbInstance;
+    }
+    lastIdentityCheckAt = now;
+    const identity = fileIdentity(dbPath);
+    if (identity === null || identity === openFileId) {
+      return dbInstance;
+    }
+    dbInstance.close();
+    dbInstance = null;
   }
   if (dbInstance !== null) {
     dbInstance.close();
@@ -21,6 +43,8 @@ function getDatabase(options = {}) {
   });
   dbInstance.pragma("foreign_keys = ON");
   currentDbPath = dbPath;
+  openFileId = fileIdentity(dbPath);
+  lastIdentityCheckAt = Date.now();
   return dbInstance;
 }
 

package/dist/index.js

@@ -1,10 +1,10 @@
 'use strict';
 
 var Database = require('better-sqlite3');
+var fs = require('fs');
 var url = require('url');
 var path = require('path');
 var crypto = require('crypto');
-var fs = require('fs');
 
 function _interopDefault (e) { return e && e.__esModule ? e : { default: e }; }
 
@@ -18,13 +18,34 @@ var __dirname$1 = path.dirname(__filename2);
 var DEFAULT_DB_PATH = process.env.LSP_DB_PATH || process.env.HYPERSCRIPT_LSP_DB || path.join(__dirname$1, "../data/patterns.db");
 var dbInstance = null;
 var currentDbPath = null;
+var openFileId = null;
+var lastIdentityCheckAt = 0;
+var IDENTITY_TTL_MS = 1e3;
+function fileIdentity(path) {
+  try {
+    const s = fs.statSync(path);
+    return `${s.dev}:${s.ino}:${s.size}:${s.mtimeMs}`;
+  } catch {
+    return null;
+  }
+}
 function getDefaultDbPath() {
   return DEFAULT_DB_PATH;
 }
 function getDatabase(options = {}) {
   const dbPath = options.dbPath ?? DEFAULT_DB_PATH;
   if (dbInstance !== null && currentDbPath === dbPath) {
-    return dbInstance;
+    const now = Date.now();
+    if (now - lastIdentityCheckAt < IDENTITY_TTL_MS) {
+      return dbInstance;
+    }
+    lastIdentityCheckAt = now;
+    const identity = fileIdentity(dbPath);
+    if (identity === null || identity === openFileId) {
+      return dbInstance;
+    }
+    dbInstance.close();
+    dbInstance = null;
   }
   if (dbInstance !== null) {
     dbInstance.close();
@@ -34,6 +55,8 @@ function getDatabase(options = {}) {
   });
   dbInstance.pragma("foreign_keys = ON");
   currentDbPath = dbPath;
+  openFileId = fileIdentity(dbPath);
+  lastIdentityCheckAt = Date.now();
   return dbInstance;
 }
 function closeDatabase() {
@@ -41,11 +64,15 @@ function closeDatabase() {
     dbInstance.close();
     dbInstance = null;
     currentDbPath = null;
+    openFileId = null;
+    lastIdentityCheckAt = 0;
   }
 }
 function resetConnection() {
   dbInstance = null;
   currentDbPath = null;
+  openFileId = null;
+  lastIdentityCheckAt = 0;
 }
 function isConnected() {
   return dbInstance !== null;

package/dist/index.mjs

@@ -1,8 +1,8 @@
 import Database from 'better-sqlite3';
+import { readFileSync, writeFileSync, existsSync, statSync, readdirSync } from 'fs';
 import { fileURLToPath } from 'url';
 import { dirname, join, relative, sep, resolve } from 'path';
 import { createHash } from 'crypto';
-import { readFileSync, writeFileSync, existsSync, readdirSync, statSync } from 'fs';
 
 // src/database/connection.ts
 var __filename$1 = fileURLToPath(import.meta.url);
@@ -10,13 +10,34 @@ var __dirname$1 = dirname(__filename$1);
 var DEFAULT_DB_PATH = process.env.LSP_DB_PATH || process.env.HYPERSCRIPT_LSP_DB || join(__dirname$1, "../data/patterns.db");
 var dbInstance = null;
 var currentDbPath = null;
+var openFileId = null;
+var lastIdentityCheckAt = 0;
+var IDENTITY_TTL_MS = 1e3;
+function fileIdentity(path) {
+  try {
+    const s = statSync(path);
+    return `${s.dev}:${s.ino}:${s.size}:${s.mtimeMs}`;
+  } catch {
+    return null;
+  }
+}
 function getDefaultDbPath() {
   return DEFAULT_DB_PATH;
 }
 function getDatabase(options = {}) {
   const dbPath = options.dbPath ?? DEFAULT_DB_PATH;
   if (dbInstance !== null && currentDbPath === dbPath) {
-    return dbInstance;
+    const now = Date.now();
+    if (now - lastIdentityCheckAt < IDENTITY_TTL_MS) {
+      return dbInstance;
+    }
+    lastIdentityCheckAt = now;
+    const identity = fileIdentity(dbPath);
+    if (identity === null || identity === openFileId) {
+      return dbInstance;
+    }
+    dbInstance.close();
+    dbInstance = null;
   }
   if (dbInstance !== null) {
     dbInstance.close();
@@ -26,6 +47,8 @@ function getDatabase(options = {}) {
   });
   dbInstance.pragma("foreign_keys = ON");
   currentDbPath = dbPath;
+  openFileId = fileIdentity(dbPath);
+  lastIdentityCheckAt = Date.now();
   return dbInstance;
 }
 function closeDatabase() {
@@ -33,11 +56,15 @@ function closeDatabase() {
     dbInstance.close();
     dbInstance = null;
     currentDbPath = null;
+    openFileId = null;
+    lastIdentityCheckAt = 0;
   }
 }
 function resetConnection() {
   dbInstance = null;
   currentDbPath = null;
+  openFileId = null;
+  lastIdentityCheckAt = 0;
 }
 function isConnected() {
   return dbInstance !== null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hyperfixi/patterns-reference",
-  "version": "2.7.1",
+  "version": "2.8.0",
   "description": "Queryable patterns database for hyperscript with multilingual translations and LLM support",
   "main": "dist/index.js",
   "module": "dist/index.mjs",
@@ -66,19 +66,19 @@
   },
   "homepage": "https://github.com/codetalcott/hyperfixi/tree/main/packages/patterns-reference#readme",
   "dependencies": {
-    "@lokascript/semantic": "^2.7.1",
-    "better-sqlite3": "^12.6.2"
+    "@lokascript/semantic": "^2.8.0",
+    "better-sqlite3": "^12.11.1"
   },
   "devDependencies": {
-    "@hyperfixi/core": "^2.7.1",
-    "@hyperfixi/reactivity": "^2.7.1",
-    "@hyperfixi/realtime": "^2.7.1",
+    "@hyperfixi/core": "^2.8.0",
+    "@hyperfixi/reactivity": "^2.8.0",
+    "@hyperfixi/realtime": "^2.8.0",
     "@types/better-sqlite3": "^7.6.0",
     "@types/node": "^20.0.0",
     "hyperscript.org": "0.9.93",
     "jsdom": "^26.0.0",
     "tsup": "^8.0.0",
-    "tsx": "^4.0.0",
+    "tsx": "^4.23.1",
     "typescript": "^5.0.0",
     "vitest": "^4.1.5"
   },

package/src/database/connection.ts

@@ -5,6 +5,7 @@
  */
 
 import Database from 'better-sqlite3';
+import { statSync } from 'fs';
 import { fileURLToPath } from 'url';
 import { dirname, join } from 'path';
 import type { ConnectionOptions } from '../types';
@@ -24,6 +25,30 @@ const DEFAULT_DB_PATH =
 let dbInstance: InstanceType<typeof Database> | null = null;
 let currentDbPath: string | null = null;
 
+/**
+ * Identity of the FILE the open handle is reading, captured when it was opened.
+ *
+ * `npm run populate` REPLACES patterns.db rather than mutating it, so a long-lived handle
+ * (the MCP server's, most notably) keeps reading the old inode: every query answers from
+ * the pre-populate database while the file on disk says otherwise. Path equality cannot
+ * see this — the path is identical; only the inode moved.
+ */
+let openFileId: string | null = null;
+let lastIdentityCheckAt = 0;
+
+/** Re-stat at most this often, so a bulk query loop pays one syscall, not thousands. */
+const IDENTITY_TTL_MS = 1000;
+
+/** inode+device+size+mtime — changes iff the file was replaced or rewritten. */
+function fileIdentity(path: string): string | null {
+  try {
+    const s = statSync(path);
+    return `${s.dev}:${s.ino}:${s.size}:${s.mtimeMs}`;
+  } catch {
+    return null; // mid-replace: treat as unknown, don't thrash the handle
+  }
+}
+
 /**
  * The resolved default DB path (honouring `LSP_DB_PATH` / `HYPERSCRIPT_LSP_DB`).
  * Exposed so callers (e.g. the multilingual gate's freshness check) can target the
@@ -39,9 +64,23 @@ export function getDefaultDbPath(): string {
 export function getDatabase(options: ConnectionOptions = {}): InstanceType<typeof Database> {
   const dbPath = options.dbPath ?? DEFAULT_DB_PATH;
 
-  // Return existing connection if same path
+  // Reuse the open handle only if it is reading the same PATH *and* the same FILE. The
+  // second half matters because `populate` swaps the file underneath us: without it a
+  // long-lived consumer serves pre-populate rows indefinitely, which looks like a parser
+  // bug and is not one. Behind a TTL so a query loop doesn't stat per row.
   if (dbInstance !== null && currentDbPath === dbPath) {
-    return dbInstance;
+    const now = Date.now();
+    if (now - lastIdentityCheckAt < IDENTITY_TTL_MS) {
+      return dbInstance;
+    }
+    lastIdentityCheckAt = now;
+    const identity = fileIdentity(dbPath);
+    if (identity === null || identity === openFileId) {
+      return dbInstance;
+    }
+    // The file was replaced — reopen so callers see the DB that is actually on disk.
+    dbInstance.close();
+    dbInstance = null;
   }
 
   // Close existing connection if path changed
@@ -57,6 +96,8 @@ export function getDatabase(options: ConnectionOptions = {}): InstanceType<typeo
   dbInstance.pragma('foreign_keys = ON');
 
   currentDbPath = dbPath;
+  openFileId = fileIdentity(dbPath);
+  lastIdentityCheckAt = Date.now();
   return dbInstance;
 }
 
@@ -68,6 +109,8 @@ export function closeDatabase(): void {
     dbInstance.close();
     dbInstance = null;
     currentDbPath = null;
+    openFileId = null;
+    lastIdentityCheckAt = 0;
   }
 }
 
@@ -77,6 +120,8 @@ export function closeDatabase(): void {
 export function resetConnection(): void {
   dbInstance = null;
   currentDbPath = null;
+  openFileId = null;
+  lastIdentityCheckAt = 0;
 }
 
 /**