@mariozechner/pi-coding-agent 0.55.3 → 0.56.0

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

@@ -6,7 +6,6 @@ import * as crypto from "node:crypto";
 import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
-import { getOAuthProviders, } from "@mariozechner/pi-ai";
 import { CombinedAutocompleteProvider, Container, fuzzyFilter, Loader, Markdown, matchesKey, ProcessTerminal, Spacer, Text, TruncatedText, TUI, visibleWidth, } from "@mariozechner/pi-tui";
 import { spawn, spawnSync } from "child_process";
 import { APP_NAME, getAuthPath, getDebugLogPath, getShareViewerUrl, getUpdateInstruction, VERSION, } from "../../config.js";
@@ -1725,7 +1724,6 @@ export class InteractiveMode {
                     for (const content of this.streamingMessage.content) {
                         if (content.type === "toolCall") {
                             if (!this.pendingTools.has(content.id)) {
-                                this.chatContainer.addChild(new Text("", 0, 0));
                                 const component = new ToolExecutionComponent(content.name, content.arguments, {
                                     showImages: this.settingsManager.getShowImages(),
                                 }, this.getRegisteredToolDefinition(content.name), this.ui);
@@ -2780,12 +2778,10 @@ export class InteractiveMode {
         // Helper to update session's scoped models (session-only, no persist)
         const updateSessionModels = async (enabledIds) => {
             if (enabledIds.size > 0 && enabledIds.size < allModels.length) {
-                // Use current session thinking level, not settings default
-                const currentThinkingLevel = this.session.thinkingLevel;
                 const newScopedModels = await resolveModelScope(Array.from(enabledIds), this.session.modelRegistry);
                 this.session.setScopedModels(newScopedModels.map((sm) => ({
                     model: sm.model,
-                    thinkingLevel: sm.thinkingLevel ?? currentThinkingLevel,
+                    thinkingLevel: sm.thinkingLevel,
                 })));
             }
             else {
@@ -2899,27 +2895,30 @@ export class InteractiveMode {
                 // Loop until user makes a complete choice or cancels to tree
                 let wantsSummary = false;
                 let customInstructions;
-                while (true) {
-                    const summaryChoice = await this.showExtensionSelector("Summarize branch?", [
-                        "No summary",
-                        "Summarize",
-                        "Summarize with custom prompt",
-                    ]);
-                    if (summaryChoice === undefined) {
-                        // User pressed escape - re-show tree selector with same selection
-                        this.showTreeSelector(entryId);
-                        return;
-                    }
-                    wantsSummary = summaryChoice !== "No summary";
-                    if (summaryChoice === "Summarize with custom prompt") {
-                        customInstructions = await this.showExtensionEditor("Custom summarization instructions");
-                        if (customInstructions === undefined) {
-                            // User cancelled - loop back to summary selector
-                            continue;
+                // Check if we should skip the prompt (user preference to always default to no summary)
+                if (!this.settingsManager.getBranchSummarySkipPrompt()) {
+                    while (true) {
+                        const summaryChoice = await this.showExtensionSelector("Summarize branch?", [
+                            "No summary",
+                            "Summarize",
+                            "Summarize with custom prompt",
+                        ]);
+                        if (summaryChoice === undefined) {
+                            // User pressed escape - re-show tree selector with same selection
+                            this.showTreeSelector(entryId);
+                            return;
                         }
+                        wantsSummary = summaryChoice !== "No summary";
+                        if (summaryChoice === "Summarize with custom prompt") {
+                            customInstructions = await this.showExtensionEditor("Custom summarization instructions");
+                            if (customInstructions === undefined) {
+                                // User cancelled - loop back to summary selector
+                                continue;
+                            }
+                        }
+                        // User made a complete choice
+                        break;
                     }
-                    // User made a complete choice
-                    break;
                 }
                 // Set up escape handler and loader if summarizing
                 let summaryLoader;
@@ -3037,7 +3036,9 @@ export class InteractiveMode {
                 }
                 else {
                     // Logout flow
-                    const providerInfo = getOAuthProviders().find((p) => p.id === providerId);
+                    const providerInfo = this.session.modelRegistry.authStorage
+                        .getOAuthProviders()
+                        .find((p) => p.id === providerId);
                     const providerName = providerInfo?.name || providerId;
                     try {
                         this.session.modelRegistry.authStorage.logout(providerId);
@@ -3057,7 +3058,7 @@ export class InteractiveMode {
         });
     }
     async showLoginDialog(providerId) {
-        const providerInfo = getOAuthProviders().find((p) => p.id === providerId);
+        const providerInfo = this.session.modelRegistry.authStorage.getOAuthProviders().find((p) => p.id === providerId);
         const providerName = providerInfo?.name || providerId;
         // Providers that use callback servers (can paste redirect URL)
         const usesCallbackServer = providerInfo?.usesCallbackServer ?? false;