style: update ESLint configurations and remove unnecessary disable comments

Author: patriksimek

eslint.config.mjs

@@ -3,17 +3,17 @@ import globals from 'globals';
 export default [
 	{
 		languageOptions: {
-			ecmaVersion: 2017,
-			sourceType: "module",
+			ecmaVersion: 2022,
+			sourceType: 'script',
 			globals: {
-				...globals.node
+				...globals.node,
 			},
 			parserOptions: {
 				ecmaFeatures: {
 					globalReturn: true,
-				}
-			}
+				},
+			},
 		},
-		ignores: ["eslint.config.mjs"],
+		ignores: ['eslint.config.mjs', '.claude/worktrees/**'],
 	},
-];
+];

lib/bridge.js

@@ -272,7 +272,7 @@ function thisIsConstructor(obj) {
 	// Note: obj@any(unsafe)
 	const Func = new ThisProxy(obj, TEST_PROXY_HANDLER);
 	try {
-		// eslint-disable-next-line no-new
+		 
 		new Func();
 		return true;
 	} catch (e) {

lib/builtin.js

@@ -88,7 +88,7 @@ const BUILTIN_MODULES = (nmod.builtinModules || Object.getOwnPropertyNames(proce
 
 let EventEmitterReferencingAsyncResourceClass = null;
 if (EventEmitter.EventEmitterAsyncResource) {
-	// eslint-disable-next-line global-require
+	 
 	const {AsyncResource} = require('async_hooks');
 	const kEventEmitter = Symbol('kEventEmitter');
 	class EventEmitterReferencingAsyncResource extends AsyncResource {

lib/compiler.js

@@ -17,7 +17,7 @@ function getCoffeeScriptCompiler(options) {
 	try {
 		// The warning generated by webpack can be disabled by setting:
 		// ignoreWarnings[].message = /Can't resolve 'coffee-script'/
-		/* eslint-disable-next-line global-require */
+		 
 		const coffeeScript = require('coffee-script');
 		return (code, filename) => {
 			return coffeeScript.compile(code, Object.assign({header: false, bare: true}, options));
@@ -40,7 +40,7 @@ function getTypeScriptCompiler(options) {
 	try {
 		// The warning generated by webpack can be disabled by setting:
 		// ignoreWarnings[].message = /Can't resolve 'typescript'/
-		/* eslint-disable-next-line global-require */
+		 
 		const typescript = require('typescript');
 		return (code, filename) => {
 			return typescript.transpileModule(code, {

lib/events.js

@@ -671,7 +671,7 @@ function _addListener(target, type, listener, prepend) {
     if (m > 0 && existing.length > m && !existing.warned) {
       existing.warned = true;
       // No error code for this since it is a Warning
-      // eslint-disable-next-line no-restricted-syntax
+       
       const w = new Error('Possible EventEmitter memory leak detected. ' +
                           `${existing.length} ${String(type)} listeners ` +
                           `added to ${inspect(target, { depth: -1 })}. Use ` +

lib/resolver-compat.js

@@ -19,7 +19,7 @@ const {jsCompiler} = require('./compiler');
  */
 function defaultRequire(moduleName) {
 	// Set module.parser.javascript.commonjsMagicComments=true in your webpack config.
-	// eslint-disable-next-line global-require
+	 
 	return require(/* webpackIgnore: true */ moduleName);
 }
 

lib/resolver.js

@@ -87,7 +87,7 @@ class Resolver {
 
 	getExtensions(vm) {
 		return {
-			// eslint-disable-next-line quote-props
+			 
 			__proto__: null,
 			'.js': this.makeExtensionHandler(vm, 'loadJS'),
 			'.json': this.makeExtensionHandler(vm, 'loadJSON'),
@@ -697,7 +697,7 @@ class DefaultResolver extends Resolver {
 				if (internal && !target.startsWith('../') && !target.startsWith('/')) {
 					let isURL = false;
 					try {
-						// eslint-disable-next-line no-new
+						 
 						new URL(target);
 						isURL = true;
 					} catch (e) {}

lib/script.js

@@ -163,7 +163,7 @@ class VMScript {
 			}
 		} else if (arguments.length > 2) {
 			// We do it this way so that there are no more arguments in the function.
-			// eslint-disable-next-line prefer-rest-params
+			 
 			useOptions = arguments[2] || {__proto__: null};
 			useFileName = options || useOptions.filename;
 		} else {

lib/setup-node-sandbox.js

@@ -166,7 +166,7 @@ function findBestExtensionHandler(filename) {
 }
 
 function createRequireForModule(mod) {
-	// eslint-disable-next-line no-shadow
+	 
 	function require(id) {
 		return requireImpl(mod, id, true);
 	}
@@ -209,7 +209,7 @@ function clearTimer(timer) {
 }
 
 // This is a function and not an arrow function, since the original is also a function
-// eslint-disable-next-line no-shadow
+ 
 global.setTimeout = function setTimeout(callback, delay, ...args) {
 	if (typeof callback !== 'function') throw new LocalTypeError('"callback" argument must be a function');
 	const obj = new Timeout(callback, args);
@@ -228,7 +228,7 @@ global.setTimeout = function setTimeout(callback, delay, ...args) {
 	return obj;
 };
 
-// eslint-disable-next-line no-shadow
+ 
 global.setInterval = function setInterval(callback, interval, ...args) {
 	if (typeof callback !== 'function') throw new LocalTypeError('"callback" argument must be a function');
 	const obj = new Interval();
@@ -247,7 +247,7 @@ global.setInterval = function setInterval(callback, interval, ...args) {
 	return obj;
 };
 
-// eslint-disable-next-line no-shadow
+ 
 global.setImmediate = function setImmediate(callback, ...args) {
 	if (typeof callback !== 'function') throw new LocalTypeError('"callback" argument must be a function');
 	const obj = new Immediate();
@@ -266,17 +266,17 @@ global.setImmediate = function setImmediate(callback, ...args) {
 	return obj;
 };
 
-// eslint-disable-next-line no-shadow
+ 
 global.clearTimeout = function clearTimeout(timeout) {
 	clearTimer(timeout);
 };
 
-// eslint-disable-next-line no-shadow
+ 
 global.clearInterval = function clearInterval(interval) {
 	clearTimer(interval);
 };
 
-// eslint-disable-next-line no-shadow
+ 
 global.clearImmediate = function clearImmediate(immediate) {
 	clearTimer(immediate);
 };
@@ -317,7 +317,7 @@ function addListener(name, handler) {
  * @this process
  * @return {this}
  */
-// eslint-disable-next-line no-shadow
+ 
 function process() {
 	return this;
 }

lib/setup-sandbox.js

@@ -400,7 +400,7 @@ if (
 	throw localUnexpected();
 
 // Fixes buffer unsafe allocation
-/* eslint-disable no-use-before-define */
+ 
 class BufferHandler extends ReadOnlyHandler {
 
 	// SECURITY (GHSA-v37h-5mfm-c47c): forward every arg (token + object)
@@ -430,7 +430,7 @@ class BufferHandler extends ReadOnlyHandler {
 		return apply(LocalBuffer.from, LocalBuffer, args);
 	}
 }
-/* eslint-enable no-use-before-define */
+ 
 
 // SECURITY (post-GHSA-v37h hardening): rebind BufferHandler.prototype.constructor
 // to the throw-always sentinel so `Object.getPrototypeOf(handler).constructor`