@modusensus/dsh-mneme 0.2.5 → 0.2.7

package/README.md

@@ -5,7 +5,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)
 [![Awesome](https://awesome-dsh-plugin.com/badge.svg)](https://github.com/awesome-dsh-plugin/awesome-dsh-plugin)
-[![tests](https://img.shields.io/badge/tests-258%20passed-success)](https://github.com/modusensus/dsh-mneme)
+[![tests](https://img.shields.io/badge/tests-263%20passed-success)](https://github.com/modusensus/dsh-mneme)
 
 > 给 DeepSeek Harness 的跨会话记忆插件：让 Agent 记住你、记住项目、自动整理记忆。**Mneme**（Μνήμη）——希腊记忆女神 Mnemosyne 之名，掌管记忆与梦境，正如 autoDream 在后台巩固记忆。
 
@@ -225,7 +225,7 @@ src/
 lib/
 ├── client.js         # Web 面板（手写 ModuleLoader bundle）
 └── *.js              # src 的同步分发产物
-test/                 # 258 个 node:test 测试（含审计与三轴线压测不变量）
+test/                 # 263 个 node:test 测试（含审计与三轴线压测不变量）
 scripts/              # e2e-dsh.js 端到端演示 · stress-dsh.js 三轴线压测 · sync-lib.js 同步
 ```
 
@@ -234,7 +234,7 @@ scripts/              # e2e-dsh.js 端到端演示 · stress-dsh.js 三轴线压
 ```bash
 cd dsh-mneme
 npm install        # 安装 peer 依赖（以 devDependencies 形式，用于本地测试）
-npm test           # 运行 258 个测试
+npm test           # 运行 263 个测试
 npm run stress     # 三轴线压测：长会话检索 / 冲突仲裁 / 多 Agent 并发（离线 mock LLM）
 npm run sync       # 把 src/ 同步到 lib/（发布时由 prepack 钩子自动执行）
 ```

package/lib/client.js

@@ -178,7 +178,7 @@ window.__ModuleLoader__.load({
 
       const tabs = ["all", "preference", "project", "decision", "history"];
 
-      const body = react.createElement("div", { style: embedded ? { ...styles.panel, width: "100%", maxWidth: "100%", maxHeight: "none" } : styles.panel },
+      const body = react.createElement("div", { style: embedded ? { width: "100%", display: "flex", flexDirection: "column" } : styles.panel },
         !embedded && react.createElement("div", { style: styles.header },
           react.createElement("span", { style: styles.title }, t("memory.panel.title")),
           react.createElement("button", { style: styles.close, onClick: onClose }, "×")
@@ -365,7 +365,7 @@ window.__ModuleLoader__.load({
       const hintStyle = { fontSize: 11, color: "var(--dsw-alias-label-tertiary, #999)", marginBottom: 8 };
       const rowStyle = { display: "flex", justifyContent: "space-between", alignItems: "center", gap: 8, padding: "6px 0", borderBottom: "1px solid var(--dsw-alias-border-l1, #eee)" };
 
-      const body = h("div", { style: embedded ? { ...styles.panel, width: "100%", maxWidth: "100%", maxHeight: "none" } : styles.panel },
+      const body = h("div", { style: embedded ? { width: "100%", display: "flex", flexDirection: "column" } : styles.panel },
         !embedded && h("div", { style: styles.header },
           h("span", { style: styles.title }, t("memory.settings.title")),
           h("button", { style: styles.close, onClick: onClose }, "×")

package/lib/dream.js

@@ -68,7 +68,7 @@ export function parseReceipt(receipt) {
   // reconcile = decisions validated but one or more did not commit (CAS
   // conflict / transaction rollback) — the store diverges from the decision
   // list and the run must be reconciled, never reported as a fake ok.
-  if (!runId || !/^(ok|failed|reconcile)$/.test(status)) return undefined;
+  if (!runId || !/^(ok|noop|degraded|reconcile|failed)$/.test(status)) return undefined;
   const count = Number(inputCount);
   const appliedN = Number(applied);
   if (!Number.isInteger(count) || !Number.isInteger(appliedN)) return undefined;
@@ -286,9 +286,11 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
     // replayable too.
     const finish = (result) => {
       // status is derived from what actually committed: ok only when the full
-      // decision list landed; reconcile when decisions were validated but some
-      // did not commit (CAS conflict / rollback); failed on any LLM/validation
-      // error. No fake "ok" for a partial commit.
+      // decision list landed (or a summary was refreshed); noop when nothing
+      // changed; degraded when real changes landed without a summary;
+      // reconcile when decisions were validated but some did not commit (CAS
+      // conflict / rollback); failed on any LLM/validation error. No fake "ok"
+      // for an empty or partial run.
       const status = result.status ?? (result.ok ? "ok" : "failed");
       const applied = result.applied ?? 0;
       const summaryStored = result.summary ?? false;
@@ -432,6 +434,11 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
     const outcome = { ...buildOutcome(committed), conflicts, failures };
     // Decisions validated but not fully committed → reconcile (not ok).
     const partial = conflicts.length > 0 || failures.length > 0;
+    // No decision landed (all-keep, or every decision skipped as an idempotent
+    // replay) → nothing substantive changed. Distinct from a success: such a
+    // run must never be reported as ok, or the audit claims work that never
+    // happened and the scheduler refreshes the baseline on a false positive.
+    const noChange = applied === 0 && committed.every((c) => c.action === "keep");
 
     // Keep the vector index consistent with the post-dream store state.
     if (semantic?.embedder && semantic?.vectorIndex) {
@@ -476,9 +483,30 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
         } catch { /* best-effort */ }
       }
     }
+    // Honest status assignment (never a fake ok):
+    //   reconcile — some decisions validated but did not commit (CAS/rollback).
+    //   noop      — nothing changed and no summary persisted: truly an empty
+    //               run. ok:false keeps the scheduler from moving the baseline.
+    //   ok        — either real changes landed, or a fresh summary was stored
+    //               (all-keep + summary is a substantive summary refresh).
+    //   degraded  — real consolidation landed but the summary came back empty/
+    //               missing: the store was absorbed (ok for the baseline) but
+    //               the run did not produce its full output (marked, not faked).
+    let status;
+    let okResult;
+    if (partial) {
+      status = "reconcile";
+      okResult = false;
+    } else if (noChange) {
+      status = summaryStored ? "ok" : "noop";
+      okResult = summaryStored;
+    } else {
+      status = summaryStored ? "ok" : "degraded";
+      okResult = true;
+    }
     return finish({
-      ok: !partial,
-      status: partial ? "reconcile" : "ok",
+      ok: okResult,
+      status,
       applied,
       decisions: auditDecisions,
       outcome,

package/lib/store.js

@@ -26,7 +26,7 @@ CREATE INDEX IF NOT EXISTS idx_memories_importance ON memories(importance);
 CREATE TABLE IF NOT EXISTS dream_runs (
   id             TEXT PRIMARY KEY,
   created_at     TEXT NOT NULL,
-  status         TEXT NOT NULL,          -- ok | failed
+  status         TEXT NOT NULL,          -- ok | noop | degraded | reconcile | failed
   error          TEXT,
   provider       TEXT,
   model          TEXT,

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.2.5",
+  "version": "0.2.7",
   "license": "MIT",
   "type": "module",
   "main": "lib/index.js",

package/src/dream.js

@@ -68,7 +68,7 @@ export function parseReceipt(receipt) {
   // reconcile = decisions validated but one or more did not commit (CAS
   // conflict / transaction rollback) — the store diverges from the decision
   // list and the run must be reconciled, never reported as a fake ok.
-  if (!runId || !/^(ok|failed|reconcile)$/.test(status)) return undefined;
+  if (!runId || !/^(ok|noop|degraded|reconcile|failed)$/.test(status)) return undefined;
   const count = Number(inputCount);
   const appliedN = Number(applied);
   if (!Number.isInteger(count) || !Number.isInteger(appliedN)) return undefined;
@@ -286,9 +286,11 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
     // replayable too.
     const finish = (result) => {
       // status is derived from what actually committed: ok only when the full
-      // decision list landed; reconcile when decisions were validated but some
-      // did not commit (CAS conflict / rollback); failed on any LLM/validation
-      // error. No fake "ok" for a partial commit.
+      // decision list landed (or a summary was refreshed); noop when nothing
+      // changed; degraded when real changes landed without a summary;
+      // reconcile when decisions were validated but some did not commit (CAS
+      // conflict / rollback); failed on any LLM/validation error. No fake "ok"
+      // for an empty or partial run.
       const status = result.status ?? (result.ok ? "ok" : "failed");
       const applied = result.applied ?? 0;
       const summaryStored = result.summary ?? false;
@@ -432,6 +434,11 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
     const outcome = { ...buildOutcome(committed), conflicts, failures };
     // Decisions validated but not fully committed → reconcile (not ok).
     const partial = conflicts.length > 0 || failures.length > 0;
+    // No decision landed (all-keep, or every decision skipped as an idempotent
+    // replay) → nothing substantive changed. Distinct from a success: such a
+    // run must never be reported as ok, or the audit claims work that never
+    // happened and the scheduler refreshes the baseline on a false positive.
+    const noChange = applied === 0 && committed.every((c) => c.action === "keep");
 
     // Keep the vector index consistent with the post-dream store state.
     if (semantic?.embedder && semantic?.vectorIndex) {
@@ -476,9 +483,30 @@ export function createDreamScheduler({ onRun, thresholdCount = 10, thresholdChar
         } catch { /* best-effort */ }
       }
     }
+    // Honest status assignment (never a fake ok):
+    //   reconcile — some decisions validated but did not commit (CAS/rollback).
+    //   noop      — nothing changed and no summary persisted: truly an empty
+    //               run. ok:false keeps the scheduler from moving the baseline.
+    //   ok        — either real changes landed, or a fresh summary was stored
+    //               (all-keep + summary is a substantive summary refresh).
+    //   degraded  — real consolidation landed but the summary came back empty/
+    //               missing: the store was absorbed (ok for the baseline) but
+    //               the run did not produce its full output (marked, not faked).
+    let status;
+    let okResult;
+    if (partial) {
+      status = "reconcile";
+      okResult = false;
+    } else if (noChange) {
+      status = summaryStored ? "ok" : "noop";
+      okResult = summaryStored;
+    } else {
+      status = summaryStored ? "ok" : "degraded";
+      okResult = true;
+    }
     return finish({
-      ok: !partial,
-      status: partial ? "reconcile" : "ok",
+      ok: okResult,
+      status,
       applied,
       decisions: auditDecisions,
       outcome,

package/src/store.js

@@ -26,7 +26,7 @@ CREATE INDEX IF NOT EXISTS idx_memories_importance ON memories(importance);
 CREATE TABLE IF NOT EXISTS dream_runs (
   id             TEXT PRIMARY KEY,
   created_at     TEXT NOT NULL,
-  status         TEXT NOT NULL,          -- ok | failed
+  status         TEXT NOT NULL,          -- ok | noop | degraded | reconcile | failed
   error          TEXT,
   provider       TEXT,
   model          TEXT,

package/test/audit.test.js

@@ -288,3 +288,69 @@ test("buildOutcome over actually-committed sub-steps never claims a rolled-back
   assert.equal(outcome.byId.m2, "merge-archived");
   assert.equal(outcome.byId.k, "keep");
 });
+
+// ---------------------------------------------------------------- noop & degraded (F-03)
+
+test("runDream records status noop when all decisions are keep and summary is empty", async () => {
+  const { store, service, dream } = setup();
+  const { memory: m } = service.saveWithDedupe({ type: "preference", title: "语言", content: "中文" });
+  const ctx = mockCtx({
+    onConsolidation: () => JSON.stringify([{ action: "keep", ids: [m.id] }]),
+    summaryText: ""
+  });
+  const result = await dream.runDream(ctx, service, {});
+  assert.equal(result.ok, false, "no-change + empty summary is not a success");
+  assert.equal(result.status, "noop", "explicit noop status");
+  assert.equal(result.applied, 0);
+  assert.equal(result.summary, false);
+
+  const run = store.listDreamRuns()[0];
+  assert.equal(run.status, "noop", "audit row records noop, never ok");
+  assert.equal(run.applied, 0);
+  assert.equal(run.summary_stored, false);
+  const parsed = parseReceipt(run.receipt);
+  assert.equal(parsed.status, "noop", "receipt records noop");
+  assert.equal(parsed.applied, 0);
+  assert.equal(parsed.summaryStored, false);
+  assert.equal(run.outcome.byId[m.id], "keep", "keep disposition still recorded for replay");
+  store.close();
+});
+
+test("runDream records status ok when all decisions are keep but a summary was stored", async () => {
+  const { store, service, dream } = setup();
+  const { memory: m } = service.saveWithDedupe({ type: "preference", title: "语言", content: "中文" });
+  const ctx = mockCtx({ onConsolidation: () => JSON.stringify([{ action: "keep", ids: [m.id] }]) });
+  const result = await dream.runDream(ctx, service, {});
+  assert.equal(result.status, "ok", "summary refresh is substantive output");
+  assert.equal(result.ok, true);
+  assert.equal(result.applied, 0);
+  assert.equal(result.summary, true);
+  const run = store.listDreamRuns()[0];
+  assert.equal(run.status, "ok");
+  assert.equal(parseReceipt(run.receipt).status, "ok");
+  assert.equal(parseReceipt(run.receipt).summaryStored, true);
+  store.close();
+});
+
+test("runDream records status degraded when changes land but the summary is empty", async () => {
+  const { store, service, dream } = setup();
+  const { memory: a } = service.saveWithDedupe({ type: "project", title: "旧", content: "A", importance: 3 });
+  const { memory: b } = service.saveWithDedupe({ type: "project", title: "新", content: "B", importance: 4 });
+  const ctx = mockCtx({
+    onConsolidation: () => JSON.stringify([
+      { action: "merge", ids: [a.id, b.id], title: "合并", content: "合并内容", importance: 4, keepSource: b.id }
+    ]),
+    summaryText: ""
+  });
+  const result = await dream.runDream(ctx, service, {});
+  assert.equal(result.status, "degraded", "real changes without a summary are not a clean ok");
+  assert.equal(result.ok, true, "consolidation landed so the baseline may advance");
+  assert.equal(result.applied, 1);
+  assert.equal(result.summary, false, "summary honestly reported missing");
+  const run = store.listDreamRuns()[0];
+  assert.equal(run.status, "degraded", "audit row records degraded");
+  assert.equal(run.summary_stored, false);
+  assert.equal(parseReceipt(run.receipt).status, "degraded");
+  assert.equal(parseReceipt(run.receipt).summaryStored, false);
+  store.close();
+});

package/test/client.test.js

@@ -24,3 +24,21 @@ test("client bundle is lib-only with no src counterpart", () => {
   assert.equal(existsSync(join(root, "src/client.js")), false, "src/ must not contain client.js");
   assert.equal(existsSync(join(root, "lib/client.js")), true, "lib/client.js must exist");
 });
+
+// Both panels render in two modes: a portal modal (styles.panel, full chrome)
+// and an embedded settings-section variant. The embedded variant must drop the
+// modal-only chrome (background, radius, padding, shadow) — the settings host
+// already provides its own surface, so reusing the modal panel style makes the
+// section render as a floating card with an inset box and shadow inside the
+// settings page.
+test("embedded variant drops the modal panel chrome", () => {
+  const branches = clientSource.match(/embedded \? \{[^}]+\}/g);
+  assert.ok(branches, "panels must branch on the embedded flag");
+  assert.equal(branches.length, 2, "both MemoryPanel and SettingsPanel must branch");
+  for (const branch of branches) {
+    assert.equal(branch.includes("styles.panel"), false, "embedded branch must not reuse the modal panel style");
+    for (const chrome of ["boxShadow", "background", "borderRadius", "padding"]) {
+      assert.equal(branch.includes(chrome), false, `embedded branch must not carry ${chrome}`);
+    }
+  }
+});

package/test/dream.test.js

@@ -296,6 +296,27 @@ test("failed run does not refresh baseline: next write re-triggers", async () =>
   store.close();
 });
 
+test("noop run (nothing changed) does not refresh baseline: next write re-triggers", async () => {
+  const { store, service } = dreamSetup();
+  let calls = 0;
+  const dream = createDreamScheduler({
+    onRun: async () => { calls++; return { ok: false, status: "noop", applied: 0, summary: false }; },
+    thresholdCount: 2, thresholdChars: 5000, delayMs: 5,
+    logger: { warn: () => {} }
+  });
+  service.saveWithDedupe({ type: "project", title: "a", content: "x".repeat(10) });
+  service.saveWithDedupe({ type: "project", title: "b", content: "y".repeat(10) });
+  assert.equal(dream.maybeSchedule(service), true, "scheduled");
+  await new Promise((r) => setTimeout(r, 50));
+  assert.equal(calls, 1, "run attempted once");
+  // a noop is not a success: the baseline stays put so the accumulated writes
+  // are still owed and the next write re-schedules instead of being absorbed
+  assert.equal(dream.maybeSchedule(service), true, "noop keeps baseline, re-schedules");
+  await new Promise((r) => setTimeout(r, 50));
+  assert.equal(calls, 2, "retried after noop");
+  store.close();
+});
+
 test("throwing run does not refresh baseline and is logged", async () => {
   const { store, service } = dreamSetup();
   const warnings = [];