Skip to content

backport https://github.com/github/copilot-sdk/pull/1483#237

Merged
edburns merged 3 commits into
mainfrom
edburns/pr-1483-backsync
May 28, 2026
Merged

backport https://github.com/github/copilot-sdk/pull/1483#237
edburns merged 3 commits into
mainfrom
edburns/pr-1483-backsync

Conversation

@edburns
Copy link
Copy Markdown
Collaborator

@edburns edburns commented May 28, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 28, 2026 22:35
@edburns
Copy link
Copy Markdown
Collaborator Author

edburns commented May 28, 2026

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR backports upstream Java SDK changes from github/copilot-sdk#1483, adding new session/request options, JDK 25 multi-release JAR executor behavior, and related CI/test/documentation updates.

Changes:

  • Adds new session/message configuration fields such as reasoning summaries, plugin directories, large tool output, MCP OAuth token storage, MCP Apps, and display prompts.
  • Introduces an internal executor provider with a JDK 25 virtual-thread MR-JAR overlay and owned-executor shutdown handling.
  • Updates tests, CI workflow, version metadata, README content, and standalone/monorepo comparison tooling.
Show a summary per file
File Description
pom.xml Bumps version and adds Failsafe/MR-JAR build support for JDK 25 overlay testing.
src/main/java/com/github/copilot/CopilotClient.java Adds default executor strategy, owned executor shutdown, cloud session ID handling, and MCP token defaults in EMPTY mode.
src/main/java/com/github/copilot/CopilotSession.java Threads displayPrompt into send requests and adds reasoning-summary model switching.
src/main/java/com/github/copilot/InternalExecutorProvider.java Adds baseline internal executor provider using the common pool.
src/main/java25/com/github/copilot/InternalExecutorProvider.java Adds JDK 25 MR-JAR executor provider using virtual threads.
src/main/java/com/github/copilot/SessionRequestBuilder.java Maps new session config fields into create/resume requests.
src/main/java/com/github/copilot/rpc/SessionConfig.java Adds new public session configuration fields and renames config directory accessors.
src/main/java/com/github/copilot/rpc/ResumeSessionConfig.java Adds matching resume configuration fields and renamed config directory accessors.
src/main/java/com/github/copilot/rpc/CreateSessionRequest.java Adds new create-session wire fields while preserving configDir JSON name.
src/main/java/com/github/copilot/rpc/ResumeSessionRequest.java Adds new resume-session wire fields while preserving configDir JSON name.
src/main/java/com/github/copilot/rpc/MessageOptions.java Adds displayPrompt to message options and cloning.
src/main/java/com/github/copilot/rpc/SendMessageRequest.java Adds the displayPrompt wire field.
src/main/java/com/github/copilot/rpc/LargeToolOutputConfig.java Adds DTO for large tool output settings.
src/main/java/com/github/copilot/rpc/SessionUiCapabilities.java Adds MCP Apps capability exposure.
src/main/java/com/github/copilot/rpc/CopilotClientOptions.java Updates executor Javadoc for SDK default strategy and ownership.
src/test/java/com/github/copilot/SessionRequestBuilderTest.java Adds request-building coverage for new session fields.
src/test/java/com/github/copilot/InternalExecutorProviderTest.java Adds baseline executor-provider and user executor ownership tests.
src/test/java/com/github/copilot/InternalExecutorProviderProbe.java Adds child-JVM probe for packaged MR-JAR executor selection.
src/test/java/com/github/copilot/InternalExecutorProviderIT.java Adds Failsafe integration test for packaged MR-JAR behavior.
src/test/java/com/github/copilot/CreateSessionReKeyEntryTest.java Updates create-session mismatch tests to expect rejection and cleanup.
src/test/java/com/github/copilot/CopilotSessionTest.java Updates config-directory setter usage.
src/test/java/com/github/copilot/ConfigCloneTest.java Adds clone assertions for new config fields and renamed accessor.
scripts/compare-standalone-to-monorepo.sh Adds README comparison support.
README.md Updates installation, runtime/development requirements, versions, and monorepo links.
jbang-example.java Updates the JBang dependency version.
.github/workflows/build-test.yml Adds JDK 17/25 matrix behavior and MR-JAR test flow.
.github/actions/test-report/action.yml Adds summary title input, Failsafe report glob, and generated-package prefix update.

Copilot's findings

  • Files reviewed: 27/27 changed files
  • Comments generated: 5

Comment thread README.md
Comment thread src/main/java/com/github/copilot/rpc/SessionConfig.java Outdated
Comment thread src/main/java/com/github/copilot/rpc/ResumeSessionConfig.java Outdated
Comment thread src/main/java/com/github/copilot/rpc/LargeToolOutputConfig.java
Comment thread .github/workflows/build-test.yml
@edburns edburns merged commit fc3a0c6 into main May 28, 2026
11 checks passed
@edburns edburns deleted the edburns/pr-1483-backsync branch May 28, 2026 23:08
edburns added a commit to github/copilot-sdk that referenced this pull request May 28, 2026
* Address github/copilot-sdk-java#237 (comment)

* Apply review comments from github/copilot-sdk-java#237

* fix: correct CopilotClientMode Javadoc links in java rpc configs

Co-authored-by: edburns <75821+edburns@users.noreply.github.com>

* fix: fully qualify CopilotClientMode Javadoc links

Co-authored-by: edburns <75821+edburns@users.noreply.github.com>

* Fix path for failsafe reports in workflow

* Correct Javadoc formatting in ResumeSessionConfig

Fix formatting of Javadoc comment for clarity.

* Fix formatting in ResumeSessionConfig.java documentation

* Fix formatting in SessionConfig.java documentation

* Spotless

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
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.

2 participants