dsh-memory-recall 0.2.0 → 0.2.1

package/README.md

@@ -173,7 +173,7 @@ Other guards:
 
 ## Requirements
 
-- dsh 0.1.5 or later, with a `web` profile
+- dsh 0.1.5 or later, with a `web` profile — from 0.2.1 also dsh 0.1.7+ and 0.2.x
 - Node 22+
 - No runtime dependencies — the client bundle requires only `react`
 

package/dsh.plugin.json

@@ -1,6 +1,6 @@
 {
   "id": "dsh-memory-recall",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "main": "./src/index.js",
   "description": "Persistent memory for dsh in the style of Hermes Agent: a memory tool that keeps a user profile and the model's own notes, delivered to every chat as a logged context message — at the start, after a change and after compaction — and a session_search tool that reads earlier chats only after the user approves, with a Memory panel to see and edit what is kept.",
   "engines": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-memory-recall",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Persistent memory for dsh in the style of Hermes Agent: a memory tool that keeps a user profile and the model's own notes, delivered to every chat as a logged context message — at the start, after a change and after compaction — and a session_search tool that reads earlier chats only after the user approves, with a Memory panel to see and edit what is kept.",
   "type": "module",
   "main": "./src/index.js",

package/src/index.js

@@ -113,30 +113,62 @@ export async function apply(ctx, config = {}, deps = {}) {
     tooling.error = 'this dsh has no tool registry';
   }
 
-  // ---- each new chat: memory in its prompt, frozen for its lifetime ---
-  const briefed = new WeakSet();
+  // ---- each new chat: memory guidance in its prompt, frozen for its
+  // lifetime. dsh 0.1.7 removed `agent/session-start`; `agent/created` and
+  // `agent/disposed` exist on every dsh from 0.1.0-rc.7 through 0.2.x, so
+  // they are the seam — session-start stays wired as a back-compat path,
+  // deduped by the same map.
+  /** Agent contexts already carrying the section -> the registering fiber. */
+  const briefed = new WeakMap();
   // sessionId -> what that chat's prompt was given, for the composer chip.
   const briefings = createBriefings();
-  ctx.on?.('agent/session-start', (payload) => {
-    const agent = payload?.agent;
+
+  const installGuidance = (agent) => {
     const agentCtx = agent?.ctx;
     if (!agentCtx || briefed.has(agentCtx) || !isTopLevel(agent)) return;
-    briefed.add(agentCtx);
     try {
-      const prompt = service(agentCtx, 'systemPrompt');
-      if (typeof prompt?.section !== 'function') return;
       // Read now, once: this text is what the chat keeps, so a save made later
       // in it never rewrites its prompt (which would cost the cached prefix).
       // The guidance only: fixed text, so the cached prompt prefix holds.
       const text = renderSection(null, store.limits, { tools: tooling.ok });
-      const register = () => prompt.section({ name: SECTION_NAME, order: SECTION_ORDER, text });
-      if (typeof agentCtx.effect === 'function') agentCtx.effect(register, 'dsh-memory-recall: memory snapshot');
-      else register();
+      // `inject` rather than `get`: at agent/created the prompt service may not
+      // be resolvable yet; the callback runs once it is, and the fiber's
+      // dispose() takes the section back out.
+      const fiber = agentCtx.inject?.(['systemPrompt'], (scope) => {
+        if (!isTopLevel(agent)) return; // the header may have attached since created
+        try {
+          scope.systemPrompt?.section?.({ name: SECTION_NAME, order: SECTION_ORDER, text });
+        } catch {
+          // Already registered (e.g. after a plugin reload): a duplicate name throws.
+        }
+      });
+      if (fiber) briefed.set(agentCtx, fiber);
     } catch (error) {
-      // Losing the snapshot costs this chat its memory, never the chat.
+      // Losing the guidance costs this chat its memory, never the chat.
       ctx.logger?.warn?.(`[dsh-memory-recall] no memory for this chat: ${error?.message ?? error}`);
     }
+  };
+
+  const uninstallGuidance = (agent) => {
+    const agentCtx = agent?.ctx;
+    const fiber = agentCtx && briefed.get(agentCtx);
+    if (!fiber) return;
+    briefed.delete(agentCtx);
+    try {
+      fiber.dispose?.();
+    } catch {
+      // Teardown must not throw.
+    }
+  };
+
+  // Agents that already exist when this plugin mounts (a reload).
+  ctx.inject?.(['agents'], (scope) => {
+    for (const agent of scope.agents?.list?.() ?? []) installGuidance(agent);
   });
+  ctx.on?.('agent/created', (payload) => installGuidance(payload?.agent));
+  ctx.on?.('agent/disposed', (payload) => uninstallGuidance(payload?.agent));
+  // Older hosts fire this instead of agent/created; the map dedupes.
+  ctx.on?.('agent/session-start', (payload) => installGuidance(payload?.agent));
 
   // ---- the memory itself: a logged context message, whenever the chat
   // cannot see the current version (start, after compaction, after a change).