thailint 0.11.0__tar.gz → 0.12.0__tar.gz

{thailint-0.11.0 → thailint-0.12.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: thailint
-Version: 0.11.0
+Version: 0.12.0
 Summary: The AI Linter - Enterprise-grade linting and governance for AI-generated code across multiple languages
 License: MIT
 License-File: LICENSE
@@ -37,8 +37,8 @@ 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-884%2F884%20passing-brightgreen.svg)](tests/)
-[![Coverage](https://img.shields.io/badge/coverage-88%25-brightgreen.svg)](htmlcov/)
+[![Tests](https://img.shields.io/badge/tests-1076%2F1076%20passing-brightgreen.svg)](tests/)
+[![Coverage](https://img.shields.io/badge/coverage-89%25-brightgreen.svg)](htmlcov/)
 [![Documentation Status](https://readthedocs.org/projects/thai-lint/badge/?version=latest)](https://thai-lint.readthedocs.io/en/latest/?badge=latest)
 [![SARIF 2.1.0](https://img.shields.io/badge/SARIF-2.1.0-orange.svg)](docs/sarif-output.md)
 
@@ -116,6 +116,12 @@ thailint complements your existing linting stack by catching the patterns AI too
   - No instance state (self.attr) detection
   - Excludes ABC, Protocol, and decorated classes
   - Helpful refactoring suggestions
+- **Stringly-Typed Linting** - Detect string patterns that should use enums
+  - Python and TypeScript support
+  - Cross-file detection with SQLite storage
+  - Detects membership validation, equality chains, function call patterns
+  - False positive filtering (200+ exclusion patterns)
+  - Inline ignore directive support
 - **Pluggable Architecture** - Easy to extend with custom linters
 - **Multi-Language Support** - Python, TypeScript, JavaScript, and more
 - **Flexible Configuration** - YAML/JSON configs with pattern matching

{thailint-0.11.0 → thailint-0.12.0}/README.md

@@ -2,8 +2,8 @@
 
 [![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-884%2F884%20passing-brightgreen.svg)](tests/)
-[![Coverage](https://img.shields.io/badge/coverage-88%25-brightgreen.svg)](htmlcov/)
+[![Tests](https://img.shields.io/badge/tests-1076%2F1076%20passing-brightgreen.svg)](tests/)
+[![Coverage](https://img.shields.io/badge/coverage-89%25-brightgreen.svg)](htmlcov/)
 [![Documentation Status](https://readthedocs.org/projects/thai-lint/badge/?version=latest)](https://thai-lint.readthedocs.io/en/latest/?badge=latest)
 [![SARIF 2.1.0](https://img.shields.io/badge/SARIF-2.1.0-orange.svg)](docs/sarif-output.md)
 
@@ -81,6 +81,12 @@ thailint complements your existing linting stack by catching the patterns AI too
   - No instance state (self.attr) detection
   - Excludes ABC, Protocol, and decorated classes
   - Helpful refactoring suggestions
+- **Stringly-Typed Linting** - Detect string patterns that should use enums
+  - Python and TypeScript support
+  - Cross-file detection with SQLite storage
+  - Detects membership validation, equality chains, function call patterns
+  - False positive filtering (200+ exclusion patterns)
+  - Inline ignore directive support
 - **Pluggable Architecture** - Easy to extend with custom linters
 - **Multi-Language Support** - Python, TypeScript, JavaScript, and more
 - **Flexible Configuration** - YAML/JSON configs with pattern matching

{thailint-0.11.0 → thailint-0.12.0}/pyproject.toml

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

{thailint-0.11.0 → thailint-0.12.0}/src/cli/linters/code_smells.py

@@ -1,18 +1,18 @@
 """
-Purpose: CLI commands for code smell linters (dry, magic-numbers)
+Purpose: CLI commands for code smell linters (dry, magic-numbers, stringly-typed)
 
-Scope: Commands that detect code smells like duplicate code and magic numbers
+Scope: Commands that detect code smells like duplicate code, magic numbers, and stringly-typed patterns
 
 Overview: Provides CLI commands for code smell detection: dry finds duplicate code blocks using
-    token-based hashing with SQLite caching, and magic-numbers detects unnamed numeric literals that
-    should be extracted as named constants. Each command supports standard options (config, format,
-    recursive) plus linter-specific options (min-lines, no-cache, clear-cache) and integrates with
-    the orchestrator for execution.
+    token-based hashing with SQLite caching, magic-numbers detects unnamed numeric literals that
+    should be extracted as named constants, and stringly-typed detects string patterns that should
+    use enums. Each command supports standard options (config, format, recursive) plus linter-specific
+    options and integrates with the orchestrator for execution.
 
 Dependencies: click for CLI framework, src.cli.main for CLI group, src.cli.utils for shared utilities,
     src.cli.linters.shared for linter-specific helpers, yaml for config loading
 
-Exports: dry command, magic_numbers command
+Exports: dry command, magic_numbers command, stringly_typed command
 
 Interfaces: Click CLI commands registered to main CLI group
 
@@ -341,3 +341,110 @@ def _execute_magic_numbers_lint(  # pylint: disable=too-many-arguments,too-many-
 
     format_violations(magic_numbers_violations, format)
     sys.exit(1 if magic_numbers_violations else 0)
+
+
+# =============================================================================
+# Stringly-Typed Command
+# =============================================================================
+
+
+def _setup_stringly_typed_orchestrator(
+    path_objs: list[Path], config_file: str | None, verbose: bool, project_root: Path | None = None
+) -> "Orchestrator":
+    """Set up orchestrator for stringly-typed command."""
+    return setup_base_orchestrator(path_objs, config_file, verbose, project_root)
+
+
+def _run_stringly_typed_lint(
+    orchestrator: "Orchestrator", path_objs: list[Path], recursive: bool
+) -> list[Violation]:
+    """Execute stringly-typed lint on files or directories."""
+    all_violations = execute_linting_on_paths(orchestrator, path_objs, recursive)
+    return [v for v in all_violations if "stringly-typed" in v.rule_id]
+
+
+@cli.command("stringly-typed")
+@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 stringly_typed(  # pylint: disable=too-many-arguments,too-many-positional-arguments
+    ctx: click.Context,
+    paths: tuple[str, ...],
+    config_file: str | None,
+    format: str,
+    recursive: bool,
+) -> None:
+    """Check for stringly-typed patterns in code.
+
+    Detects string patterns in Python and TypeScript/JavaScript code that should
+    use enums or typed alternatives. Finds membership validation, equality chains,
+    and function calls with limited string values across multiple files.
+
+    PATHS: Files or directories to lint (defaults to current directory if none provided)
+
+    Examples:
+
+        \b
+        # Check current directory (all files recursively)
+        thai-lint stringly-typed
+
+        \b
+        # Check specific directory
+        thai-lint stringly-typed src/
+
+        \b
+        # Check single file
+        thai-lint stringly-typed src/handlers.py
+
+        \b
+        # Check multiple files
+        thai-lint stringly-typed src/handlers.py src/services.py
+
+        \b
+        # Get JSON output
+        thai-lint stringly-typed --format json .
+
+        \b
+        # Get SARIF output for IDE integration
+        thai-lint stringly-typed --format sarif .
+
+        \b
+        # Use custom config file
+        thai-lint stringly-typed --config .thailint.yaml src/
+    """
+    verbose: bool = 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_stringly_typed_lint(
+            path_objs, config_file, format, recursive, verbose, project_root
+        )
+    except Exception as e:
+        handle_linting_error(e, verbose)
+
+
+def _execute_stringly_typed_lint(  # pylint: disable=too-many-arguments,too-many-positional-arguments
+    path_objs: list[Path],
+    config_file: str | None,
+    format: str,
+    recursive: bool,
+    verbose: bool,
+    project_root: Path | None = None,
+) -> NoReturn:
+    """Execute stringly-typed lint."""
+    validate_paths_exist(path_objs)
+    orchestrator = _setup_stringly_typed_orchestrator(path_objs, config_file, verbose, project_root)
+    stringly_violations = _run_stringly_typed_lint(orchestrator, path_objs, recursive)
+
+    if verbose:
+        logger.info(f"Found {len(stringly_violations)} stringly-typed violation(s)")
+
+    format_violations(stringly_violations, format)
+    sys.exit(1 if stringly_violations else 0)

{thailint-0.11.0 → thailint-0.12.0}/src/cli/utils.py

@@ -172,34 +172,54 @@ def _autodetect_project_root(
     return auto_root
 
 
-def get_project_root_from_context(ctx: click.Context) -> Path:
+def get_project_root_from_context(ctx: click.Context) -> Path | None:
     """Get or determine project root from Click context.
 
     This function defers the actual determination until needed to avoid
     importing pyprojroot in test environments where it may not be available.
 
+    Returns None when no explicit root is specified (via --project-root or --config),
+    allowing the orchestrator to auto-detect from target paths instead of CWD.
+
     Args:
         ctx: Click context containing CLI options
 
     Returns:
-        Path to determined project root
+        Path to determined project root, or None for auto-detection from target paths
     """
     # Check if already determined and cached
     if "project_root" in ctx.obj:
-        cached_root: Path = ctx.obj["project_root"]
-        return cached_root
+        cached: Path | None = ctx.obj["project_root"]
+        return cached
+
+    project_root = _determine_project_root_for_context(ctx)
+    ctx.obj["project_root"] = project_root
+    return project_root
+
+
+def _determine_project_root_for_context(ctx: click.Context) -> Path | None:
+    """Determine project root from context options.
+
+    Args:
+        ctx: Click context containing CLI options
 
-    # Determine project root using stored CLI options
+    Returns:
+        Path if explicit root or config specified, None for auto-detection
+    """
     explicit_root = ctx.obj.get("cli_project_root")
     config_path = ctx.obj.get("cli_config_path")
     verbose = ctx.obj.get("verbose", False)
 
-    project_root = _determine_project_root(explicit_root, config_path, verbose)
+    if explicit_root:
+        return _resolve_explicit_project_root(explicit_root, verbose)
 
-    # Cache for future use
-    ctx.obj["project_root"] = project_root
+    if config_path:
+        return _infer_root_from_config(config_path, verbose)
 
-    return project_root
+    # No explicit root - return None for auto-detection from target paths
+    if verbose:
+        logger.debug("No explicit project root, will auto-detect from target paths")
+    return None
 
 
 # =============================================================================

thailint-0.12.0/src/linters/stringly_typed/__init__.py

@@ -0,0 +1,36 @@
+"""
+Purpose: Stringly-typed linter package exports
+
+Scope: Public API for stringly-typed linter module
+
+Overview: Provides the public interface for the stringly-typed linter package. Exports
+    StringlyTypedConfig for configuration and StringlyTypedRule for linting. The stringly-typed
+    linter detects code patterns where plain strings are used instead of proper enums or typed
+    alternatives, helping identify potential type safety improvements. Supports cross-file
+    detection to find repeated string patterns across the codebase. Includes IgnoreChecker
+    for inline ignore directive support.
+
+Dependencies: .config for StringlyTypedConfig, .linter for StringlyTypedRule,
+    .storage for StringlyTypedStorage, .ignore_checker for IgnoreChecker
+
+Exports: StringlyTypedConfig, StringlyTypedRule, StringlyTypedStorage, StoredPattern,
+    IgnoreChecker
+
+Interfaces: Configuration loading via StringlyTypedConfig.from_dict(),
+    StringlyTypedRule.check() and finalize() for linting, IgnoreChecker.filter_violations()
+
+Implementation: Module-level exports with __all__ definition
+"""
+
+from src.linters.stringly_typed.config import StringlyTypedConfig
+from src.linters.stringly_typed.ignore_checker import IgnoreChecker
+from src.linters.stringly_typed.linter import StringlyTypedRule
+from src.linters.stringly_typed.storage import StoredPattern, StringlyTypedStorage
+
+__all__ = [
+    "StringlyTypedConfig",
+    "IgnoreChecker",
+    "StringlyTypedRule",
+    "StringlyTypedStorage",
+    "StoredPattern",
+]

{thailint-0.11.0 → thailint-0.12.0}/src/linters/stringly_typed/config.py

@@ -33,6 +33,23 @@ DEFAULT_MIN_OCCURRENCES = 2
 DEFAULT_MIN_VALUES_FOR_ENUM = 2
 DEFAULT_MAX_VALUES_FOR_ENUM = 6
 
+# Default ignore patterns - test directories are excluded by default
+# because test fixtures commonly use string literals for mocking
+DEFAULT_IGNORE_PATTERNS: list[str] = [
+    "**/tests/**",
+    "**/test/**",
+    "**/*_test.py",
+    "**/*_test.ts",
+    "**/*.test.ts",
+    "**/*.test.tsx",
+    "**/*.spec.ts",
+    "**/*.spec.tsx",
+    "**/*.stories.ts",
+    "**/*.stories.tsx",
+    "**/conftest.py",
+    "**/fixtures/**",
+]
+
 
 @dataclass
 class StringlyTypedConfig:  # pylint: disable=too-many-instance-attributes
@@ -62,7 +79,7 @@ class StringlyTypedConfig:  # pylint: disable=too-many-instance-attributes
     """Whether to require cross-file occurrences to flag violations."""
 
     ignore: list[str] = field(default_factory=list)
-    """File patterns to ignore."""
+    """File patterns to ignore. Defaults merged with test directories in from_dict."""
 
     allowed_string_sets: list[list[str]] = field(default_factory=list)
     """String sets that are allowed and should not be flagged."""
@@ -114,13 +131,17 @@ class StringlyTypedConfig:  # pylint: disable=too-many-instance-attributes
         Returns:
             StringlyTypedConfig instance
         """
+        # Merge user ignore patterns with defaults
+        user_ignore = config.get("ignore", [])
+        merged_ignore = DEFAULT_IGNORE_PATTERNS.copy() + user_ignore
+
         return cls(
             enabled=config.get("enabled", True),
             min_occurrences=config.get("min_occurrences", DEFAULT_MIN_OCCURRENCES),
             min_values_for_enum=config.get("min_values_for_enum", DEFAULT_MIN_VALUES_FOR_ENUM),
             max_values_for_enum=config.get("max_values_for_enum", DEFAULT_MAX_VALUES_FOR_ENUM),
             require_cross_file=config.get("require_cross_file", True),
-            ignore=config.get("ignore", []),
+            ignore=merged_ignore,
             allowed_string_sets=config.get("allowed_string_sets", []),
             exclude_variables=config.get("exclude_variables", []),
         )
@@ -138,6 +159,10 @@ class StringlyTypedConfig:  # pylint: disable=too-many-instance-attributes
         Returns:
             StringlyTypedConfig instance with merged values
         """
+        # Merge user ignore patterns with defaults
+        user_ignore = lang_config.get("ignore", base_config.get("ignore", []))
+        merged_ignore = DEFAULT_IGNORE_PATTERNS.copy() + user_ignore
+
         return cls(
             enabled=lang_config.get("enabled", base_config.get("enabled", True)),
             min_occurrences=lang_config.get(
@@ -155,7 +180,7 @@ class StringlyTypedConfig:  # pylint: disable=too-many-instance-attributes
             require_cross_file=lang_config.get(
                 "require_cross_file", base_config.get("require_cross_file", True)
             ),
-            ignore=lang_config.get("ignore", base_config.get("ignore", [])),
+            ignore=merged_ignore,
             allowed_string_sets=lang_config.get(
                 "allowed_string_sets", base_config.get("allowed_string_sets", [])
             ),