pi-smart-compact 7.9.0 → 7.9.1

package/CHANGELOG.md

@@ -1,5 +1,34 @@
 # Changelog
 
+## [7.9.1] - 2026-05-17
+
+### Fixed
+- **`/smart-compact` race condition** — Added `waitForIdle()` to slash command handler. Previously the command could execute concurrently with agent streaming since extension commands bypass the input queue.
+- **Tool `skipCompact` safety** — `ctx.compact()` internally calls `this.abort()`, which would kill the running agent loop if called from within a tool. Tool path now correctly keeps `skipCompact: true` and caches summary in `pendingRef` for the next natural compact.
+- **Tool context-usage guard** — Tool now checks `getContextUsage()` before running and returns early with token info if context is too small.
+- **Tool description enriched** — Better description and parameter hints so the agent knows when and how to call `smart_compact`.
+
+## [7.9.0] - 2026-05-17
+
+### Fixed
+- **40 TypeScript strict-mode errors resolved** — `bunx tsc --noEmit` now passes cleanly with zero errors against latest `@earendil-works/pi-ai`, `pi-coding-agent`, `pi-tui` peer types.
+- **`notify()` level mismatch** — `"success"` is not a valid level in `ExtensionUIContext.notify()`. All instances mapped to `"info"`.
+- **`UserMessage.timestamp` mandatory** — 10 inline message objects across `explore.ts`, `synthesize.ts`, `verify.ts` were missing the required `timestamp: Date.now()` field.
+- **`AgentToolResult.details` mandatory** — All tool return objects in `index.ts` now include explicit `details: undefined`.
+- **`SelectList.selectedIndex` private accessor** — Replaced with `setSelectedIndex()` in `overlays.ts`.
+- **`ThemeColor` union vs arbitrary string** — `theme.fg()` cast to `(c: string, t: string) => string` in overlays for dynamic color lookup.
+- **`CacheAwareOptions` not assignable to `ProviderStreamOptions`** — Added explicit cast in `cache.ts`.
+- **`blocks: unknown` after `Array.isArray`** — Explicit `unknown[]` typing in `extraction.ts` and `pruning.ts`.
+- **`ExtensionContext` vs `ExtensionCommandContext`** — Proper `as unknown as` double-cast in `index.ts` for tool and hook contexts.
+- **`apiKey?: string` (optional) used as `string`** — Extracted `apiKey` and `apiHeaders` as separate non-optional variables after guard check in `core.ts`.
+- **`pendingRef.value` type narrowed to `never`** — TypeScript control flow after `pendingRef.value = null` prevented re-reading after `runSmartCompact`. Fixed with explicit cast.
+
+### Changed
+- **`/smart-compact` command now uses `waitForIdle()`** — Prevents race condition when slash command is entered while agent is streaming. Agent finishes current turn before compaction starts.
+- **Tool (`smart_compact`) keeps `skipCompact: true`** — Mid-turn compaction via `ctx.compact()` would abort the running agent loop (`this.abort()` internally). Tool prepares summary in `pendingRef` for the next natural compact to apply.
+- **Tool description enriched** — Better description, parameter descriptions, and context-usage guard help the agent decide when to call smart_compact.
+- **README.md updated** — version 7.9.0, module count 18, line count ~5,091, typecheck status now passing, added `logger.ts` and `type-guards.ts` to source table.
+
 ## [7.8.0] - 2026-05-17
 
 ### Fixed

package/README.md

@@ -57,12 +57,12 @@ The extension keeps a short-lived pending compaction in memory, then hands that
 
 **Observed from the current codebase (`README`, `src/`, `test/`, `package.json`)**
 
-- **Package version:** `7.8.0`
+- **Package version:** `7.9.1`
 - **Runtime entrypoint:** `dist/index.js`
 - **Source entrypoint:** `src/index.ts`
-- **Source modules:** 16 TypeScript files under `src/`
+- **Source modules:** 18 TypeScript files under `src/`
 - **Tests:** 9 test files, **93 passing tests**
-- **Approx repo footprint:** ~5,053 lines across `src/` + `test/`
+- **Approx repo footprint:** ~5,091 lines across `src/` + `test/`
 - **Documentation asset:** `docs/assets/pi-smart-compact.png`
 - **Published package files:** `dist/`, `docs/`, `README.md`, `LICENSE`, `CHANGELOG.md`
 
@@ -72,9 +72,9 @@ The extension keeps a short-lived pending compaction in memory, then hands that
 | --- | --- | --- |
 | `bun test` | ✅ Pass | 93/93 tests passing |
 | `bun run build` | ✅ Pass | Bundles `src/index.ts` to `dist/index.js` |
-| `bun run typecheck` | ⚠️ Fails | Current repo has TypeScript compatibility issues against the installed Pi typings / `.ts` import style |
+| `bun run typecheck` | ✅ Pass | Strict TypeScript compatibility clean |
 
-So the project is currently **buildable and tested**, but **not fully typecheck-clean**.
+So the project is currently **buildable, tested, and typecheck-clean**.
 
 ---
 
@@ -348,11 +348,13 @@ That pending summary is valid for **5 minutes** and is consumed by:
 | `src/utils/cache.ts` | Metrics, cache-aware LLM options, extraction cache |
 | `src/utils/damage.ts` | Post-compaction regression signal detection |
 | `src/utils/extraction.ts` | Deterministic extraction and open-loop detection |
+| `src/utils/logger.ts` | Centralized logging with shared prefix |
 | `src/utils/fingerprint.ts` | Cross-session project fingerprinting |
 | `src/utils/helpers.ts` | Config loading, backups, batching, prompt helpers |
 | `src/utils/pruning.ts` | Redundancy pruning before compaction |
 | `src/utils/state.ts` | Compaction state persistence and delta logic |
 | `src/utils/tokens.ts` | Provider capabilities and token estimation |
+| `src/utils/type-guards.ts` | Shared type guard functions |
 
 ### Tests
 
@@ -589,10 +591,9 @@ The current codebase includes these safeguards:
 
 To keep this README aligned with the repository's **actual** current state:
 
-1. **`bun run typecheck` is currently failing.** The repo builds with Bun and passes tests, but strict TypeScript compatibility is not clean right now.
-2. **One legacy test filename remains:** `test/semantic-compact.test.ts`.
-3. **The package is published from `dist/`, not directly from `src/`.** Source and tests are not included in the package tarball.
-4. **The extension depends on Pi runtime APIs and peer packages** (`@earendil-works/pi-ai`, `@earendil-works/pi-coding-agent`, `@earendil-works/pi-tui`, `typebox`).
+1. **One legacy test filename remains:** `test/semantic-compact.test.ts`.
+2. **The package is published from `dist/`**, not directly from `src/`. Source and tests are not included in the package tarball.
+3. **The extension depends on Pi runtime APIs and peer packages** (`@earendil-works/pi-ai`, `@earendil-works/pi-coding-agent`, `@earendil-works/pi-tui`, `typebox`).
 
 ---
 
@@ -629,7 +630,7 @@ rm -rf dist && mkdir dist && bun build ./src/index.ts --outdir ./dist --target b
 bun run typecheck
 ```
 
-At the moment, this command is expected to report errors.
+At the moment, this command passes cleanly.
 
 ### Typical local path inside Pi
 

package/dist/constants.d.ts

@@ -2,7 +2,7 @@
  * Constants, prompts, and profile defaults.
  */
 import type { CompressionProfile, ProfileConfig } from "./types.ts";
-export declare const VERSION = "7.9.0";
+export declare const VERSION = "7.9.1";
 export declare const CHARS_PER_TOKEN = 3.8;
 export declare const COMPACT_SYSTEM_PREFIX: string;
 export declare const PROFILES: Record<CompressionProfile, ProfileConfig>;

package/dist/core.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"core.d.ts","sourceRoot":"","sources":["../src/core.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,iCAAiC,CAAC;AAE/E,OAAO,KAAK,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,uBAAuB,CAAC;AACxD,OAAO,KAAK,EACV,kBAAkB,EAAE,iBAAiB,EAEtC,MAAM,YAAY,CAAC;AAsBpB,yEAAyE;AACzE,MAAM,WAAW,mBAAmB;IAClC,GAAG,EAAE,uBAAuB,CAAC;IAC7B,YAAY,EAAE,KAAK,CAAC,GAAG,CAAC,CAAC;IACzB,QAAQ,EAAE,KAAK,CAAC,GAAG,CAAC,CAAC;IACrB,OAAO,EAAE,kBAAkB,CAAC;IAC5B,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,UAAU,EAAE;QAAE,KAAK,EAAE,iBAAiB,GAAG,IAAI,CAAC;QAAC,SAAS,EAAE,MAAM,CAAA;KAAE,CAAC;IACnE,SAAS,EAAE;QAAE,KAAK,EAAE,OAAO,CAAA;KAAE,CAAC;IAC9B,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,OAAO,CAAC;CACvB;AAED,wBAAsB,eAAe,CAAC,IAAI,EAAE,mBAAmB,GAAG,OAAO,CAAC,IAAI,CAAC,CA8V9E"}
+{"version":3,"file":"core.d.ts","sourceRoot":"","sources":["../src/core.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,iCAAiC,CAAC;AAE/E,OAAO,KAAK,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,uBAAuB,CAAC;AACxD,OAAO,KAAK,EACV,kBAAkB,EAAE,iBAAiB,EAEtC,MAAM,YAAY,CAAC;AAsBpB,yEAAyE;AACzE,MAAM,WAAW,mBAAmB;IAClC,GAAG,EAAE,uBAAuB,CAAC;IAC7B,YAAY,EAAE,KAAK,CAAC,GAAG,CAAC,CAAC;IACzB,QAAQ,EAAE,KAAK,CAAC,GAAG,CAAC,CAAC;IACrB,OAAO,EAAE,kBAAkB,CAAC;IAC5B,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,UAAU,EAAE;QAAE,KAAK,EAAE,iBAAiB,GAAG,IAAI,CAAC;QAAC,SAAS,EAAE,MAAM,CAAA;KAAE,CAAC;IACnE,SAAS,EAAE;QAAE,KAAK,EAAE,OAAO,CAAA;KAAE,CAAC;IAC9B,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,OAAO,CAAC;CACvB;AAED,wBAAsB,eAAe,CAAC,IAAI,EAAE,mBAAmB,GAAG,OAAO,CAAC,IAAI,CAAC,CAgW9E"}

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,KAAK,EAAE,YAAY,EAA2B,MAAM,iCAAiC,CAAC;AAwC7F,MAAM,CAAC,OAAO,UAAU,qBAAqB,CAAC,EAAE,EAAE,YAAY,QA0H7D"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,KAAK,EAAE,YAAY,EAA2B,MAAM,iCAAiC,CAAC;AAwC7F,MAAM,CAAC,OAAO,UAAU,qBAAqB,CAAC,EAAE,EAAE,YAAY,QA2I7D"}

package/dist/index.js

@@ -1,6 +1,6 @@
 // @bun
 // src/constants.ts
-var VERSION = "7.9.0";
+var VERSION = "7.9.1";
 var CHARS_PER_TOKEN = 3.8;
 var COMPACT_SYSTEM_PREFIX = "You are an expert conversation summarizer for a coding agent. " + "Produce structured markdown summaries. " + "Follow output format exactly. " + "Use EXACT names \u2014 never paraphrase code identifiers. " + "Trust deterministic extraction data over intuition.";
 var PROFILES = {
@@ -782,7 +782,8 @@ function catalogErrors(msgs, _tcIdx) {
   for (const err of errors) {
     for (let j = err.index + 1;j < Math.min(msgs.length, err.index + 6); j++) {
       if (msgs[j]?.role === "assistant") {
-        const blocks = Array.isArray(msgs[j]?.content) ? msgs[j].content : [];
+        const rawBlocks = msgs[j]?.content;
+        const blocks = Array.isArray(rawBlocks) ? rawBlocks : [];
         for (const b of blocks) {
           if (isToolCallBlock(b) && b.name === err.tool) {
             err.retryAttempted = true;
@@ -1312,7 +1313,8 @@ function pruneRedundant(msgs) {
   for (let idx = 0;idx < msgs.length; idx++) {
     if (msgs[idx].role !== "assistant")
       continue;
-    const blocks = Array.isArray(msgs[idx].content) ? msgs[idx].content : [];
+    const rawBlocks = msgs[idx].content;
+    const blocks = Array.isArray(rawBlocks) ? rawBlocks : [];
     const hasToolCall = blocks.some((b) => isToolCallBlock(b));
     if (hasToolCall)
       continue;
@@ -1832,7 +1834,7 @@ async function exploreConversation(llmMessages, extraction, model, auth, prevSum
     }
     const probeResp = await trackedComplete("explore", model, {
       systemPrompt: COMPACT_SYSTEM_PREFIX,
-      messages: [{ role: "user", content: [{ type: "text", text: userContent }] }],
+      messages: [{ role: "user", content: [{ type: "text", text: userContent }], timestamp: Date.now() }],
       tools: EXPLORATION_TOOLS
     }, cacheOpts({ apiKey: auth.apiKey, headers: auth.headers, signal }, model.provider));
     const toolCalls = probeResp.content.filter((c) => c.type === "toolCall");
@@ -1929,7 +1931,7 @@ User steering: ` + userNote : "");
   try {
     const resp = await trackedComplete("explore-retry", model, {
       systemPrompt: COMPACT_SYSTEM_PREFIX,
-      messages: [{ role: "user", content: [{ type: "text", text: retryPrompt }] }]
+      messages: [{ role: "user", content: [{ type: "text", text: retryPrompt }], timestamp: Date.now() }]
     }, cacheOpts({ apiKey: auth.apiKey, headers: auth.headers, maxTokens: Math.min(4096, getProviderCaps(model.provider).maxOutputTokens), signal }, model.provider));
     const text = resp.content.filter((c) => c.type === "text").map((c) => c.text).join("").trim();
     return parseExplorationReport(text, llmMessages);
@@ -1967,7 +1969,7 @@ Output ONLY JSON: {"mainGoal":"...","sessionType":"implementation|review|debuggi
   try {
     const resp = await trackedComplete("explore-direct", model, {
       systemPrompt: COMPACT_SYSTEM_PREFIX,
-      messages: [{ role: "user", content: [{ type: "text", text: prompt }] }]
+      messages: [{ role: "user", content: [{ type: "text", text: prompt }], timestamp: Date.now() }]
     }, cacheOpts({ apiKey: auth.apiKey, headers: auth.headers, maxTokens: Math.min(4096, getProviderCaps(model.provider).maxOutputTokens), signal }, model.provider));
     const text = resp.content.filter((c) => c.type === "text").map((c) => c.text).join(`
 `).trim();
@@ -2051,8 +2053,8 @@ Session-specific instructions:
   const resp = await trackedComplete("single-pass", model, {
     systemPrompt: COMPACT_SYSTEM_PREFIX,
     messages: [
-      { role: "user", content: [{ type: "text", text: adaptedPrefix }] },
-      { role: "user", content: [{ type: "text", text: dynamicSuffix }] }
+      { role: "user", content: [{ type: "text", text: adaptedPrefix }], timestamp: Date.now() },
+      { role: "user", content: [{ type: "text", text: dynamicSuffix }], timestamp: Date.now() }
     ]
   }, cacheOpts({ apiKey: auth.apiKey, headers: auth.headers, maxTokens: getProviderCaps(model.provider).maxOutputTokens, signal }, model.provider));
   const summary = resp.content.filter((c) => c.type === "text").map((c) => c.text).join(`
@@ -2082,8 +2084,8 @@ async function summarizeBatch(batch, extraction, model, auth, signal) {
   const resp = await trackedComplete("batch", model, {
     systemPrompt: COMPACT_SYSTEM_PREFIX,
     messages: [
-      { role: "user", content: [{ type: "text", text: BATCH_PROMPT_PREFIX }] },
-      { role: "user", content: [{ type: "text", text: dynamicSuffix }] }
+      { role: "user", content: [{ type: "text", text: BATCH_PROMPT_PREFIX }], timestamp: Date.now() },
+      { role: "user", content: [{ type: "text", text: dynamicSuffix }], timestamp: Date.now() }
     ]
   }, cacheOpts({ apiKey: auth.apiKey, headers: auth.headers, maxTokens: Math.min(4096, getProviderCaps(model.provider).maxOutputTokens), signal }, model.provider));
   const output = resp.content.filter((c) => c.type === "text").map((c) => c.text).join(`
@@ -2123,8 +2125,8 @@ async function assembleLLM(summaries, extraction, report, model, auth, budget, p
   const resp = await trackedComplete("assemble", model, {
     systemPrompt: COMPACT_SYSTEM_PREFIX,
     messages: [
-      { role: "user", content: [{ type: "text", text: ASSEMBLY_PROMPT_PREFIX }] },
-      { role: "user", content: [{ type: "text", text: dynamicSuffix }] }
+      { role: "user", content: [{ type: "text", text: ASSEMBLY_PROMPT_PREFIX }], timestamp: Date.now() },
+      { role: "user", content: [{ type: "text", text: dynamicSuffix }], timestamp: Date.now() }
     ]
   }, cacheOpts({ apiKey: auth.apiKey, headers: auth.headers, maxTokens: Math.min(budget, getProviderCaps(model.provider).maxOutputTokens), signal }, model.provider));
   return resp.content.filter((c) => c.type === "text").map((c) => c.text).join(`
@@ -2337,7 +2339,7 @@ Return the COMPLETE updated summary with missing items integrated. Keep the same
   try {
     const resp = await trackedComplete("patch", model, {
       systemPrompt: COMPACT_SYSTEM_PREFIX,
-      messages: [{ role: "user", content: [{ type: "text", text: patchPrompt }] }]
+      messages: [{ role: "user", content: [{ type: "text", text: patchPrompt }], timestamp: Date.now() }]
     }, cacheOpts({ apiKey: auth.apiKey, headers: auth.headers, maxTokens: 8192, signal }));
     const patched = resp.content.filter((c) => c.type === "text").map((c) => c.text).join(`
 `).trim();
@@ -2357,7 +2359,8 @@ function renderContextBar(theme, pct, tokens, barLen = 24) {
   const filled = Math.min(barLen, Math.round(clamped / 100 * barLen));
   const bar = "\u2588".repeat(filled) + "\u2591".repeat(barLen - filled);
   const color = clamped > 80 ? "error" : clamped > 50 ? "warning" : "success";
-  return theme.fg("text", "  Context: ") + theme.fg(color, bar) + theme.fg("text", " " + clamped + "%") + theme.fg("dim", " (" + (tokens ?? 0).toLocaleString() + "t)");
+  const fg = theme.fg;
+  return fg("text", "  Context: ") + fg(color, bar) + fg("text", " " + clamped + "%") + fg("dim", " (" + (tokens ?? 0).toLocaleString() + "t)");
 }
 function renderTokenBar(theme, before, after, label, barLen = 30) {
   const ratio = before > 0 ? after / before : 0;
@@ -2398,7 +2401,7 @@ async function selectModel(ctx, opts) {
       scrollInfo: (t) => theme.fg("dim", t),
       noMatch: (t) => theme.fg("warning", t)
     });
-    sel.selectedIndex = opts.defaultModelIndex;
+    sel.setSelectedIndex(opts.defaultModelIndex);
     sel.onSelect = (item) => done(item.value);
     sel.onCancel = () => done(null);
     c.addChild(sel);
@@ -2447,7 +2450,7 @@ async function selectProfile(ctx, selectedModel, opts) {
       scrollInfo: (t) => theme.fg("dim", t),
       noMatch: (t) => theme.fg("warning", t)
     });
-    sel.selectedIndex = 1;
+    sel.setSelectedIndex(1);
     sel.onSelect = (item) => done(item.value);
     sel.onCancel = () => done(null);
     c.addChild(sel);
@@ -2472,8 +2475,7 @@ function showProgressOverlay(ctx, state) {
   const progress = Math.round(state.phase / 4 * 100);
   const name = phaseNames[state.phase - 1] ?? "?";
   const detail = state.detail ? " (" + state.detail + ")" : "";
-  const type = state.phase >= 4 ? "success" : "info";
-  ctx.ui.notify("EESV [" + progress + "%] Phase " + state.phase + "/4: " + name + detail, type);
+  ctx.ui.notify("EESV [" + progress + "%] Phase " + state.phase + "/4: " + name + detail, state.phase >= 4 ? "info" : "info");
 }
 async function showResultScreen(ctx, details, extraction) {
   await ctx.ui.custom((tui, theme, _kb, done) => {
@@ -2487,7 +2489,8 @@ async function showResultScreen(ctx, details, extraction) {
     c.addChild(new Text("", 0, 0));
     const methodColors = { eesv: "accent", "single-pass": "success", heuristic: "warning" };
     const methodColor = methodColors[details.method] ?? "text";
-    c.addChild(new Text(theme.fg("text", "  Method: ") + theme.fg(methodColor, details.method.toUpperCase()) + theme.fg("dim", " \u2022 " + details.llmCalls + " LLM call(s) \u2022 Profile: " + details.profile), 0, 0));
+    const fg = theme.fg;
+    c.addChild(new Text(theme.fg("text", "  Method: ") + fg(methodColor, details.method.toUpperCase()) + theme.fg("dim", " \u2022 " + details.llmCalls + " LLM call(s) \u2022 Profile: " + details.profile), 0, 0));
     if (details.model) {
       c.addChild(new Text(theme.fg("dim", "  Model: " + details.model), 0, 0));
     }
@@ -2615,6 +2618,8 @@ async function runSmartCompact(opts) {
         ctx.ui.notify("Auth failed", "error");
       return;
     }
+    const apiKey = auth.apiKey;
+    const apiHeaders = auth.headers;
     const usage = ctx.getContextUsage();
     const totalTokens = usage?.tokens ?? 0;
     if (!totalTokens || totalTokens < MIN_TOKEN_THRESHOLD) {
@@ -2624,7 +2629,7 @@ async function runSmartCompact(opts) {
       return;
     }
     const notify = (msg, type = "info") => {
-      ctx.ui.notify(msg, type);
+      ctx.ui.notify(msg, type === "success" ? "info" : type);
     };
     const ctrl = new AbortController;
     const signal = ctrl.signal;
@@ -2654,7 +2659,7 @@ async function runSmartCompact(opts) {
       isRunning.value = false;
       return;
     }
-    const firstKeptId = msgs[keepFrom]?.id ?? msgs[msgs.length - 1]?.id ?? "";
+    const firstKeptId = msgs[keepFrom]?.id ?? msgs[msgs.length - 1]?.id;
     if (!autoTriggered) {
       showProgressOverlay(ctx, { phase: 1, phaseName: "Extract", detail: "Preparing...", model: modelLabel, profile });
     }
@@ -2698,7 +2703,7 @@ async function runSmartCompact(opts) {
       if (!autoTriggered)
         showProgressOverlay(ctx, { phase: 2, phaseName: "Explore", detail: "Single-pass (" + convTokens.toLocaleString() + "t)", model: modelLabel, profile, extraction });
       try {
-        const r = await singlePassCompact(convText, extraction, null, prevContext + projectCtx, summaryModel, { apiKey: auth.apiKey, headers: auth.headers }, signal);
+        const r = await singlePassCompact(convText, extraction, null, prevContext + projectCtx, summaryModel, { apiKey, headers: apiHeaders }, signal);
         finalSummary = r.summary;
         method = "single-pass";
         llmCalls = r.llmCalls;
@@ -2763,7 +2768,7 @@ async function runSmartCompact(opts) {
       const concurrency = caps.concurrencyLimit;
       if (totalBatches <= 1) {
         try {
-          summaries.push(...await summarizeBatch(batches[0], extraction, summaryModel, { apiKey: auth.apiKey, headers: auth.headers }, signal));
+          summaries.push(...await summarizeBatch(batches[0], extraction, summaryModel, { apiKey, headers: apiHeaders }, signal));
         } catch (err) {
           summaries.push(...batches[0].map((ch) => ({
             topic: ch.topic,
@@ -2786,7 +2791,7 @@ async function runSmartCompact(opts) {
           const wavePromises = waveBatches.map(async (batch, i) => {
             const idx = wave + i;
             try {
-              results[idx] = await summarizeBatch(batch, extraction, summaryModel, { apiKey: auth.apiKey, headers: auth.headers }, signal);
+              results[idx] = await summarizeBatch(batch, extraction, summaryModel, { apiKey, headers: apiHeaders }, signal);
             } catch (err) {
               errors[idx] = err instanceof Error ? err : new Error(String(err));
               results[idx] = batch.map((ch) => ({
@@ -2818,7 +2823,7 @@ async function runSmartCompact(opts) {
         showProgressOverlay(ctx, { phase: 3, phaseName: "Synthesize", detail: "Assembling...", model: modelLabel, profile, extraction, totalBatches: batches.length });
       let assemblyCalls = 1;
       try {
-        const r = await assembleLLM(summaries, extraction, explorationReport, summaryModel, { apiKey: auth.apiKey, headers: auth.headers }, pc.summaryBudgetTokens, prevContext, signal);
+        const r = await assembleLLM(summaries, extraction, explorationReport, summaryModel, { apiKey, headers: apiHeaders }, pc.summaryBudgetTokens, prevContext, signal);
         if (r?.startsWith("##"))
           finalSummary = r;
         else
@@ -2842,7 +2847,7 @@ async function runSmartCompact(opts) {
         if (!recheck.ok && recheck.score < 75) {
           notify("Phase 4 Verify: deterministic patch insufficient (score=" + recheck.score + "), trying LLM patch", "warning");
           try {
-            finalSummary = await patchSummary(finalSummary, recheck.gaps, summaryModel, { apiKey: auth.apiKey, headers: auth.headers }, signal);
+            finalSummary = await patchSummary(finalSummary, recheck.gaps, summaryModel, { apiKey, headers: apiHeaders }, signal);
             llmCalls++;
           } catch (err) {
             warn("LLM patch failed", err);
@@ -2941,7 +2946,7 @@ async function runSmartCompact(opts) {
         customInstructions: "Use pre-computed smart summary from /smart-compact",
         onComplete: () => {
           if (!autoTriggered)
-            ctx.ui.notify("Applied \u2713", "success");
+            ctx.ui.notify("Applied \u2713", "info");
         },
         onError: (e) => {
           if (!autoTriggered)
@@ -2996,6 +3001,7 @@ function smartCompactExtension(pi) {
       return m.length ? m : null;
     },
     handler: async (args, ctx) => {
+      await ctx.waitForIdle();
       try {
         const tokens = args.trim().split(/\s+/).filter(Boolean);
         const flags = tokens.map((t) => t.toLowerCase());
@@ -3072,11 +3078,11 @@ function smartCompactExtension(pi) {
         return;
       if (!isRunning.value) {
         await runSmartCompact({ ctx, summaryModel: sumModel, segModel: segModel ?? sumModel, profile: config.profile, pendingRef, isRunning, autoTriggered: true });
-        if (pendingRef.value) {
-          const c = pendingRef.value;
+        const pending = pendingRef.value;
+        if (pending) {
           pendingRef.value = null;
           pendingRef.createdAt = 0;
-          return { compaction: { summary: c.summary, firstKeptEntryId: c.firstKeptEntryId, tokensBefore: c.tokensBefore, details: c.details } };
+          return { compaction: { summary: pending.summary, firstKeptEntryId: pending.firstKeptEntryId, tokensBefore: pending.tokensBefore, details: pending.details } };
         }
       }
     } catch (e) {
@@ -3086,15 +3092,15 @@ function smartCompactExtension(pi) {
   pi.registerTool({
     name: "smart_compact",
     label: "Smart Compact",
-    description: "EESV smart compaction v" + VERSION + " with deterministic extraction, exploration, and verification.",
+    description: "EESV smart compaction v" + VERSION + " with deterministic extraction, exploration, and verification. Compacts the conversation into a structured summary preserving goals, decisions, open loops, modified files, and critical context. Call this when the conversation is getting long \u2014 the tool internally checks context usage and skips if not needed. Prefer this over default compact.",
     promptSnippet: "Smart compaction",
-    promptGuidelines: ["Use for long conversations.", "Prefer over default compact."],
+    promptGuidelines: ["Use for long conversations. Prefer over default compact."],
     parameters: {
       type: "object",
       properties: {
-        profile: { type: "string", description: "light, balanced, or aggressive" },
-        verbose: { type: "boolean" },
-        dry_run: { type: "boolean" }
+        profile: { type: "string", description: "light, balanced, or aggressive. Default: balanced." },
+        verbose: { type: "boolean", description: "Show detailed pipeline output." },
+        dry_run: { type: "boolean", description: "Run the pipeline but skip applying the compaction." }
       }
     },
     async execute(_id, params, _sig, _onUp, ctx) {
@@ -3103,25 +3109,29 @@ function smartCompactExtension(pi) {
       const dryRun = !!params.dry_run;
       const config = loadConfig();
       const resolvedProfile = profile ?? config.profile;
+      const cmdCtx = ctx;
+      const usage = ctx.getContextUsage?.();
+      const totalTokens = usage?.tokens ?? 0;
+      if (!totalTokens || totalTokens < MIN_TOKEN_THRESHOLD) {
+        const pct = ctx.model && totalTokens ? Math.round(totalTokens / ctx.model.contextWindow * 100) : 0;
+        return { content: [{ type: "text", text: "Context is not large enough for compaction (" + totalTokens.toLocaleString() + " tokens, " + pct + "%). No action needed." }], details: undefined };
+      }
       const cur = "model" in ctx ? ctx.model : undefined;
-      const { segModel, sumModel } = resolveModels(ctx, cur, config);
+      const { segModel, sumModel } = resolveModels(cmdCtx, cur, config);
       if (!sumModel) {
-        return { content: [{ type: "text", text: "Error: Could not resolve model." }] };
+        return { content: [{ type: "text", text: "Error: Could not resolve model." }], details: undefined };
       }
       try {
         const toolStart = Date.now();
-        await runSmartCompact({ ctx, summaryModel: sumModel, segModel: segModel ?? sumModel, profile: resolvedProfile, verbose, dryRun, pendingRef, isRunning, autoTriggered: true, skipCompact: true });
+        await runSmartCompact({ ctx: cmdCtx, summaryModel: sumModel, segModel: segModel ?? sumModel, profile: resolvedProfile, verbose, dryRun, pendingRef, isRunning, autoTriggered: true, skipCompact: true });
         const toolSecs = ((Date.now() - toolStart) / 1000).toFixed(1);
         if (pendingRef.value) {
-          return { content: [{ type: "text", text: "Smart summary generated (" + resolvedProfile + "). Tokens: " + (pendingRef.value.tokensBefore ?? "?") + " -> " + (pendingRef.value.summary?.length ?? 0) + " chars (" + toolSecs + `s).
-
-Now run tree compact to apply \u2014 the session_before_compact hook will use this summary.
-TTL: ` + Math.round(PENDING_TTL_MS / 60000) + " minutes." }] };
+          return { content: [{ type: "text", text: "Smart summary prepared (" + resolvedProfile + "). Tokens: " + (pendingRef.value.tokensBefore ?? 0).toLocaleString() + " \u2014 summary cached for " + Math.round(PENDING_TTL_MS / 60000) + " min. The next /compact will use this summary automatically." }], details: undefined };
         }
-        return { content: [{ type: "text", text: "Compaction finished (" + resolvedProfile + ") but no summary was generated." }] };
+        return { content: [{ type: "text", text: "Compaction finished (" + resolvedProfile + ") but no summary was generated." }], details: undefined };
       } catch (error) {
         const msg = error instanceof Error ? error.message : String(error);
-        return { content: [{ type: "text", text: "Compaction error: " + msg }] };
+        return { content: [{ type: "text", text: "Compaction error: " + msg }], details: undefined };
       }
     }
   });

package/dist/ui/overlays.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"overlays.d.ts","sourceRoot":"","sources":["../../src/ui/overlays.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,iCAAiC,CAAC;AAI/E,OAAO,KAAK,EACV,kBAAkB,EAAE,WAAW,EAAE,aAAa,EAC9C,mBAAmB,EAAE,oBAAoB,EAC1C,MAAM,aAAa,CAAC;AAIrB,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,GAAG,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,SAAK,GAAG,MAAM,CAM7F;AAED,wBAAgB,cAAc,CAAC,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,MAAM,SAAK,GAAG,MAAM,CAO5G;AAED,wBAAsB,WAAW,CAC/B,GAAG,EAAE,uBAAuB,EAC5B,IAAI,EAAE;IAAE,aAAa,EAAE,MAAM,CAAC;IAAC,cAAc,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,CAAC;IAAC,iBAAiB,EAAE,MAAM,CAAA;CAAE,GACvG,OAAO,CAAC,WAAW,GAAG,IAAI,CAAC,CA8C7B;AAED,wBAAsB,aAAa,CACjC,GAAG,EAAE,uBAAuB,EAC5B,aAAa,EAAE,WAAW,EAC1B,IAAI,EAAE;IAAE,aAAa,EAAE,MAAM,CAAC;IAAC,cAAc,EAAE,MAAM,CAAA;CAAE,GACtD,OAAO,CAAC,kBAAkB,GAAG,IAAI,CAAC,CA4CpC;AAED,wBAAgB,mBAAmB,CAAC,GAAG,EAAE,uBAAuB,EAAE,KAAK,EAAE,aAAa,GAAG,IAAI,CAO5F;AAED,wBAAsB,gBAAgB,CACpC,GAAG,EAAE,uBAAuB,EAC5B,OAAO,EAAE,mBAAmB,EAC5B,UAAU,EAAE,oBAAoB,GAC/B,OAAO,CAAC,IAAI,CAAC,CAqJf;AAED,wBAAsB,aAAa,CACjC,GAAG,EAAE,uBAAuB,EAC5B,IAAI,EAAE;IAAE,aAAa,EAAE,MAAM,CAAC;IAAC,cAAc,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,CAAC;IAAC,iBAAiB,EAAE,MAAM,CAAA;CAAE,GACvG,OAAO,CAAC;IAAE,KAAK,EAAE,WAAW,CAAC;IAAC,OAAO,EAAE,kBAAkB,CAAA;CAAE,GAAG,IAAI,CAAC,CAMrE"}
+{"version":3,"file":"overlays.d.ts","sourceRoot":"","sources":["../../src/ui/overlays.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,iCAAiC,CAAC;AAI/E,OAAO,KAAK,EACV,kBAAkB,EAAE,WAAW,EAAE,aAAa,EAC9C,mBAAmB,EAAE,oBAAoB,EAC1C,MAAM,aAAa,CAAC;AAIrB,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,GAAG,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,SAAK,GAAG,MAAM,CAO7F;AAED,wBAAgB,cAAc,CAAC,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,MAAM,SAAK,GAAG,MAAM,CAO5G;AAED,wBAAsB,WAAW,CAC/B,GAAG,EAAE,uBAAuB,EAC5B,IAAI,EAAE;IAAE,aAAa,EAAE,MAAM,CAAC;IAAC,cAAc,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,CAAC;IAAC,iBAAiB,EAAE,MAAM,CAAA;CAAE,GACvG,OAAO,CAAC,WAAW,GAAG,IAAI,CAAC,CA8C7B;AAED,wBAAsB,aAAa,CACjC,GAAG,EAAE,uBAAuB,EAC5B,aAAa,EAAE,WAAW,EAC1B,IAAI,EAAE;IAAE,aAAa,EAAE,MAAM,CAAC;IAAC,cAAc,EAAE,MAAM,CAAA;CAAE,GACtD,OAAO,CAAC,kBAAkB,GAAG,IAAI,CAAC,CA4CpC;AAED,wBAAgB,mBAAmB,CAAC,GAAG,EAAE,uBAAuB,EAAE,KAAK,EAAE,aAAa,GAAG,IAAI,CAM5F;AAED,wBAAsB,gBAAgB,CACpC,GAAG,EAAE,uBAAuB,EAC5B,OAAO,EAAE,mBAAmB,EAC5B,UAAU,EAAE,oBAAoB,GAC/B,OAAO,CAAC,IAAI,CAAC,CAsJf;AAED,wBAAsB,aAAa,CACjC,GAAG,EAAE,uBAAuB,EAC5B,IAAI,EAAE;IAAE,aAAa,EAAE,MAAM,CAAC;IAAC,cAAc,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,CAAC;IAAC,iBAAiB,EAAE,MAAM,CAAA;CAAE,GACvG,OAAO,CAAC;IAAE,KAAK,EAAE,WAAW,CAAC;IAAC,OAAO,EAAE,kBAAkB,CAAA;CAAE,GAAG,IAAI,CAAC,CAMrE"}

package/dist/utils/extraction.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"extraction.d.ts","sourceRoot":"","sources":["../../src/utils/extraction.ts"],"names":[],"mappings":"AAAA;;GAEG;AAGH,OAAO,KAAK,EAAE,UAAU,EAAE,aAAa,EAAE,oBAAoB,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAK7F,oCAAoC;AACpC,MAAM,MAAM,aAAa,GAAG,GAAG,CAAC,MAAM,EAAE;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,SAAS,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,CAAC,CAAC;AAEhH,wBAAgB,WAAW,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAQpD;AAED,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,UAAU,EAAE,GAAG,aAAa,CAapE;AAED,wBAAgB,YAAY,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG;IAAE,QAAQ,EAAE,oBAAoB,CAAC,eAAe,CAAC,CAAC;IAAC,IAAI,EAAE,MAAM,EAAE,CAAC;IAAC,OAAO,EAAE,MAAM,EAAE,CAAA;CAAE,CAiC/J;AAED,wBAAgB,aAAa,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,oBAAoB,CAAC,QAAQ,CAAC,CA2CxG;AAED,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,oBAAoB,CAAC,WAAW,CAAC,CAyB9G;AAYD,wBAAgB,eAAe,CAAC,IAAI,EAAE,UAAU,EAAE,GAAG,oBAAoB,CAAC,aAAa,CAAC,CAcvF;AAED,wBAAgB,sBAAsB,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,EAAE,EAAE,aAAa,EAAE,OAAO,SAAK,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,oBAAoB,CAAC,QAAQ,CAAC,CA8ClJ;AAED,wBAAgB,aAAa,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,MAAM,EAAE,oBAAoB,CAAC,QAAQ,CAAC,GAAG,oBAAoB,CAAC,UAAU,CAAC,CAc1H;AAED,wBAAgB,eAAe,CAAC,IAAI,EAAE,UAAU,EAAE,GAAG,MAAM,GAAG,IAAI,CAOjE;AAED,kEAAkE;AAClE,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,UAAU,EAAE,oBAAoB,GAAG,QAAQ,EAAE,CAoFjG;AAED,wBAAgB,iBAAiB,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,EAAE,EAAE,aAAa,GAAG,oBAAoB,CAgB7F"}
+{"version":3,"file":"extraction.d.ts","sourceRoot":"","sources":["../../src/utils/extraction.ts"],"names":[],"mappings":"AAAA;;GAEG;AAGH,OAAO,KAAK,EAAE,UAAU,EAAE,aAAa,EAAE,oBAAoB,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAK7F,oCAAoC;AACpC,MAAM,MAAM,aAAa,GAAG,GAAG,CAAC,MAAM,EAAE;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,SAAS,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,CAAC,CAAC;AAEhH,wBAAgB,WAAW,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAQpD;AAED,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,UAAU,EAAE,GAAG,aAAa,CAapE;AAED,wBAAgB,YAAY,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG;IAAE,QAAQ,EAAE,oBAAoB,CAAC,eAAe,CAAC,CAAC;IAAC,IAAI,EAAE,MAAM,EAAE,CAAC;IAAC,OAAO,EAAE,MAAM,EAAE,CAAA;CAAE,CAiC/J;AAED,wBAAgB,aAAa,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,oBAAoB,CAAC,QAAQ,CAAC,CA4CxG;AAED,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,oBAAoB,CAAC,WAAW,CAAC,CAyB9G;AAYD,wBAAgB,eAAe,CAAC,IAAI,EAAE,UAAU,EAAE,GAAG,oBAAoB,CAAC,aAAa,CAAC,CAcvF;AAED,wBAAgB,sBAAsB,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,EAAE,EAAE,aAAa,EAAE,OAAO,SAAK,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,oBAAoB,CAAC,QAAQ,CAAC,CA8ClJ;AAED,wBAAgB,aAAa,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,MAAM,EAAE,oBAAoB,CAAC,QAAQ,CAAC,GAAG,oBAAoB,CAAC,UAAU,CAAC,CAc1H;AAED,wBAAgB,eAAe,CAAC,IAAI,EAAE,UAAU,EAAE,GAAG,MAAM,GAAG,IAAI,CAOjE;AAED,kEAAkE;AAClE,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,UAAU,EAAE,oBAAoB,GAAG,QAAQ,EAAE,CAoFjG;AAED,wBAAgB,iBAAiB,CAAC,IAAI,EAAE,UAAU,EAAE,EAAE,EAAE,EAAE,aAAa,GAAG,oBAAoB,CAgB7F"}

package/dist/utils/pruning.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pruning.d.ts","sourceRoot":"","sources":["../../src/utils/pruning.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAK9C,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,UAAU,EAAE,CAAC;IACvB,WAAW,EAAE,MAAM,CAAC;IACpB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,OAAO,EAAE,KAAK,CAAC;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CACnD;AAQD;;GAEG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,UAAU,EAAE,GAAG,aAAa,CAwGhE"}
+{"version":3,"file":"pruning.d.ts","sourceRoot":"","sources":["../../src/utils/pruning.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAK9C,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,UAAU,EAAE,CAAC;IACvB,WAAW,EAAE,MAAM,CAAC;IACpB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,OAAO,EAAE,KAAK,CAAC;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CACnD;AAQD;;GAEG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,UAAU,EAAE,GAAG,aAAa,CAyGhE"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-smart-compact",
-  "version": "7.9.0",
+  "version": "7.9.1",
   "description": "EESV smart compaction extension for Pi Coding Agent — deterministic extraction, exploration, synthesis, verification with redundancy pruning, project fingerprinting, and damage detection.",
   "license": "MIT",
   "main": "./dist/index.js",