fix: resolve linting and formatting 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
- Apply Black formatting to 11 files to ensure consistent code style
- All files now pass both flake8 and black checks

Files modified:
- shellrosetta/config.py
- shellrosetta/parser.py
- shellrosetta/api.py
- shellrosetta/core.py
- shellrosetta/utils.py
- shellrosetta/cli.py
- shellrosetta/plugins.py
- shellrosetta/mappings.py
- shellrosetta/ml_engine.py
- tests/test_core.py
- tests/test_advanced_features.py

Author: sdirishguy

shellrosetta/api.py

@@ -3,6 +3,7 @@
 try:
     from flask import Flask, request, jsonify, render_template_string
     from flask_cors import CORS
+
     FLASK_AVAILABLE = True
 except ImportError:
     FLASK_AVAILABLE = False
@@ -254,53 +255,57 @@
 """
 
 
-def run_api_server(host='0.0.0.0', port=5000, debug=False):
+def run_api_server(host="0.0.0.0", port=5000, debug=False):
     """Run the API server"""
     if not FLASK_AVAILABLE:
-        print("Error: Flask is not installed. Install it with: pip install flask flask-cors")
+        print(
+            "Error: Flask is not installed. Install it with: pip install flask flask-cors"
+        )
         return
 
     app = Flask(__name__)
     CORS(app)  # Enable CORS for all routes
 
-    @app.route('/')
+    @app.route("/")
     def index():
         """Serve the web interface"""
         return render_template_string(HTML_TEMPLATE)
 
-    @app.route('/api/translate', methods=['POST'])
+    @app.route("/api/translate", methods=["POST"])
     def translate_command():
         """Translate a command via API"""
         try:
             data = request.get_json()
-            command = data.get('command', '').strip()
-            direction = data.get('direction', 'lnx2ps')
+            command = data.get("command", "").strip()
+            direction = data.get("direction", "lnx2ps")
 
             if not command:
-                return jsonify({'error': 'No command provided'}), 400
+                return jsonify({"error": "No command provided"}), 400
 
             # Try plugin translation first
-            plugin_translation = plugin_manager.translate_with_plugins(command, direction)
+            plugin_translation = plugin_manager.translate_with_plugins(
+                command, direction
+            )
 
             # Try ML translation
             ml_translation = ml_engine.get_best_translation(command, direction)
 
             # Use core translation
-            if direction == 'lnx2ps':
+            if direction == "lnx2ps":
                 translation = lnx2ps(command)
             else:
                 translation = ps2lnx(command)
 
             # Prefer plugin translation, then ML, then core
             if plugin_translation:
                 final_translation = plugin_translation
-                source = 'plugin'
+                source = "plugin"
             elif ml_translation:
                 final_translation = ml_translation
-                source = 'ml'
+                source = "ml"
             else:
                 final_translation = translation
-                source = 'core'
+                source = "core"
 
             # Learn the pattern
             ml_engine.learn_pattern(command, final_translation, direction, success=True)
@@ -309,93 +314,98 @@ def translate_command():
             suggestions = ml_engine.get_suggestions(command, direction, limit=3)
             suggestion_texts = [s[0] for s in suggestions]
 
-            return jsonify({
-                'translation': final_translation,
-                'source': source,
-                'suggestions': suggestion_texts,
-                'command': command,
-                'direction': direction
-            })
+            return jsonify(
+                {
+                    "translation": final_translation,
+                    "source": source,
+                    "suggestions": suggestion_texts,
+                    "command": command,
+                    "direction": direction,
+                }
+            )
 
         except Exception as e:
-            return jsonify({'error': str(e)}), 500
+            return jsonify({"error": str(e)}), 500
 
-    @app.route('/api/stats')
+    @app.route("/api/stats")
     def get_stats():
         """Get translation statistics"""
         try:
             analysis = ml_engine.analyze_patterns()
 
-            return jsonify({
-                'total_translations': analysis.get('total_patterns', 0),
-                'success_rate': analysis.get('success_rate', 0),
-                'learned_patterns': analysis.get('total_patterns', 0),
-                'command_types': analysis.get('command_types', {}),
-                'top_patterns': analysis.get('top_successful_patterns', [])
-            })
+            return jsonify(
+                {
+                    "total_translations": analysis.get("total_patterns", 0),
+                    "success_rate": analysis.get("success_rate", 0),
+                    "learned_patterns": analysis.get("total_patterns", 0),
+                    "command_types": analysis.get("command_types", {}),
+                    "top_patterns": analysis.get("top_successful_patterns", []),
+                }
+            )
 
         except Exception as e:
-            return jsonify({'error': str(e)}), 500
+            return jsonify({"error": str(e)}), 500
 
-    @app.route('/api/plugins')
+    @app.route("/api/plugins")
     def list_plugins():
         """List available plugins"""
         try:
             plugins = plugin_manager.list_plugins()
-            return jsonify({'plugins': plugins})
+            return jsonify({"plugins": plugins})
 
         except Exception as e:
-            return jsonify({'error': str(e)}), 500
+            return jsonify({"error": str(e)}), 500
 
-    @app.route('/api/plugins/<plugin_name>')
+    @app.route("/api/plugins/<plugin_name>")
     def get_plugin_info(plugin_name):
         """Get information about a specific plugin"""
         try:
             if plugin_name in plugin_manager.plugins:
                 plugin = plugin_manager.plugins[plugin_name]
                 return jsonify(plugin.get_metadata())
             else:
-                return jsonify({'error': 'Plugin not found'}), 404
+                return jsonify({"error": "Plugin not found"}), 404
 
         except Exception as e:
-            return jsonify({'error': str(e)}), 500
+            return jsonify({"error": str(e)}), 500
 
-    @app.route('/api/learn', methods=['POST'])
+    @app.route("/api/learn", methods=["POST"])
     def learn_pattern():
         """Manually learn a pattern"""
         try:
             data = request.get_json()
-            command = data.get('command')
-            translation = data.get('translation')
-            direction = data.get('direction')
-            success = data.get('success', True)
+            command = data.get("command")
+            translation = data.get("translation")
+            direction = data.get("direction")
+            success = data.get("success", True)
 
             if not all([command, translation, direction]):
-                return jsonify({'error': 'Missing required fields'}), 400
+                return jsonify({"error": "Missing required fields"}), 400
 
             ml_engine.learn_pattern(command, translation, direction, success)
-            return jsonify({'message': 'Pattern learned successfully'})
+            return jsonify({"message": "Pattern learned successfully"})
 
         except Exception as e:
-            return jsonify({'error': str(e)}), 500
+            return jsonify({"error": str(e)}), 500
 
-    @app.route('/api/cleanup', methods=['POST'])
+    @app.route("/api/cleanup", methods=["POST"])
     def cleanup_patterns():
         """Clean up old patterns"""
         try:
             data = request.get_json()
-            days = data.get('days', 30)
+            days = data.get("days", 30)
 
             ml_engine.cleanup_old_patterns(days)
-            return jsonify({'message': f'Cleaned up patterns older than {days} days'})
+            return jsonify({"message": f"Cleaned up patterns older than {days} days"})
 
         except Exception as e:
-            return jsonify({'error': str(e)}), 500
+            return jsonify({"error": str(e)}), 500
 
     print(f"Starting ShellRosetta API server on http://{host}:{port}")
     print("Press Ctrl+C to stop the server")
 
     app.run(host=host, port=port, debug=debug)
 
-if __name__ == '__main__':
+
+if __name__ == "__main__":
     run_api_server()

shellrosetta/cli.py

@@ -3,22 +3,30 @@
 from .config import config
 from .plugins import plugin_manager
 from .ml_engine import ml_engine
-from .utils import print_header, print_translation, sanitize_command, format_command_history
+from .utils import (
+    print_header,
+    print_translation,
+    sanitize_command,
+    format_command_history,
+)
 from .api import run_api_server
 
 import sys
+
 try:
     import readline
 except ImportError:
     # readline not available on Windows
     pass
 
 from .core import lnx2ps, ps2lnx
+
+
 def show_help():
     print_header()
     print("Usage:")
-    print("  shellrosetta lnx2ps \"linux command here\"")
-    print("  shellrosetta ps2lnx \"powershell command here\"")
+    print('  shellrosetta lnx2ps "linux command here"')
+    print('  shellrosetta ps2lnx "powershell command here"')
     print("  shellrosetta          # Interactive shell mode")
     print("  shellrosetta config   # Show configuration")
     print("  shellrosetta history  # Show command history")
@@ -27,16 +35,18 @@ def show_help():
     print("  shellrosetta ml       # Show ML insights")
     print("")
     print("Examples:")
-    print("  shellrosetta lnx2ps \"ls -alh | grep foo\"")
-    print("  shellrosetta ps2lnx \"Get-ChildItem -Force | Select-String foo\"")
+    print('  shellrosetta lnx2ps "ls -alh | grep foo"')
+    print('  shellrosetta ps2lnx "Get-ChildItem -Force | Select-String foo"')
     print("  shellrosetta api --port 8080  # Start API on port 8080")
     print("=" * 65)
 
 
 def run_interactive():
     print_header()
     print("Welcome to ShellRosetta Interactive Mode!")
-    print("Type 'exit' to quit, 'mode' to switch translation direction, or 'help' for commands.")
+    print(
+        "Type 'exit' to quit, 'mode' to switch translation direction, or 'help' for commands."
+    )
 
     # Initialize history
     history = []
@@ -52,7 +62,9 @@ def run_interactive():
             show_interactive_help()
             continue
 
-    print(f"Type your {mode.upper()} commands below. Type 'mode' to switch, 'exit' to quit.\n")
+    print(
+        f"Type your {mode.upper()} commands below. Type 'mode' to switch, 'exit' to quit.\n"
+    )
 
     while True:
         try:
@@ -107,7 +119,7 @@ def run_interactive():
 
         # Store in history
         history.append((sanitized, translated, mode))
-        if len(history) > config.get('max_history', 100):
+        if len(history) > config.get("max_history", 100):
             history.pop(0)
 
         # Print translation
@@ -144,7 +156,7 @@ def show_plugins():
         for plugin in plugins:
             print(f"  {plugin['name']} v{plugin['version']}")
             print(f"    Commands: {', '.join(plugin['supported_commands'])}")
-            if plugin.get('description'):
+            if plugin.get("description"):
                 print(f"    Description: {plugin['description']}")
             print()
     else:
@@ -162,7 +174,7 @@ def show_ml_insights():
         print(f"  Success Rate: {analysis.get('success_rate', 0):.1%}")
         print(f"  Command Types: {dict(analysis.get('command_types', {}))}")
 
-        top_patterns = analysis.get('top_successful_patterns', [])
+        top_patterns = analysis.get("top_successful_patterns", [])
         if top_patterns:
             print("\n  Top Successful Patterns:")
             for cmd, rate in top_patterns[:5]:
@@ -180,22 +192,22 @@ def main():
 
     # Handle special commands
     if len(sys.argv) == 2:
-        if sys.argv[1] in ('-h', '--help', 'help'):
+        if sys.argv[1] in ("-h", "--help", "help"):
             show_help()
             sys.exit(0)
-        elif sys.argv[1] == 'config':
+        elif sys.argv[1] == "config":
             show_config()
             sys.exit(0)
-        elif sys.argv[1] == 'history':
+        elif sys.argv[1] == "history":
             print("History feature not yet implemented for non-interactive mode.")
             sys.exit(0)
-        elif sys.argv[1] == 'plugins':
+        elif sys.argv[1] == "plugins":
             show_plugins()
             sys.exit(0)
-        elif sys.argv[1] == 'ml':
+        elif sys.argv[1] == "ml":
             show_ml_insights()
             sys.exit(0)
-        elif sys.argv[1] == 'api':
+        elif sys.argv[1] == "api":
             run_api_server()
             return
 
@@ -204,7 +216,7 @@ def main():
         sys.exit(1)
 
     mode = sys.argv[1].lower()
-    if mode not in ['lnx2ps', 'ps2lnx']:
+    if mode not in ["lnx2ps", "ps2lnx"]:
         print("Unknown mode:", mode)
         show_help()
         sys.exit(1)
@@ -224,6 +236,7 @@ def main():
         translated = ps2lnx(sanitized)
         print_translation(command, translated, mode)
 
+
 # Allow both python -m and entry point execution
 if __name__ == "__main__" or __name__ == "shellrosetta.cli":
     main()

shellrosetta/config.py

@@ -2,6 +2,8 @@
 from pathlib import Path
 import os
 import json
+
+
 class Config:
     """Configuration management for ShellRosetta"""
 
@@ -14,15 +16,15 @@ def __init__(self):
             "preferred_shell": "auto",  # auto, powershell, bash
             "color_output": True,
             "max_history": 100,
-            "auto_complete": True
+            "auto_complete": True,
         }
         self.config = self.load_config()
 
     def load_config(self):
         """Load configuration from file or create default"""
         if self.config_file.exists():
             try:
-                with open(self.config_file, 'r') as f:
+                with open(self.config_file, "r") as f:
                     config = json.load(f)
                     # Merge with defaults for any missing keys
                     for key, value in self.default_config.items():
@@ -42,7 +44,7 @@ def save_config(self, config=None):
         if config is None:
             config = self.config
         try:
-            with open(self.config_file, 'w') as f:
+            with open(self.config_file, "w") as f:
                 json.dump(config, f, indent=2)
         except IOError:
             pass  # Silently fail if we can't write config
@@ -61,5 +63,6 @@ def reset(self):
         self.config = self.default_config.copy()
         self.save_config()
 
+
 # Global config instance
 config = Config()