dsh-checklists 0.5.0 → 0.5.1

package/lib/index.js

@@ -54,10 +54,12 @@ export function apply(ctx, config = {}) {
     const activity = new Map();
     const gateLog = [];
     const events = ctx;
-    ctx.effect(() => events.on('tools/result', (payload) => {
-        const event = payload;
-        const toolName = event.exec?.name;
-        const agentId = event.exec?.agent?.id;
+    ctx.effect(() => events.on('tools/result', (exec) => {
+        // `tools/result` is emitted as (execution, result); the execution carries
+        // the tool name and the calling agent.
+        const call = exec;
+        const toolName = call?.name;
+        const agentId = call?.agent?.id;
         if (typeof toolName !== 'string' || typeof agentId !== 'string')
             return;
         const current = activity.get(agentId) ?? { mutated: false, checklistWrite: false };
@@ -67,11 +69,7 @@ export function apply(ctx, config = {}) {
             current.checklistWrite = true;
         activity.set(agentId, current);
     }));
-    ctx.effect(() => events.on('agent/turn-stopping', (payload) => {
-        const event = payload;
-        const agentId = event.agent?.id;
-        if (typeof agentId !== 'string')
-            return;
+    const evaluate = (agentId, source) => {
         const current = activity.get(agentId) ?? { mutated: false, checklistWrite: false };
         activity.set(agentId, { mutated: false, checklistWrite: false });
         const doc = store.peek(agentId);
@@ -80,17 +78,34 @@ export function apply(ctx, config = {}) {
             for (const list of doc.lists)
                 doing += countState(list.items, 'doing');
         const decision = decideNudge(current, doing);
-        if (!decision.nudge && process.env.DSH_CHECKLIST_DEBUG !== '1')
-            return;
+        // Log every decision while the gate is in observe mode: an empty log must
+        // mean "the event never fired", never "the answer was no".
         gateLog.push({
             at: new Date().toISOString(),
             sessionId: agentId,
+            source,
             nudge: decision.nudge,
             reason: decision.reason,
             doing,
         });
         if (gateLog.length > 20)
             gateLog.shift();
+    };
+    ctx.effect(() => events.on('agent/turn-stopping', (payload) => {
+        const event = payload;
+        if (typeof event.agent?.id === 'string')
+            evaluate(event.agent.id, 'turn-stopping');
+    }));
+    // Second observation point. `agent/status` → idle also marks a turn end, so
+    // keeping both in the log tells us which one the runtime actually delivers —
+    // and leaves the gate working if `turn-stopping` turns out not to reach a
+    // root listener.
+    ctx.effect(() => events.on('agent/status', (payload) => {
+        const event = payload;
+        if (event.status !== 'idle')
+            return;
+        if (typeof event.agent?.id === 'string')
+            evaluate(event.agent.id, 'status');
     }));
     let loggedScope = false;
     ctx.inject(['systemPrompt'], (promptCtx) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-checklists",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "Session-scoped checklists for DeepSeek Harness: the agent tracks work as a tree, the model always sees a compact digest, and a read-only sidebar tab mirrors it.",
   "type": "module",
   "main": "lib/index.js",