sqlite-zod-orm 3.7.2 → 3.7.3

package/dist/index.js

@@ -361,18 +361,25 @@ class QueryBuilder {
   subscribe(callback, options = {}) {
     const { interval = this.defaultPollInterval, immediate = true } = options;
     const fingerprintSQL = this.buildFingerprintSQL();
-    let lastFingerprint = null;
+    let lastCount = null;
+    let lastMax = null;
+    let lastInMemoryRev = null;
     let stopped = false;
     const poll = async () => {
       if (stopped)
         return;
       try {
+        const rev = this.revisionGetter?.() ?? "0";
+        const inMemoryChanged = rev !== lastInMemoryRev;
+        lastInMemoryRev = rev;
         const fpRows = this.executor(fingerprintSQL.sql, fingerprintSQL.params, true);
         const fpRow = fpRows[0];
-        const rev = this.revisionGetter?.() ?? "0";
-        const currentFingerprint = `${fpRow?._cnt ?? 0}:${fpRow?._max ?? 0}:${rev}`;
-        if (currentFingerprint !== lastFingerprint) {
-          lastFingerprint = currentFingerprint;
+        const cnt = fpRow?._cnt ?? 0;
+        const max = fpRow?._max ?? 0;
+        const fpChanged = cnt !== lastCount || max !== lastMax;
+        lastCount = cnt;
+        lastMax = max;
+        if (inMemoryChanged || fpChanged) {
           const rows = this.all();
           await callback(rows);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sqlite-zod-orm",
-  "version": "3.7.2",
+  "version": "3.7.3",
   "description": "Type-safe SQLite ORM for Bun — Zod schemas, fluent queries, auto relationships, zero SQL",
   "type": "module",
   "main": "./dist/index.js",

package/src/query-builder.ts

@@ -507,23 +507,38 @@ export class QueryBuilder<T extends Record<string, any>> {
 
         // Build the fingerprint SQL (COUNT + MAX(id)) using the same WHERE
         const fingerprintSQL = this.buildFingerprintSQL();
-        let lastFingerprint: string | null = null;
+        let lastCount: number | null = null;
+        let lastMax: number | null = null;
+        let lastInMemoryRev: string | null = null;
         let stopped = false;
 
         const poll = async () => {
             if (stopped) return;
             try {
-                // Run lightweight fingerprint check
-                const fpRows = this.executor(fingerprintSQL.sql, fingerprintSQL.params, true);
-                const fpRow = fpRows[0] as any;
-                // Include revision in fingerprint (combines in-memory counter + PRAGMA data_version).
-                // This detects ALL changes: same-process and cross-process.
+                // Two-signal change detection:
+                //   1. In-memory revision (table-specific) → catches same-process writes
+                //   2. COUNT+MAX fingerprint (table-specific) → catches cross-process inserts/deletes
+                //
+                // Note: cross-process UPDATEs that don't change count/max are only caught
+                // by PRAGMA data_version, which is database-wide. We accept this tradeoff
+                // to avoid re-querying on writes to OTHER tables.
                 const rev = this.revisionGetter?.() ?? '0';
-                const currentFingerprint = `${fpRow?._cnt ?? 0}:${fpRow?._max ?? 0}:${rev}`;
 
-                if (currentFingerprint !== lastFingerprint) {
-                    lastFingerprint = currentFingerprint;
-                    // Fingerprint changed → re-execute the full query
+                // Fast path: in-memory revision changed → our CRUD wrote to this table
+                const inMemoryChanged = rev !== lastInMemoryRev;
+                lastInMemoryRev = rev;
+
+                // Check table-specific fingerprint (COUNT + MAX(id))
+                const fpRows = this.executor(fingerprintSQL.sql, fingerprintSQL.params, true);
+                const fpRow = fpRows[0] as any;
+                const cnt = fpRow?._cnt ?? 0;
+                const max = fpRow?._max ?? 0;
+                const fpChanged = cnt !== lastCount || max !== lastMax;
+                lastCount = cnt;
+                lastMax = max;
+
+                // Fire callback only if THIS table actually changed
+                if (inMemoryChanged || fpChanged) {
                     const rows = this.all();
                     await callback(rows);
                 }