mcli-framework 7.12.2__py3-none-any.whl → 7.12.4__py3-none-any.whl

mcli/chat/command_rag.py

@@ -2,12 +2,8 @@
 RAG-based Command Search and Self-Referential System for MCLI Chatbot
 """
 
-import asyncio
-import json
-import os
 import re
 import subprocess
-import sys
 from dataclasses import asdict, dataclass
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Tuple

mcli/chat/enhanced_chat.py

@@ -3,25 +3,18 @@ Enhanced MCLI Chat Assistant with Self-Referential Capabilities and RAG-based Co
 """
 
 import asyncio
-import json
 import os
 import re
-import readline
 from datetime import datetime
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, Optional
 
-import requests
 from rich.console import Console
 from rich.panel import Panel
 from rich.table import Table
-from rich.text import Text
 
-from mcli.chat.command_rag import get_command_rag_system
-from mcli.chat.system_integration import handle_system_request
 from mcli.lib.api.daemon_client import get_daemon_client
 from mcli.lib.logger.logger import get_logger
 from mcli.lib.toml.toml import read_from_toml
-from mcli.lib.ui.styling import console
 
 logger = get_logger(__name__)
 
@@ -81,7 +74,7 @@ I can execute system commands and searches to provide real-time information and
 
 
 class EnhancedChatClient:
-    """Enhanced chat client with RAG-based command search and self-referential capabilities"""
+    """Enhanced chat client with RAG-based command search and self-referential capabilities."""
 
     def __init__(self, use_remote: bool = False, model_override: str = None):
         self.daemon = get_daemon_client()
@@ -125,7 +118,7 @@ class EnhancedChatClient:
             self.commands_list = []
 
     async def _initialize_rag_system(self):
-        """Initialize the RAG system for command search"""
+        """Initialize the RAG system for command search."""
         try:
             # Use a simplified command discovery for now to avoid hanging
             from mcli.lib.discovery.command_discovery import ClickCommandDiscovery
@@ -152,9 +145,7 @@ class EnhancedChatClient:
             self.simple_command_lookup = {}
 
     def _configure_model_settings(self):
-        """Configure model settings with enhanced system prompt"""
-        global config
-
+        """Configure model settings with enhanced system prompt."""
         if not self.use_remote:
             config.update(
                 {
@@ -175,7 +166,7 @@ class EnhancedChatClient:
                 self._configure_model_settings()
 
     def _ensure_daemon_running(self):
-        """Ensure the daemon is running for system integration"""
+        """Ensure the daemon is running for system integration."""
         try:
             if not self.daemon.health_check():
                 self.daemon.start_daemon()
@@ -183,7 +174,7 @@ class EnhancedChatClient:
             logger.debug(f"Daemon check failed: {e}")
 
     async def _enrich_message_with_context(self, user_message: str) -> str:
-        """Enrich user message with relevant command context and system information"""
+        """Enrich user message with relevant command context and system information."""
         enriched_parts = [user_message]
 
         # Initialize RAG system if not already done
@@ -195,7 +186,7 @@ class EnhancedChatClient:
             # Add system status only
             system_status = await self._get_system_status()
             if system_status:
-                enriched_parts.append(f"\n--- SYSTEM STATUS ---")
+                enriched_parts.append("\n--- SYSTEM STATUS ---")
                 enriched_parts.append(system_status)
 
         except Exception as e:
@@ -204,7 +195,7 @@ class EnhancedChatClient:
         return "\n".join(enriched_parts)
 
     async def _get_system_status(self) -> str:
-        """Get current system status for context"""
+        """Get current system status for context."""
         status_parts = []
 
         try:
@@ -221,7 +212,7 @@ class EnhancedChatClient:
                 status_parts.append("Redis cache: Active")
             else:
                 status_parts.append("Redis cache: Inactive")
-        except:
+        except Exception:
             pass
 
         try:
@@ -235,13 +226,13 @@ class EnhancedChatClient:
                 )
             else:
                 status_parts.append("Rust extensions: Inactive")
-        except:
+        except Exception:
             pass
 
         return " | ".join(status_parts) if status_parts else "System status unavailable"
 
     def start_interactive_session(self):
-        """Start enhanced interactive chat session"""
+        """Start enhanced interactive chat session."""
         self.console.print("\n🤖 MCLI Enhanced Chat Assistant", style="bold cyan")
         self.console.print(
             "I can help you discover and use MCLI commands through intelligent search!",
@@ -287,7 +278,7 @@ class EnhancedChatClient:
                 self.console.print(f"❌ Error: {e}", style="red")
 
     async def _process_enhanced_message(self, user_message: str):
-        """Process message with RAG enhancement"""
+        """Process message with RAG enhancement."""
         try:
             # Enrich message with command context
             enriched_message = await self._enrich_message_with_context(user_message)
@@ -317,7 +308,7 @@ class EnhancedChatClient:
             self.console.print(f"❌ Error processing message: {e}", style="red")
 
     async def _get_ai_response(self, message: str) -> Optional[str]:
-        """Get AI response using configured provider"""
+        """Get AI response using configured provider."""
         try:
             if self.use_remote and config.get("openai_api_key"):
                 return await self._get_openai_response(message)
@@ -330,7 +321,7 @@ class EnhancedChatClient:
             return None
 
     async def _get_openai_response(self, message: str) -> str:
-        """Get response from OpenAI"""
+        """Get response from OpenAI."""
         try:
             import openai
 
@@ -355,7 +346,7 @@ class EnhancedChatClient:
             return None
 
     async def _get_anthropic_response(self, message: str) -> str:
-        """Get response from Anthropic"""
+        """Get response from Anthropic."""
         try:
             import anthropic
 
@@ -375,7 +366,7 @@ class EnhancedChatClient:
             return None
 
     async def _get_local_response(self, message: str) -> str:
-        """Get response from local lightweight model"""
+        """Get response from local lightweight model."""
         try:
             # Use improved command matching for local responses
             if hasattr(self, "commands_list") and self.commands_list:
@@ -392,7 +383,7 @@ class EnhancedChatClient:
                     cmd_name_clean = cmd.full_name.lower().replace(".", " ")
                     if user_lower in cmd_name_clean:
                         score += 10
-                        matched_reasons.append(f"exact phrase match")
+                        matched_reasons.append("exact phrase match")
 
                     # High priority: Check if user input matches command structure
                     user_parts = user_lower.split()
@@ -404,7 +395,7 @@ class EnhancedChatClient:
                         for user_word in user_parts
                     ):
                         score += 8
-                        matched_reasons.append(f"matches command structure")
+                        matched_reasons.append("matches command structure")
 
                     # Medium priority: Individual word matches in name
                     for word in user_words:
@@ -422,7 +413,7 @@ class EnhancedChatClient:
                     keyword_map = {
                         "video": ["video", "mp4", "avi", "mov", "ffmpeg", "frames"],
                         "redis": ["redis", "cache"],
-                        "file": ["file", "convert", "pdf", "oxps"],
+                        "file": ["file", "convert", "pd", "oxps"],
                         "daemon": ["daemon", "service", "api"],
                         "workflow": ["workflow", "automation"],
                         "system": ["system", "status", "monitor"],
@@ -468,9 +459,9 @@ class EnhancedChatClient:
                     # Suggest broader search
                     return (
                         f"I didn't find specific commands for '{message}', but I can help you explore!\n\n"
-                        f"Try these approaches:\n"
-                        f"• Use 'commands' to browse all available commands\n"
-                        f"• Ask about specific topics like 'video processing', 'file conversion', 'system monitoring'\n"
+                        "Try these approaches:\n"
+                        "• Use 'commands' to browse all available commands\n"
+                        "• Ask about specific topics like 'video processing', 'file conversion', 'system monitoring'\n"
                         f"• I have {len(self.commands_list)} commands available across categories like workflow, redis, files, and more!"
                     )
 
@@ -478,10 +469,10 @@ class EnhancedChatClient:
 
         except Exception as e:
             logger.error(f"Local response failed: {e}")
-            return f"I'm here to help with MCLI commands. Try asking about specific tasks like 'video processing' or 'file conversion'."
+            return "I'm here to help with MCLI commands. Try asking about specific tasks like 'video processing' or 'file conversion'."
 
     def _display_response(self, response: str):
-        """Display AI response with enhanced formatting"""
+        """Display AI response with enhanced formatting."""
         # Create a panel for the response
         panel = Panel(
             response,
@@ -493,7 +484,7 @@ class EnhancedChatClient:
         self.console.print(panel)
 
     async def _highlight_mcli_commands(self, response: str):
-        """Extract and highlight MCLI commands from response"""
+        """Extract and highlight MCLI commands from response."""
         # Look for command patterns in the response
         command_pattern = r"`mcli\s+([^`]+)`|mcli\s+([\w\s\-\.]+)"
         matches = re.findall(command_pattern, response, re.IGNORECASE)
@@ -506,7 +497,7 @@ class EnhancedChatClient:
                     self.console.print(f"   • `mcli {command.strip()}`", style="green")
 
     async def _handle_command_search(self, query: str):
-        """Handle command search queries"""
+        """Handle command search queries."""
         search_term = query[8:].strip() if len(query) > 8 else ""  # Remove "commands"
 
         if not self.rag_system:
@@ -524,7 +515,7 @@ class EnhancedChatClient:
             await self._search_and_display_commands(search_term)
 
     def _show_command_categories(self, capabilities: Dict[str, Any]):
-        """Show command categories"""
+        """Show command categories."""
         table = Table(title="📋 MCLI Command Categories")
         table.add_column("Category", style="cyan")
         table.add_column("Commands", justify="right", style="magenta")
@@ -543,7 +534,7 @@ class EnhancedChatClient:
         )
 
     async def _search_and_display_commands(self, search_term: str):
-        """Search and display matching commands"""
+        """Search and display matching commands."""
         try:
             results = await self.rag_system.search_commands(search_term, limit=8)
 
@@ -569,7 +560,7 @@ class EnhancedChatClient:
             self.console.print(f"❌ Search failed: {e}", style="red")
 
     async def _show_system_status(self):
-        """Show comprehensive system status"""
+        """Show comprehensive system status."""
         try:
             status_info = await self._get_system_status()
 
@@ -590,7 +581,7 @@ class EnhancedChatClient:
             self.console.print(f"❌ Status check failed: {e}", style="red")
 
     def _show_help(self):
-        """Show enhanced help information"""
+        """Show enhanced help information."""
         help_text = """
 🤖 **MCLI Enhanced Chat Assistant Help**
 
@@ -629,7 +620,7 @@ class EnhancedChatClient:
         self.console.print(help_panel)
 
     def _handle_quit(self):
-        """Handle quit command"""
+        """Handle quit command."""
         self.session_active = False
         self.console.print("\n🎯 **Session Summary:**")
         if self.conversation_context:
@@ -648,5 +639,5 @@ class EnhancedChatClient:
 def create_enhanced_chat_client(
     use_remote: bool = False, model_override: str = None
 ) -> EnhancedChatClient:
-    """Create enhanced chat client instance"""
+    """Create enhanced chat client instance."""
     return EnhancedChatClient(use_remote=use_remote, model_override=model_override)

mcli/chat/system_controller.py

@@ -9,7 +9,7 @@ import subprocess
 import time
 from datetime import datetime
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, List
 
 from mcli.lib.logger.logger import get_logger
 
@@ -17,7 +17,7 @@ logger = get_logger(__name__)
 
 
 class SystemController:
-    """Handles real-time system control for MCLI chat"""
+    """Handles real-time system control for MCLI chat."""
 
     def __init__(self):
         self.system = platform.system()
@@ -25,7 +25,7 @@ class SystemController:
         self.current_directory = os.getcwd()  # Track current working directory
 
     def execute_command(self, command: str, description: str = "") -> Dict[str, Any]:
-        """Execute a system command and return results"""
+        """Execute a system command and return results."""
         try:
             logger.info(f"Executing: {description or command}")
 
@@ -57,7 +57,7 @@ class SystemController:
             }
 
     def open_textedit_and_write(self, text: str, filename: str = None) -> Dict[str, Any]:
-        """Open TextEdit, write text, and optionally save to file"""
+        """Open TextEdit, write text, and optionally save to file."""
         if self.system != "Darwin":
             return {
                 "success": False,
@@ -93,7 +93,7 @@ class SystemController:
         )
 
     def control_application(self, app_name: str, action: str, **kwargs) -> Dict[str, Any]:
-        """Control various applications with different actions"""
+        """Control various applications with different actions."""
 
         if self.system == "Darwin":  # macOS
             return self._control_macos_app(app_name, action, **kwargs)
@@ -103,7 +103,7 @@ class SystemController:
             return self._control_linux_app(app_name, action, **kwargs)
 
     def _control_macos_app(self, app_name: str, action: str, **kwargs) -> Dict[str, Any]:
-        """Control macOS applications using AppleScript"""
+        """Control macOS applications using AppleScript."""
 
         if action == "open":
             applescript = f'tell application "{app_name}" to activate'
@@ -149,7 +149,7 @@ class SystemController:
         return self.execute_command(f"osascript -e '{applescript}'", f"{action} {app_name}")
 
     def _control_windows_app(self, app_name: str, action: str, **kwargs) -> Dict[str, Any]:
-        """Control Windows applications using PowerShell"""
+        """Control Windows applications using PowerShell."""
 
         if action == "open":
             # Try to start the application
@@ -173,7 +173,7 @@ class SystemController:
         return self.execute_command(command, f"{action} {app_name}")
 
     def _control_linux_app(self, app_name: str, action: str, **kwargs) -> Dict[str, Any]:
-        """Control Linux applications using various tools"""
+        """Control Linux applications using various tools."""
 
         if action == "open":
             # Try different methods to open applications
@@ -212,7 +212,7 @@ class SystemController:
         return self.execute_command(command, f"{action} {app_name}")
 
     def get_system_info(self) -> Dict[str, Any]:
-        """Get comprehensive system information"""
+        """Get comprehensive system information."""
         try:
             import platform
             from datetime import datetime
@@ -261,7 +261,7 @@ class SystemController:
 
             # Disk information
             disk_info = []
-            try:
+            try:  # noqa: SIM105
                 for partition in psutil.disk_partitions():
                     try:
                         partition_usage = psutil.disk_usage(partition.mountpoint)
@@ -278,7 +278,7 @@ class SystemController:
                                 ),
                             }
                         )
-                    except (PermissionError, OSError):
+                    except OSError:
                         # Skip partitions we can't access
                         continue
             except Exception:
@@ -291,12 +291,12 @@ class SystemController:
                 "connections": 0,
             }
 
-            try:
+            try:  # noqa: SIM105
                 network_info["connections"] = len(psutil.net_connections())
             except Exception:
                 pass
 
-            try:
+            try:  # noqa: SIM105
                 for interface, addresses in psutil.net_if_addrs().items():
                     network_info["interfaces"][interface] = []
                     for addr in addresses:
@@ -365,7 +365,7 @@ class SystemController:
             return {"success": False, "error": str(e), "description": "Get system information"}
 
     def get_system_time(self) -> Dict[str, Any]:
-        """Get current system time and timezone information"""
+        """Get current system time and timezone information."""
         try:
             import time
             from datetime import datetime
@@ -388,7 +388,7 @@ class SystemController:
             return {"success": False, "error": str(e), "description": "Get system time"}
 
     def get_memory_usage(self) -> Dict[str, Any]:
-        """Get detailed memory usage information"""
+        """Get detailed memory usage information."""
         try:
             import psutil
 
@@ -443,7 +443,7 @@ class SystemController:
             return {"success": False, "error": str(e), "description": "Get memory usage"}
 
     def get_disk_usage(self) -> Dict[str, Any]:
-        """Get detailed disk usage information"""
+        """Get detailed disk usage information."""
         try:
             import psutil
 
@@ -503,7 +503,7 @@ class SystemController:
             return {"success": False, "error": str(e), "description": "Get disk usage"}
 
     def clear_system_caches(self) -> Dict[str, Any]:
-        """Clear system caches and temporary files"""
+        """Clear system caches and temporary files."""
         try:
             cleared_items = []
             total_freed_mb = 0
@@ -598,22 +598,22 @@ class SystemController:
             return {"success": False, "error": str(e), "description": "Clear system caches"}
 
     def _get_directory_size(self, path: str) -> int:
-        """Get total size of directory in bytes"""
+        """Get total size of directory in bytes."""
         total_size = 0
-        try:
+        try:  # noqa: SIM105
             for dirpath, dirnames, filenames in os.walk(path):
                 for filename in filenames:
                     file_path = os.path.join(dirpath, filename)
                     try:
                         total_size += os.path.getsize(file_path)
-                    except (OSError, FileNotFoundError):
+                    except OSError:
                         continue
         except Exception:
             pass
         return total_size
 
     def get_running_applications(self) -> List[str]:
-        """Get list of currently running applications"""
+        """Get list of currently running applications."""
         try:
             if self.system == "Darwin":
                 result = subprocess.run(
@@ -664,7 +664,7 @@ class SystemController:
         return []
 
     def take_screenshot(self, filename: str = None) -> Dict[str, Any]:
-        """Take a screenshot and save it"""
+        """Take a screenshot and save it."""
         if not filename:
             timestamp = time.strftime("%Y%m%d_%H%M%S")
             filename = f"mcli_screenshot_{timestamp}.png"
@@ -694,7 +694,7 @@ class SystemController:
             }
 
     def open_file_or_url(self, path_or_url: str) -> Dict[str, Any]:
-        """Open a file or URL using the system default application"""
+        """Open a file or URL using the system default application."""
         try:
             if self.system == "Darwin":
                 command = f"open '{path_or_url}'"
@@ -709,7 +709,7 @@ class SystemController:
             return {"success": False, "error": str(e), "description": f"Open: {path_or_url}"}
 
     def change_directory(self, path: str) -> Dict[str, Any]:
-        """Navigate to a directory and update current working directory"""
+        """Navigate to a directory and update current working directory."""
         try:
             # Expand user path and resolve relative paths
             expanded_path = os.path.expanduser(path)
@@ -751,7 +751,7 @@ class SystemController:
     def list_directory(
         self, path: str = None, show_hidden: bool = False, detailed: bool = False
     ) -> Dict[str, Any]:
-        """List contents of a directory"""
+        """List contents of a directory."""
         try:
             target_path = path if path else self.current_directory
             expanded_path = os.path.expanduser(target_path)
@@ -826,7 +826,7 @@ class SystemController:
             }
 
     def clean_simulator_data(self) -> Dict[str, Any]:
-        """Clean iOS/watchOS simulator data specifically"""
+        """Clean iOS/watchOS simulator data specifically."""
         try:
             if self.system != "Darwin":
                 return {
@@ -910,7 +910,7 @@ class SystemController:
             return {"success": False, "error": str(e), "description": "Clean simulator data"}
 
     def execute_shell_command(self, command: str, working_directory: str = None) -> Dict[str, Any]:
-        """Execute a shell command in a specific directory"""
+        """Execute a shell command in a specific directory."""
         try:
             # Use working directory if specified, otherwise use current directory
             cwd = working_directory if working_directory else self.current_directory
@@ -959,52 +959,52 @@ system_controller = SystemController()
 
 # Helper functions for easy use in chat
 def open_textedit_and_write(text: str, filename: str = None) -> Dict[str, Any]:
-    """Helper function to open TextEdit and write text"""
+    """Helper function to open TextEdit and write text."""
     return system_controller.open_textedit_and_write(text, filename)
 
 
 def control_app(app_name: str, action: str, **kwargs) -> Dict[str, Any]:
-    """Helper function to control applications"""
+    """Helper function to control applications."""
     return system_controller.control_application(app_name, action, **kwargs)
 
 
 def execute_system_command(command: str, description: str = "") -> Dict[str, Any]:
-    """Helper function to execute system commands"""
+    """Helper function to execute system commands."""
     return system_controller.execute_command(command, description)
 
 
 def take_screenshot(filename: str = None) -> Dict[str, Any]:
-    """Helper function to take screenshots"""
+    """Helper function to take screenshots."""
     return system_controller.take_screenshot(filename)
 
 
 def open_file_or_url(path_or_url: str) -> Dict[str, Any]:
-    """Helper function to open files or URLs"""
+    """Helper function to open files or URLs."""
     return system_controller.open_file_or_url(path_or_url)
 
 
 def change_directory(path: str) -> Dict[str, Any]:
-    """Helper function to navigate to a directory"""
+    """Helper function to navigate to a directory."""
     return system_controller.change_directory(path)
 
 
 def list_directory(
     path: str = None, show_hidden: bool = False, detailed: bool = False
 ) -> Dict[str, Any]:
-    """Helper function to list directory contents"""
+    """Helper function to list directory contents."""
     return system_controller.list_directory(path, show_hidden, detailed)
 
 
 def clean_simulator_data() -> Dict[str, Any]:
-    """Helper function to clean iOS/watchOS simulator data"""
+    """Helper function to clean iOS/watchOS simulator data."""
     return system_controller.clean_simulator_data()
 
 
 def execute_shell_command(command: str, working_directory: str = None) -> Dict[str, Any]:
-    """Helper function to execute shell commands"""
+    """Helper function to execute shell commands."""
     return system_controller.execute_shell_command(command, working_directory)
 
 
 def get_current_directory() -> str:
-    """Helper function to get current working directory"""
+    """Helper function to get current working directory."""
     return system_controller.current_directory

mcli/chat/system_integration.py

@@ -3,8 +3,7 @@ System Integration for MCLI Chat
 Provides system control capabilities directly within chat conversations
 """
 
-import json
-from typing import Any, Dict, List
+from typing import Any, Dict
 
 from mcli.lib.logger.logger import get_logger
 
@@ -692,7 +691,7 @@ class ChatSystemIntegration:
                 swap = memory_data["swap_memory"]
 
                 summary = [
-                    f"💾 Memory Usage:",
+                    "💾 Memory Usage:",
                     f"  RAM: {vm['used_gb']:.1f}GB used / {vm['total_gb']:.1f}GB total ({vm['usage_percent']}%)",
                     f"  Available: {vm['available_gb']:.1f}GB",
                     f"  Swap: {swap['used_gb']:.1f}GB used / {swap['total_gb']:.1f}GB total ({swap['usage_percent']}%)",
@@ -723,7 +722,7 @@ class ChatSystemIntegration:
             if result["success"]:
                 disk_data = result["data"]
 
-                summary = [f"💽 Disk Usage:"]
+                summary = ["💽 Disk Usage:"]
 
                 # Show main disk first
                 if disk_data["total_disk_gb"] > 0:
@@ -763,7 +762,7 @@ class ChatSystemIntegration:
             if result["success"]:
                 cache_data = result["data"]
 
-                summary = [f"🧹 Cache Cleanup Results:"]
+                summary = ["🧹 Cache Cleanup Results:"]
 
                 if cache_data["cleared_items"]:
                     for item in cache_data["cleared_items"]:

mcli/cli.py

@@ -5,8 +5,7 @@ from mcli.lib.api.mcli_decorators import chat
 
 @click.group()
 def cli():
-    """MCLI - Modern Command Line Interface"""
-    pass
+    """MCLI - Modern Command Line Interface."""
 
 
 # Add the chat command group
@@ -15,7 +14,7 @@ cli.add_command(chat(name="chat"))
 
 @cli.command()
 def version():
-    """Show MCLI version"""
+    """Show MCLI version."""
     from mcli import __version__
 
     click.echo(f"MCLI version {__version__}")