error-translator-cli-v2 1.0.7__tar.gz → 1.0.8__tar.gz

{error_translator_cli_v2-1.0.7 → error_translator_cli_v2-1.0.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: error-translator-cli-v2
-Version: 1.0.7
+Version: 1.0.8
 Summary: A CLI tool that explains Python errors in simple human language.
 Author-email: Gourabananda Datta <gourabanandadatta@zohomail.com>
 Requires-Python: >=3.7

{error_translator_cli_v2-1.0.7 → error_translator_cli_v2-1.0.8}/error_translator/core.py

@@ -1,50 +1,62 @@
 import json
 import os
+import re
+import linecache
+from functools import lru_cache
 from .ast_handlers import AST_REGISTRY
 
+
+@lru_cache(maxsize=1)
 def load_rules():
-    """Loads the error rules from the JSON file."""
-    # Find the absolute path to the json file next to this script
+    """Load the error rules from disk once and cache them."""
     current_dir = os.path.dirname(os.path.abspath(__file__))
-    json_path = os.path.join(current_dir, 'rules.json')
-    
-    with open(json_path, 'r') as file:
+    json_path = os.path.join(current_dir, "rules.json")
+
+    with open(json_path, "r", encoding="utf-8") as file:
         return json.load(file)
 
+
+@lru_cache(maxsize=1)
+def compiled_rules():
+    """Compile regex patterns once to avoid repeated work per translation."""
+    data = load_rules()
+    compiled = []
+    for rule in data["rules"]:
+        compiled.append((re.compile(rule["pattern"]), rule))
+    return compiled
+
+
+def _extract_location(traceback_text: str) -> tuple[str, str]:
+    location_match = re.search(r'File\s+[\'"]?(.*?)[\'"]?,\s+line\s+(\d+)', traceback_text)
+    if not location_match:
+        return "Unknown File", "Unknown Line"
+    return location_match.group(1), location_match.group(2)
+
+
 def translate_error(traceback_text: str) -> dict:
-    import re
-    import linecache  # <-- NEW: Lazy import the linecache module
-    
     data = load_rules()
-    rules = data["rules"]
+    rules = compiled_rules()
     default_error = data["default"]
 
-    lines = [line.strip() for line in traceback_text.strip().split('\n') if line.strip()]
+    lines = [line.strip() for line in traceback_text.strip().split("\n") if line.strip()]
     if not lines:
         return {"explanation": "No error text provided.", "fix": "Provide a valid Python error."}
-    
+
     actual_error_line = lines[-1]
 
-    location_match = re.search(r'File\s+[\'"]?(.*?)[\'"]?,\s+line\s+(\d+)', traceback_text)
-    file_name = location_match.group(1) if location_match else "Unknown File"
-    line_number = location_match.group(2) if location_match else "Unknown Line"
+    file_name, line_number = _extract_location(traceback_text)
 
-    # --- NEW CONTEXT ENGINE LOGIC ---
     code_context = ""
     if file_name != "Unknown File" and line_number != "Unknown Line":
         try:
-            # linecache safely fetches the exact line of code as a string
             raw_line = linecache.getline(file_name, int(line_number))
             if raw_line:
-                code_context = raw_line.strip() # Remove extra spaces/newlines
+                code_context = raw_line.strip()
         except Exception:
-            pass # If the file can't be read for any reason, fail silently
-    # --------------------------------
+            pass
 
-    for rule in rules:
-        pattern = re.compile(rule["pattern"])
+    for pattern, rule in rules:
         match = pattern.search(actual_error_line)
-        
         if match:
             extracted_values = list(match.groups())
             fix_text = rule["fix"].format(*extracted_values)
@@ -61,7 +73,7 @@ def translate_error(traceback_text: str) -> dict:
                 "matched_error": actual_error_line,
                 "file": file_name,
                 "line": line_number,
-                "code": code_context 
+                "code": code_context,
             }
 
     return {
@@ -70,5 +82,5 @@ def translate_error(traceback_text: str) -> dict:
         "matched_error": actual_error_line,
         "file": file_name,
         "line": line_number,
-        "code": code_context      
-    }
+        "code": code_context,
+    }

{error_translator_cli_v2-1.0.7 → error_translator_cli_v2-1.0.8}/error_translator_cli_v2.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: error-translator-cli-v2
-Version: 1.0.7
+Version: 1.0.8
 Summary: A CLI tool that explains Python errors in simple human language.
 Author-email: Gourabananda Datta <gourabanandadatta@zohomail.com>
 Requires-Python: >=3.7

{error_translator_cli_v2-1.0.7 → error_translator_cli_v2-1.0.8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "error-translator-cli-v2"
-version = "1.0.7"
+version = "1.0.8"
 description = "A CLI tool that explains Python errors in simple human language."
 readme = "README.md"
 requires-python = ">=3.7"

{error_translator_cli_v2-1.0.7 → error_translator_cli_v2-1.0.8}/tests/test_core.py

@@ -1,5 +1,5 @@
 import pytest
-from error_translator.core import translate_error
+from error_translator.core import translate_error, load_rules, compiled_rules
 
 # --- 1. EDGE CASE TESTS ---
 
@@ -35,6 +35,24 @@ def test_unknown_error_fallback():
     assert result["matched_error"] == "Something completely random went wrong here."
 
 
+def test_empty_input_returns_helpful_message():
+    result = translate_error("   \n   ")
+    assert result["explanation"] == "No error text provided."
+    assert result["fix"] == "Provide a valid Python error."
+
+
+def test_rule_loading_is_cached():
+    first = load_rules()
+    second = load_rules()
+    assert first is second
+
+
+def test_compiled_rules_are_cached():
+    first = compiled_rules()
+    second = compiled_rules()
+    assert first is second
+
+
 # --- 2. THE PARAMETERIZED ENGINE FOR ALL ERRORS ---
 
 @pytest.mark.parametrize("mock_traceback, expected_in_explanation", [
@@ -128,4 +146,4 @@ def test_regex_extraction_for_supported_errors(mock_traceback, expected_in_expla
     
     # 2. Prove the Context Engine successfully parsed the file location
     assert result["file"] == "script.py"
-    assert result["line"] == "5"
+    assert result["line"] == "5"