@modusensus/dsh-mneme 0.1.2 → 0.1.3

package/README.md

@@ -3,7 +3,7 @@
 [![npm version](https://img.shields.io/npm/v/@modusensus/dsh-mneme?color=blue&label=npm)](https://www.npmjs.com/package/@modusensus/dsh-mneme)
 [![license](https://img.shields.io/badge/license-MIT-green)](LICENSE)
 [![dsh-plugin](https://img.shields.io/badge/dsh-plugin-awesome-orange)](https://github.com/awesome-dsh-plugin/awesome-dsh-plugin)
-[![tests](https://img.shields.io/badge/tests-106%20passed-success)](https://github.com/modusensus/dsh-mneme)
+[![tests](https://img.shields.io/badge/tests-108%20passed-success)](https://github.com/modusensus/dsh-mneme)
 
 > 给 DeepSeek Harness 的跨会话记忆插件：让 Agent 记住你、记住项目、自动整理记忆。**Mneme**（Μνήμη）——希腊记忆女神 Mnemosyne 之名，掌管记忆与梦境，正如 autoDream 在后台巩固记忆。
 
@@ -150,7 +150,7 @@ src/
 lib/
 ├── client.js         # Web 面板（手写 ModuleLoader bundle）
 └── *.js              # src 的同步分发产物
-test/                 # 106 个 node:test 测试
+test/                 # 108 个 node:test 测试
 ```
 
 ## 🧪 开发
@@ -158,7 +158,7 @@ test/                 # 106 个 node:test 测试
 ```bash
 cd dsh-mneme
 npm install        # 安装 peer 依赖（以 devDependencies 形式，用于本地测试）
-npm test           # 运行 106 个测试（--test-isolation=none 用于受限沙箱，禁止子进程 spawn）
+npm test           # 运行 108 个测试（--test-isolation=none 用于受限沙箱，禁止子进程 spawn）
 npm run sync       # 把 src/ 同步到 lib/（发布时由 prepack 钩子自动执行）
 ```
 

package/lib/dream.js

@@ -62,6 +62,7 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
   let running = false;
   let disposed = false;
   let baseline = { count: 0, chars: 0 };
+  let inFlight = null;
 
   function shouldTrigger(service) {
     const memories = service.all().filter((m) => !m.archived && m.type !== "summary");
@@ -81,8 +82,9 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
       running = true;
       // Defer the onRun invocation so a synchronous throw cannot escape the
       // timer callback (which would crash the process) and skip the teardown.
-      // Errors are logged, never swallowed silently.
-      Promise.resolve()
+      // Errors are logged, never swallowed silently. inFlight lets dispose()
+      // await the running consolidation before the caller closes the store.
+      inFlight = Promise.resolve()
         .then(() => (onRun ? onRun() : Promise.resolve({ ok: true, skipped: true })))
         .then((result) => {
           // Refresh the baseline only for a successful run (design §5.3: an
@@ -105,17 +107,19 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
         })
         .finally(() => {
           running = false;
+          inFlight = null;
         });
     }, delayMs);
     return true;
   }
 
-  function dispose() {
+  async function dispose() {
     disposed = true;
     if (pendingTimer) { clearTimeout(pendingTimer); pendingTimer = null; }
-    // An in-flight run is left to complete naturally: its LLM calls are
-    // already paid for and aborting would discard the work. The caller is
-    // responsible for closing the store only after the run has finished.
+    // An in-flight run is left to complete naturally (its LLM calls are
+    // already paid for and aborting would discard the work). Await it so the
+    // caller can close the store only after every write has landed.
+    if (inFlight) await inFlight.catch(() => {});
   }
 
   async function runDream(ctx, service, config) {

package/lib/index.js

@@ -34,9 +34,15 @@ export const apply = (ctx, config) => {
   const service = createService({ store, mirror, config: cfg });
 
   // Human edits in mirror files win on every sync; merge them back first.
-  // TYPE_FILE maps each memory type to its mirror filename.
+  // TYPE_FILE maps each memory type to its mirror filename. Read every type's
+  // edits up front: mergeHumanEdits re-renders ALL mirror files on success, so
+  // a per-type read-then-merge loop would overwrite edits in files not yet read
+  // (e.g. preferences.md merging would clobber unsynced projects.md edits).
+  const humanEdits = new Map();
   for (const type of Object.keys(TYPE_FILE)) {
-    const edits = mirror.readHumanEdits(type);
+    humanEdits.set(type, mirror.readHumanEdits(type));
+  }
+  for (const [type, edits] of humanEdits) {
     if (edits.length) service.mergeHumanEdits(type, edits);
   }
 
@@ -76,11 +82,14 @@ export const apply = (ctx, config) => {
     disposers.push(api.dispose);
   }
 
-  return () => {
+  // Async disposer: cordis awaits the returned promise on unload (runDisposable),
+  // so an in-flight dream run is allowed to finish before the SQLite store is
+  // closed — dream.dispose() resolves only after its current run settles.
+  return async () => {
     for (const dispose of disposers) {
       if (typeof dispose === "function") dispose();
     }
-    if (dream) dream.dispose();
+    if (dream) await dream.dispose();
     store.close();
   };
 };

package/lib/store.js

@@ -193,8 +193,9 @@ export function createStore(path) {
   function search(query, { limit = 20, includeArchived = false } = {}) {
     const q = String(query).trim();
     if (!q) return [];
-    // FTS5 over unicode61 (English + long phrases); LIKE fallback covers CJK substring.
-    // LIKE wildcards in the query are escaped so user input is matched literally.
+    // Plain LIKE substring scan over title/content/tags (wildcards escaped so
+    // user input matches literally). No FTS5: CJK substring matching needs
+    // LIKE, and typical memory stores are small enough that a scan is fine.
     const like = `%${escapeLike(q)}%`;
     const { limit: lim } = sanitizePage(limit, 0, 20);
     const archivedFilter = includeArchived ? "" : "archived = 0 AND ";

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@modusensus/dsh-mneme",
-  "description": "Cross-session memory plugin for DeepSeek Harness with autoDream consolidation: SQLite+FTS5 store, Markdown mirrors, 6 model tools, automatic injection, session summarization, and a Web GUI panel",
-  "version": "0.1.2",
+  "description": "Cross-session memory plugin for DeepSeek Harness with autoDream consolidation: SQLite store, Markdown mirrors, 6 model tools, automatic injection, session summarization, and a Web GUI panel",
+  "version": "0.1.3",
   "license": "MIT",
   "type": "module",
   "main": "lib/index.js",

package/src/dream.js

@@ -62,6 +62,7 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
   let running = false;
   let disposed = false;
   let baseline = { count: 0, chars: 0 };
+  let inFlight = null;
 
   function shouldTrigger(service) {
     const memories = service.all().filter((m) => !m.archived && m.type !== "summary");
@@ -81,8 +82,9 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
       running = true;
       // Defer the onRun invocation so a synchronous throw cannot escape the
       // timer callback (which would crash the process) and skip the teardown.
-      // Errors are logged, never swallowed silently.
-      Promise.resolve()
+      // Errors are logged, never swallowed silently. inFlight lets dispose()
+      // await the running consolidation before the caller closes the store.
+      inFlight = Promise.resolve()
         .then(() => (onRun ? onRun() : Promise.resolve({ ok: true, skipped: true })))
         .then((result) => {
           // Refresh the baseline only for a successful run (design §5.3: an
@@ -105,17 +107,19 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
         })
         .finally(() => {
           running = false;
+          inFlight = null;
         });
     }, delayMs);
     return true;
   }
 
-  function dispose() {
+  async function dispose() {
     disposed = true;
     if (pendingTimer) { clearTimeout(pendingTimer); pendingTimer = null; }
-    // An in-flight run is left to complete naturally: its LLM calls are
-    // already paid for and aborting would discard the work. The caller is
-    // responsible for closing the store only after the run has finished.
+    // An in-flight run is left to complete naturally (its LLM calls are
+    // already paid for and aborting would discard the work). Await it so the
+    // caller can close the store only after every write has landed.
+    if (inFlight) await inFlight.catch(() => {});
   }
 
   async function runDream(ctx, service, config) {

package/src/index.js

@@ -34,9 +34,15 @@ export const apply = (ctx, config) => {
   const service = createService({ store, mirror, config: cfg });
 
   // Human edits in mirror files win on every sync; merge them back first.
-  // TYPE_FILE maps each memory type to its mirror filename.
+  // TYPE_FILE maps each memory type to its mirror filename. Read every type's
+  // edits up front: mergeHumanEdits re-renders ALL mirror files on success, so
+  // a per-type read-then-merge loop would overwrite edits in files not yet read
+  // (e.g. preferences.md merging would clobber unsynced projects.md edits).
+  const humanEdits = new Map();
   for (const type of Object.keys(TYPE_FILE)) {
-    const edits = mirror.readHumanEdits(type);
+    humanEdits.set(type, mirror.readHumanEdits(type));
+  }
+  for (const [type, edits] of humanEdits) {
     if (edits.length) service.mergeHumanEdits(type, edits);
   }
 
@@ -76,11 +82,14 @@ export const apply = (ctx, config) => {
     disposers.push(api.dispose);
   }
 
-  return () => {
+  // Async disposer: cordis awaits the returned promise on unload (runDisposable),
+  // so an in-flight dream run is allowed to finish before the SQLite store is
+  // closed — dream.dispose() resolves only after its current run settles.
+  return async () => {
     for (const dispose of disposers) {
       if (typeof dispose === "function") dispose();
     }
-    if (dream) dream.dispose();
+    if (dream) await dream.dispose();
     store.close();
   };
 };

package/src/store.js

@@ -193,8 +193,9 @@ export function createStore(path) {
   function search(query, { limit = 20, includeArchived = false } = {}) {
     const q = String(query).trim();
     if (!q) return [];
-    // FTS5 over unicode61 (English + long phrases); LIKE fallback covers CJK substring.
-    // LIKE wildcards in the query are escaped so user input is matched literally.
+    // Plain LIKE substring scan over title/content/tags (wildcards escaped so
+    // user input matches literally). No FTS5: CJK substring matching needs
+    // LIKE, and typical memory stores are small enough that a scan is fine.
     const like = `%${escapeLike(q)}%`;
     const { limit: lim } = sanitizePage(limit, 0, 20);
     const archivedFilter = includeArchived ? "" : "archived = 0 AND ";