refactor: Update project to pass CI linting and formatting issues

birdcar · birdcar · commit e89180c195a0 · 2025-12-12T10:24:36.000-06:00
diff --git a/src/workos/api_keys.py b/src/workos/api_keys.py
diff --git a/src/workos/directory_sync.py b/src/workos/directory_sync.py
@@ -176,7 +176,6 @@ def list_users(
         after: Optional[str] = None,
         order: PaginationOrder = "desc",
     ) -> DirectoryUsersListResource:
-
         list_params: DirectoryUserListFilters = {
             "limit": limit,
             "before": before,
@@ -315,7 +314,6 @@ async def list_users(
         after: Optional[str] = None,
         order: PaginationOrder = "desc",
     ) -> DirectoryUsersListResource:
-
         list_params: DirectoryUserListFilters = {
             "limit": limit,
             "before": before,
diff --git a/src/workos/portal.py b/src/workos/portal.py
@@ -1,4 +1,4 @@
-from typing import Optional, Protocol, Dict, Literal, Union
+from typing import Optional, Protocol
 from workos.types.portal.portal_link import PortalLink
 from workos.types.portal.portal_link_intent import PortalLinkIntent
 from workos.types.portal.portal_link_intent_options import IntentOptions
@@ -36,7 +36,6 @@ def generate_link(
 
 
 class Portal(PortalModule):
-
     _http_client: SyncHTTPClient
 
     def __init__(self, http_client: SyncHTTPClient):
diff --git a/src/workos/types/api_keys/__init__.py b/src/workos/types/api_keys/__init__.py
diff --git a/src/workos/types/api_keys/api_keys.py b/src/workos/types/api_keys/api_keys.py
diff --git a/src/workos/types/fga/warnings.py b/src/workos/types/fga/warnings.py
@@ -1,8 +1,8 @@
-from typing import Sequence, Union, Any, Dict, Literal
-from typing_extensions import Annotated
+from typing import Any, Dict, Literal, Sequence, Union
 
 from pydantic import BeforeValidator
 from pydantic_core.core_schema import ValidationInfo
+from typing_extensions import Annotated
 
 from workos.types.workos_model import WorkOSModel
 
@@ -12,7 +12,7 @@ class FGABaseWarning(WorkOSModel):
     message: str
 
 
-class MissingContextKeysWarning(FGABaseWarning):
+class MissingContextKeysWarning(FGABaseWarning):  # type: ignore[override]
     code: Literal["missing_context_keys"]
     keys: Sequence[str]
 
diff --git a/src/workos/types/mfa/authentication_factor.py b/src/workos/types/mfa/authentication_factor.py
@@ -43,21 +43,21 @@ class AuthenticationFactorBase(WorkOSModel):
     user_id: Optional[str] = None
 
 
-class AuthenticationFactorTotp(AuthenticationFactorBase):
+class AuthenticationFactorTotp(AuthenticationFactorBase):  # type: ignore[override]
     """Representation of a MFA Authentication Factor Response as returned by WorkOS through the MFA feature."""
 
     type: TotpAuthenticationFactorType
     totp: TotpFactor
 
 
-class AuthenticationFactorTotpExtended(AuthenticationFactorBase):
+class AuthenticationFactorTotpExtended(AuthenticationFactorBase):  # type: ignore[override]
     """Representation of a MFA Authentication Factor Response when enrolling an authentication factor."""
 
     type: TotpAuthenticationFactorType
     totp: ExtendedTotpFactor
 
 
-class AuthenticationFactorSms(AuthenticationFactorBase):
+class AuthenticationFactorSms(AuthenticationFactorBase):  # type: ignore[override]
     """Representation of a SMS Authentication Factor Response as returned by WorkOS through the MFA feature."""
 
     type: SmsAuthenticationFactorType
diff --git a/src/workos/types/user_management/__init__.py b/src/workos/types/user_management/__init__.py
@@ -7,6 +7,6 @@
 from .organization_membership import *
 from .password_hash_type import *
 from .password_reset import *
-from .user_management_provider_type import *
-from .user import *
 from .session import *
+from .user import *
+from .user_management_provider_type import *
diff --git a/src/workos/types/user_management/session.py b/src/workos/types/user_management/session.py
@@ -1,7 +1,5 @@
 from enum import Enum
-from typing import Optional, Sequence, TypedDict, Union
-
-from typing_extensions import Literal
+from typing import Literal, Optional, Sequence, TypedDict, Union
 
 from workos.types.user_management.impersonator import Impersonator
 from workos.types.user_management.user import User
diff --git a/src/workos/types/workos_model.py b/src/workos/types/workos_model.py
@@ -14,7 +14,7 @@ def dict(
         by_alias: bool = False,
         exclude_unset: bool = False,
         exclude_defaults: bool = False,
-        exclude_none: bool = False
+        exclude_none: bool = False,
     ) -> Dict[str, Any]:
         return self.model_dump(
             include=include,
diff --git a/src/workos/user_management.py b/src/workos/user_management.py
@@ -1,5 +1,6 @@
 from typing import Awaitable, Optional, Protocol, Sequence, Type, Union, cast
 from urllib.parse import urlencode
+
 from workos._client_configuration import ClientConfiguration
 from workos.session import AsyncSession, Session
 from workos.types.list_resource import (
@@ -36,19 +37,20 @@
     AuthenticateWithTotpParameters,
 )
 from workos.types.user_management.authentication_response import (
-    AuthKitAuthenticationResponse,
     AuthenticationResponseType,
+    AuthKitAuthenticationResponse,
 )
 from workos.types.user_management.list_filters import (
     AuthenticationFactorsListFilters,
     InvitationsListFilters,
     OrganizationMembershipsListFilters,
+    SessionsListFilters,
     UsersListFilters,
 )
 from workos.types.user_management.password_hash_type import PasswordHashType
 from workos.types.user_management.screen_hint import ScreenHintType
-from workos.types.user_management.session import SessionConfig
 from workos.types.user_management.session import Session as UserManagementSession
+from workos.types.user_management.session import SessionConfig
 from workos.types.user_management.user_management_provider_type import (
     UserManagementProviderType,
 )
@@ -57,11 +59,11 @@
 from workos.utils.pagination_order import PaginationOrder
 from workos.utils.request_helper import (
     DEFAULT_LIST_RESPONSE_LIMIT,
-    RESPONSE_TYPE_CODE,
-    REQUEST_METHOD_POST,
-    REQUEST_METHOD_GET,
     REQUEST_METHOD_DELETE,
+    REQUEST_METHOD_GET,
+    REQUEST_METHOD_POST,
     REQUEST_METHOD_PUT,
+    RESPONSE_TYPE_CODE,
     QueryParameters,
     RequestHelper,
 )
@@ -113,8 +115,6 @@
     Invitation, InvitationsListFilters, ListMetadata
 ]
 
-from workos.types.user_management.list_filters import SessionsListFilters
-
 SessionsListResource = WorkOSListResource[
     UserManagementSession, SessionsListFilters, ListMetadata
 ]
diff --git a/src/workos/utils/request_helper.py b/src/workos/utils/request_helper.py
@@ -14,7 +14,6 @@
 
 
 class RequestHelper:
-
     @classmethod
     def build_parameterized_url(cls, url: str, **params: QueryParameterValue) -> str:
         escaped_params = {k: urllib.parse.quote(str(v)) for k, v in params.items()}
diff --git a/src/workos/vault.py b/src/workos/vault.py
@@ -9,7 +9,6 @@
     WorkOSListResource,
 )
 from workos.utils.http_client import SyncHTTPClient
-from workos.utils.pagination_order import PaginationOrder
 from workos.utils.request_helper import (
     DEFAULT_LIST_RESPONSE_LIMIT,
     REQUEST_METHOD_DELETE,
diff --git a/src/workos/widgets.py b/src/workos/widgets.py
@@ -30,7 +30,6 @@ def get_token(
 
 
 class Widgets(WidgetsModule):
-
     _http_client: SyncHTTPClient
 
     def __init__(self, http_client: SyncHTTPClient):
diff --git a/tests/test_async_http_client.py b/tests/test_async_http_client.py
@@ -246,7 +246,7 @@ async def test_request_bad_body_raises_expected_exception_with_request_data(self
         try:
             await self.http_client.request("bad_place")
         except ServerException as ex:
-            assert ex.message == None
+            assert ex.message is None
             assert ex.request_id == request_id
             assert ex.__class__ == ServerException
 
diff --git a/tests/test_directory_sync.py b/tests/test_directory_sync.py
@@ -337,7 +337,7 @@ def test_primary_email_none(
 
         me = mock_user_instance.primary_email()
 
-        assert me == None
+        assert me is None
 
     def test_list_directories_auto_pagination(
         self,
diff --git a/tests/test_passwordless.py b/tests/test_passwordless.py
@@ -1,4 +1,5 @@
 import pytest
+
 from workos.passwordless import Passwordless
 
 
@@ -53,4 +54,4 @@ def test_get_send_session_succeeds(self, capture_and_mock_http_client_request):
             "/passwordless/sessions/passwordless_session_01EHDAK2BFGWCSZXP9HGZ3VK8C/send"
         )
         assert request_kwargs["method"] == "post"
-        assert response == True
+        assert response
diff --git a/tests/test_session.py b/tests/test_session.py
@@ -616,6 +616,6 @@ def get_client():
 
         first_client = clients[0]
         for client in clients[1:]:
-            assert (
-                client is first_client
-            ), "All concurrent calls should return the same instance"
+            assert client is first_client, (
+                "All concurrent calls should return the same instance"
+            )
diff --git a/tests/test_sync_http_client.py b/tests/test_sync_http_client.py
@@ -1,8 +1,8 @@
 from platform import python_version
+from unittest.mock import MagicMock
 
 import httpx
 import pytest
-from unittest.mock import MagicMock
 
 from workos.exceptions import (
     AuthenticationException,
@@ -15,7 +15,6 @@
 )
 from workos.utils.http_client import SyncHTTPClient
 
-
 STATUS_CODE_TO_EXCEPTION_MAPPING = [
     (400, BadRequestException),
     (401, AuthenticationException),
@@ -244,7 +243,7 @@ def test_request_bad_body_raises_expected_exception_with_request_data(self):
         try:
             self.http_client.request("bad_place")
         except ServerException as ex:
-            assert ex.message == None
+            assert ex.message is None
             assert ex.request_id == request_id
             assert ex.__class__ == ServerException
 
diff --git a/tests/test_vault.py b/tests/test_vault.py
@@ -1,14 +1,14 @@
 import pytest
+
 from tests.utils.fixtures.mock_vault_object import (
-    MockVaultObject,
-    MockObjectVersion,
     MockObjectDigest,
     MockObjectMetadata,
+    MockObjectVersion,
+    MockVaultObject,
 )
 from tests.utils.list_resource import list_response_of
-from tests.utils.syncify import syncify
-from workos.vault import Vault
 from workos.types.vault.key import KeyContext
+from workos.vault import Vault
 
 
 class TestVault:
@@ -132,9 +132,7 @@ def test_list_objects_with_params(
             self.http_client, mock_vault_objects_list, 200
         )
 
-        vault_objects = self.vault.list_objects(
-            limit=5, before="vault_before", after="vault_after"
-        )
+        self.vault.list_objects(limit=5, before="vault_before", after="vault_after")
 
         assert request_kwargs["method"] == "get"
         assert request_kwargs["url"].endswith("/vault/v1/kv")
@@ -257,7 +255,7 @@ def test_update_object_with_version_check(
             self.http_client, mock_vault_object, 200
         )
 
-        vault_object = self.vault.update_object(
+        self.vault.update_object(
             object_id="vault_01234567890abcdef",
             value="updated-value",
             version_check="version_123",
diff --git a/tests/utils/fixtures/mock_event.py b/tests/utils/fixtures/mock_event.py
@@ -2,7 +2,6 @@
 
 from workos.types.events import DirectoryActivatedEvent
 from workos.types.events.directory_payload_with_legacy_fields import (
-    DirectoryPayloadWithLegacyFields,
     DirectoryPayloadWithLegacyFieldsForEventsApi,
 )
 
diff --git a/tests/utils/fixtures/mock_profile.py b/tests/utils/fixtures/mock_profile.py
@@ -2,7 +2,6 @@
 
 
 class MockProfile(Profile):
-
     def __init__(self, id: str):
         super().__init__(
             object="profile",
diff --git a/tests/utils/test_request_helper.py b/tests/utils/test_request_helper.py
@@ -2,7 +2,6 @@
 
 
 class TestRequestHelper:
-
     def test_build_parameterized_url(self):
         assert RequestHelper.build_parameterized_url("a/b/c") == "a/b/c"
         assert RequestHelper.build_parameterized_url("a/{b}/c", b="b") == "a/b/c"
diff --git a/workos/__about__.py b/workos/__about__.py
