claude-mem-lite 3.86.0 → 3.88.0

package/server.mjs

@@ -50,7 +50,7 @@ import { ensureRegistryDb, collectRegistryStats, listResourcesRanked, formatRegi
 import { IMPORT_STRING_FIELDS, importResource, removeResource, reindexResources } from './lib/registry-core.mjs';
 import { searchResources } from './registry-retriever.mjs';
 import { probeOtherSources as probeIdSources, bucketIdTokens, splitDeferredTokens } from './lib/id-routing.mjs';
-import { saveObservation } from './lib/save-observation.mjs';
+import { saveObservation, formatSupersedeSkipped } from './lib/save-observation.mjs';
 import { applyObsUpdate } from './lib/observation-write.mjs';
 import { EXPORT_COLUMNS_SQL } from './lib/export-columns.mjs';
 import { liveObsFilterSql } from './lib/inject-search-core.mjs';
@@ -58,7 +58,7 @@ import { recallByFile } from './lib/recall-core.mjs';
 import { fetchRecent } from './lib/recent-core.mjs';
 import { AUTO_MERGE_THRESHOLD } from './lib/dedup-constants.mjs';
 import {
-  insertDeferred, listOpenWithOrdinal, dropDeferred,
+  insertDeferred, listOpenWithOrdinal, dropDeferred, formatDropReasonHint,
   resolveDeferredIds, closeDeferredItems,
   getDeferredByIds, formatDeferredDetail,
   searchDeferredWork, formatDeferredSearchTrailer,
@@ -853,7 +853,9 @@ server.registerTool(
         // Resolve INSIDE tx + after dedup check so duplicate replays don't throw on
         // already-closed items. Mirrors mem-cli.mjs cmdSave shape.
         if (args.closes_deferred && args.closes_deferred.length > 0) {
-          closesIds = resolveDeferredIds(db, project, args.closes_deferred);
+          // D#195: 'dropped' is closable by the close verb (kept in sync with
+          // mem-cli.mjs cmdSave — same policy, both faces).
+          closesIds = resolveDeferredIds(db, project, args.closes_deferred, { allowStatuses: ['open', 'dropped'] });
           closeDeferredItems(db, closesIds, r.id);
         }
         return r;
@@ -869,7 +871,10 @@ server.registerTool(
     }
 
     if (result.kind === 'duplicate') {
-      return { content: [{ type: 'text', text: `Skipped: similar to existing #${result.existingId} in project "${project}". Use mem_get(ids=[${result.existingId}]) to review.` }] };
+      // D#201: this branch returns before the note below, so it renders its own.
+      const dupSkip = formatSupersedeSkipped(result.supersedeSkipped);
+      const dupText = `Skipped: similar to existing #${result.existingId} in project "${project}". Use mem_get(ids=[${result.existingId}]) to review.`;
+      return { content: [{ type: 'text', text: dupSkip ? `${dupText}\n${dupSkip}` : dupText }] };
     }
 
     const lessonNote = result.lessonCaptured ? ` 💡lesson captured` : '';
@@ -884,7 +889,10 @@ server.registerTool(
     // every save) — fill-only-empty, so an agent acting on the nudge still wins.
     const enrichNote = shouldQueueSaveEnrich(result) && queueSaveEnrich(result.id)
       ? ' (background enrichment queued)' : '';
-    return { content: [{ type: 'text', text: `Saved as observation #${result.id} [${result.type}] in project "${project}".${lessonNote}${closedNote}${supersededNote}${enrichNote}${nudge}` }] };
+    // D#201: on its own line rather than inside the success sentence.
+    const skipNote = formatSupersedeSkipped(result.supersedeSkipped);
+    const savedText = `Saved as observation #${result.id} [${result.type}] in project "${project}".${lessonNote}${closedNote}${supersededNote}${enrichNote}${nudge}`;
+    return { content: [{ type: 'text', text: skipNote ? `${savedText}\n${skipNote}` : savedText }] };
   })
 );
 
@@ -960,7 +968,11 @@ server.registerTool(
     if (r.changed === 0) {
       return { content: [{ type: 'text', text: `D#${realId} was not in 'open' status — drop is a no-op.` }] };
     }
-    return { content: [{ type: 'text', text: `Dropped D#${realId} in project "${project}". Reason: ${args.reason}` }] };
+    // D#195 (c): same advisory as the CLI's `defer drop`, so an agent reaching
+    // this through MCP gets the same steer toward `mem_save(closes_deferred)`.
+    const hint = formatDropReasonHint(args.reason);
+    const dropText = `Dropped D#${realId} in project "${project}". Reason: ${args.reason}`;
+    return { content: [{ type: 'text', text: hint ? `${dropText}\n${hint}` : dropText }] };
   })
 );