Skip to content

Port AOT-safe SetForegroundSessionRequest from reference impl (c63feb2)#125

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/sync-reference-implementation
Draft

Port AOT-safe SetForegroundSessionRequest from reference impl (c63feb2)#125
Copilot wants to merge 3 commits intomainfrom
copilot/sync-reference-implementation

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 27, 2026

Ports reference implementation commit c63feb2: replaces the ad-hoc Map.of("sessionId", sessionId) parameter bag in setForegroundSessionId() with a dedicated named request type, matching the .NET SDK's shift from an anonymous object to an explicit record for serialization clarity.

Changes

  • json/SetForegroundSessionRequest.java (new) — named record with @JsonProperty("sessionId"), parallel to the existing SetForegroundSessionResponse
  • CopilotClient.javasetForegroundSessionId() now constructs new SetForegroundSessionRequest(sessionId) instead of Map.of(...)
  • .lastmerge — updated to c63feb2794786342d57936c13d28c250e723c676
// Before
.invoke("session.setForeground", Map.of("sessionId", sessionId), SetForegroundSessionResponse.class)

// After
.invoke("session.setForeground", new SetForegroundSessionRequest(sessionId), SetForegroundSessionResponse.class)

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • mvn spotless:apply has been run to format the code
  • mvn clean verify passes locally

Does this introduce a breaking change?

  • Yes
  • No

Copilot AI and others added 2 commits April 27, 2026 11:37
Copilot AI changed the title [WIP] Sync reference implementation with new Copilot SDK commits Port AOT-safe SetForegroundSessionRequest from reference impl (c63feb2) Apr 27, 2026
Copilot AI requested a review from edburns April 27, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reference implementation sync: 1 new commits (2026-04-27)

2 participants