sarj-python-lint 0.67.7__tar.gz → 0.67.9__tar.gz

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sarj-python-lint
-Version: 0.67.7
+Version: 0.67.9
 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.67.7 → sarj_python_lint-0.67.9}/pyproject.toml

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

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/docstring_args_restate_signature.py

@@ -69,12 +69,12 @@ class DocstringArgsRestateSignature(Rule):
             ),
             RuleExample(
                 example_id="argument-documents-unit",
-                title="Argument description records a unit",
+                title="Redundant argument section removed",
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
                         "app/widgets.py",
-                        'def set_timeout_ms(timeout_ms: int) -> None:\n    """Configure the request deadline.\n\n    Args:\n        timeout_ms: Timeout in ms\n    """\n',
+                        'def count_widgets(tenant_id: str) -> int:\n    """Count active widgets."""\n    return 0\n',
                     ),
                 ),
                 focus_path=PurePosixPath("app/widgets.py"),

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/docstring_returns_restate_signature.py

@@ -135,7 +135,7 @@ class DocstringReturnsRestateSignature(Rule):
                 files=(
                     ExampleFile.python(
                         "app/lines.py",
-                        'def get_line_length(line: list[str]) -> int:\n    """Measure a rendered line.\n\n    Returns:\n        The width in terminal cells, which is not the character count.\n    """\n    return len(line)\n',
+                        'def get_line_length(line: list[str]) -> int:\n    """Measure a rendered line.\n\n    Returns:\n        The number of render fragments queued for output.\n    """\n    return len(line)\n',
                     ),
                 ),
                 focus_path=PurePosixPath("app/lines.py"),

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/duplicated_override_docstring.py

@@ -84,7 +84,7 @@ class DuplicatedOverrideDocstring(Rule):
             ),
             RuleExample(
                 example_id="override-specific-docstring",
-                title="Override documents its distinct behavior",
+                title="Override relies on the base contract",
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
@@ -93,9 +93,8 @@ class DuplicatedOverrideDocstring(Rule):
                         "    def get(self, key: str) -> str:\n"
                         '        """Get a value by key."""\n'
                         "        return key\n\n"
-                        "class ReplicaStore(Store):\n"
+                        "class MemoryStore(Store):\n"
                         "    def get(self, key: str) -> str:\n"
-                        '        """Get a value from the read replica."""\n'
                         "        return key\n",
                     ),
                 ),

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/fastapi_openapi_contract.py

@@ -66,9 +66,9 @@ class FastapiOpenapiContract(Rule):
     id: str = "fastapi-openapi-contract"
     code: str = "SARJ094"
     documentation: ClassVar[RuleDocumentation | None] = RuleDocumentation(
-        summary="FastAPI operations must publish explicit request, response, and OpenAPI contracts.",
-        rationale="Complete route metadata keeps generated OpenAPI accurate for clients, validation, and review.",
-        remediation="Declare route metadata, typed parameters, response schemas, and documented alternate responses.",
+        summary="FastAPI operations must publish accurate request, response, and status contracts.",
+        rationale="Typed routes and explicit response behavior keep generated OpenAPI accurate without duplicating self-documenting names in prose.",
+        remediation="Declare status codes, typed parameters, response schemas, and alternate responses raised directly by the handler.",
         category=RuleCategory.CORRECTNESS,
         limitations=(
             "Hidden routes, WebSocket handlers, tests, generated files, documentation-source examples, and unrelated decorators are excluded.",
@@ -78,7 +78,7 @@ class FastapiOpenapiContract(Rule):
         examples=(
             RuleExample(
                 example_id="missing-operation-metadata",
-                title="Visible operation without required metadata",
+                title="Visible operation without an explicit success status",
                 outcome=ExampleOutcome.MATCH,
                 files=(
                     ExampleFile.python(
@@ -97,7 +97,7 @@ class FastapiOpenapiContract(Rule):
                 files=(
                     ExampleFile.python(
                         "api.py",
-                        "from fastapi import APIRouter\n\nrouter = APIRouter()\n\n@router.get('/users', summary='Read users', description='Returns visible users.', status_code=200)\nasync def users() -> list[UserResponse]:\n    return []\n",
+                        "from fastapi import APIRouter\n\nrouter = APIRouter()\n\n@router.get('/users', status_code=200)\nasync def read_users() -> list[UserResponse]:\n    return []\n",
                     ),
                 ),
                 focus_path=PurePosixPath("api.py"),
@@ -134,7 +134,7 @@ class FastapiOpenapiContract(Rule):
             findings.extend(_check_raw_request(function, routes, index))
             findings.extend(_check_direct_responses(function, routes, index))
             for route in _operations(routes):
-                findings.extend(_check_metadata(function, route))
+                findings.extend(_check_metadata(route))
                 findings.extend(_check_projection(route))
         findings.extend(_check_route_conflicts(declared))
         return [
@@ -155,29 +155,13 @@ def _operations(routes: tuple[Route, ...]) -> tuple[Route, ...]:
     return tuple({id(route.decorator): route for route in routes}.values())
 
 
-def _check_metadata(function: ast.FunctionDef | ast.AsyncFunctionDef, route: Route) -> list[_Finding]:
+def _check_metadata(route: Route) -> list[_Finding]:
     if route.has_unpack:
         return []
     keywords = route.keywords
-    missing: list[str] = []
-    if not _nonblank(keywords.get("summary")):
-        missing.append("summary")
-    docstring = ast.get_docstring(function, clean=False)
-    if not (_nonblank(keywords.get("description")) or bool(docstring and docstring.strip())):
-        missing.append("description or handler docstring")
-    if "status_code" not in keywords or _literal_none(keywords["status_code"]):
-        missing.append("status_code")
-    if not missing:
+    if "status_code" in keywords and not _literal_none(keywords["status_code"]):
         return []
-    return [_Finding(route.decorator, f"[metadata] operation requires explicit {', '.join(missing)}.")]
-
-
-def _nonblank(node: ast.expr | None) -> bool:
-    if node is None:
-        return False
-    if isinstance(node, ast.Constant):
-        return isinstance(node.value, str) and bool(node.value.strip())
-    return True
+    return [_Finding(route.decorator, "[metadata] operation requires explicit status_code.")]
 
 
 def _literal_none(node: ast.expr) -> bool:
@@ -238,10 +222,6 @@ def _check_parameters(
             findings.append(
                 _Finding(parameter, f"[parameter] `{parameter.arg}` must not duplicate FastAPI marker metadata.")
             )
-        if marker_name in SCHEMA_MARKERS and not _nonblank(_keyword(marker_call, "description")):
-            findings.append(
-                _Finding(parameter, f"[parameter] `{parameter.arg}` marker requires a non-empty description.")
-            )
         if marker_name in SCHEMA_MARKERS and _schema_erasing(value_type, index):
             findings.append(
                 _Finding(

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/kwarg_heavy_construction_in_test.py

@@ -92,12 +92,13 @@ class KwargHeavyConstructionInTest(Rule):
                         "tests/test_call.py",
                         "def build_style(**overrides):\n"
                         "    defaults = dict(\n"
+                        '        color="red", bgcolor="black", bold=True,\n'
                         "        dim=True, italic=True, underline=True,\n"
                         "        blink=True, blink2=True, reverse=True,\n"
                         "    )\n"
                         "    return Style(**(defaults | overrides))\n\n"
                         "def test_style():\n"
-                        '    style = build_style(color="red", bgcolor="black", bold=True)\n'
+                        "    style = build_style()\n"
                         "    assert str(style)\n\n"
                         "def test_style_again():\n"
                         '    other = build_style(color="blue", bgcolor="white", bold=False)\n'

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_file_level_escape_hatch_noqa.py

@@ -68,7 +68,7 @@ class NoFileLevelEscapeHatchNoqa(Rule):
                 files=(
                     ExampleFile.python(
                         "app/service.py",
-                        "from unittest import mock  # noqa: TID251 — vendor SDK boundary\n",
+                        "import os  # noqa: TID251 — vendor SDK boundary\n",
                     ),
                 ),
                 focus_path=PurePosixPath("app/service.py"),

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_file_level_suppression.py

@@ -76,7 +76,7 @@ class NoFileLevelSuppression(Rule):
                 example_id="scoped-type-checker-suppression",
                 title="Mypy suppression names its code",
                 outcome=ExampleOutcome.NO_MATCH,
-                files=(ExampleFile.python("service.py", "# type: ignore[attr-defined]\nimport os\n"),),
+                files=(ExampleFile.python("service.py", "# type: ignore[import-not-found]\nimport os\n"),),
                 focus_path=PurePosixPath("service.py"),
                 expected_count=0,
                 public=True,

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_frozen_after_validator_field_write.py

@@ -86,20 +86,19 @@ class NoFrozenAfterValidatorFieldWrite(Rule):
             ),
             RuleExample(
                 example_id="after-validator-only-validates-frozen-field",
-                title="After-validator checks without mutating the model",
+                title="Before-validator normalizes the input value",
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
                         "app/models.py",
-                        "from pydantic import BaseModel, ConfigDict, model_validator\n\n"
+                        "from pydantic import BaseModel, ConfigDict, field_validator\n\n"
                         "class Counter(BaseModel):\n"
                         "    model_config = ConfigDict(frozen=True)\n"
                         "    value: int\n\n"
-                        '    @model_validator(mode="after")\n'
-                        "    def require_non_negative(self):\n"
-                        "        if self.value < 0:\n"
-                        '            raise ValueError("value must be non-negative")\n'
-                        "        return self\n",
+                        '    @field_validator("value", mode="before")\n'
+                        "    @classmethod\n"
+                        "    def normalize(cls, value: int) -> int:\n"
+                        "        return abs(value)\n",
                     ),
                 ),
                 focus_path=PurePosixPath("app/models.py"),

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_generic_single_export_module.py

@@ -72,8 +72,8 @@ class NoGenericSingleExportModule(Rule):
                 example_id="specific-single-export-module",
                 title="Specific module contains one public definition",
                 outcome=ExampleOutcome.NO_MATCH,
-                files=(ExampleFile.python("pagination.py", "class InvalidPaginationCursorError(Exception): ...\n"),),
-                focus_path=PurePosixPath("pagination.py"),
+                files=(ExampleFile.python("text_case.py", "def snake_case_text(value: str) -> str: ...\n"),),
+                focus_path=PurePosixPath("text_case.py"),
                 expected_count=0,
                 public=True,
             ),

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_secret_in_log.py

@@ -114,7 +114,12 @@ class NoSecretInLog(Rule):
                 example_id="redacted-logging-keyword",
                 title="Token prefix logged under a redacted name",
                 outcome=ExampleOutcome.NO_MATCH,
-                files=(ExampleFile.python("service.py", "logger.info('request', token_prefix=token[:6])\n"),),
+                files=(
+                    ExampleFile.python(
+                        "service.py",
+                        "token_prefix = token[:6]\nlogger.info('request', token_prefix=token_prefix)\n",
+                    ),
+                ),
                 focus_path=PurePosixPath("service.py"),
                 expected_count=0,
                 public=True,

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_string_concat_in_loop.py

@@ -75,7 +75,7 @@ class NoStringConcatInLoop(Rule):
                 files=(
                     ExampleFile.python(
                         "app/render.py",
-                        'def render(items):\n    lines = []\n    for item in items:\n        lines.append(str(item))\n    return "\\n".join(lines)\n',
+                        'def render(items):\n    lines = []\n    for item in items:\n        lines.append(f"{item}\\n")\n    return "".join(lines)\n',
                     ),
                 ),
                 focus_path=PurePosixPath("app/render.py"),

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_typed_doc_sections.py

@@ -60,11 +60,11 @@ class NoTypedDocSections(Rule):
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
-                        "publisher.py",
-                        'def publish(message: str, *, retry: bool) -> None:\n    """Publish one message.\n\n    Args:\n        message: Wire payload retained for the audit record.\n        retry: Whether a prior partial write may be attempted again.\n    """\n',
+                        "app.py",
+                        'def decode(value: str) -> dict[str, object]:\n    """Decode the value.\n\n    Args:\n        value: Text to decode.\n    """\n    return {}\n',
                     ),
                 ),
-                focus_path=PurePosixPath("publisher.py"),
+                focus_path=PurePosixPath("app.py"),
                 expected_count=0,
                 public=True,
             ),

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/prefer_match_type_dispatch.py

@@ -855,12 +855,14 @@ class PreferMatchTypeDispatch(Rule):
                         "app/parser.py",
                         "def parse(value: object):\n"
                         "    match value:\n"
-                        "        case None | Unset():\n"
+                        "        case Text():\n"
+                        "            return value\n"
+                        "        case Binary():\n"
+                        "            return value\n"
+                        "        case PathValue():\n"
                         "            return value\n"
-                        "        case int():\n"
-                        "            return str(value)\n"
                         "        case _:\n"
-                        "            return None\n",
+                        "            return coerce(value)\n",
                     ),
                 ),
                 focus_path=PurePosixPath("app/parser.py"),

{sarj_python_lint-0.67.7 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/restated_test_docstring.py

@@ -158,15 +158,15 @@ class RestatedTestDocstring(Rule):
             ),
             RuleExample(
                 example_id="test-regression-context",
-                title="Docstring explains the regression",
+                title="Redundant docstring removed",
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
-                        "tests/test_scheduler.py",
-                        'def test_keeps_the_lock():\n    """The scheduler would spin forever without this."""\n    assert acquire()\n',
+                        "tests/test_widget.py",
+                        "def test_widget_renders():\n    assert render(widget)\n",
                     ),
                 ),
-                focus_path=PurePosixPath("tests/test_scheduler.py"),
+                focus_path=PurePosixPath("tests/test_widget.py"),
                 expected_count=0,
                 public=True,
             ),