refactor: improve type hints and formatting in keyring provider

Author: putradimas

pydantic_settings/sources/providers/keyring.py

@@ -1,4 +1,5 @@
-from typing import TYPE_CHECKING, Mapping
+from collections.abc import Mapping
+from typing import TYPE_CHECKING
 
 from pydantic_settings import BaseSettings, EnvSettingsSource
 
@@ -7,7 +8,8 @@
 else:
     keyring = None
 
-def import_keyring():
+
+def import_keyring() -> None:
     global keyring
     if keyring is not None:
         return
@@ -16,7 +18,7 @@ def import_keyring():
 
         return
     except ImportError as e:
-        raise ImportError("Keyring is not installed, run `pip install keyring`") from e
+        raise ImportError('Keyring is not installed, run `pip install keyring`') from e
 
 
 class KeyringConfigSettingsSource(EnvSettingsSource):
@@ -27,20 +29,16 @@ def __init__(
         case_sensitive: bool | None = None,
         env_prefix: str | None = None,
     ):
-        self.keyring_service = (
-            keyring_service if case_sensitive else keyring_service.lower()
-        )
-        super().__init__(
-            settings_cls, case_sensitive=case_sensitive, env_prefix=env_prefix
-        )
+        self.keyring_service = keyring_service if case_sensitive else keyring_service.lower()
+        super().__init__(settings_cls, case_sensitive=case_sensitive, env_prefix=env_prefix)
 
     def _load_env_vars(self) -> Mapping[str, str | None]:
         import_keyring()
 
         prefix = self.env_prefix
         if not self.case_sensitive:
-                prefix = self.env_prefix.lower()
-        env_vars = {}
+            prefix = self.env_prefix.lower()
+        env_vars: dict[str, str | None] = {}
         for field in self.settings_cls.model_fields.keys():
             if not self.case_sensitive:
                 field = field.lower()
@@ -54,9 +52,9 @@ def _load_env_vars(self) -> Mapping[str, str | None]:
         return env_vars
 
     def __repr__(self) -> str:
-        return f"{self.__class__.__name__}(keyring_service={self.keyring_service})"
+        return f'{self.__class__.__name__}(keyring_service={self.keyring_service})'
 
 
 __all__ = [
-    "KeyringConfigSettingsSource",
+    'KeyringConfigSettingsSource',
 ]