build: 🛠 Switch to pyright + basic project cleanup

Author: ddanier

.github/workflows/lint.yml

@@ -1,4 +1,4 @@
-name: "LINT: Run ruff & mypy"
+name: "LINT: Run ruff & pyright"
 on:
   push:
   pull_request:
@@ -17,7 +17,7 @@ jobs:
         run: |
           python -m pip install --upgrade poetry
           poetry install -E fastapi
-      - name: Lint with ruff & mypy
+      - name: Lint with ruff & pyright
         run: |
           poetry run ruff check pydantic_async_validation tests
-          poetry run mypy pydantic_async_validation
+          poetry run pyright pydantic_async_validation

.pre-commit-config.yaml

@@ -1,23 +1,23 @@
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.4.0
+    rev: v4.5.0
     hooks:
       - id: end-of-file-fixer
       - id: check-added-large-files
       - id: check-merge-conflict
       - id: check-docstring-first
       - id: debug-statements
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.0.291
+    rev: v0.2.1
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
   - repo: https://github.com/Lucas-C/pre-commit-hooks-safety
-    rev: v1.3.2
+    rev: v1.3.3
     hooks:
       - id: python-safety-dependencies-check
   - repo: https://github.com/alessandrojcm/commitlint-pre-commit-hook
-    rev: v9.5.0
+    rev: v9.11.0
     hooks:
       - id: commitlint
         stages: [commit-msg]

justfile

@@ -32,8 +32,8 @@ test-all: (poetry "run" "tox")
 
 ruff *args: (poetry "run" "ruff" "check" "pydantic_async_validation" "tests" args)
 
-mypy *args: (poetry "run" "mypy" "pydantic_async_validation" args)
+pyright *args: (poetry "run" "pyright" "pydantic_async_validation" args)
 
-lint: ruff mypy
+lint: ruff pyright
 
 publish: (poetry "publish" "--build")

pydantic_async_validation/utils.py

@@ -182,8 +182,8 @@ def prefix_errors(
                 **error,
                 'type': (
                     PydanticCustomError(
-                        error['type'],
-                        cast(ErrorDetails, error)['msg'],
+                        error['type'],  # type: ignore
+                        cast(ErrorDetails, error)['msg'],  # type: ignore
                     )
                     if isinstance(error['type'], str)
                     else error['type']
@@ -197,7 +197,7 @@ def prefix_errors(
             {
                 # Original data is InitErrorDetails, all fine
                 **error,
-                'loc': (*prefix, *error['loc']),
+                'loc': (*prefix, *error.get('loc', [])),
             },
         )
         for error

pyproject.toml

@@ -20,19 +20,21 @@ fastapi = ["fastapi"]
 pytest = ">=7.1.2,<9.0.0"
 pytest-cov = ">=3,<5"
 pytest-asyncio = ">=0.21.1,<0.24.0"
-mypy = ">=0.971,<2.0"
 tox = ">=3.26,<5.0"
 httpx = ">=0.24.1,<0.27.0"
-ruff = ">=0.0.284,<0.3.0"
+ruff = ">=0.2.0,<0.3.0"
+pyright = ">=1.1.350,<1.2"
 
 [tool.ruff]
-select = ["F","E","W","C","I","N","UP","ANN","S","B","A","COM","C4","T20","PT","ARG","TD","RUF"]
 line-length = 115
 target-version = "py38"
-ignore = ["A001","A002","A003","ANN101","ANN102","ANN401","C901","N8","B008","F405","F821"]
 output-format = "grouped"
 
-[tool.ruff.per-file-ignores]
+[tool.ruff.lint]
+select = ["F","E","W","C","I","N","UP","ANN","S","B","A","COM","C4","T20","PT","ARG","TD","RUF"]
+ignore = ["A001","A002","A003","ANN101","ANN102","ANN401","C901","N8","B008","F405","F821"]
+
+[tool.ruff.lint.per-file-ignores]
 "__init__.py" = ["F401"]
 "conftest.py" = ["S101","ANN","F401"]
 "test_*.py" = ["S101","ANN","F401"]