sarj-python-lint 1.28.2__tar.gz → 1.28.3__tar.gz

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: sarj-python-lint
-Version: 1.28.2
+Version: 1.28.3
 Summary: Custom Python lint rules — AST-based, pre-commit-friendly, hypermodern defaults
 Project-URL: Homepage, https://code-standards.sarj.ai/rules/python/
 Project-URL: Documentation, https://code-standards.sarj.ai/rules/python/
@@ -19,7 +19,7 @@ Classifier: Topic :: Software Development :: Quality Assurance
 Requires-Python: >=3.14
 Requires-Dist: packaging==26.3
 Requires-Dist: ruff>=0.16.10
-Requires-Dist: sarj-rule-contracts==0.2.2
+Requires-Dist: sarj-rule-contracts==0.2.3
 Requires-Dist: sqlglot==30.19.0
 Requires-Dist: typer==0.27.2
 Description-Content-Type: text/markdown

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "sarj-python-lint"
-version = "1.28.2"
+version = "1.28.3"
 description = "Custom Python lint rules — AST-based, pre-commit-friendly, hypermodern defaults"
 readme = "README.md"
 authors = [{ name = "sarj-ai" }]
@@ -15,7 +15,7 @@ classifiers = [
   "Programming Language :: Python :: 3.14",
   "Topic :: Software Development :: Quality Assurance",
 ]
-dependencies = ["sarj-rule-contracts==0.2.2", "packaging==26.3", "ruff>=0.16.10", "sqlglot==30.19.0", "typer==0.27.2"]
+dependencies = ["sarj-rule-contracts==0.2.3", "packaging==26.3", "ruff>=0.16.10", "sqlglot==30.19.0", "typer==0.27.2"]
 
 [project.scripts]
 sarj-python-lint = "sarj_python_lint.__main__:main"

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/src/sarj_python_lint/_file_context.py

@@ -57,6 +57,10 @@ class PythonFileContext:
     def nodes[NodeT: ast.AST](self, *types: type[NodeT]) -> list[NodeT]:
         return [node for node in self.node_index.query(types) if isinstance(node, types)]
 
+    @cached_property
+    def has_wildcard_import(self) -> bool:
+        return any(alias.name == "*" for node in self.nodes(ast.ImportFrom) for alias in node.names)
+
     @property
     def parents(self) -> Mapping[ast.AST, ast.AST]:
         return self.node_index.parents

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/src/sarj_python_lint/rules/no_conftest_test_module_import.py

@@ -131,11 +131,12 @@ def _pytest_plugin_test_targets(tree: ast.Module) -> list[_PluginTarget]:
     for statement in tree.body:
         value: ast.expr | None = None
         match statement:
-            case (
-                ast.Assign(targets=[ast.Name(id="pytest_plugins")])
-                | ast.AnnAssign(target=ast.Name(id="pytest_plugins"))
+            case ast.Assign(targets=targets) if any(
+                isinstance(target, ast.Name) and target.id == "pytest_plugins" for target in targets
             ):
                 value = statement.value
+            case ast.AnnAssign(target=ast.Name(id="pytest_plugins")):
+                value = statement.value
             case _:
                 continue
         findings.extend(

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/src/sarj_python_lint/rules/prefer_match_exception_dispatch.py

@@ -177,7 +177,7 @@ class PreferMatchExceptionDispatch(Rule):
         imports = context.imports
         if (
             not imports.builtin_is_unshadowed("isinstance")
-            or _has_wildcard_import(tree, node_index=context.node_index)
+            or context.has_wildcard_import
             or _has_local_import(tree, node_index=context.node_index)
         ):
             return []
@@ -412,13 +412,6 @@ def _body_terminates(body: list[ast.stmt]) -> bool:
             return False
 
 
-def _has_wildcard_import(tree: ast.Module, *, node_index: NodeIndex | None = None) -> bool:
-    return any(
-        isinstance(node, ast.ImportFrom) and any(alias.name == "*" for alias in node.names)
-        for node in walk_ast(tree, index=node_index)
-    )
-
-
 def _has_local_import(tree: ast.Module, *, node_index: NodeIndex | None = None) -> bool:
     module_imports = {id(statement) for statement in tree.body if isinstance(statement, (ast.Import, ast.ImportFrom))}
     return any(

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/src/sarj_python_lint/rules/prefer_namedtuple_over_tuple_return.py

@@ -16,12 +16,10 @@ from sarj_python_lint.rule_base import (
     Severity,
     parse_or_none,
 )
-from sarj_python_lint.rules._ast_index import walk as walk_ast
 
 
 if TYPE_CHECKING:
     from sarj_python_lint._file_context import PythonFileContext
-    from sarj_python_lint.rules._ast_index import NodeIndex
     from sarj_python_lint.rules._imports import ImportIndex
 
 
@@ -101,7 +99,7 @@ class PreferNamedtupleOverTupleReturn(Rule):
         if context.generated or _is_documentation_path(path):
             return []
         tree = context.tree
-        if tree is None or _has_wildcard_import(tree, node_index=context.node_index):
+        if tree is None or context.has_wildcard_import:
             return []
         imports = context.imports
         diagnostics: list[Diagnostic] = []
@@ -127,13 +125,6 @@ def _is_documentation_path(path: Path) -> bool:
     return any(part.lower() in _DOCUMENTATION_DIR_NAMES for part in path.parts)
 
 
-def _has_wildcard_import(tree: ast.Module, *, node_index: NodeIndex | None = None) -> bool:
-    return any(
-        isinstance(node, ast.ImportFrom) and any(alias.name == "*" for alias in node.names)
-        for node in walk_ast(tree, index=node_index)
-    )
-
-
 def _is_public_record_tuple(annotation: ast.expr, imports: ImportIndex) -> bool:
     annotation = _unwrap_return_annotation(annotation, imports)
     if not isinstance(annotation, ast.Subscript) or not isinstance(annotation.slice, ast.Tuple):

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/src/sarj_python_lint/rules/prefer_nominal_id_types.py

@@ -27,7 +27,6 @@ if TYPE_CHECKING:
     from pathlib import Path
 
     from sarj_python_lint._file_context import PythonFileContext
-    from sarj_python_lint.rules._ast_index import NodeIndex
     from sarj_python_lint.rules._imports import ImportIndex
 
 
@@ -221,7 +220,7 @@ class PreferNominalIdTypes(Rule):
         if _is_excluded_path(path) or context.generated:
             return []
         tree = context.tree
-        if tree is None or _has_wildcard_import(tree, node_index=context.node_index):
+        if tree is None or context.has_wildcard_import:
             return []
 
         imports = context.imports
@@ -311,13 +310,6 @@ def _is_excluded_path(path: Path) -> bool:
     )
 
 
-def _has_wildcard_import(tree: ast.Module, *, node_index: NodeIndex | None = None) -> bool:
-    return any(
-        isinstance(node, ast.ImportFrom) and any(alias.name == "*" for alias in node.names)
-        for node in walk_ast(tree, index=node_index)
-    )
-
-
 def _boundary_nodes(
     tree: ast.Module,
     imports: ImportIndex,

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/src/sarj_python_lint/rules/prefer_non_nullable_collection.py

@@ -23,7 +23,6 @@ if TYPE_CHECKING:
     from collections.abc import Iterator
 
     from sarj_python_lint._file_context import PythonFileContext
-    from sarj_python_lint.rules._ast_index import NodeIndex
     from sarj_python_lint.rules._imports import ImportIndex
 
 
@@ -99,7 +98,7 @@ class PreferNonNullableCollection(Rule):
         if is_test_path(path) or is_test_support_path(path) or context.generated:
             return []
         tree = context.tree
-        if tree is None or _has_wildcard_import(tree, node_index=context.node_index):
+        if tree is None or context.has_wildcard_import:
             return []
 
         imports = context.imports
@@ -125,13 +124,6 @@ class PreferNonNullableCollection(Rule):
         return sorted(diagnostics, key=lambda diagnostic: (diagnostic.line, diagnostic.col))
 
 
-def _has_wildcard_import(tree: ast.Module, *, node_index: NodeIndex | None = None) -> bool:
-    return any(
-        isinstance(node, ast.ImportFrom) and any(alias.name == "*" for alias in node.names)
-        for node in walk_ast(tree, index=node_index)
-    )
-
-
 def _eligible_functions(tree: ast.Module, imports: ImportIndex) -> list[ast.FunctionDef | ast.AsyncFunctionDef]:
     functions: list[ast.FunctionDef | ast.AsyncFunctionDef] = []
     for statement in tree.body:

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/src/sarj_python_lint/rules/redundant_test_constructor_forwarder.py

@@ -61,7 +61,10 @@ class RedundantTestConstructorForwarder(ProjectRule):
                 example_id=name,
                 title=title,
                 outcome=outcome,
-                files=(ExampleFile.python("tests/test_record.py", source),),
+                files=(
+                    ExampleFile.python("tests/__init__.py", "# Example test package.\n"),
+                    ExampleFile.python("tests/test_record.py", source),
+                ),
                 focus_path=PurePosixPath("tests/test_record.py"),
                 expected_count=count,
                 public=True,

{sarj_python_lint-1.28.2 → sarj_python_lint-1.28.3}/src/sarj_python_lint/rules/repeated_unimplemented_test_contract.py

@@ -60,6 +60,7 @@ class RepeatedUnimplementedTestContract(ProjectRule):
                 title="Two test fakes repeat one unconfigured operation",
                 outcome=ExampleOutcome.MATCH,
                 files=(
+                    ExampleFile.python("tests/__init__.py", "# Example test package.\n"),
                     ExampleFile.python(
                         "tests/test_store.py",
                         "from abc import ABC, abstractmethod\nclass Store(ABC):\n    @abstractmethod\n    def read(self, key: str) -> str:\n        pass\nclass First(Store):\n    def read(self, key: str) -> str:\n        raise NotImplementedError\nclass Second(Store):\n    def read(self, key: str) -> str:\n        raise NotImplementedError\n",
@@ -74,6 +75,7 @@ class RepeatedUnimplementedTestContract(ProjectRule):
                 title="Configured fake behavior belongs to the leaf",
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
+                    ExampleFile.python("tests/__init__.py", "# Example test package.\n"),
                     ExampleFile.python(
                         "tests/test_store.py",
                         "from abc import ABC, abstractmethod\nclass Store(ABC):\n    @abstractmethod\n    def read(self, key: str) -> str:\n        pass\nclass First(Store):\n    def read(self, key: str) -> str:\n        raise NotImplementedError\nclass Second(Store):\n    def read(self, key: str) -> str:\n        return key\n",