Fixes duplication issues: Added events based cache invalidation and rm mcp debug comments

Author: STetsing

libs/remix-ai-core/src/agents/codeExplainAgent.ts

@@ -29,7 +29,6 @@ export class CodeExplainAgent {
         return content
       } else return prompt
     } catch {
-      console.log('There is No file selected')
       return 'There is No file selected'
     }
   }

libs/remix-ai-core/src/agents/contractAgent.ts

@@ -36,7 +36,6 @@ export class ContractAgent {
   async writeContracts(payload, userPrompt, statusCallback?: (status: string) => Promise<void>) {
     const prev_statusCallback = statusCallback
     statusCallback = async (status: string) => {
-      console.log('Generation status:', status)
       if (prev_statusCallback) {
         await prev_statusCallback(status)
       }

libs/remix-ai-core/src/agents/securityAgent.ts

@@ -123,7 +123,6 @@ export class SecurityAgent {
 
     try {
       this.processFile(file);
-      console.log('Checking for vulnerabilities after compilation', this.reports);
     } catch (error) {
       console.error('Error checking for vulnerabilities after compilation: ', error);
     }

libs/remix-ai-core/src/helpers/chatCommandParser.ts

@@ -47,7 +47,6 @@ export class ChatCommandParser {
     if (handler) {
       return handler(rawArgs, this, statusCallback);
     } else {
-      console.log(`Unknown command: ${commandName}`);
       return "";
     }
   }

libs/remix-ai-core/src/helpers/compile.ts

@@ -11,13 +11,11 @@ export const compilecontracts = async (contracts, plugin): Promise<CompilationRe
   // do not compile tests files
   let result
   try {
-    // console.log('Compiling contracts:', contracts)
     result = await plugin.call('solidity' as any, 'compileWithParameters', contracts, compilationParams)
     const data = result.data
     let error = false
 
     if (data.errors) {
-      // console.log('Compilation errors:', data.errors)
       error = data.errors.find((error) => error.type !== 'Warning')
     }
 

libs/remix-ai-core/src/inferencers/mcp/codeExecutor.ts

@@ -57,9 +57,7 @@ export class CodeExecutor {
 
       // CRITICAL: race condition - Wait for all pending tool calls to complete before returning
       if (this.pendingToolCalls.length > 0) {
-        console.log(`[MCP Code mode] - Waiting for ${this.pendingToolCalls.length} pending tool call(s) to complete...`);
         await Promise.allSettled(this.pendingToolCalls);
-        console.log(`[MCP Code mode] - All tool calls completed`);
       }
 
       const executionTime = Date.now() - startTime;

libs/remix-ai-core/src/inferencers/mcp/mcpClient.ts

@@ -183,7 +183,6 @@ export class MCPClient {
         let initialized = false;
 
         this.wsConnection.onopen = () => {
-          console.log(`[MCP] WebSocket connection opened to ${this.server.name}`);
 
           // Send initialize message
           const initMessage = {
@@ -395,7 +394,6 @@ export class MCPClient {
       // Check cache for HTTP servers
       const now = Date.now();
       if (this.resourceListCache && (now - this.resourceListCache.timestamp) < this.CACHE_TTL) {
-        console.log(`[MCP] Using cached resource list for ${this.server.name}`);
         return this.resourceListCache.resources;
       }
 

libs/remix-ai-core/src/inferencers/mcp/mcpInferencer.ts

@@ -405,7 +405,6 @@ export class MCPInferencer extends RemoteInferencer implements ICompletions, IGe
               const mcpToolCall = this.convertLLMToolCallToMCP(llmToolCall);
               const result = await this.executeToolForLLM(mcpToolCall, uiCallback);
               console.log(`[MCP] Tool ${mcpToolCall.name} executed successfully`);
-              console.log("[MCP] Tool result", result);
 
               // Extract full text content from MCP result
               const extractContent = (mcpResult: any): string => {
@@ -712,8 +711,6 @@ export class MCPInferencer extends RemoteInferencer implements ICompletions, IGe
 
   async getToolsForLLMRequest(provider?: string, prompt?: string): Promise<any[]> {
     const mcpTools = await this.getAvailableToolsForLLM();
-    console.log('[MCPInferencer] Total available tools:', mcpTools.length)
-
     if (mcpTools.length === 0) return [];
 
     // Use keyword-based tool selection if prompt provided and more than 15 tools
@@ -731,7 +728,6 @@ export class MCPInferencer extends RemoteInferencer implements ICompletions, IGe
         });
 
         console.log(`[MCPInferencer] Tool selection: ${mcpTools.length} → ${selectedTools.length} tools (${Math.round((1 - selectedTools.length / mcpTools.length) * 100)}% reduction)`)
-        console.log('[MCPInferencer] Selected tools:', selectedTools.map(t => t.name).join(', '))
       } catch (error) {
         console.warn('[MCPInferencer] Tool selection failed, using all tools:', error)
         selectedTools = mcpTools
@@ -953,7 +949,6 @@ ${toolsList}`,
     if (!targetServer) {
       throw new Error(`Tool '${toolCall.name}' not found in any connected MCP server`);
     }
-    console.log(`[MCP Legacy Mode] Executing tool ${toolCall.name} from server ${targetServer}`)
     return this.executeTool(targetServer, toolCall);
   }
 

libs/remix-ai-core/src/inferencers/remote/remoteInference.ts

@@ -131,7 +131,6 @@ export class RemoteInferencer implements ICompletions, IGeneration {
         if (done) break;
 
         try {
-          console.log("value" + decoder.decode(value))
           const chunk = parser.safeJsonParse<{ generatedText: string; isGenerating: boolean }>(decoder.decode(value, { stream: true }));
           for (const parsedData of chunk) {
             if (parsedData.isGenerating) {

libs/remix-ai-core/src/remix-mcp-server/RemixMCPServer.ts

@@ -181,12 +181,6 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
 
       try {
         await this._configManager.loadConfig();
-        const securityConfig = this._configManager.getSecurityConfig();
-        const validationConfig = this._configManager.getValidationConfig();
-        console.log('[RemixMCPServer] Middlewares connected:');
-        console.log(`  - SecurityMiddleware: using ${securityConfig.excludeTools?.length || 0} excluded tools`);
-        console.log(`  - ValidationMiddleware: strictMode=${validationConfig.strictMode}, ${Object.keys(validationConfig.toolValidation || {}).length} tool-specific rules`);
-
       } catch (error) {
         console.log(`[RemixMCPServer] Failed to load MCP config: ${error.message}, using defaults`);
       }
@@ -331,17 +325,21 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
 
     // Get current user (default to 'default' role)
     const currentUser = 'default'; // Can be extended to get from plugin context
-    console.log("checking permissions")
     const permissionCheckResult = await this.checkPermissions(call.name, currentUser);
-    console.log("permissions checked", permissionCheckResult)
+
+    const timestamp = Date.now();
+    const [workspace, currentFile] = await Promise.all([
+      this.getCurrentWorkspace(),
+      this.getCurrentFile()
+    ]);
 
     const execution: ToolExecutionStatus = {
       id: executionId,
       toolName: call.name,
       startTime,
       status: 'running',
       context: {
-        workspace: await this.getCurrentWorkspace(),
+        workspace,
         user: currentUser,
         permissions: permissionCheckResult.userPermissions
       }
@@ -352,28 +350,22 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
 
     try {
       const context = {
-        workspace: execution.context.workspace,
-        currentFile: await this.getCurrentFile(),
+        workspace,
+        currentFile,
         permissions: permissionCheckResult.userPermissions,
-        timestamp: Date.now(),
+        timestamp,
         requestId: executionId
       };
 
-      // STEP 1: Security Validation (uses MCPConfigManager for dynamic config)
-      console.log(`[RemixMCPServer] Step 1: Security validation for tool '${call.name}' (using MCPConfigManager)`);
       const securityResult = await this._securityMiddleware.validateToolCall(call, context, this._plugin);
 
       if (!securityResult.allowed) {
         console.log(`[RemixMCPServer] Security validation FAILED for tool '${call.name}': ${securityResult.reason}`);
         throw new Error(`Security validation failed: ${securityResult.reason}`);
       }
-      console.log(`[RemixMCPServer] Security validation PASSED for tool '${call.name}'`);
 
-      // STEP 2: Input Validation (uses MCPConfigManager for dynamic config)
-      console.log(`[RemixMCPServer] Step 2: Input validation for tool '${call.name}' (using MCPConfigManager)`);
       const toolDefinition = this._tools.get(call.name);
       const inputSchema = toolDefinition?.inputSchema;
-
       const validationResult = await this._validationMiddleware.validateToolCall(
         call,
         inputSchema,
@@ -391,14 +383,12 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
       if (validationResult.warnings.length > 0) {
         const warnings = validationResult.warnings.map(w => w.message).join(', ');
         console.log(`[RemixMCPServer] Input validation warnings for tool '${call.name}': ${warnings}`);
-      } else {
-        console.log(`[RemixMCPServer] Input validation PASSED for tool '${call.name}'`);
       }
 
-      // STEP 3: File Write Permission Check (for file_write and file_create tools)
-      if (call.name === 'file_write' || call.name === 'file_create') {
-        console.log(`[RemixMCPServer] Step 3: File write permission check for tool '${call.name}'`);
-        const filePath = call.arguments?.path || call.arguments?.filePath;
+      // STEP 3: File Permision Check (for file operations)
+      const fileOperations = ['file_write', 'file_create', 'file_delete', 'file_move', 'file_copy'];
+      if (fileOperations.includes(call.name)) {
+        const filePath = call.arguments?.path || call.arguments?.filePath || call.arguments?.from || call.arguments?.source;
 
         if (filePath) {
           const permissionResult = await this._filePermissionMiddleware.checkFileWritePermission(
@@ -407,10 +397,10 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
           );
 
           if (!permissionResult.allowed) {
-            console.log(`[RemixMCPServer] File write permission DENIED for '${filePath}': ${permissionResult.reason}`);
-            throw new Error(`File write permission denied: ${permissionResult.reason || 'User denied the operation'}`);
+            console.log(`[RemixMCPServer] File operation permission DENIED for '${filePath}': ${permissionResult.reason}`);
+            throw new Error(`File operation permission denied: ${permissionResult.reason || 'User denied the operation'}`);
           }
-          console.log(`[RemixMCPServer] File write permission GRANTED for '${filePath}'`);
+          console.log(`[RemixMCPServer] File operation permission GRANTED for '${filePath}'`);
         }
       }
 
@@ -429,7 +419,6 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
       this._stats.totalToolCalls++;
 
       trackMatomoEvent('ai', 'remixAI', `mcp_tool_executed_${call.name}`);
-      console.log(`[RemixMCPServer] Tool '${call.name}' executed successfully`);
       this.emit('tool-executed', execution);
       return result;
 
@@ -491,8 +480,6 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
   async checkPermissions(operation: string, user: string, resource?: string): Promise<PermissionCheckResult> {
     try {
       const securityConfig = this._configManager.getSecurityConfig();
-      console.log("securityConfig", securityConfig)
-      // If permissions
 
       if (!securityConfig.permissions.requirePermissions) {
         return {
@@ -504,9 +491,7 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
       }
 
       const userPermissions = this.getUserPermissions(user, securityConfig);
-      console.log("userPermissions", userPermissions)
       const requiredPermissions = this.getOperationPermissions(operation);
-      console.log("requiredPermissions", requiredPermissions)
 
       if (userPermissions.includes('*')) {
         return {
@@ -555,7 +540,6 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
       // Additional resource-specific checks
       if (resource) {
         const resourceCheck = this.checkResourcePermissions(resource, userPermissions, securityConfig);
-        console.log("Resource check:", resourceCheck)
         if (!resourceCheck.allowed) {
           // Log denied resource access
           this.logAuditEntry({
@@ -647,7 +631,6 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
   }
 
   private getUserPermissions(user: string, securityConfig: any): string[] {
-    console.log('security config', securityConfig)
     const permissions: string[] = [];
 
     if (securityConfig.permissions?.defaultPermissions) {
@@ -701,10 +684,14 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
       // Analysis
       'analyze_code': ['analysis:static'],
       'security_scan': ['analysis:security'],
-      'estimate_gas': ['analysis:gas']
+      'estimate_gas': ['analysis:gas'],
+
+      // Additional tools
+      'run_script': ['transaction:send'],
+      'simulate_transaction': ['transaction:simulate']
     };
 
-    return defaultPermissionMap[operation] || ['*'];
+    return defaultPermissionMap[operation] || [`tool:${operation}`];
   }
 
   private checkResourcePermissions(resource: string, userPermissions: string[], securityConfig: any): { allowed: boolean; reason?: string } {
@@ -779,10 +766,7 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
    */
   async reloadConfig(): Promise<void> {
     try {
-      console.log('[RemixMCPServer] Reloading MCP configuration...');
       const mcpConfig = await this._configManager.reloadConfig();
-      console.log('[RemixMCPServer] Configuration reloaded successfully');
-      console.log('[RemixMCPServer] Configuration summary:', this._configManager.getConfigSummary());
       this.emit('config-reloaded', mcpConfig);
     } catch (error) {
       console.log(`[RemixMCPServer] Failed to reload config: ${error.message}`);
@@ -796,7 +780,6 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
 
   updateMCPConfig(partialConfig: Partial<any>): void {
     this._configManager.updateConfig(partialConfig);
-    console.log('[RemixMCPServer] Configuration updated at runtime');
     this.emit('config-updated', this._configManager.getConfig());
   }
 
@@ -894,19 +877,13 @@ export class RemixMCPServer extends EventEmitter implements IRemixMCPServer {
       this._resources.register(compilationProvider);
 
       // Register deployment resource provider
-      const deploymentProvider = new DeploymentResourceProvider();
+      const deploymentProvider = new DeploymentResourceProvider(this._plugin);
       this._resources.register(deploymentProvider);
 
       // Register tutorial resource provider
       const tutorialsProvider = new TutorialsResourceProvider(this._plugin);
       this._resources.register(tutorialsProvider);
 
-      // Register Amp resource provider
-      /*
-        const ampProvider = new AmpResourceProvider(this._plugin);
-        this._resources.register(ampProvider);
-      */
-
       // Register debugging resource provider
       const debuggingProvider = new DebuggingResourceProvider(this._plugin);
       this._resources.register(debuggingProvider);