sarj-python-lint 0.81.16__tar.gz → 0.81.18__tar.gz

{sarj_python_lint-0.81.16 → sarj_python_lint-0.81.18}/PKG-INFO

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

@@ -1,6 +1,6 @@
 [project]
 name = "sarj-python-lint"
-version = "0.81.16"
+version = "0.81.18"
 description = "Custom Python lint rules — AST-based, pre-commit-friendly, hypermodern defaults"
 readme = "README.md"
 authors = [{ name = "sarj-ai" }]
@@ -64,3 +64,8 @@ markers = [
 # Dogfooding: linted/formatted by this repo's own published config (root-synced).
 [tool.ruff]
 extend = "../../.ruff-strict.toml"
+
+[tool.deptry.per_rule_ignores]
+# Ruff is invoked at runtime through `sys.executable -m ruff`, which Deptry's
+# import analysis cannot observe.
+DEP002 = ["ruff"]

{sarj_python_lint-0.81.16 → sarj_python_lint-0.81.18}/src/sarj_python_lint/rules/prefer_match_type_dispatch.py

@@ -436,7 +436,7 @@ def _two_arm_ast_projection_findings(
                         line=first.lineno,
                         col=first.col_offset + 1,
                         code=code,
-                        severity=Severity.WARNING,
+                        severity=Severity.ERROR,
                         message=(
                             f"two-arm ast.Name/ast.Attribute projection on '{subject}' — use one match/case "
                             "statement so the structural alternatives are explicit."
@@ -581,7 +581,7 @@ def _diagnostic(path: Path, code: str, node: ast.If, dispatch: _Dispatch, shape:
         line=node.lineno,
         col=node.col_offset + 1,
         code=code,
-        severity=Severity.WARNING,
+        severity=Severity.ERROR,
         message=(
             f"{dispatch.branch_count}-branch {shape} on '{dispatch.subject}' — use match/case class patterns "
             "so the mutually exclusive runtime type dispatch is explicit."

{sarj_python_lint-0.81.16 → sarj_python_lint-0.81.18}/src/sarj_python_lint/rules/prefer_walrus_awaited_none_guard.py

@@ -50,14 +50,16 @@ class PreferWalrusAwaitedNoneGuard(Rule):
         remediation=(
             "Rewrite `value = await lookup(); if value is None: return` as "
             "`if (value := await lookup()) is None: return`. Preserve `is not None` when the terminal branch "
-            "returns or raises with the bound value. Keep two statements when the assignment is an intentional "
-            "debugging or tracing boundary."
+            "returns or raises with the bound value. A short message assignment may remain inside the guard before "
+            "the terminal statement. Keep two statements when the awaited assignment is an intentional debugging "
+            "or tracing boundary."
         ),
         category=RuleCategory.STYLE,
         autofix=AutofixPolicy.NONE,
         limitations=(
-            "Only one-line awaited calls followed immediately by an exact `is None` or `is not None` guard containing one return or raise are checked.",
-            "A negative guard requires a later use on the surviving path; a positive guard must consume the binding in its terminal statement and leave no later use before rebinding.",
+            "Only one-line awaited calls followed immediately by an exact `is None` or `is not None` guard ending in return or raise are checked.",
+            "Statements before the terminal operation must be simple assignments; expressions and compound control flow are excluded.",
+            "An `is None` guard requires a later use on the surviving path and must not use or rebind the name inside the guard; an `is not None` guard must consume the binding inside the terminal branch and leave no later use before rebinding.",
             "The combined condition must fit 120 columns; comments, else branches, ambiguous name flow, generated files, and broader assignment-expression preferences are excluded.",
         ),
         examples=(
@@ -167,17 +169,19 @@ def _candidate(
         and guard.col_offset == assignment.col_offset
         and guard.test.end_lineno == guard.lineno
         and not guard.orelse
-        and len(guard.body) == 1
-        and isinstance(guard.body[0], (ast.Return, ast.Raise))
+        and guard.body
+        and all(isinstance(statement, (ast.Assign, ast.AnnAssign)) for statement in guard.body[:-1])
+        and isinstance(guard.body[-1], (ast.Return, ast.Raise))
     ):
         return None
     name = assignment.targets[0].id
     is_not_none = _none_guard_kind(guard.test, name)
     if is_not_none is None or _loads_name(assignment.value, name):
         return None
-    terminal = guard.body[0]
+    terminal = guard.body[-1]
     usage = _NameUsage(name)
-    usage.visit(terminal)
+    for statement in guard.body:
+        usage.visit(statement)
     if usage.rebound or (is_not_none != usage.loaded):
         return None
     terminal_end = terminal.end_lineno or terminal.lineno