@kolisachint/hoocode-agent 0.5.28 → 0.5.30

package/dist/modes/interactive/components/settings-selector.js

@@ -57,9 +57,9 @@ const LEARN_KEYS = new Set(LEARN_SETTINGS.map((setting) => setting.key));
  * Preset list for a numeric row, guaranteed to contain the value in force.
  *
  * Without this a value set by hand in settings.json — say 45 days — is absent
- * from the cycle, so the first keypress silently snaps it to the first preset.
- * These particular settings gate whether `/learn` finds anything at all, so a
- * stray keystroke narrowing the window is exactly the surprise to avoid.
+ * from the cycle, so the first keypress silently snaps it to the first preset,
+ * discarding a deliberate choice the pane never showed as unusual. Every numeric
+ * cycle row goes through this for that reason.
  */
 function presetValues(presets, current) {
     const all = presets.includes(current) ? presets : [...presets, current].sort((a, b) => a - b);
@@ -97,6 +97,73 @@ class WarningSettingsSubmenu extends Container {
         this.settingsList.handleInput(data);
     }
 }
+/**
+ * The artifact platform targets, in the order the pane shows them. Each row is an
+ * independent on/off toggle because `--platform` (and the `platform` setting it
+ * mirrors) takes a list: emitting for two platforms at once is a supported shape.
+ * The tokens here are the canonical ones — the CLI aliases (`copilot`, `gh`,
+ * `native`) fold into these before anything reads them.
+ */
+const PLATFORM_ROWS = [
+    {
+        platform: "claude",
+        label: "claude",
+        description: "Claude Code layout: .claude/ scaffolds, and authored plugins drop into ~/.claude/skills/<id>/. The default when nothing is set.",
+    },
+    {
+        platform: "github",
+        label: "github (copilot, gh)",
+        description: "Copilot layout: .github/ scaffolds, and authored plugins are produced under ~/.agents/publish/github/<id>/.",
+    },
+    {
+        platform: "agents",
+        label: "agents (native)",
+        description: "Cross-vendor .agents/ layout. Scaffolds only - a plugin belongs to no marketplace in this layout, so plugin authoring ignores it.",
+    },
+];
+/** Row value for the platform setting: the selection, or the fallback when it is empty. */
+function platformSummary(platforms) {
+    return platforms.length > 0 ? platforms.join(", ") : "default (claude)";
+}
+/**
+ * Submenu for the session's artifact platform targets (the `platform` setting,
+ * same knob as `--platform`).
+ *
+ * A list rather than a cycle: the setting is a list, and the three tokens are not
+ * mutually exclusive. Turning everything off is legal and means "unset" — the
+ * per-consumer defaults come back (claude for plugins, .hoocode/ for scaffolds).
+ */
+class PlatformSubmenu extends Container {
+    settingsList;
+    selected;
+    constructor(platforms, onChange, onDone) {
+        super();
+        this.selected = new Set(platforms);
+        const items = PLATFORM_ROWS.map(({ platform, label, description }) => ({
+            id: platform,
+            label,
+            description,
+            currentValue: this.selected.has(platform) ? "on" : "off",
+            values: ["on", "off"],
+        }));
+        this.settingsList = new SettingsList(items, Math.min(items.length, 10), getSettingsListTheme(), (id, newValue) => {
+            const platform = id;
+            if (newValue === "on")
+                this.selected.add(platform);
+            else
+                this.selected.delete(platform);
+            onChange(this.ordered());
+        }, () => onDone(platformSummary(this.ordered())));
+        this.addChild(this.settingsList);
+    }
+    /** Selection in the pane's order, so the persisted list does not depend on click order. */
+    ordered() {
+        return PLATFORM_ROWS.map((row) => row.platform).filter((platform) => this.selected.has(platform));
+    }
+    handleInput(data) {
+        this.settingsList.handleInput(data);
+    }
+}
 /**
  * Submenu for tool availability. The first rows are group switches (web,
  * semantic search) that decide whether a group's tools
@@ -130,6 +197,10 @@ class ToolsSubmenu extends Container {
                 ? "Core tool. Disabling leaves the agent unable to perform this action in every session."
                 : "Disable to remove this tool from the agent this session and every future session.",
             currentValue: tool.enabled ? "on" : "off",
+            // What the schema costs on every request, whether the tool is on or off:
+            // off, it is the price of turning it back on. This is the number that
+            // makes a tool worth disabling, so it belongs beside the switch.
+            valueSuffix: tool.tokens !== undefined ? `${tokenCount(tool.tokens)} tok/turn` : undefined,
             values: ["on", "off"],
         }));
         const items = [...groupItems, ...toolItems];
@@ -168,52 +239,65 @@ function bytesToLabel(bytes) {
     const match = TOOL_OUTPUT_BYTE_PRESETS.find(([, b]) => b === bytes);
     return match ? match[0] : `${Math.round(bytes / 1024)} KB`;
 }
+/** Byte-cap labels to cycle through, including a hand-set cap that matches no preset. */
+function byteLabels(current) {
+    const all = TOOL_OUTPUT_BYTE_PRESETS.some(([, bytes]) => bytes === current)
+        ? [...TOOL_OUTPUT_BYTE_PRESETS]
+        : [...TOOL_OUTPUT_BYTE_PRESETS, [bytesToLabel(current), current]];
+    return all.sort((a, b) => a[1] - b[1]).map(([label]) => label);
+}
 /**
- * Submenu for per-tool runtime settings. These feed the tool runtime the next
- * time it is built (next session / rebuild), so changes apply to future tool
- * calls rather than retroactively.
+ * Submenu for what a tool result looks like: how much of it is rendered, and
+ * where it is truncated. The display level applies to the transcript at once;
+ * the caps feed the tool runtime, so they bind on future tool calls.
+ *
+ * Context GC is deliberately not here. It is not about a tool's output but about
+ * what stays in the outgoing context, which is the Context category's subject.
  */
 class ToolSettingsSubmenu extends Container {
     settingsList;
     constructor(config, callbacks, onCancel) {
         super();
         const items = [
+            {
+                id: "tool-output-display",
+                label: "Display",
+                description: "How tool results render. 'standard': shown (expandable). 'collapsed': hidden. 'peek': hidden with a ▸ reveal caret (press the expand key to reveal).",
+                currentValue: config.toolOutputDisplay,
+                values: ["standard", "collapsed", "peek"],
+            },
             {
                 id: "output-max-bytes",
-                label: "Output max bytes",
+                label: "Max bytes",
                 description: "Byte cap on a single read/bash result before truncation. Applies to future tool calls.",
                 currentValue: bytesToLabel(config.toolOutputMaxBytes),
-                values: TOOL_OUTPUT_BYTE_PRESETS.map(([label]) => label),
+                values: byteLabels(config.toolOutputMaxBytes),
             },
             {
                 id: "output-max-lines",
-                label: "Output max lines",
+                label: "Max lines",
                 description: "Line cap on a single read/bash result before truncation. Applies to future tool calls.",
                 currentValue: String(config.toolOutputMaxLines),
-                values: ["200", "400", "800", "1600", "3200"],
-            },
-            {
-                id: "context-gc",
-                label: "Context GC",
-                description: "Stub superseded read results (files later edited/re-read) out of the outgoing context.",
-                currentValue: config.contextGc ? "true" : "false",
-                values: ["true", "false"],
+                values: presetValues([200, 400, 800, 1600, 3200], config.toolOutputMaxLines),
             },
         ];
         this.settingsList = new SettingsList(items, Math.min(items.length, 10), getSettingsListTheme(), (id, newValue) => {
             switch (id) {
+                case "tool-output-display":
+                    callbacks.onToolOutputDisplayChange(newValue);
+                    break;
                 case "output-max-bytes": {
                     const preset = TOOL_OUTPUT_BYTE_PRESETS.find(([label]) => label === newValue);
-                    if (preset)
-                        callbacks.onToolOutputMaxBytesChange(preset[1]);
+                    // A hand-set cap has no preset entry; its label is "<n> KB" by
+                    // construction, so read the number back out of it.
+                    const bytes = preset ? preset[1] : Math.round(parseFloat(newValue) * 1024);
+                    if (Number.isFinite(bytes) && bytes > 0)
+                        callbacks.onToolOutputMaxBytesChange(bytes);
                     break;
                 }
                 case "output-max-lines":
                     callbacks.onToolOutputMaxLinesChange(parseInt(newValue, 10));
                     break;
-                case "context-gc":
-                    callbacks.onContextGcChange(newValue === "true");
-                    break;
             }
         }, onCancel);
         this.addChild(this.settingsList);
@@ -343,16 +427,52 @@ class SelectSubmenu extends Container {
         this.selectList.handleInput(data);
     }
 }
+/**
+ * Token counts, grouped and exact.
+ *
+ * `formatTokens` (2.7k) exists for the fixed-width chrome, where a count must
+ * never grow the box it sits in. This pane is the opposite case: the point of
+ * showing a number here is to compare it with another one, and "2.7k" hides the
+ * difference between the tool that costs 2,710 and the one that costs 2,749.
+ */
+function tokenCount(tokens) {
+    return tokens.toString().replace(/\B(?=(\d{3})+(?!\d))/g, ",");
+}
+/**
+ * The fixed per-turn cost, as one line under the pane.
+ *
+ * Every byte of system prompt and active tool schema is re-sent on every
+ * request, and the pane is where that number is decided - so the pane is where
+ * it should be visible. It reports the *live* session: a tool toggle applies at
+ * once and moves it, while a setting that only takes effect next session (a tool
+ * group, the light preset) leaves it alone until then.
+ */
+function formatSurfaceLine(surface) {
+    const tools = `${surface.tools.length} tool${surface.tools.length === 1 ? "" : "s"}`;
+    return (`  Per-turn surface: ${tokenCount(surface.totalTokens)} tokens  ` +
+        `${theme.fg("dim", `(${tokenCount(surface.systemPromptTokens)} system prompt + ${tokenCount(surface.toolSchemaTokens)} schemas, ${tools})`)}`);
+}
 /**
  * Main settings selector component.
  */
 export class SettingsSelectorComponent extends Container {
     settingsList;
+    surfaceLine;
+    measureTokenSurface;
     constructor(config, callbacks) {
         super();
+        this.measureTokenSurface = config.measureTokenSurface;
         const supportsImages = getCapabilities().images;
         const followUpKey = keyDisplayText("app.message.followUp");
         let currentWarnings = { ...config.warnings };
+        // A row writes the user settings file; a key the project file also sets is
+        // merged over it on the next session, so say so rather than letting the row
+        // look like it took.
+        const projectPinned = new Set(config.projectPinnedSettings);
+        const pinnedNote = (key) => projectPinned.has(key)
+            ? ` This repo's .hoocode/settings.json sets ${key}, which overrides this row from the next session on.`
+            : "";
+        const initialSurface = config.measureTokenSurface?.();
         const toolsOn = config.tools.filter((t) => t.enabled).length;
         const toolsOff = config.tools.length - toolsOn;
         const items = [
@@ -487,7 +607,7 @@ export class SettingsSelectorComponent extends Container {
                 label: "Image width",
                 description: "Preferred inline image width in terminal cells",
                 currentValue: String(config.imageWidthCells),
-                values: ["60", "80", "120"],
+                values: presetValues([60, 80, 120], config.imageWidthCells),
             });
         }
         // Image auto-resize toggle (always available, affects both attached and read images)
@@ -507,8 +627,26 @@ export class SettingsSelectorComponent extends Container {
             currentValue: config.blockImages ? "true" : "false",
             values: ["true", "false"],
         });
-        // Skill commands toggle (insert after block-images)
-        const blockImagesIndex = items.findIndex((item) => item.id === "block-images");
+        // Context GC (insert after block-images), a leaf the Context category picks up.
+        items.splice(items.findIndex((item) => item.id === "block-images") + 1, 0, {
+            id: "context-gc",
+            label: "Context GC",
+            description: "Stub superseded read results (files later edited or re-read) out of the outgoing context.",
+            currentValue: config.contextGc ? "true" : "false",
+            values: ["true", "false"],
+        });
+        // The light preset (insert after context GC). Read at startup to pick the
+        // tool set and the system prompt, so it lands on the next session.
+        const blockImagesIdx = items.findIndex((item) => item.id === "context-gc");
+        items.splice(blockImagesIdx + 1, 0, {
+            id: "light",
+            label: "Light preset",
+            description: "Low-token preset for small or local models: read/write/edit/bash only with stripped schemas, a terse system prompt, and no subagents/TodoWrite/skills/context files. Applies on the next session.",
+            currentValue: config.light ? "true" : "false",
+            values: ["true", "false"],
+        });
+        // Skill commands toggle (insert after the light preset)
+        const blockImagesIndex = items.findIndex((item) => item.id === "light");
         items.splice(blockImagesIndex + 1, 0, {
             id: "skill-commands",
             label: "Skill commands",
@@ -516,19 +654,44 @@ export class SettingsSelectorComponent extends Container {
             currentValue: config.enableSkillCommands ? "true" : "false",
             values: ["true", "false"],
         });
-        // Plugin install scope (insert after skill-commands). Governs the
-        // autonomous InstallPlugin only — /plugin install asks per install.
+        // The autonomous plugin system's master switch (insert after skill-commands).
+        // One flag for the lifecycle tools and the reuse nudge, so both flip together.
         const skillCommandsIdx = items.findIndex((item) => item.id === "skill-commands");
         items.splice(skillCommandsIdx + 1, 0, {
+            id: "plugin-tools",
+            label: "Plugin system",
+            description: `Autonomous plugin system: the lifecycle tools (SearchPlugins, InstallPlugin, ...), ProposePlugin, and the plugin-reuse nudge. Tools arrive on the next session; the nudge follows at once.${pinnedNote("enablePluginTools")}`,
+            currentValue: config.enablePluginTools ? "true" : "false",
+            values: ["true", "false"],
+        });
+        // Plugin install scope (insert after the master switch). Governs the
+        // autonomous InstallPlugin only — /plugin install asks per install.
+        const pluginToolsIdx = items.findIndex((item) => item.id === "plugin-tools");
+        items.splice(pluginToolsIdx + 1, 0, {
             id: "plugin-install-scope",
             label: "Plugin install scope",
             description: "Where autonomous plugin installs go: user (~/.agents) or project (this repo, shared)",
             currentValue: config.pluginInstallScope,
             values: ["user", "project"],
         });
-        // Hardware cursor toggle (insert after plugin-install-scope)
-        const skillCommandsIndex = items.findIndex((item) => item.id === "plugin-install-scope");
-        items.splice(skillCommandsIndex + 1, 0, {
+        // Artifact platform targets (insert after plugin-install-scope). Set once and
+        // it holds for every later session: it is the `platform` setting, which
+        // `--platform` overrides for a single run.
+        const pluginScopeIdx = items.findIndex((item) => item.id === "plugin-install-scope");
+        let currentPlatforms = [...config.platform];
+        items.splice(pluginScopeIdx + 1, 0, {
+            id: "platform",
+            label: "Platform",
+            description: `Vendor layout(s) hoocode writes artifacts in: authored plugins and the /new-skill //new-agent //new-command scaffolds.${pinnedNote("platform")}`,
+            currentValue: platformSummary(currentPlatforms),
+            submenu: (_currentValue, done) => new PlatformSubmenu(currentPlatforms, (platforms) => {
+                currentPlatforms = platforms;
+                callbacks.onPlatformChange(platforms);
+            }, (summary) => done(summary)),
+        });
+        // Hardware cursor toggle (insert after the platform row)
+        const platformIndex = items.findIndex((item) => item.id === "platform");
+        items.splice(platformIndex + 1, 0, {
             id: "show-hardware-cursor",
             label: "Show hardware cursor",
             description: "Show the terminal cursor while still positioning it for IME support",
@@ -560,7 +723,7 @@ export class SettingsSelectorComponent extends Container {
             label: "Autocomplete max items",
             description: "Max visible items in autocomplete dropdown (3-20)",
             currentValue: String(config.autocompleteMaxVisible),
-            values: ["3", "5", "7", "10", "15", "20"],
+            values: presetValues([3, 5, 7, 10, 15, 20], config.autocompleteMaxVisible),
         });
         // Clear on shrink toggle (insert after autocomplete-max-visible)
         const autocompleteIndex = items.findIndex((item) => item.id === "autocomplete-max-visible");
@@ -587,7 +750,7 @@ export class SettingsSelectorComponent extends Container {
             label: "Voice silence window",
             description: "Trailing-silence (ms) before voice capture auto-stops (300-10000). Env: VOICETOOLS_SILENCE_MS.",
             currentValue: String(config.voiceSilenceMs),
-            values: ["300", "500", "800", "1200", "2000", "3000", "5000", "8000", "10000"],
+            values: presetValues([300, 500, 800, 1200, 2000, 3000, 5000, 8000, 10000], config.voiceSilenceMs),
         });
         // Webtools request timeout (insert after voice-silence-ms)
         const voiceSilenceIndex = items.findIndex((item) => item.id === "voice-silence-ms");
@@ -596,7 +759,7 @@ export class SettingsSelectorComponent extends Container {
             label: "Web tools timeout",
             description: "Per-request timeout (secs) for webfetch/websearch (1-120). Env: HOOCODE_WEBTOOLS_TIMEOUT.",
             currentValue: String(config.webtoolsTimeoutSecs),
-            values: ["5", "10", "15", "30", "60", "120"],
+            values: presetValues([5, 10, 15, 30, 60, 120], config.webtoolsTimeoutSecs),
         });
         // The /learn thresholds, appended as leaf rows and gathered into their own
         // category below. They are written to the user settings.json, which /learn
@@ -615,30 +778,28 @@ export class SettingsSelectorComponent extends Container {
             {
                 id: "tools",
                 label: "Tools",
-                description: "Enable/disable tools and tool groups (web, semantic search). Changes persist across sessions.",
+                description: "Enable/disable tools and tool groups (web, semantic search), each priced by what its schema costs per turn. Changes persist across sessions.",
                 currentValue: toolsOff > 0 ? `${toolsOn} on · ${toolsOff} off` : `${toolsOn} on`,
-                submenu: (_currentValue, done) => new ToolsSubmenu(config.tools, config.toolGroups, (name, enabled) => callbacks.onToolEnabledChange(name, enabled), (id, enabled) => callbacks.onToolGroupChange(id, enabled), () => done()),
+                valueSuffix: initialSurface ? `${tokenCount(initialSurface.toolSchemaTokens)} tok/turn` : undefined,
+                submenu: (_currentValue, done) => new ToolsSubmenu(config.tools, config.toolGroups, (name, enabled) => {
+                    callbacks.onToolEnabledChange(name, enabled);
+                    // Applied live by the host, so the surface below is already stale.
+                    this.refreshTokenSurface();
+                }, (id, enabled) => callbacks.onToolGroupChange(id, enabled), () => done()),
             },
             {
-                id: "tool-output-display",
-                label: "Tool output display",
-                description: "How tool results render. 'standard': shown (expandable). 'collapsed': hidden. 'peek': hidden with a ▸ reveal caret (press the expand key to reveal).",
+                id: "tool-output",
+                label: "Tool output",
+                description: "How much of a tool result is rendered, and where it is truncated.",
                 currentValue: config.toolOutputDisplay,
-                values: ["standard", "collapsed", "peek"],
-            },
-            {
-                id: "tool-settings",
-                label: "Tool settings",
-                description: "Per-tool runtime settings: output truncation caps and context garbage collection.",
-                currentValue: "configure",
                 submenu: (_currentValue, done) => new ToolSettingsSubmenu({
+                    toolOutputDisplay: config.toolOutputDisplay,
                     toolOutputMaxBytes: config.toolOutputMaxBytes,
                     toolOutputMaxLines: config.toolOutputMaxLines,
-                    contextGc: config.contextGc,
                 }, {
+                    onToolOutputDisplayChange: callbacks.onToolOutputDisplayChange,
                     onToolOutputMaxBytesChange: callbacks.onToolOutputMaxBytesChange,
                     onToolOutputMaxLinesChange: callbacks.onToolOutputMaxLinesChange,
-                    onContextGcChange: callbacks.onContextGcChange,
                 }, () => done()),
             },
         ];
@@ -660,9 +821,6 @@ export class SettingsSelectorComponent extends Container {
                 case "autocompact":
                     callbacks.onAutoCompactChange(newValue === "true");
                     break;
-                case "tool-output-display":
-                    callbacks.onToolOutputDisplayChange(newValue);
-                    break;
                 case "show-images":
                     callbacks.onShowImagesChange(newValue === "true");
                     break;
@@ -678,9 +836,19 @@ export class SettingsSelectorComponent extends Container {
                 case "skill-commands":
                     callbacks.onEnableSkillCommandsChange(newValue === "true");
                     break;
+                case "light":
+                    callbacks.onLightChange(newValue === "true");
+                    break;
+                case "plugin-tools":
+                    callbacks.onEnablePluginToolsChange(newValue === "true");
+                    break;
                 case "plugin-install-scope":
                     callbacks.onPluginInstallScopeChange(newValue);
                     break;
+                case "platform":
+                    // Display only. Each toggle inside the submenu already applied itself;
+                    // closing it hands back the summary purely to refresh this row's value.
+                    break;
                 case "steering-mode":
                     callbacks.onSteeringModeChange(newValue);
                     break;
@@ -739,6 +907,10 @@ export class SettingsSelectorComponent extends Container {
                         callbacks.onLearnSettingChange(id, parseInt(newValue, 10));
                     break;
             }
+            // Most rows leave the per-turn surface alone; the ones that do not (a tool
+            // toggle, anything that rebuilds the system prompt) move it immediately.
+            // Re-measuring after every change is cheaper than knowing which is which.
+            this.refreshTokenSurface();
         };
         // Partition the flat leaf settings into named category submenus so the
         // top level stays short. `items` holds autocompact + every leaf setting.
@@ -751,12 +923,21 @@ export class SettingsSelectorComponent extends Container {
                 label,
                 description,
                 currentValue: `${members.length} setting${members.length === 1 ? "" : "s"}`,
+                // Categories shortened the top level but also hid every setting from
+                // its search: searching "theme" matched no category label. The member
+                // labels ride along as search text so the query lands on the category
+                // that holds the setting.
+                keywords: members.map((member) => member.label).join(" "),
                 submenu: (_currentValue, done) => new CategorySubmenu(members, applyChange, () => done()),
             };
         };
         const topItems = [
-            ...(byId.has("autocompact") ? [byId.get("autocompact")] : []),
             ...toolFlagGroup,
+            // What the model is sent, and how much of it. Auto-compact used to sit
+            // alone at the top level and context GC was filed under tool settings,
+            // which left the three settings that decide the token budget in three
+            // different places - with the light preset in none of them.
+            categoryRow("cat-context", "Context", "What the model is sent and how much of it: compaction, superseded reads, and the low-token preset.", ["autocompact", "context-gc", "light"]),
             categoryRow("cat-behavior", "Behavior", "Agent and session behavior: steering, follow-up, thinking, escape, tree filter, transport.", ["steering-mode", "follow-up-mode", "thinking", "double-escape-action", "tree-filter-mode", "transport"]),
             categoryRow("cat-interface", "Interface", "Appearance and editor: theme, thinking visibility, cursor, border, padding, autocomplete, terminal.", [
                 "theme",
@@ -768,6 +949,9 @@ export class SettingsSelectorComponent extends Container {
                 "clear-on-shrink",
                 "terminal-progress",
             ]),
+            // Artifact production. `plugin-install-scope` had no category and was
+            // therefore unreachable from the pane despite having a live callback.
+            categoryRow("cat-plugins", "Plugins", "The autonomous plugin system's master switch, the vendor layout hoocode writes, and where autonomous installs land.", ["plugin-tools", "platform", "plugin-install-scope"]),
             categoryRow("cat-images", "Images", "Inline image rendering and resizing.", [
                 "show-images",
                 "image-width-cells",
@@ -792,8 +976,18 @@ export class SettingsSelectorComponent extends Container {
             enableSearch: true,
         });
         this.addChild(this.settingsList);
+        if (initialSurface) {
+            this.surfaceLine = new Text(formatSurfaceLine(initialSurface), 0, 0);
+            this.addChild(this.surfaceLine);
+        }
         this.addChild(new DynamicBorder());
     }
+    /** Re-price the pane after a change that may have altered what each turn sends. */
+    refreshTokenSurface() {
+        if (!this.surfaceLine || !this.measureTokenSurface)
+            return;
+        this.surfaceLine.setText(formatSurfaceLine(this.measureTokenSurface()));
+    }
     getSettingsList() {
         return this.settingsList;
     }