sarj-python-lint 0.87.1__tar.gz → 0.87.3__tar.gz

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/PKG-INFO

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

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/pyproject.toml

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

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rule_base.py

@@ -155,6 +155,7 @@ class RuleDocumentation:
     rationale: str
     remediation: str
     category: RuleCategory
+    default_level: Severity = Severity.ERROR
     autofix: AutofixPolicy = AutofixPolicy.NONE
     aliases: tuple[str, ...] = ()
     limitations: tuple[str, ...] = ()
@@ -207,6 +208,7 @@ class NativeRuleSpec:
     rationale: str
     remediation: str
     category: RuleCategory
+    default_level: Severity
     autofix: AutofixPolicy
     aliases: tuple[str, ...]
     limitations: tuple[str, ...]
@@ -283,6 +285,7 @@ class Rule(ABC):
             rationale=authored.rationale,
             remediation=authored.remediation,
             category=authored.category,
+            default_level=authored.default_level,
             autofix=authored.autofix,
             aliases=authored.aliases,
             limitations=authored.limitations,

sarj_python_lint-0.87.3/src/sarj_python_lint/rules/_ast_position.py

@@ -0,0 +1,21 @@
+from __future__ import annotations
+
+from dataclasses import dataclass
+from typing import TYPE_CHECKING
+
+
+if TYPE_CHECKING:
+    import ast
+
+
+@dataclass(frozen=True, order=True, slots=True)
+class AstPosition:
+    line: int
+    column: int
+
+
+def ast_position(node: ast.AST, *, missing: int) -> AstPosition:
+    return AstPosition(
+        line=getattr(node, "lineno", missing),
+        column=getattr(node, "col_offset", missing),
+    )

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/complex_postgres_query_requires_architecture_review.py

@@ -264,6 +264,7 @@ class ComplexPostgresQueryRequiresArchitectureReview(Rule):
     id: str = "complex-postgres-query-requires-architecture-review"
     code: str = "SARJ437"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Complex executable PostgreSQL query shapes require architecture review.",
         rationale=(
             "Join-heavy, deeply staged, or wide joined reads can obscure cardinality, bounds, ordering, and locking "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/fakes_in_shared_location.py

@@ -50,6 +50,7 @@ class FakesInSharedLocation(Rule):
     id = "fakes-in-shared-location"
     code = "SARJ428"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Review named top-level test doubles for shared-support ownership unless they are intentionally scenario-local.",
         rationale=(
             "Reusable doubles hidden in an individual test module are difficult to discover and are often recreated. "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/fastapi_explicit_openapi_contract.py

@@ -73,6 +73,7 @@ class FastapiExplicitOpenapiContract(Rule):
     id: str = "fastapi-explicit-openapi-contract"
     code: str = "SARJ094"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Visible FastAPI operations pin locally reviewable metadata and avoid statically provable OpenAPI gaps.",
         rationale=(
             "FastAPI can infer valid schemas and default statuses. This stricter organizational policy pins locally "
@@ -674,12 +675,12 @@ def _check_direct_responses(
     routes: tuple[Route, ...],
     index: FastapiIndex,
 ) -> list[_Finding]:
-    statuses, invalid = _direct_response_statuses(function, index)
+    direct = _direct_response_statuses(function, index)
     findings = [
         _Finding(node, f"[status] {status} is outside the HTTP status range 100..599.")
-        for status, node in invalid.items()
+        for status, node in direct.invalid.items()
     ]
-    if not statuses:
+    if not direct.statuses:
         return findings
     for route in _operations(routes):
         responses = _route_response_nodes(route)
@@ -689,7 +690,7 @@ def _check_direct_responses(
             status for node in responses if isinstance(node, ast.Dict) for status in _response_codes(node, index)
         }
         primary = _status_code(route.keywords.get("status_code"), index)
-        missing = sorted(statuses - documented - ({primary} if primary is not None else set()))
+        missing = sorted(direct.statuses - documented - ({primary} if primary is not None else set()))
         if missing:
             findings.append(
                 _Finding(
@@ -700,9 +701,15 @@ def _check_direct_responses(
     return findings
 
 
+@dataclass(frozen=True, slots=True)
+class _DirectResponseStatuses:
+    statuses: set[int]
+    invalid: dict[int, ast.Call]
+
+
 def _direct_response_statuses(
     function: ast.FunctionDef | ast.AsyncFunctionDef, index: FastapiIndex
-) -> tuple[set[int], dict[int, ast.Call]]:
+) -> _DirectResponseStatuses:
     statuses: set[int] = set()
     invalid: dict[int, ast.Call] = {}
     stack: list[tuple[ast.AST, bool]] = [(node, False) for node in function.body]
@@ -721,7 +728,7 @@ def _direct_response_statuses(
             else:
                 invalid.setdefault(status, call)
         stack.extend((child, catches_http_exception) for child in children(current))
-    return statuses, invalid
+    return _DirectResponseStatuses(statuses=statuses, invalid=invalid)
 
 
 def _extend_response_try(

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/iac_source_coupled_test.py

@@ -32,6 +32,7 @@ class IacSourceCoupledTest(Rule):
     id = "iac-source-coupled-test"
     code = "SARJ412"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Test uses raw Terraform/HCL text as an infrastructure-behavior oracle.",
         rationale=(
             "Substring and regex checks can pass on comments, formatting, or unreachable Terraform configuration "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/mock_without_spec.py

@@ -121,6 +121,7 @@ class MockWithoutSpec(Rule):
     id: str = "mock-without-spec"
     code: str = "SARJ040"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Unrestricted mock permits attributes outside the collaborator contract.",
         rationale=(
             "A mock without a spec permits removed or misspelled attributes; a callable mock without autospec may "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/named_record_at_boundaries.py

@@ -13,6 +13,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     is_suppressed,
     parse_or_none,
 )
@@ -42,6 +43,7 @@ class NamedRecordAtBoundaries(Rule):
     id: str = "named-record-at-boundaries"
     code: str = "SARJ008"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Public Python API returns an unnamed fixed-shape record.",
         rationale="A named record makes field types and required keys explicit to callers and static tooling.",
         remediation="Define and return a `TypedDict`, Pydantic model, or frozen dataclass for the fixed record shape.",

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/negative_only_http_status_assertion.py

@@ -204,6 +204,7 @@ class NegativeOnlyHttpStatusAssertion(Rule):
     id = "negative-only-http-status-assertion"
     code = "SARJ408"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="HTTP test assertion only excludes a server error instead of identifying the intended response.",
         rationale=(
             "Authentication, routing, validation, and domain failures can all replace the intended response while "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_analytical_aggregation_in_postgres_store.py

@@ -152,6 +152,7 @@ class NoAnalyticalAggregationInPostgresStore(Rule):
     id: str = "no-analytical-aggregation-in-postgres-store"
     code: str = "SARJ020"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Potentially analytical PostgreSQL store queries require review.",
         rationale=(
             "Unbounded reporting and statistical scans can compete with transactional reads. "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_any_mapping_types.py

@@ -13,6 +13,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     is_suppressed,
 )
 from sarj_python_lint.rules._annotation_semantics import AnnotationSemantics, scope_bound_names
@@ -37,6 +38,7 @@ class NoAnyMappingTypes(Rule):
     id = "no-any-mapping-types"
     code = "SARJ447"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="String-keyed mapping types must not erase their values with `Any`.",
         rationale=(
             "An `Any`-valued mapping disables checking transitively and conceals whether the value is a fixed record, "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_positional_psycopg_row_escape.py

@@ -20,6 +20,7 @@ from sarj_python_lint.rule_base import (
     is_suppressed,
     parse_or_none,
 )
+from sarj_python_lint.rules._ast_position import AstPosition, ast_position
 from sarj_python_lint.rules._imports import ImportIndex
 from sarj_python_lint.rules._paths import is_generated, is_test_path, is_test_support_path
 
@@ -954,8 +955,8 @@ def _target_names(node: ast.expr) -> set[str]:
     return set()
 
 
-def _position(node: ast.AST) -> tuple[int, int]:
-    return getattr(node, "lineno", -1), getattr(node, "col_offset", -1)
+def _position(node: ast.AST) -> AstPosition:
+    return ast_position(node, missing=-1)
 
 
 def _scope_nodes(statements: Iterable[ast.stmt], *, stop_rebinding: str | None = None) -> Iterator[ast.AST]:

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_random_uuid_in_sql.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import ast
+from dataclasses import dataclass
 from pathlib import PurePosixPath
 import re
 from typing import TYPE_CHECKING, ClassVar, final, override
@@ -26,6 +27,13 @@ if TYPE_CHECKING:
 
 _RANDOM_UUID_RE = re.compile(r"\b(?:gen_random_uuid|uuid_generate_v4)\s*\(", re.IGNORECASE)
 
+
+@dataclass(frozen=True, slots=True)
+class _IdentifierEnd:
+    end: int
+    after_whitespace: int
+
+
 # Require SQL structure so prose that merely names the function stays valid.
 _SQL_SHAPE_RE = re.compile(
     r"\b(?:CREATE|ALTER|INSERT|UPDATE|SELECT|DEFAULT|VALUES)\b",
@@ -132,12 +140,12 @@ def _inside_uuidv7_builder(sql: str, position: int) -> bool:
     index = 0
     while index < position:
         if sql[index].isalpha() or sql[index] == "_":
-            end, after = _identifier_end(sql, index, position)
-            if after < position and sql[after] == "(":
-                calls.append(sql[index:end].lower())
-                index = after + 1
+            identifier = _identifier_end(sql, index, position)
+            if identifier.after_whitespace < position and sql[identifier.after_whitespace] == "(":
+                calls.append(sql[index : identifier.end].lower())
+                index = identifier.after_whitespace + 1
                 continue
-            index = end
+            index = identifier.end
             continue
         if sql[index] == "(":
             calls.append(None)
@@ -147,11 +155,11 @@ def _inside_uuidv7_builder(sql: str, position: int) -> bool:
     return any(call in _UUIDV7_BUILDERS for call in calls)
 
 
-def _identifier_end(sql: str, index: int, position: int) -> tuple[int, int]:
+def _identifier_end(sql: str, index: int, position: int) -> _IdentifierEnd:
     end = index + 1
     while end < position and (sql[end].isalnum() or sql[end] == "_"):
         end += 1
     after = end
     while after < position and sql[after].isspace():
         after += 1
-    return end, after
+    return _IdentifierEnd(end=end, after_whitespace=after)

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_raw_connection_in_tests.py

@@ -13,6 +13,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     parse_or_none,
 )
 from sarj_python_lint.rules._paths import is_generated, is_test_path, is_test_support_path
@@ -30,6 +31,7 @@ class NoRawConnectionInTests(Rule):
     id = "no-raw-connection-in-tests"
     code = "SARJ429"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Do not acquire raw database connections in tests.",
         rationale=(
             "Tests that reach through a pool couple assertions and setup to persistence internals, bypass the "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_raw_source_text_test_oracle.py

@@ -112,6 +112,7 @@ class NoRawSourceTextTestOracle(Rule):
     id = "no-raw-source-text-test-oracle"
     code = "SARJ402"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Test uses raw text from a source-like project path as its oracle.",
         rationale="Substring and regex checks can pass on comments or unreachable configuration and fail after behavior-preserving formatting changes.",
         remediation=(

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_redundant_literal_description.py

@@ -44,6 +44,7 @@ class NoRedundantLiteralDescription(Rule):
     id = "no-restated-closed-domain-description"
     code = "SARJ423"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Do not restate a string Literal or local string Enum domain in its Pydantic description.",
         rationale=(
             "Pydantic already publishes literal and enum domains in JSON Schema; duplicate must-be prose can "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_redundant_module_alias_exports.py

@@ -14,6 +14,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     parse_or_none,
 )
 from sarj_python_lint.rules._imports import ImportIndex
@@ -33,6 +34,7 @@ class NoRedundantModuleAliasExports(Rule):
     id: str = "no-redundant-module-alias-exports"
     code: str = "SARJ440"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Do not manufacture public APIs from private names or replace the current module.",
         rationale=(
             "Binding a public name to a private implementation creates two names for one API and disguises the "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_restated_comment.py

@@ -253,6 +253,7 @@ class NoRestatedComment(Rule):
     id: str = "no-restated-comment"
     code: str = "SARJ049"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Short standalone comment lexically restates the immediately following simple action.",
         rationale=(
             "When a comment adds no rationale, scope, ordering, or constraint, it duplicates code and can become stale. "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_typed_doc_sections.py

@@ -28,6 +28,7 @@ class NoTypedDocSections(Rule):
     id = "no-docstring-type-restatement"
     code = "SARJ092"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="A docstring type label repeats an annotation from the fully typed signature.",
         rationale=(
             "A repeated type spelling can drift from the annotation and obscures the behavioral contract that only prose "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_unused_value_marker.py

@@ -29,6 +29,7 @@ class NoUnusedValueMarker(Rule):
     id = "no-unused-value-marker"
     code = "SARJ452"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Do not use standalone assignments to `_` to discard values.",
         rationale=(
             "A standalone `_ = value` assignment adds a binding without clarifying intent. A side-effecting call can "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/no_whole_request_response_payload_in_log.py

@@ -114,6 +114,7 @@ class NoWholeRequestResponsePayloadInLog(Rule):
     id: str = "no-whole-request-response-payload-in-log"
     code: str = "SARJ436"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Whole request or response payloads passed to logging calls require review.",
         rationale=(
             "Request and response bodies often contain personal, financial, or echoed authentication data, while "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/opaque_parametrize_case_needs_id.py

@@ -56,6 +56,7 @@ class OpaqueParametrizeCaseNeedsId(Rule):
     id: str = "opaque-parametrize-case-needs-id"
     code: str = "SARJ042"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Opaque static pytest parameter cases rely on argument-name-and-index fallback IDs.",
         rationale="Fallback IDs such as `payload0` are hard to diagnose and silently change when the table is reordered.",
         remediation=(

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/over_mocked_test.py

@@ -179,6 +179,7 @@ class OverMockedTest(Rule):
     id: str = "over-mocked-test"
     code: str = "SARJ062"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Tests should not use more than five independently rooted test doubles or collaborator substitutions.",
         rationale="Broad double setup obscures the behavior under test and often couples tests to implementation wiring.",
         remediation=(

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_collection_comprehension.py

@@ -18,6 +18,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     is_suppressed,
     parse_or_none,
 )
@@ -77,6 +78,7 @@ class PreferCollectionComprehension(Rule):
     id = "prefer-collection-comprehension"
     code = "SARJ430"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Single-purpose fresh collection builder loop — prefer a direct comprehension.",
         rationale=(
             "An empty collection followed by a loop whose only behavior is one projection or filtered insertion "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_constant_time_secret_compare.py

@@ -94,6 +94,7 @@ class PreferConstantTimeSecretCompare(Rule):
     id: str = "prefer-constant-time-secret-compare"
     code: str = "SARJ011"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Externally supplied authenticators are compared with timing-sensitive equality.",
         rationale=(
             "Ordinary equality may short-circuit based on matching content when attacker-controlled credentials are "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_fstring_over_concat.py

@@ -120,6 +120,7 @@ class PreferFstringOverConcat(Rule):
     id: str = "prefer-fstring-over-concat"
     code: str = "SARJ068"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Prefer f-strings for short human-readable interpolation when they make the result clearer.",
         rationale="For prose-like strings, f-strings keep interpolated values and surrounding text visible together.",
         remediation=(

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_immutable_module_constant.py

@@ -73,6 +73,7 @@ class PreferImmutableModuleConstant(Rule):
     id: str = "prefer-immutable-module-constant"
     code: str = "SARJ096"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary=("Nonempty uppercase module collections allow top-level membership or keys to change at runtime."),
         rationale=(
             "A constant-looking collection can expose process-wide top-level mutation even when callers intend it as a "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_injected_dependency_over_monkeypatch.py

@@ -16,6 +16,7 @@ from sarj_python_lint.rule_base import (
     Severity,
     parse_or_none,
 )
+from sarj_python_lint.rules._ast_position import AstPosition, ast_position
 from sarj_python_lint.rules._imports import ImportIndex
 from sarj_python_lint.rules._paths import is_generated, is_test_path
 
@@ -36,6 +37,7 @@ class PreferInjectedDependencyOverMonkeypatch(Rule):
     id: str = "prefer-injected-dependency-over-monkeypatch"
     code: str = "SARJ445"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Tests should inject dependencies instead of replacing attributes through ambient patching.",
         rationale=(
             "Attribute patching hides collaborators and configuration behind ambient module or object state, coupling "
@@ -250,10 +252,10 @@ def _parameter_rebound_before(
     return any(handle in _bound_targets(node) and _position(node) < use_position for node in nodes)
 
 
-def _position(node: ast.AST) -> tuple[int, int]:
+def _position(node: ast.AST) -> AstPosition:
     if isinstance(node, ast.withitem):
         return _position(node.context_expr)
-    return (getattr(node, "lineno", 0), getattr(node, "col_offset", 0))
+    return ast_position(node, missing=0)
 
 
 def _direct_aliases(nodes: tuple[ast.AST, ...], handles: set[str], parameters: set[str]) -> set[str]:

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_library_fake.py

@@ -341,6 +341,7 @@ class PreferLibraryFake(Rule):
     id: str = "prefer-library-fake"
     code: str = "SARJ059"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Prefer a maintained fake, emulator, recorder, or test service for substantial third-party protocols.",
         rationale=(
             "A substantial hand-written protocol model can drift from supported request, state, and error behavior."

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_match_exception_dispatch.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import ast
+from dataclasses import dataclass
 from pathlib import PurePosixPath
 from typing import TYPE_CHECKING, ClassVar, NamedTuple, final, override
 
@@ -85,6 +86,7 @@ class PreferMatchExceptionDispatch(Rule):
     id: str = "prefer-match-exception-dispatch"
     code: str = "SARJ453"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Prefer guarded match/case for refined exception type dispatch.",
         rationale=(
             "A long exception classifier that repeats isinstance checks and then refines one exception type hides "
@@ -215,17 +217,23 @@ def _classifier_starts(
     for statements in _nested_statement_lists(body):
         index = 0
         while index < len(statements):
-            candidates, cursor = _adjacent_bare_ifs(statements, index)
-            branches = _validated_branches(candidates, subject, imports)
-            if branches is not None and _is_classifier(branches, statements, cursor):
+            adjacent = _adjacent_bare_ifs(statements, index)
+            branches = _validated_branches(adjacent.candidates, subject, imports)
+            if branches is not None and _is_classifier(branches, statements, adjacent.cursor):
                 findings.append(branches[0][0])
-                index = cursor + 1
+                index = adjacent.cursor + 1
             else:
-                index = max(cursor, index + 1)
+                index = max(adjacent.cursor, index + 1)
     return tuple(findings)
 
 
-def _adjacent_bare_ifs(statements: list[ast.stmt], start: int) -> tuple[list[ast.If], int]:
+@dataclass(frozen=True, slots=True)
+class _AdjacentBareIfs:
+    candidates: list[ast.If]
+    cursor: int
+
+
+def _adjacent_bare_ifs(statements: list[ast.stmt], start: int) -> _AdjacentBareIfs:
     cursor = start
     candidates: list[ast.If] = []
     while cursor < len(statements):
@@ -234,7 +242,7 @@ def _adjacent_bare_ifs(statements: list[ast.stmt], start: int) -> tuple[list[ast
             break
         candidates.append(candidate)
         cursor += 1
-    return candidates, cursor
+    return _AdjacentBareIfs(candidates=candidates, cursor=cursor)
 
 
 def _validated_branches(

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_match_value_dispatch.py

@@ -2,6 +2,7 @@ from __future__ import annotations
 
 import ast
 from collections import Counter
+from dataclasses import dataclass
 from pathlib import PurePosixPath
 from typing import TYPE_CHECKING, ClassVar, final, override
 
@@ -37,6 +38,7 @@ class PreferMatchValueDispatch(Rule):
     id: str = "prefer-match-value-dispatch"
     code: str = "SARJ439"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Prefer match/case for repeated dispatch on one value with a fallback.",
         rationale="A single dispatch subject makes distinct value cases and their fallback easier to review.",
         remediation=(
@@ -164,12 +166,12 @@ class PreferMatchValueDispatch(Rule):
         for node in ast.walk(tree):
             if not isinstance(node, ast.If) or id(node) in continuations:
                 continue
-            branches, current = _elif_chain(node, lines, continuations)
-            if len(branches) < _MIN_TESTS or not current.orelse:
+            chain = _elif_chain(node, lines, continuations)
+            if len(chain.branches) < _MIN_TESTS or not chain.last.orelse:
                 continue
-            subject = _dispatch_subject(branches, constants)
+            subject = _dispatch_subject(chain.branches, constants)
             if subject is None:
-                subject = _expanded_dispatch_subject(branches, constants)
+                subject = _expanded_dispatch_subject(chain.branches, constants)
             if subject is not None:
                 candidates.append((node, subject))
         for branches in _terminal_sibling_dispatches(tree):
@@ -368,7 +370,13 @@ def _subject(expression: ast.expr) -> str | None:
             return None
 
 
-def _elif_chain(node: ast.If, lines: list[str], continuations: set[int]) -> tuple[list[ast.If], ast.If]:
+@dataclass(frozen=True, slots=True)
+class _ElifChain:
+    branches: list[ast.If]
+    last: ast.If
+
+
+def _elif_chain(node: ast.If, lines: list[str], continuations: set[int]) -> _ElifChain:
     branches = [node]
     current = node
     while len(current.orelse) == 1:
@@ -378,7 +386,7 @@ def _elif_chain(node: ast.If, lines: list[str], continuations: set[int]) -> tupl
         continuations.add(id(child))
         branches.append(child)
         current = child
-    return branches, current
+    return _ElifChain(branches=branches, last=current)
 
 
 def _collect_terminal_siblings(field: list[ast.stmt], candidates: list[list[ast.If]]) -> None:

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_module_level_constant.py

@@ -96,6 +96,7 @@ class PreferModuleLevelConstant(Rule):
     id: str = "prefer-module-level-constant"
     code: str = "SARJ039"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Hoist repeatedly read static values when an immutable module representation preserves behavior.",
         rationale=(
             "Rebuilding a substantial static collection repeats allocation, while calling `re.compile` with a "

{sarj_python_lint-0.87.1 → sarj_python_lint-0.87.3}/src/sarj_python_lint/rules/prefer_monkeypatch_for_process_state_in_test.py

@@ -16,6 +16,7 @@ from sarj_python_lint.rule_base import (
     Severity,
     parse_or_none,
 )
+from sarj_python_lint.rules._ast_position import AstPosition, ast_position
 from sarj_python_lint.rules._imports import ImportIndex
 from sarj_python_lint.rules._paths import is_generated, is_test_path
 
@@ -43,6 +44,7 @@ class PreferMonkeypatchForProcessStateInTest(Rule):
     id = "prefer-monkeypatch-for-process-state-in-test"
     code = "SARJ446"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Test mutates process-wide state without a restoring test scope.",
         rationale=(
             "The working directory and Python import registries are shared by every test in the process. A direct "
@@ -294,8 +296,8 @@ def _import_root_rebound_before(
     )
 
 
-def _position(node: ast.AST) -> tuple[int, int]:
-    return (getattr(node, "lineno", 0), getattr(node, "col_offset", 0))
+def _position(node: ast.AST) -> AstPosition:
+    return ast_position(node, missing=0)
 
 
 def _restored_mutations(