@scotthamilton77/sidekick 0.1.29 → 0.1.31

package/assets/sidekick/defaults/features/session-summary.defaults.yaml

@@ -124,3 +124,8 @@ settings:
     # When non-off, the persona-voice reminder is replaced by a compact caveman
     # directive and the statusline shows the level.
     cavemanMode: "off"
+    # Inject persona/caveman into non-interactive (claude -p) sessions. Default
+    # false — no human reads a programmatic session, so the voice is wasted tokens.
+    # Set true to opt back in.
+    injectPersonaIntoNonInteractiveSessions: false
+    injectCavemanIntoNonInteractiveSessions: false

package/dist/bin.js

@@ -47579,7 +47579,7 @@ var require_build_identity = __commonJS({
     var path_1 = __importDefault2(require("path"));
     var DEV_PREFIX = "dev:";
     var NO_DIST_SENTINEL = "nodist";
-    function computeBuildIdentity(injectedVersion = true ? "0.1.29" : void 0, devFingerprint = computeDevFingerprint) {
+    function computeBuildIdentity(injectedVersion = true ? "0.1.31" : void 0, devFingerprint = computeDevFingerprint) {
       return injectedVersion !== void 0 ? injectedVersion : DEV_PREFIX + devFingerprint();
     }
     function computeDevFingerprint() {
@@ -61989,7 +61989,7 @@ var require_transcript_metrics_engine = __commonJS({
     exports2.isExcludedBuiltinCommandInvocation = isExcludedBuiltinCommandInvocation;
     exports2.updateToolsPerTurn = updateToolsPerTurn;
     var transcript_helpers_js_1 = require_transcript_helpers();
-    async function processEntry(entry, lineNumber, metrics, toolUseIdToName, emitEvent) {
+    async function processEntry(entry, lineNumber, metrics, toolUseIdToName, emitEvent, signalCompaction) {
       const entryType = entry.type;
       switch (entryType) {
         case "user": {
@@ -62018,8 +62018,13 @@ var require_transcript_metrics_engine = __commonJS({
           await emitEvent("AssistantMessage", entry, lineNumber);
           await processNestedToolUses(entry, lineNumber, metrics, toolUseIdToName, emitEvent);
           break;
-        case "system":
+        case "system": {
+          const subtype = entry.subtype;
+          if (subtype === "compact_boundary") {
+            await signalCompaction?.(lineNumber);
+          }
           break;
+        }
       }
     }
     function extractTokenUsage(entry, metrics) {
@@ -64477,10 +64482,12 @@ var require_transcript_service = __commonJS({
       // ============================================================================
       // Compaction Management
       // ============================================================================
-      async signalCompaction() {
+      async signalCompaction(lineNumber) {
+        if (this.metrics.isPostCompactIndeterminate)
+          return;
         this.metrics.currentContextTokens = null;
         this.metrics.isPostCompactIndeterminate = true;
-        await this.emitEvent("Compact", {}, this.metrics.lastProcessedLine);
+        await this.emitEvent("Compact", {}, lineNumber ?? this.metrics.lastProcessedLine);
         this.notifyMetricsChange();
         this.schedulePersistence();
       }
@@ -64538,7 +64545,7 @@ var require_transcript_service = __commonJS({
           return;
         const self = this;
         await (0, transcript_file_watcher_js_1.processTranscriptFile)(this.transcriptPath, this.streamingState, this.metrics, async (entry, lineNumber) => {
-          await (0, transcript_metrics_engine_js_1.processEntry)(entry, lineNumber, this.metrics, this.toolUseIdToName, (eventType, e, ln) => this.emitEvent(eventType, e, ln));
+          await (0, transcript_metrics_engine_js_1.processEntry)(entry, lineNumber, this.metrics, this.toolUseIdToName, (eventType, e, ln) => this.emitEvent(eventType, e, ln), (ln) => this.signalCompaction(ln));
         }, async (lineNumber, _durationMs) => {
           await this.emitEvent("BulkProcessingComplete", {}, lineNumber);
         }, this.options.logger, this.sessionId, {
@@ -76837,7 +76844,9 @@ var require_types3 = __commonJS({
         llmProfiles: {},
         weights: {},
         persistThroughClear: true,
-        cavemanMode: "off"
+        cavemanMode: "off",
+        injectPersonaIntoNonInteractiveSessions: false,
+        injectCavemanIntoNonInteractiveSessions: false
       }
     };
     exports2.RESUME_MIN_CONFIDENCE = 0.7;
@@ -77776,6 +77785,10 @@ var require_update_summary = __commonJS({
         ctx.logger.debug("Session summary disabled; skipping analysis", { sessionId, reason });
         return;
       }
+      if (ctx.sessionType === "non-interactive") {
+        ctx.logger.debug("Non-interactive session; skipping summary analysis", { sessionId, reason });
+        return;
+      }
       try {
         const countdown = await loadCountdownState(summaryState, sessionId);
         const startTime = Date.now();
@@ -78155,7 +78168,6 @@ var require_stage_persona_reminders = __commonJS({
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.restagePersonaRemindersForActiveSessions = restagePersonaRemindersForActiveSessions;
     exports2.stagePersonaRemindersForSession = stagePersonaRemindersForSession;
-    exports2.resolvePersonaContextForSnapshot = resolvePersonaContextForSnapshot;
     exports2.registerStagePersonaReminders = registerStagePersonaReminders;
     var core_1 = require_dist4();
     var feature_session_summary_1 = require_dist5();
@@ -78375,20 +78387,6 @@ var require_stage_persona_reminders = __commonJS({
         includeChanged: options?.includeChangedReminder ?? false
       });
     }
-    async function resolvePersonaContextForSnapshot(ctx, sessionId) {
-      if (!isPersonaInjectionEnabled(ctx))
-        return void 0;
-      const persona = await loadPersonaForSession(ctx, sessionId);
-      if (!persona)
-        return void 0;
-      const templateContext = buildPersonaTemplateContext(persona);
-      const reminder = (0, reminder_utils_js_1.resolveReminder)(types_js_1.ReminderIds.REMEMBER_YOUR_PERSONA, {
-        context: templateContext,
-        assets: ctx.assets,
-        logger: ctx.logger
-      });
-      return reminder?.additionalContext ?? void 0;
-    }
     function registerStagePersonaReminders(context) {
       if (!(0, types_1.isDaemonContext)(context))
         return;
@@ -78417,7 +78415,6 @@ var require_stage_user_profile_reminders = __commonJS({
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.stageUserProfileRemindersForSession = stageUserProfileRemindersForSession;
-    exports2.resolveUserProfileContextForSnapshot = resolveUserProfileContextForSnapshot;
     exports2.registerStageUserProfileReminders = registerStageUserProfileReminders;
     var core_1 = require_dist4();
     var events_js_1 = require_events2();
@@ -78457,21 +78454,6 @@ var require_stage_user_profile_reminders = __commonJS({
         ctx.logger.warn("Failed to resolve user-profile reminder", { sessionId });
       }
     }
-    function resolveUserProfileContextForSnapshot(ctx) {
-      const profile = (0, core_1.loadUserProfile)({ logger: ctx.logger });
-      if (!profile)
-        return void 0;
-      const templateContext = {
-        user_name: profile.name,
-        user_role: profile.role,
-        user_interests: profile.interests.join(", ")
-      };
-      const reminder = (0, reminder_utils_js_1.resolveReminder)(types_js_1.ReminderIds.USER_PROFILE, {
-        context: templateContext,
-        assets: ctx.assets
-      });
-      return reminder?.additionalContext ?? void 0;
-    }
     function registerStageUserProfileReminders(context) {
       if (!(0, types_1.isDaemonContext)(context))
         return;
@@ -78763,6 +78745,28 @@ var require_cli_staging_reader = __commonJS({
   }
 });
 
+// ../feature-reminders/dist/handlers/consumption/session-type-gate.js
+var require_session_type_gate = __commonJS({
+  "../feature-reminders/dist/handlers/consumption/session-type-gate.js"(exports2) {
+    "use strict";
+    Object.defineProperty(exports2, "__esModule", { value: true });
+    exports2.suppressReminderForSessionType = suppressReminderForSessionType;
+    var types_js_1 = require_types2();
+    var PERSONA_REMINDERS = /* @__PURE__ */ new Set([types_js_1.ReminderIds.REMEMBER_YOUR_PERSONA, types_js_1.ReminderIds.PERSONA_CHANGED]);
+    function suppressReminderForSessionType(reminderName, sessionType, config) {
+      if (sessionType !== "non-interactive")
+        return false;
+      if (PERSONA_REMINDERS.has(reminderName)) {
+        return config.injectPersonaIntoNonInteractiveSessions !== true;
+      }
+      if (reminderName === types_js_1.ReminderIds.CAVEMAN_MODE) {
+        return config.injectCavemanIntoNonInteractiveSessions !== true;
+      }
+      return false;
+    }
+  }
+});
+
 // ../feature-reminders/dist/handlers/consumption/consumption-handler-factory.js
 var require_consumption_handler_factory = __commonJS({
   "../feature-reminders/dist/handlers/consumption/consumption-handler-factory.js"(exports2) {
@@ -78774,6 +78778,7 @@ var require_consumption_handler_factory = __commonJS({
     var events_js_1 = require_events2();
     var types_1 = require_dist();
     var cli_staging_reader_js_1 = require_cli_staging_reader();
+    var session_type_gate_js_1 = require_session_type_gate();
     function buildDefaultResponse(reminder, supportsBlocking) {
       const response = {};
       if (supportsBlocking && reminder.blocking) {
@@ -78807,7 +78812,8 @@ var require_consumption_handler_factory = __commonJS({
             paths: cliCtx.paths,
             sessionId
           });
-          const reminders = reader.listReminders(hook);
+          const personaConfig = cliCtx.config.getFeature("session-summary").settings?.personas ?? {};
+          const reminders = reader.listReminders(hook).filter((r) => !(0, session_type_gate_js_1.suppressReminderForSessionType)(r.name, event.context.sessionType, personaConfig));
           if (reminders.length === 0) {
             return { response: {} };
           }
@@ -79073,22 +79079,6 @@ var require_inject_session_start = __commonJS({
   }
 });
 
-// ../feature-reminders/dist/handlers/consumption/inject-post-compact.js
-var require_inject_post_compact = __commonJS({
-  "../feature-reminders/dist/handlers/consumption/inject-post-compact.js"(exports2) {
-    "use strict";
-    Object.defineProperty(exports2, "__esModule", { value: true });
-    exports2.registerInjectPostCompact = registerInjectPostCompact;
-    var consumption_handler_factory_js_1 = require_consumption_handler_factory();
-    function registerInjectPostCompact(context) {
-      (0, consumption_handler_factory_js_1.createConsumptionHandler)(context, {
-        id: "reminders:inject-post-compact",
-        hook: "PostCompact"
-      });
-    }
-  }
-});
-
 // ../feature-reminders/dist/handlers/consumption/index.js
 var require_consumption = __commonJS({
   "../feature-reminders/dist/handlers/consumption/index.js"(exports2) {
@@ -79100,14 +79090,12 @@ var require_consumption = __commonJS({
     var inject_post_tool_use_1 = require_inject_post_tool_use();
     var inject_stop_1 = require_inject_stop();
     var inject_session_start_1 = require_inject_session_start();
-    var inject_post_compact_js_1 = require_inject_post_compact();
     function registerConsumptionHandlers(context) {
       (0, inject_user_prompt_submit_1.registerInjectUserPromptSubmit)(context);
       (0, inject_pre_tool_use_1.registerInjectPreToolUse)(context);
       (0, inject_post_tool_use_1.registerInjectPostToolUse)(context);
       (0, inject_stop_1.registerInjectStop)(context);
       (0, inject_session_start_1.registerInjectSessionStart)(context);
-      (0, inject_post_compact_js_1.registerInjectPostCompact)(context);
     }
   }
 });
@@ -79552,7 +79540,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.createRemindersState = exports2.ReminderOrchestrator = exports2.ReminderEvents = exports2.handleVCUnverifiedClear = exports2.handleVCUnverifiedSet = exports2.handleReminderConsumed = exports2.classifyCompletion = exports2.resolveUserProfileContextForSnapshot = exports2.resolvePersonaContextForSnapshot = exports2.restagePersonaRemindersForActiveSessions = exports2.stagePersonaRemindersForSession = exports2.registerStagingHandlers = exports2.registerConsumptionHandlers = exports2.manifest = void 0;
+    exports2.createRemindersState = exports2.ReminderOrchestrator = exports2.ReminderEvents = exports2.handleVCUnverifiedClear = exports2.handleVCUnverifiedSet = exports2.handleReminderConsumed = exports2.classifyCompletion = exports2.restagePersonaRemindersForActiveSessions = exports2.stagePersonaRemindersForSession = exports2.registerStagingHandlers = exports2.registerConsumptionHandlers = exports2.manifest = void 0;
     exports2.register = register;
     var staging_1 = require_staging2();
     var consumption_1 = require_consumption();
@@ -79586,13 +79574,6 @@ var require_dist6 = __commonJS({
     Object.defineProperty(exports2, "restagePersonaRemindersForActiveSessions", { enumerable: true, get: function() {
       return stage_persona_reminders_1.restagePersonaRemindersForActiveSessions;
     } });
-    Object.defineProperty(exports2, "resolvePersonaContextForSnapshot", { enumerable: true, get: function() {
-      return stage_persona_reminders_1.resolvePersonaContextForSnapshot;
-    } });
-    var stage_user_profile_reminders_1 = require_stage_user_profile_reminders();
-    Object.defineProperty(exports2, "resolveUserProfileContextForSnapshot", { enumerable: true, get: function() {
-      return stage_user_profile_reminders_1.resolveUserProfileContextForSnapshot;
-    } });
     var completion_classifier_1 = require_completion_classifier();
     Object.defineProperty(exports2, "classifyCompletion", { enumerable: true, get: function() {
       return completion_classifier_1.classifyCompletion;
@@ -79879,7 +79860,8 @@ var require_hook = __commonJS({
         timestamp: Date.now(),
         correlationId,
         ...input.agentId !== void 0 && { agentId: input.agentId },
-        ...input.agentType !== void 0 && { agentType: input.agentType }
+        ...input.agentType !== void 0 && { agentType: input.agentType },
+        sessionType: input.sessionType
       };
       switch (hookName) {
         case "SessionStart":
@@ -80143,12 +80125,6 @@ var require_hook_command = __commonJS({
     }
     function translatePostCompact(internal) {
       const response = {};
-      if (internal.additionalContext) {
-        response.hookSpecificOutput = {
-          hookEventName: "PostCompact",
-          additionalContext: internal.additionalContext
-        };
-      }
       addUserMessage(response, internal.userMessage);
       return response;
     }
@@ -87523,6 +87499,7 @@ var require_cli = __commonJS({
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.UnknownOptionError = void 0;
     exports2.parseArgs = parseArgs;
+    exports2.resolveSessionType = resolveSessionType;
     exports2.parseHookInput = parseHookInput;
     exports2.initializeRuntime = initializeRuntime;
     exports2.initializeSession = initializeSession;
@@ -87531,7 +87508,7 @@ var require_cli = __commonJS({
     var promises_12 = require("node:fs/promises");
     var node_stream_1 = require("node:stream");
     var yargs_parser_1 = __importDefault2(require_build());
-    var VERSION = true ? "0.1.29" : "dev";
+    var VERSION = true ? "0.1.31" : "dev";
     var SANDBOX_ERROR_MESSAGE = `Error: Daemon commands cannot run in sandbox mode.
 
 Claude Code's sandbox blocks Unix socket operations required for daemon IPC.
@@ -87661,7 +87638,14 @@ Example: { "command": "pnpm sidekick daemon status", "dangerouslyDisableSandbox"
         writeStatus: Boolean(parsed["write-status"])
       };
     }
-    function parseHookInput(stdinData) {
+    function resolveSessionType(agentId, entrypoint) {
+      if (agentId)
+        return "agent";
+      if (entrypoint === "sdk-cli")
+        return "non-interactive";
+      return "interactive";
+    }
+    function parseHookInput(stdinData, entrypoint) {
       if (!stdinData?.trim()) {
         return void 0;
       }
@@ -87686,6 +87670,7 @@ Example: { "command": "pnpm sidekick daemon status", "dangerouslyDisableSandbox"
           permissionMode,
           agentId,
           agentType,
+          sessionType: resolveSessionType(agentId, entrypoint),
           raw
         };
       } catch {
@@ -87696,7 +87681,7 @@ Example: { "command": "pnpm sidekick daemon status", "dangerouslyDisableSandbox"
       const stderr = options.stderr ?? new node_stream_1.PassThrough();
       const parsed = parseArgs(options.argv);
       const homeDir = options.homeDir ?? options.env?.HOME;
-      const hookInput = parseHookInput(options.stdinData);
+      const hookInput = parseHookInput(options.stdinData, options.env?.CLAUDE_CODE_ENTRYPOINT);
       const runtime = (0, runtime_1.bootstrapRuntime)({
         projectDir: parsed.projectDir,
         logLevel: parsed.logLevel,

package/dist/daemon.js

@@ -46603,7 +46603,7 @@ var require_build_identity = __commonJS({
     var path_1 = __importDefault(require("path"));
     var DEV_PREFIX = "dev:";
     var NO_DIST_SENTINEL = "nodist";
-    function computeBuildIdentity(injectedVersion = true ? "0.1.29" : void 0, devFingerprint = computeDevFingerprint) {
+    function computeBuildIdentity(injectedVersion = true ? "0.1.31" : void 0, devFingerprint = computeDevFingerprint) {
       return injectedVersion !== void 0 ? injectedVersion : DEV_PREFIX + devFingerprint();
     }
     function computeDevFingerprint() {
@@ -61013,7 +61013,7 @@ var require_transcript_metrics_engine = __commonJS({
     exports2.isExcludedBuiltinCommandInvocation = isExcludedBuiltinCommandInvocation;
     exports2.updateToolsPerTurn = updateToolsPerTurn;
     var transcript_helpers_js_1 = require_transcript_helpers();
-    async function processEntry(entry, lineNumber, metrics, toolUseIdToName, emitEvent) {
+    async function processEntry(entry, lineNumber, metrics, toolUseIdToName, emitEvent, signalCompaction) {
       const entryType = entry.type;
       switch (entryType) {
         case "user": {
@@ -61042,8 +61042,13 @@ var require_transcript_metrics_engine = __commonJS({
           await emitEvent("AssistantMessage", entry, lineNumber);
           await processNestedToolUses(entry, lineNumber, metrics, toolUseIdToName, emitEvent);
           break;
-        case "system":
+        case "system": {
+          const subtype = entry.subtype;
+          if (subtype === "compact_boundary") {
+            await signalCompaction?.(lineNumber);
+          }
           break;
+        }
       }
     }
     function extractTokenUsage(entry, metrics) {
@@ -63501,10 +63506,12 @@ var require_transcript_service = __commonJS({
       // ============================================================================
       // Compaction Management
       // ============================================================================
-      async signalCompaction() {
+      async signalCompaction(lineNumber) {
+        if (this.metrics.isPostCompactIndeterminate)
+          return;
         this.metrics.currentContextTokens = null;
         this.metrics.isPostCompactIndeterminate = true;
-        await this.emitEvent("Compact", {}, this.metrics.lastProcessedLine);
+        await this.emitEvent("Compact", {}, lineNumber ?? this.metrics.lastProcessedLine);
         this.notifyMetricsChange();
         this.schedulePersistence();
       }
@@ -63562,7 +63569,7 @@ var require_transcript_service = __commonJS({
           return;
         const self = this;
         await (0, transcript_file_watcher_js_1.processTranscriptFile)(this.transcriptPath, this.streamingState, this.metrics, async (entry, lineNumber) => {
-          await (0, transcript_metrics_engine_js_1.processEntry)(entry, lineNumber, this.metrics, this.toolUseIdToName, (eventType, e, ln) => this.emitEvent(eventType, e, ln));
+          await (0, transcript_metrics_engine_js_1.processEntry)(entry, lineNumber, this.metrics, this.toolUseIdToName, (eventType, e, ln) => this.emitEvent(eventType, e, ln), (ln) => this.signalCompaction(ln));
         }, async (lineNumber, _durationMs) => {
           await this.emitEvent("BulkProcessingComplete", {}, lineNumber);
         }, this.options.logger, this.sessionId, {
@@ -75601,7 +75608,9 @@ var require_types3 = __commonJS({
         llmProfiles: {},
         weights: {},
         persistThroughClear: true,
-        cavemanMode: "off"
+        cavemanMode: "off",
+        injectPersonaIntoNonInteractiveSessions: false,
+        injectCavemanIntoNonInteractiveSessions: false
       }
     };
     exports2.RESUME_MIN_CONFIDENCE = 0.7;
@@ -76540,6 +76549,10 @@ var require_update_summary = __commonJS({
         ctx.logger.debug("Session summary disabled; skipping analysis", { sessionId, reason });
         return;
       }
+      if (ctx.sessionType === "non-interactive") {
+        ctx.logger.debug("Non-interactive session; skipping summary analysis", { sessionId, reason });
+        return;
+      }
       try {
         const countdown = await loadCountdownState(summaryState, sessionId);
         const startTime = Date.now();
@@ -76919,7 +76932,6 @@ var require_stage_persona_reminders = __commonJS({
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.restagePersonaRemindersForActiveSessions = restagePersonaRemindersForActiveSessions;
     exports2.stagePersonaRemindersForSession = stagePersonaRemindersForSession;
-    exports2.resolvePersonaContextForSnapshot = resolvePersonaContextForSnapshot;
     exports2.registerStagePersonaReminders = registerStagePersonaReminders;
     var core_1 = require_dist4();
     var feature_session_summary_1 = require_dist5();
@@ -77139,20 +77151,6 @@ var require_stage_persona_reminders = __commonJS({
         includeChanged: options?.includeChangedReminder ?? false
       });
     }
-    async function resolvePersonaContextForSnapshot(ctx, sessionId) {
-      if (!isPersonaInjectionEnabled(ctx))
-        return void 0;
-      const persona = await loadPersonaForSession(ctx, sessionId);
-      if (!persona)
-        return void 0;
-      const templateContext = buildPersonaTemplateContext(persona);
-      const reminder = (0, reminder_utils_js_1.resolveReminder)(types_js_1.ReminderIds.REMEMBER_YOUR_PERSONA, {
-        context: templateContext,
-        assets: ctx.assets,
-        logger: ctx.logger
-      });
-      return reminder?.additionalContext ?? void 0;
-    }
     function registerStagePersonaReminders(context) {
       if (!(0, types_1.isDaemonContext)(context))
         return;
@@ -77181,7 +77179,6 @@ var require_stage_user_profile_reminders = __commonJS({
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.stageUserProfileRemindersForSession = stageUserProfileRemindersForSession;
-    exports2.resolveUserProfileContextForSnapshot = resolveUserProfileContextForSnapshot;
     exports2.registerStageUserProfileReminders = registerStageUserProfileReminders;
     var core_1 = require_dist4();
     var events_js_1 = require_events2();
@@ -77221,21 +77218,6 @@ var require_stage_user_profile_reminders = __commonJS({
         ctx.logger.warn("Failed to resolve user-profile reminder", { sessionId });
       }
     }
-    function resolveUserProfileContextForSnapshot(ctx) {
-      const profile = (0, core_1.loadUserProfile)({ logger: ctx.logger });
-      if (!profile)
-        return void 0;
-      const templateContext = {
-        user_name: profile.name,
-        user_role: profile.role,
-        user_interests: profile.interests.join(", ")
-      };
-      const reminder = (0, reminder_utils_js_1.resolveReminder)(types_js_1.ReminderIds.USER_PROFILE, {
-        context: templateContext,
-        assets: ctx.assets
-      });
-      return reminder?.additionalContext ?? void 0;
-    }
     function registerStageUserProfileReminders(context) {
       if (!(0, types_1.isDaemonContext)(context))
         return;
@@ -77527,6 +77509,28 @@ var require_cli_staging_reader = __commonJS({
   }
 });
 
+// ../feature-reminders/dist/handlers/consumption/session-type-gate.js
+var require_session_type_gate = __commonJS({
+  "../feature-reminders/dist/handlers/consumption/session-type-gate.js"(exports2) {
+    "use strict";
+    Object.defineProperty(exports2, "__esModule", { value: true });
+    exports2.suppressReminderForSessionType = suppressReminderForSessionType;
+    var types_js_1 = require_types2();
+    var PERSONA_REMINDERS = /* @__PURE__ */ new Set([types_js_1.ReminderIds.REMEMBER_YOUR_PERSONA, types_js_1.ReminderIds.PERSONA_CHANGED]);
+    function suppressReminderForSessionType(reminderName, sessionType, config) {
+      if (sessionType !== "non-interactive")
+        return false;
+      if (PERSONA_REMINDERS.has(reminderName)) {
+        return config.injectPersonaIntoNonInteractiveSessions !== true;
+      }
+      if (reminderName === types_js_1.ReminderIds.CAVEMAN_MODE) {
+        return config.injectCavemanIntoNonInteractiveSessions !== true;
+      }
+      return false;
+    }
+  }
+});
+
 // ../feature-reminders/dist/handlers/consumption/consumption-handler-factory.js
 var require_consumption_handler_factory = __commonJS({
   "../feature-reminders/dist/handlers/consumption/consumption-handler-factory.js"(exports2) {
@@ -77538,6 +77542,7 @@ var require_consumption_handler_factory = __commonJS({
     var events_js_1 = require_events2();
     var types_1 = require_dist();
     var cli_staging_reader_js_1 = require_cli_staging_reader();
+    var session_type_gate_js_1 = require_session_type_gate();
     function buildDefaultResponse(reminder, supportsBlocking) {
       const response = {};
       if (supportsBlocking && reminder.blocking) {
@@ -77571,7 +77576,8 @@ var require_consumption_handler_factory = __commonJS({
             paths: cliCtx.paths,
             sessionId
           });
-          const reminders = reader.listReminders(hook);
+          const personaConfig = cliCtx.config.getFeature("session-summary").settings?.personas ?? {};
+          const reminders = reader.listReminders(hook).filter((r) => !(0, session_type_gate_js_1.suppressReminderForSessionType)(r.name, event.context.sessionType, personaConfig));
           if (reminders.length === 0) {
             return { response: {} };
           }
@@ -77837,22 +77843,6 @@ var require_inject_session_start = __commonJS({
   }
 });
 
-// ../feature-reminders/dist/handlers/consumption/inject-post-compact.js
-var require_inject_post_compact = __commonJS({
-  "../feature-reminders/dist/handlers/consumption/inject-post-compact.js"(exports2) {
-    "use strict";
-    Object.defineProperty(exports2, "__esModule", { value: true });
-    exports2.registerInjectPostCompact = registerInjectPostCompact;
-    var consumption_handler_factory_js_1 = require_consumption_handler_factory();
-    function registerInjectPostCompact(context) {
-      (0, consumption_handler_factory_js_1.createConsumptionHandler)(context, {
-        id: "reminders:inject-post-compact",
-        hook: "PostCompact"
-      });
-    }
-  }
-});
-
 // ../feature-reminders/dist/handlers/consumption/index.js
 var require_consumption = __commonJS({
   "../feature-reminders/dist/handlers/consumption/index.js"(exports2) {
@@ -77864,14 +77854,12 @@ var require_consumption = __commonJS({
     var inject_post_tool_use_1 = require_inject_post_tool_use();
     var inject_stop_1 = require_inject_stop();
     var inject_session_start_1 = require_inject_session_start();
-    var inject_post_compact_js_1 = require_inject_post_compact();
     function registerConsumptionHandlers(context) {
       (0, inject_user_prompt_submit_1.registerInjectUserPromptSubmit)(context);
       (0, inject_pre_tool_use_1.registerInjectPreToolUse)(context);
       (0, inject_post_tool_use_1.registerInjectPostToolUse)(context);
       (0, inject_stop_1.registerInjectStop)(context);
       (0, inject_session_start_1.registerInjectSessionStart)(context);
-      (0, inject_post_compact_js_1.registerInjectPostCompact)(context);
     }
   }
 });
@@ -78316,7 +78304,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.createRemindersState = exports2.ReminderOrchestrator = exports2.ReminderEvents = exports2.handleVCUnverifiedClear = exports2.handleVCUnverifiedSet = exports2.handleReminderConsumed = exports2.classifyCompletion = exports2.resolveUserProfileContextForSnapshot = exports2.resolvePersonaContextForSnapshot = exports2.restagePersonaRemindersForActiveSessions = exports2.stagePersonaRemindersForSession = exports2.registerStagingHandlers = exports2.registerConsumptionHandlers = exports2.manifest = void 0;
+    exports2.createRemindersState = exports2.ReminderOrchestrator = exports2.ReminderEvents = exports2.handleVCUnverifiedClear = exports2.handleVCUnverifiedSet = exports2.handleReminderConsumed = exports2.classifyCompletion = exports2.restagePersonaRemindersForActiveSessions = exports2.stagePersonaRemindersForSession = exports2.registerStagingHandlers = exports2.registerConsumptionHandlers = exports2.manifest = void 0;
     exports2.register = register;
     var staging_1 = require_staging2();
     var consumption_1 = require_consumption();
@@ -78350,13 +78338,6 @@ var require_dist6 = __commonJS({
     Object.defineProperty(exports2, "restagePersonaRemindersForActiveSessions", { enumerable: true, get: function() {
       return stage_persona_reminders_1.restagePersonaRemindersForActiveSessions;
     } });
-    Object.defineProperty(exports2, "resolvePersonaContextForSnapshot", { enumerable: true, get: function() {
-      return stage_persona_reminders_1.resolvePersonaContextForSnapshot;
-    } });
-    var stage_user_profile_reminders_1 = require_stage_user_profile_reminders();
-    Object.defineProperty(exports2, "resolveUserProfileContextForSnapshot", { enumerable: true, get: function() {
-      return stage_user_profile_reminders_1.resolveUserProfileContextForSnapshot;
-    } });
     var completion_classifier_1 = require_completion_classifier();
     Object.defineProperty(exports2, "classifyCompletion", { enumerable: true, get: function() {
       return completion_classifier_1.classifyCompletion;
@@ -80018,12 +79999,18 @@ var require_daemon_helpers = __commonJS({
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
     exports2.REGISTRY_HEARTBEAT_INTERVAL_MS = exports2.EVICTION_INTERVAL_MS = exports2.HEARTBEAT_INTERVAL_MS = exports2.IDLE_CHECK_INTERVAL_MS = exports2.VERSION = void 0;
+    exports2.nextStoredSessionType = nextStoredSessionType;
     exports2.diffConfigs = diffConfigs;
     exports2.resolveTranscriptPath = resolveTranscriptPath;
     exports2.shouldRunStartupCleanup = shouldRunStartupCleanup;
     exports2.getPersonaInjectionEnabled = getPersonaInjectionEnabled;
     exports2.getCavemanConfigDefault = getCavemanConfigDefault;
     var core_1 = require_dist4();
+    function nextStoredSessionType(current, incoming) {
+      if (incoming && incoming !== "agent")
+        return incoming;
+      return current;
+    }
     exports2.VERSION = (0, core_1.computeBuildIdentity)();
     exports2.IDLE_CHECK_INTERVAL_MS = 30 * 1e3;
     exports2.HEARTBEAT_INTERVAL_MS = 5 * 1e3;
@@ -80661,8 +80648,13 @@ var require_daemon = __commonJS({
       registryService;
       timerManager;
       sessionLogWriter;
-      /** In-memory compaction snapshots keyed by sessionId. Populated by PreCompact, consumed by PostCompact. */
-      compactionSnapshots = /* @__PURE__ */ new Map();
+      /**
+       * Parent session type keyed by sessionId. Only updated by parent hooks
+       * (interactive/non-interactive); subagent 'agent' hooks share the parent
+       * sessionId and are ignored so they cannot flip the stored type. Surfaced onto
+       * DaemonContext.sessionType to gate expensive session-summary work.
+       */
+      sessionTypes = /* @__PURE__ */ new Map();
       /** Cache persona for handoff on clear. */
       cachePersonaForClear(personaId) {
         this.lastClearedPersona = { personaId, timestamp: Date.now() };
@@ -81026,6 +81018,10 @@ var require_daemon = __commonJS({
        * Uses the same infrastructure as the IPC handlers for message generation.
        */
       async regenerateMessagesForSession(sessionId) {
+        if (this.sessionTypes.get(sessionId) === "non-interactive") {
+          this.logger.debug("Non-interactive session; skipping message regeneration", { sessionId });
+          return;
+        }
         this.logger.info("Regenerating messages after persona change", { sessionId });
         try {
           await (0, persona_transition_js_1.stagePersonaTransition)(this.stateService, sessionId);
@@ -81111,6 +81107,11 @@ var require_daemon = __commonJS({
           throw new Error("hook.invoke requires hook and event parameters");
         }
         const { sessionId, correlationId } = event.context ?? {};
+        if (sessionId) {
+          const next = (0, daemon_helpers_js_1.nextStoredSessionType)(this.sessionTypes.get(sessionId), event.context?.sessionType);
+          if (next !== void 0)
+            this.sessionTypes.set(sessionId, next);
+        }
         const requestLogger = this.logger.child({
           context: { sessionId, correlationId }
         });
@@ -81132,36 +81133,29 @@ var require_daemon = __commonJS({
         if (hook === "UserPromptSubmit") {
           await this.handleUserPromptSubmitCleanup(event, { logger: requestLogger });
         }
-        if (hook === "PreCompact") {
-          await this.handlePreCompact(event, { logger: requestLogger });
-        }
-        let postCompactResponse;
         if (hook === "PostCompact") {
-          postCompactResponse = await this.handlePostCompact(event, { logger: requestLogger });
+          await this.handlePostCompact(event, { logger: requestLogger });
         }
         if (sessionId && !this.logMetrics.hasSession(sessionId)) {
           await this.logMetrics.initSessionCounters(sessionId, false);
           requestLogger.debug("Log counters initialized from file for hook", { hook });
         }
-        const response = await this.handlerRegistry.invokeHook(hook, event, { logger: requestLogger });
-        if (postCompactResponse?.additionalContext) {
-          const registryContext = response.additionalContext;
-          response.additionalContext = registryContext ? `${registryContext}
-
-${postCompactResponse.additionalContext}` : postCompactResponse.additionalContext;
-        }
-        return response;
+        return this.handlerRegistry.invokeHook(hook, event, { logger: requestLogger });
       }
       /**
-       * Handle PostCompact: signal compaction to the session's TranscriptService (UC1)
-       * and return the pre-compact session context snapshot for re-injection (UC2).
+       * Handle PostCompact: signal compaction to the session's TranscriptService.
+       *
+       * PostCompact does not re-inject session context: Claude Code's PostCompact hook-output
+       * schema has no additionalContext variant, so any snapshot would be rejected. Persona and
+       * user-profile context are re-established after compaction via the SessionStart:compact
+       * staging path instead.
        */
       async handlePostCompact(event, options) {
         const log = options?.logger ?? this.logger;
         const sessionId = event.context?.sessionId;
         if (!sessionId) {
           log.warn("PostCompact event missing sessionId");
-          return {};
+          return;
         }
         const transcriptService = this.serviceFactory.getTranscriptService(sessionId);
         if (transcriptService) {
@@ -81170,81 +81164,6 @@ ${postCompactResponse.additionalContext}` : postCompactResponse.additionalContex
         } else {
           log.warn("PostCompact: no transcript service for session", { sessionId });
         }
-        const snapshot = this.compactionSnapshots.get(sessionId);
-        if (snapshot) {
-          this.compactionSnapshots.delete(sessionId);
-          log.info("PostCompact: re-injecting compaction snapshot", {
-            sessionId,
-            snapshotLength: snapshot.length
-          });
-          return { additionalContext: snapshot };
-        }
-        return {};
-      }
-      /**
-       * Handle PreCompact: capture staged SessionStart reminders as a compaction snapshot.
-       * The snapshot is returned by the next PostCompact call for context re-injection.
-       */
-      async handlePreCompact(event, options) {
-        const log = options?.logger ?? this.logger;
-        const sessionId = event.context?.sessionId;
-        if (!sessionId)
-          return;
-        try {
-          const minimalCtx = {
-            role: "daemon",
-            config: {
-              core: {
-                logging: {
-                  level: this.configService.core.logging.level,
-                  components: this.configService.core.logging.components
-                },
-                development: { enabled: this.configService.core.development.enabled }
-              },
-              llm: {
-                defaultProfile: this.configService.llm.defaultProfile,
-                defaultFallbackProfileId: this.configService.llm.defaultFallbackProfileId,
-                profiles: this.configService.llm.profiles,
-                fallbackProfiles: this.configService.llm.fallbackProfiles
-              },
-              getAll: () => this.configService.getAll(),
-              getFeature: (name) => this.configService.getFeature(name)
-            },
-            logger: log,
-            assets: this.assetResolver,
-            paths: this.buildRuntimePaths(),
-            stateService: this.stateService,
-            // Remaining DaemonContext fields are not needed by the resolve functions
-            handlers: this.handlerRegistry,
-            staging: this.serviceFactory.getStagingService(sessionId),
-            transcript: void 0,
-            llm: void 0,
-            profileFactory: void 0,
-            orchestrator: this.orchestrator,
-            personaClearCache: { consume: () => this.consumeCachedPersona() }
-          };
-          const [personaContext, userProfileContext] = await Promise.all([
-            (0, feature_reminders_1.resolvePersonaContextForSnapshot)(minimalCtx, sessionId),
-            Promise.resolve((0, feature_reminders_1.resolveUserProfileContextForSnapshot)(minimalCtx))
-          ]);
-          const contextParts = [personaContext, userProfileContext].filter((c) => typeof c === "string" && c.trim().length > 0);
-          if (contextParts.length === 0) {
-            log.debug("PreCompact: no context to snapshot", { sessionId });
-            return;
-          }
-          const snapshot = ["Context was compacted. Re-establishing session context:", "", ...contextParts].join("\n");
-          this.compactionSnapshots.set(sessionId, snapshot);
-          log.info("PreCompact: compaction snapshot captured", {
-            sessionId,
-            reminderCount: contextParts.length,
-            snapshotLength: snapshot.length
-          });
-        } catch (err) {
-          log.warn("PreCompact: failed to capture snapshot", {
-            sessionId,
-            error: (0, core_1.toErrorMessage)(err)
-          });
-        }
       }
       /**
        * Handle SessionStart-specific logic: clear staging on startup/clear.
@@ -81290,6 +81209,7 @@ ${postCompactResponse.additionalContext}` : postCompactResponse.additionalContex
         const log = options?.logger ?? this.logger;
         const sessionId = event.context?.sessionId;
         if (sessionId) {
+          this.sessionTypes.delete(sessionId);
           const payload = event.payload;
           log.info("[persona-lifecycle] ClearHandoff: handleSessionEnd entered", {
             sessionId,
@@ -81329,7 +81249,6 @@ ${postCompactResponse.additionalContext}` : postCompactResponse.additionalContex
           }
           await this.llmManager.shutdownSessionProvider(sessionId, log);
           this.logMetrics.deleteSessionCounters(sessionId);
-          this.compactionSnapshots.delete(sessionId);
           await this.serviceFactory.shutdownSession(sessionId);
           await this.sessionLogWriter.closeSession(sessionId);
           log.info("Session ended");
@@ -81668,7 +81587,8 @@ ${postCompactResponse.additionalContext}` : postCompactResponse.additionalContex
           orchestrator: this.orchestrator,
           personaClearCache: {
             consume: () => this.consumeCachedPersona()
-          }
+          },
+          sessionType: this.sessionTypes.get(sessionId) ?? "interactive"
         };
         this.handlerRegistry.updateSession({ sessionId, transcriptPath });
         this.handlerRegistry.setStagingProvider(() => stagingService);

package/package.json

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