fix: resolve all linting, formatting, and type checking issues

- Fix flake8 E231 error: add missing whitespace after comma in ml_engine.py:138
- Fix flake8 E128 error: correct continuation line indentation in ml_engine.py:191
- Fix mypy type errors in parser.py: use field(default_factory=...) for dataclass fields
- Fix mypy type errors in ml_engine.py: add return type annotations and proper type hints
- Fix mypy type errors in api.py: add type: ignore comments for Flask imports
- Fix mypy type errors in plugins.py: add null checks for importlib operations
- Apply Black formatting to all files for consistent code style
- Add proper imports for field and Any types where needed

All files now pass:
- flake8 (no linting errors)
- black (properly formatted)
- mypy (no type errors)

Author: sdirishguy

shellrosetta/api.py

@@ -7,11 +7,11 @@
     FLASK_AVAILABLE = True
 except ImportError:
     FLASK_AVAILABLE = False
-    Flask = None
-    request = None
-    jsonify = None
-    render_template_string = None
-    CORS = None
+    Flask = None  # type: ignore
+    request = None  # type: ignore
+    jsonify = None  # type: ignore
+    render_template_string = None  # type: ignore
+    CORS = None  # type: ignore
 
 from .core import lnx2ps, ps2lnx
 from .ml_engine import ml_engine

shellrosetta/ml_engine.py

@@ -88,7 +88,7 @@ def __init__(self):
 
         self.load_data()
 
-    def load_data(self):
+    def load_data(self) -> None:
         """Load learned patterns and context"""
         # Load patterns
         if self.patterns_file.exists():
@@ -116,7 +116,7 @@ def load_data(self):
             except Exception as e:
                 print(f"Failed to load suggestions: {e}")
 
-    def save_data(self):
+    def save_data(self) -> None:
         """Save learned patterns and context"""
         # Save patterns
         patterns_data = {}
@@ -145,7 +145,7 @@ def save_data(self):
 
     def learn_pattern(
         self, command: str, translation: str, direction: str, success: bool = True
-    ):
+    ) -> None:
         """Learn a new command pattern"""
         key = f"{direction}:{command}"
 
@@ -282,7 +282,7 @@ def analyze_patterns(self) -> Dict[str, Any]:
         )
 
         # Most common command types
-        command_types = Counter()
+        command_types: Counter[str] = Counter()
         for pattern in self.patterns.values():
             command_types[self._classify_command(pattern.command)] += 1
 

shellrosetta/parser.py

@@ -4,8 +4,8 @@
 import re
 import shlex
 from enum import Enum
-from dataclasses import dataclass
-from typing import Dict, List, Optional, Tuple, Union
+from dataclasses import dataclass, field
+from typing import Dict, List, Optional, Tuple, Union, Any
 
 
 class NodeType(Enum):
@@ -24,14 +24,8 @@ class ASTNode:
 
     node_type: NodeType
     value: str
-    children: List["ASTNode"] = None
-    metadata: Dict = None
-
-    def __post_init__(self):
-        if self.children is None:
-            self.children = []
-        if self.metadata is None:
-            self.metadata = {}
+    children: List["ASTNode"] = field(default_factory=list)
+    metadata: Dict[str, Any] = field(default_factory=dict)
 
 
 class CommandParser:

shellrosetta/plugins.py

@@ -54,31 +54,34 @@ def __init__(self):
         self.plugin_dir.mkdir(parents=True, exist_ok=True)
         self.load_plugins()
 
-    def load_plugins(self):
+    def load_plugins(self) -> None:
         """Load all available plugins"""
         # Load built-in plugins
         self._load_builtin_plugins()
 
         # Load user plugins
         self._load_user_plugins()
 
-    def _load_builtin_plugins(self):
+    def _load_builtin_plugins(self) -> None:
         """Load built-in plugins"""
         # Create plugin instances directly instead of importing modules
         self.plugins["docker"] = docker_plugin
         self.plugins["kubernetes"] = kubernetes_plugin
         self.plugins["aws"] = aws_plugin
         self.plugins["git"] = git_plugin
 
-    def _load_user_plugins(self):
+    def _load_user_plugins(self) -> None:
         """Load user-installed plugins"""
         for plugin_file in self.plugin_dir.glob("*.py"):
             try:
                 spec = importlib.util.spec_from_file_location(
                     plugin_file.stem, plugin_file
                 )
+                if spec is None:
+                    continue
                 module = importlib.util.module_from_spec(spec)
-                spec.loader.exec_module(module)
+                if spec.loader is not None:
+                    spec.loader.exec_module(module)
 
                 if hasattr(module, "plugin"):
                     plugin = module.plugin