mirror of
https://github.com/microsoft/semantic-kernel.git
synced 2026-10-12 02:19:06 +00:00
### Motivation and Context Solves #4864 The KernelFunction was configured as a Python class without regards to using Pydantic fields and its validations. As an aside, while running a kernel example related to function/tool calling, it didn't succeed. <!-- Thank you for your contribution to the semantic-kernel repo! Please help reviewers and future users, providing the following information: 1. Why is this change required? 2. What problem does it solve? 3. What scenario does it contribute to? 4. If it fixes an open issue, please link to the issue here. --> ### Description This PR cleans up the KernelFunction by creating Pydantic fields. We're also able to remove the KernelFunctionBase as it didn't contain a lot of logic after removing the properties. While testing other code, I found that the tool call kernel example wasn't succeeding. The tool_call payload needed a small reformat, and we needed to handle the tool_call_id and correctly handle the ToolCall object. As further work towards v1, we should look at removing support for FunctionCall as it is deprecated in Azure OpenAI. We now see: ``` User:> What is 3+3? Mosscap:> Ah, the timeless question of arithmetic! The sum of 3 and 3 is undoubtedly a splendid number. By utilizing my mathematical prowess, I can confidently declare that 3 plus 3 equals 6. It is a harmonious union of two threes, joining forces to create a new entity, greater in magnitude. The addition of these two numbers results in a perfect balance, a symphony of numerical harmony. ``` <!-- Describe your changes, the overall approach, the underlying design. These notes will help understanding how your code works. Thanks! --> ### Contribution Checklist <!-- Before submitting this PR, please make sure: --> - [X] The code builds clean without any errors or warnings - [X] The PR follows the [SK Contribution Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md) and the [pre-submission formatting script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts) raises no violations - [X] All unit tests pass, and I have added new tests where possible - [X] I didn't break anyone 😄 --------- Co-authored-by: Evan Mattson <[email protected]>
262 lines
9.0 KiB
Python
262 lines
9.0 KiB
Python
import typing as t
|
|
|
|
import pytest
|
|
import typing_extensions as te
|
|
from pydantic import Field, Json
|
|
|
|
from semantic_kernel.core_plugins.conversation_summary_plugin import (
|
|
ConversationSummaryPlugin,
|
|
)
|
|
from semantic_kernel.core_plugins.file_io_plugin import FileIOPlugin
|
|
from semantic_kernel.core_plugins.http_plugin import HttpPlugin
|
|
from semantic_kernel.core_plugins.math_plugin import MathPlugin
|
|
from semantic_kernel.core_plugins.text_memory_plugin import TextMemoryPlugin
|
|
from semantic_kernel.core_plugins.text_plugin import TextPlugin
|
|
from semantic_kernel.core_plugins.time_plugin import TimePlugin
|
|
from semantic_kernel.core_plugins.wait_plugin import WaitPlugin
|
|
from semantic_kernel.core_plugins.web_search_engine_plugin import WebSearchEnginePlugin
|
|
from semantic_kernel.kernel_pydantic import KernelBaseModel
|
|
from semantic_kernel.memory.null_memory import NullMemory
|
|
from semantic_kernel.memory.semantic_text_memory_base import SemanticTextMemoryBase
|
|
from semantic_kernel.orchestration.context_variables import ContextVariables
|
|
from semantic_kernel.orchestration.delegate_handlers import DelegateHandlers
|
|
from semantic_kernel.orchestration.delegate_inference import DelegateInference
|
|
from semantic_kernel.orchestration.kernel_context import KernelContext
|
|
from semantic_kernel.orchestration.kernel_function import KernelFunction
|
|
from semantic_kernel.plugin_definition.function_view import FunctionView
|
|
from semantic_kernel.plugin_definition.functions_view import FunctionsView
|
|
from semantic_kernel.plugin_definition.kernel_function_decorator import kernel_function
|
|
from semantic_kernel.plugin_definition.kernel_plugin_collection import (
|
|
KernelPluginCollection,
|
|
)
|
|
from semantic_kernel.plugin_definition.parameter_view import ParameterView
|
|
from semantic_kernel.template_engine.blocks.block import Block
|
|
from semantic_kernel.template_engine.blocks.block_types import BlockTypes
|
|
from semantic_kernel.template_engine.blocks.code_block import CodeBlock
|
|
from semantic_kernel.template_engine.blocks.function_id_block import FunctionIdBlock
|
|
from semantic_kernel.template_engine.blocks.text_block import TextBlock
|
|
from semantic_kernel.template_engine.blocks.val_block import ValBlock
|
|
from semantic_kernel.template_engine.blocks.var_block import VarBlock
|
|
from semantic_kernel.template_engine.code_tokenizer import CodeTokenizer
|
|
from semantic_kernel.template_engine.prompt_template_engine import PromptTemplateEngine
|
|
from semantic_kernel.template_engine.protocols.code_renderer import CodeRenderer
|
|
from semantic_kernel.template_engine.protocols.prompt_templating_engine import (
|
|
PromptTemplatingEngine,
|
|
)
|
|
from semantic_kernel.template_engine.protocols.text_renderer import TextRenderer
|
|
from semantic_kernel.template_engine.template_tokenizer import TemplateTokenizer
|
|
|
|
KernelBaseModelFieldT = t.TypeVar("KernelBaseModelFieldT", bound=KernelBaseModel)
|
|
|
|
|
|
class _Serializable(t.Protocol):
|
|
"""A serializable object."""
|
|
|
|
def json(self) -> Json:
|
|
"""Return a JSON representation of the object."""
|
|
raise NotImplementedError
|
|
|
|
@classmethod
|
|
def parse_raw(cls: t.Type[te.Self], json: Json) -> te.Self:
|
|
"""Return the constructed object from a JSON representation."""
|
|
raise NotImplementedError
|
|
|
|
|
|
@pytest.fixture()
|
|
def kernel_factory() -> t.Callable[[t.Type[_Serializable]], _Serializable]:
|
|
"""Return a factory for various objects in semantic-kernel."""
|
|
|
|
def create_functions_view() -> FunctionsView:
|
|
"""Return a functions view."""
|
|
result = FunctionsView()
|
|
result.add_function(
|
|
FunctionView(
|
|
name="function1",
|
|
plugin_name="plugin1",
|
|
description="Native function",
|
|
parameters=[],
|
|
is_semantic=False,
|
|
is_asynchronous=True,
|
|
)
|
|
)
|
|
result.add_function(
|
|
FunctionView(
|
|
name="function1",
|
|
plugin_name="plugin1",
|
|
description="Semantic function",
|
|
parameters=[],
|
|
is_semantic=True,
|
|
is_asynchronous=True,
|
|
)
|
|
)
|
|
return result
|
|
|
|
def create_kernel_function() -> KernelFunction:
|
|
"""Return an KernelFunction."""
|
|
|
|
@kernel_function(name="function")
|
|
def my_function(cx: KernelContext) -> str:
|
|
return f"F({cx.variables.input})"
|
|
|
|
return KernelFunction.from_native_method(my_function, "plugin")
|
|
|
|
def create_context_variables() -> ContextVariables:
|
|
"""Return a context variables object."""
|
|
return ContextVariables(
|
|
content="content",
|
|
variables={"foo": "bar"},
|
|
)
|
|
|
|
def create_plugin_collection() -> KernelPluginCollection:
|
|
"""Return a plugin collection."""
|
|
# TODO: Add a few plugins to this collection.
|
|
return KernelPluginCollection()
|
|
|
|
cls_obj_map = {
|
|
Block: Block(content="foo"),
|
|
CodeBlock: CodeBlock(content="foo"),
|
|
FunctionIdBlock: FunctionIdBlock(content="bar"),
|
|
TextBlock: TextBlock(content="baz"),
|
|
ValBlock: ValBlock(content="qux"),
|
|
VarBlock: VarBlock(content="quux"),
|
|
CodeTokenizer: CodeTokenizer(),
|
|
PromptTemplateEngine: PromptTemplateEngine(),
|
|
TemplateTokenizer: TemplateTokenizer(),
|
|
ParameterView: ParameterView(
|
|
name="foo",
|
|
description="bar",
|
|
default_value="baz",
|
|
type="string",
|
|
required=True,
|
|
),
|
|
FunctionView: FunctionView(
|
|
"foo",
|
|
"bar",
|
|
"baz",
|
|
[ParameterView(name="qux", description="bar", default_value="baz")],
|
|
True,
|
|
False,
|
|
),
|
|
FunctionsView: create_functions_view(),
|
|
KernelPluginCollection: create_plugin_collection(),
|
|
DelegateHandlers: DelegateHandlers(),
|
|
DelegateInference: DelegateInference(),
|
|
ContextVariables: create_context_variables(),
|
|
KernelContext[NullMemory]: KernelContext[NullMemory](
|
|
# TODO: Test serialization with different types of memories.
|
|
variables=create_context_variables(),
|
|
memory=NullMemory(),
|
|
plugins=create_plugin_collection(),
|
|
),
|
|
NullMemory: NullMemory(),
|
|
KernelFunction: create_kernel_function(),
|
|
}
|
|
|
|
def constructor(cls: t.Type[_Serializable]) -> _Serializable:
|
|
"""Return a serializable object."""
|
|
return cls_obj_map[cls]
|
|
|
|
return constructor
|
|
|
|
|
|
PROTOCOLS = [
|
|
pytest.param(ConversationSummaryPlugin, marks=pytest.mark.xfail(reason="Contains data")),
|
|
FileIOPlugin,
|
|
HttpPlugin,
|
|
MathPlugin,
|
|
TextMemoryPlugin,
|
|
TextPlugin,
|
|
TimePlugin,
|
|
WaitPlugin,
|
|
pytest.param(WebSearchEnginePlugin, marks=pytest.mark.xfail(reason="Contains data")),
|
|
CodeRenderer,
|
|
PromptTemplatingEngine,
|
|
TextRenderer,
|
|
]
|
|
|
|
BASE_CLASSES = [
|
|
SemanticTextMemoryBase,
|
|
]
|
|
|
|
STATELESS_CLASSES = [
|
|
CodeTokenizer,
|
|
PromptTemplateEngine,
|
|
TemplateTokenizer,
|
|
DelegateHandlers,
|
|
DelegateInference,
|
|
NullMemory,
|
|
]
|
|
|
|
ENUMS = [
|
|
BlockTypes,
|
|
DelegateInference,
|
|
]
|
|
|
|
PYDANTIC_MODELS = [
|
|
Block,
|
|
CodeBlock,
|
|
FunctionIdBlock,
|
|
TextBlock,
|
|
ValBlock,
|
|
VarBlock,
|
|
ParameterView,
|
|
FunctionView,
|
|
FunctionsView,
|
|
KernelPluginCollection,
|
|
ContextVariables,
|
|
KernelContext[NullMemory],
|
|
pytest.param(
|
|
KernelFunction,
|
|
marks=pytest.mark.xfail(reason="Need to implement Pickle serialization."),
|
|
),
|
|
]
|
|
|
|
|
|
class TestUsageInPydanticFields:
|
|
@pytest.mark.parametrize(
|
|
"kernel_type",
|
|
BASE_CLASSES + PROTOCOLS + ENUMS + PYDANTIC_MODELS + STATELESS_CLASSES,
|
|
)
|
|
def test_usage_as_optional_field(
|
|
self,
|
|
kernel_type: t.Type[KernelBaseModelFieldT],
|
|
) -> None:
|
|
"""Semantic Kernel objects should be valid Pydantic fields.
|
|
|
|
Otherwise, they cannot be used in Pydantic models.
|
|
"""
|
|
|
|
class TestModel(KernelBaseModel):
|
|
"""A test model."""
|
|
|
|
field: t.Optional[kernel_type] = None
|
|
|
|
assert_serializable(TestModel(), TestModel)
|
|
|
|
@pytest.mark.parametrize("kernel_type", PYDANTIC_MODELS + STATELESS_CLASSES)
|
|
def test_usage_as_required_field(
|
|
self,
|
|
kernel_factory: t.Callable[[t.Type[KernelBaseModelFieldT]], KernelBaseModelFieldT],
|
|
kernel_type: t.Type[KernelBaseModelFieldT],
|
|
) -> None:
|
|
"""Semantic Kernel objects should be valid Pydantic fields.
|
|
|
|
Otherwise, they cannot be used in Pydantic models.
|
|
"""
|
|
|
|
class TestModel(KernelBaseModel):
|
|
"""A test model."""
|
|
|
|
field: kernel_type = Field(default_factory=lambda: kernel_factory(kernel_type))
|
|
|
|
assert_serializable(TestModel(), TestModel)
|
|
assert_serializable(TestModel(field=kernel_factory(kernel_type)), TestModel)
|
|
|
|
|
|
def assert_serializable(obj: _Serializable, obj_type) -> None:
|
|
"""Assert that an object is serializable, uses both dump and dump_json methods."""
|
|
assert obj is not None
|
|
serialized = obj.model_dump_json()
|
|
assert isinstance(serialized, str)
|
|
assert obj_type.model_validate_json(serialized).model_dump() == obj.model_dump()
|