fix!: Move azure auth provider to separate module (#4912)

* Move azure auth provider to separate module

* Update lock

* fix component gen
This commit is contained in:
Jack Gerrits 2025-01-07 12:21:50 -05:00 committed by GitHub
parent bdfdc6ab5e
commit 310564908b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
7 changed files with 11 additions and 16 deletions

View File

@ -25,7 +25,7 @@ m1 = "autogen_ext.teams.magentic_one_cli:main"
langchain = ["langchain_core~= 0.3.3"]
azure = ["azure-core", "azure-identity"]
docker = ["docker~=7.0"]
openai = ["openai>=1.52.2", "aiofiles", "azure-core", "azure-identity"]
openai = ["openai>=1.52.2", "aiofiles"]
file-surfer = [
"autogen-agentchat==0.4.0.dev13",
"markitdown>=0.0.1a2",

View File

@ -1,11 +1,12 @@
from typing import List
from autogen_core import Component
from azure.core.credentials import TokenProvider
from azure.identity import DefaultAzureCredential, get_bearer_token_provider
from pydantic import BaseModel
from typing_extensions import Self
from azure.core.credentials import TokenProvider
from azure.identity import DefaultAzureCredential, get_bearer_token_provider
class TokenProviderConfig(BaseModel):
provider_kind: str

View File

@ -1,4 +1,3 @@
from ._azure_token_provider import AzureTokenProvider
from ._openai_client import (
AzureOpenAIChatCompletionClient,
OpenAIChatCompletionClient,
@ -10,5 +9,4 @@ __all__ = [
"AzureOpenAIChatCompletionClient",
"OpenAIClientConfiguration",
"OpenAIChatCompletionClient",
"AzureTokenProvider",
]

View File

@ -69,8 +69,6 @@ from openai.types.shared_params import FunctionDefinition, FunctionParameters
from pydantic import BaseModel
from typing_extensions import Self, Unpack
from autogen_ext.models.openai._azure_token_provider import AzureTokenProvider
from . import _model_info
from .config import (
AzureOpenAIClientConfiguration,
@ -1076,8 +1074,9 @@ class AzureOpenAIChatCompletionClient(
self._client = _azure_openai_client_from_config(state["_raw_config"])
def _to_config(self) -> AzureOpenAIClientConfigurationConfigModel:
copied_config = self._raw_config.copy()
from ...auth.azure import AzureTokenProvider
copied_config = self._raw_config.copy()
if "azure_ad_token_provider" in copied_config:
if not isinstance(copied_config["azure_ad_token_provider"], AzureTokenProvider):
raise ValueError("azure_ad_token_provider must be a AzureTokenProvider to be component serialized")
@ -1090,6 +1089,8 @@ class AzureOpenAIChatCompletionClient(
@classmethod
def _from_config(cls, config: AzureOpenAIClientConfigurationConfigModel) -> Self:
from ...auth.azure import AzureTokenProvider
copied_config = config.model_copy().model_dump(exclude_none=True)
if "azure_ad_token_provider" in copied_config:
copied_config["azure_ad_token_provider"] = AzureTokenProvider.load_component(

View File

@ -5,8 +5,6 @@ from autogen_core.models import ModelCapabilities, ModelInfo # type: ignore
from pydantic import BaseModel
from typing_extensions import Required, TypedDict
from .._azure_token_provider import AzureTokenProvider
class ResponseFormat(TypedDict):
type: Literal["text", "json_object"]
@ -51,7 +49,7 @@ class AzureOpenAIClientConfiguration(BaseOpenAIClientConfiguration, total=False)
azure_deployment: str
api_version: Required[str]
azure_ad_token: str
azure_ad_token_provider: AsyncAzureADTokenProvider | AzureTokenProvider
azure_ad_token_provider: AsyncAzureADTokenProvider # Or AzureTokenProvider
__all__ = [

View File

@ -8,7 +8,8 @@ from autogen_core._component_config import (
ComponentConfigImpl,
_type_to_provider_str, # type: ignore
)
from autogen_ext.models.openai import AzureOpenAIChatCompletionClient, AzureTokenProvider, OpenAIChatCompletionClient
from autogen_ext.auth.azure import AzureTokenProvider
from autogen_ext.models.openai import AzureOpenAIChatCompletionClient, OpenAIChatCompletionClient
from pydantic import BaseModel
all_defs: Dict[str, Any] = {}

View File

@ -497,8 +497,6 @@ magentic-one = [
]
openai = [
{ name = "aiofiles" },
{ name = "azure-core" },
{ name = "azure-identity" },
{ name = "openai" },
]
video-surfer = [
@ -529,9 +527,7 @@ requires-dist = [
{ name = "autogen-agentchat", marker = "extra == 'web-surfer'", editable = "packages/autogen-agentchat" },
{ name = "autogen-core", editable = "packages/autogen-core" },
{ name = "azure-core", marker = "extra == 'azure'" },
{ name = "azure-core", marker = "extra == 'openai'" },
{ name = "azure-identity", marker = "extra == 'azure'" },
{ name = "azure-identity", marker = "extra == 'openai'" },
{ name = "docker", marker = "extra == 'docker'", specifier = "~=7.0" },
{ name = "ffmpeg-python", marker = "extra == 'video-surfer'" },
{ name = "grpcio", marker = "extra == 'grpc'", specifier = "~=1.62.0" },