@mariozechner/pi-coding-agent 0.51.1 → 0.51.3

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

@@ -9,13 +9,14 @@ 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, isBunBinary, isBunRuntime, VERSION, } from "../../config.js";
+import { APP_NAME, getAuthPath, getDebugLogPath, getShareViewerUrl, getUpdateInstruction, VERSION, } from "../../config.js";
 import { parseSkillBlock } from "../../core/agent-session.js";
 import { FooterDataProvider } from "../../core/footer-data-provider.js";
 import { KeybindingsManager } from "../../core/keybindings.js";
 import { createCompactionSummaryMessage } from "../../core/messages.js";
 import { resolveModelScope } from "../../core/model-resolver.js";
 import { SessionManager } from "../../core/session-manager.js";
+import { BUILTIN_SLASH_COMMANDS } from "../../core/slash-commands.js";
 import { getChangelogPath, getNewEntries, parseChangelog } from "../../utils/changelog.js";
 import { copyToClipboard } from "../../utils/clipboard.js";
 import { extensionForImageMimeType, readClipboardImage } from "../../utils/clipboard-image.js";
@@ -166,59 +167,44 @@ export class InteractiveMode {
     }
     setupAutocomplete(fdPath) {
         // Define commands for autocomplete
-        const slashCommands = [
-            { name: "settings", description: "Open settings menu" },
-            {
-                name: "model",
-                description: "Select model (opens selector UI)",
-                getArgumentCompletions: (prefix) => {
-                    // Get available models (scoped or from registry)
-                    const models = this.session.scopedModels.length > 0
-                        ? this.session.scopedModels.map((s) => s.model)
-                        : this.session.modelRegistry.getAvailable();
-                    if (models.length === 0)
-                        return null;
-                    // Create items with provider/id format
-                    const items = models.map((m) => ({
-                        id: m.id,
-                        provider: m.provider,
-                        label: `${m.provider}/${m.id}`,
-                    }));
-                    // Fuzzy filter by model ID + provider (allows "opus anthropic" to match)
-                    const filtered = fuzzyFilter(items, prefix, (item) => `${item.id} ${item.provider}`);
-                    if (filtered.length === 0)
-                        return null;
-                    return filtered.map((item) => ({
-                        value: item.label,
-                        label: item.id,
-                        description: item.provider,
-                    }));
-                },
-            },
-            { name: "scoped-models", description: "Enable/disable models for Ctrl+P cycling" },
-            { name: "export", description: "Export session to HTML file" },
-            { name: "share", description: "Share session as a secret GitHub gist" },
-            { name: "copy", description: "Copy last agent message to clipboard" },
-            { name: "name", description: "Set session display name" },
-            { name: "session", description: "Show session info and stats" },
-            { name: "changelog", description: "Show changelog entries" },
-            { name: "hotkeys", description: "Show all keyboard shortcuts" },
-            { name: "fork", description: "Create a new fork from a previous message" },
-            { name: "tree", description: "Navigate session tree (switch branches)" },
-            { name: "login", description: "Login with OAuth provider" },
-            { name: "logout", description: "Logout from OAuth provider" },
-            { name: "new", description: "Start a new session" },
-            { name: "compact", description: "Manually compact the session context" },
-            { name: "resume", description: "Resume a different session" },
-            { name: "reload", description: "Reload extensions, skills, prompts, and themes" },
-        ];
+        const slashCommands = BUILTIN_SLASH_COMMANDS.map((command) => ({
+            name: command.name,
+            description: command.description,
+        }));
+        const modelCommand = slashCommands.find((command) => command.name === "model");
+        if (modelCommand) {
+            modelCommand.getArgumentCompletions = (prefix) => {
+                // Get available models (scoped or from registry)
+                const models = this.session.scopedModels.length > 0
+                    ? this.session.scopedModels.map((s) => s.model)
+                    : this.session.modelRegistry.getAvailable();
+                if (models.length === 0)
+                    return null;
+                // Create items with provider/id format
+                const items = models.map((m) => ({
+                    id: m.id,
+                    provider: m.provider,
+                    label: `${m.provider}/${m.id}`,
+                }));
+                // Fuzzy filter by model ID + provider (allows "opus anthropic" to match)
+                const filtered = fuzzyFilter(items, prefix, (item) => `${item.id} ${item.provider}`);
+                if (filtered.length === 0)
+                    return null;
+                return filtered.map((item) => ({
+                    value: item.label,
+                    label: item.id,
+                    description: item.provider,
+                }));
+            };
+        }
         // Convert prompt templates to SlashCommand format for autocomplete
         const templateCommands = this.session.promptTemplates.map((cmd) => ({
             name: cmd.name,
             description: cmd.description,
         }));
         // Convert extension commands to SlashCommand format
-        const extensionCommands = (this.session.extensionRunner?.getRegisteredCommands() ?? []).map((cmd) => ({
+        const builtinCommandNames = new Set(slashCommands.map((c) => c.name));
+        const extensionCommands = (this.session.extensionRunner?.getRegisteredCommands(builtinCommandNames) ?? []).map((cmd) => ({
             name: cmd.name,
             description: cmd.description ?? "(extension command)",
             getArgumentCompletions: cmd.getArgumentCompletions,
@@ -724,6 +710,8 @@ export class InteractiveMode {
                 extensionDiagnostics.push({ type: "error", message: error.error, path: error.path });
             }
         }
+        const commandDiagnostics = this.session.extensionRunner?.getCommandDiagnostics() ?? [];
+        extensionDiagnostics.push(...commandDiagnostics);
         const shortcutDiagnostics = this.session.extensionRunner?.getShortcutDiagnostics() ?? [];
         extensionDiagnostics.push(...shortcutDiagnostics);
         if (extensionDiagnostics.length > 0) {
@@ -1112,6 +1100,8 @@ export class InteractiveMode {
                 }
                 return result;
             },
+            getToolsExpanded: () => this.toolOutputExpanded,
+            setToolsExpanded: (expanded) => this.setToolsExpanded(expanded),
         };
     }
     /**
@@ -2110,6 +2100,9 @@ export class InteractiveMode {
         // Wait for any pending renders to complete
         // requestRender() uses process.nextTick(), so we wait one tick
         await new Promise((resolve) => process.nextTick(resolve));
+        // Drain any in-flight Kitty key release events before stopping.
+        // This prevents escape sequences from leaking to the parent shell over slow SSH.
+        await this.ui.terminal.drainInput(1000);
         this.stop();
         process.exit(0);
     }
@@ -2211,10 +2204,13 @@ export class InteractiveMode {
         }
     }
     toggleToolOutputExpansion() {
-        this.toolOutputExpanded = !this.toolOutputExpanded;
+        this.setToolsExpanded(!this.toolOutputExpanded);
+    }
+    setToolsExpanded(expanded) {
+        this.toolOutputExpanded = expanded;
         for (const child of this.chatContainer.children) {
             if (isExpandable(child)) {
-                child.setExpanded(this.toolOutputExpanded);
+                child.setExpanded(expanded);
             }
         }
         this.ui.requestRender();
@@ -2292,9 +2288,7 @@ export class InteractiveMode {
         this.ui.requestRender();
     }
     showNewVersionNotification(newVersion) {
-        const action = isBunBinary
-            ? `Download from: ${theme.fg("accent", "https://github.com/badlogic/pi-mono/releases/latest")}`
-            : `Run: ${theme.fg("accent", `${isBunRuntime ? "bun" : "npm"} install -g @mariozechner/pi-coding-agent`)}`;
+        const action = theme.fg("accent", getUpdateInstruction("@mariozechner/pi-coding-agent"));
         const updateInstruction = theme.fg("muted", `New version ${newVersion} is available. `) + action;
         const changelogUrl = theme.fg("accent", "https://github.com/badlogic/pi-mono/blob/main/packages/coding-agent/CHANGELOG.md");
         const changelogLine = theme.fg("muted", "Changelog: ") + changelogUrl;