@try-works/dsh-memory 0.1.7 → 0.1.9

package/package.json

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

package/src/client/index.js

@@ -358,6 +358,11 @@ window.__ModuleLoader__.load({
      */
     const MEMORY_STATE_NONE = { kind: 'none', meta: null, seq: null, time: null }
     const MEMORY_STATE_UNAVAILABLE = { kind: 'unavailable', meta: null, seq: null, time: null }
+    // Distinct from NONE on purpose. PTC mode dispatches a tool from inside a run_code program and
+    // records tool/ptc-dispatch, which carries the rendered text but NO structured value, so there
+    // is genuinely nothing to draw - and telling the reader 'memory_map has not run here' would be
+    // false.
+    const MEMORY_STATE_NESTED = { kind: 'nested', meta: null, seq: null, time: null }
 
     /**
      * Find the newest memory_map result in the window and say what it carried.
@@ -370,10 +375,13 @@ window.__ModuleLoader__.load({
       const entries = window && Array.isArray(window.entries) ? window.entries : null
       if (!entries) return MEMORY_STATE_NONE
       const calls = Object.create(null)
+      let nested = false
       for (let i = 0; i < entries.length; i += 1) {
         const entry = entries[i]
         const event = entry && entry.type === 'event' ? entry.event : null
         if (event && event.type === 'tool/call' && event.data && event.data.name === 'memory_map') calls[event.data.callId] = true
+        // A nested dispatch is a real call the agent made; it simply has no structured result.
+        if (event && event.type === 'tool/ptc-dispatch' && event.data && event.data.name === 'memory_map') nested = true
       }
       // Newest first: the tab describes the session as it stands now, not as it first was.
       for (let i = entries.length - 1; i >= 0; i -= 1) {
@@ -392,7 +400,7 @@ window.__ModuleLoader__.load({
         if (meta === null || typeof meta !== 'object') return { kind: 'empty', meta: null, seq: event.seq, time: event.time }
         return { kind: 'ready', meta: meta, seq: event.seq, time: event.time }
       }
-      return MEMORY_STATE_NONE
+      return nested ? MEMORY_STATE_NESTED : MEMORY_STATE_NONE
     }
 
     /**
@@ -493,6 +501,15 @@ window.__ModuleLoader__.load({
           ],
         }
       }
+      if (state.kind === 'nested') {
+        return {
+          title: 'memory_map ran here, but not as a readable result',
+          lines: [
+            'This session called memory_map from inside a run_code program. Those calls are recorded as nested dispatches, which keep the rendered text and not the structured value this tab draws.',
+            'Calling memory_map directly - scope "session" or "all" - records a result the panels can read.',
+          ],
+        }
+      }
       return {
         title: 'No memory_map result in this session yet',
         lines: [

package/src/tools.js

@@ -321,7 +321,17 @@ export function registerTools(ctx, store, extras = {}) {
           session: { type: 'object', additionalProperties: true },
         },
       },
+      // The text face, for the agent and for any caller without a UI.
       render: (_args, value) => [{ type: 'text', text: renderMap(value) }],
+      // The STRUCTURED face, and the only reason the Memory tab and the inline card can draw
+      // anything. A tool result reaches the session event with a `meta` field ONLY when the tool
+      // declares presentationMeta (core/tools/src/index.ts:1845-1859), and that field is what
+      // ui-chat forwards to a view (conversation-nodes/tool.ts:77: meta: match.event.data.meta).
+      // Without this, memory_map rendered its snapshot to prose and threw the value away: the tab
+      // showed 'the result carried no snapshot' while the same call displayed 2,197 entries in
+      // the conversation. The rendered text is not a substitute - rebuilding panels by parsing it
+      // would be a second source of truth.
+      presentationMeta: (_args, value) => value,
     },
     async execute(args, exec) {
       const scope = args.scope === 'store' || args.scope === 'session' ? args.scope : 'all'