Skip to content

Commit f4e432f

Browse files
stephentoubCopilot
andcommitted
Fix SDK compile after verbosity schema update
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent 700d278 commit f4e432f

9 files changed

Lines changed: 13 additions & 8 deletions

File tree

dotnet/src/Session.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1787,6 +1787,7 @@ await Rpc.Model.SwitchToAsync(
17871787
model,
17881788
options.ReasoningEffort,
17891789
options.ReasoningSummary,
1790+
null,
17901791
options.ModelCapabilities,
17911792
options.ContextTier,
17921793
cancellationToken);

java/src/main/java/com/github/copilot/CopilotClient.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -926,6 +926,7 @@ CompletableFuture<Void> updateSessionOptionsForMode(CopilotSession session, Bool
926926
null, // modelCapabilitiesOverrides
927927
null, // reasoningEffort
928928
null, // reasoningSummary
929+
null, // verbosity
929930
null, // clientName
930931
null, // lspClientName
931932
null, // integrationId

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1927,7 +1927,7 @@ public CompletableFuture<Void> abort() {
19271927
public CompletableFuture<Void> setModel(String model, String reasoningEffort) {
19281928
ensureNotTerminated();
19291929
return getRpc().model
1930-
.switchTo(new SessionModelSwitchToParams(sessionId, model, reasoningEffort, null, null, null))
1930+
.switchTo(new SessionModelSwitchToParams(sessionId, model, reasoningEffort, null, null, null, null))
19311931
.thenApply(r -> null);
19321932
}
19331933

@@ -2008,7 +2008,7 @@ public CompletableFuture<Void> setModel(String model, String reasoningEffort, St
20082008
? null
20092009
: com.github.copilot.generated.rpc.ReasoningSummary.fromValue(reasoningSummary);
20102010
return getRpc().model.switchTo(new SessionModelSwitchToParams(sessionId, model, reasoningEffort,
2011-
generatedReasoningSummary, generatedCapabilities, null)).thenApply(r -> null);
2011+
generatedReasoningSummary, null, generatedCapabilities, null)).thenApply(r -> null);
20122012
}
20132013

20142014
/**

java/src/test/java/com/github/copilot/RpcSessionStateExtrasE2ETest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ void testShouldAddByokProviderAndModelAtRuntime() throws Exception {
6565

6666
var selectionId = "java-e2e-provider/small";
6767
session.getRpc().model
68-
.switchTo(new SessionModelSwitchToParams(null, selectionId, null, null, null, null))
68+
.switchTo(new SessionModelSwitchToParams(null, selectionId, null, null, null, null, null))
6969
.get(30, TimeUnit.SECONDS);
7070
var current = session.getRpc().model.getCurrent().get(30, TimeUnit.SECONDS);
7171
assertEquals(selectionId, current.modelId());

java/src/test/java/com/github/copilot/RpcWrappersTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ void sessionRpc_model_switchTo_merges_sessionId_with_extra_params() {
205205
var session = new SessionRpc(stub, "sess-xyz");
206206

207207
// switchTo takes extra params beyond sessionId
208-
var switchParams = new SessionModelSwitchToParams(null, "gpt-5", null, null, null, null);
208+
var switchParams = new SessionModelSwitchToParams(null, "gpt-5", null, null, null, null, null);
209209
session.model.switchTo(switchParams);
210210

211211
assertEquals(1, stub.calls.size());

java/src/test/java/com/github/copilot/SessionEventHandlingTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ void testHandlerReceivesCorrectEventData() {
180180

181181
SessionStartEvent startEvent = createSessionStartEvent();
182182
startEvent.setData(new SessionStartEvent.SessionStartEventData("my-session-123", null, null, null, null, null,
183-
null, null, null, null, null, null, null, null));
183+
null, null, null, null, null, null, null, null, null));
184184
dispatchEvent(startEvent);
185185

186186
AssistantMessageEvent msgEvent = createAssistantMessageEvent("Test content");
@@ -857,7 +857,7 @@ private SessionStartEvent createSessionStartEvent() {
857857
private SessionStartEvent createSessionStartEvent(String sessionId) {
858858
var event = new SessionStartEvent();
859859
var data = new SessionStartEvent.SessionStartEventData(sessionId, null, null, null, null, null, null, null,
860-
null, null, null, null, null, null);
860+
null, null, null, null, null, null, null);
861861
event.setData(data);
862862
return event;
863863
}

java/src/test/java/com/github/copilot/generated/rpc/GeneratedRpcRecordsCoverageTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -321,11 +321,12 @@ void sessionModelGetCurrentParams_record() {
321321

322322
@Test
323323
void sessionModelSwitchToParams_record() {
324-
var params = new SessionModelSwitchToParams("sess-32", "claude-sonnet-4.5", "high", null, null, null);
324+
var params = new SessionModelSwitchToParams("sess-32", "claude-sonnet-4.5", "high", null, null, null, null);
325325
assertEquals("sess-32", params.sessionId());
326326
assertEquals("claude-sonnet-4.5", params.modelId());
327327
assertEquals("high", params.reasoningEffort());
328328
assertNull(params.reasoningSummary());
329+
assertNull(params.verbosity());
329330
assertNull(params.modelCapabilities());
330331
}
331332

@@ -836,7 +837,7 @@ void sessionModelSwitchToParams_nested_records() {
836837
var limits = new ModelCapabilitiesOverrideLimits(100000L, 8192L, 128000L, limitsVision);
837838
var supports = new ModelCapabilitiesOverrideSupports(true, true, null);
838839
var capabilities = new ModelCapabilitiesOverride(supports, limits);
839-
var params = new SessionModelSwitchToParams("sess-m", "gpt-5", null, null, capabilities, null);
840+
var params = new SessionModelSwitchToParams("sess-m", "gpt-5", null, null, null, capabilities, null);
840841

841842
assertEquals("gpt-5", params.modelId());
842843
assertNotNull(params.modelCapabilities());

rust/src/session.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -529,6 +529,7 @@ impl Session {
529529
model_id: model.to_string(),
530530
reasoning_effort: opts.reasoning_effort,
531531
reasoning_summary: opts.reasoning_summary,
532+
verbosity: None,
532533
context_tier: opts.context_tier,
533534
model_capabilities: opts.model_capabilities,
534535
};

rust/tests/e2e/rpc_session_state_extras.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -319,6 +319,7 @@ async fn should_add_byok_provider_and_model_at_runtime() {
319319
model_id: selection_id.to_string(),
320320
reasoning_effort: None,
321321
reasoning_summary: None,
322+
verbosity: None,
322323
})
323324
.await
324325
.expect("switch to added model");

0 commit comments

Comments
 (0)