@heixui/dsh-session-suite 0.6.0 → 0.6.2

package/index.js

@@ -28,24 +28,14 @@ import { stripStepTagsFromStream } from './lib/step-tag-filter.js'
 export const name = 'session-suite'
 
 /**
- * `agents` is required for implicit memory extraction.
+ * The HTTP carrier is the only hard requirement; everything else degrades gracefully.
  *
- * Cordis dispatch is scope-filtered: "agent-scoped listeners receive only events from
- * sessions entered through that agent's context". A listener on the plugin's ROOT
- * context therefore never sees an agent's session events — which is why our
- * `session/event` handler was registered successfully, never fired, and left the pending
- * queue empty for six rounds of fixes.
- *
- * dsh-fish-memory (16k downloads) solves the same problem by injecting `agents`, taking
- * one with `ctx.agents.get(sid)`, and listening on `agent.ctx` — the agent's own context.
- * Its source records the trap in a comment: the Cordis events that exist are
- * `agent/inbox/inserted` and `agent/turn-stopping`, NOT `user/message` /
- * `assistant/message` (those are SessionEvent types carried by `session/event`).
- *
- * 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.
+ * 0.6.2: `agents` and `timer` were added for an implicit-extraction polling loop. That
+ * loop crashed the Host with `cannot get property "timer" without inject` on every
+ * 500ms tick. Both are removed. Do not re-add them without also proving the tick body
+ * cannot throw and that the timer is disposed with the effect.
  */
-export const inject = ['connection', 'agents']
+export const inject = ['connection']
 
 /**
  * @param {any} ctx plugin context

package/lib/memory-host.js

@@ -261,66 +261,20 @@ export function installMemory({ ctx, root, config, log }) {
   ),
   )
 
-  // --- 2b. Per-agent extraction trigger -----------------------------------
+  // --- 2b. Per-agent extraction trigger: REMOVED (0.6.2) -----------------
   //
-  // This is the trigger that actually works, copied from how dsh-fish-memory does it
-  // (16k downloads, demonstrably working). Two things had to be true and neither was:
+  // The 500ms agent-poll was the source of a Host crash: its timer callback
+  // called `ctx.interval` / `ctx.get("timer")` on a context that had not
+  // injected timer, producing `TypeError: cannot get property "timer" without
+  // inject` on EVERY tick until the Host died. The WorkBuddy transcript showed
+  // `at mount, _onTimeout, listOnTimeout` from our bundle.
   //
-  //   1. the listener must be registered on the AGENT's context, not the plugin root.
-  //      Cordis dispatch is scope-filtered, so a root-context `session/event` listener
-  //      receives nothing from an agent's session — which is exactly what we observed:
-  //      registered successfully, `lastExtraction` stayed null forever.
-  //   2. `agent/turn-stopping` is the hook that fires at the end of a turn on that
-  //      scope. `user/message` and `assistant/message` are SessionEvent TYPES carried
-  //      by `session/event`, not Cordis events — a trap fish-memory documents in a
-  //      comment after hitting it themselves.
-  //
-  // The agent is reached through `ctx.agents`, so this requires the `agents` service;
-  // index.js now injects it. Without it the feature degrades to off rather than throwing.
-  safe('per-agent extraction hook', () => {
-    const agents = ctx.get?.('agents')
-    if (!agents) return
-
-    /** @param {any} agent live agent @returns {void} */
-    const mountFor = agent => {
-      const agentCtx = agent?.ctx
-      if (!agentCtx || typeof agentCtx.on !== 'function') return
-      // 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
-        // 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
-            const cwd = resolveCwd(ctx, agent)
-            return extractAfterTurn({ ctx, engine, sessionId, cwd, log, config })
-          })
-          .finally(() => publishExtractionTrace(engine))
-          .catch(error => {
-            extractionTrace.stage = 'threw'
-            extractionTrace.reason = errorMessage(error)
-            publishExtractionTrace(engine)
-            log('implicit memory extraction failed', error)
-          })
-      })
-    }
-
-    // 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 => {
-        try {
-          mountFor(payload?.agent ?? payload)
-        } catch {
-          /* one bad agent must not stop the rest */
-        }
-      })
-    }
-  })
-
+  // A recurring timer that can throw on every tick is not an acceptable design
+  // for a plugin that runs inside a shared Host process. This whole trigger is
+  // removed, and `agents`/`timer` are dropped from inject. Implicit extraction
+  // stays off until a trigger can be proven safe. Do not re-add a polling loop
+  // without (1) injecting timer, (2) wrapping the tick body in try/catch, and
+  // (3) clearing the timer on dispose.
   // --- 3. Implicit extraction after a turn --------------------------------
   safe('implicit extraction', () => {
     // A startup marker, recorded before the listener is even registered.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@heixui/dsh-session-suite",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "Session checkpoints for DSH: rewind a conversation with its files and memory, permanent session deletion, and shareable exports",
   "type": "module",
   "main": "./index.js",