@try-works/dsh-memory 0.2.0 → 0.2.1

package/STATUS.md

@@ -28,10 +28,12 @@ What does NOT exist, by design: content indexing. The store documents a file ("W
 to read it"); it never extracts or indexes the file's contents, and a token that appears only
 inside a PDF is deliberately not searchable (pinned by the acceptance suite).
 
-Two integration decisions worth keeping: `refresh` skips the asset layer entirely (a supersede
-would hide the entry while its inventory refcount stayed, orphaning the blob), and an automatic
-capture over quota records a notice instead of asking, because `session/event` is reentrancy-
-sensitive and has no answerer; only `memory_asset_put` asks.
+Three integration decisions worth keeping: `refresh` skips the asset layer entirely (a supersede
+would hide the entry while its inventory refcount stayed, orphaning the blob); the generic
+`memory_forget` REFUSES an asset entry and names `memory_asset_forget`, which is the only path
+that decrements the inventory; and an automatic capture over quota records a notice instead of
+asking, because `session/event` is reentrancy-sensitive and has no answerer — only
+`memory_asset_put` asks.
 
 ## Store performance, 2026-11-01
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@try-works/dsh-memory",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Git-backed, workspace-scoped memory for the DeepSeek Harness (global layer + per-workspace layers)",
   "type": "module",
   "main": "src/index.js",

package/src/forget.js

@@ -40,6 +40,13 @@ export function isSuppressed(root, scope, text) {
 export function forget(store, id) {
   const entry = buildLedger(store.root).find((e) => e.id === id)
   if (!entry) throw new Error('no such entry: ' + id)
+  // An ASSET entry is removed by memory_asset_forget, which also decrements the inventory and
+  // deletes the object when it is the last reference. This function only marks an entry
+  // superseded, so allowing it here would hide the asset while the blob stayed in the store and in
+  // LFS with nothing pointing at it - the same orphan the refresh pass refuses to create.
+  if (entry.layer === 'asset') {
+    throw new Error('asset entries are removed with memory_asset_forget, not memory_forget (id ' + id + ')')
+  }
   const scope = entry.file.startsWith('global/') ? 'global' : entry.file.split('/')[1]
   const hash = suppressionHash(scope, entry.content)
   appendTombstone(store.root, { hash, scope, id, createdAt: new Date().toISOString(), reason: 'user-forget' })