@openeryc/pi-coding-agent 0.75.42 → 0.75.43

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

@@ -4419,26 +4419,33 @@ export class InteractiveMode {
     }
     showMcpSelector(highlightServer) {
         const configured = this.settingsManager.getMcpServers();
-        const allTools = this.session.getAllTools();
-        const mcpTools = allTools.filter((t) => t.name.startsWith("mcp_"));
-        const toolCountByServer = new Map();
-        for (const tool of mcpTools) {
-            const rest = tool.name.slice(4);
-            const idx = rest.indexOf("_");
-            const srv = idx === -1 ? rest : rest.slice(0, idx);
-            toolCountByServer.set(srv, (toolCountByServer.get(srv) ?? 0) + 1);
-        }
-        const items = (configured ? Object.entries(configured) : []).map(([name, cfg]) => {
-            const disabled = cfg.enabled === false;
-            const connected = (toolCountByServer.get(name) ?? 0) > 0;
-            const transport = cfg.url ? (cfg.transport ?? "sse") : cfg.command ? "stdio" : "unknown";
-            return {
-                name,
-                status: disabled ? "disabled" : connected ? "connected" : "disconnected",
-                toolCount: toolCountByServer.get(name) ?? 0,
-                transport,
-            };
-        });
+        const buildItems = () => {
+            const allTools = this.session.getAllTools();
+            const mcpTools = allTools.filter((t) => t.name.startsWith("mcp_"));
+            const cnt = new Map();
+            for (const t of mcpTools) {
+                const rest = t.name.slice(4);
+                const idx = rest.indexOf("_");
+                const srv = idx === -1 ? rest : rest.slice(0, idx);
+                cnt.set(srv, (cnt.get(srv) ?? 0) + 1);
+            }
+            return (configured ? Object.entries(configured) : []).map(([name, c]) => {
+                const disabled = c.enabled === false;
+                const connected = (cnt.get(name) ?? 0) > 0;
+                const transport = c.url ? (c.transport ?? "sse") : c.command ? "stdio" : "unknown";
+                return {
+                    name,
+                    status: disabled
+                        ? "disabled"
+                        : connected
+                            ? "connected"
+                            : "disconnected",
+                    toolCount: cnt.get(name) ?? 0,
+                    transport,
+                };
+            });
+        };
+        const items = buildItems();
         if (items.length === 0) {
             this.chatContainer.addChild(new Spacer(1));
             this.chatContainer.addChild(new Text(`${theme.bold("MCP Servers")}\n\n${theme.fg("dim", "No MCP servers configured. Add 'mcpServers' to settings.json.")}`, 1, 0));
@@ -4454,15 +4461,14 @@ export class InteractiveMode {
             }
         }
         this.showSelector((done) => {
-            const selector = new McpSelectorComponent(items, async (name) => {
+            const selector = new McpSelectorComponent("MCP Servers", items, async (name) => {
                 const server = configured?.[name];
                 if (!server)
                     return;
                 const currentEnabled = server.enabled !== false;
                 this.settingsManager.setMcpServerEnabled(name, !currentEnabled);
                 await this.session.reloadMcp();
-                done();
-                this.showStatus(`${name} ${currentEnabled ? "disabled" : "enabled"}`);
+                selector.setItems(buildItems());
             }, () => done());
             return { component: selector, focus: selector };
         });
@@ -4475,14 +4481,23 @@ export class InteractiveMode {
             this.ui.requestRender();
             return;
         }
+        const buildItems = () => {
+            const disabledSkills = this.settingsManager.getDisabledSkills();
+            return skills.map((s) => {
+                const isDisabled = disabledSkills.includes(s.name) || s.disableModelInvocation;
+                return {
+                    name: s.name,
+                    status: isDisabled ? "disabled" : "connected",
+                    toolCount: 0,
+                    transport: s.description.length > 60 ? `${s.description.slice(0, 57)}...` : s.description,
+                };
+            });
+        };
         this.showSelector((done) => {
-            const selector = new McpSelectorComponent(skills.map((s) => ({
-                name: s.name,
-                status: s.disableModelInvocation ? "disabled" : "connected",
-                toolCount: 0,
-                transport: s.description.slice(0, 60),
-            })), (_name) => {
-                done();
+            const selector = new McpSelectorComponent("Skills", buildItems(), (name) => {
+                this.settingsManager.toggleSkillDisabled(name);
+                this.session.setActiveToolsByName(this.session.getActiveToolNames());
+                selector.setItems(buildItems());
             }, () => done());
             return { component: selector, focus: selector };
         });