@heixui/dsh-session-suite 0.3.0 → 0.3.1

package/lib/memory-host.js

@@ -110,6 +110,21 @@ export function installMemory({ ctx, root, config, log }) {
    */
   const recallByScope = new Map()
   let lastRecall = ''
+  // Diagnostics for the settings page: "did the last rebuild fail, and why" must
+  // be answerable, because the failure mode is invisible otherwise — the model
+  // simply behaves as if the memory is missing.
+  let lastRecallFailed = false
+  let lastRecallError = ''
+
+  /**
+   * How long a computed recall may be reused before it must be recomputed.
+   *
+   * `agent/pre-step` normally refreshes it every step, so this only matters when
+   * a step does not run the hook (a resumed session, a step served from a cached
+   * decision). Without a bound, a memory written elsewhere could stay invisible
+   * for the life of the process.
+   */
+  const RECALL_TTL_MS = 60_000
 
   // --- 1. Recall into the system prompt -----------------------------------
   const systemPrompt = ctx.get('systemPrompt')
@@ -122,7 +137,11 @@ export function installMemory({ ctx, root, config, log }) {
             order: 900,
             text: context => {
               const scoped = context?.scope ? recallByScope.get(context.scope) : undefined
-              return scoped?.text || lastRecall || ''
+              // Expiry is checked here rather than only at write time: a cached
+              // value outlives its usefulness the moment the library changes
+              // underneath it, and this is the only place that can see the age.
+              if (scoped && Date.now() - scoped.at < RECALL_TTL_MS) return scoped.text
+              return lastRecall || ''
             },
           }),
         'session-suite: memory context',
@@ -160,7 +179,19 @@ export function installMemory({ ctx, root, config, log }) {
           const text = await buildRecallText({ engine, cwd, question, config })
           if (agent) recallByScope.set(agent, { text, at: Date.now() })
           lastRecall = text
+          lastRecallFailed = false
         } catch (error) {
+          // A failed rebuild must not leave the previous answer in place.
+          //
+          // Observed in the wild: one failed rebuild, and every later turn kept
+          // injecting the memory text computed BEFORE it — a memory written after
+          // that point was invisible for the rest of the process, which reads as
+          // "the other workspace does not know". Stale recall is worse than no
+          // recall: empty at least tells the truth.
+          lastRecall = ''
+          if (agent) recallByScope.delete(agent)
+          lastRecallFailed = true
+          lastRecallError = error instanceof Error ? error.message : String(error)
           log('memory recall failed', error)
         }
         return decision
@@ -329,7 +360,19 @@ export function installMemory({ ctx, root, config, log }) {
     }, 'session-suite: memory tool')
   }
 
-  return { engine, dispose: () => {} }
+  return {
+    engine,
+    dispose: () => {},
+    // Exposed so the settings page and the status endpoint can report whether the
+    // last recall rebuild failed. Without this the failure is silent: the model
+    // simply acts as if the memory is not there.
+    recallStatus: () => ({
+      failed: lastRecallFailed,
+      error: lastRecallError,
+      hasText: Boolean(lastRecall) || recallByScope.size > 0,
+      scopes: recallByScope.size,
+    }),
+  }
 }
 
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@heixui/dsh-session-suite",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Session checkpoints for DSH: rewind a conversation with its files and memory, permanent session deletion, and shareable exports",
   "type": "module",
   "main": "./index.js",
@@ -56,6 +56,7 @@
     "THIRD-PARTY.md"
   ],
   "dsh": {
+    "icon": "./icon.svg",
     "bundle": {
       "patch": "./cordis.patch.yml"
     },