resplite 1.4.12 → 1.4.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "resplite",
-  "version": "1.4.12",
+  "version": "1.4.14",
   "description": "A RESP2 server with practical Redis compatibility, backed by SQLite",
   "type": "module",
   "main": "src/index.js",

package/src/storage/sqlite/search.js

@@ -154,7 +154,11 @@ export function addDocument(db, idx, docId, score, replace, fields) {
     let ftsRowid;
     if (existing) {
       if (!replace) throw new Error('ERR document exists');
-      ftsRowid = existing.fts_rowid;
+      // FTS5 contentless bug: INSERT OR REPLACE doesn't remove old tokens.
+      // Solution: assign a new fts_rowid to avoid token pollution.
+      const maxRow = db.prepare(`SELECT COALESCE(MAX(fts_rowid), 0) AS m FROM ${docmapT}`).get();
+      ftsRowid = maxRow.m + 1;
+      db.prepare(`UPDATE ${docmapT} SET fts_rowid = ? WHERE doc_id = ?`).run(ftsRowid, docId);
     } else {
       const maxRow = db.prepare(`SELECT COALESCE(MAX(fts_rowid), 0) AS m FROM ${docmapT}`).get();
       ftsRowid = maxRow.m + 1;
@@ -172,12 +176,12 @@ export function addDocument(db, idx, docId, score, replace, fields) {
       ).run(docId, score, fieldsJson, now, now);
     }
 
-    // FTS5 contentless: cannot DELETE; use INSERT OR REPLACE to overwrite by rowid when replacing.
+    // FTS5 contentless: insert with new rowid (old rowid becomes orphaned and won't match via docmap join).
     const ftsColumns = ['rowid', ...fieldNames.sort()];
     const ftsValues = [ftsRowid, ...fieldNames.sort().map((f) => fields[f] ?? '')];
     const placeholders = ftsValues.map(() => '?').join(', ');
     const colList = ftsColumns.join(', ');
-    db.prepare(`INSERT OR REPLACE INTO ${ftsT}(${colList}) VALUES (${placeholders})`).run(...ftsValues);
+    db.prepare(`INSERT INTO ${ftsT}(${colList}) VALUES (${placeholders})`).run(...ftsValues);
   });
 
   run();

package/test/integration/search.test.js

@@ -196,6 +196,40 @@ describe('Search integration', () => {
     const err = v?.error ?? v;
     assert.ok(String(err).includes('Unknown index name'));
   });
+
+  it('FT.SEARCH prefix query should not match unrelated terms', async () => {
+    // Create a fresh index for this test
+    await sendCommand(port, argv('FT.CREATE', 'bugtest', 'SCHEMA', 'payload', 'TEXT'));
+    
+    // Add document with "gorge" first
+    await sendCommand(port, argv('FT.ADD', 'bugtest', 'DOC1', '1', 'REPLACE', 'FIELDS', 'payload', 'gorge'));
+    
+    // Verify it matches gorge*
+    const gorge1 = await sendCommand(port, argv('FT.SEARCH', 'bugtest', 'gorge*', 'NOCONTENT', 'LIMIT', '0', '10'));
+    const g1 = tryParseValue(gorge1, 0).value;
+    assert.equal(g1[0], 1, 'gorge* should match gorge');
+    assert.equal(g1[1].toString?.('utf8') ?? g1[1], 'DOC1');
+    
+    // Replace with "martan"
+    await sendCommand(port, argv('FT.ADD', 'bugtest', 'DOC1', '1', 'REPLACE', 'FIELDS', 'payload', 'martan'));
+    
+    // Verify FT.GET shows only martan
+    const get = await sendCommand(port, argv('FT.GET', 'bugtest', 'DOC1'));
+    const getArr = tryParseValue(get, 0).value;
+    assert.equal(getArr[0].toString?.('utf8') ?? getArr[0], 'payload');
+    assert.equal(getArr[1].toString?.('utf8') ?? getArr[1], 'martan');
+    
+    // martan* should match
+    const martan = await sendCommand(port, argv('FT.SEARCH', 'bugtest', 'martan*', 'NOCONTENT', 'LIMIT', '0', '10'));
+    const m = tryParseValue(martan, 0).value;
+    assert.equal(m[0], 1, 'martan* should match martan');
+    assert.equal(m[1].toString?.('utf8') ?? m[1], 'DOC1');
+    
+    // gorge* should NOT match martan
+    const gorge2 = await sendCommand(port, argv('FT.SEARCH', 'bugtest', 'gorge*', 'NOCONTENT', 'LIMIT', '0', '10'));
+    const g2 = tryParseValue(gorge2, 0).value;
+    assert.equal(g2[0], 0, 'gorge* should NOT match martan - this is the bug');
+  });
 });
 
 describe('Search persistence', () => {