@scotthamilton77/sidekick 0.1.13 → 0.1.15

package/dist/daemon.js

@@ -54,7 +54,7 @@ var require_events = __commonJS({
   "../types/dist/events.js"(exports2) {
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
-    exports2.UI_EVENT_VISIBILITY = exports2.UI_EVENT_TYPES = exports2.HOOK_NAMES = void 0;
+    exports2.UI_EVENT_VISIBILITY = exports2.DecisionEvents = exports2.UI_EVENT_TYPES = exports2.HOOK_NAMES = void 0;
     exports2.isHookEvent = isHookEvent;
     exports2.isTranscriptEvent = isTranscriptEvent;
     exports2.isSessionStartEvent = isSessionStartEvent;
@@ -162,9 +162,30 @@ var require_events = __commonJS({
       // Transcript events
       "transcript:emitted",
       "transcript:pre-compact",
+      // Bulk processing lifecycle events
+      "bulk-processing:start",
+      "bulk-processing:finish",
       // General error
       "error:occurred"
     ];
+    exports2.DecisionEvents = {
+      /** Emitted when a runtime decision is recorded. */
+      decisionRecorded(context, payload) {
+        return {
+          type: "decision:recorded",
+          time: Date.now(),
+          source: "daemon",
+          context: {
+            sessionId: context.sessionId,
+            correlationId: context.correlationId,
+            traceId: context.traceId,
+            hook: context.hook,
+            taskId: context.taskId
+          },
+          payload
+        };
+      }
+    };
     exports2.UI_EVENT_VISIBILITY = {
       // Timeline events (user-visible state changes)
       "reminder:staged": "timeline",
@@ -200,7 +221,9 @@ var require_events = __commonJS({
       "session-summary:skipped": "log",
       "resume-message:skipped": "log",
       "transcript:emitted": "log",
-      "transcript:pre-compact": "log"
+      "transcript:pre-compact": "log",
+      "bulk-processing:start": "log",
+      "bulk-processing:finish": "log"
     };
   }
 });
@@ -44798,7 +44821,8 @@ var require_structured_logging = __commonJS({
           payload: {
             hook: context.hook,
             cwd: metadata.cwd,
-            mode: metadata.mode
+            mode: metadata.mode,
+            input: metadata.input
           }
         };
       },
@@ -44820,7 +44844,8 @@ var require_structured_logging = __commonJS({
             hook: context.hook,
             durationMs: metadata.durationMs,
             reminderReturned: state?.reminderReturned,
-            responseType: state?.responseType
+            responseType: state?.responseType,
+            returnValue: state?.returnValue
           }
         };
       },
@@ -45233,6 +45258,48 @@ var require_structured_logging = __commonJS({
             errorStack: state.errorStack
           }
         };
+      },
+      // --- Bulk Processing Lifecycle Events ---
+      /**
+       * Create a BulkProcessingStart event (logged when transcript bulk replay begins).
+       */
+      bulkProcessingStart(context, metadata) {
+        return {
+          type: "bulk-processing:start",
+          time: Date.now(),
+          source: "transcript",
+          context: {
+            sessionId: context.sessionId,
+            correlationId: context.correlationId,
+            traceId: context.traceId,
+            hook: context.hook,
+            taskId: context.taskId
+          },
+          payload: {
+            fileSize: metadata.fileSize
+          }
+        };
+      },
+      /**
+       * Create a BulkProcessingFinish event (logged when transcript bulk replay completes).
+       */
+      bulkProcessingFinish(context, metadata) {
+        return {
+          type: "bulk-processing:finish",
+          time: Date.now(),
+          source: "transcript",
+          context: {
+            sessionId: context.sessionId,
+            correlationId: context.correlationId,
+            traceId: context.traceId,
+            hook: context.hook,
+            taskId: context.taskId
+          },
+          payload: {
+            totalLinesProcessed: metadata.totalLinesProcessed,
+            durationMs: metadata.durationMs
+          }
+        };
       }
     };
     function logEvent(logger, event) {
@@ -45317,7 +45384,7 @@ var require_package3 = __commonJS({
         "eslint-config-prettier": "^9.1.0",
         "eslint-plugin-prettier": "^5.1.3",
         prettier: "^3.2.5",
-        typescript: "^5.6.3",
+        typescript: "^5.9.3",
         "typescript-eslint": "^8.56.0",
         vitest: "^4.0.18"
       }
@@ -57281,6 +57348,7 @@ var require_staging_service = __commonJS({
         const hookDir = this.getHookDirPath(sessionId, hookName);
         const reminderPath = this.getReminderFilePath(sessionId, hookName, reminderName);
         await this.ensureDir(hookDir);
+        const isRestage = (0, node_fs_1.existsSync)(reminderPath);
         try {
           await this.options.stateService.write(reminderPath, data, types_1.StagedReminderSchema);
         } catch (err) {
@@ -57296,18 +57364,20 @@ var require_staging_service = __commonJS({
             throw err;
           }
         }
-        const event = structured_logging_1.LogEvents.reminderStaged({
-          sessionId,
-          hook: hookName
-        }, {
-          reminderName: data.name,
-          hookName,
-          blocking: data.blocking,
-          priority: data.priority,
-          persistent: data.persistent,
-          ...enrichment
-        }, { stagingPath: reminderPath });
-        (0, structured_logging_1.logEvent)(this.options.logger.child({ context: { sessionId } }), event);
+        if (!isRestage) {
+          const event = structured_logging_1.LogEvents.reminderStaged({
+            sessionId,
+            hook: hookName
+          }, {
+            reminderName: data.name,
+            hookName,
+            blocking: data.blocking,
+            priority: data.priority,
+            persistent: data.persistent,
+            ...enrichment
+          }, { stagingPath: reminderPath });
+          (0, structured_logging_1.logEvent)(this.options.logger.child({ context: { sessionId } }), event);
+        }
       }
       /**
        * Read a staged reminder.
@@ -57386,13 +57456,14 @@ var require_staging_service = __commonJS({
        * Delete a specific staged reminder.
        * Used after consumption of one-shot reminders.
        *
+       * @returns true if the file was actually deleted, false if it didn't exist
        * @throws Error if hookName or reminderName contain path traversal characters
        */
       async deleteReminder(sessionId, hookName, reminderName) {
         (0, staging_paths_js_1.validatePathSegment)(hookName, "hookName");
         (0, staging_paths_js_1.validatePathSegment)(reminderName, "reminderName");
         const reminderPath = this.getReminderFilePath(sessionId, hookName, reminderName);
-        await this.options.stateService.delete(reminderPath);
+        return this.options.stateService.delete(reminderPath);
       }
       /**
        * List consumed reminder files for a specific reminder ID.
@@ -59933,7 +60004,7 @@ var require_state_service = __commonJS({
       }
       /**
        * Delete state file if it exists.
-       * No-op if file doesn't exist.
+       * @returns true if the file was actually deleted, false if it didn't exist
        */
       async delete(path) {
         try {
@@ -59942,10 +60013,12 @@ var require_state_service = __commonJS({
             this.cache.delete(path);
           }
           this.logger?.debug("State deleted", { path });
+          return true;
         } catch (err) {
           if (!isEnoent(err)) {
             throw err;
           }
+          return false;
         }
       }
       /**
@@ -60200,7 +60273,7 @@ var require_typed_accessor = __commonJS({
        */
       async delete(sessionId) {
         const path = this.stateService.sessionStatePath(sessionId, this.descriptor.filename);
-        return this.stateService.delete(path);
+        await this.stateService.delete(path);
       }
       /**
        * Get the path for a session state file.
@@ -60246,7 +60319,7 @@ var require_typed_accessor = __commonJS({
        */
       async delete() {
         const path = this.stateService.globalStatePath(this.descriptor.filename);
-        return this.stateService.delete(path);
+        await this.stateService.delete(path);
       }
       /**
        * Get the path for the global state file.
@@ -60568,6 +60641,10 @@ var require_transcript_service = __commonJS({
       prepared = false;
       /** Track whether we're in bulk processing mode (first-time transcript replay) */
       isBulkProcessing = false;
+      /** One-shot guard — prevents BulkProcessingComplete from firing more than once per instance */
+      hasFiredBulkComplete = false;
+      /** Timestamp when bulk processing started (for duration calculation in finish event) */
+      bulkStartTime = 0;
       /** Map tool_use_id → tool name so ToolResult events can include the tool name */
       toolUseIdToName = /* @__PURE__ */ new Map();
       // Streaming state for incremental file processing
@@ -61307,12 +61384,10 @@ var require_transcript_service = __commonJS({
         }
         const startLine = this.metrics.lastProcessedLine;
         const isBulkStart = startLine === 0 && this.lastProcessedByteOffset === 0;
-        if (isBulkStart) {
+        if (isBulkStart && !this.hasFiredBulkComplete) {
           this.isBulkProcessing = true;
-          this.options.logger.info("Bulk processing started (streaming)", {
-            sessionId: this.sessionId,
-            fileSize: currentFileSize
-          });
+          this.bulkStartTime = Date.now();
+          (0, structured_logging_js_1.logEvent)(this.options.logger, structured_logging_js_1.LogEvents.bulkProcessingStart({ sessionId: this.sessionId }, { fileSize: currentFileSize }));
         }
         const stream = (0, node_fs_1.createReadStream)(this.transcriptPath, {
           encoding: "utf-8",
@@ -61370,13 +61445,12 @@ var require_transcript_service = __commonJS({
           totalLines: lineNumber,
           newByteOffset: bytesRead
         });
-        if (isBulkStart && this.isBulkProcessing) {
+        if (isBulkStart && this.isBulkProcessing && !this.hasFiredBulkComplete) {
           this.isBulkProcessing = false;
+          this.hasFiredBulkComplete = true;
+          const durationMs = Date.now() - this.bulkStartTime;
+          (0, structured_logging_js_1.logEvent)(this.options.logger, structured_logging_js_1.LogEvents.bulkProcessingFinish({ sessionId: this.sessionId }, { totalLinesProcessed: lineNumber, durationMs }));
           await this.emitEvent("BulkProcessingComplete", {}, lineNumber);
-          this.options.logger.info("Bulk processing complete", {
-            sessionId: this.sessionId,
-            totalLinesProcessed: lineNumber
-          });
         }
         this.notifyMetricsChange();
         this.schedulePersistence();
@@ -70614,6 +70688,7 @@ var require_stage_default_user_prompt = __commonJS({
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.registerStageDefaultUserPrompt = registerStageDefaultUserPrompt;
+    var core_1 = require_dist4();
     var types_1 = require_dist();
     var staging_handler_utils_js_1 = require_staging_handler_utils();
     var types_js_1 = require_types2();
@@ -70753,6 +70828,12 @@ var require_stage_default_user_prompt = __commonJS({
                 ...typedEntry.cachedReminder,
                 stagedAt
               });
+              (0, core_1.logEvent)(handlerCtx.logger, types_1.DecisionEvents.decisionRecorded({ sessionId }, {
+                decision: "staged",
+                reason: `message count reached threshold (${newCount}/${threshold})`,
+                subsystem: "user-prompt-reminders",
+                title: "Stage user-prompt reminder"
+              }));
               state[reminderId] = { ...typedEntry, messagesSinceLastStaging: 0 };
               handlerCtx.logger.debug("Throttle: re-staged reminder", {
                 sessionId,
@@ -70950,6 +71031,12 @@ var require_stage_pause_and_reflect = __commonJS({
             }));
             return void 0;
           }
+          (0, core_1.logEvent)(ctx.logger, types_1.DecisionEvents.decisionRecorded({ sessionId }, {
+            decision: "staged",
+            reason: `tools since baseline reached threshold (${toolsSinceBaseline}/${config.pause_and_reflect_threshold})`,
+            subsystem: "pause-reflect",
+            title: "Stage pause-and-reflect reminder"
+          }));
           return {
             reminderId: types_js_1.ReminderIds.PAUSE_AND_REFLECT,
             targetHook: "PreToolUse",
@@ -72637,12 +72724,21 @@ var require_track_verification_tools = __commonJS({
           } else {
             const newEdits = current.editsSinceVerified + 1;
             if (newEdits >= toolConfig.clearing_threshold) {
+              const wasAlreadyStaged = stagedNames.has(reminderId);
               const staged = await ensureToolReminderStaged(daemonCtx, reminderId, stagedNames, {
                 reason: "threshold_reached",
                 triggeredBy: "file_edit",
                 thresholdState: { current: newEdits, threshold: toolConfig.clearing_threshold }
               });
               if (staged) {
+                if (!wasAlreadyStaged) {
+                  (0, core_1.logEvent)(daemonCtx.logger, types_1.DecisionEvents.decisionRecorded({ sessionId }, {
+                    decision: "staged",
+                    reason: `edits reached clearing threshold (${newEdits}/${toolConfig.clearing_threshold})`,
+                    subsystem: "vc-reminders",
+                    title: "Re-stage VC reminder (threshold reached)"
+                  }));
+                }
                 toolsState[toolName] = {
                   ...current,
                   status: "staged",
@@ -72710,7 +72806,15 @@ var require_track_verification_tools = __commonJS({
           lastMatchedToolId: match.tool_id,
           lastMatchedScope: match.scope
         };
-        await daemonCtx.staging.deleteReminder("Stop", reminderId);
+        const deleted = await daemonCtx.staging.deleteReminder("Stop", reminderId);
+        if (deleted) {
+          (0, core_1.logEvent)(daemonCtx.logger, types_1.DecisionEvents.decisionRecorded({ sessionId }, {
+            decision: "unstaged",
+            reason: `verification passed for ${toolName} (matched ${match.tool_id})`,
+            subsystem: "vc-reminders",
+            title: "Unstage VC reminder (verified)"
+          }));
+        }
         (0, core_1.logEvent)(daemonCtx.logger, events_js_1.ReminderEvents.reminderUnstaged({ sessionId }, {
           reminderName: reminderId,
           hookName: "Stop",
@@ -72913,8 +73017,9 @@ var require_unstage_verify_completion = __commonJS({
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.registerUnstageVerifyCompletion = registerUnstageVerifyCompletion;
     var core_1 = require_dist4();
-    var events_js_1 = require_events2();
     var types_1 = require_dist();
+    var events_js_1 = require_events2();
+    var types_2 = require_dist();
     var types_js_1 = require_types2();
     var reminder_utils_js_1 = require_reminder_utils();
     var state_js_1 = require_state4();
@@ -72931,7 +73036,7 @@ var require_unstage_verify_completion = __commonJS({
       return false;
     }
     function registerUnstageVerifyCompletion(context) {
-      if (!(0, types_1.isDaemonContext)(context))
+      if (!(0, types_2.isDaemonContext)(context))
         return;
       context.handlers.register({
         id: "reminders:unstage-verify-completion",
@@ -72939,17 +73044,19 @@ var require_unstage_verify_completion = __commonJS({
         // Before consumption handlers (50)
         filter: { kind: "hook", hooks: ["UserPromptSubmit"] },
         handler: async (event, ctx) => {
-          if (!(0, types_1.isHookEvent)(event))
+          if (!(0, types_2.isHookEvent)(event))
             return;
-          if (!(0, types_1.isDaemonContext)(ctx))
+          if (!(0, types_2.isDaemonContext)(ctx))
             return;
           const daemonCtx = ctx;
           const sessionId = event.context?.sessionId;
           if (!sessionId) {
             daemonCtx.logger.warn("No sessionId in UserPromptSubmit event");
             for (const vcId of types_js_1.ALL_VC_REMINDER_IDS) {
-              await daemonCtx.staging.deleteReminder("Stop", vcId);
-              (0, core_1.logEvent)(daemonCtx.logger, events_js_1.ReminderEvents.reminderUnstaged({ sessionId: "" }, { reminderName: vcId, hookName: "Stop", reason: "no_session_id" }));
+              const deleted = await daemonCtx.staging.deleteReminder("Stop", vcId);
+              if (deleted) {
+                (0, core_1.logEvent)(daemonCtx.logger, events_js_1.ReminderEvents.reminderUnstaged({ sessionId: "" }, { reminderName: vcId, hookName: "Stop", reason: "no_session_id" }));
+              }
             }
             return;
           }
@@ -73034,6 +73141,12 @@ var require_unstage_verify_completion = __commonJS({
                 maxCycles
               });
               await remindersState.vcUnverified.delete(sessionId);
+              (0, core_1.logEvent)(daemonCtx.logger, types_1.DecisionEvents.decisionRecorded({ sessionId }, {
+                decision: "unstaged-all",
+                reason: `verification cycle limit reached (${unverifiedState.cycleCount}/${maxCycles})`,
+                subsystem: "vc-reminders",
+                title: "Unstage all VC reminders (cycle limit)"
+              }));
             }
           } else {
             (0, core_1.logEvent)(daemonCtx.logger, events_js_1.ReminderEvents.reminderNotStaged({ sessionId }, {
@@ -73048,16 +73161,24 @@ var require_unstage_verify_completion = __commonJS({
           }
           const eventContext = { sessionId };
           const reason = unverifiedState?.hasUnverifiedChanges ? "cycle_limit_reached" : "no_unverified_changes";
+          let deletedCount = 0;
           for (const vcId of types_js_1.ALL_VC_REMINDER_IDS) {
-            await daemonCtx.staging.deleteReminder("Stop", vcId);
-            (0, core_1.logEvent)(daemonCtx.logger, events_js_1.ReminderEvents.reminderUnstaged(eventContext, {
-              reminderName: vcId,
-              hookName: "Stop",
-              reason,
-              triggeredBy: unverifiedState?.hasUnverifiedChanges ? "cycle_limit" : "no_unverified_changes"
-            }));
+            const deleted = await daemonCtx.staging.deleteReminder("Stop", vcId);
+            if (deleted) {
+              deletedCount++;
+              (0, core_1.logEvent)(daemonCtx.logger, events_js_1.ReminderEvents.reminderUnstaged(eventContext, {
+                reminderName: vcId,
+                hookName: "Stop",
+                reason,
+                triggeredBy: unverifiedState?.hasUnverifiedChanges ? "cycle_limit" : "no_unverified_changes"
+              }));
+            }
           }
-          daemonCtx.logger.debug("VC unstage: deleted all VC reminders");
+          daemonCtx.logger.debug("VC unstage: cleanup complete", {
+            reason,
+            deletedCount,
+            totalChecked: types_js_1.ALL_VC_REMINDER_IDS.length
+          });
         }
       });
     }
@@ -74019,16 +74140,24 @@ var require_orchestrator = __commonJS({
             const staging = this.deps.getStagingService(sessionId);
             const eventContext = { sessionId };
             const sessionLogger = this.deps.logger.child({ context: { sessionId } });
+            let deletedCount = 0;
             for (const vcId of types_js_1.ALL_VC_REMINDER_IDS) {
-              await staging.deleteReminder("Stop", vcId);
-              (0, core_1.logEvent)(sessionLogger, events_js_1.ReminderEvents.reminderUnstaged(eventContext, {
-                reminderName: vcId,
-                hookName: "Stop",
-                reason: "pause_and_reflect_cascade",
-                triggeredBy: "cascade_from_pause_and_reflect"
-              }));
+              const deleted = await staging.deleteReminder("Stop", vcId);
+              if (deleted) {
+                deletedCount++;
+                (0, core_1.logEvent)(sessionLogger, events_js_1.ReminderEvents.reminderUnstaged(eventContext, {
+                  reminderName: vcId,
+                  hookName: "Stop",
+                  reason: "pause_and_reflect_cascade",
+                  triggeredBy: "cascade_from_pause_and_reflect"
+                }));
+              }
             }
-            this.deps.logger.debug("Unstaged all VC reminders after P&R staged", { sessionId });
+            this.deps.logger.debug("VC unstage: P&R cascade complete", {
+              sessionId,
+              deletedCount,
+              totalChecked: types_js_1.ALL_VC_REMINDER_IDS.length
+            });
           } catch (err) {
             this.deps.logger.warn("Failed to unstage VC reminders after P&R staged", {
               sessionId,
@@ -74061,14 +74190,16 @@ var require_orchestrator = __commonJS({
           }
           try {
             const staging = this.deps.getStagingService(sessionId);
-            await staging.deleteReminder("PreToolUse", types_js_1.ReminderIds.PAUSE_AND_REFLECT);
-            (0, core_1.logEvent)(this.deps.logger.child({ context: { sessionId } }), events_js_1.ReminderEvents.reminderUnstaged({ sessionId }, {
-              reminderName: types_js_1.ReminderIds.PAUSE_AND_REFLECT,
-              hookName: "PreToolUse",
-              reason: "vc_consumed_cascade",
-              triggeredBy: "cascade_from_verify_completion"
-            }));
-            this.deps.logger.debug("Unstaged P&R after VC consumed", { sessionId });
+            const deleted = await staging.deleteReminder("PreToolUse", types_js_1.ReminderIds.PAUSE_AND_REFLECT);
+            if (deleted) {
+              (0, core_1.logEvent)(this.deps.logger.child({ context: { sessionId } }), events_js_1.ReminderEvents.reminderUnstaged({ sessionId }, {
+                reminderName: types_js_1.ReminderIds.PAUSE_AND_REFLECT,
+                hookName: "PreToolUse",
+                reason: "vc_consumed_cascade",
+                triggeredBy: "cascade_from_verify_completion"
+              }));
+            }
+            this.deps.logger.debug("VC unstage: P&R cascade from VC consumed", { sessionId, deleted });
           } catch (err) {
             this.deps.logger.warn("Failed to unstage P&R after VC consumed", {
               sessionId,
@@ -74534,7 +74665,7 @@ var require_events3 = __commonJS({
   "../feature-session-summary/dist/events.js"(exports2) {
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
-    exports2.DecisionEvents = exports2.SessionSummaryEvents = void 0;
+    exports2.SessionSummaryEvents = void 0;
     exports2.SessionSummaryEvents = {
       /** Emitted when summary generation begins. */
       summaryStart(context, payload) {
@@ -74656,24 +74787,6 @@ var require_events3 = __commonJS({
         };
       }
     };
-    exports2.DecisionEvents = {
-      /** Emitted when an LLM decision is recorded. */
-      decisionRecorded(context, payload) {
-        return {
-          type: "decision:recorded",
-          time: Date.now(),
-          source: "daemon",
-          context: {
-            sessionId: context.sessionId,
-            correlationId: context.correlationId,
-            traceId: context.traceId,
-            hook: context.hook,
-            taskId: context.taskId
-          },
-          payload
-        };
-      }
-    };
   }
 });
 
@@ -74833,11 +74946,14 @@ var require_update_summary = __commonJS({
     exports2.updateSessionSummary = updateSessionSummary;
     var core_1 = require_dist4();
     var events_js_1 = require_events3();
+    var types_1 = require_dist();
     var zod_1 = require_zod();
     var types_js_1 = require_types3();
     var state_js_1 = require_state5();
     var persona_utils_js_1 = require_persona_utils();
     var persona_selection_js_1 = require_persona_selection();
+    var DECISION_TITLE_SKIP = "Skip session analysis";
+    var DECISION_TITLE_RUN = "Run session analysis";
     var PROMPT_FILE = "prompts/session-summary.prompt.txt";
     var SNARKY_PROMPT_FILE = "prompts/snarky-message.prompt.txt";
     var RESUME_PROMPT_FILE = "prompts/resume-message.prompt.txt";
@@ -74888,17 +75004,19 @@ var require_update_summary = __commonJS({
         await (0, persona_selection_js_1.ensurePersonaForSession)(sessionId, ctx);
         const metrics = ctx.transcript.getMetrics();
         if (metrics.turnCount === 0) {
-          (0, core_1.logEvent)(ctx.logger, events_js_1.DecisionEvents.decisionRecorded(event.context, {
+          (0, core_1.logEvent)(ctx.logger, types_1.DecisionEvents.decisionRecorded(event.context, {
             decision: "skipped",
             reason: "BulkProcessingComplete with no user turns (turnCount=0)",
-            detail: "session-summary analysis"
+            subsystem: "session-summary",
+            title: DECISION_TITLE_SKIP
           }));
           return;
         }
-        (0, core_1.logEvent)(ctx.logger, events_js_1.DecisionEvents.decisionRecorded(event.context, {
+        (0, core_1.logEvent)(ctx.logger, types_1.DecisionEvents.decisionRecorded(event.context, {
           decision: "calling",
           reason: "BulkProcessingComplete - analyzing full transcript",
-          detail: "session-summary analysis"
+          subsystem: "session-summary",
+          title: DECISION_TITLE_RUN
         }));
         const countdown2 = await loadCountdownState(summaryState, sessionId);
         void performAnalysis(event, ctx, summaryState, countdown2, "user_prompt_forced");
@@ -74906,28 +75024,19 @@ var require_update_summary = __commonJS({
       }
       const countdown = await loadCountdownState(summaryState, sessionId);
       if (isUserPrompt) {
-        (0, core_1.logEvent)(ctx.logger, events_js_1.DecisionEvents.decisionRecorded(event.context, {
-          decision: "calling",
-          reason: "UserPrompt event forces immediate analysis",
-          detail: "session-summary analysis"
-        }));
         void performAnalysis(event, ctx, summaryState, countdown, "user_prompt_forced");
         return;
       }
       if (countdown.countdown > 0) {
-        (0, core_1.logEvent)(ctx.logger, events_js_1.DecisionEvents.decisionRecorded(event.context, {
-          decision: "skipped",
-          reason: `countdown not reached (${countdown.countdown} tool results remaining)`,
-          detail: "session-summary analysis"
-        }));
         countdown.countdown--;
         await saveCountdownState(summaryState, sessionId, countdown);
         return;
       }
-      (0, core_1.logEvent)(ctx.logger, events_js_1.DecisionEvents.decisionRecorded(event.context, {
+      (0, core_1.logEvent)(ctx.logger, types_1.DecisionEvents.decisionRecorded(event.context, {
         decision: "calling",
         reason: "countdown reached zero after ToolResult",
-        detail: "session-summary analysis"
+        subsystem: "session-summary",
+        title: DECISION_TITLE_RUN
       }));
       void performAnalysis(event, ctx, summaryState, countdown, "countdown_reached");
     }
@@ -75069,14 +75178,6 @@ var require_update_summary = __commonJS({
         if (sideEffects.length > 0) {
           await Promise.all(sideEffects);
         }
-        (0, core_1.logEvent)(ctx.logger, events_js_1.SessionSummaryEvents.summaryFinish(event.context, {
-          session_title: updatedSummary.session_title,
-          session_title_confidence: updatedSummary.session_title_confidence,
-          latest_intent: updatedSummary.latest_intent,
-          latest_intent_confidence: updatedSummary.latest_intent_confidence,
-          processing_time_ms: updatedSummary.stats?.processing_time_ms ?? 0,
-          pivot_detected: updatedSummary.pivot_detected ?? false
-        }));
         if (currentSummary && updatedSummary.session_title !== currentSummary.session_title) {
           (0, core_1.logEvent)(ctx.logger, events_js_1.SessionSummaryEvents.titleChanged(event.context, {
             previousValue: currentSummary.session_title,
@@ -75091,6 +75192,14 @@ var require_update_summary = __commonJS({
             confidence: updatedSummary.latest_intent_confidence
           }));
         }
+        (0, core_1.logEvent)(ctx.logger, events_js_1.SessionSummaryEvents.summaryFinish(event.context, {
+          session_title: updatedSummary.session_title,
+          session_title_confidence: updatedSummary.session_title_confidence,
+          latest_intent: updatedSummary.latest_intent,
+          latest_intent_confidence: updatedSummary.latest_intent_confidence,
+          processing_time_ms: updatedSummary.stats?.processing_time_ms ?? 0,
+          pivot_detected: updatedSummary.pivot_detected ?? false
+        }));
         ctx.logger.info("Updated session summary", {
           sessionId,
           reason,
@@ -75574,7 +75683,7 @@ var require_dist6 = __commonJS({
       for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports3, p)) __createBinding(exports3, m, p);
     };
     Object.defineProperty(exports2, "__esModule", { value: true });
-    exports2.DecisionEvents = exports2.SessionSummaryEvents = exports2.manifest = void 0;
+    exports2.SessionSummaryEvents = exports2.manifest = void 0;
     exports2.register = register;
     var index_js_1 = require_handlers();
     exports2.manifest = {
@@ -75596,9 +75705,6 @@ var require_dist6 = __commonJS({
     Object.defineProperty(exports2, "SessionSummaryEvents", { enumerable: true, get: function() {
       return events_js_1.SessionSummaryEvents;
     } });
-    Object.defineProperty(exports2, "DecisionEvents", { enumerable: true, get: function() {
-      return events_js_1.DecisionEvents;
-    } });
   }
 });
 
@@ -77950,8 +78056,7 @@ var require_daemon = __commonJS({
           clearStaging: async () => {
           },
           listReminders: () => Promise.resolve([]),
-          deleteReminder: async () => {
-          },
+          deleteReminder: () => Promise.resolve(false),
           listConsumedReminders: () => Promise.resolve([]),
           getLastConsumed: () => Promise.resolve(null)
         };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scotthamilton77/sidekick",
-  "version": "0.1.13",
+  "version": "0.1.15",
   "description": "AI pair programming assistant with personas, session tracking, and contextual nudges",
   "bin": {
     "sidekick": "dist/bin.js"