sarj-python-lint 1.25.0__tar.gz → 1.25.1__tar.gz

{sarj_python_lint-1.25.0 → sarj_python_lint-1.25.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: sarj-python-lint
-Version: 1.25.0
+Version: 1.25.1
 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.25.0 → sarj_python_lint-1.25.1}/pyproject.toml

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

{sarj_python_lint-1.25.0 → sarj_python_lint-1.25.1}/src/sarj_python_lint/rules/prefer_collection_comprehension.py

@@ -112,7 +112,8 @@ class PreferCollectionComprehension(Rule):
             (
                 "Comments, loop-target leakage, try blocks, aliases, complex projections, and replacements that "
                 "do not fit a 120-column line or a simple formatter-style multiline comprehension are excluded. "
-                "Namespace observers and collections referenced by nested functions are excluded. No autofix is offered because dict key/value evaluation order can differ."
+                "Namespace observers, explicit inspect/sys frame access, and collections referenced by nested functions "
+                "are excluded. No autofix is offered because dict key/value evaluation order can differ."
             ),
             (
                 "Attribute projections can invoke properties or descriptors, so reviewers should keep the loop "
@@ -672,29 +673,33 @@ def _owner_bindings(owner: _Callable, namespace_observers: frozenset[int]) -> _O
 
 
 def _namespace_observer_calls(context: PythonFileContext) -> frozenset[int]:
-    observers = {"locals", "vars", "eval", "exec"}
+    observers = {
+        "builtins": {"locals", "vars", "eval", "exec"},
+        "inspect": {"currentframe"},
+        "sys": {"_getframe"},
+    }
     # Explicit aliases remain conservative exclusions even when another scope shadows them.
-    modules: set[str] = set()
-    symbols = observers.copy()
+    modules: defaultdict[str, set[str]] = defaultdict(set)
+    symbols = observers["builtins"].copy()
     for node in context.nodes(ast.Import):
         for alias in node.names:
-            if alias.name == "builtins":
-                modules.add(alias.asname or alias.name)
+            if alias.name in observers:
+                modules[alias.asname or alias.name].update(observers[alias.name])
     for node in context.nodes(ast.ImportFrom):
-        if node.module != "builtins":
+        if node.module not in observers:
             continue
-        symbols.update(alias.asname or alias.name for alias in node.names if alias.name in observers)
+        symbols.update(alias.asname or alias.name for alias in node.names if alias.name in observers[node.module])
     return frozenset(
         id(call) for call in context.nodes(ast.Call) if _is_namespace_observer(call.func, symbols, modules)
     )
 
 
-def _is_namespace_observer(function: ast.expr, symbols: set[str], modules: set[str]) -> bool:
+def _is_namespace_observer(function: ast.expr, symbols: set[str], modules: dict[str, set[str]]) -> bool:
     match function:
         case ast.Name(id=name):
             return name in symbols
         case ast.Attribute(value=ast.Name(id=module), attr=symbol):
-            return module in modules and symbol in {"locals", "vars", "eval", "exec"}
+            return module in modules and symbol in modules[module]
         case _:
             return False