chore: lint with eslint v9

Author: pi0

.eslintignore

@@ -186,11 +186,13 @@ The above package doesn't work outside of Node.js and neither we need any platfo
 We use this proxy for auto-mocking unimplemented internals. Imagine a package does this:
 
 ```js
-const os = require("os");
+const os = require("node:os");
 if (os.platform() === "windows") {
   /* do some fix */
 }
-module.exports = () => "Hello world";
+module.exports = function main() {
+  return "Hello world"
+}
 ```
 
 By aliasing `os` to `unenv/runtime/mock/proxy-cjs`, the code will be compatible with other platforms.

.eslintrc

@@ -0,0 +1,32 @@
+import unjs from "eslint-config-unjs";
+
+// https://github.com/unjs/eslint-config
+export default unjs({
+  ignores: [
+    "runtime"
+  ],
+  rules: {
+  "@typescript-eslint/no-unused-vars": 0,
+  "unicorn/no-null": 0,
+  "@typescript-eslint/no-inferrable-types": 0,
+  "unicorn/prefer-math-trunc": 0,
+  "unicorn/prefer-code-point": 0,
+  "unicorn/text-encoding-identifier-case": 0,
+  "prefer-rest-params": 0,
+  "prefer-spread": 0,
+  "unicorn/number-literal-case": 0,
+  "generator-star-spacing": 0,
+  "indent": 0,
+  "unicorn/no-nested-ternary": 0,
+  "require-await": 0,
+  "unicorn/switch-case-braces": 0,
+  "unicorn/prefer-string-replace-all": 0,
+  "unicorn/prefer-event-target": 0,
+},
+}, {
+  languageOptions: {
+    globals: {
+      Deno: true
+    }
+  }
+});

README.md

@@ -31,7 +31,7 @@ class _AsyncLocalStorage<T> implements asyncHooks.AsyncLocalStorage<T> {
     ...args: TArgs
   ): R {
     this._currentStore = store;
-    const res = callback(...args); // eslint-disable-line n/no-callback-literal
+    const res = callback(...args);
     this._currentStore = undefined;
     return res;
   }
@@ -42,7 +42,7 @@ class _AsyncLocalStorage<T> implements asyncHooks.AsyncLocalStorage<T> {
   ): R {
     const _previousStore = this._currentStore;
     this._currentStore = undefined;
-    const res = callback(...args); // eslint-disable-line n/no-callback-literal
+    const res = callback(...args);
     this._currentStore = _previousStore;
     return res;
   }

eslint.config.mjs

@@ -12,8 +12,8 @@ import * as base64 from "./_base64";
 import * as ieee754 from "./_ieee754";
 
 const customInspectSymbol =
-  typeof Symbol === "function" && typeof Symbol["for"] === "function" // eslint-disable-line dot-notation
-    ? Symbol["for"]("nodejs.util.inspect.custom") // eslint-disable-line dot-notation
+  typeof Symbol === "function" && typeof Symbol["for"] === "function"
+    ? Symbol["for"]("nodejs.util.inspect.custom")
     : null;
 
 export const INSPECT_MAX_BYTES = 50;
@@ -363,7 +363,6 @@ function checked(length) {
 }
 
 export function SlowBuffer(length) {
-  // eslint-disable-next-line eqeqeq
   if (+length != length) {
     length = 0;
   }
@@ -2161,7 +2160,7 @@ function E(sym, getMessage, Base) {
       this.name = `${this.name} [${sym}]`;
       // Access the stack to generate the error message including the error code
       // from the name.
-      this.stack; // eslint-disable-line no-unused-expressions
+      this.stack;
       // Reset the name to the actual name.
       delete this.name;
     }
@@ -2457,7 +2456,7 @@ function isInstance(obj, type) {
 }
 function numberIsNaN(obj) {
   // For IE11 support
-  return obj !== obj; // eslint-disable-line no-self-compare
+  return obj !== obj;
 }
 
 // Create lookup table for `toString('hex')`

src/runtime/node/async_hooks/_async-local-storage.ts

@@ -208,7 +208,7 @@ function _addListener<T extends EventEmitter>(
   ) {
     target._events[type].warned = true;
     // No error code for this since it is a Warning
-    // eslint-disable-next-line no-restricted-syntax
+
     const warning = new Error(
       `[unenv] Possible EventEmitter memory leak detected. ${target._events[type].length} ${type} listeners added. Use emitter.setMaxListeners() to increase limit`,
     ) as Error & {

src/runtime/node/buffer/_buffer.ts

@@ -52,7 +52,7 @@ function runTimeout(fun: () => void) {
   } catch {
     try {
       // When we are in I.E. but the script has been evaled so I.E. doesn't trust the global object when called normally
-      // eslint-disable-next-line no-useless-call
+
       return cachedSetTimeout.call(null, fun, 0);
     } catch {
       // same as above but when it's a version of I.E. that must have the global object for 'this', hopfully our context correct otherwise it will throw a global error
@@ -80,7 +80,7 @@ function runClearTimeout(marker) {
   } catch {
     try {
       // When we are in I.E. but the script has been evaled so I.E. doesn't  trust the global object when called normally
-      // eslint-disable-next-line no-useless-call
+
       return cachedClearTimeout.call(null, marker);
     } catch {
       // same as above but when it's a version of I.E. that must have the global object for 'this', hopfully our context correct otherwise it will throw a global error.

src/runtime/node/events/_events.ts

@@ -13,7 +13,6 @@ interface ArrayOptions {
   signal?: AbortSignal;
 }
 
-// eslint-disable-next-line unicorn/prefer-event-target
 export class _Readable extends EventEmitter implements stream.Readable {
   __unenv__: unknown = true;
 
@@ -102,6 +101,7 @@ export class _Readable extends EventEmitter implements stream.Readable {
     return Promise.resolve();
   }
 
+  // eslint-disable-next-line require-yield
   async *[Symbol.asyncIterator](): AsyncIterableIterator<any> {
     throw createNotImplementedError("Readable.asyncIterator");
   }

src/runtime/node/process/_process.ts

@@ -5,7 +5,6 @@ import { EventEmitter } from "../events";
 
 // Docs: https://nodejs.org/api/stream.html#stream_writable_streams
 // Implementation: https://github.com/nodejs/node/blob/master/lib/internal/streams/writable.js
-// eslint-disable-next-line unicorn/prefer-event-target
 class _Writable extends EventEmitter implements stream.Writable {
   readonly __unenv__ = true;