sarj-python-lint 0.84.0__tar.gz → 0.84.2__tar.gz

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/PKG-INFO

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

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

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/__main__.py

@@ -209,7 +209,7 @@ def _suppressed_diagnostics(
 
 
 def _suppressed_codes(codes: dict[str, set[Severity]]) -> set[tuple[str, Severity]]:
-    return {
+    suppressed = {
         (generic, generic_severity)
         for specific, generics in _DIAGNOSTIC_PRECEDENCE.items()
         if specific in codes
@@ -217,6 +217,13 @@ def _suppressed_codes(codes: dict[str, set[Severity]]) -> set[tuple[str, Severit
         for generic_severity in codes.get(generic, set())
         if generic_severity is Severity.WARNING or Severity.ERROR in codes[specific]
     }
+    # The promoted signature-restatement rules now block. Keep their errors and
+    # drop only the lower-severity typed-docstring twin at the same owner.
+    if Severity.WARNING in codes.get("SARJ092", set()) and any(
+        Severity.ERROR in codes.get(generic, set()) for generic in ("SARJ086", "SARJ087")
+    ):
+        suppressed.add(("SARJ092", Severity.WARNING))
+    return suppressed
 
 
 def _function_signature_owner_locations(source: str) -> dict[int, _OwnerLocation]:

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/defect_xfail_requires_explicit_strict.py

@@ -129,7 +129,7 @@ class DefectXfailRequiresExplicitStrict(Rule):
                     "fixed its XPASS may not fail the suite. Add literal `strict=True`; if only some "
                     "parameter cases fail, move the marker to those `pytest.param` cases first."
                 ),
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
             )
             for node in _rotting_bug_pins(tree, imports)
         ]

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/docstring_args_restate_signature.py

@@ -181,7 +181,7 @@ class DocstringArgsRestateSignature(Rule):
                     f"`{node.name}` has an Args section whose entries only repeat matching parameter names or types; "
                     "remove that section or document a constraint not evident from the signature."
                 ),
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
             )
         )
 

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/docstring_returns_restate_signature.py

@@ -252,7 +252,7 @@ class DocstringReturnsRestateSignature(Rule):
                     f"`{node.name}` {section.name} section only repeats its callable name or annotated result type; "
                     "remove that section or document result semantics not expressed by the signature."
                 ),
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
             )
         )
 

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/no_copied_inherited_docstring.py

@@ -158,7 +158,7 @@ class NoCopiedInheritedDocstring(Rule):
                         f"Docstring has the same cleaned content as {parent.name}.{name}'s. Remove it only if no "
                         "consumer requires the override's direct `__doc__`; otherwise retain it with a narrow rationale."
                     ),
-                    severity=Severity.WARNING,
+                    severity=Severity.ERROR,
                 )
             )
 

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/no_offset_pagination.py

@@ -128,7 +128,7 @@ class NoOffsetPagination(Rule):
                     line=node.lineno,
                     col=node.col_offset + 1,
                     code=self.code,
-                    severity=Severity.WARNING,
+                    severity=Severity.ERROR,
                     message=(
                         "Dynamic OFFSET pagination may scan skipped rows and drift under writes; use a stable unique "
                         "keyset cursor, or document why bounded/random page access requires OFFSET."

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/no_positional_psycopg_row_escape.py

@@ -157,7 +157,7 @@ class NoPositionalPsycopgRowEscape(Rule):
                 line=cursor.call.lineno,
                 col=cursor.call.col_offset + 1,
                 code=self.code,
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
                 message=(
                     "this positional Psycopg record escapes unchanged; return a named `class_row(Model)` "
                     "record, use `scalar_row` for a scalar, or transform the tuple locally"

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/no_psycopg_execution_outside_injected_owner.py

@@ -148,7 +148,7 @@ class NoPsycopgExecutionOutsideInjectedOwner(Rule):
                 line=call.lineno,
                 col=call.col_offset + 1,
                 code=self.code,
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
                 message=(
                     "Psycopg execution is outside a constructor-injected persistence owner; route it through the "
                     "owning store, repository, or transaction API"

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/no_repeated_string_literal.py

@@ -187,7 +187,7 @@ class NoRepeatedStringLiteral(Rule):
                             f"structured string literal {_preview(value)} duplicates module constant "
                             f"`{constant_name}` — reuse the canonical constant so the copies cannot drift."
                         ),
-                        severity=Severity.WARNING,
+                        severity=Severity.ERROR,
                     )
                     for node in function_nodes
                 )
@@ -209,7 +209,7 @@ class NoRepeatedStringLiteral(Rule):
                         f"callable scopes (first use at line {first.lineno}) — reuse a named binding "
                         "when they share ownership, or suppress with an independent-ownership rationale."
                     ),
-                    severity=Severity.WARNING,
+                    severity=Severity.ERROR,
                 )
                 for node in repeats
             )

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/no_repeated_test_body.py

@@ -204,7 +204,7 @@ class NoRepeatedTestBody(Rule):
                 col=group[1].node.col_offset + 1,
                 code=self.code,
                 message=_message(group),
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
             )
             for group in groups
         ]

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/no_service_behavior_in_settings.py

@@ -141,7 +141,7 @@ class NoServiceBehaviorInSettings(Rule):
                     line=node.lineno,
                     col=node.col_offset + 1,
                     code=self.code,
-                    severity=Severity.WARNING,
+                    severity=Severity.ERROR,
                     message=(
                         f"`{node.name}` is named as passive data but {methods} calls an injected collaborator — "
                         "move orchestration to a service/coordinator and keep settings or configuration data-only."

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/no_tautological_expect.py

@@ -102,7 +102,7 @@ class NoTautologicalExpect(Rule):
                 col=node.col_offset + 1,
                 code=self.code,
                 message=_message(node, reason),
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
             )
             for node, reason in _tautologies(tree, exempt)
         ]

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/no_vague_suppression_description.py

@@ -123,7 +123,7 @@ class NoVagueSuppressionDescription(Rule):
                     message=(
                         f'Suppression reason "{vague_reason}" is generic; name the concrete mismatch or safety invariant.'
                     ),
-                    severity=Severity.WARNING,
+                    severity=Severity.ERROR,
                     column_encoding=ColumnEncoding.CODEPOINTS,
                 )
             )

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/phase_label_comment.py

@@ -112,7 +112,7 @@ class TestPhaseLabelComment(Rule):
                     f"Bare test phase comment {body.strip()!r} narrates structure; delete it or replace it "
                     "with non-obvious rationale."
                 ),
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
                 column_encoding=ColumnEncoding.CODEPOINTS,
             )
             for line, col, body in standalone

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/prefer_class_row.py

@@ -123,7 +123,7 @@ class PreferClassRow(Rule):
                         line=cursor.factory.lineno,
                         col=cursor.factory.col_offset + 1,
                         code=self.code,
-                        severity=Severity.WARNING,
+                        severity=Severity.ERROR,
                         message=(
                             f"this dict row is immediately converted to `{model}`; use `class_row({model})` to "
                             "construct it during fetch"

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/prefer_match_assert_never.py

@@ -128,7 +128,7 @@ class PreferMatchAssertNever(Rule):
                             line=wildcard.pattern.lineno,
                             col=wildcard.pattern.col_offset + 1,
                             code=self.code,
-                            severity=Severity.WARNING,
+                            severity=Severity.ERROR,
                             message=(
                                 f"typed `{enum_name}` match has a no-op catch-all — an unhandled member "
                                 "is silently ignored; bind it and call `assert_never`, or raise."
@@ -146,7 +146,7 @@ class PreferMatchAssertNever(Rule):
                             line=node.lineno,
                             col=node.col_offset + 1,
                             code=self.code,
-                            severity=Severity.WARNING,
+                            severity=Severity.ERROR,
                             message=(
                                 f"typed `{enum_name}` if/elif dispatch has a no-op `else` — an unhandled "
                                 "member is silently ignored; prefer match/case with `assert_never`, or raise."

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/prefer_self_documenting_constant.py

@@ -345,7 +345,7 @@ class PreferSelfDocumentingConstant(Rule):
                                 f"`{name}` contains bare HTTP status integers; use `http.HTTPStatus` members, "
                                 "or their `.value` at an exact-integer boundary. Keep non-obvious rationale."
                             ),
-                            Severity.WARNING,
+                            Severity.ERROR,
                         )
                     )
                     continue
@@ -371,7 +371,7 @@ class PreferSelfDocumentingConstant(Rule):
                             f"`{name}` relies on its comment to identify {unit.label}; encode the unit in "
                             f"the constant name{alternative}.{compatibility} Keep non-obvious rationale."
                         ),
-                        Severity.WARNING,
+                        Severity.ERROR,
                     )
                 )
 

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/prefer_struct_over_namedtuple.py

@@ -121,7 +121,7 @@ class PreferStructOverNamedtuple(Rule):
                 "static `collections.namedtuple` fields have no type declarations — prefer a class-based "
                 "`typing.NamedTuple`; change tuple semantics only after reviewing callers"
             ),
-            severity=Severity.WARNING,
+            severity=Severity.ERROR,
         )
 
 

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/redundant_class_docstring.py

@@ -107,7 +107,7 @@ class RedundantClassDocstring(Rule):
                         col=expr.col_offset + 1,
                         code=self.code,
                         message=self.description,
-                        severity=Severity.WARNING,
+                        severity=Severity.ERROR,
                     )
                 )
         return sorted(diags, key=lambda d: d.line)

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/redundant_module_docstring.py

@@ -126,7 +126,7 @@ class RedundantModuleDocstring(Rule):
                 col=expression.col_offset + 1,
                 code=self.code,
                 message=self.description,
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
             )
         ]
 

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/require_nodecode_for_splitting_settings_field.py

@@ -181,7 +181,7 @@ def _settings_split_findings(
                 line=field.annotation.lineno,
                 col=field.annotation.col_offset + 1,
                 code=code,
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
                 message=(
                     f"Complex setting `{field_name}` is unconditionally split by `{function.name}` but "
                     "environment sources may JSON-decode it first; if the input contract is raw text, "

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/restated_test_docstring.py

@@ -333,7 +333,7 @@ class RestatedTestDocstring(Rule):
                 col=expr.col_offset + 1,
                 code=self.code,
                 message=_MESSAGE,
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
             )
         )
 
@@ -369,7 +369,7 @@ class RestatedTestDocstring(Rule):
                 col=expr.col_offset + 1,
                 code=self.code,
                 message=_MESSAGE,
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
             )
         )
 

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/trailing_value_narration.py

@@ -234,7 +234,7 @@ class TrailingValueNarration(Rule):
                             f"Trailing numeric-unit label {body.strip()!r} can drift; encode or reconcile the "
                             "unit in the assigned name or value type, then remove the comment."
                         ),
-                        severity=Severity.WARNING,
+                        severity=Severity.ERROR,
                         column_encoding=ColumnEncoding.CODEPOINTS,
                     )
                 )

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/uncontrolled_randomness_in_test.py

@@ -479,7 +479,7 @@ def _test_randomness_findings(
                 line=call.lineno,
                 col=call.col_offset + 1,
                 code=code,
-                severity=Severity.WARNING,
+                severity=Severity.ERROR,
                 message=(
                     "a standard-library PRNG sample runs in a potentially repeated test region without a "
                     "dominating deterministic seed; use an isolated `random.Random(seed)`, inject a "

{sarj_python_lint-0.84.0 → sarj_python_lint-0.84.2}/src/sarj_python_lint/rules/unused_mock_setup.py

@@ -155,7 +155,7 @@ class UnusedMockSetup(Rule):
                         line=position[0],
                         col=position[1],
                         code=self.code,
-                        severity=Severity.WARNING,
+                        severity=Severity.ERROR,
                         message=finding.message,
                     )
                 )