sarj-python-lint 0.87.2__tar.gz → 0.87.4__tar.gz

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/PKG-INFO

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

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

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/__main__.py

@@ -14,6 +14,7 @@ import typer
 from sarj_python_lint import __version__
 from sarj_python_lint._analysis_session import AnalysisSession
 from sarj_python_lint._filesystem import atomic_write_text
+from sarj_python_lint.json_boundary import is_object_mapping, parse_json
 from sarj_python_lint.rule_base import Diagnostic, ProjectRule, Rule, Severity, is_suppressed
 from sarj_python_lint.rules import REGISTRY
 from sarj_python_lint.rules._paths import clear_path_caches
@@ -335,20 +336,14 @@ def _baseline_path(path: Path, *, root: Path | None = None) -> str:
 
 
 def _read_baseline(path: Path) -> dict[str, dict[str, int]]:
-    raw: object = json.loads(  # pyright: ignore[reportAny] — json.loads is an untyped stdlib boundary; the shape is narrowed below
-        path.read_text(encoding="utf-8")
-    )
-    if not isinstance(raw, dict):
+    raw = parse_json(path.read_text(encoding="utf-8"))
+    if not is_object_mapping(raw):
         return {}
     counts: dict[str, dict[str, int]] = {}
-    for file_key, per_code in raw.items():  # pyright: ignore[reportUnknownVariableType] — json.loads yields Any leaves
-        if not isinstance(file_key, str) or not isinstance(per_code, dict):
+    for file_key, per_code in raw.items():
+        if not isinstance(file_key, str) or not is_object_mapping(per_code):
             continue
-        counts[file_key] = {
-            code: n
-            for code, n in per_code.items()  # pyright: ignore[reportUnknownVariableType] — same
-            if isinstance(code, str) and isinstance(n, int)
-        }
+        counts[file_key] = {code: n for code, n in per_code.items() if isinstance(code, str) and isinstance(n, int)}
     return counts
 
 

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/_ratchet_cli.py

@@ -1,7 +1,6 @@
 from __future__ import annotations
 
 from dataclasses import dataclass
-import json
 from pathlib import Path
 import subprocess  # ruff: ignore[suspicious-subprocess-import] -- fixed local Ruff introspection only
 import sys
@@ -11,6 +10,7 @@ import typer
 
 from sarj_python_lint._filesystem import atomic_write_text
 from sarj_python_lint._version import __version__
+from sarj_python_lint.json_boundary import parse_json
 from sarj_python_lint.ratchet import (
     DEFAULT_PER_FILE_CEILING,
     Baseline,
@@ -142,7 +142,7 @@ def _ruff_selector_aliases() -> dict[str, str]:
         encoding="utf-8",
         text=True,
     )
-    raw: object = json.loads(completed.stdout)  # pyright: ignore[reportAny] -- validated below
+    raw = parse_json(completed.stdout)
     if not _is_object_list(raw):
         msg = "Ruff rule catalog was not a JSON list"
         raise TypeError(msg)

sarj_python_lint-0.87.4/src/sarj_python_lint/json_boundary.py

@@ -0,0 +1,10 @@
+import json
+from typing import TypeIs
+
+
+def parse_json(text: str) -> object:
+    return json.loads(text)  # pyright: ignore[reportAny] -- untyped stdlib parser boundary.
+
+
+def is_object_mapping(value: object) -> TypeIs[dict[object, object]]:
+    return isinstance(value, dict)

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/ratchet.py

@@ -9,6 +9,8 @@ import tokenize
 from types import MappingProxyType
 from typing import TYPE_CHECKING, Final, NamedTuple, TypeGuard
 
+from sarj_python_lint.json_boundary import is_object_mapping, parse_json
+
 
 if TYPE_CHECKING:
     from collections.abc import Iterable, Iterator, Mapping
@@ -216,9 +218,7 @@ def seed(measurement: Measurement, baseline: Baseline) -> Baseline:
 
 
 def load_baseline(path: Path) -> Baseline:
-    raw: object = json.loads(  # pyright: ignore[reportAny] — json.loads is an untyped stdlib boundary; every read below narrows
-        path.read_text(encoding="utf-8")
-    )
+    raw = parse_json(path.read_text(encoding="utf-8"))
     schema = _get(raw, "schema_version")
     if schema is not None and schema != BASELINE_SCHEMA_VERSION:
         msg = f"unsupported suppression baseline schema_version: {schema!r}"
@@ -238,9 +238,9 @@ def load_baseline(path: Path) -> Baseline:
 
 
 def _get(mapping: object, key: str) -> object:
-    if not isinstance(mapping, dict):
+    if not is_object_mapping(mapping):
         return None
-    return mapping.get(key)  # pyright: ignore[reportUnknownMemberType, reportUnknownVariableType] — json leaves are Any
+    return mapping.get(key)
 
 
 def dump_baseline(baseline: Baseline, packages: Iterable[str]) -> str:
@@ -308,10 +308,7 @@ def _is_object_list(value: object) -> TypeGuard[list[object]]:
 
 
 def _is_string_object_mapping(value: object) -> TypeGuard[dict[str, object]]:
-    return isinstance(value, dict) and all(
-        isinstance(key, str)
-        for key in value  # pyright: ignore[reportUnknownVariableType] — runtime validation narrows untyped JSON keys.
-    )
+    return is_object_mapping(value) and all(isinstance(key, str) for key in value)
 
 
 def _normalized_subtree(value: str) -> str:

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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 "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_injected_dependency_over_monkeypatch.py

@@ -37,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 "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_match_exception_dispatch.py

@@ -86,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 "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_match_value_dispatch.py

@@ -38,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=(

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/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.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_monkeypatch_for_process_state_in_test.py

@@ -44,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 "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_non_nullable_collection.py

@@ -40,6 +40,7 @@ class PreferNonNullableCollection(Rule):
     id: str = "prefer-non-nullable-collection"
     code: str = "SARJ082"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Avoid nullable list parameters that are immediately collapsed to an empty list.",
         rationale=(
             "When an implementation immediately replaces both `None` and an empty list with the same fresh empty list, "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_or_pattern.py

@@ -39,6 +39,7 @@ class PreferOrPattern(Rule):
     id: str = "prefer-or-pattern"
     code: str = "SARJ070"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Merge adjacent `case` arms with identical bodies into one or-pattern.",
         rationale="An or-pattern expresses shared handling once and prevents identical arms from drifting apart.",
         remediation="Join the equivalent patterns with `|` and keep their shared body under the merged arm.",

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_pydantic_json_value.py

@@ -19,6 +19,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     is_suppressed,
     parse_or_none,
 )
@@ -52,6 +53,7 @@ class PreferPydanticJsonValue(Rule):
     id = "prefer-pydantic-json-value"
     code = "SARJ454"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Recursive JSON value alias duplicates `pydantic.JsonValue`.",
         rationale=(
             "Pydantic's canonical JSON value alias keeps the static contract aligned with runtime validation and "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_self_type_annotation.py

@@ -114,6 +114,7 @@ class PreferSelfTypeAnnotation(Rule):
     id: str = "prefer-self-type-annotation"
     code: str = "SARJ078"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Prefer `Self` for self-returning methods and alternate constructors.",
         rationale="`Self` preserves the concrete subclass type when an inherited method returns its receiver or constructs through its class receiver.",
         remediation="Import `Self` from `typing` (or `typing_extensions` on Python before 3.11) and use it as the return annotation.",

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_set_isdisjoint.py

@@ -13,6 +13,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     is_suppressed,
     parse_or_none,
 )
@@ -46,6 +47,7 @@ class PreferSetIsdisjoint(Rule):
     id = "prefer-set-isdisjoint"
     code = "SARJ431"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Prefer `set.isdisjoint` when a built-in set intersection is used only as a boolean predicate.",
         rationale="`isdisjoint` names the overlap predicate directly and avoids allocating an intersection that is immediately discarded.",
         remediation="Use `left.isdisjoint(right)` and negate it when the condition requires overlap.",

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_walrus_awaited_none_guard.py

@@ -42,6 +42,7 @@ class PreferWalrusAwaitedNoneGuard(Rule):
     id = "prefer-walrus-awaited-none-guard"
     code = "SARJ432"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Bind a compact awaited lookup in its immediately following terminal None guard.",
         rationale=(
             "When an awaited lookup and its terminal None guard are adjacent, binding in the condition keeps "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_walrus_comprehension_filter.py

@@ -13,6 +13,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     is_suppressed,
     parse_or_none,
 )
@@ -199,6 +200,7 @@ class PreferWalrusComprehensionFilter(Rule):
     id: str = "prefer-walrus-comprehension-filter"
     code: str = "SARJ076"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="The same call runs in a comprehension filter and its result.",
         rationale="When both evaluations are intended to produce one stable value, repeating the call wastes work and obscures that relationship.",
         remediation=(

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_walrus_regex_match.py

@@ -14,6 +14,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     is_suppressed,
     parse_or_none,
 )
@@ -58,6 +59,7 @@ class PreferWalrusRegexMatch(Rule):
     id: str = "prefer-walrus-regex-match"
     code: str = "SARJ081"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="A proven regex Match-or-None result is assigned only for the following condition.",
         rationale="A named expression can keep a short regex operation with its only condition while preserving access to the result.",
         remediation=(

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/prefer_walrus_stream_loop.py

@@ -14,6 +14,7 @@ from sarj_python_lint.rule_base import (
     RuleCategory,
     RuleDocumentation,
     RuleExample,
+    Severity,
     is_suppressed,
     parse_or_none,
 )
@@ -30,6 +31,7 @@ class PreferWalrusStreamLoop(Rule):
     id: str = "prefer-walrus-stream-loop"
     code: str = "SARJ077"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Collapse a compact producer assignment and immediate sentinel break into a named-expression loop.",
         rationale="A named-expression loop can state a repeated producer call and its termination condition together.",
         remediation=(

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/pytest_fixture_returns_bare_tuple.py

@@ -45,6 +45,7 @@ class PytestFixtureReturnsBareTuple(Rule):
     id: str = "pytest-fixture-returns-bare-tuple"
     code: str = "SARJ044"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Pytest fixture exposes a fixed positional record as an unnamed tuple.",
         rationale=(
             "Tuple-shaped fixture APIs encode each value's role in its position, so call sites are opaque and "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/redundant_docstring.py

@@ -46,6 +46,7 @@ class RedundantDocstring(Rule):
     id: str = "redundant-docstring"
     code: str = "SARJ050"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Function or plain-method docstring only repeats its declaration.",
         rationale="Restating a clear name and signature creates maintenance work without helping callers.",
         remediation=(

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/repeated_kwarg_heavy_call_in_test.py

@@ -61,6 +61,7 @@ class RepeatedKwargHeavyCallInTest(Rule):
     id: str = "repeated-kwarg-heavy-call-in-test"
     code: str = "SARJ045"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Tests repeat at least seven explicit keyword names across calls to the same callee.",
         rationale=(
             "Large repeated argument lists duplicate incidental setup, bury scenario differences, and make signature "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/repeated_static_call_cases.py

@@ -80,6 +80,7 @@ class RepeatedStaticCallCases(Rule):
     id = "repeated-static-call-cases"
     code = "SARJ413"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Three same-shape literal call assertions may be independent parameter cases.",
         rationale=(
             "When the calls are independent, named parameters isolate failures and identify the input that failed. "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/require_keyword_only_swap_prone_params.py

@@ -118,6 +118,7 @@ class RequireKeywordOnlySwapProneParams(Rule):
     id: str = "require-keyword-only-swap-prone-params"
     code: str = "SARJ034"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Risky-name positional parameters sharing a primitive annotation may be confused.",
         rationale="A caller can exchange semantically distinct positional values without a type-checking failure.",
         remediation=(

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/require_port_for_service.py

@@ -194,6 +194,7 @@ class RequirePortForService(ProjectRule):
     id: str = "require-port-for-service"
     code: str = "SARJ071"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Consider a consumer-owned port when visible service structure suggests a substitution boundary.",
         rationale="A small port can decouple consumers when they genuinely need to substitute a concrete service boundary.",
         remediation=(

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/require_pydantic_for_external_json.py

@@ -97,6 +97,7 @@ class RequirePydanticForExternalJson(Rule):
     id = "require-pydantic-for-external-json"
     code = "SARJ411"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Proven external JSON record fields are consumed before runtime schema validation.",
         rationale=(
             "Annotations, casts, and partial key checks do not validate a wire protocol; a maintained runtime "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/require_pydantic_ordinal_lower_bound.py

@@ -95,6 +95,7 @@ class RequirePydanticOrdinalLowerBound(Rule):
     id = "require-pydantic-ordinal-lower-bound"
     code = "SARJ418"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="A Pydantic ordinal field maps its first position to N but accepts smaller integers.",
         rationale=(
             "An unqualified `N for the first ...` mapping defines the origin of an ordinal field; matching schema "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/store_get_delegates_to_bulk_read.py

@@ -95,6 +95,7 @@ class StoreGetDelegatesToBulkRead(Rule):
     id = "store-get-delegates-to-bulk-read"
     code = "SARJ421"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="A store singleton operation should reuse its equivalent bulk implementation.",
         rationale=(
             "Independent singleton and bulk queries can drift in filtering, mutation semantics, row conversion, "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/store_insert_requires_on_conflict.py

@@ -65,6 +65,7 @@ class StoreInsertRequiresOnConflict(Rule):
     id: str = "replay-contract-insert-requires-duplicate-policy"
     code: str = "SARJ018"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="A literal INSERT in a replay-named store callable must declare duplicate behavior.",
         rationale=(
             "A store callable named ensure, enqueue, record_once, get_or_create, create_if_absent, or "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/timestamp_order_requires_tiebreaker.py

@@ -56,6 +56,7 @@ class TimestampOrderRequiresTiebreaker(Rule):
     id = "timestamp-order-requires-tiebreaker"
     code = "SARJ407"
     documentation = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary=(
             "Bounded store SQL whose final result-order key looks like a `*_at` timestamp should include a "
             "deterministic secondary key."

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/typed_error_reasons.py

@@ -32,6 +32,7 @@ class TypedErrorReasons(Rule):
     id = "typed-error-reasons"
     code = "SARJ435"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="Review joined exception strings for fixed reason identities versus dynamic context.",
         rationale=(
             "Fixed reason identities benefit from stable codes for API clients, UI formatting, telemetry, and "

{sarj_python_lint-0.87.2 → sarj_python_lint-0.87.4}/src/sarj_python_lint/rules/unused_test_factory_option.py

@@ -35,6 +35,7 @@ class UnusedTestFactoryOption(Rule):
     id = "unused-test-factory-option"
     code = "SARJ443"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
+        default_level=Severity.WARNING,
         summary="A private test factory exposes a literal option that visible callers never vary.",
         rationale="Unused customization obscures the values that actually distinguish test scenarios.",
         remediation="Keep the value in the factory's construction instead of exposing an unused option; retain it if external callers need it.",