@aexol/spectral 0.7.7 → 0.8.0

package/dist/relay/auto-research.js

@@ -1,6 +1,6 @@
 /**
  * Auto-research handler — sends an auto-research task through the existing
- * PiBridge (backend proxy) instead of spawning a separate pi process.
+ * AgentBridge (backend proxy) instead of spawning a separate pi process.
  *
  * This ensures auto-research uses the same model and API keys as the active
  * session — no separate subprocess, no missing API key errors.
@@ -47,12 +47,12 @@ function makeRelaySubscriber(sessionId, relay) {
     };
 }
 /**
- * Scan the project's .pi/extensions/auto-research/ directory for generated
+ * Scan the project's .spectral/extensions/auto-research/ directory for generated
  * extension directories. Each subdirectory with .ts files counts as one
  * extension.
  */
 function scanGeneratedExtensions(projectPath) {
-    const arDir = path.join(projectPath, ".pi", "extensions", "auto-research");
+    const arDir = path.join(projectPath, ".spectral", "extensions", "auto-research");
     const extensions = [];
     try {
         if (!fs.existsSync(arDir))
@@ -105,7 +105,7 @@ function scanGeneratedExtensions(projectPath) {
             }
             extensions.push({
                 name: entry.name,
-                path: `.pi/extensions/auto-research/${entry.name}`,
+                path: `.spectral/extensions/auto-research/${entry.name}`,
                 description,
                 usesLLM,
                 fileCount,
@@ -137,7 +137,7 @@ function hasAgentsMdUpdate(projectPath) {
  */
 function readManifest(projectPath) {
     try {
-        const mPath = path.join(projectPath, ".pi", "extensions", "auto-research", "manifest.json");
+        const mPath = path.join(projectPath, ".spectral", "extensions", "auto-research", "manifest.json");
         if (!fs.existsSync(mPath))
             return null;
         const raw = fs.readFileSync(mPath, "utf-8");
@@ -164,7 +164,7 @@ function gatherPreRunContext(projectPath) {
     const isIncremental = manifest !== null;
     const existingExtensions = [];
     try {
-        const arDir = path.join(projectPath, ".pi", "extensions", "auto-research");
+        const arDir = path.join(projectPath, ".spectral", "extensions", "auto-research");
         if (fs.existsSync(arDir)) {
             for (const entry of fs.readdirSync(arDir, { withFileTypes: true })) {
                 if (entry.isDirectory() && entry.name !== "node_modules") {
@@ -209,7 +209,7 @@ function buildIncrementalSection(ctx) {
     if (ctx.existingExtensions.length > 0) {
         lines.push("### Existing extensions to review:");
         for (const name of ctx.existingExtensions) {
-            lines.push(`  - \`.pi/extensions/auto-research/${name}/\``);
+            lines.push(`  - \`.spectral/extensions/auto-research/${name}/\``);
         }
         lines.push("");
     }
@@ -224,7 +224,7 @@ function buildIncrementalSection(ctx) {
     lines.push("   was removed from the project, delete the extension directory entirely.", "");
     lines.push("4. **Update AGENTS.md** — The AUTO-RESEARCH section should reflect the CURRENT");
     lines.push("   set of extensions (remove stale entries, add new ones).", "");
-    lines.push("5. **Save manifest.json** — Write/update .pi/extensions/auto-research/manifest.json");
+    lines.push("5. **Save manifest.json** — Write/update .spectral/extensions/auto-research/manifest.json");
     lines.push(`   with: lastRun (ISO), lastCommit (git HEAD), runCount (${ctx.manifest ? ctx.manifest.runCount + 1 : 1}),`);
     lines.push("   and extensions array with name/path/category for each generated extension.", "");
     return lines.join("\n");
@@ -234,7 +234,7 @@ function buildIncrementalSection(ctx) {
  */
 function writeManifest(projectPath, extensions) {
     try {
-        const arDir = path.join(projectPath, ".pi", "extensions", "auto-research");
+        const arDir = path.join(projectPath, ".spectral", "extensions", "auto-research");
         if (!fs.existsSync(arDir))
             fs.mkdirSync(arDir, { recursive: true });
         let currentCommit = "unknown";
@@ -257,7 +257,7 @@ function writeManifest(projectPath, extensions) {
 }
 /**
  * Build the auto-research task prompt. This is sent as a user message
- * through the existing PiBridge, so the agent uses the session's model
+ * through the existing AgentBridge, so the agent uses the session's model
  * and backend proxy.
  *
  * @param projectPath Absolute path to the project root
@@ -297,8 +297,8 @@ function buildAutoResearchTask(projectPath, projectName, preRunContext = null) {
         "}",
         "```",
         "",
-        "Extensions live in: `.pi/extensions/` (project-local) or `~/.pi/agent/extensions/` (user-global).",
-        "Auto-research generates extensions into `.pi/extensions/auto-research/<name>/`.",
+        "Extensions live in: `.spectral/extensions/` (project-local) or `~/.spectral/agent/extensions/` (user-global).",
+        "Auto-research generates extensions into `.spectral/extensions/auto-research/<name>/`.",
         "",
         "### Tools (pi.registerTool)",
         "",
@@ -400,10 +400,10 @@ function buildAutoResearchTask(projectPath, projectName, preRunContext = null) {
         "",
         "### Extension File Structure",
         "",
-        "Each extension is a directory under `.pi/extensions/auto-research/`:",
+        "Each extension is a directory under `.spectral/extensions/auto-research/`:",
         "",
         "```",
-        ".pi/extensions/auto-research/",
+        ".spectral/extensions/auto-research/",
         "  <extension-name>/",
         "    index.ts          # Entry point — default export activate(pi)",
         "    utils.ts          # [optional] Helper functions",
@@ -440,7 +440,7 @@ function buildAutoResearchTask(projectPath, projectName, preRunContext = null) {
         "",
         "1. **Context Collection** — Explore the project structure:",
         "   - Read package.json, tsconfig.json, deno.json (if present)",
-        "   - Check existing extensions under .pi/extensions/",
+        "   - Check existing extensions under .spectral/extensions/",
         "   - Review key source files to understand architecture",
         "   - Check git log for recent changes and patterns",
         "",
@@ -454,9 +454,9 @@ function buildAutoResearchTask(projectPath, projectName, preRunContext = null) {
         "   - Use `pi.registerTool()` for simple tools with TypeBox validation",
         "   - Use `pi.registerCommand()` for custom slash commands",
         "   - For LLM-powered extensions, use `ctx.modelRegistry` to call models",
-        "   - Create files under `.pi/extensions/auto-research/<name>/`",
+        "   - Create files under `.spectral/extensions/auto-research/<name>/`",
         "   - Each extension needs an `index.ts` that registers its tools/commands",
-        "   - Read `.pi/agents/auto-research-templates.md` for proven extension templates",
+        "   - Read `.spectral/agents/auto-research-templates.md` for proven extension templates",
         "",
         "4. **Validation** — Verify generated extensions:",
         "   - Ensure all imports resolve to available packages (see reference above)",
@@ -478,7 +478,7 @@ function buildAutoResearchTask(projectPath, projectName, preRunContext = null) {
         "",
         "### Important Rules",
         "",
-        "- Write each extension as TypeScript files under `.pi/extensions/auto-research/<name>/`",
+        "- Write each extension as TypeScript files under `.spectral/extensions/auto-research/<name>/`",
         "- Every extension directory MUST have an `index.ts` entry point",
         "- Use proper TypeScript with type annotations and error handling",
         "- Extensions must handle errors gracefully (never crash the agent)",
@@ -491,7 +491,7 @@ function buildAutoResearchTask(projectPath, projectName, preRunContext = null) {
         "agent sessions automatically know about the new capabilities.",
         "",
         "**How AGENTS.md works:**",
-        "- Pi loads AGENTS.md at startup from the project root and parent directories",
+        "- Spectral loads AGENTS.md at startup from the project root and parent directories",
         "- All found AGENTS.md files are concatenated and injected into the system prompt",
         "- This means documented extensions get discovered by the agent automatically",
         "",
@@ -502,7 +502,7 @@ function buildAutoResearchTask(projectPath, projectName, preRunContext = null) {
         "## Auto-Generated Extensions",
         "",
         "These extensions were generated by auto-research. They are available",
-        "in every session. Pi loads them automatically from `.pi/extensions/`.",
+        "in every session. Spectral loads them automatically from `.spectral/extensions/`.",
         "",
         "### `<extension-name>`",
         "",
@@ -539,7 +539,7 @@ function buildAutoResearchTask(projectPath, projectName, preRunContext = null) {
 // ---------------------------------------------------------------------------
 const AR_TIMEOUT_MS = 10 * 60 * 1000; // 10 minutes — generous for LLM-based analysis
 /**
- * Execute auto-research for a project through the existing PiBridge.
+ * Execute auto-research for a project through the existing AgentBridge.
  *
  * Caller (dispatcher) is fire-and-forget — this function is `void` and
  * all errors are surfaced as `auto_research_error` events on the wire
@@ -547,7 +547,7 @@ const AR_TIMEOUT_MS = 10 * 60 * 1000; // 10 minutes — generous for LLM-based a
  *
  * This replaces the old subprocess-spawning approach. Instead of launching
  * a separate pi process (which lacks backend proxy credentials), we send
- * the auto-research task through the session's existing PiBridge. The agent
+ * the auto-research task through the session's existing AgentBridge. The agent
  * uses the session's model, backend proxy, and all available tools.
  */
 export function handleAutoResearch(input, deps) {
@@ -700,7 +700,7 @@ export function handleAutoResearch(input, deps) {
     // then timeout fires and calls finalize() again — but finalize() is
     // gated by watcherFired, so it's a no-op. The only issue is we don't
     // clearTimeout on watcher fire — but that's fine, the timer is harmless.
-    // --- Send the prompt through the existing PiBridge (backend proxy) ---
+    // --- Send the prompt through the existing AgentBridge (backend proxy) ---
     manager.prompt(sessionId, taskContent, storedModelId).catch((err) => {
         if (watcherFired)
             return; // already handled by watcher

package/dist/relay/dispatcher.js

@@ -41,7 +41,7 @@
 import { BadRequestError, NotFoundError } from "../server/handlers/errors.js";
 import { handlePathAutocomplete } from "../server/handlers/paths-autocomplete.js";
 import { handleBindStudioProject, handleCreateProject, handleDeleteProject, handleListProjects, handleListSessionsByProject, handleUpdateProject, } from "../server/handlers/projects.js";
-import { handleCompactSession, handleCreateSession, handleDeleteSession, handleForkSession, handleGetSessionDetail, handleGetSessionMemoryDetails, handleGetSessionMemoryStatus, handleUpdateSession, } from "../server/handlers/sessions.js";
+import { handleCompactSession, handleCreateSession, handleDeleteSession, handleForkSession, handleRememberAndDeleteSession, handleGetSessionDetail, handleGetSessionMemoryDetails, handleGetSessionMemoryStatus, handleUpdateSession, } from "../server/handlers/sessions.js";
 import { handleClearPromptQueue, handleEnqueuePrompt, handleGetPromptQueue, handleRemovePrompt, } from "../server/handlers/queue.js";
 import { shutdownState } from "../server/shutdown.js";
 import { handleAutoResearch } from "./auto-research.js";
@@ -126,6 +126,14 @@ export function matchRoute(method, path) {
             return { route: "compact_session", id };
         return null;
     }
+    // /api/sessions/:id/remember-and-delete
+    const rememberDeleteMatch = /^\/api\/sessions\/([^/]+)\/remember-and-delete$/.exec(cleanPath);
+    if (rememberDeleteMatch) {
+        const id = decodeURIComponent(rememberDeleteMatch[1]);
+        if (method === "POST")
+            return { route: "remember_and_delete_session", id };
+        return null;
+    }
     // /api/sessions/:id/fork
     const forkMatch = /^\/api\/sessions\/([^/]+)\/fork$/.exec(cleanPath);
     if (forkMatch) {
@@ -341,6 +349,24 @@ async function dispatchRoute(match, body, deps) {
         }
         case "compact_session":
             return await handleCompactSession(store, manager, id);
+        case "remember_and_delete_session": {
+            // Compact first — the compaction hook's persistProjectObservations
+            // runs inside compactSession, writing reflections to cross-session
+            // durable memory. On failure (e.g. no API key), we let the error
+            // propagate so the session is NOT deleted.
+            await handleRememberAndDeleteSession(store, manager, id);
+            const detail = store.getSession(id);
+            manager.disposeSessionStream(id);
+            handleDeleteSession(store, id);
+            if (detail) {
+                safePublish(publishMetaEvent, logger, {
+                    type: "session_deleted",
+                    projectId: detail.projectId,
+                    sessionId: id,
+                });
+            }
+            return { ok: true };
+        }
         case "fork_session": {
             const session = handleForkSession(store, id, asObject(body));
             safePublish(publishMetaEvent, logger, {
@@ -682,7 +708,7 @@ export function detachAllSubscribers(manager, subscribers) {
 }
 /**
  * Dispatch an `auto_research` frame. Sends the auto-research task through
- * the existing PiBridge (backend proxy) instead of spawning a separate pi
+ * the existing AgentBridge (backend proxy) instead of spawning a separate pi
  * subprocess. This ensures auto-research uses the same model and API keys
  * as the active session.
  *

package/dist/relay/models-fetch.js

@@ -1,7 +1,7 @@
 /**
  * Fetch the admin-managed list of allowed base models from the backend.
  *
- * Used by `PiBridge` at startup to register synthetic providers
+ * Used by `AgentBridge` at startup to register synthetic providers
  * (`spectral-proxy-anthropic` / `spectral-proxy-openai`) that route every
  * inference call through the backend's `/v1/messages` and
  * `/v1/chat/completions` endpoints. The backend authenticates the call
@@ -28,11 +28,11 @@ const cache = new Map();
 export function clearAllowedModelsCache() {
     cache.clear();
 }
-const QUERY = `query AvailableBaseModels { availableBaseModels { name provider userModelId agentEnabled creditInputPer1M creditOutputPer1M creditCachedInputPer1M creditCacheReadPer1M creditCacheWritePer1M contextWindow supportsImages } }`;
+const QUERY = `query AvailableBaseModels { availableBaseModels { name provider userModelId agentEnabled creditInputPer1M creditOutputPer1M creditCachedInputPer1M creditCacheReadPer1M creditCacheWritePer1M contextWindow supportsImages supportsReasoning isDefault isVisionDefault } }`;
 /**
  * Fetch the whitelist of allowed base models. Throws on any failure with a
  * message tailored for an operator running `spectral serve` — the caller
- * (PiBridge.start) lets the throw propagate so the WS subscriber sees a
+ * (AgentBridge.start) lets the throw propagate so the WS subscriber sees a
  * clear error event instead of a silent fall-through to "no models".
  */
 export async function fetchAllowedModels(opts) {
@@ -104,6 +104,15 @@ export async function fetchAllowedModels(opts) {
         const supportsImages = typeof row?.supportsImages === "boolean"
             ? row.supportsImages
             : null;
+        const supportsReasoning = typeof row?.supportsReasoning === "boolean"
+            ? row.supportsReasoning
+            : null;
+        const isDefault = typeof row?.isDefault === "boolean"
+            ? row.isDefault
+            : null;
+        const isVisionDefault = typeof row?.isVisionDefault === "boolean"
+            ? row.isVisionDefault
+            : null;
         const model = {
             modelId: name,
             displayName: name,
@@ -115,6 +124,9 @@ export async function fetchAllowedModels(opts) {
             creditCacheWritePer1M: asOptionalNumber(row?.creditCacheWritePer1M),
             contextWindow,
             supportsImages,
+            supportsReasoning,
+            isDefault,
+            isVisionDefault,
         };
         if (typeof row?.userModelId === "string") {
             model.userModelId = row.userModelId;

package/dist/{pi → sdk}/coding-agent/cli/args.js

@@ -232,7 +232,7 @@ ${chalk.bold("Options:")}
   --export <file>                Export session file to HTML and exit
   --list-models [search]         List available models (with optional fuzzy search)
   --verbose                      Force verbose startup (overrides quietStartup setting)
-  --offline                      Disable startup network operations (same as PI_OFFLINE=1)
+  --offline                      Disable startup network operations (same as SPECTRAL_OFFLINE=1)
   --help, -h                     Show this help
   --version, -v                  Show version number
 
@@ -324,9 +324,9 @@ ${chalk.bold("Environment Variables:")}
   ${ENV_AGENT_DIR.padEnd(32)} - Config directory (default: ~/${CONFIG_DIR_NAME}/agent)
   ${ENV_SESSION_DIR.padEnd(32)} - Session storage directory (overridden by --session-dir)
   PI_PACKAGE_DIR                   - Override package directory (for Nix/Guix store paths)
-  PI_OFFLINE                       - Disable startup network operations when set to 1/true/yes
-  PI_TELEMETRY                     - Override install telemetry when set to 1/true/yes or 0/false/no
-  PI_SHARE_VIEWER_URL              - Base URL for /share command (default: https://pi.dev/session/)
+  SPECTRAL_OFFLINE                       - Disable startup network operations when set to 1/true/yes
+  SPECTRAL_TELEMETRY                     - Override install telemetry when set to 1/true/yes or 0/false/no
+  SPECTRAL_SHARE_VIEWER_URL              - Base URL for /share command (default: https://spectral.dev/session/)
 
 ${chalk.bold("Built-in Tool Names:")}
   read   - Read file contents

package/dist/{pi → sdk}/coding-agent/config.js

@@ -297,17 +297,6 @@ export function getThemesDir() {
 /**
  * Get path to HTML export template directory (shipped with package)
  * - For Bun binary: export-html/ next to executable
- * - For Node.js (dist/): dist/core/export-html/
- * - For tsx (src/): src/core/export-html/
- */
-export function getExportTemplateDir() {
-    if (isBunBinary) {
-        return join(getPackageDir(), "export-html");
-    }
-    const packageDir = getPackageDir();
-    const srcOrDist = existsSync(join(packageDir, "src")) ? "src" : "dist";
-    return join(packageDir, srcOrDist, "core", "export-html");
-}
 /** Get path to package.json */
 export function getPackageJsonPath() {
     return join(getPackageDir(), "package.json");
@@ -347,28 +336,28 @@ export function getBundledInteractiveAssetPath(name) {
     return join(getInteractiveAssetsDir(), name);
 }
 const pkg = JSON.parse(readFileSync(getPackageJsonPath(), "utf-8"));
-const piConfigName = pkg.piConfig?.name;
+const spectralConfigName = pkg.spectralConfig?.name;
 export const PACKAGE_NAME = pkg.name || "index.ts";
-export const APP_NAME = piConfigName || "pi";
-export const APP_TITLE = piConfigName ? APP_NAME : "π";
-export const CONFIG_DIR_NAME = pkg.piConfig?.configDir || ".pi";
+export const APP_NAME = spectralConfigName || "spectral";
+export const APP_TITLE = spectralConfigName ? APP_NAME : "spectral";
+export const CONFIG_DIR_NAME = pkg.spectralConfig?.configDir || ".spectral";
 export const VERSION = pkg.version || "0.0.0";
-// e.g., PI_CODING_AGENT_DIR or TAU_CODING_AGENT_DIR
+// e.g., SPECTRAL_CODING_AGENT_DIR
 export const ENV_AGENT_DIR = `${APP_NAME.toUpperCase()}_CODING_AGENT_DIR`;
 export const ENV_SESSION_DIR = `${APP_NAME.toUpperCase()}_CODING_AGENT_SESSION_DIR`;
 export function expandTildePath(path) {
     return normalizePath(path);
 }
-const DEFAULT_SHARE_VIEWER_URL = "https://pi.dev/session/";
+const DEFAULT_SHARE_VIEWER_URL = "https://spectral.dev/session/";
 /** Get the share viewer URL for a gist ID */
 export function getShareViewerUrl(gistId) {
-    const baseUrl = process.env.PI_SHARE_VIEWER_URL || DEFAULT_SHARE_VIEWER_URL;
+    const baseUrl = process.env.SPECTRAL_SHARE_VIEWER_URL || DEFAULT_SHARE_VIEWER_URL;
     return `${baseUrl}#${gistId}`;
 }
 // =============================================================================
-// User Config Paths (~/.pi/agent/*)
+// User Config Paths (~/.spectral/agent/*)
 // =============================================================================
-/** Get the agent config directory (e.g., ~/.pi/agent/) */
+/** Get the agent config directory (e.g., ~/.spectral/agent/) */
 export function getAgentDir() {
     const envDir = process.env[ENV_AGENT_DIR];
     if (envDir) {

package/dist/{pi → sdk}/coding-agent/core/agent-session.js

@@ -15,7 +15,6 @@
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { basename, dirname } from "node:path";
 import { clampThinkingLevel, cleanupSessionResources, getSupportedThinkingLevels, isContextOverflow, modelsAreEqual, resetApiProviders, streamSimple, } from "../../ai/index.js";
-import { theme } from "../modes/interactive/theme/theme.js";
 import { stripFrontmatter } from "../utils/frontmatter.js";
 import { resolvePath } from "../utils/paths.js";
 import { sleep } from "../utils/sleep.js";
@@ -23,8 +22,6 @@ import { formatNoApiKeyFoundMessage, formatNoModelSelectedMessage } from "./auth
 import { executeBashWithOperations } from "./bash-executor.js";
 import { calculateContextTokens, collectEntriesForBranchSummary, compact, estimateContextTokens, generateBranchSummary, prepareCompaction, shouldCompact, } from "./compaction/index.js";
 import { DEFAULT_THINKING_LEVEL } from "./defaults.js";
-import { exportSessionToHtml } from "./export-html/index.js";
-import { createToolHtmlRenderer } from "./export-html/tool-renderer.js";
 import { ExtensionRunner, wrapRegisteredTools, } from "./extensions/index.js";
 import { emitSessionShutdownEvent } from "./extensions/runner.js";
 import { expandPromptTemplate } from "./prompt-templates.js";
@@ -468,6 +465,8 @@ export class AgentSession {
      */
     dispose() {
         this._extensionRunner.invalidate("This extension ctx is stale after session replacement or reload. Do not use a captured pi or command ctx after ctx.newSession(), ctx.fork(), ctx.switchSession(), or ctx.reload(). For newSession, fork, and switchSession, move post-replacement work into withSession and use the ctx passed to withSession. For reload, do not use the old ctx after await ctx.reload().");
+        this.abortRetry();
+        this.agent.abort();
         this._disconnectFromAgent();
         this._eventListeners = [];
         cleanupSessionResources(this.sessionId);
@@ -1889,7 +1888,7 @@ export class AgentSession {
                 extensionsResult.runtime.flagValues.set(name, value);
             }
         }
-        this._extensionRunner = new ExtensionRunner(extensionsResult.extensions, extensionsResult.runtime, this._cwd, this.sessionManager, this._modelRegistry);
+        this._extensionRunner = new ExtensionRunner(extensionsResult.extensions, extensionsResult.runtime, this._cwd, this.sessionManager, this._modelRegistry, this.settingsManager);
         if (this._extensionRunnerRef) {
             this._extensionRunnerRef.current = this._extensionRunner;
         }
@@ -2398,19 +2397,8 @@ export class AgentSession {
      * @param outputPath Optional output path (defaults to session directory)
      * @returns Path to exported file
      */
-    async exportToHtml(outputPath) {
-        const themeName = this.settingsManager.getTheme();
-        // Create tool renderer if we have an extension runner (for custom tool HTML rendering)
-        const toolRenderer = createToolHtmlRenderer({
-            getToolDefinition: (name) => this.getToolDefinition(name),
-            theme,
-            cwd: this.sessionManager.getCwd(),
-        });
-        return await exportSessionToHtml(this.sessionManager, this.state, {
-            outputPath,
-            themeName,
-            toolRenderer,
-        });
+    async exportToHtml(_outputPath) {
+        throw new Error("HTML export has been removed. Use spectral serve instead.");
     }
     /**
      * Export the current session branch to a JSONL file.

package/dist/{pi → sdk}/coding-agent/core/compaction/compaction.js

@@ -543,6 +543,157 @@ export function prepareCompaction(pathEntries, settings) {
     };
 }
 // ============================================================================
+// Tool Call Deduplication
+// ============================================================================
+/**
+ * Tools that always return the same result for the same arguments.
+ * Same (name, args) from any point in the session = duplicate.
+ * Only the most recent call is kept.
+ */
+const IDEMPOTENT_READ_TOOLS = new Set(["read"]);
+/**
+ * Tools whose output may differ between calls with the same arguments.
+ * Deduplication requires comparing actual outputs to determine equivalence.
+ */
+const OUTPUT_DEPENDENT_TOOLS = new Set(["bash"]);
+/**
+ * Tools that mutate state. Never deduplicated — chronological ordering matters.
+ */
+const MUTABLE_TOOLS = new Set(["edit", "write"]);
+/**
+ * Build a stable string key from a Record's sorted keys.
+ * Ensures {a:1, b:2} and {b:2, a:1} produce the same key.
+ */
+function stableArgs(args) {
+    const sortedKeys = Object.keys(args).sort();
+    const sorted = {};
+    for (const key of sortedKeys) {
+        sorted[key] = args[key];
+    }
+    return JSON.stringify(sorted);
+}
+/**
+ * Compute the deduplication key for a tool call.
+ * - Idempotent-read tools: keyed by (name, args) — same args always same result
+ * - Output-dependent tools: keyed by (name, args, output) — output comparison needed
+ */
+function toolCallDedupKey(toolName, args, toolCallId, resultMap) {
+    const argsKey = stableArgs(args);
+    if (OUTPUT_DEPENDENT_TOOLS.has(toolName)) {
+        const output = resultMap.get(toolCallId) ?? "";
+        return `${toolName}:${argsKey}:${output}`;
+    }
+    return `${toolName}:${argsKey}`;
+}
+/**
+ * Check whether a tool should be excluded from deduplication.
+ * Mutating tools and unknown tools are never deduplicated (conservative).
+ */
+function isMutableOrUnknownTool(toolName) {
+    if (MUTABLE_TOOLS.has(toolName))
+        return true;
+    if (IDEMPOTENT_READ_TOOLS.has(toolName))
+        return false;
+    if (OUTPUT_DEPENDENT_TOOLS.has(toolName))
+        return false;
+    // Unknown tools: conservative — never deduplicate
+    return true;
+}
+/**
+ * Deduplicate repeated tool calls in a message array by keeping only the
+ * most recent occurrence of each (tool, args) pair.
+ *
+ * Strategy:
+ * - **Idempotent-read tools** (read, grep, glob): same (name, args) → keep last only.
+ *   These tools return the same content for the same arguments.
+ * - **Output-dependent tools** (bash): same (name, args, output) → keep last only.
+ *   Two bash calls with same command but different output are NOT duplicates.
+ * - **Mutating tools** (edit, write): never deduplicated. Chronological ordering
+ *   of mutations matters for correctness.
+ * - **Unknown tools** (extensions, MCP, custom): never deduplicated. Conservative
+ *   by default — only tools in the known sets above participate.
+ *
+ * Deduplication removes both the ToolCall block from the assistant message
+ * and the corresponding ToolResultMessage from the array.
+ *
+ * Recalculated each time compaction runs — prompt cache is only impacted
+ * alongside compression, not on every turn.
+ */
+export function deduplicateToolCalls(messages) {
+    if (messages.length === 0)
+        return messages;
+    // Phase 1: Build result lookup for output-dependent tools
+    const resultMap = new Map();
+    for (const msg of messages) {
+        if (msg.role === "toolResult" && Array.isArray(msg.content)) {
+            const text = msg.content
+                .filter((c) => c.type === "text")
+                .map((c) => c.text)
+                .join("\n");
+            resultMap.set(msg.toolCallId, text);
+        }
+    }
+    // Phase 2: Walk newest→oldest, collect keys. The first encounter
+    // (newest) wins; all earlier tool calls with the same key are duplicates.
+    const seen = new Map(); // key → toolCallId (keep newest)
+    const duplicateIds = new Set();
+    for (let i = messages.length - 1; i >= 0; i--) {
+        const msg = messages[i];
+        if (msg.role !== "assistant")
+            continue;
+        if (!("content" in msg) || !Array.isArray(msg.content))
+            continue;
+        // Iterate blocks newest-first: within a single assistant message,
+        // the rightmost tool call is the "most recent" one.
+        for (let j = msg.content.length - 1; j >= 0; j--) {
+            const block = msg.content[j];
+            if (typeof block !== "object" || block === null)
+                continue;
+            if (!("type" in block) || block.type !== "toolCall")
+                continue;
+            const toolBlock = block;
+            if (isMutableOrUnknownTool(toolBlock.name))
+                continue;
+            const key = toolCallDedupKey(toolBlock.name, toolBlock.arguments, toolBlock.id, resultMap);
+            if (seen.has(key)) {
+                duplicateIds.add(toolBlock.id);
+            }
+            else {
+                seen.set(key, toolBlock.id);
+            }
+        }
+    }
+    if (duplicateIds.size === 0)
+        return messages;
+    // Phase 3: Filter out duplicate tool results and strip duplicate
+    // ToolCall blocks from assistant messages.
+    const deduped = [];
+    let modified = false;
+    for (const msg of messages) {
+        if (msg.role === "toolResult" && duplicateIds.has(msg.toolCallId)) {
+            modified = true;
+            continue;
+        }
+        if (msg.role === "assistant" && "content" in msg && Array.isArray(msg.content)) {
+            const originalLength = msg.content.length;
+            const filteredContent = msg.content.filter((block) => {
+                if (typeof block !== "object" || block === null)
+                    return true;
+                if (!("type" in block) || block.type !== "toolCall")
+                    return true;
+                return !duplicateIds.has(block.id);
+            });
+            if (filteredContent.length < originalLength) {
+                modified = true;
+                deduped.push({ ...msg, content: filteredContent });
+                continue;
+            }
+        }
+        deduped.push(msg);
+    }
+    return modified ? deduped : messages;
+}
+// ============================================================================
 // Main compaction function
 // ============================================================================
 const TURN_PREFIX_SUMMARIZATION_PROMPT = `This is the PREFIX of a turn that was too large to keep. The SUFFIX (recent work) is retained.
@@ -568,22 +719,27 @@ Be concise. Focus on what's needed to understand the kept suffix.`;
  */
 export async function compact(preparation, model, apiKey, headers, customInstructions, signal, thinkingLevel, streamFn) {
     const { firstKeptEntryId, messagesToSummarize, turnPrefixMessages, isSplitTurn, tokensBefore, previousSummary, fileOps, settings, } = preparation;
+    // Remove redundant tool calls before summarization.
+    // Deduplication is applied here (not in prepareCompaction) so that
+    // the session_before_compact extension hook receives raw messages.
+    const dedupedMessages = deduplicateToolCalls(messagesToSummarize);
+    const dedupedTurnPrefix = deduplicateToolCalls(turnPrefixMessages);
     // Generate summaries (can be parallel if both needed) and merge into one
     let summary;
-    if (isSplitTurn && turnPrefixMessages.length > 0) {
+    if (isSplitTurn && dedupedTurnPrefix.length > 0) {
         // Generate both summaries in parallel
         const [historyResult, turnPrefixResult] = await Promise.all([
-            messagesToSummarize.length > 0
-                ? generateSummary(messagesToSummarize, model, settings.reserveTokens, apiKey, headers, signal, customInstructions, previousSummary, thinkingLevel, streamFn)
+            dedupedMessages.length > 0
+                ? generateSummary(dedupedMessages, model, settings.reserveTokens, apiKey, headers, signal, customInstructions, previousSummary, thinkingLevel, streamFn)
                 : Promise.resolve("No prior history."),
-            generateTurnPrefixSummary(turnPrefixMessages, model, settings.reserveTokens, apiKey, headers, signal, thinkingLevel, streamFn),
+            generateTurnPrefixSummary(dedupedTurnPrefix, model, settings.reserveTokens, apiKey, headers, signal, thinkingLevel, streamFn),
         ]);
         // Merge into single summary
         summary = `${historyResult}\n\n---\n\n**Turn Context (split turn):**\n\n${turnPrefixResult}`;
     }
     else {
         // Just generate history summary
-        summary = await generateSummary(messagesToSummarize, model, settings.reserveTokens, apiKey, headers, signal, customInstructions, previousSummary, thinkingLevel, streamFn);
+        summary = await generateSummary(dedupedMessages, model, settings.reserveTokens, apiKey, headers, signal, customInstructions, previousSummary, thinkingLevel, streamFn);
     }
     // Compute file lists and append to summary
     const { readFiles, modifiedFiles } = computeFileLists(fileOps);

package/dist/{pi → sdk}/coding-agent/core/extensions/loader.js

@@ -9,7 +9,6 @@ import { fileURLToPath } from "node:url";
 import * as _bundledPiAgentCore from "../../../agent-core/index.js";
 import * as _bundledPiAi from "../../../ai/index.js";
 import * as _bundledPiAiOauth from "../../../ai/oauth.js";
-import * as _bundledPiTui from "../../../tui/index.js";
 import { createJiti } from "@mariozechner/jiti";
 // Static imports of packages that extensions may use.
 // These MUST be static so Bun bundles them into the compiled binary.
@@ -34,12 +33,10 @@ const VIRTUAL_MODULES = {
     "@sinclair/typebox/compile": _bundledTypeboxCompile,
     "@sinclair/typebox/value": _bundledTypeboxValue,
     "../../../agent-core/index.ts": _bundledPiAgentCore,
-    "../../../tui/index.ts": _bundledPiTui,
     "../../../ai/index.ts": _bundledPiAi,
     "../../../ai/oauth.ts": _bundledPiAiOauth,
     "../../index.ts": _bundledPiCodingAgent,
     "@mariozechner/pi-agent": _bundledPiAgentCore,
-    "@mariozechner/pi-tui": _bundledPiTui,
     "@mariozechner/pi-ai": _bundledPiAi,
     "@mariozechner/pi-ai/oauth": _bundledPiAiOauth,
     "@mariozechner/pi-coding-agent": _bundledPiCodingAgent,
@@ -68,18 +65,15 @@ function getAliases() {
     };
     const piCodingAgentEntry = packageIndex;
     const piAgentCoreEntry = resolveWorkspaceOrImport("agent/dist/index.js", "../../../agent-core/index.ts");
-    const piTuiEntry = resolveWorkspaceOrImport("tui/dist/index.js", "../../../tui/index.ts");
     const piAiEntry = resolveWorkspaceOrImport("ai/dist/index.js", "../../../ai/index.ts");
     const piAiOauthEntry = resolveWorkspaceOrImport("ai/dist/oauth.js", "../../../ai/oauth.ts");
     _aliases = {
         "../../index.ts": piCodingAgentEntry,
         "../../../agent-core/index.ts": piAgentCoreEntry,
-        "../../../tui/index.ts": piTuiEntry,
         "../../../ai/index.ts": piAiEntry,
         "../../../ai/oauth.ts": piAiOauthEntry,
         "@mariozechner/pi-coding-agent": piCodingAgentEntry,
         "@mariozechner/pi-agent": piAgentCoreEntry,
-        "@mariozechner/pi-tui": piTuiEntry,
         "@mariozechner/pi-ai": piAiEntry,
         "@mariozechner/pi-ai/oauth": piAiOauthEntry,
         typebox: typeboxEntry,