refactor: update imports to use node: prefix and improve code structure

- Changed imports from 'fs' and 'path' to 'node:fs' and 'node:path' across multiple files.
- Refactored BuildToolService to use a BatchCommandOptions interface for better argument handling.
- Enhanced error handling and logging in various methods.
- Simplified regex handling for XML parsing and control character stripping.
- Updated test files to reflect changes in BuildToolService and ConfigurationService.
- Adjusted TypeScript configuration to target ES2021.
- Updated vitest configuration to use 'node:url' for consistency.

Author: tomaszrup

package.json

@@ -2,7 +2,7 @@
   "name": "spock-test-runner-vscode-tr",
   "displayName": "Spock Test Runner & Debugger",
   "description": "Run, debug, and measure code coverage of Spock Framework (Groovy) tests using Gradle or Maven, with native VS Code Test Explorer integration",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "copyright": "Copyright 2025 Lukas Zaruba, Copyright 2026 Tomasz Rup",
   "publisher": "TomaszRup",
   "author": "Tomasz Rup",

src/TestRunCoordinator.test.ts

@@ -445,7 +445,7 @@ describe('TestRunCoordinator', () => {
         { test: test2, data: { type: 'test' as const, className: 'Spec', testName: 'test two' } },
       ];
 
-      await coordinator.runBatch('/workspace/project', tests, run as any, false, createCancellationToken());
+      await coordinator.runBatch('/workspace/project', { uri: vscode.Uri.file('/workspace'), name: 'workspace', index: 0 }, tests, run as any, false, createCancellationToken());
 
       expect(run.started).toHaveBeenCalledWith(test1);
       expect(run.started).toHaveBeenCalledWith(test2);
@@ -459,7 +459,7 @@ describe('TestRunCoordinator', () => {
         { test: test1, data: { type: 'test' as const } }, // no className/testName
       ];
 
-      await coordinator.runBatch('/workspace/project', tests, run as any, false, createCancellationToken());
+      await coordinator.runBatch('/workspace/project', { uri: vscode.Uri.file('/workspace'), name: 'workspace', index: 0 }, tests, run as any, false, createCancellationToken());
 
       expect(run.skipped).toHaveBeenCalledWith(test1);
     });
@@ -472,7 +472,7 @@ describe('TestRunCoordinator', () => {
         { test: test1, data: { type: 'test' as const, className: 'Spec', testName: 'data test', isDataDriven: true } },
       ];
 
-      await coordinator.runBatch('/workspace/project', tests, run as any, false, createCancellationToken());
+      await coordinator.runBatch('/workspace/project', { uri: vscode.Uri.file('/workspace'), name: 'workspace', index: 0 }, tests, run as any, false, createCancellationToken());
 
       expect(resultProcessor.handleDataDrivenTestResults).toHaveBeenCalled();
     });
@@ -490,7 +490,15 @@ describe('TestRunCoordinator', () => {
       ]);
       coverageService.parseJacocoReport.mockResolvedValue([{ uri: vscode.Uri.file('/Spec.groovy') }]);
 
-      await coordinator.runBatch('/workspace/project', tests, run as any, false, createCancellationToken(), true);
+      await coordinator.runBatch(
+        '/workspace/project',
+        { uri: vscode.Uri.file('/workspace'), name: 'workspace', index: 0 },
+        tests,
+        run as any,
+        false,
+        createCancellationToken(),
+        { coverage: true },
+      );
 
       expect(coverageService.findAllJacocoXmlReports).toHaveBeenCalled();
       expect(run.addCoverage).toHaveBeenCalled();
@@ -517,7 +525,7 @@ describe('TestRunCoordinator', () => {
         return args;
       });
 
-      await coordinator.runBatch('/workspace/project', tests, run as any, false, createCancellationToken());
+      await coordinator.runBatch('/workspace/project', { uri: vscode.Uri.file('/workspace'), name: 'workspace', index: 0 }, tests, run as any, false, createCancellationToken());
 
       // Should have called executeBatch (at least once — exact count depends on platform limits)
       expect(executionService.executeBatch).toHaveBeenCalled();
@@ -540,7 +548,7 @@ describe('TestRunCoordinator', () => {
         { test: test2, data: { type: 'test' as const, className: 'MySpec', testName: 'test two' } },
       ];
 
-      await coordinator.runBatch('/workspace/project', tests, run as any, false, createCancellationToken());
+      await coordinator.runBatch('/workspace/project', { uri: vscode.Uri.file('/workspace'), name: 'workspace', index: 0 }, tests, run as any, false, createCancellationToken());
 
       // buildBatchCommandArgs should have been called with classTestCounts as the last arg
       const calls = buildToolService.buildBatchCommandArgs.mock.calls;
@@ -914,7 +922,7 @@ describe('TestRunCoordinator', () => {
         output: [
           '> Task :compileTestGroovy FAILED',
           '> Compilation failed; see the compiler error output for details.',
-          'C:\\work\\src\\test\\groovy\\Spec.groovy: 12: unable to resolve class MissingType',
+          String.raw`C:\work\src\test\groovy\Spec.groovy: 12: unable to resolve class MissingType`,
           'BUILD FAILED',
         ].join('\n'),
       });