reverting

AndreasArvidsson · AndreasArvidsson · commit b7bee4f8ff87 · 2025-03-28T09:59:19.000+01:00
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -31,7 +31,7 @@ jobs:
       VSCODE_LOGS_DIR: ${{ github.workspace }}/artifacts/logs
       CURSORLESS_REPO_ROOT: ${{ github.workspace }}
       TEMP_DIR: ${{ github.workspace }}/temp
-      NODE_OPTIONS: "--max-old-space-size=16384"
+      NODE_OPTIONS: "--max-old-space-size=4096"
 
     steps:
       - name: Checkout repository
diff --git a/package.json b/package.json
@@ -30,20 +30,20 @@
   "devDependencies": {
     "@pnpm/meta-updater": "2.0.3",
     "@types/node": "20.16.0",
-    "@typescript-eslint/eslint-plugin": "^8.26.1",
-    "@typescript-eslint/parser": "^8.26.1",
+    "@typescript-eslint/eslint-plugin": "^8.9.0",
+    "@typescript-eslint/parser": "^8.9.0",
     "esbuild": "^0.25.0",
     "eslint": "^8.57.1",
     "eslint-config-prettier": "^9.1.0",
-    "eslint-import-resolver-typescript": "^3.9.0",
+    "eslint-import-resolver-typescript": "3.6.3",
     "eslint-plugin-import": "2.31.0",
     "eslint-plugin-mocha": "10.5.0",
     "eslint-plugin-unicorn": "56.0.0",
     "eslint-plugin-unused-imports": "^4.1.4",
     "prettier": "3.3.3",
     "prettier-plugin-tailwindcss": "0.6.8",
     "syncpack": "13.0.0",
-    "typescript": "^5.8.2"
+    "typescript": "^5.6.3"
   },
   "pnpm": {
     "patchedDependencies": {
diff --git a/packages/cheatsheet-local/package.json b/packages/cheatsheet-local/package.json
@@ -48,7 +48,7 @@
     "tailwindcss": "3.4.14",
     "ts-loader": "9.5.1",
     "ts-node": "10.9.2",
-    "typescript": "^5.8.2",
+    "typescript": "^5.6.3",
     "webpack": "5.95.0",
     "webpack-cli": "5.1.4",
     "webpack-dev-server": "5.1.0"
diff --git a/packages/cheatsheet/package.json b/packages/cheatsheet/package.json
@@ -45,7 +45,7 @@
     "jest": "29.7.0",
     "jest-environment-jsdom": "29.7.0",
     "ts-jest": "29.2.5",
-    "typescript": "^5.8.2"
+    "typescript": "^5.6.3"
   },
   "type": "module"
 }
diff --git a/packages/common/package.json b/packages/common/package.json
@@ -22,11 +22,11 @@
   "devDependencies": {
     "@types/js-yaml": "^4.0.9",
     "@types/lodash-es": "4.17.12",
-    "@types/mocha": "^10.0.10",
+    "@types/mocha": "^10.0.9",
     "cross-spawn": "7.0.5",
     "fast-check": "3.22.0",
     "js-yaml": "^4.1.0",
-    "mocha": "^11.1.0"
+    "mocha": "^10.7.3"
   },
   "types": "./out/index.d.ts",
   "exports": {
diff --git a/packages/cursorless-engine/package.json b/packages/cursorless-engine/package.json
@@ -37,13 +37,13 @@
   "devDependencies": {
     "@types/js-yaml": "^4.0.9",
     "@types/lodash-es": "4.17.12",
-    "@types/mocha": "^10.0.10",
+    "@types/mocha": "^10.0.9",
     "@types/moo": "0.5.9",
     "@types/nearley": "2.11.5",
     "@types/sinon": "^17.0.3",
     "@types/uuid": "^10.0.0",
     "js-yaml": "^4.1.0",
-    "mocha": "^11.1.0",
+    "mocha": "^10.7.3",
     "sinon": "^19.0.2"
   },
   "types": "./out/index.d.ts",
diff --git a/packages/cursorless-everywhere-talon-e2e/package.json b/packages/cursorless-everywhere-talon-e2e/package.json
@@ -20,7 +20,7 @@
     "@cursorless/test-case-recorder": "workspace:*"
   },
   "devDependencies": {
-    "@types/mocha": "^10.0.10",
-    "mocha": "^11.1.0"
+    "@types/mocha": "^10.0.9",
+    "mocha": "^10.7.3"
   }
 }
diff --git a/packages/cursorless-org-docs/package.json b/packages/cursorless-org-docs/package.json
@@ -52,7 +52,7 @@
     "@tsconfig/docusaurus": "2.0.3",
     "@types/mdast": "4.0.4",
     "@types/react": "18.3.11",
-    "typescript": "^5.8.2",
+    "typescript": "^5.6.3",
     "unified": "11.0.5"
   },
   "license": "MIT",
diff --git a/packages/cursorless-org/package.json b/packages/cursorless-org/package.json
@@ -34,7 +34,7 @@
     "http-server": "14.1.1",
     "postcss": "8.4.47",
     "tailwindcss": "3.4.14",
-    "typescript": "^5.8.2"
+    "typescript": "^5.6.3"
   },
   "license": "MIT",
   "main": "./out/index.js",
diff --git a/packages/cursorless-vscode-e2e/package.json b/packages/cursorless-vscode-e2e/package.json
@@ -29,12 +29,12 @@
     "@types/chai": "^5.0.0",
     "@types/js-yaml": "^4.0.9",
     "@types/lodash-es": "4.17.12",
-    "@types/mocha": "^10.0.10",
+    "@types/mocha": "^10.0.9",
     "@types/sinon": "^17.0.3",
     "@types/vscode": "1.82.0",
     "chai": "^5.1.1",
     "js-yaml": "^4.1.0",
-    "mocha": "^11.1.0",
+    "mocha": "^10.7.3",
     "sinon": "^19.0.2"
   },
   "types": "./out/index.d.ts",
diff --git a/packages/cursorless-vscode-e2e/src/suite/scopes.vscode.test.ts b/packages/cursorless-vscode-e2e/src/suite/scopes.vscode.test.ts
@@ -50,7 +50,7 @@ suite("Scope test cases", async function () {
     );
   }
 
-  testPaths.slice(0, 1).forEach(({ path, name, languageId, facet }) =>
+  testPaths.forEach(({ path, name, languageId, facet }) =>
     test(
       name,
       asyncSafety(() => runTest(path, languageId, facet)),
diff --git a/packages/sentence-parser/package.json b/packages/sentence-parser/package.json
@@ -24,7 +24,7 @@
     }
   },
   "devDependencies": {
-    "@types/mocha": "^10.0.10",
-    "mocha": "^11.1.0"
+    "@types/mocha": "^10.0.9",
+    "mocha": "^10.7.3"
   }
 }
diff --git a/packages/test-harness/package.json b/packages/test-harness/package.json
@@ -37,11 +37,11 @@
   "devDependencies": {
     "@types/cross-spawn": "6.0.6",
     "@types/glob": "^8.1.0",
-    "@types/mocha": "^10.0.10",
+    "@types/mocha": "^10.0.9",
     "@types/tail": "2.2.3",
     "@vscode/test-electron": "^2.4.1",
     "cross-spawn": "7.0.5",
-    "mocha": "^11.1.0"
+    "mocha": "^10.7.3"
   },
   "types": "./out/runners/extensionTestsNeovim.d.ts",
   "exports": {
diff --git a/packages/test-harness/src/launchVscodeAndRunTests.ts b/packages/test-harness/src/launchVscodeAndRunTests.ts
@@ -35,12 +35,11 @@ export async function launchVscodeAndRunTests(extensionTestsPath: string) {
     // NB: We include the exact version here instead of in `test.yml` so that
     // we don't have to update the branch protection rules every time we bump
     // the legacy VSCode version.
-    const vscodeVersion = useLegacyVscode ? "1.82.0" : "stable";
-    // const vscodeVersion = useLegacyVscode
-    //   ? "1.82.0"
-    //   : os.platform() === "win32"
-    //     ? "stable"
-    //     : "1.97.2";
+    const vscodeVersion = useLegacyVscode
+      ? "1.82.0"
+      : os.platform() === "win32"
+        ? "stable"
+        : "1.97.2";
     const vscodeExecutablePath = await downloadAndUnzipVSCode(vscodeVersion);
     const [cli, ...args] =
       resolveCliArgsFromVSCodeExecutablePath(vscodeExecutablePath);
diff --git a/packages/test-harness/src/runAllTests.ts b/packages/test-harness/src/runAllTests.ts
@@ -76,12 +76,7 @@ async function runTestsInDir(
     await new Promise<void>((resolve, reject) => {
       const failedTests: string[] = [];
 
-      console.log(`Running ${files.length} tests...`);
-
       const runner = mocha.run((failures) => {
-        console.log(`Finished running ${files.length} tests.`);
-        console.log(`Failures: ${failures}`);
-
         if (shouldLogFailedTests()) {
           logFailedTests(failedTests);
         }
@@ -98,7 +93,6 @@ async function runTestsInDir(
       }
     });
   } catch (err) {
-    console.log("Failed to run tests:");
     console.error(err);
     throw err;
   }
diff --git a/packages/test-harness/src/scripts/runVscodeTestsCI.ts b/packages/test-harness/src/scripts/runVscodeTestsCI.ts
@@ -14,12 +14,5 @@ void (async () => {
     "packages/test-harness/dist/extensionTestsVscode.cjs",
   );
 
-  try {
-    await launchVscodeAndRunTests(extensionTestsPath);
-  } catch (error) {
-    console.log("Error running launchVscodeAndRunTests tests:");
-    const errorMessage = error instanceof Error ? error.message : String(error);
-    console.log(error);
-    console.error(errorMessage);
-  }
+  await launchVscodeAndRunTests(extensionTestsPath);
 })();
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@`
`45`	`45`	`"jest": "29.7.0",`
`46`	`46`	`"jest-environment-jsdom": "29.7.0",`
`47`	`47`	`"ts-jest": "29.2.5",`
`48`		`- "typescript": "^5.8.2"`
	`48`	`+ "typescript": "^5.6.3"`
`49`	`49`	`},`
`50`	`50`	`"type": "module"`
`51`	`51`	`}`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@`
`20`	`20`	`"@cursorless/test-case-recorder": "workspace:*"`
`21`	`21`	`},`
`22`	`22`	`"devDependencies": {`
`23`		`- "@types/mocha": "^10.0.10",`
`24`		`- "mocha": "^11.1.0"`
	`23`	`+ "@types/mocha": "^10.0.9",`
	`24`	`+ "mocha": "^10.7.3"`
`25`	`25`	`}`
`26`	`26`	`}`