feat: add more workspace linting rules

Makefile

@@ -30,6 +30,9 @@ help::
 	@echo "  stop-infra     -- Stop all infra used by tests."
 	@echo "                    NOTE: run this command to ensure all containers are stopped after tests"
 
+mypy: $(call FORALL_PKGS,mypy)
+help::
+	@echo "  mypy          -- Run mypy on all files"
 
 ruff:
 	@uv run ruff check --fix

pyproject.toml

@@ -18,3 +18,28 @@ supabase = { workspace = true }
 
 [tool.pytest.ini_options]
 asyncio_mode = "auto"
+
+[tool.ruff]
+target-version = "py39"
+
+[tool.ruff.lint]
+select = [
+  # pycodestyle
+  "E",
+  # Pyflakes
+  "F",
+  # pyupgrade
+  "UP",
+  # flake8-bugbear
+  "B",
+  # flake8-simplify
+  # "SIM",
+  # isort
+  "I",
+  "ANN2"
+]
+ignore = ["F403", "E501", "E402", "UP006", "UP035"]
+
+[tool.ruff.lint.pyupgrade]
+# Preserve types, even if a file imports `from __future__ import annotations`.
+keep-runtime-typing = true

src/auth/pyproject.toml

@@ -50,29 +50,6 @@ lints = [
 ]
 dev = [{ include-group = "lints" }, {include-group = "tests" }]
 
-[tool.ruff.lint]
-select = [
-  # pycodestyle
-  "E",
-  # Pyflakes
-  "F",
-  # pyupgrade
-  "UP",
-  # flake8-bugbear
-  "B",
-  # flake8-simplify
-  # "SIM",
-  # isort
-  "I",
-  "ANN2"
-]
-ignore = ["F403", "E501", "E402", "UP006", "UP035"]
-
-[tool.ruff.lint.pyupgrade]
-# Preserve types, even if a file imports `from __future__ import annotations`.
-keep-runtime-typing = true
-
-
 [tool.pytest.ini_options]
 asyncio_mode = "auto"
 

src/auth/tests/_async/clients.py

@@ -5,10 +5,9 @@
 
 from faker import Faker
 from jwt import encode
-from typing_extensions import NotRequired, TypedDict
-
 from supabase_auth import AsyncGoTrueAdminAPI, AsyncGoTrueClient
 from supabase_auth.types import User
+from typing_extensions import NotRequired, TypedDict
 
 
 def mock_access_token() -> str:

src/auth/tests/_async/test_gotrue.py

@@ -3,7 +3,6 @@
 
 import pytest
 from jwt import encode
-
 from supabase_auth.errors import (
     AuthApiError,
     AuthInvalidJwtError,
@@ -489,7 +488,6 @@ async def test_sign_in_with_otp() -> None:
     from unittest.mock import patch
 
     from httpx import Response
-
     from supabase_auth.types import AuthOtpResponse
 
     # First test for email OTP

src/auth/tests/_async/test_gotrue_admin_api.py

@@ -1,7 +1,6 @@
 import uuid
 
 import pytest
-
 from supabase_auth.errors import (
     AuthApiError,
     AuthError,

src/auth/tests/_sync/clients.py

@@ -5,10 +5,9 @@
 
 from faker import Faker
 from jwt import encode
-from typing_extensions import NotRequired, TypedDict
-
 from supabase_auth import SyncGoTrueAdminAPI, SyncGoTrueClient
 from supabase_auth.types import User
+from typing_extensions import NotRequired, TypedDict
 
 
 def mock_access_token() -> str:

src/auth/tests/_sync/test_gotrue.py

@@ -3,7 +3,6 @@
 
 import pytest
 from jwt import encode
-
 from supabase_auth.errors import (
     AuthApiError,
     AuthInvalidJwtError,
@@ -487,7 +486,6 @@ def test_sign_in_with_otp() -> None:
     from unittest.mock import patch
 
     from httpx import Response
-
     from supabase_auth.types import AuthOtpResponse
 
     # First test for email OTP

src/auth/tests/_sync/test_gotrue_admin_api.py

@@ -1,7 +1,6 @@
 import uuid
 
 import pytest
-
 from supabase_auth.errors import (
     AuthApiError,
     AuthError,

src/auth/tests/test_helpers.py

@@ -6,7 +6,6 @@
 import respx
 from httpx import Headers, HTTPStatusError, Response
 from pydantic import BaseModel
-
 from supabase_auth.constants import (
     API_VERSION_HEADER_NAME,
 )
@@ -339,7 +338,6 @@ def test_handle_exception_weak_password_branch() -> None:
     which is logically impossible, so we'll test it by mocking the implementation details.
     """
     import httpx
-
     from supabase_auth.errors import AuthWeakPasswordError
     from supabase_auth.helpers import handle_exception
 

src/functions/conftest.py

@@ -7,7 +7,7 @@
 _test_failed_incremental: Dict[str, Dict[Tuple[int, ...], str]] = {}
 
 
-def pytest_runtest_makereport(item, call):
+def pytest_runtest_makereport(item, call) -> None:
     if "incremental" in item.keywords:
         # incremental marker is used
         if call.excinfo is not None:
@@ -29,7 +29,7 @@ def pytest_runtest_makereport(item, call):
             )
 
 
-def pytest_runtest_setup(item):
+def pytest_runtest_setup(item) -> None:
     if "incremental" in item.keywords:
         # retrieve the class name of the test
         cls_name = str(item.cls)

src/functions/run-unasync.py

@@ -1,6 +1,7 @@
-import unasync
 from pathlib import Path
 
+import unasync
+
 paths = Path("src/supabase_functions").glob("**/*.py")
 tests = Path("tests").glob("**/*.py")
 

src/functions/src/supabase_functions/_async/functions_client.py

@@ -1,7 +1,7 @@
 from typing import Any, Dict, Literal, Optional, Union
 from warnings import warn
 
-from httpx import AsyncClient, HTTPError, Response, QueryParams
+from httpx import AsyncClient, HTTPError, QueryParams, Response
 from yarl import URL
 
 from ..errors import FunctionsHttpError, FunctionsRelayError
@@ -22,7 +22,7 @@ def __init__(
         verify: Optional[bool] = None,
         proxy: Optional[str] = None,
         http_client: Optional[AsyncClient] = None,
-    ):
+    ) -> None:
         if not is_http_url(url):
             raise ValueError("url must be a valid HTTP URL string")
         self.url = URL(url)
@@ -133,7 +133,7 @@ async def invoke(
             region = invoke_options.get("region")
             if region:
                 if not isinstance(region, FunctionRegion):
-                    warn(f"Use FunctionRegion({region})")
+                    warn(f"Use FunctionRegion({region})", stacklevel=2)
                     region = FunctionRegion(region)
 
                 if region.value != "any":

src/functions/src/supabase_functions/_sync/functions_client.py

@@ -1,7 +1,7 @@
 from typing import Any, Dict, Literal, Optional, Union
 from warnings import warn
 
-from httpx import Client, HTTPError, Response, QueryParams
+from httpx import Client, HTTPError, QueryParams, Response
 from yarl import URL
 
 from ..errors import FunctionsHttpError, FunctionsRelayError
@@ -22,7 +22,7 @@ def __init__(
         verify: Optional[bool] = None,
         proxy: Optional[str] = None,
         http_client: Optional[Client] = None,
-    ):
+    ) -> None:
         if not is_http_url(url):
             raise ValueError("url must be a valid HTTP URL string")
         self.url = URL(url)
@@ -133,7 +133,7 @@ def invoke(
             region = invoke_options.get("region")
             if region:
                 if not isinstance(region, FunctionRegion):
-                    warn(f"Use FunctionRegion({region})")
+                    warn(f"Use FunctionRegion({region})", stacklevel=2)
                     region = FunctionRegion(region)
 
                 if region.value != "any":

src/functions/src/supabase_functions/utils.py

@@ -1,9 +1,7 @@
 import sys
 from urllib.parse import urlparse
-from warnings import warn
 
 from httpx import AsyncClient as AsyncClient  # noqa: F401
-from httpx import Client as BaseClient
 
 if sys.version_info >= (3, 11):
     from enum import StrEnum
@@ -33,25 +31,6 @@ class FunctionRegion(StrEnum):
     UsWest2 = "us-west-2"
 
 
-class SyncClient(BaseClient):
-    def __init__(self, *args, **kwargs):
-        warn(
-            "The 'SyncClient' class is deprecated. Please use `Client` from the httpx package instead.",
-            DeprecationWarning,
-            stacklevel=2,
-        )
-
-        super().__init__(*args, **kwargs)
-
-    def aclose(self) -> None:
-        warn(
-            "The 'aclose' method is deprecated. Please use `close` method from `Client` in the httpx package instead.",
-            DeprecationWarning,
-            stacklevel=2,
-        )
-        self.close()
-
-
 def is_valid_str_arg(target: str) -> bool:
     return isinstance(target, str) and len(target.strip()) > 0