thailint 0.4.0__py3-none-any.whl → 0.4.2__py3-none-any.whl

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)
 

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)

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:

src/linters/magic_numbers/config.py

@@ -33,6 +33,7 @@ class MagicNumberConfig:
         default_factory=lambda: {-1, 0, 1, 2, 3, 4, 5, 10, 100, 1000}
     )
     max_small_integer: int = 10
+    ignore: list[str] = field(default_factory=list)
 
     def __post_init__(self) -> None:
         """Validate configuration values."""
@@ -69,8 +70,13 @@ class MagicNumberConfig:
             )
             max_small_integer = config.get("max_small_integer", 10)
 
+        ignore_patterns = config.get("ignore", [])
+        if not isinstance(ignore_patterns, list):
+            ignore_patterns = []
+
         return cls(
             enabled=config.get("enabled", True),
             allowed_numbers=allowed_numbers,
             max_small_integer=max_small_integer,
+            ignore=ignore_patterns,
         )

src/linters/magic_numbers/linter.py

@@ -24,6 +24,7 @@ Implementation: Composition pattern with helper classes, AST-based analysis with
 """
 
 import ast
+from pathlib import Path
 
 from src.core.base import BaseLintContext, MultiLanguageLintRule
 from src.core.linter_utils import load_linter_config
@@ -96,7 +97,64 @@ 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.
+
+        Args:
+            context: Lint context
+            config: Magic numbers configuration
+
+        Returns:
+            True if file should be ignored
+        """
+        if not config.ignore:
+            return False
+
+        if not context.file_path:
+            return False
+
+        file_path = Path(context.file_path)
+        for pattern in config.ignore:
+            if self._matches_pattern(file_path, pattern):
+                return True
+        return False
+
+    def _matches_pattern(self, file_path: Path, pattern: str) -> bool:
+        """Check if file path matches a glob pattern.
+
+        Args:
+            file_path: Path to check
+            pattern: Glob pattern (e.g., "test/**", "**/test_*.py", "specific/file.py")
+
+        Returns:
+            True if path matches pattern
+        """
+        # Try glob pattern matching first (handles **, *, etc.)
+        if file_path.match(pattern):
+            return True
+
+        # Also check if pattern is a substring (for partial path matching)
+        if pattern in str(file_path):
+            return True
+
+        return False
 
     def _check_python(self, context: BaseLintContext, config: MagicNumberConfig) -> list[Violation]:
         """Check Python code for magic number violations.
@@ -108,6 +166,9 @@ class MagicNumberRule(MultiLanguageLintRule):  # thailint: ignore[srp]
         Returns:
             List of violations found in Python code
         """
+        if self._is_file_ignored(context, config):
+            return []
+
         tree = self._parse_python_code(context.file_content)
         if tree is None:
             return []
@@ -267,6 +328,9 @@ class MagicNumberRule(MultiLanguageLintRule):  # thailint: ignore[srp]
         Returns:
             List of violations found in TypeScript/JavaScript code
         """
+        if self._is_file_ignored(context, config):
+            return []
+
         analyzer = TypeScriptMagicNumberAnalyzer()
         root_node = analyzer.parse_typescript(context.file_content or "")
         if root_node is None:

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

{thailint-0.4.0.dist-info → thailint-0.4.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: thailint
-Version: 0.4.0
+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,11 +35,20 @@ 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-253%2F253%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.
 
+## Documentation
+
+**New to thailint?** Start here:
+- **[Quick Start Guide](docs/quick-start.md)** - Get running in 5 minutes
+- **[Configuration Reference](docs/configuration.md)** - Complete config options for all linters
+- **[Troubleshooting Guide](docs/troubleshooting.md)** - Common issues and solutions
+
+**Full Documentation:** Browse the **[docs/](docs/)** folder for comprehensive guides covering installation, all linters, configuration patterns, and integration examples.
+
 ## Overview
 
 thailint is a modern, enterprise-ready multi-language linter designed specifically for AI-generated code. It focuses on common mistakes and anti-patterns that AI coding assistants frequently introduce—issues that existing linters don't catch or don't handle consistently across languages.
@@ -55,6 +64,8 @@ We're not trying to replace the wonderful existing linters like Pylint, ESLint,
 
 thailint complements your existing linting stack by catching the patterns AI tools repeatedly miss.
 
+**Complete documentation available in the [docs/](docs/) folder** covering installation, configuration, all linters, and troubleshooting.
+
 ## Features
 
 ### Core Capabilities
@@ -151,6 +162,8 @@ thailint dry --config .thailint.yaml src/
 thailint dry --format json src/
 ```
 
+**New to thailint?** See the **[Quick Start Guide](docs/quick-start.md)** for a complete walkthrough including config generation, understanding output, and next steps.
+
 ### Library Mode
 
 ```python

{thailint-0.4.0.dist-info → thailint-0.4.2.dist-info}/RECORD

@@ -2,12 +2,12 @@ src/__init__.py,sha256=f601zncODr2twrUHqTLS5wyOdZqZi9tMjAe2INhRKqU,2175
 src/analyzers/__init__.py,sha256=fFloZtjkBGwYbAhKTxS3Qy3yDr2_3i3WSfKTw1mAioo,972
 src/analyzers/typescript_base.py,sha256=4I7fAcMOAY9vY1AXh52QpohgFmguBECwOkvBRP4zCS4,5054
 src/api.py,sha256=pJ5l3qxccKBEY-BkANwzTgLAl1ZFq7OP6hx6LSxbhDw,4664
-src/cli.py,sha256=zRl6FHyr24puWli77ogFtuPFWKbTNMI93kmu1V4kPEQ,40369
+src/cli.py,sha256=54g24wpF1tzyJW6gflZoIDYfeewjP123no3XliJdVsQ,40544
 src/config.py,sha256=2ebAjIpAhw4bHbOxViEA5nCjfBlDEIrMR59DBrzcYzM,12460
 src/core/__init__.py,sha256=5FtsDvhMt4SNRx3pbcGURrxn135XRbeRrjSUxiXwkNc,381
 src/core/base.py,sha256=Eklcagi2ktfY4Kytl_ObXov2U49N9OGDpw4cu4PUzGY,7824
 src/core/cli_utils.py,sha256=rYOJz4mnr8RLP-nJdHOy-GJyxGNqkWtK3_rvKriHXj4,6083
-src/core/config_parser.py,sha256=0PfLNSC0awV21SrpqdcJ2L8Cn2-B2YcwbA8znaC5YnE,3195
+src/core/config_parser.py,sha256=zAY4bDptNlVID0a4JDXN0YlUKXLM92cFqTAwhp_8uGc,4183
 src/core/linter_utils.py,sha256=4jmC2YfpPvGhS_XHlHXa5SBIJh9CQlNj5zuW_GpdPKc,5273
 src/core/registry.py,sha256=IMkic1ukmo8HCX1TY5YoKjwKT_IT-ZmVx6sdntC5X2M,3289
 src/core/rule_discovery.py,sha256=smxJ9PEyMqEAIicsWaHOaSHD1PHUAOeFZT_a3DNRwgE,4163
@@ -40,16 +40,16 @@ src/linters/dry/violation_generator.py,sha256=cc6aKvTxtHSZm0F7Y-gL1bmD3JUphRmAvc
 src/linters/file_placement/__init__.py,sha256=vJ43GZujcbAk-K3DwfsQZ0J3yP_5G35CKssatLyntXk,862
 src/linters/file_placement/config_loader.py,sha256=Of5sTG2S-04efn3KOlXrSxpMcC1ipBpSvCjtJOMmWno,2640
 src/linters/file_placement/directory_matcher.py,sha256=YaBeLGiT4bgqN_v4FmEmSASOBxkMC1lyEYpL17wLIDY,2607
-src/linters/file_placement/linter.py,sha256=r3wnfGkss6HsPCqxQjBZ1dvyZdNdxppbSu9VRh0PMzw,13802
+src/linters/file_placement/linter.py,sha256=HtQf08UmKCe5YAHoPvVgWmxdUFu6ZeP2qicg-q7BgzM,14438
 src/linters/file_placement/path_resolver.py,sha256=S6g7xOYsoSc0O_RDJh8j4Z2klcwzp16rSUfEAErGOTI,1972
 src/linters/file_placement/pattern_matcher.py,sha256=3HZWYgQKXz_y13z3lO1YHn51khCaiGOrneGxKXGWGw0,1898
 src/linters/file_placement/pattern_validator.py,sha256=eMt5GB5lgJMhhQACOlfDXQFfSfNrOY-wJN1JanGka6Q,3717
 src/linters/file_placement/rule_checker.py,sha256=JONXcaYxZ8CM_7Zg6Th01p5cve1rJ8YkReAUJ44nfUg,7795
 src/linters/file_placement/violation_factory.py,sha256=NkQmBcgpa3g3W2ZdFZNQ5djLVP4x9OKs65d7F1rCKvM,6040
 src/linters/magic_numbers/__init__.py,sha256=17dkCUf0uiYLvpOZF01VDojj92NzxXZMtRhrSBUzsdc,1689
-src/linters/magic_numbers/config.py,sha256=yL4E440FaT1RDsiQgIRcEs-MozMylOubHrQ3rT2usOM,3047
+src/linters/magic_numbers/config.py,sha256=3zV6ZNezouBWUYy4kMw5PUlPNvIWXVwOxTz1moZfRoI,3270
 src/linters/magic_numbers/context_analyzer.py,sha256=cGXozlKll10Zao56c2E8ThIyH2mSQaPaUau_g7ngRLw,8446
-src/linters/magic_numbers/linter.py,sha256=f240nyxsSN9-ttF6c7gA0LNwo6mBd85bWVO2I45G3DI,16354
+src/linters/magic_numbers/linter.py,sha256=maj4NgrDapv0RurKvaVgOI1BUujixZv4E7UeYy4eGT4,18394
 src/linters/magic_numbers/python_analyzer.py,sha256=0u1cFaaFCqOW5yeW-YbmPoZuVIeN_KtmkFyyxup6aR0,2803
 src/linters/magic_numbers/typescript_analyzer.py,sha256=DCYRdxjgMd6PkhJWKnc1W-S1T0sa-F9AHCLV2JwcR8g,7468
 src/linters/magic_numbers/violation_builder.py,sha256=SqIQv3N9lpP2GRC1TC5InrvaEdrAq24V7Ec2Xj5olb0,3308
@@ -75,9 +75,9 @@ src/orchestrator/core.py,sha256=zb4H4HtDNLmnsRCUXI3oNtfM3T-nTPW9Q2pAbI61VEs,8374
 src/orchestrator/language_detector.py,sha256=rHyVMApit80NTTNyDH1ObD1usKD8LjGmH3DwqNAWYGc,2736
 src/templates/thailint_config_template.yaml,sha256=u8WFv2coE4uqfgf_slw7xjo4kGYIowDm1RIgxsKQzrE,4275
 src/utils/__init__.py,sha256=NiBtKeQ09Y3kuUzeN4O1JNfUIYPQDS2AP1l5ODq-Dec,125
-src/utils/project_root.py,sha256=ldv2-XeMT0IElpSgrHdTaP2CUfwmdZix8vQ2qXO1O5s,2735
-thailint-0.4.0.dist-info/LICENSE,sha256=kxh1J0Sb62XvhNJ6MZsVNe8PqNVJ7LHRn_EWa-T3djw,1070
-thailint-0.4.0.dist-info/METADATA,sha256=c_ZXWle6WqMTEnSzLWodRM38CcS1O4ZnzRbJD0u-IeQ,33249
-thailint-0.4.0.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-thailint-0.4.0.dist-info/entry_points.txt,sha256=l7DQJgU18sVLDpSaXOXY3lLhnQHQIRrSJZTQjG1cEAk,62
-thailint-0.4.0.dist-info/RECORD,,
+src/utils/project_root.py,sha256=kGxQU1nHItffiR6iIHT_xCy9RQJtl1Y6dgSUnJHRJQ8,3498
+thailint-0.4.2.dist-info/LICENSE,sha256=kxh1J0Sb62XvhNJ6MZsVNe8PqNVJ7LHRn_EWa-T3djw,1070
+thailint-0.4.2.dist-info/METADATA,sha256=HT1yu78t_cI-2Mv3kw0Kci1o6iBAEdz3jpwJwS6dOkg,34038
+thailint-0.4.2.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+thailint-0.4.2.dist-info/entry_points.txt,sha256=l7DQJgU18sVLDpSaXOXY3lLhnQHQIRrSJZTQjG1cEAk,62
+thailint-0.4.2.dist-info/RECORD,,