thailint 0.4.6__tar.gz → 0.5.0__tar.gz

{thailint-0.4.6 → thailint-0.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: thailint
-Version: 0.4.6
+Version: 0.5.0
 Summary: The AI Linter - Enterprise-grade linting and governance for AI-generated code across multiple languages
 License: MIT
 License-File: LICENSE
@@ -37,7 +37,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-356%2F356%20passing-brightgreen.svg)](tests/)
+[![Tests](https://img.shields.io/badge/tests-393%2F393%20passing-brightgreen.svg)](tests/)
 [![Coverage](https://img.shields.io/badge/coverage-87%25-brightgreen.svg)](htmlcov/)
 [![Documentation Status](https://readthedocs.org/projects/thai-lint/badge/?version=latest)](https://thai-lint.readthedocs.io/en/latest/?badge=latest)
 
@@ -122,7 +122,7 @@ cd thai-lint
 pip install -e ".[dev]"
 ```
 
-### From PyPI (once published)
+### From PyPI
 
 ```bash
 pip install thai-lint

{thailint-0.4.6 → thailint-0.5.0}/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-356%2F356%20passing-brightgreen.svg)](tests/)
+[![Tests](https://img.shields.io/badge/tests-393%2F393%20passing-brightgreen.svg)](tests/)
 [![Coverage](https://img.shields.io/badge/coverage-87%25-brightgreen.svg)](htmlcov/)
 [![Documentation Status](https://readthedocs.org/projects/thai-lint/badge/?version=latest)](https://thai-lint.readthedocs.io/en/latest/?badge=latest)
 
@@ -87,7 +87,7 @@ cd thai-lint
 pip install -e ".[dev]"
 ```
 
-### From PyPI (once published)
+### From PyPI
 
 ```bash
 pip install thai-lint

{thailint-0.4.6 → thailint-0.5.0}/pyproject.toml

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

{thailint-0.4.6 → thailint-0.5.0}/src/cli.py

@@ -1551,5 +1551,115 @@ def _execute_magic_numbers_lint(  # pylint: disable=too-many-arguments,too-many-
     sys.exit(1 if magic_numbers_violations else 0)
 
 
+# =============================================================================
+# Print Statements Linter Command
+# =============================================================================
+
+
+def _setup_print_statements_orchestrator(
+    path_objs: list[Path], config_file: str | None, verbose: bool, project_root: Path | None = None
+):
+    """Set up orchestrator for print-statements command."""
+    from src.orchestrator.core import Orchestrator
+    from src.utils.project_root import get_project_root
+
+    if project_root is None:
+        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)
+
+    if config_file:
+        _load_config_file(orchestrator, config_file, verbose)
+
+    return orchestrator
+
+
+def _run_print_statements_lint(orchestrator, path_objs: list[Path], recursive: bool):
+    """Execute print-statements lint on files or directories."""
+    all_violations = _execute_linting_on_paths(orchestrator, path_objs, recursive)
+    return [v for v in all_violations if "print-statement" in v.rule_id]
+
+
+@cli.command("print-statements")
+@click.argument("paths", nargs=-1, type=click.Path())
+@click.option("--config", "-c", "config_file", type=click.Path(), help="Path to config file")
+@format_option
+@click.option("--recursive/--no-recursive", default=True, help="Scan directories recursively")
+@click.pass_context
+def print_statements(  # pylint: disable=too-many-arguments,too-many-positional-arguments
+    ctx,
+    paths: tuple[str, ...],
+    config_file: str | None,
+    format: str,
+    recursive: bool,
+):
+    """Check for print/console statements in code.
+
+    Detects print() calls in Python and console.log/warn/error/debug/info calls
+    in TypeScript/JavaScript that should be replaced with proper logging.
+
+    PATHS: Files or directories to lint (defaults to current directory if none provided)
+
+    Examples:
+
+        \b
+        # Check current directory (all files recursively)
+        thai-lint print-statements
+
+        \b
+        # Check specific directory
+        thai-lint print-statements src/
+
+        \b
+        # Check single file
+        thai-lint print-statements src/app.py
+
+        \b
+        # Check multiple files
+        thai-lint print-statements src/app.py src/utils.ts tests/test_app.py
+
+        \b
+        # Get JSON output
+        thai-lint print-statements --format json .
+
+        \b
+        # Use custom config file
+        thai-lint print-statements --config .thailint.yaml src/
+    """
+    verbose = ctx.obj.get("verbose", False)
+    project_root = _get_project_root_from_context(ctx)
+
+    if not paths:
+        paths = (".",)
+
+    path_objs = [Path(p) for p in paths]
+
+    try:
+        _execute_print_statements_lint(
+            path_objs, config_file, format, recursive, verbose, project_root
+        )
+    except Exception as e:
+        _handle_linting_error(e, verbose)
+
+
+def _execute_print_statements_lint(  # pylint: disable=too-many-arguments,too-many-positional-arguments
+    path_objs, config_file, format, recursive, verbose, project_root=None
+):
+    """Execute print-statements lint."""
+    _validate_paths_exist(path_objs)
+    orchestrator = _setup_print_statements_orchestrator(
+        path_objs, config_file, verbose, project_root
+    )
+    print_statements_violations = _run_print_statements_lint(orchestrator, path_objs, recursive)
+
+    if verbose:
+        logger.info(f"Found {len(print_statements_violations)} print statement violation(s)")
+
+    format_violations(print_statements_violations, format)
+    sys.exit(1 if print_statements_violations else 0)
+
+
 if __name__ == "__main__":
     cli()

{thailint-0.4.6 → thailint-0.5.0}/src/linters/magic_numbers/typescript_analyzer.py

@@ -26,6 +26,7 @@ from typing import Any
 
 from src.analyzers.typescript_base import TypeScriptBaseAnalyzer
 
+# dry: ignore-block - tree-sitter import pattern (common across TypeScript analyzers)
 try:
     from tree_sitter import Node
 

{thailint-0.4.6 → thailint-0.5.0}/src/linters/nesting/typescript_analyzer.py

@@ -22,6 +22,7 @@ from typing import Any
 
 from src.analyzers.typescript_base import TypeScriptBaseAnalyzer
 
+# dry: ignore-block - tree-sitter import pattern (common across TypeScript analyzers)
 try:
     from tree_sitter import Node
 

thailint-0.5.0/src/linters/print_statements/__init__.py

@@ -0,0 +1,53 @@
+"""
+File: src/linters/print_statements/__init__.py
+
+Purpose: Print statements linter package exports and convenience functions
+
+Exports: PrintStatementRule class, PrintStatementConfig dataclass, lint() convenience function
+
+Depends: .linter for PrintStatementRule, .config for PrintStatementConfig
+
+Implements: lint(file_path, config) -> list[Violation] for simple linting operations
+
+Related: src/linters/magic_numbers/__init__.py, src/core/base.py
+
+Overview: Provides the public interface for the print statements linter package. Exports main
+    PrintStatementRule class for use by the orchestrator and PrintStatementConfig for configuration.
+    Includes lint() convenience function that provides a simple API for running the print statements
+    linter on a file without directly interacting with the orchestrator. This module serves as the
+    entry point for users of the print statements linter, hiding implementation details and exposing
+    only the essential components needed for linting operations.
+
+Usage: from src.linters.print_statements import PrintStatementRule, lint
+    violations = lint("path/to/file.py")
+
+Notes: Module-level exports with __all__ definition, convenience function wrapper
+"""
+
+from .config import PrintStatementConfig
+from .linter import PrintStatementRule
+
+__all__ = ["PrintStatementRule", "PrintStatementConfig", "lint"]
+
+
+def lint(file_path: str, config: dict | None = None) -> list:
+    """Convenience function for linting a file for print statements.
+
+    Args:
+        file_path: Path to the file to lint
+        config: Optional configuration dictionary
+
+    Returns:
+        List of violations found
+    """
+    from pathlib import Path
+
+    from src.orchestrator.core import FileLintContext
+
+    rule = PrintStatementRule()
+    context = FileLintContext(
+        path=Path(file_path),
+        lang="python",
+    )
+
+    return rule.check(context)

thailint-0.5.0/src/linters/print_statements/config.py

@@ -0,0 +1,83 @@
+"""
+File: src/linters/print_statements/config.py
+
+Purpose: Configuration schema for print statements linter
+
+Exports: PrintStatementConfig dataclass
+
+Depends: dataclasses, typing
+
+Implements: PrintStatementConfig(enabled, ignore, allow_in_scripts, console_methods),
+    from_dict class method for loading configuration from dictionary
+
+Related: src/linters/magic_numbers/config.py, src/core/types.py
+
+Overview: Defines configuration schema for print statements linter. Provides PrintStatementConfig
+    dataclass with enabled flag, ignore patterns list, allow_in_scripts setting (default True to
+    allow print in __main__ blocks), and console_methods set (default includes log, warn, error,
+    debug, info) for TypeScript/JavaScript console method detection. Supports per-file and
+    per-directory config overrides through from_dict class method. Integrates with orchestrator's
+    configuration system to allow users to customize detection via .thailint.yaml configuration.
+
+Usage: config = PrintStatementConfig.from_dict(yaml_config, language="python")
+
+Notes: Dataclass with defaults matching common use cases, language-specific override support
+"""
+
+from dataclasses import dataclass, field
+from typing import Any
+
+
+@dataclass
+class PrintStatementConfig:
+    """Configuration for print statements linter."""
+
+    enabled: bool = True
+    ignore: list[str] = field(default_factory=list)
+    allow_in_scripts: bool = True
+    console_methods: set[str] = field(
+        default_factory=lambda: {"log", "warn", "error", "debug", "info"}
+    )
+
+    @classmethod
+    def from_dict(
+        cls, config: dict[str, Any], language: str | None = None
+    ) -> "PrintStatementConfig":
+        """Load configuration from dictionary with language-specific overrides.
+
+        Args:
+            config: Dictionary containing configuration values
+            language: Programming language (python, typescript, javascript)
+                for language-specific settings
+
+        Returns:
+            PrintStatementConfig instance with values from dictionary
+        """
+        # Get language-specific config if available
+        if language and language in config:
+            lang_config = config[language]
+            allow_in_scripts = lang_config.get(
+                "allow_in_scripts", config.get("allow_in_scripts", True)
+            )
+            console_methods = set(
+                lang_config.get(
+                    "console_methods",
+                    config.get("console_methods", ["log", "warn", "error", "debug", "info"]),
+                )
+            )
+        else:
+            allow_in_scripts = config.get("allow_in_scripts", True)
+            console_methods = set(
+                config.get("console_methods", ["log", "warn", "error", "debug", "info"])
+            )
+
+        ignore_patterns = config.get("ignore", [])
+        if not isinstance(ignore_patterns, list):
+            ignore_patterns = []
+
+        return cls(
+            enabled=config.get("enabled", True),
+            ignore=ignore_patterns,
+            allow_in_scripts=allow_in_scripts,
+            console_methods=console_methods,
+        )