Apply review comments from standalone backport. (#1486)

* 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>

Author: edburns

.githooks/pre-commit

@@ -147,4 +147,5 @@ jobs:
           path: |
             java/target/surefire-reports/
             java/target/surefire-reports-isolated/
+            java/target/failsafe-reports/
           retention-days: 7

.github/workflows/java-sdk-tests.yml

@@ -38,7 +38,7 @@ Java SDK for programmatic control of GitHub Copilot CLI, enabling you to build A
 
 ```groovy
 implementation 'com.github:copilot-sdk-java:1.0.0-beta-10-java.0'
-
+```
 
 #### Snapshot Builds
 

java/README.md

@@ -1,3 +1,8 @@
+/*
+ * Copyright (c) Microsoft Corporation. All rights reserved.
+ * Licensed under the MIT License.
+ */
+
 package com.github.copilot.rpc;
 
 import com.fasterxml.jackson.annotation.JsonInclude;

java/src/main/java/com/github/copilot/rpc/LargeToolOutputConfig.java

@@ -258,7 +258,7 @@ public ResumeSessionConfig setProvider(ProviderConfig provider) {
      * {@code true}, telemetry is enabled for GitHub-authenticated sessions. When a
      * custom {@link ProviderConfig} (BYOK) is configured, session telemetry is
      * always disabled regardless of this setting. This is independent of
-     * {@link com.github.copilot.rpc.CopilotClientOptions#getTelemetry()
+     * {@link CopilotClientOptions#getTelemetry()
      * CopilotClientOptions.TelemetryConfig}, which configures OpenTelemetry export
      * for observability.
      *
@@ -1060,8 +1060,9 @@ public String getMcpOAuthTokenStorage() {
      * <li>{@code "in-memory"} — tokens are stored in memory and discarded when the
      * session ends</li>
      * </ul>
-     * If not set, the SDK defaults to {@code "in-memory"} for safe multitenant
-     * behavior.
+     * If not set and the client is in {@link CopilotClientMode#EMPTY EMPTY} mode,
+     * the SDK defaults to {@code "in-memory"} for safe multitenant behavior. In
+     * other modes this field is left unset.
      *
      * @param mcpOAuthTokenStorage
      *            the storage mode

java/src/main/java/com/github/copilot/rpc/ResumeSessionConfig.java

@@ -705,8 +705,9 @@ public String getMcpOAuthTokenStorage() {
      * <li>{@code "in-memory"} — tokens are stored in memory and discarded when the
      * session ends</li>
      * </ul>
-     * If not set, the SDK defaults to {@code "in-memory"} for safe multitenant
-     * behavior.
+     * If not set and the client is in {@link CopilotClientMode#EMPTY EMPTY} mode,
+     * the SDK defaults to {@code "in-memory"} for safe multitenant behavior. In
+     * other modes this field is left unset.
      *
      * @param mcpOAuthTokenStorage
      *            the storage mode