open-multi-agent-kit 0.80.6 → 0.80.8

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

@@ -1,22 +1,22 @@
 /**
  * Interactive mode for the coding agent.
+ * allow: SIZE_OK - legacy TUI coordinator; this change only adds a two-line CONTROL status bridge.
  * Handles TUI rendering and user interaction, delegating business logic to AgentSession.
  */
 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 { clampThinkingLevel, getProviders, getSupportedThinkingLevels, modelsAreEqual, } from "@earendil-works/omk-ai";
-import { CombinedAutocompleteProvider, Container, fuzzyFilter, getCapabilities, hyperlink, Loader, Markdown, matchesKey, ProcessTerminal, Spacer, setKeybindings, Text, TruncatedText, TUI, visibleWidth, } from "@earendil-works/omk-tui";
 import chalk from "chalk";
 import { spawn, spawnSync } from "child_process";
+import { getProviders, } from "omk-ai";
+import { CombinedAutocompleteProvider, Container, fuzzyFilter, getCapabilities, hyperlink, Loader, Markdown, matchesKey, ProcessTerminal, Spacer, setKeybindings, Text, TruncatedText, TUI, visibleWidth, } from "omk-tui";
 import { APP_NAME, APP_TITLE, getAgentDir, getAuthPath, getDebugLogPath, getDocsPath, getShareViewerUrl, VERSION, } from "../../config.js";
 import { parseSkillBlock } from "../../core/agent-session.js";
 import { SessionImportFileNotFoundError } from "../../core/agent-session-runtime.js";
 import { FooterDataProvider } from "../../core/footer-data-provider.js";
 import { configureHttpDispatcher, formatHttpIdleTimeoutMs } from "../../core/http-dispatcher.js";
 import { KeybindingsManager } from "../../core/keybindings.js";
-import { loadMcpInventory } from "../../core/mcp-inventory.js";
 import { createCompactionSummaryMessage } from "../../core/messages.js";
 import { defaultModelPerProvider, findExactModelReferenceMatch, resolveModelScope } from "../../core/model-resolver.js";
 import { DefaultPackageManager } from "../../core/package-manager.js";
@@ -25,7 +25,6 @@ import { formatMissingSessionCwdPrompt, MissingSessionCwdError } from "../../cor
 import { SessionManager } from "../../core/session-manager.js";
 import { BUILTIN_SLASH_COMMANDS } from "../../core/slash-commands.js";
 import { isInstallTelemetryEnabled } from "../../core/telemetry.js";
-import { runHarnessControlTransaction, runHarnessControlTransactionSync } from "../../core/transaction-coordinator.js";
 import { getChangelogPath, getNewEntries, parseChangelog } from "../../utils/changelog.js";
 import { copyToClipboard } from "../../utils/clipboard.js";
 import { extensionForImageMimeType, readClipboardImage } from "../../utils/clipboard-image.js";
@@ -33,7 +32,7 @@ import { parseGitUrl } from "../../utils/git.js";
 import { getCwdRelativePath } from "../../utils/paths.js";
 import { killTrackedDetachedChildren } from "../../utils/shell.js";
 import { ensureTool } from "../../utils/tools-manager.js";
-import { checkForNewPiVersion } from "../../utils/version-check.js";
+import { checkForNewOmkVersion } from "../../utils/version-check.js";
 import { ArminComponent } from "./components/armin.js";
 import { AssistantMessageComponent } from "./components/assistant-message.js";
 import { BashExecutionComponent } from "./components/bash-execution.js";
@@ -41,6 +40,7 @@ import { BorderedLoader } from "./components/bordered-loader.js";
 import { BranchSummaryMessageComponent } from "./components/branch-summary-message.js";
 import { CompactionSummaryMessageComponent } from "./components/compaction-summary-message.js";
 import { ControlPanelComponent } from "./components/control-panel.js";
+import { createControlPanelStatusSnapshot } from "./components/control-panel-runtime-status.js";
 import { CountdownTimer } from "./components/countdown-timer.js";
 import { CustomEditor } from "./components/custom-editor.js";
 import { CustomMessageComponent } from "./components/custom-message.js";
@@ -53,14 +53,12 @@ import { ExtensionSelectorComponent } from "./components/extension-selector.js";
 import { FooterComponent } from "./components/footer.js";
 import { formatKeyText, keyDisplayText, keyHint, keyText, rawKeyHint } from "./components/keybinding-hints.js";
 import { LoginDialogComponent } from "./components/login-dialog.js";
-import { McpSelectorComponent } from "./components/mcp-selector.js";
 import { ModelSelectorComponent } from "./components/model-selector.js";
 import { OAuthSelectorComponent } from "./components/oauth-selector.js";
 import { ScopedModelsSelectorComponent } from "./components/scoped-models-selector.js";
 import { SessionSelectorComponent } from "./components/session-selector.js";
 import { SettingsSelectorComponent, ThinkingSelectorComponent } from "./components/settings-selector.js";
 import { SkillInvocationMessageComponent } from "./components/skill-invocation-message.js";
-import { SkillsSelectorComponent } from "./components/skills-selector.js";
 import { ToolExecutionComponent } from "./components/tool-execution.js";
 import { TreeSelectorComponent } from "./components/tree-selector.js";
 import { UserMessageComponent } from "./components/user-message.js";
@@ -526,6 +524,7 @@ export class InteractiveMode {
                 expandedInstructions,
                 compactOnboarding: () => `Press ${keyText("app.tools.expand")} to show full startup help and loaded resources.`,
                 onboarding: () => `OMK can explain its own features and look up its docs. Ask it how to use or extend OMK.`,
+                statusSnapshot: () => createControlPanelStatusSnapshot(this.session, this.sessionManager),
             });
             controlPanel.setExpanded(this.getStartupExpansionState());
             this.builtInHeader = controlPanel;
@@ -596,7 +595,7 @@ export class InteractiveMode {
     async run() {
         await this.init();
         // Start version check asynchronously
-        checkForNewPiVersion(this.version).then((newRelease) => {
+        checkForNewOmkVersion(this.version).then((newRelease) => {
             if (newRelease) {
                 this.showNewVersionNotification(newRelease);
             }
@@ -660,7 +659,7 @@ export class InteractiveMode {
         }
     }
     async checkForPackageUpdates() {
-        if (process.env.OMK_OFFLINE || process.env.PI_OFFLINE) {
+        if (process.env.OMK_OFFLINE) {
             return [];
         }
         try {
@@ -744,7 +743,7 @@ export class InteractiveMode {
         return undefined;
     }
     reportInstallTelemetry(version) {
-        if (process.env.OMK_OFFLINE || process.env.PI_OFFLINE) {
+        if (process.env.OMK_OFFLINE) {
             return;
         }
         if (!isInstallTelemetryEnabled(this.settingsManager)) {
@@ -999,6 +998,18 @@ export class InteractiveMode {
         }
         return this.formatDisplayPath(p);
     }
+    formatCollisionPathWithSource(p, sourceInfos, fallback) {
+        const sourceInfo = this.findSourceInfoForPath(p, sourceInfos) ??
+            (fallback?.source
+                ? {
+                    path: p,
+                    source: fallback.source,
+                    scope: fallback.scope ?? "temporary",
+                    origin: fallback.origin ?? "top-level",
+                }
+                : undefined);
+        return this.formatPathWithSource(p, sourceInfo);
+    }
     formatDiagnostics(diagnostics, sourceInfos) {
         const lines = [];
         // Group collision diagnostics by name
@@ -1020,10 +1031,18 @@ export class InteractiveMode {
             if (!first)
                 continue;
             lines.push(theme.fg("warning", `  "${name}" collision:`));
-            lines.push(theme.fg("dim", `    ${theme.fg("success", "✓")} ${this.formatPathWithSource(first.winnerPath, this.findSourceInfoForPath(first.winnerPath, sourceInfos))}`));
+            lines.push(theme.fg("dim", `    ${theme.fg("success", "✓")} ${this.formatCollisionPathWithSource(first.winnerPath, sourceInfos, {
+                source: first.winnerSource,
+                scope: first.winnerScope,
+                origin: first.winnerOrigin,
+            })}`));
             for (const d of collisionList) {
                 if (d.collision) {
-                    lines.push(theme.fg("dim", `    ${theme.fg("warning", "✗")} ${this.formatPathWithSource(d.collision.loserPath, this.findSourceInfoForPath(d.collision.loserPath, sourceInfos))} (skipped)`));
+                    lines.push(theme.fg("dim", `    ${theme.fg("warning", "✗")} ${this.formatCollisionPathWithSource(d.collision.loserPath, sourceInfos, {
+                        source: d.collision.loserSource,
+                        scope: d.collision.loserScope,
+                        origin: d.collision.loserOrigin,
+                    })} (skipped)`));
                 }
             }
         }
@@ -1063,12 +1082,6 @@ export class InteractiveMode {
         if (!showListing && !showDiagnostics) {
             return;
         }
-        const diagnosticsForStartup = (diagnostics) => {
-            if (showListing) {
-                return [...diagnostics];
-            }
-            return diagnostics.filter((diagnostic) => diagnostic.type !== "collision");
-        };
         const sectionHeader = (name, color = "mdHeading") => theme.fg(color, `[${name}]`);
         const formatCompactList = (items, options) => {
             const labels = items.map((item) => item.trim()).filter((item) => item.length > 0);
@@ -1170,15 +1183,19 @@ export class InteractiveMode {
             }
         }
         if (showDiagnostics) {
+            const showCollisionDiagnostics = options?.force === true || this.options.verbose;
+            const diagnosticsVisibleForMode = (diagnostics) => showCollisionDiagnostics
+                ? diagnostics
+                : diagnostics.filter((diagnostic) => diagnostic.type !== "collision");
             const addDiagnosticsSection = (name, diagnostics) => {
                 if (diagnostics.length === 0) {
                     return;
                 }
                 addLoadedSection(name, () => this.formatDiagnosticsSummary(diagnostics), () => this.formatDiagnostics(diagnostics, sourceInfos), "warning");
             };
-            const skillDiagnostics = diagnosticsForStartup(skillsResult.diagnostics);
+            const skillDiagnostics = diagnosticsVisibleForMode(skillsResult.diagnostics);
             addDiagnosticsSection("Skill conflicts", skillDiagnostics);
-            const promptDiagnostics = diagnosticsForStartup(promptsResult.diagnostics);
+            const promptDiagnostics = diagnosticsVisibleForMode(promptsResult.diagnostics);
             addDiagnosticsSection("Prompt conflicts", promptDiagnostics);
             const extensionDiagnostics = [];
             const extensionErrors = this.session.resourceLoader.getExtensions().errors;
@@ -1192,8 +1209,8 @@ export class InteractiveMode {
             extensionDiagnostics.push(...this.getBuiltInCommandConflictDiagnostics(this.session.extensionRunner));
             const shortcutDiagnostics = this.session.extensionRunner.getShortcutDiagnostics();
             extensionDiagnostics.push(...shortcutDiagnostics);
-            addDiagnosticsSection("Extension issues", extensionDiagnostics);
-            const themeDiagnostics = diagnosticsForStartup(themesResult.diagnostics);
+            addDiagnosticsSection("Extension issues", diagnosticsVisibleForMode(extensionDiagnostics));
+            const themeDiagnostics = diagnosticsVisibleForMode(themesResult.diagnostics);
             addDiagnosticsSection("Theme conflicts", themeDiagnostics);
         }
     }
@@ -2046,7 +2063,7 @@ export class InteractiveMode {
             // Write to temp file
             const tmpDir = os.tmpdir();
             const ext = extensionForImageMimeType(image.mimeType) ?? "png";
-            const fileName = `pi-clipboard-${crypto.randomUUID()}.${ext}`;
+            const fileName = `omk-clipboard-${crypto.randomUUID()}.${ext}`;
             const filePath = path.join(tmpDir, fileName);
             fs.writeFileSync(filePath, Buffer.from(image.bytes));
             // Insert file path directly
@@ -2166,16 +2183,6 @@ export class InteractiveMode {
                 await this.handleReloadCommand();
                 return;
             }
-            if (text === "/mcp") {
-                this.editor.setText("");
-                this.handleMcpCommand();
-                return;
-            }
-            if (text === "/skills") {
-                this.editor.setText("");
-                this.handleSkillsCommand();
-                return;
-            }
             if (text === "/debug") {
                 this.handleDebugCommand();
                 this.editor.setText("");
@@ -2836,7 +2843,7 @@ export class InteractiveMode {
             this.ui.stop();
         }
         catch { }
-        console.error("pi exiting due to uncaughtException:");
+        console.error("OMK exiting due to uncaughtException:");
         console.error(error);
         process.exit(1);
     }
@@ -2878,7 +2885,7 @@ export class InteractiveMode {
         this.signalCleanupHandlers.push(() => process.stderr.off("error", terminalErrorHandler));
         // Restore the terminal before the process dies on any uncaught throw.
         // Without this, an unhandled exception from extension code (or anywhere
-        // in pi) leaves the terminal in raw mode with no cursor.
+        // in OMK) leaves the terminal in raw mode with no cursor.
         const uncaughtExceptionHandler = (error) => this.uncaughtCrash(error);
         process.prependListener("uncaughtException", uncaughtExceptionHandler);
         this.signalCleanupHandlers.push(() => process.off("uncaughtException", uncaughtExceptionHandler));
@@ -3039,7 +3046,7 @@ export class InteractiveMode {
             return;
         }
         const currentText = this.editor.getExpandedText?.() ?? this.editor.getText();
-        const tmpFile = path.join(os.tmpdir(), `pi-editor-${Date.now()}.omk.md`);
+        const tmpFile = path.join(os.tmpdir(), `omk-editor-${Date.now()}.omk.md`);
         try {
             // Write current content to temp file
             fs.writeFileSync(tmpFile, currentText, "utf-8");
@@ -3047,7 +3054,7 @@ export class InteractiveMode {
             this.ui.stop();
             // Split by space to support editor arguments (e.g., "code --wait")
             const [editor, ...editorArgs] = editorCmd.split(" ");
-            process.stdout.write(`Launching external editor: ${editorCmd}\nPi will resume when the editor exits.\n`);
+            process.stdout.write(`Launching external editor: ${editorCmd}\nOMK will resume when the editor exits.\n`);
             // Do not use spawnSync here. On Windows, synchronous child_process calls can keep
             // Node/libuv's console input read active after ui.stop() pauses stdin, racing
             // vim/nvim for the console input buffer until Ctrl+C cancels the pending read.
@@ -3395,7 +3402,12 @@ export class InteractiveMode {
                     this.updateEditorBorderColor();
                 },
                 onThemeChange: (themeName) => {
-                    this.applySettingsThemeChange(themeName);
+                    const result = setTheme(themeName, true);
+                    this.settingsManager.setTheme(themeName);
+                    this.ui.invalidate();
+                    if (!result.success) {
+                        this.showError(`Failed to load theme "${themeName}": ${result.error}\nFell back to dark theme.`);
+                    }
                 },
                 onThemePreview: (themeName) => {
                     const result = setTheme(themeName, true);
@@ -3466,37 +3478,6 @@ export class InteractiveMode {
             return { component: selector, focus: selector.getSettingsList() };
         });
     }
-    applySettingsThemeChange(themeName) {
-        const previousTheme = this.settingsManager.getTheme?.() || "dark";
-        let applyResult = {
-            success: false,
-            error: "theme transaction did not run",
-        };
-        const transaction = runHarnessControlTransactionSync({
-            kind: "interactive.theme.apply",
-            data: { themeName, previousTheme, persistenceScope: "default" },
-            beforeState: { theme: previousTheme },
-            afterState: () => ({ theme: themeName }),
-            commit: () => {
-                const result = setTheme(themeName, true);
-                applyResult = result;
-                if (!result.success) {
-                    throw new Error(result.error ?? `Failed to load theme "${themeName}"`);
-                }
-                this.settingsManager.setTheme(themeName);
-                this.ui.invalidate();
-                return { theme: themeName };
-            },
-            rollback: () => {
-                setTheme(previousTheme, true);
-            },
-        });
-        if (transaction.status !== "completed") {
-            this.showError(`Failed to load theme "${themeName}": ${applyResult.error}\nRestored theme "${previousTheme}".`);
-            return applyResult;
-        }
-        return applyResult;
-    }
     handleThinkCommand(level) {
         if (!level) {
             this.showThinkingSelector();
@@ -3510,25 +3491,10 @@ export class InteractiveMode {
         this.applyThinkingLevel(level);
     }
     applyThinkingLevel(level) {
-        const previousThinkingLevel = this.session.thinkingLevel;
-        const transaction = runHarnessControlTransactionSync({
-            kind: "interactive.model_thinking.commit",
-            data: { thinkingLevel: level, previousThinkingLevel, persistenceScope: "session" },
-            beforeState: { thinkingLevel: previousThinkingLevel },
-            afterState: () => ({ thinkingLevel: level }),
-            commit: () => {
-                this.session.setThinkingLevel(level);
-                this.footer.invalidate();
-                this.updateEditorBorderColor();
-                return { thinkingLevel: level };
-            },
-            rollback: () => {
-                this.session.setThinkingLevel(previousThinkingLevel);
-            },
-        });
-        if (transaction.status === "completed") {
-            this.showStatus(`Thinking: ${this.session.thinkingLevel}`);
-        }
+        this.session.setThinkingLevel(level);
+        this.footer.invalidate();
+        this.updateEditorBorderColor();
+        this.showStatus(`Thinking: ${this.session.thinkingLevel}`);
     }
     async handleModelCommand(searchTerm) {
         if (!searchTerm) {
@@ -3537,7 +3503,18 @@ export class InteractiveMode {
         }
         const model = await this.findExactModelMatch(searchTerm);
         if (model) {
-            this.showModelThinkingTransaction(model);
+            try {
+                await this.session.setModel(model);
+                this.footer.invalidate();
+                this.updateEditorBorderColor();
+                this.showStatus(`Model: ${model.id}`);
+                void this.maybeWarnAboutAnthropicSubscriptionAuth(model);
+                this.checkDaxnutsEasterEgg(model);
+                this.showThinkingSelector();
+            }
+            catch (error) {
+                this.showError(error instanceof Error ? error.message : String(error));
+            }
             return;
         }
         this.showModelSelector(searchTerm);
@@ -3613,81 +3590,23 @@ export class InteractiveMode {
             return { component: selector, focus: selector };
         });
     }
-    showModelThinkingTransaction(model) {
-        const availableThinkingLevels = getSupportedThinkingLevels(model);
-        const thinkingLevel = clampThinkingLevel(model, this.session.thinkingLevel);
-        // Auto-skip selector when the model offers at most one thinking level (T04 routing safeguard).
-        if (availableThinkingLevels.length <= 1) {
-            const chosen = (availableThinkingLevels[0] ?? thinkingLevel ?? "off");
-            void this.commitModelThinkingSelection(model, chosen);
-            return;
-        }
+    showModelSelector(initialSearchInput) {
         this.showSelector((done) => {
-            const selector = new ThinkingSelectorComponent({
-                thinkingLevel,
-                availableThinkingLevels,
-            }, {
-                onThinkingLevelChange: () => { },
-                onSelectComplete: (level) => {
+            const selector = new ModelSelectorComponent(this.ui, this.session.model, this.settingsManager, this.session.modelRegistry, this.session.scopedModels, async (model) => {
+                try {
+                    await this.session.setModel(model);
+                    this.footer.invalidate();
+                    this.updateEditorBorderColor();
                     done();
-                    void this.commitModelThinkingSelection(model, level);
-                },
-                onCancel: () => {
+                    this.showStatus(`Model: ${model.id}`);
+                    void this.maybeWarnAboutAnthropicSubscriptionAuth(model);
+                    this.checkDaxnutsEasterEgg(model);
+                    this.showThinkingSelector();
+                }
+                catch (error) {
                     done();
-                    this.ui.requestRender();
-                },
-            });
-            return { component: selector, focus: selector };
-        });
-    }
-    async commitModelThinkingSelection(model, thinkingLevel) {
-        const previousModel = this.session.model;
-        const previousThinkingLevel = this.session.thinkingLevel;
-        const transaction = await runHarnessControlTransaction({
-            kind: "interactive.model_thinking.commit",
-            data: {
-                provider: model.provider,
-                model: model.id,
-                thinkingLevel,
-                previousProvider: previousModel?.provider,
-                previousModel: previousModel?.id,
-                previousThinkingLevel,
-                persistenceScope: "session",
-            },
-            beforeState: {
-                provider: previousModel?.provider,
-                model: previousModel?.id,
-                thinkingLevel: previousThinkingLevel,
-            },
-            afterState: () => ({ provider: model.provider, model: model.id, thinkingLevel }),
-            commit: async () => {
-                await this.session.setModel(model);
-                this.session.setThinkingLevel(thinkingLevel);
-                this.footer.invalidate();
-                this.updateEditorBorderColor();
-                return { provider: model.provider, model: model.id, thinkingLevel };
-            },
-            rollback: async () => {
-                if (previousModel && !modelsAreEqual(this.session.model, previousModel)) {
-                    await this.session.setModel(previousModel);
+                    this.showError(error instanceof Error ? error.message : String(error));
                 }
-                this.session.setThinkingLevel(previousThinkingLevel);
-            },
-        });
-        if (transaction.status === "completed") {
-            this.showStatus(`Model: ${model.id} · Thinking: ${this.session.thinkingLevel}`);
-            void this.maybeWarnAboutAnthropicSubscriptionAuth(model);
-            this.checkDaxnutsEasterEgg(model);
-            return;
-        }
-        const error = transaction.error ?? transaction.rollbackError ?? "Model transaction failed";
-        this.showError(error instanceof Error ? error.message : String(error));
-    }
-    showModelSelector(initialSearchInput) {
-        this.showSelector((done) => {
-            const selector = new ModelSelectorComponent(this.ui, this.session.model, this.settingsManager, this.session.modelRegistry, this.session.scopedModels, (model) => {
-                done();
-                this.showModelThinkingTransaction(model);
             }, () => {
                 done();
                 this.ui.requestRender();
@@ -4619,35 +4538,6 @@ export class InteractiveMode {
         this.chatContainer.addChild(new Text(info, 1, 0));
         this.ui.requestRender();
     }
-    handleMcpCommand() {
-        const inventory = loadMcpInventory(this.sessionManager.getCwd());
-        this.showSelector((done) => {
-            const selector = new McpSelectorComponent(inventory, () => {
-                done();
-                this.ui.requestRender();
-            });
-            return { component: selector, focus: selector };
-        });
-        const summary = inventory.entries.length === 1 ? "1 MCP server" : `${inventory.entries.length} MCP servers`;
-        this.showStatus(`${summary} loaded (env values hidden)`);
-    }
-    handleSkillsCommand() {
-        const result = this.session.resourceLoader.getSkills();
-        const enableSkillCommands = this.settingsManager.getEnableSkillCommands();
-        this.showSelector((done) => {
-            const selector = new SkillsSelectorComponent({
-                skills: result.skills,
-                diagnostics: result.diagnostics,
-                enableSkillCommands,
-            }, () => {
-                done();
-                this.ui.requestRender();
-            });
-            return { component: selector, focus: selector };
-        });
-        const summary = result.skills.length === 1 ? "1 skill" : `${result.skills.length} skills`;
-        this.showStatus(`${summary} loaded`);
-    }
     handleChangelogCommand() {
         const changelogPath = getChangelogPath();
         const allEntries = parseChangelog(changelogPath);