fix(ci): bump minimum Python to 3.9 and fix type errors from #37

PR #37 broke all three CI jobs:

- test / compat-build: cn.py uses PEP 585 builtins (`list[str]`) which
  fail at Pydantic model-build time on Python 3.8. Raise the minimum
  supported version to 3.9 (pyproject + compat pyproject + CI matrix,
  poetry.lock refreshed) so the existing annotations are valid.
- compile (mypy): fix the type errors introduced alongside the new
  regional/CN feature:
  * Remove the `@overload` stubs on Agent/Agora `__new__`/`__init__`
    (they could not be paired with their implementations by mypy).
    Runtime dispatch is unchanged; only static return-type precision
    is dropped.
  * __init__.py: import AgentClient/AsyncAgentClient from .pool_client
    (where they are defined) instead of .agentkit.
  * Agent.turn_detection / _resolve_asr_config: accept dict as well as
    the model, matching runtime behaviour.
  * regional_agent.py: type: ignore[override] for the intentional
    regional-vendor parameter narrowing.
  * test_regional_vendors.py: narrow Optional before indexing.

Verified: mypy clean (303 files), 193 passed / 1 skipped, lock consistent.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: plutoless

.github/workflows/ci.yml

@@ -9,7 +9,7 @@ jobs:
       - name: Set up python
         uses: actions/setup-python@v4
         with:
-          python-version: 3.8
+          python-version: 3.9
       - name: Bootstrap poetry
         run: |
           curl -sSL https://install.python-poetry.org | python - -y --version 1.5.1
@@ -27,7 +27,7 @@ jobs:
       - name: Set up python
         uses: actions/setup-python@v4
         with:
-          python-version: 3.8
+          python-version: 3.9
       - name: Bootstrap poetry
         run: |
           curl -sSL https://install.python-poetry.org | python - -y --version 1.5.1
@@ -45,7 +45,7 @@ jobs:
       - name: Set up python
         uses: actions/setup-python@v4
         with:
-          python-version: 3.8
+          python-version: 3.9
       - name: Bootstrap poetry
         run: |
           curl -sSL https://install.python-poetry.org | python - -y --version 1.5.1
@@ -77,7 +77,7 @@ jobs:
       - name: Set up python
         uses: actions/setup-python@v4
         with:
-          python-version: 3.8
+          python-version: 3.9
       - name: Bootstrap poetry
         run: |
           curl -sSL https://install.python-poetry.org | python - -y --version 1.5.1

compat/agora-agent-server-sdk/pyproject.toml

@@ -13,7 +13,6 @@ classifiers = [
     "Intended Audience :: Developers",
     "Programming Language :: Python",
     "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.8",
     "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
@@ -34,7 +33,7 @@ packages = [
 Repository = 'https://github.com/AgoraIO-Conversational-AI/agent-server-sdk-python'
 
 [tool.poetry.dependencies]
-python = "^3.8"
+python = "^3.9"
 agora-agents = ">=2.2.0,<3.0.0"
 
 [build-system]

poetry.lock

@@ -13,7 +13,6 @@ classifiers = [
     "Intended Audience :: Developers",
     "Programming Language :: Python",
     "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.8",
     "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
@@ -34,7 +33,7 @@ packages = [
 Repository = 'https://github.com/AgoraIO-Conversational-AI/agent-server-sdk-python'
 
 [tool.poetry.dependencies]
-python = "^3.8"
+python = "^3.9"
 httpx = ">=0.21.2"
 pydantic = ">= 1.9.2"
 pydantic-core = ">=2.18.2"

pyproject.toml

@@ -12,14 +12,12 @@
 if typing.TYPE_CHECKING:
     from . import agents, agentkit, core, phone_numbers, telephony
     from .core.domain import Area, Pool, create_pool
-    from .pool_client import Agora, AsyncAgora
+    from .pool_client import Agora, AsyncAgora, AgentClient, AsyncAgentClient
     from .version import __version__
     from .agentkit import (
         Agent,
         AgentSession,
         AgentSessionOptions,
-        AgentClient,
-        AsyncAgentClient,
         CNAgent,
         GlobalAgent,
         GenericAvatar,

src/agora_agent/__init__.py

@@ -350,54 +350,6 @@ class Agent:
     ... )
     """
 
-    if typing.TYPE_CHECKING:
-        _GlobalArea = typing_extensions.Literal[Area.US, Area.EU, Area.AP]
-
-        @typing.overload
-        def __new__(
-            cls,
-            client: "Agora[typing_extensions.Literal[Area.CN]]",
-            *args: typing.Any,
-            **kwargs: typing.Any,
-        ) -> "CNAgent":
-            ...
-
-        @typing.overload
-        def __new__(
-            cls,
-            client: "Agora[_GlobalArea]",
-            *args: typing.Any,
-            **kwargs: typing.Any,
-        ) -> "GlobalAgent":
-            ...
-
-        @typing.overload
-        def __new__(
-            cls,
-            client: "AsyncAgora[typing_extensions.Literal[Area.CN]]",
-            *args: typing.Any,
-            **kwargs: typing.Any,
-        ) -> "CNAgent":
-            ...
-
-        @typing.overload
-        def __new__(
-            cls,
-            client: "AsyncAgora[_GlobalArea]",
-            *args: typing.Any,
-            **kwargs: typing.Any,
-        ) -> "GlobalAgent":
-            ...
-
-        @typing.overload
-        def __new__(
-            cls,
-            client: typing.Any,
-            *args: typing.Any,
-            **kwargs: typing.Any,
-        ) -> "Agent":
-            ...
-
     def __new__(
         cls,
         client: typing.Any = None,
@@ -422,7 +374,7 @@ def __init__(
         self,
         client: typing.Any,
         instructions: typing.Optional[str] = None,
-        turn_detection: typing.Optional[TurnDetectionConfig] = None,
+        turn_detection: typing.Optional[typing.Union[TurnDetectionConfig, typing.Dict[str, typing.Any]]] = None,
         interruption: typing.Optional[InterruptionConfig] = None,
         sal: typing.Optional[SalConfig] = None,
         advanced_features: typing.Optional[AdvancedFeatures] = None,
@@ -733,7 +685,7 @@ def mllm(self) -> typing.Optional[typing.Dict[str, typing.Any]]:
         return self._mllm
 
     @property
-    def turn_detection(self) -> typing.Optional[TurnDetectionConfig]:
+    def turn_detection(self) -> typing.Optional[typing.Union[TurnDetectionConfig, typing.Dict[str, typing.Any]]]:
         return self._turn_detection
 
     @property
@@ -1072,7 +1024,10 @@ def _resolve_llm_config(self) -> typing.Dict[str, typing.Any]:
             llm_config["max_history"] = self._max_history
         return llm_config
 
-    def _resolve_asr_config(self, turn_detection_config: TurnDetectionConfig) -> typing.Dict[str, typing.Any]:
+    def _resolve_asr_config(
+        self,
+        turn_detection_config: typing.Union[TurnDetectionConfig, typing.Dict[str, typing.Any]],
+    ) -> typing.Dict[str, typing.Any]:
         asr_config = dict(self._stt or {})
         if not asr_config:
             asr_config["vendor"] = "ares"

src/agora_agent/agentkit/agent.py

@@ -108,30 +108,30 @@
 
 
 class CNAgent(Agent):
-    def with_stt(self, vendor: CNSTT) -> "CNAgent":
+    def with_stt(self, vendor: CNSTT) -> "CNAgent":  # type: ignore[override]
         return typing.cast("CNAgent", super().with_stt(typing.cast(BaseSTT, vendor)))
 
-    def with_llm(self, vendor: CNLLM) -> "CNAgent":
+    def with_llm(self, vendor: CNLLM) -> "CNAgent":  # type: ignore[override]
         return typing.cast("CNAgent", super().with_llm(typing.cast(BaseLLM, vendor)))
 
-    def with_tts(self, vendor: CNTTS) -> "CNAgent":
+    def with_tts(self, vendor: CNTTS) -> "CNAgent":  # type: ignore[override]
         return typing.cast("CNAgent", super().with_tts(typing.cast(BaseTTS, vendor)))
 
-    def with_avatar(self, vendor: CNAvatar) -> "CNAgent":
+    def with_avatar(self, vendor: CNAvatar) -> "CNAgent":  # type: ignore[override]
         return typing.cast("CNAgent", super().with_avatar(typing.cast(BaseAvatar, vendor)))
 
 
 class GlobalAgent(Agent):
-    def with_stt(self, vendor: GlobalSTT) -> "GlobalAgent":
+    def with_stt(self, vendor: GlobalSTT) -> "GlobalAgent":  # type: ignore[override]
         return typing.cast("GlobalAgent", super().with_stt(typing.cast(BaseSTT, vendor)))
 
-    def with_llm(self, vendor: GlobalLLM) -> "GlobalAgent":
+    def with_llm(self, vendor: GlobalLLM) -> "GlobalAgent":  # type: ignore[override]
         return typing.cast("GlobalAgent", super().with_llm(typing.cast(BaseLLM, vendor)))
 
-    def with_tts(self, vendor: GlobalTTS) -> "GlobalAgent":
+    def with_tts(self, vendor: GlobalTTS) -> "GlobalAgent":  # type: ignore[override]
         return typing.cast("GlobalAgent", super().with_tts(typing.cast(BaseTTS, vendor)))
 
-    def with_avatar(self, vendor: GlobalAvatar) -> "GlobalAgent":
+    def with_avatar(self, vendor: GlobalAvatar) -> "GlobalAgent":  # type: ignore[override]
         return typing.cast("GlobalAgent", super().with_avatar(typing.cast(BaseAvatar, vendor)))