@curie-agent/daemon 0.4.2 → 0.4.4

package/dist/src/jsonrpc-handler.js

@@ -1,22 +1,12 @@
 import os, { homedir } from 'node:os';
 import path, { join } from 'node:path';
 import { readFileSync, existsSync, readdirSync, statSync, writeFileSync, mkdirSync, realpathSync } from 'node:fs';
-import { fileURLToPath } from 'node:url';
 import { Method, renderSlashCommandHelp, findSlashCommand } from '@curie-agent/protocol';
 import { TurnLoop, parseReminderTime, listSnapshots, revertTo, createIdentityFilesAuto, PURE_TOOLS } from '@curie-agent/core';
+import { compactMessages, reconstructMessagesFromEvents, resolveBudget, estimateRequestTokens, breakdownChars, fillPct, formatTokens, DEFAULT_CALIBRATION, DEFAULT_SETTINGS, } from '@curie-agent/core';
 import { listSkills, discoverAllSkills } from '@curie-agent/tools';
 import { executeCd } from './slash-cd.js';
-/** Real package version — previously hardcoded and years out of date. */
-const VERSION = (() => {
-    try {
-        const here = path.dirname(fileURLToPath(import.meta.url));
-        const pkg = JSON.parse(readFileSync(join(here, '..', '..', 'package.json'), 'utf-8'));
-        return pkg.version ?? '0.0.0';
-    }
-    catch {
-        return '0.0.0';
-    }
-})();
+import { VERSION } from './version.js';
 export class JsonRpcHandler {
     sessionStore;
     settingsManager;
@@ -305,10 +295,11 @@ export class JsonRpcHandler {
                 }
                 case Method.CONFIG_GET: {
                     const key = this.getStringParam(params, 'key');
-                    if (!key)
-                        return this.paramError('key');
                     const settings = this.settingsManager.get();
-                    result = this.getNestedValue(settings, key);
+                    // '*' or omitted → the whole tree, in one consistent snapshot. Reading
+                    // ~23 top-level keys one at a time can tear if another client writes
+                    // mid-fetch, yielding a snapshot that never existed on disk.
+                    result = !key || key === '*' ? settings : this.getNestedValue(settings, key);
                     break;
                 }
                 case Method.CONFIG_SET: {
@@ -340,14 +331,7 @@ export class JsonRpcHandler {
                     else {
                         this.settingsManager.update({ [key]: value });
                     }
-                    this.settingsManager.save();
-                    this.sharedEventBus?.emit({
-                        type: 'config-changed',
-                        id: Math.random().toString(36).substring(7),
-                        timestamp: Date.now(),
-                        key,
-                        value,
-                    });
+                    this.emitConfigChanged(key, value);
                     if (key.startsWith('heartbeat') && this.daemonApp) {
                         const updatedSettings = this.settingsManager.get();
                         if (updatedSettings.heartbeat?.schedule === 'on') {
@@ -363,6 +347,17 @@ export class JsonRpcHandler {
                             this.daemonApp.taskManager.cancelAllHeartbeats();
                         }
                     }
+                    // Keep the derived top-level `model` in sync with the active provider.
+                    // SettingsManager.load() recomputes it from providers[current_provider].model,
+                    // so a stale top-level value silently reverts on the next daemon start.
+                    if (key === 'current_provider' || /^providers\.[^.]+\.model$/.test(key)) {
+                        const synced = this.settingsManager.get();
+                        const active = synced.providers[synced.current_provider];
+                        if (active && synced.model !== active.model) {
+                            this.settingsManager.update({ model: active.model });
+                            this.emitConfigChanged('model', active.model);
+                        }
+                    }
                     result = { status: 'ok', key, value };
                     break;
                 }
@@ -824,21 +819,26 @@ export class JsonRpcHandler {
                     if (typeof p.status === 'string') {
                         this.daemonApp.taskManager.updateTaskStatus(taskId, p.status);
                     }
+                    // Patch through updateTask() rather than mutating the object in place —
+                    // an in-place edit followed by save() writes a stale array.
+                    const patch = {};
                     if (typeof p.priority === 'string')
-                        task.priority = p.priority;
+                        patch.priority = p.priority;
                     if (typeof p.title === 'string')
-                        task.title = p.title;
+                        patch.title = p.title;
                     if (typeof p.description === 'string')
-                        task.description = p.description;
+                        patch.description = p.description;
                     if (Array.isArray(p.tags))
-                        task.tags = p.tags;
+                        patch.tags = p.tags;
                     if (typeof p.mode === 'string')
-                        task.mode = p.mode;
+                        patch.mode = p.mode;
                     if (typeof p.scope === 'string')
-                        task.scope = p.scope;
+                        patch.scope = p.scope;
                     if (typeof p.scheduled_at === 'number')
-                        task.scheduled_at = p.scheduled_at;
-                    this.daemonApp.taskManager.save();
+                        patch.scheduled_at = p.scheduled_at;
+                    if (Object.keys(patch).length > 0) {
+                        this.daemonApp.taskManager.updateTask(task.id, patch);
+                    }
                     this.sharedEventBus?.emit({ type: 'todo-changed', id: crypto.randomUUID(), timestamp: Date.now(), action: 'updated', taskId });
                     result = { ok: true, task: this.daemonApp.taskManager.findTask(taskId) };
                     break;
@@ -905,6 +905,7 @@ export class JsonRpcHandler {
             'tool-result', 'approval-decision', 'usage',
             'error', 'session-start', 'session-stop', 'hook', 'status',
             'session-resumed', 'context-warning', 'thinking-delta',
+            'compaction', 'context-report',
             // Subagent events
             'agent-start', 'agent-text-delta', 'agent-thinking-delta',
             'agent-tool-call', 'agent-tool-result', 'agent-usage',
@@ -918,7 +919,9 @@ export class JsonRpcHandler {
         }
         try {
             const result = await loop.run(text);
-            await this.checkContextThresholds(sessionId);
+            // No post-run threshold check: TurnLoop enforces the budget before every
+            // provider call, on every code path (channels, tasks, heartbeat,
+            // subagents) — not just between prompts on this one.
             return { status: 'completed', sessionId: result.sessionId, events: result.events.length };
         }
         catch (err) {
@@ -936,6 +939,21 @@ export class JsonRpcHandler {
     paramError(key) {
         return { jsonrpc: '2.0', id: 0, error: { code: -32602, message: `Missing required parameter: ${key}` } };
     }
+    /**
+     * Broadcast a settings change so every connected client re-reads config.
+     * 'config-changed' is already in ws-handler's newEventTypes, so this reaches
+     * the browser. Every settings write must emit — a write that doesn't leaves
+     * the web dashboard showing a stale value until the user reloads.
+     */
+    emitConfigChanged(key, value) {
+        this.sharedEventBus?.emit({
+            type: 'config-changed',
+            id: Math.random().toString(36).substring(7),
+            timestamp: Date.now(),
+            key,
+            value,
+        });
+    }
     /** Get a nested value from an object using dot notation (e.g. "providers.anthropic.model"). */
     getNestedValue(obj, path) {
         const parts = path.split('.');
@@ -1008,6 +1026,7 @@ export class JsonRpcHandler {
                     const pricing = pConfig?.model_cost
                         ? `\`${pConfig.model_cost}\` (per million)`
                         : 'Not configured';
+                    const statusBudget = resolveBudget(settings);
                     const statusText = `### Curie Agent Status
 * **Version:** \`${VERSION}\`
 * **Active Model:** \`${settings.model}\`
@@ -1015,11 +1034,11 @@ export class JsonRpcHandler {
 * **Approval Mode:** \`${settings.mode || 'auto'}\`
 * **Reasoning Effort:** \`${settings.effort || 'auto'}\`
 * **Workspace CWD:** \`${this.getSessionCwd(sessionId) || process.cwd()}\`
-* **Tools per Turn Limit:** \`${settings.tools_per_call || 10}\`
-* **Web Search per Turn Limit:** \`${settings.websearch_per_call || 5}\`
-* **Model Context Window:** \`${(pConfig?.model_context_window || 200000).toLocaleString()} tokens\`
+* **Tools per Turn / Run:** \`${String(settings.tools_per_call || DEFAULT_SETTINGS.tools_per_call)}\` / \`${String(settings.tools_per_run || DEFAULT_SETTINGS.tools_per_run)}\`
+* **Web Search per Turn / Run:** \`${String(settings.websearch_per_call || DEFAULT_SETTINGS.websearch_per_call)}\` / \`${String(settings.websearch_per_run || DEFAULT_SETTINGS.websearch_per_run)}\`
+* **Model Context Window:** \`${statusBudget.windowTokens.toLocaleString()} tokens\` (\`${statusBudget.usableTokens.toLocaleString()}\` usable, \`${statusBudget.reservedOutput.toLocaleString()}\` reserved for output)
 * **Model Pricing:** ${pricing}
-* **Auto-Compaction:** \`${settings.auto_compact?.enabled || 'on'}\` (Threshold: \`${settings.auto_compact?.threshold ?? 80}%\`)`;
+* **Auto-Compaction:** \`${settings.auto_compact?.enabled ?? DEFAULT_SETTINGS.auto_compact.enabled}\` (warn \`${String(settings.auto_compact?.warn_threshold ?? DEFAULT_SETTINGS.auto_compact.warn_threshold)}%\`, suggest \`${String(settings.auto_compact?.threshold ?? DEFAULT_SETTINGS.auto_compact.threshold)}%\`, compact \`${String(settings.auto_compact?.forced_threshold ?? DEFAULT_SETTINGS.auto_compact.forced_threshold)}%\`)`;
                     emitDelta(statusText);
                     break;
                 }
@@ -1126,13 +1145,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                     else {
                         settings.theme = theme;
                         this.settingsManager.update(settings);
-                        this.sharedEventBus?.emit({
-                            type: 'config-changed',
-                            id: Math.random().toString(36).substring(7),
-                            timestamp: Date.now(),
-                            key: 'theme',
-                            value: theme,
-                        });
+                        this.emitConfigChanged('theme', theme);
                         emitDelta(`Successfully switched theme to: **${theme}**`);
                     }
                     break;
@@ -1149,6 +1162,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                     else {
                         settings.mode = args.toLowerCase();
                         this.settingsManager.update(settings);
+                        this.emitConfigChanged('mode', settings.mode);
                         emitDelta(`Successfully switched approval mode to: **${args.toLowerCase()}**`);
                     }
                     break;
@@ -1165,6 +1179,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                     else {
                         settings.effort = args.toLowerCase();
                         this.settingsManager.update(settings);
+                        this.emitConfigChanged('effort', settings.effort);
                         emitDelta(`Successfully switched reasoning effort to: **${args.toLowerCase()}**`);
                     }
                     break;
@@ -1192,8 +1207,11 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                                     settings.websearch_per_call = wsVal;
                                 }
                             }
-                            this.settingsManager.save();
-                            emitDelta(`Tool limits updated! Tools per turn: **${settings.tools_per_call}**, WebSearch per turn: **${settings.websearch_per_call ?? 5}**`);
+                            this.settingsManager.update({
+                                tools_per_call: settings.tools_per_call,
+                                websearch_per_call: settings.websearch_per_call,
+                            });
+                            emitDelta(`Tool limits updated! Tools per turn: **${String(settings.tools_per_call)}**, WebSearch per turn: **${String(settings.websearch_per_call ?? 5)}**`);
                         }
                     }
                     break;
@@ -1212,9 +1230,8 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                             emitDelta(`Invalid value: "${args}". Must be a positive integer.`);
                         }
                         else {
-                            settings.websearch_per_call = val;
-                            this.settingsManager.save();
-                            emitDelta(`WebSearch/WebFetch limit per turn set to: **${val}**`);
+                            this.settingsManager.update({ websearch_per_call: val });
+                            emitDelta(`WebSearch/WebFetch limit per turn set to: **${String(val)}**`);
                         }
                     }
                     break;
@@ -1343,122 +1360,121 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                     const sub = parts[0]?.toLowerCase();
                     const arg1 = parts[1]?.toLowerCase();
                     const arg2 = parts[2]?.toLowerCase();
+                    const d = DEFAULT_SETTINGS.auto_compact;
+                    /** Persist through update() — get() returns a deep clone, so mutating it is a no-op. */
+                    const saveAutoCompact = (patch) => {
+                        this.settingsManager.update({ auto_compact: { ...settings.auto_compact, ...patch } });
+                    };
                     if (sub === 'auto') {
-                        const s = settings;
-                        if (!s.auto_compact) {
-                            s.auto_compact = { enabled: 'on', threshold: 80, warn_threshold: 15, forced_threshold: 85 };
-                        }
+                        const ac = settings.auto_compact ?? d;
                         if (!arg1) {
-                            emitDelta(`### Auto-Compaction Settings:
-* **Auto-compact**: \`${s.auto_compact.enabled}\`
-* **Threshold**: \`${s.auto_compact.threshold ?? 80}%\`
-* **Warning Threshold**: \`${s.auto_compact.warn_threshold ?? 15}%\`
-* **Pricing Warn**: \`${s.pricing_tier_warn ?? 'off'}\`
+                            emitDelta(`### Auto-Compaction Settings
 
-**Usage**: \`/context auto [on|off|threshold N|warn N|pricing on/off]\``);
-                        }
-                        else if (arg1 === 'on') {
-                            s.auto_compact.enabled = 'on';
-                            this.settingsManager.save();
-                            emitDelta(`Autocompaction enabled.`);
+* **Auto-compact**: \`${ac.enabled}\`
+* **Warn at**: \`${String(ac.warn_threshold ?? d.warn_threshold)}%\`
+* **Suggest at**: \`${String(ac.threshold ?? d.threshold)}%\`
+* **Compact at**: \`${String(ac.forced_threshold ?? d.forced_threshold)}%\`
+* **Summarizer model**: \`${ac.model || '(active model)'}\`
+* **Pricing warn**: \`${settings.pricing_tier_warn ?? 'off'}\`
+
+**Usage**: \`/context auto [on|off|threshold N|warn N|forced N|pricing on/off]\``);
                         }
-                        else if (arg1 === 'off') {
-                            s.auto_compact.enabled = 'off';
-                            this.settingsManager.save();
-                            emitDelta(`Autocompaction disabled.`);
+                        else if (arg1 === 'on' || arg1 === 'off') {
+                            saveAutoCompact({ enabled: arg1 });
+                            emitDelta(`Auto-compaction ${arg1 === 'on' ? 'enabled' : 'disabled'}.`);
                         }
-                        else if (arg1 === 'threshold' && arg2) {
+                        else if ((arg1 === 'threshold' || arg1 === 'warn' || arg1 === 'forced') && arg2) {
                             const pct = parseInt(arg2, 10);
-                            if (isNaN(pct) || pct < 10 || pct > 99) {
-                                emitDelta(`Invalid threshold. Use a value between 10 and 99.`);
+                            if (isNaN(pct) || pct < 5 || pct > 99) {
+                                emitDelta(`Invalid value. Use a percentage between 5 and 99.`);
                             }
                             else {
-                                s.auto_compact.threshold = pct;
-                                this.settingsManager.save();
-                                emitDelta(`Compaction threshold set to ${pct}%.`);
+                                const key = arg1 === 'threshold' ? 'threshold' : arg1 === 'warn' ? 'warn_threshold' : 'forced_threshold';
+                                saveAutoCompact({ [key]: pct });
+                                emitDelta(`${arg1 === 'threshold' ? 'Suggest' : arg1 === 'warn' ? 'Warn' : 'Forced compaction'} threshold set to ${String(pct)}%.`);
                             }
                         }
-                        else if (arg1 === 'warn' && arg2) {
-                            const pct = parseInt(arg2, 10);
-                            if (isNaN(pct) || pct < 5 || pct > 95) {
-                                emitDelta(`Invalid warning threshold. Use a value between 5 and 95.`);
-                            }
-                            else {
-                                s.auto_compact.warn_threshold = pct;
-                                this.settingsManager.save();
-                                emitDelta(`Warning threshold set to ${pct}%.`);
-                            }
-                        }
-                        else if (arg1 === 'pricing') {
-                            if (arg2 === 'on') {
-                                s.pricing_tier_warn = 'on';
-                                this.settingsManager.save();
-                                emitDelta(`Pricing tier warnings enabled.`);
-                            }
-                            else if (arg2 === 'off') {
-                                s.pricing_tier_warn = 'off';
-                                this.settingsManager.save();
-                                emitDelta(`Pricing tier warnings disabled.`);
-                            }
-                            else {
-                                emitDelta(`Usage: \`/context auto pricing on/off\``);
-                            }
+                        else if (arg1 === 'pricing' && (arg2 === 'on' || arg2 === 'off')) {
+                            this.settingsManager.update({ pricing_tier_warn: arg2 });
+                            emitDelta(`Pricing tier warnings ${arg2 === 'on' ? 'enabled' : 'disabled'}.`);
                         }
                         else {
-                            emitDelta(`Usage: \`/context auto [on|off|threshold N|warn N|pricing on/off]\``);
+                            emitDelta(`Usage: \`/context auto [on|off|threshold N|warn N|forced N|pricing on/off]\``);
                         }
                     }
                     else if (sub === 'compact') {
-                        emitDelta(`### ⚡ Manual Compaction Triggered\n\nAnalyzing conversation history and building summary...`);
+                        emitDelta(`Compacting conversation history…`);
                         try {
-                            const summary = await this.runAutomaticCompaction(sessionId, 'detailed');
-                            emitDelta(`\n\n⚡ **Manual Compaction Executed Successfully!**\n\nConversation history has been summarized, reducing the active context usage to just ~500 tokens. The agent will continue seamlessly!\n\n**Restored Context Summary:**\n\n${summary}`);
+                            const r = await this.runAutomaticCompaction(sessionId);
+                            emitDelta(`\n\n**Compacted.** ${String(r.summarizedMessageCount)} message(s) summarized, ` +
+                                `~${formatTokens(r.estimatedTokensBefore)} → ~${formatTokens(r.estimatedTokensAfter)} tokens. ` +
+                                `The full transcript is preserved on disk.\n\n**Summary:**\n\n${r.summary}`);
                         }
                         catch (err) {
-                            emitDelta(`\n\n❌ **Compaction Failed**: ${err instanceof Error ? err.message : String(err)}`);
+                            emitDelta(`\n\n**Compaction failed**: ${err instanceof Error ? err.message : String(err)}`);
                         }
                     }
+                    else if (sub === 'messages') {
+                        const messages = reconstructMessagesFromEvents(this.sessionStore.loadEvents(sessionId) || []);
+                        if (messages.length === 0) {
+                            emitDelta(`No messages in this session yet.`);
+                            break;
+                        }
+                        const lines = [`### Messages (${String(messages.length)})`, '', '| # | Role | Tokens | Detail |', '|---|---|---|---|'];
+                        messages.forEach((m, i) => {
+                            const tokens = estimateRequestTokens({ messages: [m] });
+                            const detail = m.role === 'tool'
+                                ? (m.toolName ?? 'tool')
+                                : m.role === 'user'
+                                    ? `${String(m.content).slice(0, 60).replace(/[\n|]/g, ' ')}…`
+                                    : m.content.map((b) => b.type === 'tool-use' ? `→ ${b.name}` : b.type).join(', ');
+                            lines.push(`| ${String(i + 1)} | ${m.role} | ${formatTokens(tokens)} | ${detail} |`);
+                        });
+                        emitDelta(lines.join('\n'));
+                    }
                     else {
+                        // Emit measurements, not markup. The TUI and the web dashboard each
+                        // render this natively — the hand-written HTML this replaces showed
+                        // a themed gauge in the browser and nothing at all in the terminal.
+                        const budget = resolveBudget(settings);
                         const activeLoop = this.turnLoops.get(sessionId);
-                        const history = activeLoop
-                            ? activeLoop.eventBus.history()
-                            : (this.sessionStore.loadEvents(sessionId) || []);
-                        const usageEvents = history.filter((e) => e.type === 'usage');
-                        const latestUsage = usageEvents[usageEvents.length - 1];
-                        const input = latestUsage ? (latestUsage.inputTokens || 0) : 0;
-                        const output = latestUsage ? (latestUsage.outputTokens || 0) : 0;
-                        const model = settings.model || 'unknown';
-                        const windowSize = settings.providers?.[settings.current_provider]?.model_context_window ?? 200000;
-                        const pct = input > 0 ? Math.min(100, Math.round((input / windowSize) * 100)) : 0;
-                        const fmt = (n) => {
-                            if (n >= 1_000_000)
-                                return `${(n / 1_000_000).toFixed(1)}m`;
-                            if (n >= 1_000)
-                                return `${(n / 1_000).toFixed(1)}k`;
-                            return String(n);
+                        const model = settings.providers[settings.current_provider]?.model || settings.model;
+                        const messages = activeLoop
+                            ? activeLoop.getMessages()
+                            : reconstructMessagesFromEvents(this.sessionStore.loadEvents(sessionId) || []);
+                        const chars = breakdownChars({
+                            system: this.systemPrompt,
+                            messages,
+                            toolDefinitions: this.tools.map((t) => t.definition),
+                        });
+                        const toTokens = (c) => Math.ceil(c / DEFAULT_CALIBRATION);
+                        const breakdown = [
+                            { label: 'System prompt', tokens: toTokens(chars.system) },
+                            { label: 'Tool definitions', tokens: toTokens(chars.toolDefinitions) },
+                            { label: 'Conversation', tokens: toTokens(chars.conversation) },
+                            { label: 'Tool results', tokens: toTokens(chars.toolResults) },
+                        ];
+                        const usedTokens = breakdown.reduce((sum, b) => sum + b.tokens, 0);
+                        const report = {
+                            type: 'context-report',
+                            id: crypto.randomUUID(),
+                            model,
+                            windowTokens: budget.windowTokens,
+                            usedTokens,
+                            reservedOutput: budget.reservedOutput,
+                            breakdown,
+                            timestamp: Date.now(),
                         };
-                        if (input === 0 && output === 0) {
-                            emitDelta(`No token data yet. Start a conversation to see context window usage.\n\nActive model: \`${model}\` (Max Context: \`${fmt(windowSize)}\` tokens).`);
-                        }
-                        else {
-                            const barColor = pct > 80 ? 'var(--red)' : pct > 50 ? 'var(--yellow)' : 'var(--green)';
-                            const barGradient = pct > 80
-                                ? 'linear-gradient(90deg, var(--red), #e08070)'
-                                : pct > 50
-                                    ? 'linear-gradient(90deg, var(--yellow), #f0d090)'
-                                    : 'linear-gradient(90deg, var(--green), #c0d8a8)';
-                            emitDelta(`<div style="background:var(--s3);border-radius:8px;padding:12px;margin:8px 0">` +
-                                `<div style="display:flex;justify-content:space-between;align-items:center;margin-bottom:6px">` +
-                                `<span style="font-family:monospace;font-size:12px;color:var(--text)">Context Window (${model})</span>` +
-                                `<span style="font-family:monospace;font-size:13px;font-weight:bold;color:${barColor}">${pct}%</span>` +
-                                `</div>` +
-                                `<div style="background:var(--s2);border-radius:4px;height:8px;overflow:hidden">` +
-                                `<div style="width:${pct}%;height:100%;background:${barGradient};border-radius:4px"></div>` +
-                                `</div>` +
-                                `<div style="display:flex;justify-content:space-between;margin-top:8px;font-family:monospace;font-size:11px;color:var(--muted)">` +
-                                `<span>In: ${fmt(input)}</span><span>Out: ${fmt(output)}</span><span>Max: ${fmt(windowSize)}</span>` +
-                                `</div></div>`);
-                        }
+                        this.sharedEventBus?.emit({ ...report, sessionId });
+                        // Plain-text fallback so the command still says something useful on a
+                        // surface that has not yet learned the event.
+                        const pct = fillPct(usedTokens, budget);
+                        const rows = breakdown
+                            .map((b) => `* ${b.label}: ${formatTokens(b.tokens)} (${String(usedTokens > 0 ? Math.round((b.tokens / usedTokens) * 100) : 0)}%)`)
+                            .join('\n');
+                        emitDelta(`### Context Window — \`${model}\`\n\n` +
+                            `**${String(pct)}%** — ${formatTokens(usedTokens)} of ${formatTokens(budget.usableTokens)} usable ` +
+                            `(${formatTokens(budget.windowTokens)} window, ${formatTokens(budget.reservedOutput)} reserved for output)\n\n${rows}`);
                     }
                     break;
                 }
@@ -1476,7 +1492,8 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                         emitDelta(`Failed to parse reminder time. Please format like: \`in 2 hours call developer\` or \`tomorrow at 9:00 am meeting\`.`);
                         break;
                     }
-                    const task = this.daemonApp.taskManager.create({ title: parsed.message, mode: 'notify', scope: 'personal', scheduled_at: parsed.scheduledAt });
+                    this.daemonApp.taskManager.load();
+                    this.daemonApp.taskManager.create({ title: parsed.message, mode: 'notify', scope: 'personal', scheduled_at: parsed.scheduledAt });
                     emitDelta(`Reminder scheduled! **"${parsed.message}"** at ${new Date(parsed.scheduledAt).toLocaleString()}`);
                     break;
                 }
@@ -1562,7 +1579,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                         }
                         case 'enable': {
                             settings.heartbeat.schedule = 'on';
-                            this.settingsManager.save();
+                            this.settingsManager.update({ heartbeat: settings.heartbeat });
                             if (this.daemonApp) {
                                 this.daemonApp.taskManager.rescheduleFromSettings({
                                     HEARTBEAT_INTRADAY: settings.heartbeat.intraday,
@@ -1577,7 +1594,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                         }
                         case 'disable': {
                             settings.heartbeat.schedule = 'off';
-                            this.settingsManager.save();
+                            this.settingsManager.update({ heartbeat: settings.heartbeat });
                             if (this.daemonApp) {
                                 this.daemonApp.taskManager.cancelAllHeartbeats();
                             }
@@ -1605,7 +1622,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                                 }
                                 else {
                                     settings.heartbeat.intraday = tokens.join(',');
-                                    this.settingsManager.save();
+                                    this.settingsManager.update({ heartbeat: settings.heartbeat });
                                     if (this.daemonApp && settings.heartbeat.schedule === 'on') {
                                         this.daemonApp.taskManager.rescheduleFromSettings({
                                             HEARTBEAT_INTRADAY: settings.heartbeat.intraday,
@@ -1633,7 +1650,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                                 }
                                 else {
                                     settings.heartbeat.daily = rest;
-                                    this.settingsManager.save();
+                                    this.settingsManager.update({ heartbeat: settings.heartbeat });
                                     if (this.daemonApp && settings.heartbeat.schedule === 'on') {
                                         this.daemonApp.taskManager.rescheduleFromSettings({
                                             HEARTBEAT_INTRADAY: settings.heartbeat.intraday,
@@ -1662,7 +1679,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                                 }
                                 else {
                                     settings.heartbeat.weekly = rest;
-                                    this.settingsManager.save();
+                                    this.settingsManager.update({ heartbeat: settings.heartbeat });
                                     if (this.daemonApp && settings.heartbeat.schedule === 'on') {
                                         this.daemonApp.taskManager.rescheduleFromSettings({
                                             HEARTBEAT_INTRADAY: settings.heartbeat.intraday,
@@ -1690,7 +1707,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                                 }
                                 else {
                                     settings.heartbeat.monthly = rest;
-                                    this.settingsManager.save();
+                                    this.settingsManager.update({ heartbeat: settings.heartbeat });
                                     if (this.daemonApp && settings.heartbeat.schedule === 'on') {
                                         this.daemonApp.taskManager.rescheduleFromSettings({
                                             HEARTBEAT_INTRADAY: settings.heartbeat.intraday,
@@ -1718,7 +1735,7 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
                                 }
                                 else {
                                     settings.heartbeat.dreaming = rest;
-                                    this.settingsManager.save();
+                                    this.settingsManager.update({ heartbeat: settings.heartbeat });
                                     if (this.daemonApp && settings.heartbeat.schedule === 'on') {
                                         this.daemonApp.taskManager.rescheduleFromSettings({
                                             HEARTBEAT_INTRADAY: settings.heartbeat.intraday,
@@ -2214,129 +2231,50 @@ Usage: \`/model window <tokens>\` (e.g., \`/model window 120000\`).`);
         }
         return true;
     }
-    async runAutomaticCompaction(sessionId, depth) {
+    /**
+     * Compact a session's history.
+     *
+     * The summarization itself lives in `@curie-agent/core` so the TurnLoop can
+     * run it mid-run — this is the out-of-band entry point for `/context compact`
+     * on an idle session.
+     *
+     * Nothing is deleted. A `compaction` marker is appended and message
+     * reconstruction replays forward from it, so the full transcript survives for
+     * the UI, resume and audit while the model carries only the summary.
+     */
+    async runAutomaticCompaction(sessionId) {
         if (!this.createProvider) {
             throw new Error('No provider configured');
         }
         const settings = this.settingsManager.get();
         const provider = this.createProvider(settings);
-        const model = settings.providers[settings.current_provider]?.model || settings.model;
+        const activeModel = settings.providers[settings.current_provider]?.model || settings.model;
         const events = this.sessionStore.loadEvents(sessionId) || [];
         if (events.length === 0) {
             throw new Error('No events to compact');
         }
-        // Build human-readable transcript
-        let transcriptParts = [];
-        for (const e of events) {
-            if (e.type === 'user-prompt' && e.text) {
-                transcriptParts.push(`User: ${e.text}`);
-            }
-            else if (e.type === 'assistant-delta' && e.text) {
-                const lastIdx = transcriptParts.length - 1;
-                if (lastIdx >= 0 && transcriptParts[lastIdx]?.startsWith('Assistant:')) {
-                    transcriptParts[lastIdx] += e.text;
-                }
-                else {
-                    transcriptParts.push(`Assistant: ${e.text}`);
-                }
-            }
-        }
-        const transcript = transcriptParts.join('\n\n');
-        if (!transcript.trim()) {
+        const messages = reconstructMessagesFromEvents(events);
+        if (messages.length === 0) {
             throw new Error('No conversational history found to compact');
         }
-        const systemPrompt = `You are a conversation summarizer. Summarize the provided dialogue in a dense, detailed, high-fidelity paragraph or two. Focus on capturing the original goals, what was accomplished, any modified files or configurations, current settings, and what the pending next steps are. Ensure all key technical details (like file paths, specific code adjustments, command names) are preserved. Do not add any conversational intros or outros; output ONLY the raw summary text.`;
-        const prompt = `Please summarize this conversation history:\n\n${transcript}`;
-        const summary = await provider.check(prompt, { model, system: systemPrompt });
-        const cleanSummary = summary.trim();
-        // Overwrite events log
-        const eventsPath = this.sessionStore.eventsPath(sessionId);
-        const newEvents = [
-            {
-                type: 'session-start',
-                id: crypto.randomUUID(),
-                model,
-                provider: settings.current_provider || 'unknown',
-                cwd: process.cwd(),
-                timestamp: Date.now(),
-            },
-            {
-                type: 'user-prompt',
-                id: crypto.randomUUID(),
-                text: `This is a continuation of a compacted conversation. Here is the high-fidelity summary of our session so far:\n\n${cleanSummary}\n\nLet's continue!`,
-                cwd: process.cwd(),
-                timestamp: Date.now() + 1,
-            },
-            {
-                type: 'assistant-delta',
-                id: crypto.randomUUID(),
-                text: `Got it! I have fully restored our conversation summary and details. Let let me know what you would like to do next!`,
-                timestamp: Date.now() + 2,
-            },
-            {
-                type: 'assistant-stop',
-                id: crypto.randomUUID(),
-                timestamp: Date.now() + 3,
-            }
-        ];
-        const data = newEvents.map((e) => JSON.stringify(e)).join('\n') + '\n';
-        writeFileSync(eventsPath, data, 'utf-8');
-        return cleanSummary;
-    }
-    async checkContextThresholds(sessionId) {
-        const settings = this.settingsManager.get();
-        const history = this.sessionStore.loadEvents(sessionId) || [];
-        const usageEvents = history.filter((e) => e.type === 'usage');
-        const latestUsage = usageEvents[usageEvents.length - 1];
-        const input = latestUsage?.inputTokens ?? 0;
-        if (input === 0)
-            return; // No token data yet
-        const windowSize = settings.providers?.[settings.current_provider]?.model_context_window ?? 200000;
-        const pct = Math.min(100, Math.round((input / windowSize) * 100));
-        const autoCompact = settings.auto_compact || { enabled: 'on', threshold: 80, warn_threshold: 60, forced_threshold: 85 };
-        const warnThresh = autoCompact.warn_threshold ?? 60;
-        const compactThresh = autoCompact.threshold ?? 80;
-        const forcedThresh = autoCompact.forced_threshold ?? 85;
-        const enabled = autoCompact.enabled ?? 'on';
-        if (pct >= forcedThresh && enabled === 'on') {
-            try {
-                const summary = await this.runAutomaticCompaction(sessionId, 'detailed');
-                const successMessage = `⚡ **Auto-Compaction Executed Successfully!**\n\nContext usage was at **${pct}%** (forced threshold: **${forcedThresh}%**).\nWe have summarized the conversation, reducing the history size down to just ~500 tokens. The agent will continue seamlessly!\n\n**Restored Context Summary:**\n\n${summary}`;
-                const warningEvent = {
-                    type: 'context-warning',
-                    id: crypto.randomUUID(),
-                    message: successMessage,
-                    timestamp: Date.now(),
-                };
-                this.sharedEventBus?.emit({ ...warningEvent, sessionId });
-                this.sessionStore.appendEvent(sessionId, { ...warningEvent, sessionId });
-            }
-            catch (err) {
-                console.error('[compaction] Auto-compaction failed:', err);
-            }
-        }
-        else if (pct >= compactThresh) {
-            const suggestMessage = `⚠️ **Context Fill High (${pct}%)**\n\nYour context fill is at **${pct}%** (Threshold: **${compactThresh}%**). Suggesting conversation compaction.\n\nType \`/context compact\` to run compaction, summarize history, and free memory immediately!`;
-            const warningEvent = {
-                type: 'context-warning',
-                id: crypto.randomUUID(),
-                message: suggestMessage,
-                timestamp: Date.now(),
-            };
-            this.sharedEventBus?.emit({ ...warningEvent, sessionId });
-            this.sessionStore.appendEvent(sessionId, { ...warningEvent, sessionId });
-        }
-        else if (pct >= warnThresh) {
-            const warnMessage = `⚠️ **Context Warning (${pct}%)**\n\nContext window is **${pct}%** full (Warning threshold: **${warnThresh}%**).`;
-            const warningEvent = {
-                type: 'context-warning',
-                id: crypto.randomUUID(),
-                message: warnMessage,
-                timestamp: Date.now(),
-            };
-            this.sharedEventBus?.emit({ ...warningEvent, sessionId });
-            this.sessionStore.appendEvent(sessionId, { ...warningEvent, sessionId });
-        }
+        const result = await compactMessages({
+            messages,
+            provider: provider,
+            model: settings.auto_compact?.model || activeModel,
+            budget: resolveBudget(settings),
+        });
+        const marker = {
+            type: 'compaction',
+            id: crypto.randomUUID(),
+            summary: result.summary,
+            summarizedMessageCount: result.summarizedMessageCount,
+            tokensBefore: result.estimatedTokensBefore,
+            tokensAfter: result.estimatedTokensAfter,
+            timestamp: Date.now(),
+        };
+        this.sessionStore.appendEvent(sessionId, { ...marker, sessionId });
+        this.sharedEventBus?.emit({ ...marker, sessionId });
+        return result;
     }
 }
 //# sourceMappingURL=jsonrpc-handler.js.map