Coerce inspect.getdoc() None to empty string before parsing
`inspect.getdoc()` returns `str | None`, but `docstring_parser.parse()`
requires `str`. Functions without a docstring produced `None`, which
the type checker correctly flagged.
Coerce to `""` at the call site. `docstring_parser.parse("")` returns
an empty docstring whose `.description` and `.params` are already
handled by the surrounding `or ""` fallbacks, so runtime behavior is
unchanged.
This commit is contained in:
@@ -2,11 +2,16 @@
|
|||||||
"typeCheckingMode": "basic",
|
"typeCheckingMode": "basic",
|
||||||
"pythonVersion": "3.11",
|
"pythonVersion": "3.11",
|
||||||
"pythonPlatform": "All",
|
"pythonPlatform": "All",
|
||||||
"include": ["scripts", "src/pipecat"],
|
"include": [
|
||||||
"exclude": ["**/*_pb2.py", "**/__pycache__"],
|
"scripts",
|
||||||
|
"src/pipecat"
|
||||||
|
],
|
||||||
|
"exclude": [
|
||||||
|
"**/*_pb2.py",
|
||||||
|
"**/__pycache__"
|
||||||
|
],
|
||||||
"ignore": [
|
"ignore": [
|
||||||
"tests",
|
"tests",
|
||||||
"src/pipecat/adapters/schemas/direct_function.py",
|
|
||||||
"src/pipecat/adapters/services/anthropic_adapter.py",
|
"src/pipecat/adapters/services/anthropic_adapter.py",
|
||||||
"src/pipecat/adapters/services/aws_nova_sonic_adapter.py",
|
"src/pipecat/adapters/services/aws_nova_sonic_adapter.py",
|
||||||
"src/pipecat/adapters/services/bedrock_adapter.py",
|
"src/pipecat/adapters/services/bedrock_adapter.py",
|
||||||
|
|||||||
@@ -127,7 +127,7 @@ class BaseDirectFunctionWrapper:
|
|||||||
self.name = self.function.__name__
|
self.name = self.function.__name__
|
||||||
|
|
||||||
# Parse docstring for description and parameters
|
# Parse docstring for description and parameters
|
||||||
docstring = docstring_parser.parse(inspect.getdoc(self.function))
|
docstring = docstring_parser.parse(inspect.getdoc(self.function) or "")
|
||||||
|
|
||||||
# Get function description
|
# Get function description
|
||||||
self.description = (docstring.description or "").strip()
|
self.description = (docstring.description or "").strip()
|
||||||
|
|||||||
Reference in New Issue
Block a user