@aexol/spectral 0.8.0 → 0.8.2

package/dist/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/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/sdk/coding-agent/core/theme.js

@@ -0,0 +1,202 @@
+/**
+ * Minimal shared theme for syntax highlighting and extension API compatibility.
+ *
+ * This replaces the interactive/mode TUI theme system with a stripped-down
+ * version that supports the core utilities used by the serve/relay path:
+ * - getLanguageFromPath(): maps file extensions to highlight.js language IDs
+ * - highlightCode(): syntax-highlights code blocks
+ * - loadThemeFromPath(): loads a theme JSON file from disk
+ * - Theme class: used by resource-loader and extension types
+ *
+ * All ANSI color formatting (theme.fg(), theme.bold(), etc.) is removed —
+ * tool output is plain text for LLM consumption in serve/relay mode.
+ */
+import * as fs from "node:fs";
+import { highlight, supportsLanguage } from "../utils/syntax-highlight.js";
+// ============================================================================
+// Language detection
+// ============================================================================
+const extToLang = {
+    ts: "typescript",
+    tsx: "typescript",
+    js: "javascript",
+    jsx: "javascript",
+    mjs: "javascript",
+    cjs: "javascript",
+    py: "python",
+    rb: "ruby",
+    rs: "rust",
+    go: "go",
+    java: "java",
+    kt: "kotlin",
+    swift: "swift",
+    c: "c",
+    h: "c",
+    cpp: "cpp",
+    cc: "cpp",
+    cxx: "cpp",
+    hpp: "cpp",
+    cs: "csharp",
+    php: "php",
+    sh: "bash",
+    bash: "bash",
+    zsh: "bash",
+    fish: "fish",
+    ps1: "powershell",
+    sql: "sql",
+    html: "html",
+    htm: "html",
+    css: "css",
+    scss: "scss",
+    sass: "sass",
+    less: "less",
+    json: "json",
+    yaml: "yaml",
+    yml: "yaml",
+    toml: "toml",
+    xml: "xml",
+    md: "markdown",
+    markdown: "markdown",
+    dockerfile: "dockerfile",
+    makefile: "makefile",
+    cmake: "cmake",
+    lua: "lua",
+    perl: "perl",
+    r: "r",
+    scala: "scala",
+    clj: "clojure",
+    ex: "elixir",
+    exs: "elixir",
+    erl: "erlang",
+    hs: "haskell",
+    ml: "ocaml",
+    vim: "vim",
+    graphql: "graphql",
+    proto: "protobuf",
+    tf: "hcl",
+    hcl: "hcl",
+};
+export function getLanguageFromPath(filePath) {
+    const ext = filePath.split(".").pop()?.toLowerCase();
+    if (!ext)
+        return undefined;
+    return extToLang[ext];
+}
+// ============================================================================
+// Syntax highlighting
+// ============================================================================
+/** Identity formatter — returns text as-is (no ANSI styling in serve mode). */
+function identity(text) {
+    return text;
+}
+const neutralHighlightTheme = {
+    keyword: identity,
+    built_in: identity,
+    literal: identity,
+    number: identity,
+    string: identity,
+    comment: identity,
+    function: identity,
+    title: identity,
+    class: identity,
+    type: identity,
+    attr: identity,
+    variable: identity,
+    params: identity,
+    operator: identity,
+    punctuation: identity,
+};
+/**
+ * Highlight code with syntax coloring based on file extension or language.
+ * In serve mode, returns plain text (no ANSI coloring for LLM consumption).
+ */
+export function highlightCode(code, lang) {
+    const validLang = lang && supportsLanguage(lang) ? lang : undefined;
+    if (!validLang) {
+        return code.split("\n");
+    }
+    try {
+        return highlight(code, {
+            language: validLang,
+            ignoreIllegals: true,
+            theme: neutralHighlightTheme,
+        }).split("\n");
+    }
+    catch {
+        return code.split("\n");
+    }
+}
+// ============================================================================
+// Theme loading (for resource-loader)
+// ============================================================================
+/** Resolve built-in theme directory using config path helpers. */
+function getThemesDir() {
+    try {
+        // This function is exported from config.ts and uses the actual package directory
+        const { getThemesDir: fn } = require("../../config.js");
+        return fn();
+    }
+    catch {
+        // Fallback: themes were in modes/interactive/theme/, try to find them
+        try {
+            const { join } = require("node:path");
+            const { existsSync } = require("node:fs");
+            const packageDir = join(__dirname, "../../..");
+            for (const base of ["src", "dist"]) {
+                const candidates = [
+                    join(packageDir, base, "modes", "interactive", "theme"),
+                ];
+                for (const dir of candidates) {
+                    if (existsSync(dir))
+                        return dir;
+                }
+            }
+        }
+        catch {
+            // ignore
+        }
+        return __dirname;
+    }
+}
+function getBuiltinThemes() {
+    const themesDir = getThemesDir();
+    if (!themesDir)
+        return {};
+    try {
+        const { join } = require("node:path");
+        const { readFileSync } = require("node:fs");
+        return {
+            dark: JSON.parse(readFileSync(join(themesDir, "dark.json"), "utf-8")),
+            light: JSON.parse(readFileSync(join(themesDir, "light.json"), "utf-8")),
+        };
+    }
+    catch {
+        return {};
+    }
+}
+export function loadThemeFromPath(themePath) {
+    const content = fs.readFileSync(themePath, "utf-8");
+    const json = JSON.parse(content);
+    return {
+        name: json.name,
+        sourcePath: themePath,
+    };
+}
+// ============================================================================
+// Registered themes (for extension API compatibility)
+// ============================================================================
+const registeredThemes = new Map();
+export function setRegisteredThemes(themes) {
+    registeredThemes.clear();
+    for (const t of themes) {
+        if (t.name) {
+            registeredThemes.set(t.name, t);
+        }
+    }
+}
+export function getAvailableThemes() {
+    return Array.from(new Set([
+        ...Object.keys(getBuiltinThemes()),
+        ...registeredThemes.keys(),
+    ])).sort();
+}

package/dist/sdk/coding-agent/core/tools/bash.js

@@ -1,12 +1,10 @@
 import { existsSync } from "node:fs";
 import { spawn } from "child_process";
 import { Type } from "typebox";
-import { keyHint } from "../../modes/interactive/components/keybinding-hints.js";
-import { theme } from "../../modes/interactive/theme/theme.js";
 import { waitForChildProcess } from "../../utils/child-process.js";
 import { getShellConfig, getShellEnv, killProcessTree, trackDetachedChildPid, untrackDetachedChildPid, } from "../../utils/shell.js";
 import { OutputAccumulator } from "./output-accumulator.js";
-import { getTextOutput, invalidArgText, str } from "./render-utils.js";
+import { getTextOutput, str } from "./render-utils.js";
 import { wrapToolDefinition } from "./tool-definition-wrapper.js";
 import { DEFAULT_MAX_BYTES, DEFAULT_MAX_LINES, formatSize } from "./truncate.js";
 const bashSchema = Type.Object({
@@ -103,12 +101,17 @@ const BASH_UPDATE_THROTTLE_MS = 100;
 function formatDuration(ms) {
     return `${(ms / 1000).toFixed(1)}s`;
 }
+function plainText(s) { return s; }
+const noopTheme = {
+    fg: (_name, text) => text,
+    bold: plainText,
+};
 function formatBashCall(args) {
     const command = str(args?.command);
     const timeout = args?.timeout;
-    const timeoutSuffix = timeout ? theme.fg("muted", ` (timeout ${timeout}s)`) : "";
-    const commandDisplay = command === null ? invalidArgText(theme) : command ? command : theme.fg("toolOutput", "...");
-    return theme.fg("toolTitle", theme.bold(`$ ${commandDisplay}`)) + timeoutSuffix;
+    const timeoutSuffix = timeout ? ` (timeout ${timeout}s)` : "";
+    const commandDisplay = command === null ? "[invalid]" : command || "...";
+    return `$ ${commandDisplay}${timeoutSuffix}`;
 }
 function formatBashResult(result, options, showImages, startedAt, endedAt) {
     let output = getTextOutput(result, showImages).trim();
@@ -122,24 +125,20 @@ function formatBashResult(result, options, showImages, startedAt, endedAt) {
     }
     const parts = [];
     if (output) {
-        const styledOutput = output
-            .split("\n")
-            .map((line) => theme.fg("toolOutput", line))
-            .join("\n");
         if (options.expanded) {
-            parts.push(`\n${styledOutput}`);
+            parts.push(`\n${output}`);
         }
         else {
-            const styledLines = styledOutput.split("\n");
-            if (styledLines.length > BASH_PREVIEW_LINES) {
-                const hint = theme.fg("muted", `... (${styledLines.length - BASH_PREVIEW_LINES} earlier lines, ${keyHint("app.tools.expand", "to expand")})`);
+            const lines = output.split("\n");
+            if (lines.length > BASH_PREVIEW_LINES) {
+                const hint = `... (${lines.length - BASH_PREVIEW_LINES} earlier lines)`;
                 parts.push("");
                 parts.push(hint);
-                parts.push(...styledLines.slice(-BASH_PREVIEW_LINES));
+                parts.push(...lines.slice(-BASH_PREVIEW_LINES));
             }
             else {
                 parts.push("");
-                parts.push(...styledLines);
+                parts.push(...lines);
             }
         }
     }
@@ -156,12 +155,12 @@ function formatBashResult(result, options, showImages, startedAt, endedAt) {
                 warnings.push(`Truncated: ${truncation.outputLines} lines shown (${formatSize(truncation.maxBytes ?? DEFAULT_MAX_BYTES)} limit)`);
             }
         }
-        parts.push(`\n${theme.fg("warning", `[${warnings.join(". ")}]`)}`);
+        parts.push(`\n[${warnings.join(". ")}]`);
     }
     if (startedAt !== undefined) {
         const label = options.isPartial ? "Elapsed" : "Took";
         const endTime = endedAt ?? Date.now();
-        parts.push(`\n${theme.fg("muted", `${label} ${formatDuration(endTime - startedAt)}`)}`);
+        parts.push(`\n${label} ${formatDuration(endTime - startedAt)}`);
     }
     return parts.join("");
 }

package/dist/sdk/coding-agent/core/tools/edit.js

@@ -1,11 +1,11 @@
 import { constants } from "fs";
 import { access as fsAccess, readFile as fsReadFile, writeFile as fsWriteFile } from "fs/promises";
 import { Type } from "typebox";
-import { renderDiff } from "../../modes/interactive/components/diff.js";
+import { renderDiff } from "./render-diff.js";
 import { applyEditsToNormalizedContent, detectLineEnding, generateDiffString, generateUnifiedPatch, normalizeToLF, restoreLineEndings, stripBom, } from "./edit-diff.js";
 import { withFileMutationQueue } from "./file-mutation-queue.js";
 import { resolveToCwd } from "./path-utils.js";
-import { invalidArgText, shortenPath, str } from "./render-utils.js";
+import { shortenPath, str } from "./render-utils.js";
 import { wrapToolDefinition } from "./tool-definition-wrapper.js";
 const replaceEditSchema = Type.Object({
     oldText: Type.String({
@@ -53,14 +53,13 @@ function validateEditInput(input) {
     }
     return { path: input.path, edits: input.edits };
 }
-function formatEditCall(args, theme) {
-    const invalidArg = invalidArgText(theme);
+function formatEditCall(args, _theme) {
     const rawPath = str(args?.file_path ?? args?.path);
     const path = rawPath !== null ? shortenPath(rawPath) : null;
-    const pathDisplay = path === null ? invalidArg : path ? theme.fg("accent", path) : theme.fg("toolOutput", "...");
-    return `${theme.fg("toolTitle", theme.bold("edit"))} ${pathDisplay}`;
+    const pathDisplay = path === null ? "[invalid]" : path || "...";
+    return `edit ${pathDisplay}`;
 }
-function formatEditResult(args, result, theme, isError) {
+function formatEditResult(args, result, _theme, isError) {
     const rawPath = str(args?.file_path ?? args?.path);
     if (isError) {
         const errorText = result.content
@@ -70,7 +69,7 @@ function formatEditResult(args, result, theme, isError) {
         if (!errorText) {
             return undefined;
         }
-        return theme.fg("error", errorText);
+        return errorText;
     }
     const resultDiff = result.details?.diff;
     if (resultDiff) {

package/dist/sdk/coding-agent/core/tools/find.js

@@ -3,10 +3,9 @@ import { spawn } from "child_process";
 import { existsSync } from "fs";
 import path from "path";
 import { Type } from "typebox";
-import { keyHint } from "../../modes/interactive/components/keybinding-hints.js";
 import { ensureTool } from "../../utils/tools-manager.js";
 import { resolveToCwd } from "./path-utils.js";
-import { getTextOutput, invalidArgText, shortenPath, str } from "./render-utils.js";
+import { getTextOutput, shortenPath, str } from "./render-utils.js";
 import { wrapToolDefinition } from "./tool-definition-wrapper.js";
 import { DEFAULT_MAX_BYTES, formatSize, truncateHead } from "./truncate.js";
 function toPosixPath(value) {
@@ -25,22 +24,19 @@ const defaultFindOperations = {
     // This is a placeholder. Actual fd execution happens in execute() when no custom glob is provided.
     glob: () => [],
 };
-function formatFindCall(args, theme) {
+function formatFindCall(args, _theme) {
     const pattern = str(args?.pattern);
     const rawPath = str(args?.path);
     const path = rawPath !== null ? shortenPath(rawPath || ".") : null;
     const limit = args?.limit;
-    const invalidArg = invalidArgText(theme);
-    let text = theme.fg("toolTitle", theme.bold("find")) +
-        " " +
-        (pattern === null ? invalidArg : theme.fg("accent", pattern || "")) +
-        theme.fg("toolOutput", ` in ${path === null ? invalidArg : path}`);
+    const invalidArg = "[invalid]";
+    let text = `find ${pattern === null ? invalidArg : pattern || ""} in ${path === null ? invalidArg : path}`;
     if (limit !== undefined) {
-        text += theme.fg("toolOutput", ` (limit ${limit})`);
+        text += ` (limit ${limit})`;
     }
     return text;
 }
-function formatFindResult(result, options, theme, showImages) {
+function formatFindResult(result, options, _theme, showImages) {
     const output = getTextOutput(result, showImages).trim();
     let text = "";
     if (output) {
@@ -48,9 +44,9 @@ function formatFindResult(result, options, theme, showImages) {
         const maxLines = options.expanded ? lines.length : 20;
         const displayLines = lines.slice(0, maxLines);
         const remaining = lines.length - maxLines;
-        text += `\n${displayLines.map((line) => theme.fg("toolOutput", line)).join("\n")}`;
+        text += `\n${displayLines.join("\n")}`;
         if (remaining > 0) {
-            text += `${theme.fg("muted", `\n... (${remaining} more lines,`)} ${keyHint("app.tools.expand", "to expand")})`;
+            text += `\n... (${remaining} more lines)`;
         }
     }
     const resultLimit = result.details?.resultLimitReached;
@@ -61,7 +57,7 @@ function formatFindResult(result, options, theme, showImages) {
             warnings.push(`${resultLimit} results limit`);
         if (truncation?.truncated)
             warnings.push(`${formatSize(truncation.maxBytes ?? DEFAULT_MAX_BYTES)} limit`);
-        text += `\n${theme.fg("warning", `[Truncated: ${warnings.join(", ")}]`)}`;
+        text += `\n[Truncated: ${warnings.join(", ")}]`;
     }
     return text;
 }