@kolisachint/hoocode-agent 0.5.32 → 0.5.34

package/dist/modes/interactive/interactive-mode.js

@@ -1941,7 +1941,7 @@ export class InteractiveMode {
                     this.ui.requestRender();
                 }
                 else if (event.message.role === "assistant") {
-                    this.streamingComponent = new AssistantMessageComponent(undefined, this.hideThinkingBlock, this.getMarkdownThemeWithSettings(), this.hiddenThinkingLabel);
+                    this.streamingComponent = new AssistantMessageComponent(undefined, this.thinkingDisplayForView(), this.getMarkdownThemeWithSettings(), this.hiddenThinkingLabel);
                     this.streamingMessage = event.message;
                     this.sawTextInCurrentMessage = false;
                     this.chatContainer.addChild(this.streamingComponent);
@@ -2289,7 +2289,7 @@ export class InteractiveMode {
                 break;
             }
             case "assistant": {
-                const assistantComponent = new AssistantMessageComponent(message, this.hideThinkingBlock, this.getMarkdownThemeWithSettings(), this.hiddenThinkingLabel);
+                const assistantComponent = new AssistantMessageComponent(message, this.thinkingDisplayForView(), this.getMarkdownThemeWithSettings(), this.hiddenThinkingLabel);
                 this.chatContainer.addChild(assistantComponent);
                 break;
             }
@@ -2681,6 +2681,10 @@ export class InteractiveMode {
             for (let i = chains.length - 1; i >= 0; i--) {
                 if (chains[i].isEmpty || chains[i].isOpened === wantOpen)
                     continue;
+                // A chain of one is never summarised, so there is nothing for unfold to
+                // reveal; skipping it keeps the press moving to a chain that will change.
+                if (wantOpen && !chains[i].isSummarised)
+                    continue;
                 chains[i].setOpened(wantOpen);
                 this.ui.requestRender();
                 return;
@@ -2710,15 +2714,46 @@ export class InteractiveMode {
         const next = cycleToolOutputView(this.toolOutputView, direction);
         this.applyToolOutputView(next);
     }
+    /**
+     * How much of a thinking trace this view shows.
+     *
+     * Radar drops them outright, regardless of the setting. The dial is one
+     * question asked once — "how much do I ever want to see" — and a view whose
+     * whole job is to fold a run of tool calls down to a row cannot then spend
+     * forty lines on the reasoning that led to it. Folding to the label is not
+     * enough either: a message that only thought and called tools has nothing
+     * else on screen once its calls join the chain, so its label would stand
+     * alone under the summary and one row per chain would become one row plus a
+     * label per call.
+     *
+     * It is also what keeps a running chain on screen. A chain stays open across
+     * a thinking block (thinking is not text, so it is not a chain boundary), and
+     * a trace rendering below pushes the chain's own summary line off the top —
+     * where every subsequent call rewrites a line above the viewport and forces
+     * the full redraw that clears terminal scrollback.
+     */
+    thinkingDisplayForView() {
+        if (this.toolOutputView === "radar")
+            return "omit";
+        return this.hideThinkingBlock ? "label" : "full";
+    }
     applyToolOutputView(view) {
+        const previousThinking = this.thinkingDisplayForView();
         this.toolOutputView = view;
         this.settingsManager.setToolOutputView(view);
         this.footer.setToolOutputView(view);
+        const thinking = this.thinkingDisplayForView();
         for (const child of this.chatContainer.children) {
             if (child instanceof ToolChainComponent)
                 child.setView(view);
             else if (child instanceof ToolExecutionComponent)
                 child.setView(view);
+            else if (child instanceof AssistantMessageComponent && thinking !== previousThinking) {
+                child.setThinkingDisplay(thinking);
+            }
+        }
+        if (this.streamingComponent && thinking !== previousThinking) {
+            this.streamingComponent.setThinkingDisplay(thinking);
         }
         this.ui.requestRender();
     }
@@ -2770,11 +2805,15 @@ export class InteractiveMode {
         this.rebuildChatFromMessages();
         // If streaming, re-add the streaming component with updated visibility and re-render
         if (this.streamingComponent && this.streamingMessage) {
-            this.streamingComponent.setHideThinkingBlock(this.hideThinkingBlock);
+            this.streamingComponent.setThinkingDisplay(this.thinkingDisplayForView());
             this.streamingComponent.updateContent(this.streamingMessage);
             this.chatContainer.addChild(this.streamingComponent);
         }
-        this.showStatus(`Thinking blocks: ${this.hideThinkingBlock ? "hidden" : "visible"}`);
+        // Radar overrides the setting, so say so rather than claiming a change the
+        // screen does not show.
+        this.showStatus(!this.hideThinkingBlock && this.toolOutputView === "radar"
+            ? "Thinking blocks: visible (radar hides them)"
+            : `Thinking blocks: ${this.hideThinkingBlock ? "hidden" : "visible"}`);
     }
     openExternalEditor() {
         // Determine editor (respect $VISUAL, then $EDITOR)
@@ -3166,9 +3205,10 @@ export class InteractiveMode {
                 onHideThinkingBlockChange: (hidden) => {
                     this.hideThinkingBlock = hidden;
                     this.settingsManager.setHideThinkingBlock(hidden);
+                    const effective = this.thinkingDisplayForView();
                     for (const child of this.chatContainer.children) {
                         if (child instanceof AssistantMessageComponent) {
-                            child.setHideThinkingBlock(hidden);
+                            child.setThinkingDisplay(effective);
                         }
                     }
                     this.chatContainer.clear();