@aexol/spectral 0.7.8 → 0.8.2

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

@@ -6,45 +6,15 @@ import * as fs from "node:fs";
 import { createRequire } from "node:module";
 import * as path from "node:path";
 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 { createJiti } from "@mariozechner/jiti";
-// Static imports of packages that extensions may use.
-// These MUST be static so Bun bundles them into the compiled binary.
-// The virtualModules option then makes them available to extensions.
-import * as _bundledTypebox from "typebox";
-import * as _bundledTypeboxCompile from "typebox/compile";
-import * as _bundledTypeboxValue from "typebox/value";
-import { CONFIG_DIR_NAME, getAgentDir, isBunBinary } from "../../config.js";
-// NOTE: This import works because loader.ts exports are NOT re-exported from index.ts,
-// avoiding a circular dependency. Extensions can import from ../../index.ts.
-import * as _bundledPiCodingAgent from "../../index.js";
+import { CONFIG_DIR_NAME, getAgentDir } from "../../config.js";
 import { resolvePath } from "../../utils/paths.js";
 import { createEventBus } from "../event-bus.js";
 import { execCommand } from "../exec.js";
 import { createSyntheticSourceInfo } from "../source-info.js";
-/** Modules available to extensions via virtualModules (for compiled Bun binary) */
-const VIRTUAL_MODULES = {
-    typebox: _bundledTypebox,
-    "typebox/compile": _bundledTypeboxCompile,
-    "typebox/value": _bundledTypeboxValue,
-    "@sinclair/typebox": _bundledTypebox,
-    "@sinclair/typebox/compile": _bundledTypeboxCompile,
-    "@sinclair/typebox/value": _bundledTypeboxValue,
-    "../../../agent-core/index.ts": _bundledPiAgentCore,
-    "../../../ai/index.ts": _bundledPiAi,
-    "../../../ai/oauth.ts": _bundledPiAiOauth,
-    "../../index.ts": _bundledPiCodingAgent,
-    "@mariozechner/pi-agent": _bundledPiAgentCore,
-    "@mariozechner/pi-ai": _bundledPiAi,
-    "@mariozechner/pi-ai/oauth": _bundledPiAiOauth,
-    "@mariozechner/pi-coding-agent": _bundledPiCodingAgent,
-};
 const require = createRequire(import.meta.url);
 /**
  * Get aliases for jiti (used in Node.js/development mode).
- * In Bun binary mode, virtualModules is used instead.
  */
 let _aliases = null;
 function getAliases() {
@@ -263,10 +233,7 @@ function createExtensionAPI(extension, runtime, cwd, eventBus) {
 async function loadExtensionModule(extensionPath) {
     const jiti = createJiti(import.meta.url, {
         moduleCache: false,
-        // In Bun binary: use virtualModules for bundled packages (no filesystem resolution)
-        // Also disable tryNative so jiti handles ALL imports (not just the entry point)
-        // In Node.js/dev: use aliases to resolve to node_modules paths
-        ...(isBunBinary ? { virtualModules: VIRTUAL_MODULES, tryNative: false } : { alias: getAliases() }),
+        alias: getAliases(),
     });
     const module = await jiti.import(extensionPath, { default: true });
     const factory = module;

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

@@ -1,7 +1,6 @@
 /**
  * Extension runner - executes extensions and manages their lifecycle.
  */
-import { theme } from "../../modes/interactive/theme/theme.js";
 // Extension shortcuts compete with canonical keybinding ids from keybindings.json.
 // Only editor-global shortcuts are reserved here. Picker-specific bindings are not.
 const RESERVED_KEYBINDINGS_FOR_EXTENSION_CONFLICTS = [
@@ -80,7 +79,7 @@ const noOpUIContext = {
     setEditorComponent: () => { },
     getEditorComponent: () => undefined,
     get theme() {
-        return theme;
+        return {};
     },
     getAllThemes: () => [],
     getTheme: () => undefined,

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

@@ -241,16 +241,23 @@ export class ModelRegistry {
     loadError = undefined;
     authStorage;
     modelsJsonPath;
-    constructor(authStorage, modelsJsonPath) {
+    constructor(authStorage, modelsJsonPath, loadBuiltins) {
         this.authStorage = authStorage;
         this.modelsJsonPath = modelsJsonPath ? normalizePath(modelsJsonPath) : undefined;
-        this.loadModels();
+        if (loadBuiltins) {
+            this.loadModels();
+        }
+        // When loadBuiltins is false, we start with an empty model list
+        // — the caller is responsible for populating it via registerProvider().
+        // This is used in relay/serve mode where all models come from the
+        // backend's synthetic proxy providers, and loading the 16K-line
+        // models.generated.ts is pure startup waste.
     }
     static create(authStorage, modelsJsonPath = join(getAgentDir(), "models.json")) {
-        return new ModelRegistry(authStorage, modelsJsonPath);
+        return new ModelRegistry(authStorage, modelsJsonPath, true);
     }
     static inMemory(authStorage) {
-        return new ModelRegistry(authStorage, undefined);
+        return new ModelRegistry(authStorage, undefined, false);
     }
     /**
      * Reload models from disk (built-in + custom from models.json).

package/dist/sdk/coding-agent/core/model-resolver-utils.js

@@ -0,0 +1,8 @@
+/**
+ * Shared utility for validating thinking level strings.
+ * Extracted from cli/args.ts (TUI-only, now deleted).
+ */
+const VALID_THINKING_LEVELS = ["off", "minimal", "low", "medium", "high", "xhigh"];
+export function isValidThinkingLevel(level) {
+    return VALID_THINKING_LEVELS.includes(level);
+}

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

@@ -4,7 +4,7 @@
 import { modelsAreEqual } from "../../ai/index.js";
 import chalk from "chalk";
 import { minimatch } from "minimatch";
-import { isValidThinkingLevel } from "../cli/args.js";
+import { isValidThinkingLevel } from "./model-resolver-utils.js";
 import { DEFAULT_THINKING_LEVEL } from "./defaults.js";
 /** Default model IDs for each known provider */
 export const defaultModelPerProvider = {

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

@@ -33,7 +33,7 @@ const NETWORK_TIMEOUT_MS = 10000;
 const UPDATE_CHECK_CONCURRENCY = 4;
 const GIT_UPDATE_CONCURRENCY = 4;
 function isOfflineModeEnabled() {
-    const value = process.env.PI_OFFLINE;
+    const value = process.env.SPECTRAL_OFFLINE;
     if (!value)
         return false;
     return value === "1" || value.toLowerCase() === "true" || value.toLowerCase() === "yes";
@@ -1833,9 +1833,9 @@ export class DefaultPackageManager {
                 this.addResource(target, path, metadata, enabled);
             }
         };
-        // Project extensions from .pi/
+        // Project extensions from .spectral/
         addResources("extensions", collectAutoExtensionEntries(projectDirs.extensions), projectMetadata, projectOverrides.extensions, projectBaseDir);
-        // Project skills from .pi/
+        // Project skills from .spectral/
         addResources("skills", collectAutoSkillEntries(projectDirs.skills, "pi"), projectMetadata, projectOverrides.skills, projectBaseDir);
         // Project skills from .agents/ (each with its own baseDir)
         for (const agentsSkillsDir of projectAgentsSkillDirs) {
@@ -1848,9 +1848,9 @@ export class DefaultPackageManager {
         }
         addResources("prompts", collectAutoPromptEntries(projectDirs.prompts), projectMetadata, projectOverrides.prompts, projectBaseDir);
         addResources("themes", collectAutoThemeEntries(projectDirs.themes), projectMetadata, projectOverrides.themes, projectBaseDir);
-        // User extensions from ~/.pi/agent/
+        // User extensions from ~/.spectral/agent/
         addResources("extensions", collectAutoExtensionEntries(userDirs.extensions), userMetadata, userOverrides.extensions, globalBaseDir);
-        // User skills from ~/.pi/agent/
+        // User skills from ~/.spectral/agent/
         addResources("skills", collectAutoSkillEntries(userDirs.skills, "pi"), userMetadata, userOverrides.skills, globalBaseDir);
         // User skills from ~/.agents/ (with its own baseDir)
         const userAgentsBaseDir = dirname(userAgentsSkillsDir);

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

@@ -2,7 +2,7 @@ import { existsSync, readdirSync, readFileSync, statSync } from "node:fs";
 import { join, resolve, sep } from "node:path";
 import chalk from "chalk";
 import { CONFIG_DIR_NAME } from "../config.js";
-import { loadThemeFromPath } from "../modes/interactive/theme/theme.js";
+import { loadThemeFromPath } from "./theme.js";
 import { canonicalizePath, isLocalPath, resolvePath } from "../utils/paths.js";
 import { createEventBus } from "./event-bus.js";
 import { createExtensionRuntime, loadExtensionFromFactory, loadExtensions } from "./extensions/loader.js";

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

@@ -31,7 +31,7 @@ function getAttributionHeaders(model, settingsManager) {
     }
     if (model.provider === "openrouter" || model.baseUrl.includes("openrouter.ai")) {
         return {
-            "HTTP-Referer": "https://pi.dev",
+            "HTTP-Referer": "https://spectral.dev",
             "X-OpenRouter-Title": "pi",
             "X-OpenRouter-Categories": "cli-agent",
         };

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

@@ -208,7 +208,7 @@ export function buildSessionContext(entries, leafId, byId) {
 }
 /**
  * Compute the default session directory for a cwd.
- * Encodes cwd into a safe directory name under ~/.pi/agent/sessions/.
+ * Encodes cwd into a safe directory name under ~/.spectral/agent/sessions/.
  */
 export function getDefaultSessionDir(cwd, agentDir = getDefaultAgentDir()) {
     const resolvedCwd = resolvePath(cwd);
@@ -1002,7 +1002,7 @@ export class SessionManager {
     /**
      * Create a new session.
      * @param cwd Working directory (stored in session header)
-     * @param sessionDir Optional session directory. If omitted, uses default (~/.pi/agent/sessions/<encoded-cwd>/).
+     * @param sessionDir Optional session directory. If omitted, uses default (~/.spectral/agent/sessions/<encoded-cwd>/).
      */
     static create(cwd, sessionDir) {
         const dir = sessionDir ? normalizePath(sessionDir) : getDefaultSessionDir(cwd);
@@ -1027,7 +1027,7 @@ export class SessionManager {
     /**
      * Continue the most recent session, or create new if none.
      * @param cwd Working directory
-     * @param sessionDir Optional session directory. If omitted, uses default (~/.pi/agent/sessions/<encoded-cwd>/).
+     * @param sessionDir Optional session directory. If omitted, uses default (~/.spectral/agent/sessions/<encoded-cwd>/).
      */
     static continueRecent(cwd, sessionDir) {
         const dir = sessionDir ? normalizePath(sessionDir) : getDefaultSessionDir(cwd);
@@ -1089,7 +1089,7 @@ export class SessionManager {
     /**
      * List all sessions for a directory.
      * @param cwd Working directory (used to compute default session directory)
-     * @param sessionDir Optional session directory. If omitted, uses default (~/.pi/agent/sessions/<encoded-cwd>/).
+     * @param sessionDir Optional session directory. If omitted, uses default (~/.spectral/agent/sessions/<encoded-cwd>/).
      * @param onProgress Optional callback for progress updates (loaded, total)
      */
     static async list(cwd, sessionDir, onProgress) {

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

@@ -193,9 +193,7 @@ export class SettingsManager {
             settings.skills !== null &&
             !Array.isArray(settings.skills)) {
             const skillsSettings = settings.skills;
-            if (skillsSettings.enableSkillCommands !== undefined && settings.enableSkillCommands === undefined) {
-                settings.enableSkillCommands = skillsSettings.enableSkillCommands;
-            }
+            // enableSkillCommands is deprecated — only migrate the custom directories
             if (Array.isArray(skillsSettings.customDirectories) && skillsSettings.customDirectories.length > 0) {
                 settings.skills = skillsSettings.customDirectories;
             }
@@ -367,14 +365,6 @@ export class SettingsManager {
         this.errors = [];
         return drained;
     }
-    getLastChangelogVersion() {
-        return this.settings.lastChangelogVersion;
-    }
-    setLastChangelogVersion(version) {
-        this.globalSettings.lastChangelogVersion = version;
-        this.markModified("lastChangelogVersion");
-        this.save();
-    }
     getSessionDir() {
         const sessionDir = this.settings.sessionDir;
         return sessionDir ? normalizePath(sessionDir) : sessionDir;
@@ -438,14 +428,6 @@ export class SettingsManager {
         this.markModified("followUpMode");
         this.save();
     }
-    getTheme() {
-        return this.settings.theme;
-    }
-    setTheme(theme) {
-        this.globalSettings.theme = theme;
-        this.markModified("theme");
-        this.save();
-    }
     getDefaultThinkingLevel() {
         return this.settings.defaultThinkingLevel;
     }
@@ -539,14 +521,6 @@ export class SettingsManager {
             maxRetryDelayMs: this.settings.retry?.provider?.maxRetryDelayMs ?? 60000,
         };
     }
-    getHideThinkingBlock() {
-        return this.settings.hideThinkingBlock ?? false;
-    }
-    setHideThinkingBlock(hide) {
-        this.globalSettings.hideThinkingBlock = hide;
-        this.markModified("hideThinkingBlock");
-        this.save();
-    }
     getShellPath() {
         return this.settings.shellPath;
     }
@@ -555,14 +529,6 @@ export class SettingsManager {
         this.markModified("shellPath");
         this.save();
     }
-    getQuietStartup() {
-        return this.settings.quietStartup ?? false;
-    }
-    setQuietStartup(quiet) {
-        this.globalSettings.quietStartup = quiet;
-        this.markModified("quietStartup");
-        this.save();
-    }
     getShellCommandPrefix() {
         return this.settings.shellCommandPrefix;
     }
@@ -579,14 +545,6 @@ export class SettingsManager {
         this.markModified("npmCommand");
         this.save();
     }
-    getCollapseChangelog() {
-        return this.settings.collapseChangelog ?? false;
-    }
-    setCollapseChangelog(collapse) {
-        this.globalSettings.collapseChangelog = collapse;
-        this.markModified("collapseChangelog");
-        this.save();
-    }
     getEnableInstallTelemetry() {
         return this.settings.enableInstallTelemetry ?? true;
     }
@@ -651,83 +609,9 @@ export class SettingsManager {
         this.markProjectModified("prompts");
         this.saveProjectSettings(projectSettings);
     }
-    getThemePaths() {
-        return [...(this.settings.themes ?? [])];
-    }
-    setThemePaths(paths) {
-        this.globalSettings.themes = paths;
-        this.markModified("themes");
-        this.save();
-    }
-    setProjectThemePaths(paths) {
-        const projectSettings = structuredClone(this.projectSettings);
-        projectSettings.themes = paths;
-        this.markProjectModified("themes");
-        this.saveProjectSettings(projectSettings);
-    }
-    getEnableSkillCommands() {
-        return this.settings.enableSkillCommands ?? true;
-    }
-    setEnableSkillCommands(enabled) {
-        this.globalSettings.enableSkillCommands = enabled;
-        this.markModified("enableSkillCommands");
-        this.save();
-    }
     getThinkingBudgets() {
         return this.settings.thinkingBudgets;
     }
-    getShowImages() {
-        return this.settings.terminal?.showImages ?? true;
-    }
-    setShowImages(show) {
-        if (!this.globalSettings.terminal) {
-            this.globalSettings.terminal = {};
-        }
-        this.globalSettings.terminal.showImages = show;
-        this.markModified("terminal", "showImages");
-        this.save();
-    }
-    getImageWidthCells() {
-        const width = this.settings.terminal?.imageWidthCells;
-        if (typeof width !== "number" || !Number.isFinite(width)) {
-            return 60;
-        }
-        return Math.max(1, Math.floor(width));
-    }
-    setImageWidthCells(width) {
-        if (!this.globalSettings.terminal) {
-            this.globalSettings.terminal = {};
-        }
-        this.globalSettings.terminal.imageWidthCells = Math.max(1, Math.floor(width));
-        this.markModified("terminal", "imageWidthCells");
-        this.save();
-    }
-    getClearOnShrink() {
-        // Settings takes precedence, then env var, then default false
-        if (this.settings.terminal?.clearOnShrink !== undefined) {
-            return this.settings.terminal.clearOnShrink;
-        }
-        return process.env.PI_CLEAR_ON_SHRINK === "1";
-    }
-    setClearOnShrink(enabled) {
-        if (!this.globalSettings.terminal) {
-            this.globalSettings.terminal = {};
-        }
-        this.globalSettings.terminal.clearOnShrink = enabled;
-        this.markModified("terminal", "clearOnShrink");
-        this.save();
-    }
-    getShowTerminalProgress() {
-        return this.settings.terminal?.showTerminalProgress ?? false;
-    }
-    setShowTerminalProgress(enabled) {
-        if (!this.globalSettings.terminal) {
-            this.globalSettings.terminal = {};
-        }
-        this.globalSettings.terminal.showTerminalProgress = enabled;
-        this.markModified("terminal", "showTerminalProgress");
-        this.save();
-    }
     getImageAutoResize() {
         return this.settings.images?.autoResize ?? true;
     }
@@ -758,57 +642,4 @@ export class SettingsManager {
         this.markModified("enabledModels");
         this.save();
     }
-    getDoubleEscapeAction() {
-        return this.settings.doubleEscapeAction ?? "tree";
-    }
-    setDoubleEscapeAction(action) {
-        this.globalSettings.doubleEscapeAction = action;
-        this.markModified("doubleEscapeAction");
-        this.save();
-    }
-    getTreeFilterMode() {
-        const mode = this.settings.treeFilterMode;
-        const valid = ["default", "no-tools", "user-only", "labeled-only", "all"];
-        return mode && valid.includes(mode) ? mode : "default";
-    }
-    setTreeFilterMode(mode) {
-        this.globalSettings.treeFilterMode = mode;
-        this.markModified("treeFilterMode");
-        this.save();
-    }
-    getShowHardwareCursor() {
-        return this.settings.showHardwareCursor ?? process.env.PI_HARDWARE_CURSOR === "1";
-    }
-    setShowHardwareCursor(enabled) {
-        this.globalSettings.showHardwareCursor = enabled;
-        this.markModified("showHardwareCursor");
-        this.save();
-    }
-    getEditorPaddingX() {
-        return this.settings.editorPaddingX ?? 0;
-    }
-    setEditorPaddingX(padding) {
-        this.globalSettings.editorPaddingX = Math.max(0, Math.min(3, Math.floor(padding)));
-        this.markModified("editorPaddingX");
-        this.save();
-    }
-    getAutocompleteMaxVisible() {
-        return this.settings.autocompleteMaxVisible ?? 5;
-    }
-    setAutocompleteMaxVisible(maxVisible) {
-        this.globalSettings.autocompleteMaxVisible = Math.max(3, Math.min(20, Math.floor(maxVisible)));
-        this.markModified("autocompleteMaxVisible");
-        this.save();
-    }
-    getCodeBlockIndent() {
-        return this.settings.markdown?.codeBlockIndent ?? "  ";
-    }
-    getWarnings() {
-        return { ...(this.settings.warnings ?? {}) };
-    }
-    setWarnings(warnings) {
-        this.globalSettings.warnings = { ...warnings };
-        this.markModified("warnings");
-        this.save();
-    }
 }

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

@@ -78,10 +78,12 @@ export function buildSystemPrompt(options) {
         }
     }
     // Always include these
+    addGuideline("Never assign coding or implementation work to people — you write the code. Non-coding tasks (review, design, ops, coordination) can be assigned");
+    addGuideline("When estimating effort, frame it as a feature scope (e.g. 'this is a ~20-hour coding enhancement') rather than using traditional time measures. Tie every estimate to a clearly defined feature boundary");
     addGuideline("Be concise in your responses");
     addGuideline("Show file paths clearly when working with files");
     const guidelines = guidelinesList.map((g) => `- ${g}`).join("\n");
-    let prompt = `You are an expert coding assistant operating inside pi, a coding agent harness. You help users by reading files, executing commands, editing code, and writing new files.
+    let prompt = `You are an expert coding agent operating inside pi, a coding agent harness. You read files, execute commands, edit code, and write new files. All coding and implementation work is yours — never delegate it. Non-coding tasks (review, design decisions, ops) may be assigned to people. When discussing effort, frame each enhancement around a ~20-hour coding scope tied to a concrete feature boundary, avoiding calendar-day or week-based estimates.
 
 Available tools:
 ${toolsList}

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

@@ -3,6 +3,6 @@ function isTruthyEnvFlag(value) {
         return false;
     return value === "1" || value.toLowerCase() === "true" || value.toLowerCase() === "yes";
 }
-export function isInstallTelemetryEnabled(settingsManager, telemetryEnv = process.env.PI_TELEMETRY) {
+export function isInstallTelemetryEnabled(settingsManager, telemetryEnv = process.env.SPECTRAL_TELEMETRY) {
     return telemetryEnv !== undefined ? isTruthyEnvFlag(telemetryEnv) : settingsManager.getEnableInstallTelemetry();
 }