enable lint for js

Author: cometkim

.github/workflows/get_artifact_dir_name.js

@@ -1,5 +1,5 @@
-const fs = require("fs");
-const os = require("os");
+const fs = require("node:fs");
+const os = require("node:os");
 
 const { dirName: artifactDirName } = require("../../cli/bin_path.js");
 

.github/workflows/prepare_package_upload.js

@@ -1,5 +1,5 @@
-const fs = require("fs");
-const os = require("os");
+const fs = require("node:fs");
+const os = require("node:os");
 
 const packageSpec = require("rescript/package.json");
 const { version } = packageSpec;

analysis/examples/example-project/types.json

@@ -8,6 +8,6 @@
     }
   ],
   "custom": [
-    {"module": "Belt_HashMapInt", "path": [], "name": "t", "args": 1}
+    { "module": "Belt_HashMapInt", "path": [], "name": "t", "args": 1 }
   ]
-}
+}

analysis/examples/larger-project/.watchmanconfig

@@ -7,10 +7,13 @@
     "useIgnoreFile": true
   },
   "linter": {
-    "enabled": false
+    "enabled": true,
+    "rules": {
+      "recommended": true
+    }
   },
   "organizeImports": {
-    "enabled": false
+    "enabled": true
   },
   "formatter": {
     "enabled": true,
@@ -19,23 +22,7 @@
     "indentWidth": 2,
     "lineEnding": "lf",
     "lineWidth": 80,
-    "attributePosition": "auto",
-    "ignore": [
-      "tests/build_tests/**",
-      "tests/tests/**",
-      "tests/tools_tests/**",
-      "tests/analysis_tests/**",
-      "tests/docstring_tests/**",
-      "analysis/examples/**",
-      "analysis/reanalyze/examples/**",
-      "lib/**",
-      "ninja/**",
-      "playground/**",
-      "**/*.bs.js",
-      "**/*.res.js",
-      "**/*.gen.ts*",
-      "package.json"
-    ]
+    "attributePosition": "auto"
   },
   "javascript": {
     "formatter": {
@@ -49,5 +36,25 @@
       "quoteStyle": "double",
       "attributePosition": "auto"
     }
+  },
+  "files": {
+    "include": ["tests/build_tests/**/node_modules/**"],
+    "ignore": [
+      "tests/analysis_tests/**/src/**",
+      "tests/build_tests/**/src/**",
+      "tests/docstring_tests/**",
+      "tests/tests/**/src/**",
+      "tests/tools_tests/**/src/**",
+      "analysis/examples/**/src/**",
+      "lib/es6/**",
+      "lib/js/**",
+      "ninja/**",
+      "playground/packages/**",
+      "*.bs.js",
+      "*.res.js",
+      "*.res.mjs",
+      "*.gen.ts*",
+      "package.json"
+    ]
   }
 }

biome.json

@@ -1,59 +1,61 @@
-//@ts-check
+// @ts-check
 
-const path = require("path");
+const path = require("node:path");
 
 /**
- * @type{string}
- *
  * For compatibility reasons, if the architecture is x64, omit it from the bin directory name.
  * So we'll have "darwin", "linux" and "win32" for x64 arch,
  * but "darwinarm64" and "linuxarm64" for arm64 arch.
  * Also, we do not have Windows ARM binaries yet. But the x64 binaries do work on Windows 11 ARM.
  * So omit the architecture for Windows, too.
  */
-const binDirName =
+const platformName =
   process.arch === "x64" || process.platform === "win32"
     ? process.platform
     : process.platform + process.arch;
 
 /**
- *
- * @type{string}
+ * @type {string}
  */
-const binAbsolutePath = path.join(__dirname, "..", binDirName);
+const platformDir = path.resolve(__dirname, "..", platformName);
 
 /**
- * @type{string}
+ * @type {string}
  */
-const bsc_exe = path.join(binAbsolutePath, "bsc.exe");
+const bsc_exe = path.join(platformDir, "bsc.exe");
 
 /**
- * @type{string}
+ * @type {string}
  */
-const ninja_exe = path.join(binAbsolutePath, "ninja.exe");
+const ninja_exe = path.join(platformDir, "ninja.exe");
 
 /**
- * @type{string}
+ * @type {string}
  */
-const rescript_exe = path.join(binAbsolutePath, "rescript.exe");
+const rescript_exe = path.join(platformDir, "rescript.exe");
 
 /**
- * @type{string}
+ * @type {string}
  */
-const rescript_tools_exe = path.join(binAbsolutePath, "rescript-tools.exe");
+const rescript_tools_exe = path.join(platformDir, "rescript-tools.exe");
 
 /**
- * @type{string}
+ * @type {string}
  */
 const rescript_editor_analysis_exe = path.join(
-  binAbsolutePath,
+  platformDir,
   "rescript-editor-analysis.exe",
 );
 
-exports.dirName = binDirName;
-exports.absolutePath = binAbsolutePath;
+/**
+ * @type {string}
+ */
+const rewatch_exe = path.join(platformDir, "rewatch.exe");
+
+exports.platformDir = platformDir;
 exports.bsc_exe = bsc_exe;
 exports.ninja_exe = ninja_exe;
 exports.rescript_exe = rescript_exe;
 exports.rescript_tools_exe = rescript_tools_exe;
 exports.rescript_editor_analysis_exe = rescript_editor_analysis_exe;
+exports.rewatch_exe = rewatch_exe;

cli/bin_path.js

@@ -1,10 +1,13 @@
 #!/usr/bin/env node
+
+// @ts-check
+
 "use strict";
 
-var child_process = require("child_process");
-var { bsc_exe } = require("./bin_path");
+const child_process = require("child_process");
+const { bsc_exe } = require("./bin_path.js");
 
-var delegate_args = process.argv.slice(2);
+const delegate_args = process.argv.slice(2);
 
 try {
   child_process.execFileSync(bsc_exe, delegate_args, { stdio: "inherit" });

cli/bsc

@@ -1,21 +1,22 @@
 #!/usr/bin/env node
+
 //@ts-check
+
 "use strict";
 
-/**
- * This script is supposed to be running in project root directory
- * It matters since we need read .sourcedirs(location)
- * and its content are file/directories with regard to project root
- */
+// This script is supposed to be running in project root directory
+// It matters since we need read .sourcedirs(location)
+// and its content are file/directories with regard to project root
 
-var { bsc_exe, rescript_exe } = require("./bin_path.js");
-var bsb = require("./rescript_bsb.js");
+const tty = require("node:tty");
+const { bsc_exe, rescript_exe } = require("./bin_path.js");
+const bsb = require("./rescript_bsb.js");
 
-var cwd = process.cwd();
+const cwd = process.cwd();
 process.env.BSB_PROJECT_ROOT = cwd;
 
 if (process.env.FORCE_COLOR === undefined) {
-  if (require("tty").isatty(1)) {
+  if (tty.isatty(1)) {
     process.env.FORCE_COLOR = "1";
     process.env.NINJA_ANSI_FORCED = "1";
   }

cli/rescript

@@ -1,13 +1,12 @@
 #!/usr/bin/env node
-//@ts-check
-"use strict";
 
-const path = require("path");
-const child_process = require("child_process");
+// @ts-check
+
+"use strict";
 
-const { absolutePath: binAbsolutePath } = require("./bin_path");
-const rewatchExe = path.join(binAbsolutePath, "rescript-tools.exe");
+const child_process = require("node:child_process");
+const { rescript_tools_exe } = require("./bin_path.js");
 
 const args = process.argv.slice(2);
 
-child_process.spawnSync(rewatchExe, args, { stdio: "inherit" });
+child_process.spawnSync(rescript_tools_exe, args, { stdio: "inherit" });

cli/rescript-tools

@@ -1,4 +1,5 @@
-//@ts-check
+// @ts-check
+
 class StringBuilder {
   constructor() {
     this.val = "";
@@ -13,23 +14,32 @@ class StringBuilder {
     return this;
   }
 }
+
 class ArgError extends Error {}
 
 /**
- *
  * @param {string} s
  */
 function bad_arg(s) {
   throw new ArgError(s);
 }
 
 /**
- * @typedef {{val : string}} stringref
- * @typedef {{val : boolean}} boolref
- * @typedef {{kind:"Unit_call",data : ()=>void } | {kind : "Unit_set", data : boolref}} unit_action
- * @typedef {{kind:"String_call",data:(s : string)=>void} | {kind : "String_set",data: stringref}} string_action
- * @typedef {{kind:"Unit",data : unit_action } | {kind:"String", data: string_action}} action
- * @typedef {Array<[string,action,string]>} specs
+ * @typedef {{ val: string }} stringref
+ * @typedef {{ val: boolean }} boolref
+ * @typedef {(
+ *   | { kind: "Unit_call", data: () => void }
+ *   | { kind: "Unit_set", data: boolref }
+ * )} unit_action
+ * @typedef {(
+ *   | { kind: "String_call", data: (s: string) => void }
+ *   | {kind: "String_set", data: stringref }
+ * )} string_action
+ * @typedef {(
+ *   | { kind: "Unit", data: unit_action }
+ *   | { kind: "String", data: string_action}
+ * )} action
+ * @typedef {Array<[string, action, string]>} specs
  * @param {StringBuilder} b
  * @param {string} usage
  * @param {specs} specs
@@ -39,33 +49,32 @@ function usage_b(b, usage, specs) {
   if (specs.length === 0) {
     return;
   }
-  b.add(`\nOptions:\n`);
-  var max_col = 0;
-  for (let [key] of specs) {
+  b.add("\nOptions:\n");
+  let max_col = 0;
+  for (const [key] of specs) {
     if (key.length > max_col) {
       max_col = key.length;
     }
   }
   for (let i = 0; i < specs.length; i++) {
-    let [key, _, doc] = specs[i];
+    const [key, _, doc] = specs[i];
     if (!doc.startsWith("*internal*")) {
       b.add("  ")
         .add(key)
         .add(" ".repeat(max_col - key.length + 2));
       let cur = 0;
-      let doc_length = doc.length;
+      const doc_length = doc.length;
       while (cur < doc_length) {
         if (cur !== 0) {
           b.add("\n").add(" ".repeat(max_col + 4));
         }
-        let i = doc.indexOf("\n", cur);
+        const i = doc.indexOf("\n", cur);
         if (i < 0) {
           b.add(doc.substring(cur));
           break;
-        } else {
-          b.add(doc.substr(cur, i - cur));
-          cur = i + 1;
         }
+        b.add(doc.substring(cur, i - cur));
+        cur = i + 1;
       }
       b.add("\n");
     }
@@ -79,7 +88,7 @@ function usage_b(b, usage, specs) {
  * @param {specs} specs
  */
 function stop_raise(usage, error, specs) {
-  var b = new StringBuilder();
+  const b = new StringBuilder();
   switch (error.kind) {
     case "Unknown":
       if (["-help", "--help", "-h"].includes(error.data)) {
@@ -89,8 +98,10 @@ function stop_raise(usage, error, specs) {
       } else {
         b.add(`Unknown option "${error.data}".\n'`);
       }
+      break;
     case "Missing":
       b.add(`Option "${error.data}" needs an argument.\n'`);
+      break;
   }
   usage_b(b, usage, specs);
   bad_arg(b.val);
@@ -114,15 +125,15 @@ function parse_exn(
   // first 3 are [node, rescript, subcommand]
   finish = argv.length,
 ) {
-  var current = start;
-  var list = [];
+  let current = start;
+  const list = [];
   while (current < finish) {
-    let s = argv[current];
+    const s = argv[current];
     ++current;
     if (s !== "" && s[0] === "-") {
-      var out = specs.find(([flag]) => flag === s);
+      const out = specs.find(([flag]) => flag === s);
       if (out !== undefined) {
-        let [_, action] = out;
+        const [_, action] = out;
         switch (action.kind) {
           case "Unit":
             switch (action.data.kind) {
@@ -139,7 +150,7 @@ function parse_exn(
             if (current >= finish) {
               stop_raise(usage, { kind: "Missing", data: s }, specs);
             } else {
-              let arg = argv[current];
+              const arg = argv[current];
               ++current;
               switch (action.data.kind) {
                 case "String_call":