wx44wx/pipecat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(types): widen TLLMInvocationParams bound to Mapping[str, Any]

Browse Source 
TypedDict types are not subtypes of dict[...] in the type system
(per PEP 589), so TypedDict-based invocation param classes could not
satisfy the TypeVar bound. Mapping[str, Any] accepts TypedDicts while
preserving the "string-keyed mapping" constraint.
This commit is contained in:
Paul Kompfner
2026-04-23 14:35:59 -04:00
parent b90ea9bf6a
commit 092b1dcb0f
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/pipecat/adapters/base_llm_adapter.py
View File

@@ -12,6 +12,7 @@ adapters that handle tool format conversion and standardization.
import warnings import warnings
from abc import ABC, abstractmethod from abc import ABC, abstractmethod
from collections.abc import Mapping
from typing import Any, Generic, TypeVar from typing import Any, Generic, TypeVar
from loguru import logger from loguru import logger
@@ -26,7 +27,7 @@ from pipecat.processors.aggregators.llm_context import (
) )
# Should be a TypedDict # Should be a TypedDict
TLLMInvocationParams = TypeVar("TLLMInvocationParams", bound=dict[str, Any]) TLLMInvocationParams = TypeVar("TLLMInvocationParams", bound=Mapping[str, Any])
class BaseLLMAdapter(ABC, Generic[TLLMInvocationParams]): class BaseLLMAdapter(ABC, Generic[TLLMInvocationParams]):