sarj-python-lint 1.28.5__tar.gz → 1.28.6__tar.gz

{sarj_python_lint-1.28.5 → sarj_python_lint-1.28.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: sarj-python-lint
-Version: 1.28.5
+Version: 1.28.6
 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/

{sarj_python_lint-1.28.5 → sarj_python_lint-1.28.6}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "sarj-python-lint"
-version = "1.28.5"
+version = "1.28.6"
 description = "Custom Python lint rules — AST-based, pre-commit-friendly, hypermodern defaults"
 readme = "README.md"
 authors = [{ name = "sarj-ai" }]

{sarj_python_lint-1.28.5 → sarj_python_lint-1.28.6}/src/sarj_python_lint/_file_context.py

@@ -67,6 +67,17 @@ class PythonFileContext:
 
         return consumed_docstring_names(self._valid_tree(), node_index=self.node_index)
 
+    @cached_property
+    def docstring_node_ids(self) -> frozenset[int]:
+        result: set[int] = set()
+        for owner in self.nodes(ast.Module, ast.ClassDef, ast.FunctionDef, ast.AsyncFunctionDef):
+            if not owner.body or not isinstance(owner.body[0], ast.Expr):
+                continue
+            value = owner.body[0].value
+            if isinstance(value, ast.Constant) and isinstance(value.value, str):
+                result.add(id(value))
+        return frozenset(result)
+
     @property
     def parents(self) -> Mapping[ast.AST, ast.AST]:
         return self.node_index.parents

{sarj_python_lint-1.28.5 → sarj_python_lint-1.28.6}/src/sarj_python_lint/rules/complex_postgres_query_requires_architecture_review.py

@@ -71,18 +71,6 @@ class _ConstructorImport(NamedTuple):
     name: str
 
 
-def _docstring_node_ids(tree: ast.AST, *, node_index: NodeIndex | None = None) -> set[int]:
-    result: set[int] = set()
-    for owner in nodes(tree, ast.Module, ast.ClassDef, ast.FunctionDef, ast.AsyncFunctionDef, index=node_index):
-        body = owner.body
-        if not body or not isinstance(body[0], ast.Expr):
-            continue
-        value = body[0].value
-        if isinstance(value, ast.Constant) and isinstance(value.value, str):
-            result.add(id(value))
-    return result
-
-
 def _is_query_context(node: ast.expr, parents: Mapping[ast.AST, ast.AST], roots: frozenset[int]) -> bool:
     current: ast.AST = node
     while True:
@@ -368,7 +356,7 @@ class ComplexPostgresQueryRequiresArchitectureReview(Rule):
         if not _is_postgres_module(tree, node_index=context.node_index):
             return []
 
-        docstrings = _docstring_node_ids(tree, node_index=context.node_index)
+        docstrings = context.docstring_node_ids
         parents = context.parents
         query_roots = _query_roots(tree, parents, node_index=context.node_index)
         diagnostics: list[Diagnostic] = []

{sarj_python_lint-1.28.5 → sarj_python_lint-1.28.6}/src/sarj_python_lint/rules/no_analytical_aggregation_in_postgres_store.py

@@ -16,7 +16,7 @@ from sarj_python_lint.rule_base import (
     RuleExample,
     Severity,
 )
-from sarj_python_lint.rules._ast_index import nodes, walk
+from sarj_python_lint.rules._ast_index import walk
 from sarj_python_lint.rules._sql import is_store_module, sql_string_value, strip_sql_noise
 
 
@@ -24,7 +24,6 @@ if TYPE_CHECKING:
     from sqlglot import exp
 
     from sarj_python_lint._file_context import PythonFileContext
-    from sarj_python_lint.rules._ast_index import NodeIndex
 
 
 _QUERY_SHAPE = re.compile(r"\bSELECT\b[\s\S]*?\bFROM\b", re.IGNORECASE)
@@ -65,18 +64,6 @@ _STRONG_ANALYTICAL: tuple[tuple[str, re.Pattern[str]], ...] = (
 )
 
 
-def _docstring_node_ids(tree: ast.AST, *, node_index: NodeIndex | None = None) -> set[int]:
-    result: set[int] = set()
-    for owner in nodes(tree, ast.Module, ast.ClassDef, ast.FunctionDef, ast.AsyncFunctionDef, index=node_index):
-        body = owner.body
-        if not body or not isinstance(body[0], ast.Expr):
-            continue
-        value = body[0].value
-        if isinstance(value, ast.Constant) and isinstance(value.value, str):
-            result.add(id(value))
-    return result
-
-
 def analytical_signal(sql: str) -> str | None:
     if _LOCKING_READ.search(sql) is not None:
         return None
@@ -217,7 +204,7 @@ class NoAnalyticalAggregationInPostgresStore(Rule):
         if tree is None:
             return []
 
-        docstrings = _docstring_node_ids(tree, node_index=context.node_index)
+        docstrings = context.docstring_node_ids
         diagnostics: list[Diagnostic] = []
         consumed: set[int] = set()
         for node in context.nodes(ast.Constant, ast.BinOp):

{sarj_python_lint-1.28.5 → sarj_python_lint-1.28.6}/src/sarj_python_lint/rules/no_offset_pagination.py

@@ -16,13 +16,12 @@ from sarj_python_lint.rule_base import (
     RuleExample,
     Severity,
 )
-from sarj_python_lint.rules._ast_index import nodes, walk
+from sarj_python_lint.rules._ast_index import walk
 from sarj_python_lint.rules._sql import is_store_module, sql_string_value, strip_sql_noise
 
 
 if TYPE_CHECKING:
     from sarj_python_lint._file_context import PythonFileContext
-    from sarj_python_lint.rules._ast_index import NodeIndex
 
 
 _DYNAMIC_VALUE = r"(?:%s|%\(\w+\)s|\?\d*|:\w+|@\w+|\$\d+|:__sarj_dynamic__)"
@@ -107,7 +106,7 @@ class NoOffsetPagination(Rule):
 
         diags: list[Diagnostic] = []
         consumed: set[int] = set()
-        docstrings = _docstring_node_ids(tree, node_index=context.node_index)
+        docstrings = context.docstring_node_ids
         for node in context.nodes(ast.Constant, ast.BinOp, ast.JoinedStr):
             if id(node) in consumed or id(node) in docstrings:
                 continue
@@ -151,17 +150,6 @@ def _sql_value(node: ast.expr) -> str | None:
     return sql_string_value(node)
 
 
-def _docstring_node_ids(tree: ast.Module, *, node_index: NodeIndex | None = None) -> set[int]:
-    result: set[int] = set()
-    for owner in nodes(tree, ast.Module, ast.ClassDef, ast.FunctionDef, ast.AsyncFunctionDef, index=node_index):
-        if not owner.body or not isinstance(owner.body[0], ast.Expr):
-            continue
-        value = owner.body[0].value
-        if isinstance(value, ast.Constant) and isinstance(value.value, str):
-            result.add(id(value))
-    return result
-
-
 def _mask_dialect_noise(sql: str) -> str:
     sql = _QUOTED_IDENTIFIER.sub(lambda match: " " * len(match.group(0)), sql)
     return _MYSQL_COMMENT.sub(lambda match: " " * len(match.group(0)), sql)