Python: Fix FC stepwise planner. (microsoft#6357)

### Motivation and Context

The FC stepwise planner was breaking when trying to process the function
call result because if a filter is not configured, then the context will
be None. We need to only try to extract the FC result if the context is
not none, otherwise it will already be present in the chat history.

<!-- 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

Check that the FC result context is not None, if so, then extract the FC
result, otherwise proceed as the FC result is in the chat history. Fixes
microsoft#6350.
- Removes some extranous logging from json schema addition.

<!-- 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 😄

Author: moonbox3

python/samples/concepts/plugins/openai_function_calling_with_custom_plugin.py

@@ -1,6 +1,5 @@
 # Copyright (c) Microsoft. All rights reserved.
 
-
 import asyncio
 from typing import Annotated
 

python/semantic_kernel/planners/function_calling_stepwise_planner/function_calling_stepwise_planner.py

@@ -196,10 +196,13 @@ async def invoke(
                         request_index=0,
                         function_call_behavior=prompt_execution_settings.function_call_behavior,
                     )
-                    frc = FunctionResultContent.from_function_call_content_and_result(
-                        function_call_content=item, result=context.function_result
-                    )
-                    chat_history_for_steps.add_message(message=frc.to_chat_message_content())
+                    if context is not None:
+                        # Only add the function result content to the chat history if the context is present
+                        # which means it wasn't added in the _process_function_call method
+                        frc = FunctionResultContent.from_function_call_content_and_result(
+                            function_call_content=item, result=context.function_result
+                        )
+                        chat_history_for_steps.add_message(message=frc.to_chat_message_content())
                 except Exception as exc:
                     frc = FunctionResultContent.from_function_call_content_and_result(
                         function_call_content=item,

python/semantic_kernel/prompt_template/utils/jinja2_system_helpers.py

@@ -61,7 +61,6 @@ def _double_close():
 
 
 def _array(*args, **kwargs):
-    print(f"Received args: {args}")
     return list(args)
 
 

python/semantic_kernel/schema/kernel_json_schema_builder.py

@@ -26,7 +26,6 @@ class KernelJsonSchemaBuilder:
     @classmethod
     def build(cls, parameter_type: type | str, description: str | None = None) -> dict[str, Any]:
         """Builds JSON schema for a given parameter type."""
-        print(f"Building schema for type: {parameter_type}")
 
         if isinstance(parameter_type, str):
             return cls.build_from_type_name(parameter_type, description)
@@ -56,7 +55,6 @@ def build_model_schema(cls, model: type, description: str | None = None) -> dict
         if description:
             schema["description"] = description
 
-        print(f"Generated schema for model {model}: {schema}")
         return schema
 
     @classmethod
@@ -67,13 +65,11 @@ def build_from_type_name(cls, parameter_type: str, description: str | None = Non
         if description:
             schema["description"] = description
 
-        print(f"Generated schema from type name {parameter_type}: {schema}")
         return schema
 
     @classmethod
     def get_json_schema(cls, parameter_type: type) -> dict[str, Any]:
         """Gets JSON schema for a given parameter type."""
         type_name = TYPE_MAPPING.get(parameter_type, "object")
         schema = {"type": type_name}
-        print(f"Generated JSON schema for type {parameter_type}: {schema}")
         return schema

python/tests/integration/planning/function_calling_stepwise_planner/test_int_function_calling_stepwise_planner.py

@@ -19,7 +19,6 @@
 
 
 @pytest.mark.asyncio
-@pytest.mark.xfail(reason="This test is flaky and needs investigation.")
 async def test_can_execute_function_calling_stepwise_plan(kernel: Kernel):
 
     service_id = "planner"
@@ -47,4 +46,4 @@ async def test_can_execute_function_calling_stepwise_plan(kernel: Kernel):
         result = await planner.invoke(kernel, question)
         print(f"Q: {question}\nA: {result.final_answer}\n")
         assert isinstance(result, FunctionCallingStepwisePlannerResult)
-        assert 0 < len(result.final_answer) < 100
+        assert 0 < len(result.final_answer)