[WIP] Achieve 100% InProcess and out-of-process test parity

[Copilot]

• Gap 1: Removed failsafe <includes> restriction in -Pinprocess profile so all ITs run
• Gap 2: Updated E2ETestContext.createClient()/createClient(options) to detect InProcess mode and route env vars through InProcessEnvGuard instead of cliPath/environment/cwd; added COPILOT_SDK_DEFAULT_CONNECTION=inprocess to surefire env vars and forkCount/parallel settings in the inprocess profile
• Gap 3: Annotated tests with inherently incompatible direct client construction (ClientOptionsE2ETest, parts of CopilotClientTest, PerSessionAuthTest, RpcServerMiscE2ETest, SlashCommandsIT) with @SkipInProcess("reason")
• Verified mvn compile test-compile succeeds
• Full mvn clean verify -Pinprocess / mvn clean verify runs not completed due to time constraints — recommend running in CI to confirm full parity before merge

• Fixes [Java] Post-agentic 01: Achieve 100% InProcess / out-of-process test parity #2271

[Copilot]

Pull request overview

Extends Java test coverage across both subprocess and InProcess transports.

Changes:

• Runs all Surefire and Failsafe tests under the InProcess profile.
• Routes E2E environment configuration through InProcessEnvGuard.
• Skips tests requiring subprocess-specific behavior.

Show a summary per file

File	Description
java/sdk/pom.xml	Configures InProcess test execution.
E2ETestContext.java	Adds InProcess environment handling.
SkipInProcess.java	Exposes the execution condition.
RequireInProcess.java	Exposes the execution condition.
SlashCommandsIT.java	Skips subprocess-only integration tests.
RpcServerMiscE2ETest.java	Skips incompatible account-flow test.
PerSessionAuthTest.java	Skips incompatible auth-isolation test.
CopilotClientTest.java	Skips selected transport-specific tests.
ClientOptionsE2ETest.java	Skips subprocess argument-forwarding tests.

Review details

Suppressed comments (2)

java/sdk/pom.xml:631

• Failsafe also needs a fork for its configured environment variables to take effect. With forkCount=0, neither COPILOT_SDK_DEFAULT_CONNECTION nor COPILOT_CLI_PATH is injected, so removing the <includes> filter merely runs all ITs on the default transport rather than providing InProcess parity.

                            <forkCount>0</forkCount>

java/sdk/src/test/java/com/github/copilot/E2ETestContext.java:373

• An explicit non-InProcess connection must win over the ambient default, matching CopilotClient's transport-selection contract. Currently RuntimeConnection.forStdio()/forTcp() falls through to the environment check, so createClient(options) treats it as InProcess, omits the context's stdio settings, and mutates the native environment even though the client will use the explicit connection.

        RuntimeConnection connection = options.getConnection();
        if (connection instanceof InProcessRuntimeConnection) {
            return true;
        }
        String defaultConnection = System.getenv("COPILOT_SDK_DEFAULT_CONNECTION");

• Files reviewed: 9/9 changed files
• Comments generated: 3
• Review effort level: Balanced

[github-actions]

Cross-SDK Consistency Review ✅

This PR modifies only the Java SDK (test infrastructure, E2ETestContext, InProcessEnvGuard, @SkipInProcess/@RequireInProcess annotations, and planning docs).

No new public API surface is introduced — the changes are entirely internal to the Java test harness for InProcess vs. out-of-process test parity.

No cross-SDK consistency issues were identified. Other language SDKs (Node.js, Python, Go, .NET, Rust) do not require corresponding changes.

Generated by SDK Consistency Review Agent for #2272 · sonnet46 10.4 AIC · ⌖ 5.33 AIC · ⊞ 6.6K · ◷