Updates

Author: Milder Hernandez Cagua

agents/semantickernel-agents-core/src/main/java/com/microsoft/semantickernel/agents/chatcompletion/ChatCompletionAgent.java

@@ -110,7 +110,7 @@ private Mono<List<ChatMessageContent<?>>> internalInvokeAsync(
                     ? invocationContext.getPromptExecutionSettings()
                     : kernelArguments.getExecutionSettings().get(chatCompletionService.getServiceId());
 
-            final InvocationContext newMessagesContext = InvocationContext.builder()
+            final InvocationContext updatedInvocationContext = InvocationContext.builder()
                     .withPromptExecutionSettings(executionSettings)
                     .withToolCallBehavior(invocationContext.getToolCallBehavior())
                     .withTelemetry(invocationContext.getTelemetry())
@@ -119,7 +119,7 @@ private Mono<List<ChatMessageContent<?>>> internalInvokeAsync(
                     .withReturnMode(InvocationReturnMode.FULL_HISTORY)
                     .build();
 
-            return formatInstructionsAsync(kernel, arguments, newMessagesContext).flatMap(
+            return formatInstructionsAsync(kernel, arguments, updatedInvocationContext).flatMap(
                 instructions -> {
                     // Create a new chat history with the instructions
                     ChatHistory chat = new ChatHistory(
@@ -137,7 +137,7 @@ private Mono<List<ChatMessageContent<?>>> internalInvokeAsync(
                     chat.addAll(history);
                     int previousHistorySize = chat.getMessages().size();
 
-                    return chatCompletionService.getChatMessageContentsAsync(chat, kernel, newMessagesContext)
+                    return chatCompletionService.getChatMessageContentsAsync(chat, kernel, updatedInvocationContext)
                             .map(chatMessageContents -> {
                                 return chatMessageContents.subList(
                                         previousHistorySize,

samples/semantickernel-concepts/semantickernel-syntax-examples/src/main/java/com/microsoft/semantickernel/samples/plugins/github/GitHubPlugin.java

@@ -103,7 +103,7 @@ public Mono<List<GitHubModel.Issue>> getIssuesAsync(
                 });
     }
 
-    @DefineKernelFunction(name = "get_issue_detail_info", description = "Get issue detail information from GitHub",
+    @DefineKernelFunction(name = "get_issue_detail_info", description = "Get detail information of a single issue from GitHub",
             returnType = "com.microsoft.semantickernel.samples.plugins.github.GitHubModel$IssueDetail")
     public GitHubModel.IssueDetail getIssueDetailAsync(
             @KernelFunctionParameter(

samples/semantickernel-concepts/semantickernel-syntax-examples/src/main/java/com/microsoft/semantickernel/samples/syntaxexamples/agents/CompletionsAgent.java

@@ -34,7 +34,7 @@ public class CompletionsAgent {
     // Only required if AZURE_CLIENT_KEY is set
     private static final String CLIENT_ENDPOINT = System.getenv("CLIENT_ENDPOINT");
     private static final String MODEL_ID = System.getenv()
-            .getOrDefault("MODEL_ID", "gpt-35-turbo");
+            .getOrDefault("MODEL_ID", "gpt-4o");
 
     private static final String GITHUB_PAT = System.getenv("GITHUB_PAT");
     public static void main(String[] args) {
@@ -77,7 +77,7 @@ public static void main(String[] args) {
                 .build();
 
         InvocationContext invocationContext = InvocationContext.builder()
-                .withToolCallBehavior(ToolCallBehavior.allowAllKernelFunctions(true))
+                .withToolCallBehavior(ToolCallBehavior.allowAllKernelFunctions(false))
                 .build();
 
         ChatCompletionAgent agent = ChatCompletionAgent.builder()
@@ -127,7 +127,7 @@ public static void main(String[] args) {
                     .withVariable("now", System.currentTimeMillis())
                     .build();
 
-            var responses = agent.invokeAsync(
+            var response = agent.invokeAsync(
                     List.of(message),
                     agentThread,
                     AgentInvokeOptions.builder()
@@ -136,10 +136,7 @@ public static void main(String[] args) {
                             .build()
             ).block();
 
-            for (var response : responses) {
-                System.out.println("> " + response.getMessage());
-            }
+            System.out.println("> " + response.get(response.size() - 1).getMessage());
         }
     }
-
 }

semantickernel-api/src/main/java/com/microsoft/semantickernel/agents/Agent.java

@@ -53,8 +53,10 @@ public interface Agent {
      */
     Mono<List<AgentResponseItem<ChatMessageContent<?>>>> invokeAsync(List<ChatMessageContent<?>> messages, AgentThread thread, AgentInvokeOptions options);
 
-
-
-
+    /**
+     * Notifies the agent of a new message.
+     *
+     * @param thread The agent thread to use
+     */
     Mono<Void> notifyThreadOfNewMessageAsync(AgentThread thread, ChatMessageContent<?> newMessage);
 }

semantickernel-api/src/main/java/com/microsoft/semantickernel/agents/KernelAgent.java

@@ -2,6 +2,7 @@
 
 import com.microsoft.semantickernel.Kernel;
 import com.microsoft.semantickernel.orchestration.InvocationContext;
+import com.microsoft.semantickernel.orchestration.InvocationReturnMode;
 import com.microsoft.semantickernel.orchestration.PromptExecutionSettings;
 import com.microsoft.semantickernel.semanticfunctions.KernelArguments;
 import com.microsoft.semantickernel.semanticfunctions.PromptTemplate;
@@ -18,16 +19,12 @@
 public abstract class KernelAgent implements Agent {
 
     protected final String id;
-
     protected final String name;
-
     protected final String description;
-
     protected final Kernel kernel;
     protected final KernelArguments kernelArguments;
     protected final InvocationContext invocationContext;
     protected final String instructions;
-
     protected final PromptTemplate template;
 
     protected KernelAgent(
@@ -45,7 +42,8 @@ protected KernelAgent(
         this.description = description;
         this.kernel = kernel;
         this.kernelArguments = kernelArguments;
-        this.invocationContext = invocationContext;
+        this.invocationContext = invocationContext != null
+                ? invocationContext : InvocationContext.builder().withReturnMode(InvocationReturnMode.FULL_HISTORY).build();
         this.instructions = instructions;
         this.template = template;
     }