@heixui/dsh-session-suite 0.6.0 → 0.6.1

package/index.js

@@ -45,7 +45,7 @@ export const name = 'session-suite'
  * Everything else stays optional: the HTTP carrier is still the only hard requirement,
  * and a Host without `agents` degrades to no implicit extraction rather than failing.
  */
-export const inject = ['connection', 'agents']
+export const inject = ['connection', 'agents', 'timer']
 
 /**
  * @param {any} ctx plugin context

package/lib/memory-host.js

@@ -288,13 +288,21 @@ export function installMemory({ ctx, root, config, log }) {
       // One listener per agent; the context disposes it with the agent.
       agentCtx.on('agent/turn-stopping', () => {
         const sessionId = agent?.sessionId
-        if (typeof sessionId !== 'string' || !sessionId) return
+        if (typeof sessionId !== 'string' || !sessionId) {
+          extractionTrace.reason = `turn-stopping fired but agent.sessionId was ${typeof sessionId}`
+          publishExtractionTrace(engine)
+          return
+        }
         // The model choice lives in settings.json, not in the patch config.
         void engine
           .settings()
           .then(settings => {
             const chosen = settings?.implicitModel ?? config?.implicitModel
-            if (!chosen || chosen === 'off') return undefined
+            if (!chosen || chosen === 'off') {
+              extractionTrace.reason = 'no implicit model is selected in settings'
+              publishExtractionTrace(engine)
+              return undefined
+            }
             const cwd = resolveCwd(ctx, agent)
             return extractAfterTurn({ ctx, engine, sessionId, cwd, log, config })
           })
@@ -308,16 +316,48 @@ export function installMemory({ ctx, root, config, log }) {
       })
     }
 
-    // Mount for agents that already exist, and for every one created later.
-    for (const agent of agents.list?.() ?? []) mountFor(agent)
-    if (typeof ctx.on === 'function') {
-      ctx.on('agent/created', payload => {
+    // Mount by POLLING, not by an event.
+    //
+    // The first version listened for `agent/created`, which never delivered a usable
+    // agent — the hook stayed unmounted and the trace stayed null, the same symptom as
+    // every earlier attempt. dsh-fish-memory does not use an event either: it runs
+    // `ctx.effect(() => ctx.interval(tick, 500))` and calls `agents.list()` every half
+    // second, mounting on any agent it has not seen yet. That is what makes its version
+    // work, and its source notes the tradeoff — the tick is cheap because `agents.list()`
+    // reads memory.
+    //
+    // Polling also handles the case an event cannot: the agent instance is REPLACED when
+    // a session is resumed, a plugin reloads, or the Host rebuilds it. A one-shot
+    // listener dies with the old instance; a poll re-mounts on the new one.
+    const mounted = new WeakSet()
+    const tick = () => {
+      let live = []
+      try {
+        live = agents.list?.() ?? []
+      } catch {
+        return
+      }
+      for (const agent of live) {
+        if (!agent || mounted.has(agent)) continue
+        // Subagents only do one job; a memory store for them is wasted work.
+        if (agent.origin === 'subagent' || agent.delegationDepth > 0) continue
+        mounted.add(agent)
         try {
-          mountFor(payload?.agent ?? payload)
-        } catch {
-          /* one bad agent must not stop the rest */
+          mountFor(agent)
+        } catch (error) {
+          log('could not mount the extraction hook on an agent', error)
         }
-      })
+      }
+    }
+    // First pass immediately, so an already-running session is covered without waiting.
+    tick()
+    // Then every half second. `ctx.interval` is the Cordis timer facility; fall back to
+    // setInterval when it is absent so the feature still works on a leaner Host.
+    if (typeof ctx.interval === 'function') ctx.effect(() => ctx.interval(tick, 500))
+    else if (typeof setInterval === 'function') {
+      const handle = setInterval(tick, 500)
+      handle.unref?.()
+      ctx.effect(() => () => clearInterval(handle))
     }
   })
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@heixui/dsh-session-suite",
-  "version": "0.6.0",
+  "version": "0.6.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",