thailint 0.4.1__tar.gz → 0.4.2__tar.gz

{thailint-0.4.1 → thailint-0.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: thailint
-Version: 0.4.1
+Version: 0.4.2
 Summary: The AI Linter - Enterprise-grade linting and governance for AI-generated code across multiple languages
 License: MIT
 Keywords: linter,ai,code-quality,static-analysis,file-placement,governance,multi-language,cli,docker,python
@@ -35,7 +35,7 @@ Description-Content-Type: text/markdown
 
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Python 3.11+](https://img.shields.io/badge/python-3.11+-blue.svg)](https://www.python.org/downloads/)
-[![Tests](https://img.shields.io/badge/tests-263%2F263%20passing-brightgreen.svg)](tests/)
+[![Tests](https://img.shields.io/badge/tests-267%2F267%20passing-brightgreen.svg)](tests/)
 [![Coverage](https://img.shields.io/badge/coverage-87%25-brightgreen.svg)](htmlcov/)
 
 The AI Linter - Enterprise-ready linting and governance for AI-generated code across multiple languages.

{thailint-0.4.1 → thailint-0.4.2}/README.md

@@ -2,7 +2,7 @@
 
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Python 3.11+](https://img.shields.io/badge/python-3.11+-blue.svg)](https://www.python.org/downloads/)
-[![Tests](https://img.shields.io/badge/tests-263%2F263%20passing-brightgreen.svg)](tests/)
+[![Tests](https://img.shields.io/badge/tests-267%2F267%20passing-brightgreen.svg)](tests/)
 [![Coverage](https://img.shields.io/badge/coverage-87%25-brightgreen.svg)](htmlcov/)
 
 The AI Linter - Enterprise-ready linting and governance for AI-generated code across multiple languages.

{thailint-0.4.1 → thailint-0.4.2}/pyproject.toml

@@ -17,7 +17,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "thailint"
-version = "0.4.1"
+version = "0.4.2"
 description = "The AI Linter - Enterprise-grade linting and governance for AI-generated code across multiple languages"
 authors = ["Steve Jackson"]
 license = "MIT"

{thailint-0.4.1 → thailint-0.4.2}/src/cli.py

@@ -1216,10 +1216,13 @@ def _setup_magic_numbers_orchestrator(
     path_objs: list[Path], config_file: str | None, verbose: bool
 ):
     """Set up orchestrator for magic-numbers command."""
-    first_path = path_objs[0] if path_objs else Path.cwd()
-    project_root = first_path if first_path.is_dir() else first_path.parent
-
     from src.orchestrator.core import Orchestrator
+    from src.utils.project_root import get_project_root
+
+    # Find actual project root (where .git or .thailint.yaml exists)
+    first_path = path_objs[0] if path_objs else Path.cwd()
+    search_start = first_path if first_path.is_dir() else first_path.parent
+    project_root = get_project_root(search_start)
 
     orchestrator = Orchestrator(project_root=project_root)
 

{thailint-0.4.1 → thailint-0.4.2}/src/core/config_parser.py

@@ -72,18 +72,41 @@ def parse_json(file_obj: TextIO, path: Path) -> dict[str, Any]:
         raise ConfigParseError(f"Invalid JSON in {path}: {e}") from e
 
 
+def _normalize_config_keys(config: dict[str, Any]) -> dict[str, Any]:
+    """Normalize configuration keys from hyphens to underscores.
+
+    Converts top-level keys like "magic-numbers" to "magic_numbers" to match
+    internal linter expectations while maintaining backward compatibility with
+    both formats in config files.
+
+    Args:
+        config: Configuration dictionary with potentially hyphenated keys
+
+    Returns:
+        Configuration dictionary with normalized (underscored) keys
+    """
+    normalized = {}
+    for key, value in config.items():
+        # Replace hyphens with underscores in keys
+        normalized_key = key.replace("-", "_")
+        normalized[normalized_key] = value
+    return normalized
+
+
 def parse_config_file(path: Path, encoding: str = "utf-8") -> dict[str, Any]:
     """Parse configuration file based on extension.
 
     Supports .yaml, .yml, and .json formats. Automatically detects format
-    from file extension and uses appropriate parser.
+    from file extension and uses appropriate parser. Normalizes hyphenated
+    keys (e.g., "magic-numbers") to underscored keys (e.g., "magic_numbers")
+    for internal consistency.
 
     Args:
         path: Path to configuration file.
         encoding: File encoding (default: utf-8).
 
     Returns:
-        Parsed configuration dictionary.
+        Parsed configuration dictionary with normalized keys.
 
     Raises:
         ConfigParseError: If file format is unsupported or parsing fails.
@@ -95,5 +118,9 @@ def parse_config_file(path: Path, encoding: str = "utf-8") -> dict[str, Any]:
 
     with path.open(encoding=encoding) as f:
         if suffix in [".yaml", ".yml"]:
-            return parse_yaml(f, path)
-        return parse_json(f, path)
+            config = parse_yaml(f, path)
+        else:
+            config = parse_json(f, path)
+
+    # Normalize keys from hyphens to underscores
+    return _normalize_config_keys(config)

{thailint-0.4.1 → thailint-0.4.2}/src/linters/file_placement/linter.py

@@ -75,9 +75,12 @@ class FilePlacementLinter:
         # Load and validate config
         if config_obj:
             # Handle both wrapped and unwrapped config formats
-            # Wrapped: {"file-placement": {...}}
+            # Wrapped: {"file-placement": {...}} or {"file_placement": {...}}
             # Unwrapped: {"directories": {...}, "global_deny": [...], ...}
-            self.config = config_obj.get("file-placement", config_obj)
+            # Try both hyphenated and underscored keys for backward compatibility
+            self.config = config_obj.get(
+                "file-placement", config_obj.get("file_placement", config_obj)
+            )
         elif config_file:
             self.config = self._components.config_loader.load_config_file(config_file)
         else:
@@ -279,7 +282,9 @@ class FilePlacementRule(BaseLintRule):  # thailint: ignore[srp.violation]
 
     @staticmethod
     def _get_wrapped_config(context: BaseLintContext) -> dict[str, Any] | None:
-        """Get config from wrapped format: {"file-placement": {...}}.
+        """Get config from wrapped format: {"file-placement": {...}} or {"file_placement": {...}}.
+
+        Supports both hyphenated and underscored keys for backward compatibility.
 
         Args:
             context: Lint context with metadata
@@ -289,8 +294,12 @@ class FilePlacementRule(BaseLintRule):  # thailint: ignore[srp.violation]
         """
         if not hasattr(context, "metadata"):
             return None
+        # Try hyphenated format first (original format)
         if "file-placement" in context.metadata:
             return context.metadata["file-placement"]
+        # Try underscored format (normalized format)
+        if "file_placement" in context.metadata:
+            return context.metadata["file_placement"]
         return None
 
     @staticmethod
@@ -378,9 +387,11 @@ class FilePlacementRule(BaseLintRule):  # thailint: ignore[srp.violation]
         try:
             config = self._parse_layout_file(layout_path)
 
-            # Unwrap file-placement key if present
+            # Unwrap file-placement key if present (try both formats for backward compatibility)
             if "file-placement" in config:
                 return config["file-placement"]
+            if "file_placement" in config:
+                return config["file_placement"]
 
             return config
         except Exception:

{thailint-0.4.1 → thailint-0.4.2}/src/linters/magic_numbers/linter.py

@@ -97,7 +97,22 @@ class MagicNumberRule(MultiLanguageLintRule):  # thailint: ignore[srp]
         """Try to load production configuration."""
         if not hasattr(context, "metadata") or not isinstance(context.metadata, dict):
             return None
-        return load_linter_config(context, "magic_numbers", MagicNumberConfig)
+
+        # Try both hyphenated and underscored keys for backward compatibility
+        # The config parser normalizes keys when loading from YAML, but
+        # direct metadata injection (tests) may use either format
+        metadata = context.metadata
+
+        # Try underscore version first (normalized format)
+        if "magic_numbers" in metadata:
+            return load_linter_config(context, "magic_numbers", MagicNumberConfig)
+
+        # Fallback to hyphenated version (for direct test injection)
+        if "magic-numbers" in metadata:
+            return load_linter_config(context, "magic-numbers", MagicNumberConfig)
+
+        # No config found, return None to use defaults
+        return None
 
     def _is_file_ignored(self, context: BaseLintContext, config: MagicNumberConfig) -> bool:
         """Check if file matches ignore patterns.

{thailint-0.4.1 → thailint-0.4.2}/src/utils/project_root.py

@@ -52,14 +52,30 @@ def is_project_root(path: Path) -> bool:
         return False
 
 
+def _try_find_with_criterion(criterion: object, start_path: Path) -> Path | None:
+    """Try to find project root with a specific criterion.
+
+    Args:
+        criterion: pyprojroot criterion function (e.g., has_dir(".git"))
+        start_path: Path to start searching from
+
+    Returns:
+        Found project root or None if not found
+    """
+    try:
+        return find_root(criterion, start=start_path)  # type: ignore[arg-type]
+    except (OSError, RuntimeError):
+        return None
+
+
 def get_project_root(start_path: Path | None = None) -> Path:
     """Find project root by walking up the directory tree.
 
     This is the single source of truth for project root detection.
     All code that needs to find the project root should use this function.
 
-    Uses pyprojroot which searches for standard project markers defined by the
-    pyprojroot library (git repos, Python projects, etc).
+    Uses pyprojroot which searches for standard project markers (.git directory,
+    pyproject.toml, .thailint.yaml, etc) starting from start_path and walking upward.
 
     Args:
         start_path: Directory to start searching from. If None, uses current working directory.
@@ -71,14 +87,19 @@ def get_project_root(start_path: Path | None = None) -> Path:
         >>> root = get_project_root()
         >>> config_file = root / ".thailint.yaml"
     """
+    from pyprojroot import has_dir, has_file
+
     if start_path is None:
         start_path = Path.cwd()
 
     current = start_path.resolve()
 
-    try:
-        # Use pyprojroot to find the project root
-        return find_root(current)
-    except (OSError, RuntimeError):
-        # No project markers found, return the start path
-        return current
+    # Search for project root markers in priority order
+    # Try .git first (most reliable), then .thailint.yaml, then pyproject.toml
+    for criterion in [has_dir(".git"), has_file(".thailint.yaml"), has_file("pyproject.toml")]:
+        root = _try_find_with_criterion(criterion, current)
+        if root is not None:
+            return root
+
+    # No markers found, return start path
+    return current