Skip to content

Python: fix(python): respect type alias and default type in KernelParameterMetadata schema inference #14705

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Hardik K (hardikkaurani) wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Python: fix(python): respect type alias and default type in KernelParameterMetadata schema inference #14705

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d8de1a0
test(python): add regression tests for KernelParameterMetadata type a…
hardikkaurani Nov 1, 2026
1bec7b8
fix(python): respect type alias and default type in KernelParameterMe…
hardikkaurani Nov 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion python/semantic_kernel/functions/kernel_parameter_metadata.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,9 @@ def form_schema(cls, data: Any) -> Any:
"""Create a schema for the parameter metadata."""
if isinstance(data, dict) and data.get("schema_data") is None:
type_object = data.get("type_object", None)
type_ = data.get("type_", None)
type_ = data.get("type_") or data.get("type")
if type_ is None and type_object is None and "type_" not in data and "type" not in data:
type_ = "str"
default_value = data.get("default_value", None)
description = data.get("description", None)
inferred_schema = cls.infer_schema(type_object, type_, default_value, description)
Expand Down
15 changes: 15 additions & 0 deletions python/tests/unit/functions/test_kernel_parameter_metadata.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ def test_kernel_parameter_metadata_init():
assert metadata.description == "description"
assert metadata.is_required is True
assert metadata.default_value == "default"
assert metadata.schema_data == {"type": "string", "description": "description (default value: default)"}


def test_kernel_parameter_metadata_valid():
Expand Down Expand Up @@ -65,3 +66,17 @@ def test_kernel_parameter_metadata_without_schema_data():
def test_kernel_parameter_metadata_with_partial_data():
metadata = KernelParameterMetadata(name="param5", type_="float", default_value=3.14)
assert metadata.schema_data == {"type": "number", "description": "(default value: 3.14)"}


def test_kernel_parameter_metadata_infer_schema_with_type_alias():
metadata = KernelParameterMetadata(
name="param_alias", type="int", default_value=42, description="An integer parameter"
)
assert metadata.type_ == "int"
assert metadata.schema_data == {"type": "integer", "description": "An integer parameter (default value: 42)"}


def test_kernel_parameter_metadata_infer_schema_with_default_type():
metadata = KernelParameterMetadata(name="param_default")
assert metadata.type_ == "str"
assert metadata.schema_data == {"type": "string"}
Loading