chore: update to eslint v9

Author: pi0

.eslintignore

@@ -1,11 +1,11 @@
 #!/usr/bin/env node
 
-const { resolve } = require("path");
+const { resolve } = require("node:path");
 
 const script = process.argv.splice(2, 1)[0];
 
 if (!script) {
-  // eslint-disable-next-line no-console
+   
   console.error("Usage: jiti <path> [...arguments]");
   process.exit(1);
 }

.eslintrc

@@ -0,0 +1,10 @@
+import unjs from "eslint-config-unjs";
+
+export default unjs({
+  ignores: [
+    "test/fixtures/error-*"
+  ],
+  rules: {
+    "unicorn/no-null": 0
+  },
+});

bin/jiti.js

@@ -2,7 +2,7 @@ function onError(err) {
   throw err; /* ↓ Check stack trace ↓ */
 }
 
-module.exports = function (filename, opts) {
+module.exports = function jiti(filename, opts) {
   const jiti = require("../dist/jiti");
 
   opts = { onError, ...opts };

eslint.config.mjs

@@ -18,8 +18,8 @@
     "dev": "pnpm clean && pnpm webpack --watch",
     "jiti": "JITI_DEBUG=1 JITI_CACHE=false JITI_REQUIRE_CACHE=false ./bin/jiti.js",
     "jiti:legacy": "JITI_DEBUG=1 npx node@12 ./bin/jiti.js",
-    "lint": "eslint --ext .ts,.js . && prettier -c src lib test stubs",
-    "lint:fix": "eslint --fix --ext .ts,.js . && prettier -w src lib test stubs",
+    "lint": "eslint . && prettier -c src lib test stubs",
+    "lint:fix": "eslint --fix . && prettier -w src lib test stubs",
     "release": "pnpm build && pnpm test && changelogen --release --push && npm publish",
     "test": "pnpm lint && vitest run --coverage && pnpm test:bun",
     "test:bun": "bun --bun test test/bun"
@@ -53,8 +53,8 @@
     "create-require": "^1.1.1",
     "destr": "^2.0.3",
     "escape-string-regexp": "^5.0.0",
-    "eslint": "^8.57.0",
-    "eslint-config-unjs": "^0.2.1",
+    "eslint": "^9.1.1",
+    "eslint-config-unjs": "0.3.0-rc.7",
     "esm": "^3.2.25",
     "estree-walker": "^3.0.3",
     "execa": "^8.0.1",
@@ -78,4 +78,4 @@
     "webpack-cli": "^5.1.4"
   },
   "packageManager": "pnpm@8.15.8"
-}
+}

lib/index.js

@@ -1,9 +1,9 @@
-import { existsSync, readFileSync, writeFileSync, mkdirSync } from "fs";
-import { Module, builtinModules } from "module";
-import { performance } from "perf_hooks";
-import { platform } from "os";
-import vm from "vm";
-import { fileURLToPath, pathToFileURL } from "url";
+import { existsSync, readFileSync, writeFileSync, mkdirSync } from "node:fs";
+import { Module, builtinModules } from "node:module";
+import { performance } from "node:perf_hooks";
+import { platform } from "node:os";
+import vm from "node:vm";
+import { fileURLToPath, pathToFileURL } from "node:url";
 import { dirname, join, basename, extname } from "pathe";
 import { destr } from "destr";
 import escapeStringRegexp from "escape-string-regexp";
@@ -114,7 +114,6 @@ export default function createJITI(
 
   function debug(...args: string[]) {
     if (opts.debug) {
-      // eslint-disable-next-line no-console
       console.log("[jiti]", ...args);
     }
   }
@@ -151,7 +150,9 @@ export default function createJITI(
   const tryResolve = (id: string, options?: { paths?: string[] }) => {
     try {
       return nativeRequire.resolve(id, options);
-    } catch {}
+    } catch {
+      // Ignore errors
+    }
   };
 
   const _url = pathToFileURL(_filename);

package.json

@@ -1,4 +1,4 @@
-import { pathToFileURL } from "url";
+import { pathToFileURL } from "node:url";
 import { smart } from "@babel/template";
 import type { NodePath, PluginObj } from "@babel/core";
 import type { Statement, MemberExpression } from "@babel/types";

pnpm-lock.yaml

@@ -1,6 +1,6 @@
-import { lstatSync, accessSync, constants, readFileSync } from "fs";
-import { createHash } from "crypto";
-import { tmpdir } from "os";
+import { lstatSync, accessSync, constants, readFileSync } from "node:fs";
+import { createHash } from "node:crypto";
+import { tmpdir } from "node:os";
 import { join } from "pathe";
 import type { PackageJson } from "pkg-types";
 
@@ -63,8 +63,12 @@ export function readNearestPackageJSON(path: string): PackageJson | undefined {
       const pkg = readFileSync(join(path, "package.json"), "utf8");
       try {
         return JSON.parse(pkg);
-      } catch {}
+      } catch {
+        // Ignore errors
+      }
       break;
-    } catch {}
+    } catch {
+      // Ignore errors
+    }
   }
 }

src/jiti.ts

@@ -1,4 +1,4 @@
-import { join, resolve, dirname } from "path";
+import { join, resolve, dirname } from "node:path";
 import { execa } from "execa";
 import { describe, it, expect } from "vitest";
 import fg from "fast-glob";

src/plugins/babel-plugin-transform-import-meta.ts

@@ -1,6 +1,5 @@
-// eslint-disable-next-line import/no-duplicates
 import imported from "./file.json";
-// eslint-disable-next-line import/no-duplicates
+
 import importedWithAssertion from "./file.json" assert { type: "json" };
 
 const required = require("./file.json");

src/utils.ts

@@ -1,3 +1,3 @@
-import path from "path";
+import path from "node:path";
 
 console.log("Mixed works for:", path.resolve("."));

test/fixtures.test.ts

@@ -1,4 +1,3 @@
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
 import fs from "node:fs";
 
 console.log("exists:", fs.existsSync("index.js"));

test/fixtures/json/index.ts

@@ -2,6 +2,7 @@
 console.log("Optional chaining:", ({} as any)?.foo?.bar);
 
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Nullish_coalescing_operator
+// eslint-disable-next-line no-constant-binary-expression
 console.log("Nullish coalescing:", 0 ?? 42);
 
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Logical_OR_assignment

test/fixtures/mixed/esm.js

@@ -1,5 +1,5 @@
-const path = require("path");
-const fsp = require("fs/promises");
+const path = require("node:path");
+const fsp = require("node:fs/promises");
 
 const TerserPlugin = require("terser-webpack-plugin");