auto-fix by Biome

Author: cometkim

cli/rescript_arg.js

@@ -15,7 +15,7 @@ class StringBuilder {
   }
 }
 
-class ArgError extends Error { }
+class ArgError extends Error {}
 
 /**
  * @param {string} s

cli/rescript_bsb.js

@@ -32,7 +32,7 @@ function releaseBuild() {
     ownerProcess.kill("SIGHUP");
     try {
       fs.rmSync(lockFileName);
-    } catch { }
+    } catch {}
     ownerProcess = null;
   }
 }
@@ -147,7 +147,7 @@ function logFinishCompiling(code) {
 
 function logStartCompiling() {
   updateStartTime();
-  let log = `>>>> Start compiling`;
+  let log = ">>>> Start compiling";
   if (shouldColorize) {
     log = `\x1b[36m${log}\x1b[0m`;
   }
@@ -210,7 +210,7 @@ function watch(args) {
   let watchers = [];
 
   const verbose = args.includes("-verbose");
-  const dlog = verbose ? console.log : () => { };
+  const dlog = verbose ? console.log : () => {};
 
   const wsParamIndex = args.indexOf("-ws");
   if (wsParamIndex > -1) {
@@ -253,18 +253,18 @@ function watch(args) {
   function setUpWebSocket() {
     const _id = setInterval(notifyClients, 3000);
     createServer()
-      .on("upgrade", function(req, socket, upgradeHead) {
+      .on("upgrade", (req, socket, upgradeHead) => {
         dlog("connection opened");
-        var ws = new WebSocket(req, socket, upgradeHead);
-        socket.on("error", function(err) {
+        const ws = new WebSocket(req, socket, upgradeHead);
+        socket.on("error", err => {
           dlog(`Socket Error ${err}`);
         });
         wsClients.push(ws);
       })
-      .on("error", function(err) {
+      .on("error", err => {
         // @ts-ignore
         if (err !== undefined && err.code === "EADDRINUSE") {
-          var error = shouldColorize ? `\x1b[1;31mERROR:\x1b[0m` : `ERROR:`;
+          const error = shouldColorize ? "\x1b[1;31mERROR:\x1b[0m" : "ERROR:";
           console.error(`${error} The websocket port number ${webSocketPort} is in use.
 Please pick a different one using the \`-ws [host:]port\` flag from bsb.`);
         } else {

cli/rescript_format.js

@@ -70,7 +70,7 @@ async function readStdin() {
   return Buffer.concat(chunks).toString("utf8");
 }
 
-const numThreads = os.cpus().length;
+const _numThreads = os.cpus().length;
 
 /**
  * Splits an array into smaller chunks of a specified size.

lib/minisocket.js

@@ -72,14 +72,14 @@ function encodeMessage(opcode, payload) {
 const upgradeHeader =
   "HTTP/1.1 101 Web Socket Protocol Handshake\r\nUpgrade: WebSocket\r\nConnection: Upgrade\r\nsec-websocket-accept: ";
 class MiniWebSocket {
-  constructor(req, socket, upgradeHead) {
+  constructor(req, socket, _upgradeHead) {
     this.socket = socket;
     this.closed = false;
     const key = hashWebSocketKey(req.headers["sec-websocket-key"]);
 
     // http://tools.ietf.org/html/rfc6455#section-4.2.2
     socket.write(`${upgradeHeader + key}\r\n\r\n`);
-    socket.on("close", hadError => {
+    socket.on("close", _hadError => {
       if (!this.closed) {
         this.closed = true;
       }

packages/playground-bundling/rollup.config.cjs

@@ -1,7 +1,7 @@
 const resolve = require("@rollup/plugin-node-resolve");
 
 const { globSync } = require("glob");
-const path = require("path");
+const path = require("node:path");
 
 const RESCRIPT_COMPILER_ROOT_DIR = path.join(__dirname, "..", "..");
 const LIB_DIR = path.join(RESCRIPT_COMPILER_ROOT_DIR, "lib");

scripts/cppo.js

@@ -1,6 +1,6 @@
 // @ts-check
 
-const { execFileSync } = require("child_process");
+const { execFileSync } = require("node:child_process");
 
 [
   ["belt_HashSetString.res", "hashset.res.cppo", "TYPE_STRING"],

scripts/npmPack.js

@@ -26,9 +26,9 @@
  * @typedef {[PackOutputEntry]} PackOutput
  */
 
-const { spawnSync, execSync } = require("child_process");
-const path = require("path");
-const fs = require("fs");
+const { spawnSync, execSync } = require("node:child_process");
+const path = require("node:path");
+const fs = require("node:fs");
 
 const mode = process.argv.includes("-updateArtifactList")
   ? "updateArtifactList"
@@ -38,7 +38,7 @@ const rootPath = path.join(__dirname, "..");
 const fileListPath = path.join(rootPath, "packages", "artifacts.txt");
 
 const output = spawnSync(
-  "npm pack --json" + (mode === "updateArtifactList" ? " --dry-run" : ""),
+  `npm pack --json${mode === "updateArtifactList" ? " --dry-run" : ""}`,
   {
     cwd: rootPath,
     encoding: "utf8",
@@ -75,8 +75,8 @@ function getFilesAddedByCI() {
 
   const files = ["ninja.COPYING"];
 
-  for (let platform of platforms) {
-    for (let exe of exes) {
+  for (const platform of platforms) {
+    for (const exe of exes) {
       files.push(`${platform}/${exe}`);
     }
   }

tests/build_tests/build_warn_as_error/input.js

@@ -47,7 +47,7 @@ if (!third_message) {
   assert.fail(o3.stdout);
 }
 
-var cleanup = p.spawnSync(rescript_exe, ["clean"], {
+const _cleanup = p.spawnSync(rescript_exe, ["clean"], {
   encoding: "utf8",
   cwd: __dirname,
 });

tests/build_tests/cli_compile_status/input.js

@@ -14,9 +14,7 @@ let out = child_process.spawnSync("node", [rescriptPath, "build"], {
 });
 assert.match(
   normalizeNewlines(out.stdout),
-  new RegExp(`>>>> Start compiling
-Dependency Finished
->>>> Finish compiling \\d+ mseconds`),
+  />>>> Start compiling\nDependency Finished\n>>>> Finish compiling \d+ mseconds/,
 );
 
 // Shows compile time for `rescript` command
@@ -26,9 +24,7 @@ out = child_process.spawnSync("node", [rescriptPath], {
 });
 assert.match(
   normalizeNewlines(out.stdout),
-  new RegExp(`>>>> Start compiling
-Dependency Finished
->>>> Finish compiling \\d+ mseconds`),
+  />>>> Start compiling\nDependency Finished\n>>>> Finish compiling \d+ mseconds/,
 );
 
 // Doesn't show compile time for `rescript build -verbose` command
@@ -42,6 +38,6 @@ out = child_process.spawnSync("node", [rescriptPath, "build", "-verbose"], {
 
 assert.match(
   normalizeNewlines(out.stdout),
-  /Package stack: test  \nDependency Finished\n/,
+  /Package stack: test {2}\nDependency Finished\n/,
 );
 assert.match(normalizeNewlines(out.stdout), /ninja.exe"? -C lib[\\/]bs ?\n/);

tests/build_tests/cli_help/input.js

@@ -1,7 +1,7 @@
 // @ts-check
 
-const assert = require("assert");
-const path = require("path");
+const assert = require("node:assert");
+const path = require("node:path");
 const { exec, normalizeNewlines } = require("../utils.js");
 
 const rescriptPath = path.join(__dirname, "..", "..", "..", "cli", "rescript");
@@ -103,7 +103,7 @@ async function test() {
   // Exits with build help with unknown arg
   await runTest(["build", "-foo"], {
     stdout: "",
-    stderr: 'Error: Unknown option "-foo".\n' + buildHelp,
+    stderr: `Error: Unknown option "-foo".\n${buildHelp}`,
     status: 2,
   });
 
@@ -119,14 +119,14 @@ async function test() {
   // Exits with cli help with unknown command
   await runTest(["built"], {
     stdout: "",
-    stderr: `Error: Unknown command "built".\n` + cliHelp,
+    stderr: `Error: Unknown command "built".\n${cliHelp}`,
     status: 2,
   });
 
   // Exits with build help with unknown args
   await runTest(["-foo"], {
     stdout: "",
-    stderr: 'Error: Unknown option "-foo".\n' + buildHelp,
+    stderr: `Error: Unknown option "-foo".\n${buildHelp}`,
     status: 2,
   });
 
@@ -143,7 +143,7 @@ async function test() {
   // Exits with clean help with unknown arg
   await runTest(["clean", "-foo"], {
     stdout: "",
-    stderr: 'Error: Unknown option "-foo".\n' + cleanHelp,
+    stderr: `Error: Unknown option "-foo".\n${cleanHelp}`,
     status: 2,
   });
 

tests/build_tests/duplicated_symlinked_packages/input.js

@@ -22,21 +22,25 @@ if (process.platform === "win32") {
 
 child_process.execSync(`${rescript_exe} clean`, { cwd: __dirname });
 
-child_process.exec(rescript_exe, { cwd: __dirname }, (err, stdout, stderr) => {
-  const actualErrorOutput = postProcessErrorOutput(stderr.toString());
-  if (updateTests) {
-    fs.writeFileSync(expectedFilePath, actualErrorOutput);
-  } else {
-    const expectedErrorOutput = postProcessErrorOutput(
-      fs.readFileSync(expectedFilePath, { encoding: "utf-8" }),
-    );
-    if (expectedErrorOutput !== actualErrorOutput) {
-      console.error(`The old and new error output aren't the same`);
-      console.error("\n=== Old:");
-      console.error(expectedErrorOutput);
-      console.error("\n=== New:");
-      console.error(actualErrorOutput);
-      process.exit(1);
+child_process.exec(
+  rescript_exe,
+  { cwd: __dirname },
+  (_err, _stdout, stderr) => {
+    const actualErrorOutput = postProcessErrorOutput(stderr.toString());
+    if (updateTests) {
+      fs.writeFileSync(expectedFilePath, actualErrorOutput);
+    } else {
+      const expectedErrorOutput = postProcessErrorOutput(
+        fs.readFileSync(expectedFilePath, { encoding: "utf-8" }),
+      );
+      if (expectedErrorOutput !== actualErrorOutput) {
+        console.error(`The old and new error output aren't the same`);
+        console.error("\n=== Old:");
+        console.error(expectedErrorOutput);
+        console.error("\n=== New:");
+        console.error(actualErrorOutput);
+        process.exit(1);
+      }
     }
-  }
-});
+  },
+);

tests/build_tests/gpr_978/input.js

@@ -1,8 +1,8 @@
 //@ts-check
-const cp = require("child_process");
-const assert = require("assert");
-const fs = require("fs");
-const path = require("path");
+const cp = require("node:child_process");
+const assert = require("node:assert");
+const fs = require("node:fs");
+const path = require("node:path");
 const { rescript_exe } = require("#cli/bin_path");
 
 const output = cp.spawnSync(rescript_exe, { encoding: "utf8", shell: true });

tests/build_tests/in_source/input.js

@@ -5,7 +5,7 @@ const { rescript_exe } = require("#cli/bin_path");
 
 assert.throws(
   () => {
-    const output = child_process.execSync(`${rescript_exe} build -regen`, {
+    const _output = child_process.execSync(`${rescript_exe} build -regen`, {
       cwd: __dirname,
       encoding: "utf8",
     });

tests/build_tests/react_ppx/input.js

@@ -1,5 +1,5 @@
 //@ts-check
-var cp = require("child_process");
-var { rescript_exe } = require("#cli/bin_path");
+const cp = require("node:child_process");
+const { rescript_exe } = require("#cli/bin_path");
 
 cp.execSync(rescript_exe, { cwd: __dirname });

tests/build_tests/super_errors/input.js

@@ -35,7 +35,7 @@ let atLeastOneTaskFailed = false;
 for (const fileName of fixtures) {
   const fullFilePath = path.join(__dirname, "fixtures", fileName);
   const command = `${prefix} -color always ${fullFilePath}`;
-  child_process.exec(command, (err, stdout, stderr) => {
+  child_process.exec(command, (_err, _stdout, stderr) => {
     doneTasksCount++;
     // careful of:
     // - warning test that actually succeeded in compiling (warning's still in stderr, so the code path is shared here)

tests/build_tests/transitive_pinned_dependency1/a/rescript.json

@@ -18,10 +18,6 @@
     "error": true
   },
   "suffix": ".mjs",
-  "bs-dependencies": [
-    "b"
-  ],
-  "pinned-dependencies": [
-    "b"
-  ]
+  "bs-dependencies": ["b"],
+  "pinned-dependencies": ["b"]
 }

tests/gentype_tests/typescript-react-example/src/App.css

@@ -23,6 +23,10 @@
 }
 
 @keyframes App-logo-spin {
-  from { transform: rotate(0deg); }
-  to { transform: rotate(360deg); }
+  from {
+    transform: rotate(0deg);
+  }
+  to {
+    transform: rotate(360deg);
+  }
 }

tests/gentype_tests/typescript-react-example/src/MyMath.ts

@@ -21,7 +21,7 @@ export class AbsoluteValue {
 
 export type stringFunction = (_: string) => string;
 
-export const useColor = (x: "tomato" | "gray"): number => 0;
+export const useColor = (_x: "tomato" | "gray"): number => 0;
 
 export const higherOrder = (foo: (_1: number, _2: number) => number) =>
   foo(3, 4);

tests/gentype_tests/typescript-react-example/src/index.tsx

@@ -1,12 +1,12 @@
 import type * as React from "react";
 import * as ReactDOM from "react-dom";
-import App from "./App";
+import App from "./App.tsx";
 import * as ImportJsValue from "./ImportJsValue.gen";
 import * as Uncurried from "./Uncurried.gen";
 import "./index.css";
 import * as DocStrings from "./Docstrings.gen";
 import Hooks from "./Hooks.gen";
-import * as MyMath from "./MyMath";
+import * as MyMath from "./MyMath.tsx";
 import { Universe_Nested2_Nested3_nested3Value } from "./NestedModules.gen";
 import * as Records from "./Records.gen";
 import * as TestPromise from "./TestPromise.gen";
@@ -126,7 +126,7 @@ type Props = {
   readonly method?: "push" | "replace";
 };
 
-export const make: React.FC<Props> = (x: Props) => {
+export const make: React.FC<Props> = (_x: Props) => {
   return <div />;
 };
 

tsconfig.json

@@ -19,8 +19,5 @@
     "tests/**/input.js",
     "package.json"
   ],
-  "exclude": [
-    "*.res.js",
-    "*.res.mjs",
-  ]
+  "exclude": ["*.res.js", "*.res.mjs"]
 }