@kolisachint/hoocode-agent 0.5.29 → 0.5.30

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

@@ -197,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];
@@ -243,39 +247,45 @@ function byteLabels(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: 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: presetValues([200, 400, 800, 1600, 3200], config.toolOutputMaxLines),
             },
-            {
-                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"],
-            },
         ];
         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);
                     // A hand-set cap has no preset entry; its label is "<n> KB" by
@@ -288,9 +298,6 @@ class ToolSettingsSubmenu extends Container {
                 case "output-max-lines":
                     callbacks.onToolOutputMaxLinesChange(parseInt(newValue, 10));
                     break;
-                case "context-gc":
-                    callbacks.onContextGcChange(newValue === "true");
-                    break;
             }
         }, onCancel);
         this.addChild(this.settingsList);
@@ -420,13 +427,41 @@ 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 };
@@ -437,6 +472,7 @@ export class SettingsSelectorComponent extends Container {
         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 = [
@@ -591,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",
@@ -724,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()),
             },
         ];
@@ -769,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;
@@ -787,6 +836,9 @@ 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;
@@ -855,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.
@@ -876,8 +932,12 @@ export class SettingsSelectorComponent extends Container {
             };
         };
         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",
@@ -916,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;
     }