Skip to content

Commit 21d738e

Browse files
github-actions[bot]stephentoubCopilot
authored
Update @github/copilot to 1.0.81-6 (#2367)
* Update @github/copilot to 1.0.81-6 - Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code * Fix SDK compatibility with updated CLI Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * Update tests for CLI 1.0.81-6 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * Re-enable in-process telemetry tests Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * Scope re-enabled tests to stdio Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * Limit re-enabled Node coverage Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * Keep unresolved 1.0.81 regressions guarded Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Stephen Toub <stoub@microsoft.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent ea41dad commit 21d738e

29 files changed

Lines changed: 188 additions & 217 deletions

File tree

dotnet/src/Client.cs

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1480,15 +1480,6 @@ public async Task<CopilotSession> ResumeSessionAsync(string sessionId, ResumeSes
14801480
session.SetCapabilities(response.Capabilities);
14811481
session.SetOpenCanvases(response.OpenCanvases);
14821482

1483-
if (config.McpServers is not null)
1484-
{
1485-
await InvokeRpcAsync<JsonElement>(
1486-
connection.Rpc,
1487-
"session.mcp.reloadWithConfig",
1488-
[new ReloadMcpServersRequest(sessionId, new ReloadMcpServersConfig(config.McpServers))],
1489-
cancellationToken);
1490-
}
1491-
14921483
if (config.OnMcpAuthRequest is not null)
14931484
{
14941485
await session.Rpc.EventLog.RegisterInterestAsync("mcp.oauth_required", cancellationToken);
@@ -2961,13 +2952,6 @@ internal record ResumeSessionResponse(
29612952
IList<OpenCanvasInstance>? OpenCanvases = null);
29622953
#pragma warning restore GHCP001
29632954

2964-
internal record ReloadMcpServersRequest(
2965-
string SessionId,
2966-
ReloadMcpServersConfig Config);
2967-
2968-
internal record ReloadMcpServersConfig(
2969-
IDictionary<string, McpServerConfig> McpServers);
2970-
29712955
internal record CommandWireDefinition(
29722956
string Name,
29732957
string Description);
@@ -3044,8 +3028,6 @@ internal record HooksInvokeResponse(
30443028
[JsonSerializable(typeof(CapiSessionOptions))]
30453029
[JsonSerializable(typeof(NamedProviderConfig))]
30463030
[JsonSerializable(typeof(ProviderModelConfig))]
3047-
[JsonSerializable(typeof(ReloadMcpServersConfig))]
3048-
[JsonSerializable(typeof(ReloadMcpServersRequest))]
30493031
[JsonSerializable(typeof(SessionLimitsConfig))]
30503032
[JsonSerializable(typeof(ResumeSessionRequest))]
30513033
[JsonSerializable(typeof(ResumeSessionResponse))]

dotnet/src/Generated/Rpc.cs

Lines changed: 13 additions & 8 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dotnet/test/E2E/RpcServerMiscE2ETests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ public async Task Should_Login_List_GetCurrentAuth_And_Logout_Account()
9292
var initial = await client.Rpc.Account.GetCurrentAuthAsync();
9393
Assert.Null(initial.AuthInfo);
9494

95-
var loginResult = await client.Rpc.Account.LoginAsync("https://github.com", login, token);
95+
var loginResult = await client.Rpc.Account.LoginAsync("https://github.com", token, login);
9696
Assert.NotNull(loginResult);
9797

9898
var current = await client.Rpc.Account.GetCurrentAuthAsync();

go/client.go

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1356,19 +1356,6 @@ func (c *Client) ResumeSessionWithOptions(ctx context.Context, sessionID string,
13561356
return nil, fmt.Errorf("failed to unmarshal response: %w", err)
13571357
}
13581358

1359-
if config.MCPServers != nil {
1360-
internalSessionRPC := rpc.NewInternalSessionRPC(c.client, sessionID)
1361-
_, err := internalSessionRPC.MCP.ReloadWithConfig(ctx, &rpc.MCPReloadWithConfigRequest{
1362-
Config: map[string]any{"mcpServers": config.MCPServers},
1363-
})
1364-
if err != nil {
1365-
c.sessionsMux.Lock()
1366-
delete(c.sessions, sessionID)
1367-
c.sessionsMux.Unlock()
1368-
return nil, fmt.Errorf("failed to reload MCP servers after resume: %w", err)
1369-
}
1370-
}
1371-
13721359
if config.OnMCPAuthRequest != nil {
13731360
if _, err := c.client.Request(ctx, "session.eventLog.registerInterest", map[string]any{
13741361
"sessionId": sessionID,

go/internal/e2e/rpc_server_misc_e2e_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ func TestRpcServerMisc(t *testing.T) {
131131

132132
login, err := client.RPC.Account.Login(t.Context(), &rpc.AccountLoginRequest{
133133
Host: "https://github.com",
134-
Login: "go-account-user",
134+
Login: rpcPtr("go-account-user"),
135135
Token: "go-account-token",
136136
})
137137
if err != nil {

go/rpc/zrpc.go

Lines changed: 13 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

java/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@
6363
DO NOT EDIT MANUALLY. Updated by the update-copilot-dependency
6464
workflow.
6565
-->
66-
<readonly-copilot-sdk-ref-impl-version-from-lastmerge-file-updated-by-reference-impl-sync>^1.0.81-5</readonly-copilot-sdk-ref-impl-version-from-lastmerge-file-updated-by-reference-impl-sync>
66+
<readonly-copilot-sdk-ref-impl-version-from-lastmerge-file-updated-by-reference-impl-sync>^1.0.81-6</readonly-copilot-sdk-ref-impl-version-from-lastmerge-file-updated-by-reference-impl-sync>
6767
<!-- The parent POM is not published to Maven Central. -->
6868
<maven.deploy.skip>true</maven.deploy.skip>
6969
</properties>

java/scripts/codegen/package-lock.json

Lines changed: 36 additions & 36 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

java/scripts/codegen/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
"generate:java": "tsx java.ts"
88
},
99
"dependencies": {
10-
"@github/copilot": "^1.0.81-5",
10+
"@github/copilot": "^1.0.81-6",
1111
"json-schema": "^0.4.0",
1212
"tsx": "^4.23.1"
1313
}

java/sdk/src/generated/java/com/github/copilot/generated/rpc/AccountLoginParams.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
import javax.annotation.processing.Generated;
1515

1616
/**
17-
* Credentials to store after successful authentication
17+
* Credentials to validate and store. Omit login to resolve the authenticated user from the token.
1818
*
1919
* @apiNote This method is experimental and may change in a future version.
2020
* @since 1.0.0
@@ -26,7 +26,7 @@
2626
public record AccountLoginParams(
2727
/** GitHub host URL */
2828
@JsonProperty("host") String host,
29-
/** User login/username */
29+
/** User login/username. When omitted, the runtime validates the token and resolves the login from GitHub. */
3030
@JsonProperty("login") String login,
3131
/** GitHub authentication token */
3232
@JsonProperty("token") String token

0 commit comments

Comments
 (0)