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" });