Merge pull request #2135 from tisnik/lcore-2810

LCORE-2810: Consistent Optional data type in LCore sources

Author: tisnik

scripts/latest-tag.py

@@ -2,6 +2,7 @@
 
 import os
 import subprocess
+from typing import Optional
 
 
 def version_split(value: str) -> tuple[int, ...]:
@@ -19,7 +20,7 @@ def is_prerelease(tag: str) -> bool:
     return any(n in tag for n in omit)
 
 
-def get_latest_stable() -> str | None:
+def get_latest_stable() -> Optional[str]:
     """Return the latest stable tag."""
     stdout = subprocess.check_output(["git", "tag"], text=True)
     tags = [tag for tag in stdout.splitlines() if not is_prerelease(tag)]

src/models/config.py

@@ -698,7 +698,7 @@ class UnifiedInferenceProvider(ConfigurationBase):
         "Llama Stack provider_type by the synthesizer.",
     )
 
-    id: str | None = Field(
+    id: Optional[str] = Field(
         None,
         title="Provider ID",
         description="Optional identifier emitted as the Llama Stack provider_id. "
@@ -730,7 +730,7 @@ class UnifiedInferenceProvider(ConfigurationBase):
 
     @field_validator("id")
     @classmethod
-    def validate_id(cls, value: str | None) -> str | None:
+    def validate_id(cls, value: Optional[str]) -> Optional[str]:
         """Strip and validate an optional high-level provider id.
 
         Parameters:

src/pydantic_ai_lightspeed/llamastack/_model.py

@@ -360,9 +360,9 @@ def from_llama_stack_client(
         model_name: str,
         client: AsyncLlamaStackClient | AsyncLlamaStackAsLibraryClient,
         *,
-        responses_params: ResponsesApiParams | None = None,
-        model_settings: ModelSettings | None = None,
-        profile: ModelProfileSpec | None = None,
+        responses_params: Optional[ResponsesApiParams] = None,
+        model_settings: Optional[ModelSettings] = None,
+        profile: Optional[ModelProfileSpec] = None,
     ) -> LlamaStackResponsesModel:
         """Create a ``LlamaStackResponsesModel`` from a Llama Stack client.