imcp 0.0.11 → 0.0.12

package/dist/cli/commands/uninstall.js

@@ -3,19 +3,27 @@ import { serverService } from '../../services/ServerService.js';
 export function createUninstallCommand() {
     return new Command('uninstall')
         .description('Uninstall specific MCP servers')
-        .requiredOption('--category <category>', '--names <names>', 'Server names (semicolon separated)')
-        .option('--remove-data', 'Remove all associated data', false)
+        .requiredOption('--category <category>', 'Server category')
+        .requiredOption('--names <names>', 'Server names (semicolon separated)')
+        .requiredOption('--targets <targets>', 'Target clients to uninstall from (semicolon separated)')
+        .option('--remove-data', 'Remove all associated data', true // Change default to true to ensure cleanup happens
+    )
         .action(async (options) => {
         try {
             const serverNames = options.names.split(';').map((name) => name.trim());
-            if (!serverService.validateServerName(options.category, serverNames)) {
+            const validNames = await serverService.validateServerName(options.category, serverNames);
+            if (!validNames) {
                 console.error('Invalid server names provided');
                 process.exit(1);
             }
             console.log(`Uninstalling servers: ${serverNames.join(', ')}`);
-            const results = await Promise.all(serverNames.map((serverName) => serverService.uninstallMcpServer(options.category, serverName, {
-                removeData: options.removeData,
-            })));
+            const results = await Promise.all(serverNames.map((serverName) => {
+                const targets = options.targets ? options.targets.split(';').map((t) => t.trim()) : [];
+                return serverService.uninstallMcpServer(options.category, serverName, {
+                    removeData: options.removeData,
+                    targets: targets
+                });
+            }));
             const { success, messages } = serverService.formatOperationResults(results);
             messages.forEach(message => {
                 if (success) {

package/dist/core/ConfigurationProvider.d.ts

@@ -28,5 +28,6 @@ export declare class ConfigurationProvider {
     private loadFeedsIntoConfiguration;
     private loadClientMCPSettings;
     reloadClientMCPSettings(): Promise<void>;
+    removeServerFromClientMCPSettings(serverName: string, target?: string): Promise<void>;
 }
 export declare const configProvider: ConfigurationProvider;

package/dist/core/ConfigurationProvider.js

@@ -3,7 +3,7 @@ import path from 'path';
 import { exec } from 'child_process';
 import { promisify } from 'util';
 import { fileURLToPath } from 'url';
-import { GITHUB_REPO, LOCAL_FEEDS_DIR, SETTINGS_DIR } from './constants.js';
+import { GITHUB_REPO, LOCAL_FEEDS_DIR, SETTINGS_DIR, SUPPORTED_CLIENTS } from './constants.js';
 import { Logger } from '../utils/logger.js';
 import { checkGithubAuth } from '../utils/githubAuth.js';
 import { ConfigurationLoader } from './ConfigurationLoader.js';
@@ -294,6 +294,70 @@ export class ConfigurationProvider {
             await this.loadClientMCPSettings();
         });
     }
+    async removeServerFromClientMCPSettings(serverName, target) {
+        return await this.withLock(async () => {
+            // Load utils in async context to avoid circular dependencies
+            const { readJsonFile, writeJsonFile } = await import('../utils/clientUtils.js');
+            // Filter clients if target is specified
+            const clientEntries = Object.entries(SUPPORTED_CLIENTS);
+            const targetClients = target
+                ? clientEntries.filter(([clientName]) => clientName === target)
+                : clientEntries;
+            for (const [clientName, clientSettings] of targetClients) {
+                const settingPath = process.env.CODE_INSIDERS
+                    ? clientSettings.codeInsiderSettingPath
+                    : clientSettings.codeSettingPath;
+                try {
+                    const content = await readJsonFile(settingPath, true);
+                    let modified = false;
+                    // Handle GitHub Copilot's different structure
+                    if (clientName === 'GithubCopilot' && content.mcp?.servers?.[serverName]) {
+                        delete content.mcp.servers[serverName];
+                        modified = true;
+                    }
+                    else if (content.mcpServers?.[serverName]) {
+                        delete content.mcpServers[serverName];
+                        modified = true;
+                    }
+                    // Only write if we actually modified the content
+                    if (modified) {
+                        await writeJsonFile(settingPath, content);
+                        Logger.debug(`Removed server ${serverName} from client ${clientName} settings`);
+                    }
+                }
+                catch (error) {
+                    Logger.error(`Failed to remove server ${serverName} from client ${clientName} settings:`, error);
+                }
+            }
+            // Also update our in-memory configuration
+            if (this.configuration.clientMCPSettings) {
+                if (target) {
+                    // Only update settings for the target client
+                    const clientSettings = this.configuration.clientMCPSettings[target];
+                    if (clientSettings) {
+                        if (clientSettings.mcpServers?.[serverName]) {
+                            delete clientSettings.mcpServers[serverName];
+                        }
+                        if (clientSettings.servers?.[serverName]) { // For GitHub Copilot
+                            delete clientSettings.servers[serverName];
+                        }
+                    }
+                }
+                else {
+                    // Update all clients if no target specified
+                    for (const clientSettings of Object.values(this.configuration.clientMCPSettings)) {
+                        if (clientSettings.mcpServers?.[serverName]) {
+                            delete clientSettings.mcpServers[serverName];
+                        }
+                        if (clientSettings.servers?.[serverName]) { // For GitHub Copilot
+                            delete clientSettings.servers[serverName];
+                        }
+                    }
+                }
+                await this.saveConfiguration();
+            }
+        });
+    }
 }
 // Export a singleton instance
 export const configProvider = ConfigurationProvider.getInstance();

package/dist/core/MCPManager.js

@@ -1,7 +1,7 @@
 import { EventEmitter } from 'events';
 import { ConfigurationProvider } from './ConfigurationProvider.js';
 import { InstallationService } from './InstallationService.js';
-import { MCPEvent, } from './types.js';
+import { MCPEvent } from './types.js';
 export class MCPManager extends EventEmitter {
     installationService;
     configProvider;
@@ -66,12 +66,34 @@ export class MCPManager extends EventEmitter {
                     message: `Server category ${categoryName} is not onboarded`,
                 };
             }
-            // Clear installation status
-            await this.configProvider.updateInstallationStatus(categoryName, {}, {});
-            this.emit(MCPEvent.SERVER_UNINSTALLED, { serverName });
+            const { targets = [], removeData = false } = options;
+            // Clear installation status for specified targets
+            const currentStatus = serverCategory.installationStatus || {
+                requirementsStatus: {},
+                serversStatus: {},
+                lastUpdated: new Date().toISOString()
+            };
+            const serversStatus = currentStatus.serversStatus || {};
+            const serverStatus = serversStatus[serverName] || { installedStatus: {}, name: serverName };
+            // Only reset installedStatus for specified targets
+            for (const target of targets) {
+                if (serverStatus.installedStatus) {
+                    delete serverStatus.installedStatus[target];
+                }
+            }
+            if (removeData) {
+                for (const target of targets) {
+                    await this.configProvider.removeServerFromClientMCPSettings(serverName, target);
+                }
+            }
+            // Update server status
+            serversStatus[serverName] = serverStatus;
+            // Update status keeping requirements
+            await this.configProvider.updateInstallationStatus(categoryName, currentStatus.requirementsStatus || {}, serversStatus);
+            this.emit(MCPEvent.SERVER_UNINSTALLED, { serverName, targets });
             return {
                 success: true,
-                message: `Successfully uninstalled ${serverName}`,
+                message: `Successfully uninstalled ${serverName} from ${targets.join(', ')}`,
             };
         }
         catch (error) {

package/dist/core/ServerSchemaLoader.js

@@ -1,6 +1,6 @@
 import fs from 'fs/promises';
 import path from 'path';
-import { LOCAL_FEEDS_DIR } from './constants.js';
+import { LOCAL_FEEDS_SCHEMA_DIR } from './constants.js';
 import { Logger } from '../utils/logger.js';
 export class ServerSchemaLoader {
     /**
@@ -8,7 +8,7 @@ export class ServerSchemaLoader {
      */
     static async loadSchema(categoryName, serverName) {
         try {
-            const schemaPath = path.join(LOCAL_FEEDS_DIR, categoryName, `${serverName}.json`);
+            const schemaPath = path.join(LOCAL_FEEDS_SCHEMA_DIR, categoryName, `${serverName}.json`);
             const content = await fs.readFile(schemaPath, 'utf8');
             const schema = JSON.parse(content);
             // Validate schema structure

package/dist/core/constants.d.ts

@@ -22,7 +22,17 @@ export declare const LOCAL_FEEDS_DIR: string;
  * Value: Client-specific settings or configuration details.
  * TODO: Define actual client settings structure.
  */
-export declare const SUPPORTED_CLIENTS: Record<string, any>;
+export declare const SUPPORTED_CLIENTS: Record<string, {
+    extension: {
+        extensionId: string;
+        leastVersion?: string;
+        repository?: string;
+        assetName?: string;
+        private?: boolean;
+    };
+    codeSettingPath: string;
+    codeInsiderSettingPath: string;
+}>;
 /**
  * List of supported client names.
  */

package/dist/core/installers/clients/ExtensionInstaller.js

@@ -128,6 +128,9 @@ export class ExtensionInstaller {
                 else {
                     // Install private extension from GitHub release using latest version
                     try {
+                        if (!repository || !assetName) {
+                            throw new Error(`Missing repository or assetName for private extension ${extensionId}`);
+                        }
                         const { resolvedPath } = await handleGitHubRelease({ name: extensionId, version: 'latest', type: 'extension' }, { repository, assetName });
                         success = await this.installPrivateExtension(resolvedPath, isInsiders);
                     }

package/dist/core/types.d.ts

@@ -73,6 +73,7 @@ export interface UpdateRequirementOptions {
 }
 export interface ServerUninstallOptions {
     removeData?: boolean;
+    targets?: string[];
 }
 export interface EnvVariableConfig {
     Required: boolean;
@@ -130,6 +131,10 @@ export interface FeedConfiguration {
     requirements: RequirementConfig[];
     mcpServers: McpConfig[];
 }
+export interface ClientSettings {
+    codeSettingPath: string;
+    codeInsiderSettingPath: string;
+}
 export declare enum MCPEvent {
     SERVER_INSTALLED = "server:installed",
     SERVER_UNINSTALLED = "server:uninstalled",
@@ -143,6 +148,7 @@ export interface MCPEventData {
     };
     [MCPEvent.SERVER_UNINSTALLED]: {
         serverName: string;
+        targets?: string[];
     };
     [MCPEvent.SERVER_STARTED]: {
         server: MCPServerCategory;

package/dist/services/ServerService.d.ts

@@ -27,11 +27,17 @@ export declare class ServerService {
      * Installs a specific mcp tool for a server.
      * TODO: This might require enhancing MCPManager to handle category-specific installs.
      */
+    /**
+     * Uninstall MCP server from specified client targets
+     * @param category The server category
+     * @param serverName The server name to uninstall
+     * @param options Uninstall options including target clients and data removal flags
+     */
     uninstallMcpServer(category: string, serverName: string, options?: ServerUninstallOptions): Promise<ServerOperationResult>;
     /**
      * Validates server names
      */
-    validateServerName(category: string, name: string): Promise<boolean>;
+    validateServerName(category: string, names: string | string[]): Promise<boolean>;
     /**
      * Formats success/error messages for operations
      */

package/dist/services/ServerService.js

@@ -117,25 +117,40 @@ export class ServerService {
      * Installs a specific mcp tool for a server.
      * TODO: This might require enhancing MCPManager to handle category-specific installs.
      */
-    async uninstallMcpServer(category, serverName, options = {} // Reuse ServerInstallOptions for env etc.
-    ) {
-        return mcpManager.uninstallServer(category, serverName, options);
+    /**
+     * Uninstall MCP server from specified client targets
+     * @param category The server category
+     * @param serverName The server name to uninstall
+     * @param options Uninstall options including target clients and data removal flags
+     */
+    async uninstallMcpServer(category, serverName, options = {}) {
+        Logger.debug(`Uninstalling MCP server: ${JSON.stringify({ category, serverName, options })}`);
+        try {
+            const result = await mcpManager.uninstallServer(category, serverName, options);
+            Logger.debug(`Uninstallation result: ${JSON.stringify(result)}`);
+            return result;
+        }
+        catch (error) {
+            Logger.error(`Failed to uninstall MCP server: ${serverName}`, error);
+            throw error;
+        }
     }
     /**
      * Validates server names
      */
-    async validateServerName(category, name) {
-        Logger.debug(`Validating server name: ${JSON.stringify({ category, name })}`);
+    async validateServerName(category, names) {
+        const serverNames = Array.isArray(names) ? names : [names];
+        Logger.debug(`Validating server names: ${JSON.stringify({ category, names: serverNames })}`);
         // Check if category exists in feeds
         const feedConfig = await mcpManager.getFeedConfiguration(category);
         if (!feedConfig) {
             Logger.debug(`Validation failed: Category "${category}" not found in feeds`);
             return false;
         }
-        // Check if server exists in the category's mcpServers
-        const serverExists = feedConfig.mcpServers.some(server => server.name === name);
-        if (!serverExists) {
-            Logger.debug(`Validation failed: Server "${name}" not found in category "${category}"`);
+        // Check if all servers exist in the category's mcpServers
+        const invalidServers = serverNames.filter(name => !feedConfig.mcpServers.some(server => server.name === name));
+        if (invalidServers.length > 0) {
+            Logger.debug(`Validation failed: Servers "${invalidServers.join(', ')}" not found in category "${category}"`);
             return false;
         }
         return true;

package/dist/utils/osUtils.js

@@ -256,7 +256,7 @@ export function getPythonPackagePath(pythonExecutablePath) {
                 const venvRoot = path.dirname(dir);
                 return path.join(venvRoot, 'Lib', 'site-packages');
             }
-            else if (dir.toLowerCase().includes('python')) {
+            else {
                 // System Python or Conda on Windows
                 return path.join(dir, 'Lib', 'site-packages');
             }

package/dist/web/public/css/detailsWidget.css

@@ -1,56 +1,84 @@
 .details-widget {
-    max-height: 0;
-    overflow: hidden;
-    transition: max-height 0.3s ease-out;
+    transition: all 0.3s ease-in-out;
+}
+
+.tools-grid {
+    display: grid;
+    grid-template-columns: repeat(auto-fill, minmax(250px, 1fr));
+    gap: 0.5rem;
+    padding: 0.5rem;
 }
 
-.details-widget.expanded {
-    max-height: 2000px;
-    transition: max-height 0.5s ease-in;
+.tool-card {
+    transition: all 0.3s ease-out;
+    border: 1px solid #e5e7eb;
+    margin-bottom: 0.5rem;
+    padding: 0.75rem;
 }
 
-.details-widget-content {
-    padding: 1rem;
+.tool-card.active {
+    box-shadow: 0 4px 6px -1px rgba(0, 0, 0, 0.1), 0 2px 4px -1px rgba(0, 0, 0, 0.06);
+    border-left: 4px solid #3b82f6;
     background-color: #f8fafc;
-    border-top: 1px solid #e2e8f0;
 }
 
-/* Schema specific styles */
-.schema-content {
-    font-family: system-ui, -apple-system, sans-serif;
+.tool-card-header {
+    position: relative;
+    padding-right: 2rem;
 }
 
-.tool-section {
-    background-color: white;
-    border-radius: 0.5rem;
-    box-shadow: 0 1px 3px 0 rgba(0, 0, 0, 0.1);
-    margin-bottom: 1.5rem;
-    transition: transform 0.2s;
+.tool-card-header::after {
+    content: '';
+    position: absolute;
+    right: 1rem;
+    top: 50%;
+    transform: translateY(-50%);
+    width: 12px;
+    height: 12px;
+    border-right: 2px solid #64748b;
+    border-bottom: 2px solid #64748b;
+    transform-origin: 75% 75%;
+    transition: transform 0.3s ease;
+}
+
+.tool-card.active .tool-card-header::after {
+    transform: translateY(-50%) rotate(45deg);
+}
+
+.tool-details {
+    max-height: 0;
+    opacity: 0;
+    overflow: hidden;
+    transition: all 0.3s ease-out;
 }
 
-.tool-section:hover {
-    transform: translateY(-2px);
+.tool-details.visible {
+    max-height: 1500px;
+    opacity: 1;
+    padding-top: 0.75rem;
+    margin-top: 0.75rem;
+    border-top: 1px solid #e5e7eb;
 }
 
 .property-item {
-    padding: 0.75rem;
-    margin: 0.5rem 0;
-    background-color: white;
-    border-radius: 0.375rem;
-    transition: all 0.2s;
+    margin-bottom: 0.75rem;
+    padding-left: 0.75rem;
+    border-left: 2px solid #e5e7eb;
+    transition: border-color 0.2s ease;
+    font-size: 0.9rem;
 }
 
 .property-item:hover {
-    background-color: #f1f5f9;
+    border-left-color: #3b82f6;
 }
 
-.property-item.required {
-    border-left: 3px solid #3b82f6;
+.property-header {
+    margin-bottom: 0.5rem;
 }
 
 .property-name {
-    color: #1e40af;
     font-weight: 600;
+    color: #1e293b;
 }
 
 .property-type {
@@ -60,44 +88,23 @@
 
 .property-desc {
     color: #475569;
-    margin-top: 0.375rem;
+    font-size: 0.8125rem;
     line-height: 1.4;
-}
-
-.property-default {
-    color: #94a3b8;
-    font-size: 0.875rem;
-    font-family: monospace;
-}
-
-.required-badge {
-    background-color: #dbeafe;
-    color: #1e40af;
-    padding: 0.125rem 0.5rem;
-    border-radius: 9999px;
-    font-size: 0.75rem;
-    font-weight: 500;
+    margin-top: 0.25rem;
 }
 
 .required-fields {
     background-color: #fef3c7;
-    color: #92400e;
-    padding: 0.5rem;
-    border-radius: 0.375rem;
+    border-left: 4px solid #f59e0b;
+    padding: 0.5rem 0.75rem;
     margin-bottom: 1rem;
     font-size: 0.875rem;
 }
 
-.input-schema {
-    margin-top: 1rem;
-    padding: 1rem;
-    background-color: #f8fafc;
-    border-radius: 0.5rem;
-    border: 1px solid #e2e8f0;
-}
-
-.properties-list {
-    display: flex;
-    flex-direction: column;
-    gap: 0.75rem;
+.nested-properties {
+    margin-left: 0.75rem;
+    padding-left: 0.75rem;
+    border-left: 1px solid #e5e7eb;
+    margin-top: 0.5rem;
+    font-size: 0.875rem;
 }

package/dist/web/public/css/modal.css

@@ -160,6 +160,15 @@ body {
     transition: all 0.2s ease;
     cursor: pointer;
     user-select: none;
+    gap: 0.5rem;
+}
+
+/* Client actions container */
+.client-actions {
+    display: flex;
+    align-items: center;
+    gap: 0.5rem;
+    margin-left: auto;
 }
 
 .client-item:hover {
@@ -180,6 +189,14 @@ body {
     gap: 0.75rem;
 }
 
+/* Status container */
+.status-container {
+    display: flex;
+    align-items: center;
+    gap: 0.5rem;
+    margin-left: auto;
+}
+
 /* Status badges */
 .status-badge {
     display: inline-flex;
@@ -227,6 +244,31 @@ body {
     background-color: #fef3c7;
 }
 
+/* Uninstall button styling */
+.uninstall-btn {
+    display: inline-flex;
+    align-items: center;
+    justify-content: center;
+    width: 28px;
+    height: 28px;
+    border-radius: 6px;
+    border: 1px solid transparent;
+    background: transparent;
+    cursor: pointer;
+    transition: all 0.2s ease;
+    padding: 0;
+}
+
+.uninstall-btn:hover {
+    background-color: #fee2e2;
+    border-color: #ef4444;
+    transform: scale(1.05);
+}
+
+.uninstall-btn i {
+    font-size: 1.25rem;
+}
+
 /* Environment variables section */
 #modalEnvInputs input {
     width: 100%;

package/dist/web/public/css/serverDetails.css

@@ -36,8 +36,9 @@
 }
 
 .details-widget.expanded {
-    max-height: 500px; /* Adjust based on content */
+    max-height: 800px; /* Increased height to accommodate more content */
     border-color: #3b82f6;
+    transition: max-height 0.3s ease-in-out;
 }
 
 .details-widget-content {