From 9476f9865dba07fba98dc896f2e8cdc70f11411e Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 6 Feb 2026 02:02:56 +0000
Subject: [PATCH 1/2] Initial plan


From bf34e89c5c92a632fb91262e944a3fb0fb2413ca Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 6 Feb 2026 02:05:36 +0000
Subject: [PATCH 2/2] Fix: Remove unnecessary double quotes wrapping -javaagent
 argument on Windows

On Windows, the -javaagent argument for JaCoCo coverage was wrapped in
double quotes before being pushed to the vmArgs array. Since vmArgs is
an array of strings where each element is already a separate argument,
the extra quotes caused Java to interpret the entire string as a class
name, resulting in "Could not find or load main class" errors.

Removes the Windows-specific quoting block and the now-unused os import.

Co-authored-by: chagong <831821+chagong@users.noreply.github.com>
---
 src/utils/launchUtils.ts | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/utils/launchUtils.ts b/src/utils/launchUtils.ts
index 433b8486..0fb746e4 100644
--- a/src/utils/launchUtils.ts
+++ b/src/utils/launchUtils.ts
@@ -2,7 +2,6 @@
 // Licensed under the MIT license.
 
 import * as path from 'path';
-import * as os from 'os';
 import { DebugConfiguration, TestItem, TestRunProfileKind } from 'vscode';
 import { sendError, sendInfo } from 'vscode-extension-telemetry-wrapper';
 import { JavaTestRunnerDelegateCommands } from '../constants';
@@ -70,9 +69,6 @@ export async function resolveLaunchConfigurationForRunner(runner: BaseRunner, te
         if (config?.coverage?.appendResult === false) {
             agentArg += ',append=false';
         }
-        if (os.platform() === 'win32') {
-            agentArg = `"${agentArg}"`;
-        }
         (debugConfiguration.vmArgs as string[]).push(agentArg);
     }