Update Other deps (major) (major) (#1489)

* Update Other deps (major)

* Switch build files to .mjs

* Update yarn

* Freeze some deps

* Update packages

* Update knip config to .mjs scripts

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Jean-Yves Moyen <[email protected]>

.yarnrc.yml

@@ -4,8 +4,4 @@ npmScopes:
   siteimprove:
     npmRegistryServer: "https://npm.pkg.github.com"
 
-plugins:
-  - path: .yarn/plugins/@yarnpkg/plugin-workspace-tools.cjs
-    spec: "@yarnpkg/plugin-workspace-tools"
-
-yarnPath: .yarn/releases/yarn-3.6.4.cjs
+yarnPath: .yarn/releases/yarn-4.0.0.cjs

config/knip.ts

@@ -6,8 +6,8 @@ const project = ["src/**/*.ts!", "test/**/*.ts", "test/**/*.tsx"];
 const config: KnipConfig = {
   workspaces: {
     ".": {
-      entry: ["scripts/*.js"],
-      project: ["scripts/**/*.ts", "scripts/**/*.js"],
+      entry: ["scripts/*.mjs"],
+      project: ["scripts/**/*.ts", "scripts/**/*.mjs"],
       ignoreDependencies: ["prettier"],
     },
     "packages/*": { entry, project },

package.json

@@ -11,11 +11,11 @@
   },
   "bugs": "https://github.com/siteimprove/alfa/issues",
   "scripts": {
-    "build": "node --max-old-space-size=8192 scripts/build.js --pretty",
-    "clean": "node scripts/clean.js --pretty",
-    "test": "node scripts/test.js --pretty",
-    "watch": "node scripts/watch.js --pretty",
-    "extract": "node scripts/api-extractor.js --pretty",
+    "build": "node --max-old-space-size=8192 scripts/build.mjs --pretty",
+    "clean": "node scripts/clean.mjs --pretty",
+    "test": "node scripts/test.mjs --pretty",
+    "watch": "node scripts/watch.mjs --pretty",
+    "extract": "node scripts/api-extractor.mjs --pretty",
     "document": "api-documenter generate -i docs/data/api -o docs/api",
     "dependency-graph": "package-dependency-graph --config config/package-dependency-graph.ts --exclude-node_modules --graphviz --dot docs/dependency-graph.dot --png docs/dependency-graph.png",
     "knip": "knip --config config/knip.ts"
@@ -31,12 +31,12 @@
     "@types/async": "^3.2.20",
     "@types/node": "^20.5.9",
     "async": "^3.2.1",
-    "execa": "^5.1.1",
+    "execa": "^8.0.0",
     "knip": "^2.33.4",
     "minimist": "^1.2.6",
     "package-dependency-graph": "^1.14.4",
-    "prettier": "^2.7.1",
+    "prettier": "^3.0.0",
     "typescript": "^5.2.2"
   },
-  "packageManager": "yarn@3.6.4"
+  "packageManager": "yarn@4.0.0"
 }

packages/alfa-aria/package.json

@@ -46,8 +46,8 @@
   },
   "devDependencies": {
     "@siteimprove/alfa-test": "workspace:^0.68.0",
-    "prettier": "^2.7.1",
-    "puppeteer": "^19.9.1"
+    "prettier": "^3.0.0",
+    "puppeteer": "^21.0.0"
   },
   "publishConfig": {
     "access": "public",

packages/alfa-compatibility/package.json

@@ -26,8 +26,8 @@
     "browserslist": "^4.17.0"
   },
   "devDependencies": {
-    "@mdn/browser-compat-data": "^3.1.3",
-    "prettier": "^2.7.1"
+    "@mdn/browser-compat-data": "^5.0.0",
+    "prettier": "^3.0.0"
   },
   "publishConfig": {
     "access": "public",

packages/alfa-iana/package.json

@@ -29,8 +29,8 @@
   },
   "devDependencies": {
     "@siteimprove/alfa-test": "workspace:^0.68.0",
-    "axios": "^0.27.2",
-    "prettier": "^2.7.1"
+    "axios": "^1.0.0",
+    "prettier": "^3.0.0"
   },
   "publishConfig": {
     "access": "public",

packages/alfa-test/package.json

@@ -19,7 +19,7 @@
   ],
   "dependencies": {
     "@types/node": "^20.5.9",
-    "chalk": "^4.1.2"
+    "chalk": "=4.1.2"
   },
   "publishConfig": {
     "access": "public",

packages/alfa-test/src/format.ts

@@ -37,7 +37,7 @@ export function format(name: string, error: Error): string {
       "\n" +
       error.message.replace(
         "Input A expected to strictly deep-equal input B:\n",
-        ""
+        "",
       );
 
     message += error.message;
@@ -47,7 +47,7 @@ export function format(name: string, error: Error): string {
 
   const output = `
 ${chalk.underline(`${file}(${line},${column}):`)} ${chalk.bold(
-    name.trim().replace(/\s+/g, " ")
+    name.trim().replace(/\s+/g, " "),
   )}
 ${message}
   `;

packages/alfa-toolchain/package.json

@@ -38,7 +38,7 @@
     "@changesets/get-github-info": "^0.5.2",
     "@changesets/git": "^2.0.0",
     "@changesets/read": "^0.5.9",
-    "@manypkg/get-packages": "^1.1.3",
+    "@manypkg/get-packages": "=1.1.3",
     "@siteimprove/alfa-result": "workspace:^0.68.0",
     "@svitejs/changesets-changelog-github-compact": "^1.1.0",
     "resolve-from": "^5.0.0"

packages/alfa-wcag/package.json

@@ -28,8 +28,8 @@
     "@siteimprove/alfa-refinement": "workspace:^0.68.0"
   },
   "devDependencies": {
-    "prettier": "^2.7.1",
-    "puppeteer": "^19.9.1"
+    "prettier": "^3.0.0",
+    "puppeteer": "^21.0.0"
   },
   "publishConfig": {
     "access": "public",

scripts/api-extractor.js → scripts/api-extractor.mjs

@@ -1,10 +1,9 @@
-const path = require("path");
+import * as path from "path";
 
-const { Extractor, ExtractorConfig } = require("@microsoft/api-extractor");
-
-const { system } = require("./common/system");
-const { flags } = require("./common/flags");
-const { builder } = require("./common/builder");
+import { Extractor, ExtractorConfig } from "@microsoft/api-extractor";
+import { system } from "./common/system.mjs";
+import { flags } from "./common/flags.mjs";
+import { builder } from "./common/builder.mjs";
 
 const status = builder.build(flags.project);
 
@@ -25,7 +24,7 @@ function extract(root = "packages") {
     let file;
     try {
       file = require.resolve(
-        path.resolve(project, "config", "api-extractor.json")
+        path.resolve(project, "config", "api-extractor.json"),
       );
     } catch {
       continue;

scripts/build.js → scripts/build.mjs

@@ -1,12 +1,12 @@
-const { system } = require("./common/system");
-const { flags } = require("./common/flags");
-const { builder } = require("./common/builder");
+import { system } from "./common/system.mjs";
+import { flags } from "./common/flags.mjs";
+import { builder } from "./common/builder.mjs";
 
 const exitCode = builder.build(flags.project);
 
 if (flags.minimal && exitCode === /*Success*/ 0) {
   // Move cursor up one line and 13 to the left and erase
-  system.write("\033[1A\033[13D\033[K");
+  system.write("\x1B[1A\x1B[13D\x1B[K");
   system.write("done!" + system.newLine);
 }
 

scripts/clean.mjs

@@ -0,0 +1,5 @@
+import { system } from "./common/system.mjs";
+import { flags } from "./common/flags.mjs";
+import { builder } from "./common/builder.mjs";
+
+system.exit(builder.clean(flags.project));

scripts/common/builder.mjs

@@ -0,0 +1,9 @@
+import ts from "typescript";
+
+import { host } from "./host.mjs";
+import { flags } from "./flags.mjs";
+
+export const builder = ts.createSolutionBuilder(host, ["tsconfig.json"], {
+  force: flags.force,
+  verbose: true, // verbosity is now managed by different reporter implementations
+});

scripts/common/flags.js → scripts/common/flags.mjs

@@ -1,12 +1,11 @@
-const path = require("path");
-const process = require("process");
-const minimist = require("minimist");
+import * as path from "path";
+import minimist from "minimist";
 
 const argv = minimist(process.argv.slice(2), {
   boolean: ["force", "verbose", "quiet", "pretty"],
 });
 
-exports.flags = {
+export const flags = {
   project: argv._[0] || path.relative(process.cwd(), process.env.INIT_CWD),
   force: argv.force,
   verbose: argv.verbose,

scripts/common/host.mjs

@@ -0,0 +1,12 @@
+import ts from "typescript";
+
+import { system } from "./system.mjs";
+import * as reporter from "./reporter.mjs";
+
+export const host = ts.createSolutionBuilderWithWatchHost(
+  system,
+  /* createProgram */ undefined,
+  reporter.diagnostic,
+  reporter.status.build,
+  reporter.status.watch,
+);

scripts/common/reporter.js → scripts/common/reporter.mjs

@@ -1,15 +1,15 @@
-const ts = require("typescript");
+import ts from "typescript";
 
-const { system } = require("./system");
-const { flags } = require("./flags");
+import { system } from "./system.mjs";
+import { flags } from "./flags.mjs";
 const verboseStatusReporter = ts.createBuilderStatusReporter;
 
 /**
  * @type {ts.DiagnosticReporter}
  */
-exports.diagnostic = ts.createDiagnosticReporter(system, flags.pretty);
+export const diagnostic = ts.createDiagnosticReporter(system, flags.pretty);
 
-exports.status = {
+export const status = {
   /**
    * @type {ts.DiagnosticReporter}
    */
@@ -37,7 +37,7 @@ function minimalStatusReporter(system, _) {
   return function (diag) {
     if (!first) {
       // Move cursor up one line and 13 to the left and erase
-      system.write("\033[1A\033[13D\033[K");
+      system.write("\x1B[1A\x1B[13D\x1B[K");
     }
 
     const frame = frames[frameIndex % frames.length];

scripts/common/system.mjs

@@ -0,0 +1,6 @@
+import ts from "typescript";
+
+/**
+ * @type {ts.System}
+ */
+export const system = ts.sys;

scripts/test.mjs

@@ -0,0 +1,36 @@
+import async from "async";
+import { execaNode } from "execa";
+import os from "os";
+
+import { builder } from "./common/builder.mjs";
+import { flags } from "./common/flags.mjs";
+import { system } from "./common/system.mjs";
+
+const status = builder.build(flags.project);
+
+if (status !== 0) {
+  system.exit(status);
+}
+
+test(flags.project);
+
+function test(root = "packages") {
+  async.eachLimit(
+    system.readDirectory(root, [".spec.ts", ".spec.tsx"], ["node_modules"]),
+    os.cpus().length,
+    (fileName, done) => {
+      execaNode(fileName.replace(/\.tsx?$/, ".js"), [], {
+        nodeOptions: [...process.execArgv, "--enable-source-maps"],
+        stdio: "inherit",
+      }).then(
+        () => done(),
+        (err) => done(err),
+      );
+    },
+    (err) => {
+      if (err) {
+        system.exit(1);
+      }
+    },
+  );
+}

scripts/watch.js → scripts/watch.mjs

@@ -1,7 +1,7 @@
-const ts = require("typescript");
+import ts from "typescript";
 
-const { flags } = require("./common/flags");
-const { host } = require("./common/host");
+import { flags } from "./common/flags.mjs";
+import { host } from "./common/host.mjs";
 
 const watcher = ts.createSolutionBuilderWithWatch(host, ["tsconfig.json"], {
   force: flags.force,