@modusensus/dsh-mneme 0.7.9 → 0.7.11

package/{dsh-mneme/test → test}/client.test.js

@@ -5,9 +5,7 @@ import { fileURLToPath } from "node:url";
 import { dirname, join } from "node:path";
 
 const root = join(dirname(fileURLToPath(import.meta.url)), "..");
-// src/ is the single source of truth for every module (including the Web
-// client bundle); lib/ is build output produced by `npm run sync`.
-const clientSource = readFileSync(join(root, "src/client.js"), "utf8");
+const clientSource = readFileSync(join(root, "lib/client.js"), "utf8");
 const pkg = JSON.parse(readFileSync(join(root, "package.json"), "utf8"));
 
 // The Web client bundle registers itself via __ModuleLoader__.load. DSH
@@ -20,14 +18,11 @@ test("client bundle registers under the package name", () => {
   assert.equal(match[1], pkg.name, "registered id must equal package.json name");
 });
 
-// client.js is authored under src/ like every other module; lib/client.js is
-// build output generated by `npm run sync` (prepack), so it must stay a
-// byte-identical copy — drift here means the sync wasn't run.
-test("client bundle is src-authored and synced into lib", () => {
-  assert.equal(existsSync(join(root, "src/client.js")), true, "src/client.js must exist");
+// client.js is hand-authored under lib/ only (no src/ counterpart), so the
+// src->lib sync must never prune it.
+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");
-  const libClient = readFileSync(join(root, "lib/client.js"), "utf8");
-  assert.equal(libClient, clientSource, "lib/client.js must equal src/client.js (run `npm run sync`)");
 });
 
 // The memory entry lives at the sidebar foot, not in the settings modal: the
@@ -67,13 +62,24 @@ test("tab-first activation with hero-screen overlay fallback", () => {
     "a failed tab activation must open the fallback overlay"
   );
   assert.ok(
-    /if \(!tab\) \{ resolve\(false\); return; \}/.test(clientSource),
+    /if \(candidates\.length === 0\) \{ resolve\(false\); return; \}/.test(clientSource),
     "activation must resolve false when the tab ring is absent (hero screen)"
   );
   assert.ok(
     /aria-selected.*true/.test(clientSource),
     "activation is only confirmed once the host marks the tab selected"
   );
+  // Same-labelled tabs from other plugins must never be activated by mistake:
+  // the click is only trusted once OUR explorer view actually rendered, and
+  // hidden tab panes are excluded before any click happens.
+  assert.ok(
+    /querySelector\("\.mneme-x"\) !== null/.test(clientSource),
+    "activation must verify the memory explorer rendered, not just the tab state"
+  );
+  assert.ok(
+    /offsetParent === null/.test(clientSource),
+    "hidden tab panes (settings dialogs etc.) must be excluded from activation"
+  );
 });
 
 // The fallback overlay reuses the full MemoryExplorer (three columns, graph,
@@ -144,18 +150,18 @@ test("graph toggle uses a node-graph glyph, not the share icon", () => {
 });
 
 // Every memory feature lives in the main-area library now: the explorer
-// hosts three sub-views (browse / graph / settings) switched by tabs whose
+// hosts three sub-views (browse / entities / settings) switched by tabs whose
 // labels come from dedicated dictionary keys.
-test("explorer hosts browse, graph and settings sub-views", () => {
-  for (const key of ["tabMemory", "tabGraph", "tabSettings"]) {
+test("explorer hosts browse, entities and settings sub-views", () => {
+  for (const key of ["tabMemory", "tabEntities", "tabSettings"]) {
     assert.ok(
       clientSource.includes(`"memory.explorer.${key}"`),
       `sub-view labels must come from memory.explorer.${key}`
     );
   }
   assert.ok(
-    clientSource.includes('h(GraphPanel, { t, focusEntity: graphFocus, onJumpMemory: jumpToMemory })'),
-    "the graph panel must be embedded as a sub-view"
+    clientSource.includes("h(EntityPanel, { t, focusEntity: graphFocus, onJumpMemory: jumpToMemory })"),
+    "the entity panel must be embedded as a sub-view"
   );
   assert.ok(
     clientSource.includes('h(SettingsContent, { t })'),
@@ -237,194 +243,3 @@ test("explorer chrome aligns with the host design system", () => {
     "pill chips belong to the drawer era and must stay gone"
   );
 });
-
-// The v0.6.1 wiki-link feature: the detail pane mounts a BacklinksPanel that
-// fetches the two read-only link endpoints by the selected memory id and
-// renders back/forward rows, each jumping back into the browser.
-test("detail pane mounts a BacklinksPanel backed by the two link endpoints", () => {
-  assert.ok(
-    clientSource.includes("h(BacklinksPanel, { memory: selected, t, onJump: jumpToMemory })"),
-    "the detail pane must mount BacklinksPanel after the actions row"
-  );
-  assert.ok(
-    clientSource.includes("/api/dsh-mneme/wikilinks/backlinks?id="),
-    "BacklinksPanel must fetch the backlinks endpoint by memory id"
-  );
-  assert.ok(
-    clientSource.includes("/api/dsh-mneme/wikilinks/forward?id="),
-    "BacklinksPanel must fetch the forward-links endpoint by memory id"
-  );
-});
-
-// Detail content turns [[target]] / [[display|target]] into clickable links:
-// the display text survives, the target title is kept for hover, and a click
-// resolves the title through the resolve endpoint before jumping.
-test("detail content renders wiki-links and resolves them on click", () => {
-  assert.ok(
-    clientSource.includes('className: "mneme-wikilink"'),
-    "inline [[target]] links must use the .mneme-wikilink style"
-  );
-  assert.ok(
-    clientSource.includes("/api/dsh-mneme/wikilinks/resolve?title="),
-    "clicking a wiki-link must resolve the title via the resolve endpoint"
-  );
-  assert.ok(
-    clientSource.includes('className: "mneme-backlinks"'),
-    "the panel must render inside a .mneme-backlinks block"
-  );
-});
-
-// v0.6.2 tag system: the detail pane renders tags as clickable chips wired to
-// the memory/tags endpoints, and a tag: query always goes server-side.
-test("detail pane renders editable tag chips backed by the tags endpoint", () => {
-  assert.ok(
-    clientSource.includes("/api/dsh-mneme/memory/tags?id="),
-    "the panel must fetch the selected memory's tags via the GET endpoint"
-  );
-  assert.ok(
-    clientSource.includes('method: "POST"'),
-    "adding/removing a tag must POST to the tags endpoint"
-  );
-  assert.ok(
-    clientSource.includes("onClick: () => onTagClick(tag)"),
-    "a tag chip click must trigger a tag: search"
-  );
-  assert.ok(
-    clientSource.includes("tagManual"),
-    "the panel must read the manualTagEnabled gate to hide editing"
-  );
-});
-
-test("tag: queries run server-side regardless of the semantic toggle", () => {
-  assert.ok(
-    clientSource.includes('(!semantic && !q.startsWith("tag:"))'),
-    "tag: must bypass the semantic toggle and use the search endpoint"
-  );
-  assert.ok(
-    clientSource.includes('setQuery(`tag:${tag}`)'),
-    "a tag chip click must set the query to tag:<tag>"
-  );
-});
-
-test("tag UI ships localized labels in both dictionaries", () => {
-  for (const key of ["tagAdd", "tagRemove", "tagPlaceholder", "tagsEmpty"]) {
-    assert.ok(
-      clientSource.includes(`"memory.explorer.${key}"`),
-      `memory.explorer.${key} key must exist`
-    );
-  }
-});
-
-// v0.6.3 directory view: a tag-folder tree backed by the directory endpoint.
-// Folders accordion via data-expanded controlled by MemoryExplorer-lifted
-// state; a memory entry click jumps back into the browser via onJump.
-test("directory sub-view mounts DirectoryPanel backed by the directory endpoint", () => {
-  assert.ok(
-    clientSource.includes("h(DirectoryPanel, { t, onJump: jumpToMemory, collapsed: dirCollapsed, setCollapsed: setDirCollapsed })"),
-    "the directory panel must be embedded as a sub-view with lifted collapse state"
-  );
-  assert.ok(
-    clientSource.includes("/api/dsh-mneme/directory"),
-    "DirectoryPanel must fetch the directory endpoint"
-  );
-  assert.ok(
-    clientSource.includes('className: "mneme-directory"'),
-    "the directory panel must render inside the .mneme-directory block"
-  );
-});
-
-test("directory folders accordion via data-expanded driven by MemoryExplorer state", () => {
-  assert.ok(
-    clientSource.includes('"data-expanded": String(open)'),
-    "folder expansion must be driven by the data-expanded attribute"
-  );
-  assert.ok(
-    clientSource.includes("const [dirCollapsed, setDirCollapsed] = useState({})"),
-    "collapse state must live in MemoryExplorer so tab switches keep it"
-  );
-  assert.ok(
-    clientSource.includes('"aria-expanded": String(open)'),
-    "folder headers must announce their expanded state"
-  );
-});
-
-test("directory entry click jumps to the memory detail via onJump", () => {
-  assert.ok(
-    clientSource.includes("onClick: () => onJump(m)"),
-    "clicking a memory entry must call onJump with the memory"
-  );
-  assert.ok(
-    clientSource.includes("formatDate(m.updated_at || m.created_at)"),
-    "entries must render a timestamp via formatDate"
-  );
-  assert.ok(
-    clientSource.includes("memory.directory.untagged"),
-    "untagged memories must be bucketed under a labeled fallback folder"
-  );
-});
-
-test("directory ships localized labels in both dictionaries", () => {
-  for (const key of ["memory.explorer.tabDirectory", "memory.directory.untagged", "memory.directory.loading", "memory.directory.empty"]) {
-    assert.ok(
-      clientSource.includes(`"${key}"`),
-      `${key} key must exist`
-    );
-  }
-});
-
-// issue #35: the delete confirm step must not depend on the host window.confirm —
-// in the DSH web host the native dialog is unreliable/blocked, so clicks
-// appeared to do nothing. Deletion is now an inline two-step confirm
-// (✕ → 确认？ → DELETE), gated per row by confirmingId.
-test("delete flow drops window.confirm for an inline two-step confirm", () => {
-  assert.equal(
-    /window\.confirm\(/.test(clientSource),
-    false,
-    "the delete path must not call the host window.confirm"
-  );
-  assert.ok(
-    clientSource.includes("confirmingId"),
-    "a per-row confirm state must gate the second click"
-  );
-  assert.ok(
-    clientSource.includes('"directory.deleteConfirmShort"'),
-    "the short inline confirm label must be localized"
-  );
-});
-
-// issue #35: a failed DELETE (network / 401 / 500 / deleted!==true) must be
-// visible — the old code swallowed it with a bare return and .catch(() => {}).
-test("delete failure surfaces visible error feedback", () => {
-  assert.ok(
-    clientSource.includes("delErrorId"),
-    "a per-row error state must track failed deletes"
-  );
-  assert.ok(
-    clientSource.includes('"directory.deleteError"'),
-    "the delete error message must be localized"
-  );
-  assert.ok(
-    clientSource.includes(".mneme-dir-error"),
-    "a failed row must get a visible error style"
-  );
-});
-
-// issue #38: the sidebar foot trigger must be disableable so it stops colliding
-// with other plugins that also claim the bottom-left footer slot (dsh-cost-meter
-// et al). The client reads the settings-over-config toggle from the /config
-// endpoint and renders no button when it is off; the 记忆库 conversation tab is
-// unaffected, so nothing is lost by hiding the entry.
-test("sidebar trigger is hidden when showSidebarTrigger is false (issue #38)", () => {
-  assert.ok(
-    clientSource.includes('apiFetch("/api/dsh-mneme/config")'),
-    "trigger must read the effective config from /api/dsh-mneme/config"
-  );
-  assert.ok(
-    /d\.config\?\.showSidebarTrigger !== false/.test(clientSource),
-    "trigger must hide when showSidebarTrigger is explicitly false and stay visible by default"
-  );
-  assert.ok(
-    clientSource.includes('"memory.settings.sidebarTriggerTitle"'),
-    "settings panel must expose a sidebar-trigger toggle"
-  );
-});

package/{dsh-mneme/test → test}/config.test.js

@@ -27,13 +27,6 @@ test("dream sliding window + implicit keep config defaults and bounds (v0.4.4)",
   assert.equal(off.dreamImplicitKeep, false, "implicit keep can be disabled");
 });
 
-test("injectTimePrefix is opt-in: default off, enabled explicitly (issue #34)", () => {
-  const cfg = Config({});
-  assert.equal(cfg.injectTimePrefix, false, "time prefix defaults to false (behavior unchanged)");
-  const on = Config({ injectTimePrefix: true });
-  assert.equal(on.injectTimePrefix, true, "explicit opt-in enables it");
-});
-
 test("dream explicit coverage threshold config defaults and bounds (v0.4.4 fix)", () => {
   const cfg = Config({});
   assert.equal(cfg.dreamMinExplicitCoverage, 0.5, "coverage threshold defaults to 0.5");
@@ -55,17 +48,3 @@ test("startup probe: the reranker module never statically imports transformers/o
   );
   assert.match(src, /await import\("@huggingface\/transformers"\)/, "transformers.js loads lazily via dynamic import");
 });
-
-test("showSidebarTrigger is on by default, disabled explicitly (issue #38)", () => {
-  const cfg = Config({});
-  assert.equal(cfg.showSidebarTrigger, true, "sidebar trigger defaults to visible (behavior unchanged)");
-  const off = Config({ showSidebarTrigger: false });
-  assert.equal(off.showSidebarTrigger, false, "explicit false hides the trigger");
-});
-
-test("escapePromptVariables is on by default, disabled explicitly (issue #40)", () => {
-  const cfg = Config({});
-  assert.equal(cfg.escapePromptVariables, true, "{{...}} escaping defaults to on (protects interpolate)");
-  const off = Config({ escapePromptVariables: false });
-  assert.equal(off.escapePromptVariables, false, "explicit false passes {{...}} through verbatim");
-});

package/{dsh-mneme/test → test}/dream.test.js

@@ -137,100 +137,6 @@ test("merge across types rejects", () => {
   assert.ok(errors.some((e) => e.includes("multiple types")), "multi-type error present");
 });
 
-// --- Issue #26 (P0): skipInvalid — 单条非法决策跳过，合法子集照常应用 ---------
-
-test("validateDecisions skipInvalid: a single invalid decision is skipped, valid subset survives", () => {
-  const snap = new Map([
-    ["p", { id: "p", type: "preference", title: "语言", content: "中文", importance: 3, archived: false, forgotten: false }],
-    ["j", { id: "j", type: "project", title: "插件", content: "内容", importance: 3, archived: false, forgotten: false }],
-    ["a", { id: "a", type: "project", title: "旧A", content: "过时A", importance: 3, archived: false, forgotten: false }],
-    ["b", { id: "b", type: "project", title: "旧B", content: "过时B", importance: 3, archived: false, forgotten: false }]
-  ]);
-  const decisions = [
-    { action: "merge", ids: ["p", "j"], keepSource: "p", title: "跨类型", content: "不应合并", importance: 4 },
-    { action: "archive", ids: ["a"], reason: "stale" },
-    { action: "archive", ids: ["b"], reason: "stale" }
-  ];
-  const { ok, errors, skipped } = validateDecisions(decisions, snap, { skipInvalid: true });
-  assert.equal(ok, true, `valid subset should survive, got: ${errors.join("; ")}`);
-  assert.equal(skipped.length, 1, "cross-type merge recorded as skipped");
-  assert.equal(skipped[0].index, 0, "the skipped one is the cross-type merge");
-  assert.match(skipped[0].error, /multiple types/, "skip reason mentions types");
-  // invalid merge spliced out of the caller's array; valid archives + implicit
-  // keeps for p/j survive (p/j were left unclaimed by the skipped merge)
-  assert.deepEqual(decisions.map((d) => d.action), ["archive", "archive", "keep", "keep"]);
-  assert.deepEqual(decisions[0].ids, ["a"]);
-  assert.ok(decisions.some((d) => d.action === "keep" && d.ids.includes("p")), "p auto-kept");
-  assert.ok(decisions.some((d) => d.action === "keep" && d.ids.includes("j")), "j auto-kept");
-});
-
-test("validateDecisions skipInvalid: an all-invalid batch still rejects (coverage floor guards truncation)", () => {
-  const snap = new Map([
-    ["p", { id: "p", type: "preference", title: "语言", content: "中文", importance: 3, archived: false, forgotten: false }],
-    ["j", { id: "j", type: "project", title: "插件", content: "内容", importance: 3, archived: false, forgotten: false }],
-    ["d1", { id: "d1", type: "decision", title: "决定", content: "内容D", importance: 3, archived: false, forgotten: false }],
-    ["b", { id: "b", type: "project", title: "旧B", content: "过时B", importance: 3, archived: false, forgotten: false }]
-  ]);
-  // both decisions are cross-type merges → both skipped → valid claims = 0
-  const decisions = [
-    { action: "merge", ids: ["p", "j"], keepSource: "p", title: "跨类型", content: "不应合并", importance: 4 },
-    { action: "merge", ids: ["d1", "b"], keepSource: "d1", title: "跨类型2", content: "不应合并", importance: 4 }
-  ];
-  const { ok, errors, skipped } = validateDecisions(decisions, snap, { skipInvalid: true });
-  assert.equal(ok, false, "no valid decisions left → whole batch rejected");
-  assert.equal(skipped.length, 2);
-  assert.ok(errors.some((e) => e.includes("coverage")), "coverage error present");
-  assert.equal(decisions.length, 2, "rejected batch left untouched (splice only on the success path)");
-});
-
-test("validateDecisions skipInvalid: runaway update count still rejects the whole batch (global cap)", () => {
-  const snap = new Map([
-    ["a", { id: "a", type: "project", title: "A", content: "旧A", importance: 3, archived: false, forgotten: false, created_at: "2020-01-01T00:00:00.000Z" }],
-    ["b", { id: "b", type: "project", title: "B", content: "旧B", importance: 3, archived: false, forgotten: false, created_at: "2020-01-01T00:00:00.000Z" }],
-    ["c", { id: "c", type: "project", title: "C", content: "旧C", importance: 3, archived: false, forgotten: false, created_at: "2020-01-01T00:00:00.000Z" }]
-  ]);
-  const decisions = [
-    { action: "update", ids: ["a"], content: "新A" },
-    { action: "update", ids: ["b"], content: "新B" },
-    { action: "update", ids: ["c"], content: "新C" }
-  ];
-  const { ok, errors } = validateDecisions(decisions, snap, { skipInvalid: true });
-  assert.equal(ok, false, "3 updates > default cap 2 → still rejects");
-  assert.ok(errors.some((e) => e.includes("too many update decisions")), "global cap error present");
-});
-
-// --- Issue #26 (P1): allowCrossTypeMerge — 显式放宽跨类型合并 -----------------
-
-test("validateDecisions allowCrossTypeMerge: cross-type merge is allowed when the flag is on", () => {
-  const snap = new Map([
-    ["p", { id: "p", type: "preference", title: "语言", content: "中文", importance: 3, archived: false, forgotten: false }],
-    ["j", { id: "j", type: "project", title: "插件", content: "内容", importance: 3, archived: false, forgotten: false }]
-  ]);
-  const decisions = [
-    { action: "merge", ids: ["p", "j"], keepSource: "p", title: "合并", content: "合并内容", importance: 4 }
-  ];
-  const { ok, errors } = validateDecisions(decisions, snap, { allowCrossTypeMerge: true });
-  assert.equal(ok, true, `cross-type merge allowed with the flag, got: ${errors.join("; ")}`);
-  assert.equal(decisions.length, 1, "no keep appended (both snapshot ids claimed)");
-});
-
-test("validateDecisions allowCrossTypeMerge off + skipInvalid: cross-type merge is skipped, not applied", () => {
-  const snap = new Map([
-    ["p", { id: "p", type: "preference", title: "语言", content: "中文", importance: 3, archived: false, forgotten: false }],
-    ["j", { id: "j", type: "project", title: "插件", content: "内容", importance: 3, archived: false, forgotten: false }],
-    ["a", { id: "a", type: "project", title: "旧A", content: "过时A", importance: 3, archived: false, forgotten: false }],
-    ["b", { id: "b", type: "project", title: "旧B", content: "过时B", importance: 3, archived: false, forgotten: false }]
-  ]);
-  const decisions = [
-    { action: "merge", ids: ["p", "j"], keepSource: "p", title: "跨类型", content: "不应合并", importance: 4 },
-    { action: "archive", ids: ["a"], reason: "stale" },
-    { action: "archive", ids: ["b"], reason: "stale" }
-  ];
-  const { ok, skipped } = validateDecisions(decisions, snap, { skipInvalid: true, allowCrossTypeMerge: false });
-  assert.equal(ok, true);
-  assert.equal(skipped.length, 1, "flag off → cross-type merge still skipped");
-});
-
 function dreamSetup() {
   const store = createStore(":memory:");
   const service = createService({ store, mirror: null, config: {} });
@@ -577,69 +483,6 @@ test("runDream fails safe on invalid decisions", async () => {
   store.close();
 });
 
-// --- Issue #26: 部分非法决策 → 跳过非法、应用合法子集、run 记为 degraded ----
-
-test("runDream applies the valid subset and records degraded when some decisions are invalid (Issue #26)", async () => {
-  const { store, service } = dreamSetup();
-  const a = service.saveWithDedupe({ type: "project", title: "项目A", content: "内容A", importance: 3 });
-  const b = service.saveWithDedupe({ type: "project", title: "项目B", content: "内容B", importance: 3 });
-  const c = service.saveWithDedupe({ type: "preference", title: "偏好C", content: "内容C", importance: 3 });
-  const d = service.saveWithDedupe({ type: "preference", title: "偏好D", content: "内容D", importance: 3 });
-  // 2 个合法同类型 merge + 1 个非法跨类型 merge（a×c，且 a/c 已被合法决策占用）
-  const decisions = [
-    { action: "merge", ids: [a.memory.id, b.memory.id], title: "项目总览", content: "AB 合并", importance: 4, keepSource: a.memory.id },
-    { action: "merge", ids: [c.memory.id, d.memory.id], title: "偏好总览", content: "CD 合并", importance: 4, keepSource: c.memory.id },
-    { action: "merge", ids: [a.memory.id, c.memory.id], title: "非法跨类型", content: "不应合并", importance: 4, keepSource: a.memory.id }
-  ];
-  const warnings = [];
-  const ctx = mockCtx({ onConsolidation: () => JSON.stringify(decisions) });
-  ctx.logger.warn = (m) => warnings.push(m);
-  const dream = createDreamScheduler({ thresholdCount: 1, thresholdChars: 0, delayMs: 0 });
-  const result = await dream.runDream(ctx, service, { dreamProvider: "deepseek", dreamModel: "deepseek-chat" });
-  assert.equal(result.status, "degraded", "partial-invalid run recorded as degraded");
-  assert.equal(result.ok, true, "degraded is ok for the scheduler baseline");
-  assert.equal(result.applied, 2, "2 valid merges applied");
-  assert.equal(result.skipped.length, 1, "cross-type merge skipped");
-  // valid merges landed
-  assert.equal(store.getById(a.memory.id).title, "项目总览", "keeper A updated");
-  assert.equal(store.getById(b.memory.id).archived, true, "B archived");
-  assert.equal(store.getById(c.memory.id).title, "偏好总览", "keeper C updated");
-  assert.equal(store.getById(d.memory.id).archived, true, "D archived");
-  // audit row records degraded + the skipped decision, never a fake ok
-  const run = store.listDreamRuns()[0];
-  assert.equal(run.status, "degraded", "audit row records degraded");
-  assert.equal(run.applied, 2, "audit row applied count");
-  assert.equal(run.outcome.skipped.length, 1, "audit outcome records the skipped decision");
-  assert.match(run.error, /skipped 1 invalid decision/, "audit error explains the skip");
-  assert.ok(warnings.some((m) => m.includes("skipping invalid decision")), "skip logged");
-  store.close();
-});
-
-test("runDream with dreamSkipInvalid=false keeps the old whole-batch rejection (Issue #26 opt-out)", async () => {
-  const { store, service } = dreamSetup();
-  const a = service.saveWithDedupe({ type: "project", title: "项目A", content: "内容A", importance: 3 });
-  const b = service.saveWithDedupe({ type: "project", title: "项目B", content: "内容B", importance: 3 });
-  const c = service.saveWithDedupe({ type: "preference", title: "偏好C", content: "内容C", importance: 3 });
-  const d = service.saveWithDedupe({ type: "preference", title: "偏好D", content: "内容D", importance: 3 });
-  const decisions = [
-    { action: "merge", ids: [a.memory.id, c.memory.id], title: "非法跨类型", content: "不应合并", importance: 4, keepSource: a.memory.id },
-    { action: "merge", ids: [b.memory.id, d.memory.id], title: "项目B", content: "合并", importance: 4, keepSource: b.memory.id }
-  ];
-  const warnings = [];
-  const ctx = mockCtx({ onConsolidation: () => JSON.stringify(decisions) });
-  ctx.logger.warn = (m) => warnings.push(m);
-  const dream = createDreamScheduler({ thresholdCount: 1, thresholdChars: 0, delayMs: 0 });
-  const result = await dream.runDream(ctx, service, {
-    dreamProvider: "deepseek", dreamModel: "deepseek-chat",
-    dreamSkipInvalid: false
-  });
-  assert.equal(result.ok, false, "opt-out rejects the whole batch");
-  assert.equal(store.listDreamRuns()[0].status, "failed", "audited as failed");
-  assert.equal(store.getById(a.memory.id).content, "内容A", "nothing changed");
-  assert.equal(store.getById(b.memory.id).archived, false, "valid-looking decision also not applied");
-  store.close();
-});
-
 // --- item ①: CAS guard against concurrent edits ----------------------------
 
 test("applyDecisions CAS: merge onto a concurrently-edited target is skipped as a conflict, not applied", () => {
@@ -1027,9 +870,7 @@ test("Bug8: runDream records llm_audit_logs rows for consolidation and summary",
   assert.deepEqual(summarize.related_memory_ids, [], "summary audit has no related ids");
   for (const row of rows) {
     assert.equal(row.status, "success");
-    // Issue #25: config route (dreamProvider/dreamModel) wins over the agent
-    // default — the audit must reflect the actually-used config route.
-    assert.equal(row.model_id, "deepseek:deepseek-chat");
+    assert.equal(row.model_id, "mock:stress-model");
     assert.ok(Number.isInteger(row.duration_ms) && row.duration_ms >= 0, "duration recorded");
     assert.equal(row.input_tokens, 0);
     assert.equal(row.output_tokens, 0);

package/{dsh-mneme/test → test}/graph-api.test.js

@@ -173,37 +173,3 @@ test("ego 2-hop over 100+ nodes stays under 50ms (spec §7)", async () => {
   assert.ok(data.edges.length > 0);
   assert.ok(ms < 50, `2-hop query took ${ms.toFixed(1)}ms, spec budget is 50ms`);
 });
-
-test("ego node heat projects from linked memory heat (v0.7.0)", async () => {
-  const { routes, service } = setup();
-  const mem = service.saveWithDedupe({ type: "history", title: "test", content: "test entity heat", importance: 3 });
-  const entity = service.createEntity({ name: "E", type: "project" });
-  service.saveRelation({ from_entity: entity.id, to_entity: entity.id, relation_type: "evidence", memory_id: mem.memory.id });
-  const route = routes.find((r) => r.path === "/api/dsh-mneme/semantic/graph/ego");
-  assert.ok(route, "ego route exists");
-  const res = new FakeRes();
-  await route.handler(req("/api/dsh-mneme/semantic/graph/ego?entity=E"), res);
-  const data = JSON.parse(res.body);
-  const node = data.nodes.find((n) => n.name === "E");
-  assert.ok(typeof node.heat === "number" && node.heat >= 0 && node.heat <= 1, "entity node has a heat value");
-  assert.ok(Math.abs(node.heat - 1) < 0.01, "fresh memory -> heat ~1.0 -> entity heat ~1.0");
-});
-
-test("ego node heat is null when heatEnabled=false (v0.7.0)", async () => {
-  const store = createStore(":memory:");
-  const service = createService({ store, mirror: null, config: { heatEnabled: false } });
-  const settings = createSettings(store.db);
-  const commands = { add: () => {}, remove: () => {}, list: () => [] };
-  const routes = [];
-  const ctx = { webServer: { register(route) { routes.push(route); return () => {}; } } };
-  createApi(ctx, service, settings, commands, null, undefined, "");
-  const mem = service.saveWithDedupe({ type: "history", title: "关联记忆", content: "实体关联", importance: 3 });
-  const entity = service.createEntity({ name: "F", type: "project" });
-  service.saveRelation({ from_entity: entity.id, to_entity: entity.id, relation_type: "evidence", memory_id: mem.memory.id });
-  const route = routes.find((r) => r.path === "/api/dsh-mneme/semantic/graph/ego");
-  const res = new FakeRes();
-  await route.handler(req("/api/dsh-mneme/semantic/graph/ego?entity=F"), res);
-  const data = JSON.parse(res.body);
-  assert.equal(data.nodes.find((n) => n.name === "F").heat, null, "heat disabled → entity heat is null");
-  store.close();
-});

package/{dsh-mneme/test → test}/hot-memory.test.js

@@ -49,35 +49,6 @@ test("estimateTokens counts CJK heavier than ASCII", () => {
   assert.ok(estimateTokens("中文内容") > estimateTokens("abcd"));
 });
 
-// --- entry defense: non-positive / non-integer bounds fall back to defaults ---
-// Bug: createHotMemory({ maxRounds: -1 }) made the eviction while-loop
-// `while (buffer.length > maxRounds)` unbounded — after the buffer emptied,
-// `0 > -1` stayed true and buffer.shift() on an empty array is a no-op, so
-// every add() spun forever. Non-integer values (1.5, NaN, null) were also
-// silently wrong. The fix clamps them to the 5/2000 defaults at the door.
-
-test("hot memory falls back to maxRounds=5 for non-positive/invalid values", () => {
-  for (const bad of [0, -1, 1.5, NaN, null]) {
-    const hot = createHotMemory({ maxRounds: bad, maxTokens: 10000 });
-    for (let i = 0; i < 8; i++) hot.add({ query: `第${i}轮`, response: "x" });
-    assert.equal(hot.rounds().length, 5, `maxRounds=${bad} must fall back to 5, no infinite loop`);
-    assert.ok(hot.getContext().includes("第7轮"), `maxRounds=${bad}: newest round survives`);
-    assert.ok(!hot.getContext().includes("第0轮"), `maxRounds=${bad}: oldest round evicted`);
-  }
-});
-
-test("hot memory falls back to maxTokens=2000 for non-positive/infinite values", () => {
-  for (const bad of [0, -1, Infinity]) {
-    const hot = createHotMemory({ maxRounds: 50, maxTokens: bad });
-    // 50 rounds at ~74 tokens each blow a 2000-token budget; the fallback must
-    // evict into (1, 50). A broken budget of 0/-1 would squeeze to 1 round and
-    // Infinity would keep all 50 — both are the pre-fix behavior.
-    for (let i = 0; i < 50; i++) hot.add({ query: `第${i}轮`, response: "长回答".repeat(40) });
-    const n = hot.rounds().length;
-    assert.ok(n > 1 && n < 50, `maxTokens=${bad} falls back to 2000 (kept ${n} rounds)`);
-  }
-});
-
 // --- service-level: BM25 fusion + semantic dedup + selective injection ---
 
 function toyVec(text) {