@mariozechner/pi-coding-agent 0.46.0 → 0.48.0

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

@@ -65,6 +65,7 @@ export class InteractiveMode {
     keybindings;
     version;
     isInitialized = false;
+    hasRenderedInitialMessages = false;
     onInputCallback;
     loadingAnimation = undefined;
     defaultWorkingMessage = "Working...";
@@ -136,7 +137,8 @@ export class InteractiveMode {
         this.statusContainer = new Container();
         this.widgetContainer = new Container();
         this.keybindings = KeybindingsManager.create();
-        this.defaultEditor = new CustomEditor(getEditorTheme(), this.keybindings);
+        const editorPaddingX = this.settingsManager.getEditorPaddingX();
+        this.defaultEditor = new CustomEditor(this.ui, getEditorTheme(), this.keybindings, { paddingX: editorPaddingX });
         this.editor = this.defaultEditor;
         this.editorContainer = new Container();
         this.editorContainer.addChild(this.editor);
@@ -204,6 +206,7 @@ export class InteractiveMode {
         const extensionCommands = (this.session.extensionRunner?.getRegisteredCommands() ?? []).map((cmd) => ({
             name: cmd.name,
             description: cmd.description ?? "(extension command)",
+            getArgumentCompletions: cmd.getArgumentCompletions,
         }));
         // Build skill commands from session.skills (if enabled)
         this.skillCommands.clear();
@@ -230,53 +233,67 @@ export class InteractiveMode {
         // Setup autocomplete with fd tool for file path completion
         this.fdPath = await ensureTool("fd");
         this.setupAutocomplete(this.fdPath);
-        // Add header with keybindings from config
-        const logo = theme.bold(theme.fg("accent", APP_NAME)) + theme.fg("dim", ` v${this.version}`);
-        // Build startup instructions using keybinding hint helpers
-        const kb = this.keybindings;
-        const hint = (action, desc) => appKeyHint(kb, action, desc);
-        const instructions = [
-            hint("interrupt", "to interrupt"),
-            hint("clear", "to clear"),
-            rawKeyHint(`${appKey(kb, "clear")} twice`, "to exit"),
-            hint("exit", "to exit (empty)"),
-            hint("suspend", "to suspend"),
-            keyHint("deleteToLineEnd", "to delete to end"),
-            hint("cycleThinkingLevel", "to cycle thinking"),
-            rawKeyHint(`${appKey(kb, "cycleModelForward")}/${appKey(kb, "cycleModelBackward")}`, "to cycle models"),
-            hint("selectModel", "to select model"),
-            hint("expandTools", "to expand tools"),
-            hint("toggleThinking", "to toggle thinking"),
-            hint("externalEditor", "for external editor"),
-            rawKeyHint("/", "for commands"),
-            rawKeyHint("!", "to run bash"),
-            rawKeyHint("!!", "to run bash (no context)"),
-            hint("followUp", "to queue follow-up"),
-            hint("dequeue", "to edit all queued messages"),
-            hint("pasteImage", "to paste image"),
-            rawKeyHint("drop files", "to attach"),
-        ].join("\n");
-        this.builtInHeader = new Text(`${logo}\n${instructions}`, 1, 0);
-        // Setup UI layout
-        this.ui.addChild(new Spacer(1));
-        this.ui.addChild(this.builtInHeader);
-        this.ui.addChild(new Spacer(1));
-        // Add changelog if provided
-        if (this.changelogMarkdown) {
-            this.ui.addChild(new DynamicBorder());
-            if (this.settingsManager.getCollapseChangelog()) {
+        // Add header with keybindings from config (unless silenced)
+        if (!this.settingsManager.getQuietStartup()) {
+            const logo = theme.bold(theme.fg("accent", APP_NAME)) + theme.fg("dim", ` v${this.version}`);
+            // Build startup instructions using keybinding hint helpers
+            const kb = this.keybindings;
+            const hint = (action, desc) => appKeyHint(kb, action, desc);
+            const instructions = [
+                hint("interrupt", "to interrupt"),
+                hint("clear", "to clear"),
+                rawKeyHint(`${appKey(kb, "clear")} twice`, "to exit"),
+                hint("exit", "to exit (empty)"),
+                hint("suspend", "to suspend"),
+                keyHint("deleteToLineEnd", "to delete to end"),
+                hint("cycleThinkingLevel", "to cycle thinking"),
+                rawKeyHint(`${appKey(kb, "cycleModelForward")}/${appKey(kb, "cycleModelBackward")}`, "to cycle models"),
+                hint("selectModel", "to select model"),
+                hint("expandTools", "to expand tools"),
+                hint("toggleThinking", "to toggle thinking"),
+                hint("externalEditor", "for external editor"),
+                rawKeyHint("/", "for commands"),
+                rawKeyHint("!", "to run bash"),
+                rawKeyHint("!!", "to run bash (no context)"),
+                hint("followUp", "to queue follow-up"),
+                hint("dequeue", "to edit all queued messages"),
+                hint("pasteImage", "to paste image"),
+                rawKeyHint("drop files", "to attach"),
+            ].join("\n");
+            this.builtInHeader = new Text(`${logo}\n${instructions}`, 1, 0);
+            // Setup UI layout
+            this.ui.addChild(new Spacer(1));
+            this.ui.addChild(this.builtInHeader);
+            this.ui.addChild(new Spacer(1));
+            // Add changelog if provided
+            if (this.changelogMarkdown) {
+                this.ui.addChild(new DynamicBorder());
+                if (this.settingsManager.getCollapseChangelog()) {
+                    const versionMatch = this.changelogMarkdown.match(/##\s+\[?(\d+\.\d+\.\d+)\]?/);
+                    const latestVersion = versionMatch ? versionMatch[1] : this.version;
+                    const condensedText = `Updated to v${latestVersion}. Use ${theme.bold("/changelog")} to view full changelog.`;
+                    this.ui.addChild(new Text(condensedText, 1, 0));
+                }
+                else {
+                    this.ui.addChild(new Text(theme.bold(theme.fg("accent", "What's New")), 1, 0));
+                    this.ui.addChild(new Spacer(1));
+                    this.ui.addChild(new Markdown(this.changelogMarkdown.trim(), 1, 0, getMarkdownTheme()));
+                    this.ui.addChild(new Spacer(1));
+                }
+                this.ui.addChild(new DynamicBorder());
+            }
+        }
+        else {
+            // Minimal header when silenced
+            this.builtInHeader = new Text("", 0, 0);
+            if (this.changelogMarkdown) {
+                // Still show changelog notification even in silent mode
+                this.ui.addChild(new Spacer(1));
                 const versionMatch = this.changelogMarkdown.match(/##\s+\[?(\d+\.\d+\.\d+)\]?/);
                 const latestVersion = versionMatch ? versionMatch[1] : this.version;
                 const condensedText = `Updated to v${latestVersion}. Use ${theme.bold("/changelog")} to view full changelog.`;
                 this.ui.addChild(new Text(condensedText, 1, 0));
             }
-            else {
-                this.ui.addChild(new Text(theme.bold(theme.fg("accent", "What's New")), 1, 0));
-                this.ui.addChild(new Spacer(1));
-                this.ui.addChild(new Markdown(this.changelogMarkdown.trim(), 1, 0, getMarkdownTheme()));
-                this.ui.addChild(new Spacer(1));
-            }
-            this.ui.addChild(new DynamicBorder());
         }
         this.ui.addChild(this.chatContainer);
         this.ui.addChild(this.pendingMessagesContainer);
@@ -421,33 +438,38 @@ export class InteractiveMode {
      * Initialize the extension system with TUI-based UI context.
      */
     async initExtensions() {
-        // Show loaded project context files
-        const contextFiles = loadProjectContextFiles();
-        if (contextFiles.length > 0) {
-            const contextList = contextFiles.map((f) => theme.fg("dim", `  ${f.path}`)).join("\n");
-            this.chatContainer.addChild(new Text(theme.fg("muted", "Loaded context:\n") + contextList, 0, 0));
-            this.chatContainer.addChild(new Spacer(1));
-        }
-        // Show loaded skills (already discovered by SDK)
-        const skills = this.session.skills;
-        if (skills.length > 0) {
-            const skillList = skills.map((s) => theme.fg("dim", `  ${s.filePath}`)).join("\n");
-            this.chatContainer.addChild(new Text(theme.fg("muted", "Loaded skills:\n") + skillList, 0, 0));
-            this.chatContainer.addChild(new Spacer(1));
-        }
-        // Show skill warnings if any
-        const skillWarnings = this.session.skillWarnings;
-        if (skillWarnings.length > 0) {
-            const warningList = skillWarnings.map((w) => theme.fg("warning", `  ${w.skillPath}: ${w.message}`)).join("\n");
-            this.chatContainer.addChild(new Text(theme.fg("warning", "Skill warnings:\n") + warningList, 0, 0));
-            this.chatContainer.addChild(new Spacer(1));
-        }
-        // Show loaded prompt templates
-        const templates = this.session.promptTemplates;
-        if (templates.length > 0) {
-            const templateList = templates.map((t) => theme.fg("dim", `  /${t.name} ${t.source}`)).join("\n");
-            this.chatContainer.addChild(new Text(theme.fg("muted", "Loaded prompt templates:\n") + templateList, 0, 0));
-            this.chatContainer.addChild(new Spacer(1));
+        // Show discovery info unless silenced
+        if (!this.settingsManager.getQuietStartup()) {
+            // Show loaded project context files
+            const contextFiles = loadProjectContextFiles();
+            if (contextFiles.length > 0) {
+                const contextList = contextFiles.map((f) => theme.fg("dim", `  ${f.path}`)).join("\n");
+                this.chatContainer.addChild(new Text(theme.fg("muted", "Loaded context:\n") + contextList, 0, 0));
+                this.chatContainer.addChild(new Spacer(1));
+            }
+            // Show loaded skills (already discovered by SDK)
+            const skills = this.session.skills;
+            if (skills.length > 0) {
+                const skillList = skills.map((s) => theme.fg("dim", `  ${s.filePath}`)).join("\n");
+                this.chatContainer.addChild(new Text(theme.fg("muted", "Loaded skills:\n") + skillList, 0, 0));
+                this.chatContainer.addChild(new Spacer(1));
+            }
+            // Show skill warnings if any
+            const skillWarnings = this.session.skillWarnings;
+            if (skillWarnings.length > 0) {
+                const warningList = skillWarnings
+                    .map((w) => theme.fg("warning", `  ${w.skillPath}: ${w.message}`))
+                    .join("\n");
+                this.chatContainer.addChild(new Text(theme.fg("warning", "Skill warnings:\n") + warningList, 0, 0));
+                this.chatContainer.addChild(new Spacer(1));
+            }
+            // Show loaded prompt templates
+            const templates = this.session.promptTemplates;
+            if (templates.length > 0) {
+                const templateList = templates.map((t) => theme.fg("dim", `  /${t.name} ${t.source}`)).join("\n");
+                this.chatContainer.addChild(new Text(theme.fg("muted", "Loaded prompt templates:\n") + templateList, 0, 0));
+                this.chatContainer.addChild(new Spacer(1));
+            }
         }
         const extensionRunner = this.session.extensionRunner;
         if (!extensionRunner) {
@@ -463,7 +485,9 @@ export class InteractiveMode {
                 this.session
                     .sendCustomMessage(message, options)
                     .then(() => {
-                    if (!wasStreaming && message.display) {
+                    // Don't rebuild if initial render hasn't happened yet
+                    // (renderInitialMessages will handle it)
+                    if (!wasStreaming && message.display && this.hasRenderedInitialMessages) {
                         this.rebuildChatFromMessages();
                     }
                 })
@@ -547,7 +571,12 @@ export class InteractiveMode {
                 return { cancelled: false };
             },
             navigateTree: async (targetId, options) => {
-                const result = await this.session.navigateTree(targetId, { summarize: options?.summarize });
+                const result = await this.session.navigateTree(targetId, {
+                    summarize: options?.summarize,
+                    customInstructions: options?.customInstructions,
+                    replaceInstructions: options?.replaceInstructions,
+                    label: options?.label,
+                });
                 if (result.cancelled) {
                     return { cancelled: true };
                 }
@@ -566,12 +595,14 @@ export class InteractiveMode {
         });
         // Set up extension-registered shortcuts
         this.setupExtensionShortcuts(extensionRunner);
-        // Show loaded extensions
-        const extensionPaths = extensionRunner.getExtensionPaths();
-        if (extensionPaths.length > 0) {
-            const extList = extensionPaths.map((p) => theme.fg("dim", `  ${p}`)).join("\n");
-            this.chatContainer.addChild(new Text(theme.fg("muted", "Loaded extensions:\n") + extList, 0, 0));
-            this.chatContainer.addChild(new Spacer(1));
+        // Show loaded extensions (unless silenced)
+        if (!this.settingsManager.getQuietStartup()) {
+            const extensionPaths = extensionRunner.getExtensionPaths();
+            if (extensionPaths.length > 0) {
+                const extList = extensionPaths.map((p) => theme.fg("dim", `  ${p}`)).join("\n");
+                this.chatContainer.addChild(new Text(theme.fg("muted", "Loaded extensions:\n") + extList, 0, 0));
+                this.chatContainer.addChild(new Spacer(1));
+            }
         }
         // Emit session_start event
         await extensionRunner.emit({
@@ -1241,19 +1272,6 @@ export class InteractiveMode {
                 await this.shutdown();
                 return;
             }
-            // Handle skill commands (/skill:name [args])
-            if (text.startsWith("/skill:")) {
-                const spaceIndex = text.indexOf(" ");
-                const commandName = spaceIndex === -1 ? text.slice(1) : text.slice(1, spaceIndex);
-                const args = spaceIndex === -1 ? "" : text.slice(spaceIndex + 1).trim();
-                const skillPath = this.skillCommands.get(commandName);
-                if (skillPath) {
-                    this.editor.addToHistory?.(text);
-                    this.editor.setText("");
-                    await this.handleSkillCommand(skillPath, args);
-                    return;
-                }
-            }
             // Handle bash command (! for normal, !! for excluded from context)
             if (text.startsWith("!")) {
                 const isExcluded = text.startsWith("!!");
@@ -1503,6 +1521,11 @@ export class InteractiveMode {
                     });
                     this.footer.invalidate();
                 }
+                else if (event.errorMessage) {
+                    // Compaction failed (e.g., quota exceeded, API error)
+                    this.chatContainer.addChild(new Spacer(1));
+                    this.chatContainer.addChild(new Text(theme.fg("error", event.errorMessage), 1, 0));
+                }
                 void this.flushCompactionQueue({ willRetry: event.willRetry });
                 this.ui.requestRender();
                 break;
@@ -1690,6 +1713,7 @@ export class InteractiveMode {
         this.ui.requestRender();
     }
     renderInitialMessages() {
+        this.hasRenderedInitialMessages = true;
         // Get aligned messages and entries from session context
         const context = this.sessionManager.buildSessionContext();
         this.renderSessionContext(context, {
@@ -1750,6 +1774,9 @@ export class InteractiveMode {
                 type: "session_shutdown",
             });
         }
+        // Wait for any pending renders to complete
+        // requestRender() uses process.nextTick(), so we wait one tick
+        await new Promise((resolve) => process.nextTick(resolve));
         this.stop();
         process.exit(0);
     }
@@ -2117,6 +2144,7 @@ export class InteractiveMode {
                 hideThinkingBlock: this.hideThinkingBlock,
                 collapseChangelog: this.settingsManager.getCollapseChangelog(),
                 doubleEscapeAction: this.settingsManager.getDoubleEscapeAction(),
+                editorPaddingX: this.settingsManager.getEditorPaddingX(),
             }, {
                 onAutoCompactChange: (enabled) => {
                     this.session.setAutoCompactionEnabled(enabled);
@@ -2183,6 +2211,10 @@ export class InteractiveMode {
                 onDoubleEscapeActionChange: (action) => {
                     this.settingsManager.setDoubleEscapeAction(action);
                 },
+                onEditorPaddingXChange: (padding) => {
+                    this.settingsManager.setEditorPaddingX(padding);
+                    this.defaultEditor.setPaddingX(padding);
+                },
                 onCancel: () => {
                     done();
                     this.ui.requestRender();
@@ -2823,21 +2855,6 @@ export class InteractiveMode {
         this.chatContainer.addChild(new Text(info, 1, 0));
         this.ui.requestRender();
     }
-    async handleSkillCommand(skillPath, args) {
-        try {
-            const content = fs.readFileSync(skillPath, "utf-8");
-            // Strip YAML frontmatter if present
-            const body = content.replace(/^---\n[\s\S]*?\n---\n/, "").trim();
-            const skillDir = path.dirname(skillPath);
-            const header = `Skill location: ${skillPath}\nReferences are relative to ${skillDir}.`;
-            const skillMessage = `${header}\n\n${body}`;
-            const message = args ? `${skillMessage}\n\n---\n\nUser: ${args}` : skillMessage;
-            await this.session.prompt(message);
-        }
-        catch (err) {
-            this.showError(`Failed to load skill: ${err instanceof Error ? err.message : String(err)}`);
-        }
-    }
     handleChangelogCommand() {
         const changelogPath = getChangelogPath();
         const allEntries = parseChangelog(changelogPath);