thailint 0.4.4__tar.gz → 0.4.6__tar.gz

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: thailint
-Version: 0.4.4
+Version: 0.4.6
 Summary: The AI Linter - Enterprise-grade linting and governance for AI-generated code across multiple languages
 License: MIT
 License-File: LICENSE
@@ -28,7 +28,7 @@ Requires-Dist: pyprojroot (>=0.3.0,<0.4.0)
 Requires-Dist: pyyaml (>=6.0,<7.0)
 Requires-Dist: tree-sitter (>=0.25.2,<0.26.0)
 Requires-Dist: tree-sitter-typescript (>=0.23.2,<0.24.0)
-Project-URL: Documentation, https://github.com/be-wise-be-kind/thai-lint#readme
+Project-URL: Documentation, https://thai-lint.readthedocs.io/
 Project-URL: Homepage, https://github.com/be-wise-be-kind/thai-lint
 Project-URL: Repository, https://github.com/be-wise-be-kind/thai-lint
 Description-Content-Type: text/markdown
@@ -37,19 +37,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-296%2F296%20passing-brightgreen.svg)](tests/)
+[![Tests](https://img.shields.io/badge/tests-356%2F356%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)
 
 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
+- **[Quick Start Guide](https://thai-lint.readthedocs.io/en/latest/quick-start/)** - Get running in 5 minutes
+- **[Configuration Reference](https://thai-lint.readthedocs.io/en/latest/configuration/)** - Complete config options for all linters
+- **[Troubleshooting Guide](https://thai-lint.readthedocs.io/en/latest/troubleshooting/)** - Common issues and solutions
 
-**Full Documentation:** Browse the **[docs/](docs/)** folder for comprehensive guides covering installation, all linters, configuration patterns, and integration examples.
+**Full Documentation:** Browse the **[documentation site](https://thai-lint.readthedocs.io/)** for comprehensive guides covering installation, all linters, configuration patterns, and integration examples.
 
 ## Overview
 
@@ -66,7 +67,7 @@ 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.
+**Complete documentation available at [thai-lint.readthedocs.io](https://thai-lint.readthedocs.io/)** covering installation, configuration, all linters, and troubleshooting.
 
 ## Features
 
@@ -164,7 +165,7 @@ 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.
+**New to thailint?** See the **[Quick Start Guide](https://thai-lint.readthedocs.io/en/latest/quick-start/)** for a complete walkthrough including config generation, understanding output, and next steps.
 
 ### Library Mode
 
@@ -356,9 +357,9 @@ magic-numbers:
 }
 ```
 
-See [Configuration Guide](docs/configuration.md) for complete reference.
+See [Configuration Guide](https://thai-lint.readthedocs.io/en/latest/configuration/) for complete reference.
 
-**Need help with ignores?** See **[How to Ignore Violations](docs/how-to-ignore-violations.md)** for complete guide to all ignore levels (line, method, class, file, repository).
+**Need help with ignores?** See **[How to Ignore Violations](https://thai-lint.readthedocs.io/en/latest/how-to-ignore-violations/)** for complete guide to all ignore levels (line, method, class, file, repository).
 
 ## Nesting Depth Linter
 
@@ -444,7 +445,7 @@ Common patterns to reduce nesting:
 - **TypeScript**: Full support (if/for/while/try/switch)
 - **JavaScript**: Supported via TypeScript parser
 
-See [Nesting Linter Guide](docs/nesting-linter.md) for comprehensive documentation and refactoring patterns.
+See [Nesting Linter Guide](https://thai-lint.readthedocs.io/en/latest/nesting-linter/) for comprehensive documentation and refactoring patterns.
 
 ## Single Responsibility Principle (SRP) Linter
 
@@ -561,7 +562,7 @@ Common patterns to fix SRP violations (discovered during dogfooding):
 - **After**: Extract Class pattern applied - 5 focused classes (ConfigLoader, PatternValidator, RuleChecker, PathResolver, FilePlacementLinter)
 - **Result**: Each class ≤8 methods, ≤150 LOC, single responsibility
 
-See [SRP Linter Guide](docs/srp-linter.md) for comprehensive documentation and refactoring patterns.
+See [SRP Linter Guide](https://thai-lint.readthedocs.io/en/latest/srp-linter/) for comprehensive documentation and refactoring patterns.
 
 ## DRY Linter (Don't Repeat Yourself)
 
@@ -714,7 +715,7 @@ Built-in filters automatically exclude common non-duplication patterns:
 3. **Extract Utility Module**: Move helper functions to shared utilities
 4. **Template Method**: Use function parameters for variations
 
-See [DRY Linter Guide](docs/dry-linter.md) for comprehensive documentation, storage modes, and refactoring patterns.
+See [DRY Linter Guide](https://thai-lint.readthedocs.io/en/latest/dry-linter/) for comprehensive documentation, storage modes, and refactoring patterns.
 
 ## Magic Numbers Linter
 
@@ -866,7 +867,7 @@ def get_ports():  # thailint: ignore[magic-numbers] - Standard ports
 # thailint: ignore-file[magic-numbers]
 ```
 
-See **[How to Ignore Violations](docs/how-to-ignore-violations.md)** and **[Magic Numbers Linter Guide](docs/magic-numbers-linter.md)** for complete documentation.
+See **[How to Ignore Violations](https://thai-lint.readthedocs.io/en/latest/how-to-ignore-violations/)** and **[Magic Numbers Linter Guide](https://thai-lint.readthedocs.io/en/latest/magic-numbers-linter/)** for complete documentation.
 
 ## Pre-commit Hooks
 
@@ -928,7 +929,7 @@ repos:
         pass_filenames: true
 ```
 
-See **[Pre-commit Hooks Guide](docs/pre-commit-hooks.md)** for complete documentation, troubleshooting, and advanced configuration.
+See **[Pre-commit Hooks Guide](https://thai-lint.readthedocs.io/en/latest/pre-commit-hooks/)** for complete documentation, troubleshooting, and advanced configuration.
 
 ## Common Use Cases
 
@@ -1145,20 +1146,20 @@ docker run --rm -v /path/to/workspace:/workspace \
 
 ### Comprehensive Guides
 
-- **[Getting Started](docs/getting-started.md)** - Installation, first lint, basic config
-- **[Configuration Reference](docs/configuration.md)** - Complete config options (YAML/JSON)
-- **[How to Ignore Violations](docs/how-to-ignore-violations.md)** - Complete guide to all ignore levels
-- **[API Reference](docs/api-reference.md)** - Library API documentation
-- **[CLI Reference](docs/cli-reference.md)** - All CLI commands and options
-- **[Deployment Modes](docs/deployment-modes.md)** - CLI, Library, and Docker usage
-- **[File Placement Linter](docs/file-placement-linter.md)** - Detailed linter guide
-- **[Magic Numbers Linter](docs/magic-numbers-linter.md)** - Magic numbers detection guide
-- **[Nesting Depth Linter](docs/nesting-linter.md)** - Nesting depth analysis guide
-- **[SRP Linter](docs/srp-linter.md)** - Single Responsibility Principle guide
-- **[DRY Linter](docs/dry-linter.md)** - Duplicate code detection guide
-- **[Pre-commit Hooks](docs/pre-commit-hooks.md)** - Automated quality checks
-- **[Publishing Guide](docs/releasing.md)** - Release and publishing workflow
-- **[Publishing Checklist](docs/publishing-checklist.md)** - Post-publication validation
+- **[Getting Started](https://thai-lint.readthedocs.io/en/latest/getting-started/)** - Installation, first lint, basic config
+- **[Configuration Reference](https://thai-lint.readthedocs.io/en/latest/configuration/)** - Complete config options (YAML/JSON)
+- **[How to Ignore Violations](https://thai-lint.readthedocs.io/en/latest/how-to-ignore-violations/)** - Complete guide to all ignore levels
+- **[API Reference](https://thai-lint.readthedocs.io/en/latest/api-reference/)** - Library API documentation
+- **[CLI Reference](https://thai-lint.readthedocs.io/en/latest/cli-reference/)** - All CLI commands and options
+- **[Deployment Modes](https://thai-lint.readthedocs.io/en/latest/deployment-modes/)** - CLI, Library, and Docker usage
+- **[File Placement Linter](https://thai-lint.readthedocs.io/en/latest/file-placement-linter/)** - Detailed linter guide
+- **[Magic Numbers Linter](https://thai-lint.readthedocs.io/en/latest/magic-numbers-linter/)** - Magic numbers detection guide
+- **[Nesting Depth Linter](https://thai-lint.readthedocs.io/en/latest/nesting-linter/)** - Nesting depth analysis guide
+- **[SRP Linter](https://thai-lint.readthedocs.io/en/latest/srp-linter/)** - Single Responsibility Principle guide
+- **[DRY Linter](https://thai-lint.readthedocs.io/en/latest/dry-linter/)** - Duplicate code detection guide
+- **[Pre-commit Hooks](https://thai-lint.readthedocs.io/en/latest/pre-commit-hooks/)** - Automated quality checks
+- **[Publishing Guide](https://thai-lint.readthedocs.io/en/latest/releasing/)** - Release and publishing workflow
+- **[Publishing Checklist](https://thai-lint.readthedocs.io/en/latest/publishing-checklist/)** - Post-publication validation
 
 ### Examples
 

{thailint-0.4.4 → thailint-0.4.6}/README.md

@@ -2,19 +2,20 @@
 
 [![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-296%2F296%20passing-brightgreen.svg)](tests/)
+[![Tests](https://img.shields.io/badge/tests-356%2F356%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)
 
 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
+- **[Quick Start Guide](https://thai-lint.readthedocs.io/en/latest/quick-start/)** - Get running in 5 minutes
+- **[Configuration Reference](https://thai-lint.readthedocs.io/en/latest/configuration/)** - Complete config options for all linters
+- **[Troubleshooting Guide](https://thai-lint.readthedocs.io/en/latest/troubleshooting/)** - Common issues and solutions
 
-**Full Documentation:** Browse the **[docs/](docs/)** folder for comprehensive guides covering installation, all linters, configuration patterns, and integration examples.
+**Full Documentation:** Browse the **[documentation site](https://thai-lint.readthedocs.io/)** for comprehensive guides covering installation, all linters, configuration patterns, and integration examples.
 
 ## Overview
 
@@ -31,7 +32,7 @@ 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.
+**Complete documentation available at [thai-lint.readthedocs.io](https://thai-lint.readthedocs.io/)** covering installation, configuration, all linters, and troubleshooting.
 
 ## Features
 
@@ -129,7 +130,7 @@ 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.
+**New to thailint?** See the **[Quick Start Guide](https://thai-lint.readthedocs.io/en/latest/quick-start/)** for a complete walkthrough including config generation, understanding output, and next steps.
 
 ### Library Mode
 
@@ -321,9 +322,9 @@ magic-numbers:
 }
 ```
 
-See [Configuration Guide](docs/configuration.md) for complete reference.
+See [Configuration Guide](https://thai-lint.readthedocs.io/en/latest/configuration/) for complete reference.
 
-**Need help with ignores?** See **[How to Ignore Violations](docs/how-to-ignore-violations.md)** for complete guide to all ignore levels (line, method, class, file, repository).
+**Need help with ignores?** See **[How to Ignore Violations](https://thai-lint.readthedocs.io/en/latest/how-to-ignore-violations/)** for complete guide to all ignore levels (line, method, class, file, repository).
 
 ## Nesting Depth Linter
 
@@ -409,7 +410,7 @@ Common patterns to reduce nesting:
 - **TypeScript**: Full support (if/for/while/try/switch)
 - **JavaScript**: Supported via TypeScript parser
 
-See [Nesting Linter Guide](docs/nesting-linter.md) for comprehensive documentation and refactoring patterns.
+See [Nesting Linter Guide](https://thai-lint.readthedocs.io/en/latest/nesting-linter/) for comprehensive documentation and refactoring patterns.
 
 ## Single Responsibility Principle (SRP) Linter
 
@@ -526,7 +527,7 @@ Common patterns to fix SRP violations (discovered during dogfooding):
 - **After**: Extract Class pattern applied - 5 focused classes (ConfigLoader, PatternValidator, RuleChecker, PathResolver, FilePlacementLinter)
 - **Result**: Each class ≤8 methods, ≤150 LOC, single responsibility
 
-See [SRP Linter Guide](docs/srp-linter.md) for comprehensive documentation and refactoring patterns.
+See [SRP Linter Guide](https://thai-lint.readthedocs.io/en/latest/srp-linter/) for comprehensive documentation and refactoring patterns.
 
 ## DRY Linter (Don't Repeat Yourself)
 
@@ -679,7 +680,7 @@ Built-in filters automatically exclude common non-duplication patterns:
 3. **Extract Utility Module**: Move helper functions to shared utilities
 4. **Template Method**: Use function parameters for variations
 
-See [DRY Linter Guide](docs/dry-linter.md) for comprehensive documentation, storage modes, and refactoring patterns.
+See [DRY Linter Guide](https://thai-lint.readthedocs.io/en/latest/dry-linter/) for comprehensive documentation, storage modes, and refactoring patterns.
 
 ## Magic Numbers Linter
 
@@ -831,7 +832,7 @@ def get_ports():  # thailint: ignore[magic-numbers] - Standard ports
 # thailint: ignore-file[magic-numbers]
 ```
 
-See **[How to Ignore Violations](docs/how-to-ignore-violations.md)** and **[Magic Numbers Linter Guide](docs/magic-numbers-linter.md)** for complete documentation.
+See **[How to Ignore Violations](https://thai-lint.readthedocs.io/en/latest/how-to-ignore-violations/)** and **[Magic Numbers Linter Guide](https://thai-lint.readthedocs.io/en/latest/magic-numbers-linter/)** for complete documentation.
 
 ## Pre-commit Hooks
 
@@ -893,7 +894,7 @@ repos:
         pass_filenames: true
 ```
 
-See **[Pre-commit Hooks Guide](docs/pre-commit-hooks.md)** for complete documentation, troubleshooting, and advanced configuration.
+See **[Pre-commit Hooks Guide](https://thai-lint.readthedocs.io/en/latest/pre-commit-hooks/)** for complete documentation, troubleshooting, and advanced configuration.
 
 ## Common Use Cases
 
@@ -1110,20 +1111,20 @@ docker run --rm -v /path/to/workspace:/workspace \
 
 ### Comprehensive Guides
 
-- **[Getting Started](docs/getting-started.md)** - Installation, first lint, basic config
-- **[Configuration Reference](docs/configuration.md)** - Complete config options (YAML/JSON)
-- **[How to Ignore Violations](docs/how-to-ignore-violations.md)** - Complete guide to all ignore levels
-- **[API Reference](docs/api-reference.md)** - Library API documentation
-- **[CLI Reference](docs/cli-reference.md)** - All CLI commands and options
-- **[Deployment Modes](docs/deployment-modes.md)** - CLI, Library, and Docker usage
-- **[File Placement Linter](docs/file-placement-linter.md)** - Detailed linter guide
-- **[Magic Numbers Linter](docs/magic-numbers-linter.md)** - Magic numbers detection guide
-- **[Nesting Depth Linter](docs/nesting-linter.md)** - Nesting depth analysis guide
-- **[SRP Linter](docs/srp-linter.md)** - Single Responsibility Principle guide
-- **[DRY Linter](docs/dry-linter.md)** - Duplicate code detection guide
-- **[Pre-commit Hooks](docs/pre-commit-hooks.md)** - Automated quality checks
-- **[Publishing Guide](docs/releasing.md)** - Release and publishing workflow
-- **[Publishing Checklist](docs/publishing-checklist.md)** - Post-publication validation
+- **[Getting Started](https://thai-lint.readthedocs.io/en/latest/getting-started/)** - Installation, first lint, basic config
+- **[Configuration Reference](https://thai-lint.readthedocs.io/en/latest/configuration/)** - Complete config options (YAML/JSON)
+- **[How to Ignore Violations](https://thai-lint.readthedocs.io/en/latest/how-to-ignore-violations/)** - Complete guide to all ignore levels
+- **[API Reference](https://thai-lint.readthedocs.io/en/latest/api-reference/)** - Library API documentation
+- **[CLI Reference](https://thai-lint.readthedocs.io/en/latest/cli-reference/)** - All CLI commands and options
+- **[Deployment Modes](https://thai-lint.readthedocs.io/en/latest/deployment-modes/)** - CLI, Library, and Docker usage
+- **[File Placement Linter](https://thai-lint.readthedocs.io/en/latest/file-placement-linter/)** - Detailed linter guide
+- **[Magic Numbers Linter](https://thai-lint.readthedocs.io/en/latest/magic-numbers-linter/)** - Magic numbers detection guide
+- **[Nesting Depth Linter](https://thai-lint.readthedocs.io/en/latest/nesting-linter/)** - Nesting depth analysis guide
+- **[SRP Linter](https://thai-lint.readthedocs.io/en/latest/srp-linter/)** - Single Responsibility Principle guide
+- **[DRY Linter](https://thai-lint.readthedocs.io/en/latest/dry-linter/)** - Duplicate code detection guide
+- **[Pre-commit Hooks](https://thai-lint.readthedocs.io/en/latest/pre-commit-hooks/)** - Automated quality checks
+- **[Publishing Guide](https://thai-lint.readthedocs.io/en/latest/releasing/)** - Release and publishing workflow
+- **[Publishing Checklist](https://thai-lint.readthedocs.io/en/latest/publishing-checklist/)** - Post-publication validation
 
 ### Examples
 

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

@@ -17,14 +17,14 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "thailint"
-version = "0.4.4"
+version = "0.4.6"
 description = "The AI Linter - Enterprise-grade linting and governance for AI-generated code across multiple languages"
 authors = ["Steve Jackson"]
 license = "MIT"
 readme = "README.md"
 homepage = "https://github.com/be-wise-be-kind/thai-lint"
 repository = "https://github.com/be-wise-be-kind/thai-lint"
-documentation = "https://github.com/be-wise-be-kind/thai-lint#readme"
+documentation = "https://thai-lint.readthedocs.io/"
 keywords = [
     "linter",
     "ai",

{thailint-0.4.4 → thailint-0.4.6}/src/linters/dry/python_analyzer.py

@@ -65,8 +65,12 @@ class PythonDuplicateAnalyzer(BaseTokenAnalyzer):  # thailint: ignore[srp.violat
         # Performance optimization: Cache parsed AST to avoid re-parsing for each hash window
         self._cached_ast: ast.Module | None = None
         self._cached_content: str | None = None
+        # Performance optimization: Line-to-node index for O(1) lookups instead of O(n) ast.walk()
+        self._line_to_nodes: dict[int, list[ast.AST]] | None = None
 
-    def analyze(self, file_path: Path, content: str, config: DRYConfig) -> list[CodeBlock]:
+    def analyze(  # thailint: ignore[nesting.excessive-depth]
+        self, file_path: Path, content: str, config: DRYConfig
+    ) -> list[CodeBlock]:
         """Analyze Python file for duplicate code blocks, excluding docstrings.
 
         Args:
@@ -81,6 +85,9 @@ class PythonDuplicateAnalyzer(BaseTokenAnalyzer):  # thailint: ignore[srp.violat
         self._cached_ast = self._parse_content_safe(content)
         self._cached_content = content
 
+        # Performance optimization: Build line-to-node index for O(1) lookups
+        self._line_to_nodes = self._build_line_to_node_index(self._cached_ast)
+
         try:
             # Get docstring line ranges
             docstring_ranges = self._get_docstring_ranges_from_content(content)
@@ -89,34 +96,58 @@ class PythonDuplicateAnalyzer(BaseTokenAnalyzer):  # thailint: ignore[srp.violat
             lines_with_numbers = self._tokenize_with_line_numbers(content, docstring_ranges)
 
             # Generate rolling hash windows
-            windows = self._rolling_hash_with_tracking(lines_with_numbers, config.min_duplicate_lines)
-
-            blocks = []
-            for hash_val, start_line, end_line, snippet in windows:
-                # Skip blocks that are single logical statements
-                # Check the original source code, not the normalized snippet
-                if self._is_single_statement_in_source(content, start_line, end_line):
-                    continue
-
-                block = CodeBlock(
-                    file_path=file_path,
-                    start_line=start_line,
-                    end_line=end_line,
-                    snippet=snippet,
-                    hash_value=hash_val,
-                )
-
-                # Apply extensible filters (keyword arguments, imports, etc.)
-                if self._filter_registry.should_filter_block(block, content):
-                    continue
-
-                blocks.append(block)
+            windows = self._rolling_hash_with_tracking(
+                lines_with_numbers, config.min_duplicate_lines
+            )
 
-            return blocks
+            return self._filter_valid_blocks(windows, file_path, content)
         finally:
             # Clear cache after analysis to avoid memory leaks
             self._cached_ast = None
             self._cached_content = None
+            self._line_to_nodes = None
+
+    def _filter_valid_blocks(
+        self,
+        windows: list[tuple[int, int, int, str]],
+        file_path: Path,
+        content: str,
+    ) -> list[CodeBlock]:
+        """Filter hash windows and create valid CodeBlock instances."""
+        blocks = []
+        for hash_val, start_line, end_line, snippet in windows:
+            block = self._create_block_if_valid(
+                file_path, content, hash_val, start_line, end_line, snippet
+            )
+            if block:
+                blocks.append(block)
+        return blocks
+
+    def _create_block_if_valid(  # pylint: disable=too-many-arguments,too-many-positional-arguments
+        self,
+        file_path: Path,
+        content: str,
+        hash_val: int,
+        start_line: int,
+        end_line: int,
+        snippet: str,
+    ) -> CodeBlock | None:
+        """Create CodeBlock if it passes all validation checks."""
+        if self._is_single_statement_in_source(content, start_line, end_line):
+            return None
+
+        block = CodeBlock(
+            file_path=file_path,
+            start_line=start_line,
+            end_line=end_line,
+            snippet=snippet,
+            hash_value=hash_val,
+        )
+
+        if self._filter_registry.should_filter_block(block, content):
+            return None
+
+        return block
 
     def _get_docstring_ranges_from_content(self, content: str) -> set[int]:
         """Extract line numbers that are part of docstrings.
@@ -184,20 +215,21 @@ class PythonDuplicateAnalyzer(BaseTokenAnalyzer):  # thailint: ignore[srp.violat
             List of (original_line_number, normalized_code) tuples
         """
         lines_with_numbers = []
+        in_multiline_import = False
 
         for line_num, line in enumerate(content.split("\n"), start=1):
-            # Skip docstring lines
             if line_num in docstring_lines:
                 continue
 
-            # Use hasher's existing tokenization logic
-            line = self._hasher._strip_comments(line)  # pylint: disable=protected-access
-            line = " ".join(line.split())
-
+            line = self._hasher._normalize_line(line)  # pylint: disable=protected-access
             if not line:
                 continue
 
-            if self._hasher._is_import_statement(line):  # pylint: disable=protected-access
+            # Update multi-line import state and check if line should be skipped
+            in_multiline_import, should_skip = self._hasher._should_skip_import_line(  # pylint: disable=protected-access
+                line, in_multiline_import
+            )
+            if should_skip:
                 continue
 
             lines_with_numbers.append((line_num, line))
@@ -243,6 +275,7 @@ class PythonDuplicateAnalyzer(BaseTokenAnalyzer):  # thailint: ignore[srp.violat
         to avoid re-parsing the entire file for each hash window check.
         """
         # Use cached AST if available and content matches
+        tree: ast.Module | None
         if self._cached_ast is not None and content == self._cached_content:
             tree = self._cached_ast
         else:
@@ -261,23 +294,99 @@ class PythonDuplicateAnalyzer(BaseTokenAnalyzer):  # thailint: ignore[srp.violat
         except SyntaxError:
             return None
 
-    def _check_overlapping_nodes(self, tree: ast.Module, start_line: int, end_line: int) -> bool:
-        """Check if any AST node overlaps and matches single-statement pattern.
+    @staticmethod
+    def _build_line_to_node_index(tree: ast.Module | None) -> dict[int, list[ast.AST]] | None:
+        """Build an index mapping each line number to overlapping AST nodes.
+
+        Performance optimization: This allows O(1) lookups instead of O(n) ast.walk() calls.
+        For a file with 5,144 nodes and 673 hash windows, this reduces 3.46M node operations
+        to just ~3,365 relevant node checks (99.9% reduction).
 
-        Performance optimization: Pre-filter nodes by line range before expensive pattern checks.
+        Args:
+            tree: Parsed AST tree (None if parsing failed)
+
+        Returns:
+            Dictionary mapping line numbers to list of AST nodes overlapping that line,
+            or None if tree is None
         """
+        if tree is None:
+            return None
+
+        line_to_nodes: dict[int, list[ast.AST]] = {}
         for node in ast.walk(tree):
-            # Quick line range check to skip nodes that don't overlap
-            if not hasattr(node, "lineno") or not hasattr(node, "end_lineno"):
-                continue
-            if node.end_lineno < start_line or node.lineno > end_line:
-                continue  # No overlap, skip expensive pattern matching
+            if PythonDuplicateAnalyzer._node_has_line_info(node):
+                PythonDuplicateAnalyzer._add_node_to_index(node, line_to_nodes)
+
+        return line_to_nodes
 
-            # Node overlaps - check if it matches single-statement pattern
+    @staticmethod
+    def _node_has_line_info(node: ast.AST) -> bool:
+        """Check if node has valid line number information."""
+        if not hasattr(node, "lineno") or not hasattr(node, "end_lineno"):
+            return False
+        return node.lineno is not None and node.end_lineno is not None
+
+    @staticmethod
+    def _add_node_to_index(node: ast.AST, line_to_nodes: dict[int, list[ast.AST]]) -> None:
+        """Add node to all lines it overlaps in the index."""
+        for line_num in range(node.lineno, node.end_lineno + 1):  # type: ignore[attr-defined]
+            if line_num not in line_to_nodes:
+                line_to_nodes[line_num] = []
+            line_to_nodes[line_num].append(node)
+
+    def _check_overlapping_nodes(self, tree: ast.Module, start_line: int, end_line: int) -> bool:
+        """Check if any AST node overlaps and matches single-statement pattern.
+
+        Performance optimization: Use line-to-node index for O(1) lookups instead of O(n) ast.walk().
+        """
+        if self._line_to_nodes is not None:
+            return self._check_nodes_via_index(start_line, end_line)
+        return self._check_nodes_via_walk(tree, start_line, end_line)
+
+    def _check_nodes_via_index(self, start_line: int, end_line: int) -> bool:
+        """Check nodes using line-to-node index for O(1) lookups."""
+        candidates = self._collect_candidate_nodes_from_index(start_line, end_line)
+        return self._any_node_matches_pattern(candidates, start_line, end_line)
+
+    def _collect_candidate_nodes_from_index(self, start_line: int, end_line: int) -> set[ast.AST]:
+        """Collect unique nodes that overlap with the line range from index."""
+        candidate_nodes: set[ast.AST] = set()
+        for line_num in range(start_line, end_line + 1):
+            if self._line_to_nodes and line_num in self._line_to_nodes:
+                candidate_nodes.update(self._line_to_nodes[line_num])
+        return candidate_nodes
+
+    def _any_node_matches_pattern(
+        self, nodes: set[ast.AST], start_line: int, end_line: int
+    ) -> bool:
+        """Check if any node matches single-statement pattern."""
+        for node in nodes:
             if self._is_single_statement_pattern(node, start_line, end_line):
                 return True
         return False
 
+    def _check_nodes_via_walk(self, tree: ast.Module, start_line: int, end_line: int) -> bool:
+        """Check nodes using ast.walk() fallback for tests or standalone calls."""
+        for node in ast.walk(tree):
+            if self._node_matches_via_walk(node, start_line, end_line):
+                return True
+        return False
+
+    def _node_matches_via_walk(self, node: ast.AST, start_line: int, end_line: int) -> bool:
+        """Check if a single node overlaps and matches pattern."""
+        if not self._node_overlaps_range(node, start_line, end_line):
+            return False
+        return self._is_single_statement_pattern(node, start_line, end_line)
+
+    @staticmethod
+    def _node_overlaps_range(node: ast.AST, start_line: int, end_line: int) -> bool:
+        """Check if node overlaps with the given line range."""
+        if not hasattr(node, "lineno") or not hasattr(node, "end_lineno"):
+            return False
+        node_end = node.end_lineno
+        node_start = node.lineno
+        return not (node_end < start_line or node_start > end_line)
+
     def _node_overlaps_and_matches(self, node: ast.AST, start_line: int, end_line: int) -> bool:
         """Check if node overlaps with range and matches single-statement pattern."""
         if not hasattr(node, "lineno") or not hasattr(node, "end_lineno"):

{thailint-0.4.4 → thailint-0.4.6}/src/linters/dry/token_hasher.py

@@ -33,26 +33,80 @@ class TokenHasher:
             List of normalized code lines (non-empty, comments removed, imports filtered)
         """
         lines = []
+        in_multiline_import = False
 
         for line in code.split("\n"):
-            # Remove comments (language-specific logic can be added)
-            line = self._strip_comments(line)
-
-            # Normalize whitespace (collapse to single space)
-            line = " ".join(line.split())
-
-            # Skip empty lines
+            line = self._normalize_line(line)
             if not line:
                 continue
 
-            # Skip import statements (common false positive)
-            if self._is_import_statement(line):
+            # Update multi-line import state and check if line should be skipped
+            in_multiline_import, should_skip = self._should_skip_import_line(
+                line, in_multiline_import
+            )
+            if should_skip:
                 continue
 
             lines.append(line)
 
         return lines
 
+    def _normalize_line(self, line: str) -> str:
+        """Normalize a line by removing comments and excess whitespace.
+
+        Args:
+            line: Raw source code line
+
+        Returns:
+            Normalized line (empty string if line has no content)
+        """
+        line = self._strip_comments(line)
+        return " ".join(line.split())
+
+    def _should_skip_import_line(self, line: str, in_multiline_import: bool) -> tuple[bool, bool]:
+        """Determine if an import line should be skipped.
+
+        Args:
+            line: Normalized code line
+            in_multiline_import: Whether we're currently inside a multi-line import
+
+        Returns:
+            Tuple of (new_in_multiline_import_state, should_skip_line)
+        """
+        if self._is_multiline_import_start(line):
+            return True, True
+
+        if in_multiline_import:
+            return self._handle_multiline_import_continuation(line)
+
+        if self._is_import_statement(line):
+            return False, True
+
+        return False, False
+
+    def _is_multiline_import_start(self, line: str) -> bool:
+        """Check if line starts a multi-line import statement.
+
+        Args:
+            line: Normalized code line
+
+        Returns:
+            True if line starts a multi-line import (has opening paren but no closing)
+        """
+        return self._is_import_statement(line) and "(" in line and ")" not in line
+
+    def _handle_multiline_import_continuation(self, line: str) -> tuple[bool, bool]:
+        """Handle a line that's part of a multi-line import.
+
+        Args:
+            line: Normalized code line inside a multi-line import
+
+        Returns:
+            Tuple of (still_in_import, should_skip)
+        """
+        closes_import = ")" in line
+        return not closes_import, True
+
     def _strip_comments(self, line: str) -> str:
         """Remove comments from line (Python # and // style).