@agi-fans/dsh-tui 0.15.0 → 0.16.0

package/lib/runtime/provider-local.js

@@ -33,7 +33,8 @@ import { applyTranscriptSearchEvent, createTranscriptSearch, searchBlockIndexes,
 import { nextRevealStep, revealStreamingAssistant, streamingAssistantKey, streamingAssistantUnits, } from "../views/streaming-reveal.js";
 import { flushPending, parseKeys } from "../input/keys.js";
 import { MainScreenRenderer } from "../chrome/main-screen-renderer.js";
-import { createTheme, detectTrueColor, parseThemeName } from "../chrome/theme.js";
+import { colorDisabledByEnv, createTheme, detectTrueColor, parseThemeName } from "../chrome/theme.js";
+import { renderTool } from "../chrome/tool-renderers.js";
 import { TUI_SETTINGS_NAMESPACE, TuiSettingsSchema } from "../session/tui-settings.js";
 import { defaultStatusBarConfig, resolveStatusBarConfig } from "../chrome/status-config.js";
 import { HistoryStore } from "../views/history-store.js";
@@ -81,6 +82,18 @@ function detectTerminalProfile() {
     }
     return process.platform === 'win32' ? 'conpty' : 'direct';
 }
+/**
+ * Decide whether SGR is emitted. An explicit preference — plugin config or a
+ * value the user picked in `/settings` — always wins, so a non-empty `NO_COLOR`
+ * can never block someone who asked for color. Without a preference, a
+ * non-empty `NO_COLOR` or `FORCE_COLOR=0` suppresses color and the output
+ * stream's tty-ness decides.
+ */
+function resolveColors(preference, isTty) {
+    if (preference !== undefined)
+        return preference;
+    return isTty && !colorDisabledByEnv();
+}
 export class LocalTui {
     #deferInitialRender = false;
     #terminalProfile;
@@ -187,7 +200,12 @@ export class LocalTui {
     #subagentLauncherFocused = false;
     #inspected;
     #promptDocument;
-    #trueColor;
+    /**
+     * 24-bit switch for the current color preference, re-derived whenever that
+     * preference changes so a session that starts colorless can still enable
+     * hexadecimal palettes without a restart.
+     */
+    #trueColor = false;
     #copy;
     #readClipboard;
     #readClipboardImage;
@@ -212,7 +230,8 @@ export class LocalTui {
     /**
      * @param term - terminal surface (injectable for tests).
      * @param model - model label for the status line.
-     * @param colors - SGR styling switch.
+     * @param colors - SGR styling switch; `undefined` derives one from the
+     * output stream's tty-ness and the environment.
      * @param themeName - shipped palette.
      * @param copy - clipboard writer (defaults to the platform tool).
      * @param paths - cwd/home/listing used by `@` and path autocomplete.
@@ -220,7 +239,7 @@ export class LocalTui {
     constructor(term, model, colors, themeName = 'dark', copy = copyToClipboard, paths = {}) {
         this.#term = term;
         this.#model = model;
-        this.#colors = colors;
+        this.#colors = resolveColors(colors, term.output.isTTY === true);
         this.#themeName = themeName;
         this.#copy = copy;
         this.#readClipboard = paths.readClipboard ?? readFromClipboard;
@@ -245,7 +264,7 @@ export class LocalTui {
         this.#terminalProfile = paths.terminalProfile ?? detectTerminalProfile();
         this.#resizeDebounceMs = Math.max(0, paths.resizeDebounceMs ?? 120);
         this.#streamRenderMs = Math.max(0, paths.streamRenderMs ?? 8);
-        this.#trueColor = colors && detectTrueColor();
+        this.#syncTrueColor();
         this.#tty = term.input.isTTY === true;
         this.#pwd = shortenPath(project.root);
         this.#branch = project.gitLabel;
@@ -354,6 +373,13 @@ export class LocalTui {
             this.#render();
     }
     setSubagents(roster) {
+        const previous = this.#subagents?.agents ?? [];
+        const incoming = roster?.agents ?? [];
+        const statusChanged = previous.length !== incoming.length || incoming.some((agent, index) => {
+            const before = previous[index];
+            return before === undefined || before.id !== agent.id || before.label !== agent.label
+                || before.phase !== agent.phase || before.depth !== agent.depth || before.mode !== agent.mode;
+        });
         this.#subagents = roster === undefined
             ? undefined
             : { agents: roster.agents.map(agent => ({ ...agent, activity: [...agent.activity] })) };
@@ -378,11 +404,17 @@ export class LocalTui {
                 };
             }
         }
+        if (!statusChanged && this.#agentHub === null)
+            return;
         this.#syncTick();
         if (this.#tty)
-            this.#render();
+            this.#scheduleStreamRender();
     }
     setInspectedSubagent(inspected) {
+        if (this.#inspected?.id === inspected?.id && this.#inspected?.label === inspected?.label
+            && this.#inspected?.phase === inspected?.phase && this.#inspected?.mode === inspected?.mode
+            && this.#inspected?.writable === inspected?.writable)
+            return;
         this.#inspected = inspected === undefined ? undefined : { ...inspected };
         if (inspected !== undefined) {
             this.#inspectEscapeFenceUntil = 0;
@@ -433,6 +465,31 @@ export class LocalTui {
     setImageValidator(validate) {
         this.#validateImageDraft = validate;
     }
+    toolCallContext(callId) {
+        const block = this.#state.blocks.findLast(candidate => candidate.kind === 'tool' && String(candidate.callId) === callId);
+        if (block?.kind !== 'tool')
+            return undefined;
+        // Reuse the card renderer so a decision explains the call the same way the
+        // transcript already does. The approval prompt renders its detail on one
+        // line, so the parts are joined here rather than handed over as rows.
+        const view = renderTool({
+            name: block.name,
+            arguments: block.args,
+            output: block.output,
+            status: block.status,
+            expanded: false,
+            ...(block.presentation === undefined ? {} : { presentation: block.presentation }),
+        });
+        const detail = view.summary === undefined || view.summary === ''
+            ? view.input.join(' ').replace(/\s+/gu, ' ').trim()
+            : '';
+        // A card with a summary already says what the call is. Without one the raw
+        // argument text is all there is, and its pretty-printed form would reduce to
+        // a lone brace on a single row, so the whole body is folded into one line.
+        return [view.title, view.summary, detail === '' ? undefined : detail]
+            .filter((part) => part !== undefined && part !== '')
+            .join(' · ');
+    }
     notice(text, options = {}) {
         const block = {
             kind: 'notice',
@@ -538,11 +595,19 @@ export class LocalTui {
         this.#writtenWindowTitle = title;
         this.#term.output.write(`\x1b]2;${title}\x07`);
     }
+    /**
+     * Re-derive the 24-bit switch from the active color preference. Every path
+     * that changes `#colors` calls this so `/settings` takes effect immediately.
+     */
+    #syncTrueColor() {
+        this.#trueColor = this.#colors && detectTrueColor();
+    }
     /** Apply prefs loaded from the settings document (does not persist). */
     applyStoredPrefs(prefs) {
         const previousMotion = this.#motion;
         this.#themeName = prefs.theme;
         this.#colors = prefs.colors;
+        this.#syncTrueColor();
         this.#motion = prefs.motion ?? 'full';
         this.#terminalProgress = prefs.terminalProgress ?? false;
         this.#expandTools = prefs.expandTools;
@@ -1745,6 +1810,7 @@ export class LocalTui {
         const expandChanged = prefs.expandTools !== this.#expandTools;
         this.#themeName = prefs.theme;
         this.#colors = prefs.colors;
+        this.#syncTrueColor();
         const previousMotion = this.#motion;
         this.#motion = prefs.motion ?? 'full';
         this.#terminalProgress = prefs.terminalProgress ?? false;
@@ -2315,7 +2381,7 @@ export class LocalTui {
                         formatHelpText(this.#commands()),
                         '',
                         full
-                            ? `**Keyboard Shortcuts · ${hotkeyCount(this.#keybindings)} bindings**`
+                            ? `**Keyboard Shortcuts · ${hotkeyCount(this.#keybindings)} shortcuts**`
                             : '**Essential Shortcuts**',
                         '',
                         full
@@ -2617,7 +2683,7 @@ export function apply(ctx, config) {
             return () => { process.stdout.removeListener('resize', listener); };
         },
     };
-    const tui = new LocalTui(term, config.model, config.colors ?? term.output.isTTY === true, parseThemeName(config.theme), copyToClipboard, {
+    const tui = new LocalTui(term, config.model, config.colors, parseThemeName(config.theme), copyToClipboard, {
         deferInitialRender: true,
         terminalProfile,
         alternateScreenOverlays: terminalProfile === 'direct',
@@ -2627,7 +2693,7 @@ export function apply(ctx, config) {
     ctx.provide(TUI_SERVICE, tui);
     ctx.effect(() => () => { tui.dispose(); });
     ctx.inject(['settings'], (settingsCtx) => {
-        const scope = settingsCtx.settings.register(TUI_SETTINGS_NAMESPACE, TuiSettingsSchema, { base: { theme: parseThemeName(config.theme), colors: config.colors ?? term.output.isTTY === true, expandTools: false } });
+        const scope = settingsCtx.settings.register(TUI_SETTINGS_NAMESPACE, TuiSettingsSchema, { base: { theme: parseThemeName(config.theme), colors: resolveColors(config.colors, term.output.isTTY === true), expandTools: false } });
         tui.applyStoredPrefs(scope.get());
         tui.setPrefsPersist((prefs) => { void scope.update(prefs); });
         scope.watch((next) => { tui.applyStoredPrefs(next); });