@modusensus/dsh-mneme 0.3.0 → 0.3.1

package/lib/entities/extractor.js

@@ -140,7 +140,7 @@ async function resolveEntity(entity, store) {
   return created.id;
 }
 
-export async function extractEntities(memory, { store, config, callLLM }) {
+export async function extractEntities(memory, { store, config, callLLM, logger }) {
   try {
     if (!memory || !memory.content) {
       return { ok: false, error: "Invalid memory: missing content" };
@@ -181,7 +181,7 @@ export async function extractEntities(memory, { store, config, callLLM }) {
         resolvedEntities.push({ ...entity, entity_id: entityId });
       } catch (err) {
         skipCount++;
-        console.warn(`[extractor] Failed to resolve entity "${entity.name}":`, err.message);
+        logger?.warn?.(`[extractor] Failed to resolve entity "${entity.name}":`, err.message);
       }
     }
     
@@ -201,7 +201,7 @@ export async function extractEntities(memory, { store, config, callLLM }) {
           attrs.push(saved);
         } catch (err) {
           skipCount++;
-          console.warn(`[extractor] Failed to save attr "${attr.key}" for entity "${entity.name}":`, err.message);
+          logger?.warn?.(`[extractor] Failed to save attr "${attr.key}" for entity "${entity.name}":`, err.message);
         }
       }
     }
@@ -224,7 +224,7 @@ export async function extractEntities(memory, { store, config, callLLM }) {
         savedRelations.push(saved);
       } catch (err) {
         skipCount++;
-        console.warn(`[extractor] Failed to save relation "${rel.from} -> ${rel.to}":`, err.message);
+        logger?.warn?.(`[extractor] Failed to save relation "${rel.from} -> ${rel.to}":`, err.message);
       }
     }
     

package/lib/index.js

@@ -43,7 +43,7 @@ export const apply = (ctx, config) => {
     store.deleteOldFailures(new Date(Date.now() - 90 * 86400000).toISOString());
   } catch { /* non-fatal */ }
   const mirror = createMirror(memoryDir);
-  const service = createService({ store, mirror, config: cfg });
+  const service = createService({ store, mirror, config: cfg, logger: ctx.logger });
 
   // Recall-layer receipt: when searchMemories runs with recordRecall=true, the
   // retrieval scene (query/mode/topK/threshold + candidates) is persisted to
@@ -196,7 +196,7 @@ export const apply = (ctx, config) => {
       return text;
     };
     service.setEntityExtractor((memory) =>
-      extractEntities(memory, { store, config: cfg, callLLM: streamEntityText })
+      extractEntities(memory, { store, config: cfg, callLLM: streamEntityText, logger: ctx.logger })
         .catch((err) => {
           ctx.logger?.warn?.(`[dsh-mneme] entity extraction failed: ${String(err)}`);
           return { ok: false, error: String(err) };

package/lib/service.js

@@ -3,7 +3,7 @@ import { TYPE_FILE } from "./mirror.js";
 
 const INJECT_TYPES = new Set(["preference", "project", "decision", "summary"]);
 
-export function createService({ store, mirror, config, onWrite }) {
+export function createService({ store, mirror, config, onWrite, logger }) {
   // Optional dream scheduler hook, installed via setDreamHook after creation
   // (the scheduler holds a reference back to the service, so it cannot be
   // passed in the constructor). Fired on the same write events as onWrite.
@@ -56,10 +56,10 @@ export function createService({ store, mirror, config, onWrite }) {
     if (!config.entityExtractionEnabled || !entityExtractor) return;
     try {
       entityExtractor(memory).catch((err) => {
-        console.warn("entity extraction failed:", err);
+        logger?.warn?.("entity extraction failed:", err);
       });
     } catch (err) {
-      console.warn("entity extraction failed:", err);
+      logger?.warn?.("entity extraction failed:", err);
     }
   }
 
@@ -306,7 +306,7 @@ export function createService({ store, mirror, config, onWrite }) {
       try {
         syncMirror();
       } catch (error) {
-        console.warn("syncMirror failed after transaction:", error);
+        logger?.warn?.("syncMirror failed after transaction:", error);
       }
       notifyWrite();
     }
@@ -488,7 +488,7 @@ export function createService({ store, mirror, config, onWrite }) {
     try {
       mirror.sync(reconcileHumanEdits(store.list({ limit: 500, includeForgotten: false })));
     } catch (error) {
-      console.warn("syncMirror failed:", error);
+      logger?.warn?.("syncMirror failed:", error);
     }
   }
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@modusensus/dsh-mneme",
   "description": "Cross-session memory plugin for DeepSeek Harness with autoDream consolidation: SQLite store, Markdown mirrors, 6 model tools, automatic injection, session summarization, user profile/rules, custom slash commands, vector (semantic) search, and a Web GUI panel",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "license": "MIT",
   "type": "module",
   "main": "lib/index.js",

package/src/entities/extractor.js

@@ -140,7 +140,7 @@ async function resolveEntity(entity, store) {
   return created.id;
 }
 
-export async function extractEntities(memory, { store, config, callLLM }) {
+export async function extractEntities(memory, { store, config, callLLM, logger }) {
   try {
     if (!memory || !memory.content) {
       return { ok: false, error: "Invalid memory: missing content" };
@@ -181,7 +181,7 @@ export async function extractEntities(memory, { store, config, callLLM }) {
         resolvedEntities.push({ ...entity, entity_id: entityId });
       } catch (err) {
         skipCount++;
-        console.warn(`[extractor] Failed to resolve entity "${entity.name}":`, err.message);
+        logger?.warn?.(`[extractor] Failed to resolve entity "${entity.name}":`, err.message);
       }
     }
     
@@ -201,7 +201,7 @@ export async function extractEntities(memory, { store, config, callLLM }) {
           attrs.push(saved);
         } catch (err) {
           skipCount++;
-          console.warn(`[extractor] Failed to save attr "${attr.key}" for entity "${entity.name}":`, err.message);
+          logger?.warn?.(`[extractor] Failed to save attr "${attr.key}" for entity "${entity.name}":`, err.message);
         }
       }
     }
@@ -224,7 +224,7 @@ export async function extractEntities(memory, { store, config, callLLM }) {
         savedRelations.push(saved);
       } catch (err) {
         skipCount++;
-        console.warn(`[extractor] Failed to save relation "${rel.from} -> ${rel.to}":`, err.message);
+        logger?.warn?.(`[extractor] Failed to save relation "${rel.from} -> ${rel.to}":`, err.message);
       }
     }
     

package/src/index.js

@@ -43,7 +43,7 @@ export const apply = (ctx, config) => {
     store.deleteOldFailures(new Date(Date.now() - 90 * 86400000).toISOString());
   } catch { /* non-fatal */ }
   const mirror = createMirror(memoryDir);
-  const service = createService({ store, mirror, config: cfg });
+  const service = createService({ store, mirror, config: cfg, logger: ctx.logger });
 
   // Recall-layer receipt: when searchMemories runs with recordRecall=true, the
   // retrieval scene (query/mode/topK/threshold + candidates) is persisted to
@@ -196,7 +196,7 @@ export const apply = (ctx, config) => {
       return text;
     };
     service.setEntityExtractor((memory) =>
-      extractEntities(memory, { store, config: cfg, callLLM: streamEntityText })
+      extractEntities(memory, { store, config: cfg, callLLM: streamEntityText, logger: ctx.logger })
         .catch((err) => {
           ctx.logger?.warn?.(`[dsh-mneme] entity extraction failed: ${String(err)}`);
           return { ok: false, error: String(err) };

package/src/service.js

@@ -3,7 +3,7 @@ import { TYPE_FILE } from "./mirror.js";
 
 const INJECT_TYPES = new Set(["preference", "project", "decision", "summary"]);
 
-export function createService({ store, mirror, config, onWrite }) {
+export function createService({ store, mirror, config, onWrite, logger }) {
   // Optional dream scheduler hook, installed via setDreamHook after creation
   // (the scheduler holds a reference back to the service, so it cannot be
   // passed in the constructor). Fired on the same write events as onWrite.
@@ -56,10 +56,10 @@ export function createService({ store, mirror, config, onWrite }) {
     if (!config.entityExtractionEnabled || !entityExtractor) return;
     try {
       entityExtractor(memory).catch((err) => {
-        console.warn("entity extraction failed:", err);
+        logger?.warn?.("entity extraction failed:", err);
       });
     } catch (err) {
-      console.warn("entity extraction failed:", err);
+      logger?.warn?.("entity extraction failed:", err);
     }
   }
 
@@ -306,7 +306,7 @@ export function createService({ store, mirror, config, onWrite }) {
       try {
         syncMirror();
       } catch (error) {
-        console.warn("syncMirror failed after transaction:", error);
+        logger?.warn?.("syncMirror failed after transaction:", error);
       }
       notifyWrite();
     }
@@ -488,7 +488,7 @@ export function createService({ store, mirror, config, onWrite }) {
     try {
       mirror.sync(reconcileHumanEdits(store.list({ limit: 500, includeForgotten: false })));
     } catch (error) {
-      console.warn("syncMirror failed:", error);
+      logger?.warn?.("syncMirror failed:", error);
     }
   }
 

package/test/fnew-0112.test.js

@@ -201,27 +201,22 @@ test("F-NEW-02 e: COMMIT 后 mirror.sync 抛 ENOSPC → transaction 正常返回
   try {
     const store = createStore(dbPath);
     const mirror = createMirror(mirrorDir);
-    const service = createService({ store, mirror, config: {} });
+    const warns = [];
+    const logger = { warn: (...a) => warns.push(a) };
+    const service = createService({ store, mirror, config: {}, logger });
     const { original } = throwingMirrorSync(mirror);
 
-    let warns = 0;
     let committedId;
-    const origWarn = console.warn;
-    console.warn = (...a) => { warns++; origWarn(...a); };
-    try {
-      let result;
-      assert.doesNotThrow(() => {
-        result = service.transaction(() => {
-          const { memory } = service.saveWithDedupe({ type: "project", title: "原子", content: "已提交", importance: 3 });
-          committedId = memory.id;
-          return "tx-result";
-        });
-      }, "syncMirror 失败不得向外抛（已 COMMIT 不得被当成未提交）");
-      assert.equal(result, "tx-result", "transaction 必须返回 fn 的结果");
-    } finally {
-      console.warn = origWarn;
-    }
-    assert.ok(warns >= 1, "fail-safe 应走 console.warn 路径");
+    let result;
+    assert.doesNotThrow(() => {
+      result = service.transaction(() => {
+        const { memory } = service.saveWithDedupe({ type: "project", title: "原子", content: "已提交", importance: 3 });
+        committedId = memory.id;
+        return "tx-result";
+      });
+    }, "syncMirror 失败不得向外抛（已 COMMIT 不得被当成未提交）");
+    assert.equal(result, "tx-result", "transaction 必须返回 fn 的结果");
+    assert.ok(warns.length >= 1, "fail-safe 应走 logger.warn 路径");
     assert.equal(store.count(), 1, "DB 必须已提交");
     assert.equal(service.getById(committedId).content, "已提交");