Lee-W commented on code in PR #72152:
URL: https://github.com/apache/airflow/pull/72152#discussion_r3964080134
##########
providers/common/ai/tests/unit/common/ai/hooks/test_pydantic_ai.py:
##########
@@ -27,13 +29,32 @@
from pydantic_ai.providers import infer_provider_class
from airflow.models.connection import Connection
+from airflow.providers.common.ai.get_provider_info import get_provider_info
from airflow.providers.common.ai.hooks.pydantic_ai import (
PydanticAIAzureHook,
PydanticAIBedrockHook,
PydanticAIHook,
PydanticAIVertexHook,
)
+# Matches the `google...` provider key pydantic-ai expects before the
`:model-name`
+# separator, e.g. "google-cloud" out of "google-cloud:gemini-2.0-flash".
+_GOOGLE_MODEL_PREFIX_RE = re.compile(r"google[\w-]*(?=:)")
Review Comment:
`_extract_google_cloud_prefix` now checked prefix is exactly `google-cloud`,
not just provider `infer_provider_class` recognizes. Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
