Skip to content

Commit 17a920e

Browse files
author
Milder Hernandez Cagua
committed
Fixes
1 parent 393d987 commit 17a920e

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

api-test/integration-tests/src/test/java/com/microsoft/semantickernel/tests/ResponseSchemaTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -160,15 +160,19 @@ private static void verifyCalled(OpenAIAsyncClient client, String expected) {
160160
writer,
161161
new JsonOptions()
162162
);
163+
JsonWriter format = chatCompletionsOptions.getResponseFormat()
164+
.toJson(jsonWriter);
163165
jsonWriter.flush();
164166
writer.flush();
165167

166168
String json = String.valueOf(writer.getBuffer())
167169
.replaceAll("\n", "")
170+
.replaceAll("\r", "")
168171
.replaceAll(" +", "");
169172
String expectedClean = expected
170173
.stripIndent()
171174
.replaceAll("\n", "")
175+
.replaceAll("\r", "")
172176
.replaceAll(" +", "");
173177

174178
return json.equals(expectedClean);

semantickernel-api/src/main/java/com/microsoft/semantickernel/semanticfunctions/KernelArguments.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -409,6 +409,8 @@ public Builder<U> withExecutionSettings(List<PromptExecutionSettings> executionS
409409
serviceId)
410410
);
411411
}
412+
413+
this.executionSettings.put(serviceId, settings);
412414
}
413415

414416
return this;

semantickernel-api/src/main/java/com/microsoft/semantickernel/semanticfunctions/KernelFunctionFromPrompt.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -131,13 +131,12 @@ private Flux<FunctionResult<T>> invokeInternalAsync(
131131

132132
LOGGER.info(SemanticKernelResources.getString("rendered.prompt"), prompt);
133133

134-
FunctionInvokingEvent updateArguments = kernelHooks
134+
FunctionInvokingEvent invokingEvent = kernelHooks
135135
.executeHooks(new FunctionInvokingEvent(this, args));
136136

137137
args = KernelArguments.builder()
138-
.withVariables(updateArguments.getArguments())
139-
.withExecutionSettings(
140-
this.getExecutionSettings())
138+
.withVariables(invokingEvent.getArguments())
139+
.withExecutionSettings(this.getExecutionSettings())
141140
.build();
142141

143142
AIServiceSelection<?> aiServiceSelection = kernel

0 commit comments

Comments
 (0)