@modusensus/dsh-mneme 0.6.5 → 0.6.6

package/lib/dream/tag-extractor.js

@@ -78,8 +78,12 @@ export async function runAutoTag({ ctx, service, config, route }) {
     ? config.autoTagMaxPerRun
     : 10;
   // Retained = post-consolidation active memories, newest first, capped.
+  // Skip memories that already carry live tags — otherwise every run re-picks
+  // the same "newest" batch (tagging doesn't bump updated_at) and older
+  // memories starve; manual tags are also left untouched this way.
   const memories = service.all()
     .filter((m) => !m.forgotten && !m.archived && !m.session_disposed_at && m.type !== "summary")
+    .filter((m) => (service.getMemoryTags?.(m.id) ?? []).length === 0)
     .sort((a, b) => {
       const ta = String(a.updated_at ?? "");
       const tb = String(b.updated_at ?? "");
@@ -137,7 +141,10 @@ export async function runAutoTag({ ctx, service, config, route }) {
   try {
     service.transaction(() => {
       for (const { id, tags } of toWrite) {
-        service.applyMemoryTags(id, tags);
+        // Merge with existing live tags (never clobber manual tags that landed
+        // between selection and write).
+        const existing = service.getMemoryTags?.(id) ?? [];
+        service.applyMemoryTags(id, [...new Set([...existing, ...tags])]);
         tagged++;
       }
     });

package/lib/service.js

@@ -373,7 +373,9 @@ export function createService({ store, mirror, config, onWrite, logger }) {
           .sort((a, b) => b.score - a.score)
       : hits.map((m) => ({ ...m, source: "tag" }));
     const result = hits.slice(0, topK);
-    touchRecalled(result);
+    // Only count toward recall stats/forgetting when the caller asked for
+    // recording — the panel's `tag:` search must not pollute the curve.
+    if (options.recordRecall) touchRecalled(result);
     return result;
   }
 

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, 7 model tools, automatic injection, session summarization, user profile/rules, custom slash commands, vector (semantic) search, and a Web GUI panel",
-  "version": "0.6.5",
+  "version": "0.6.6",
   "license": "MIT",
   "repository": {
     "type": "git",

package/src/dream/tag-extractor.js

@@ -78,8 +78,12 @@ export async function runAutoTag({ ctx, service, config, route }) {
     ? config.autoTagMaxPerRun
     : 10;
   // Retained = post-consolidation active memories, newest first, capped.
+  // Skip memories that already carry live tags — otherwise every run re-picks
+  // the same "newest" batch (tagging doesn't bump updated_at) and older
+  // memories starve; manual tags are also left untouched this way.
   const memories = service.all()
     .filter((m) => !m.forgotten && !m.archived && !m.session_disposed_at && m.type !== "summary")
+    .filter((m) => (service.getMemoryTags?.(m.id) ?? []).length === 0)
     .sort((a, b) => {
       const ta = String(a.updated_at ?? "");
       const tb = String(b.updated_at ?? "");
@@ -137,7 +141,10 @@ export async function runAutoTag({ ctx, service, config, route }) {
   try {
     service.transaction(() => {
       for (const { id, tags } of toWrite) {
-        service.applyMemoryTags(id, tags);
+        // Merge with existing live tags (never clobber manual tags that landed
+        // between selection and write).
+        const existing = service.getMemoryTags?.(id) ?? [];
+        service.applyMemoryTags(id, [...new Set([...existing, ...tags])]);
         tagged++;
       }
     });

package/src/service.js

@@ -373,7 +373,9 @@ export function createService({ store, mirror, config, onWrite, logger }) {
           .sort((a, b) => b.score - a.score)
       : hits.map((m) => ({ ...m, source: "tag" }));
     const result = hits.slice(0, topK);
-    touchRecalled(result);
+    // Only count toward recall stats/forgetting when the caller asked for
+    // recording — the panel's `tag:` search must not pollute the curve.
+    if (options.recordRecall) touchRecalled(result);
     return result;
   }
 

package/test/tag.test.js

@@ -186,6 +186,24 @@ test("runAutoTag writes validated tags into entity_attrs", async () => {
   store.close();
 });
 
+test("runAutoTag: skips already-tagged memories, merges into fresh ones", async () => {
+  const { store, service } = makeService();
+  const a = seed(service, "Linux", "内核");
+  const b = seed(service, "考研", "公共管理");
+  store.setMemoryTags(a.id, ["已有"]); // manual tag
+  const text = JSON.stringify([
+    { id: a.id, tags: ["linux"] },
+    { id: b.id, tags: ["考研"] }
+  ]);
+  const ctx = makeCtx([text]);
+  const result = await runAutoTag({ ctx, service, config: { dreamProvider: "d", dreamModel: "m" } });
+  assert.equal(result.ok, true);
+  assert.equal(result.tagged, 1, "only the untagged memory is tagged");
+  assert.deepEqual(store.getMemoryTags(a.id), ["已有"], "manual tags left untouched");
+  assert.deepEqual(store.getMemoryTags(b.id), ["考研"]);
+  store.close();
+});
+
 test("runAutoTag fail-safe: garbage / aborted LLM output never throws and writes nothing", async () => {
   const { store, service } = makeService();
   const a = seed(service, "Alpha", "正文");