resplite 1.1.8 → 1.1.12

package/package.json

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

package/spec/SPEC_D.md

@@ -160,7 +160,7 @@ Exact fields may evolve, but these should exist.
 
 ## D.5.3 Errors
 
-* Missing index: `-ERR unknown index`
+* Missing index: `-Unknown index name`
 
 ---
 
@@ -212,7 +212,7 @@ RESP reply: `+OK`
 
 ## D.6.6 Errors
 
-* Index missing: `-ERR unknown index`
+* Index missing: `-Unknown index name`
 * Wrong syntax: `-ERR syntax error`
 * Missing `FIELDS`: `-ERR syntax error`
 * Unknown field: `-ERR unknown field`
@@ -248,7 +248,7 @@ RESP Integer:
 
 ## D.7.4 Errors
 
-* Index missing: `-ERR unknown index`
+* Index missing: `-Unknown index name`
 
 ---
 
@@ -346,7 +346,7 @@ Compute total efficiently:
 
 ## D.8.8 Errors
 
-* Index missing: `-ERR unknown index`
+* Index missing: `-Unknown index name`
 * Bad syntax: `-ERR syntax error`
 * Invalid LIMIT: `-ERR invalid limit`
 * Unsafe query: `-ERR invalid query`
@@ -383,7 +383,7 @@ RESP Integer:
 
 ### Errors
 
-* Index missing: `-ERR unknown index`
+* Index missing: `-Unknown index name`
 * Score invalid: `-ERR invalid score`
 
 ## D.9.2 FT.SUGGET
@@ -425,7 +425,7 @@ Not supported (v1):
 
 ### Errors
 
-* Index missing: `-ERR unknown index`
+* Index missing: `-Unknown index name`
 * Bad syntax: `-ERR syntax error`
 
 ## D.9.3 FT.SUGDEL
@@ -449,7 +449,7 @@ RESP Integer:
 
 ### Errors
 
-* Index missing: `-ERR unknown index`
+* Index missing: `-Unknown index name`
 
 ---
 

package/src/commands/ft-add.js

@@ -20,6 +20,6 @@ export function handleFtAdd(engine, args) {
     return { simple: 'OK' };
   } catch (e) {
     const msg = e?.message ?? String(e);
-    return { error: msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
+    return { error: msg === 'Unknown index name' ? msg : msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
   }
 }

package/src/commands/ft-create.js

@@ -15,6 +15,6 @@ export function handleFtCreate(engine, args) {
     return { simple: 'OK' };
   } catch (e) {
     const msg = e?.message ?? String(e);
-    return { error: msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
+    return { error: msg === 'Unknown index name' ? msg : msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
   }
 }

package/src/commands/ft-del.js

@@ -15,6 +15,6 @@ export function handleFtDel(engine, args) {
     return n;
   } catch (e) {
     const msg = e?.message ?? String(e);
-    return { error: msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
+    return { error: msg === 'Unknown index name' ? msg : msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
   }
 }

package/src/commands/ft-info.js

@@ -32,6 +32,6 @@ export function handleFtInfo(engine, args) {
     ];
   } catch (e) {
     const msg = e?.message ?? String(e);
-    return { error: msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
+    return { error: msg === 'Unknown index name' ? msg : msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
   }
 }

package/src/commands/ft-search.js

@@ -33,6 +33,6 @@ export function handleFtSearch(engine, args) {
     return out;
   } catch (e) {
     const msg = e?.message ?? String(e);
-    return { error: msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
+    return { error: msg === 'Unknown index name' ? msg : msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
   }
 }

package/src/commands/ft-sugadd.js

@@ -22,6 +22,6 @@ export function handleFtSugadd(engine, args) {
     return n;
   } catch (e) {
     const msg = e?.message ?? String(e);
-    return { error: msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
+    return { error: msg === 'Unknown index name' ? msg : msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
   }
 }

package/src/commands/ft-sugdel.js

@@ -15,6 +15,6 @@ export function handleFtSugdel(engine, args) {
     return n;
   } catch (e) {
     const msg = e?.message ?? String(e);
-    return { error: msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
+    return { error: msg === 'Unknown index name' ? msg : msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
   }
 }

package/src/commands/ft-sugget.js

@@ -19,6 +19,6 @@ export function handleFtSugget(engine, args) {
     return list;
   } catch (e) {
     const msg = e?.message ?? String(e);
-    return { error: msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
+    return { error: msg === 'Unknown index name' ? msg : msg.startsWith('ERR ') ? msg : 'ERR ' + msg };
   }
 }

package/src/embed.js

@@ -34,11 +34,34 @@ export async function createRESPlite({
 } = {}) {
   const db = openDb(dbPath, { pragmaTemplate });
   const engine = createEngine({ db });
-  const server = net.createServer((socket) => handleConnection(socket, engine));
+  const connections = new Set();
+
+  const server = net.createServer((socket) => {
+    connections.add(socket);
+    socket.once('close', () => connections.delete(socket));
+    handleConnection(socket, engine);
+  });
   await new Promise((resolve) => server.listen(port, host, resolve));
+
+  let closePromise = null;
+  const close = () => {
+    if (closePromise) return closePromise;
+    closePromise = new Promise((resolve) => {
+      for (const socket of connections) {
+        socket.destroy();
+      }
+      connections.clear();
+      server.close(() => {
+        db.close();
+        resolve();
+      });
+    });
+    return closePromise;
+  };
+
   return {
     port: server.address().port,
     host,
-    close: () => new Promise((resolve) => server.close(() => { db.close(); resolve(); })),
+    close,
   };
 }

package/src/storage/sqlite/search.js

@@ -116,7 +116,7 @@ export function getIndexMeta(db, name) {
   const row = db.prepare(
     'SELECT name, schema_json, created_at, updated_at FROM search_indices WHERE name = ?'
   ).get(name);
-  if (!row) throw new Error('ERR unknown index');
+  if (!row) throw new Error('Unknown index name');
   const schema = JSON.parse(row.schema_json);
   return {
     name: row.name,

package/test/integration/search.test.js

@@ -117,7 +117,7 @@ describe('Search integration', () => {
     const reply = await sendCommand(port, argv('FT.INFO', 'nonexistent'));
     const v = tryParseValue(reply, 0).value;
     const err = v?.error ?? v;
-    assert.ok(String(err).includes('unknown index'));
+    assert.ok(String(err).includes('Unknown index name'));
   });
 });
 

package/test/unit/search.test.js

@@ -135,6 +135,6 @@ describe('Search layer', () => {
   });
 
   it('getIndexMeta throws for unknown index', () => {
-    assert.throws(() => getIndexMeta(db, 'nonexistent'), /unknown index/);
+    assert.throws(() => getIndexMeta(db, 'nonexistent'), /Unknown index name/);
   });
 });

package/tasks/todo.md

@@ -1,63 +0,0 @@
-# Migration from Redis (SPEC §26)
-
-## Plan
-
-- [x] Review SPEC §26 (Migration Strategy) and §10 (Data model)
-- [x] Implement external CLI tool (not part of RESP server)
-- [x] Method: connect to Redis → SCAN → TYPE → fetch by type (GET / HGETALL / SMEMBERS) → PTTL → write to SQLite
-- [x] Migratable subset: strings, hashes, sets, TTL metadata
-- [x] Skip unsupported types (list, zset, stream, etc.) with report
-- [x] Contract test using local Redis (skip if Redis unavailable)
-
-## Implementation
-
-- **CLI**: `src/cli/import-from-redis.js` — `node src/cli/import-from-redis.js --redis-url redis://127.0.0.1:6379 --db ./migrated.db`
-- **Storage**: Reuse existing `openDb`, `createKeysStorage`, `createStringsStorage`, `createHashesStorage`, `createSetsStorage`; write via storage layer with Buffer values
-- **Binary safety**: Redis client returns strings; coerce to Buffer (utf8) when writing to SQLite
-
-## Verification
-
-- Run contract test: `npm run test:contract` (includes import-from-redis test when Redis is available)
-- Manual: populate local Redis, run CLI, start RESPlite with migrated db, verify keys/values/TTL
-
-## Not in scope (SPEC §26.3)
-
-- RDB parsing, AOF parsing, mirror mode, dual-write
-
----
-
-# Migration with Dirty Key Registry (SPEC_F)
-
-## Done
-
-- [x] Migration schema: `migration_runs`, `migration_dirty_keys`, `migration_errors` in `src/storage/sqlite/migration-schema.js`
-- [x] Registry layer: `src/migration/registry.js` (createRun, getRun, updateBulkProgress, upsertDirtyKey, getDirtyBatch, markDirtyState, logError, getDirtyCounts)
-- [x] Bulk importer: `src/migration/bulk.js` with run_id, checkpointing, resume, max_rps, batch_keys/batch_bytes, pause/abort via status
-- [x] Shared import-one: `src/migration/import-one.js` (fetch key from Redis + write to storages; used by bulk and apply-dirty)
-- [x] Delta apply: `src/migration/apply-dirty.js` (apply dirty keys from registry: reimport or delete in destination)
-- [x] Preflight: `src/migration/preflight.js` (key count, type distribution, notify-keyspace-events check, recommendations)
-- [x] Verify: `src/migration/verify.js` (sample keys, compare Redis vs RespLite)
-- [x] CLI `resplite-import`: `src/cli/resplite-import.js` (preflight, bulk, status, apply-dirty, verify)
-- [x] CLI `resplite-dirty-tracker`: `src/cli/resplite-dirty-tracker.js` (start = PSUBSCRIBE keyevent, stop = update run status)
-- [x] package.json `bin`: resplite-import, resplite-dirty-tracker
-- [x] Unit tests: `test/unit/migration-registry.test.js`
-- [x] README: minimal-downtime migration flow and commands
-
----
-
-# KEYS + MONITOR Commands
-
-## Plan
-
-- [x] Add `KEYS pattern` command with glob matching support
-- [x] Add `MONITOR` command and streaming monitor bus in TCP server
-- [x] Register commands in command registry
-- [x] Add integration tests for both commands
-- [x] Verify with targeted and full integration suites
-
-## Review
-
-- Added `KEYS` command with wildcard matching (`*`, `?`) and proper arity validation
-- Added server-side monitor broadcasting for `MONITOR` clients and restricted monitor-mode commands to `QUIT`
-- Added integration coverage in `test/integration/keys.test.js` and `test/integration/monitor.test.js`
-- Verified with `npm test -- test/integration/keys.test.js`, `npm test -- test/integration/monitor.test.js`, and `npm run test:integration`