Skip to content

Commit f219ada

Browse files
Fix SessionModelSwitchToParams constructor calls after new reasoningSummary field
The generated SessionModelSwitchToParams record gained a new reasoningSummary field, changing the constructor from 4 to 5 parameters. Updated both call sites in CopilotSession to pass null for the new reasoningSummary parameter. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 86f4695 commit f219ada

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/main/java/com/github/copilot/sdk/CopilotSession.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1657,7 +1657,7 @@ public CompletableFuture<Void> abort() {
16571657
*/
16581658
public CompletableFuture<Void> setModel(String model, String reasoningEffort) {
16591659
ensureNotTerminated();
1660-
return getRpc().model.switchTo(new SessionModelSwitchToParams(sessionId, model, reasoningEffort, null))
1660+
return getRpc().model.switchTo(new SessionModelSwitchToParams(sessionId, model, reasoningEffort, null, null))
16611661
.thenApply(r -> null);
16621662
}
16631663

@@ -1706,7 +1706,7 @@ public CompletableFuture<Void> setModel(String model, String reasoningEffort,
17061706
generatedCapabilities = new ModelCapabilitiesOverride(supports, limits);
17071707
}
17081708
return getRpc().model
1709-
.switchTo(new SessionModelSwitchToParams(sessionId, model, reasoningEffort, generatedCapabilities))
1709+
.switchTo(new SessionModelSwitchToParams(sessionId, model, reasoningEffort, null, generatedCapabilities))
17101710
.thenApply(r -> null);
17111711
}
17121712

0 commit comments

Comments
 (0)