fix(deps): complete typescript-eslint v8 migration

Merge the latest main branch, align ESLint with the v8 peer range, address review feedback, and cover malformed TestNG output diagnostics.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: Copilot

package-lock.json

@@ -558,15 +558,15 @@
         "@types/lru-cache": "^7.10.10",
         "@types/mocha": "^9.1.1",
         "@types/node": "^16.18.13",
-        "@types/sinon": "^10.0.13",
+        "@types/sinon": "^22.0.0",
         "@types/vscode": "1.125.0",
         "@typescript-eslint/eslint-plugin": "^8.67.0",
         "@typescript-eslint/parser": "^8.67.0",
         "@vscode/test-electron": "^3.1.0",
-        "eslint": "^8.43.0",
+        "eslint": "^8.57.0",
         "glob": "^7.2.3",
         "mocha": "^11.1.0",
-        "sinon": "^14.0.2",
+        "sinon": "^22.1.0",
         "ts-loader": "^9.4.2",
         "typescript": "^4.9.4",
         "webpack": "^5.109.2",

package.json

@@ -48,8 +48,8 @@ export class TestNGRunnerResultAnalyzer extends RunnerResultAnalyzer {
         while ((match = this.regex.exec(data)) !== null) {
             try {
                 this.processData(match[1]);
-            } catch {
-                this.testContext.testRun.appendOutput(`[ERROR] Failed to parse output data: ${match[1]}\r\n`);
+            } catch (error) {
+                this.testContext.testRun.appendOutput(`[ERROR] Failed to parse output data: ${match[1]}. ${String(error)}\r\n`);
             }
         }
     }

src/runners/testngRunner/TestNGRunnerResultAnalyzer.ts

@@ -57,7 +57,7 @@ function filterCandidateConfigItems(configItems: IExecutionConfig[], testItems:
             try {
                 return checkTestItems(testItems, context);
             } catch {
-                // do something with the error
+                // Ignore invalid when clauses so the configuration remains selectable.
             }
         }
 

src/utils/configUtils.ts

@@ -72,6 +72,29 @@ suite('TestNG Runner Analyzer Tests', () => {
         sinon.assert.notCalled(appendOutputSpy);
     });
 
+    test('includes parsing errors in malformed runner output diagnostics', () => {
+        const testItem = generateTestItem(testController, 'testng@example.SampleTest#test', TestKind.TestNG);
+        const testRun = testController.createTestRun(new TestRunRequest([testItem], []));
+        const appendOutputSpy = sinon.spy(testRun, 'appendOutput');
+        const runnerContext: IRunTestContext = {
+            isDebug: false,
+            kind: TestKind.TestNG,
+            projectName: 'testng',
+            testItems: [testItem],
+            testRun,
+            workspaceFolder: workspace.workspaceFolders?.[0]!,
+        };
+        const analyzer = new TestNGRunnerResultAnalyzer(runnerContext);
+
+        analyzer.analyzeData('@@<TestRunner-{invalid}-TestRunner>');
+
+        sinon.assert.calledOnce(appendOutputSpy);
+        sinon.assert.calledWith(
+            appendOutputSpy,
+            sinon.match(/^\[ERROR\] Failed to parse output data: \{invalid\}\. SyntaxError:/),
+        );
+    });
+
     test('reports method results without assigning a result state to the class', () => {
         const classItem = testController.createTestItem('testng@example.SampleTest', 'SampleTest');
         dataCache.set(classItem, {