lib: revert to using default derived class constructors

Renegade334 · richardlau · commit 10ae8684ea1c · 2025-09-20T06:16:14.000Z
PR-URL: #59650 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
diff --git a/lib/inspector/promises.js b/lib/inspector/promises.js
@@ -3,9 +3,7 @@
 const inspector = require('inspector');
 const { promisify } = require('internal/util');
 
-class Session extends inspector.Session {
-  constructor() { super(); } // eslint-disable-line no-useless-constructor
-}
+class Session extends inspector.Session {}
 Session.prototype.post = promisify(inspector.Session.prototype.post);
 
 module.exports = {
diff --git a/lib/internal/buffer.js b/lib/internal/buffer.js
@@ -953,14 +953,7 @@ function writeFloatBackwards(val, offset = 0) {
   return offset;
 }
 
-class FastBuffer extends Uint8Array {
-  // Using an explicit constructor here is necessary to avoid relying on
-  // `Array.prototype[Symbol.iterator]`, which can be mutated by users.
-  // eslint-disable-next-line no-useless-constructor
-  constructor(bufferOrLength, byteOffset, length) {
-    super(bufferOrLength, byteOffset, length);
-  }
-}
+class FastBuffer extends Uint8Array {}
 
 function addBufferPrototypeMethods(proto) {
   proto.readBigUInt64LE = readBigUInt64LE;
diff --git a/lib/internal/crypto/keys.js b/lib/internal/crypto/keys.js
@@ -235,11 +235,6 @@ const {
   }
 
   class AsymmetricKeyObject extends KeyObject {
-    // eslint-disable-next-line no-useless-constructor
-    constructor(type, handle) {
-      super(type, handle);
-    }
-
     get asymmetricKeyType() {
       return this[kAsymmetricKeyType] ||= this[kHandle].getAsymmetricKeyType();
     }
diff --git a/lib/internal/fs/glob.js b/lib/internal/fs/glob.js
@@ -240,7 +240,6 @@ class Pattern {
 class ResultSet extends SafeSet {
   #root = '.';
   #isExcluded = () => false;
-  constructor(i) { super(i); } // eslint-disable-line no-useless-constructor
 
   setup(root, isExcludedFn) {
     this.#root = root;
diff --git a/lib/internal/modules/esm/module_map.js b/lib/internal/modules/esm/module_map.js
@@ -23,8 +23,6 @@ const { validateString } = require('internal/validators');
  * This cache is *not* used when custom loaders are registered.
  */
 class ResolveCache extends SafeMap {
-  constructor(i) { super(i); } // eslint-disable-line no-useless-constructor
-
   /**
    * Generates the internal serialized cache key and returns it along the actual cache object.
    *
@@ -87,7 +85,6 @@ class ResolveCache extends SafeMap {
  * Cache the results of the `load` step of the module resolution and loading process.
  */
 class LoadCache extends SafeMap {
-  constructor(i) { super(i); } // eslint-disable-line no-useless-constructor
   get(url, type = kImplicitTypeAttribute) {
     validateString(url, 'url');
     validateString(type, 'type');
diff --git a/lib/internal/per_context/primordials.js b/lib/internal/per_context/primordials.js
@@ -401,55 +401,36 @@ primordials.makeSafe = makeSafe;
 
 // Subclass the constructors because we need to use their prototype
 // methods later.
-// Defining the `constructor` is necessary here to avoid the default
-// constructor which uses the user-mutable `%ArrayIteratorPrototype%.next`.
 primordials.SafeMap = makeSafe(
   Map,
-  class SafeMap extends Map {
-    constructor(i) { super(i); } // eslint-disable-line no-useless-constructor
-  },
+  class SafeMap extends Map {},
 );
 primordials.SafeWeakMap = makeSafe(
   WeakMap,
-  class SafeWeakMap extends WeakMap {
-    constructor(i) { super(i); } // eslint-disable-line no-useless-constructor
-  },
+  class SafeWeakMap extends WeakMap {},
 );
 
 primordials.SafeSet = makeSafe(
   Set,
-  class SafeSet extends Set {
-    constructor(i) { super(i); } // eslint-disable-line no-useless-constructor
-  },
+  class SafeSet extends Set {},
 );
 primordials.SafeWeakSet = makeSafe(
   WeakSet,
-  class SafeWeakSet extends WeakSet {
-    constructor(i) { super(i); } // eslint-disable-line no-useless-constructor
-  },
+  class SafeWeakSet extends WeakSet {},
 );
 
 primordials.SafeFinalizationRegistry = makeSafe(
   FinalizationRegistry,
-  class SafeFinalizationRegistry extends FinalizationRegistry {
-    // eslint-disable-next-line no-useless-constructor
-    constructor(cleanupCallback) { super(cleanupCallback); }
-  },
+  class SafeFinalizationRegistry extends FinalizationRegistry {},
 );
 primordials.SafeWeakRef = makeSafe(
   WeakRef,
-  class SafeWeakRef extends WeakRef {
-    // eslint-disable-next-line no-useless-constructor
-    constructor(target) { super(target); }
-  },
+  class SafeWeakRef extends WeakRef {},
 );
 
 const SafePromise = makeSafe(
   Promise,
-  class SafePromise extends Promise {
-    // eslint-disable-next-line no-useless-constructor
-    constructor(executor) { super(executor); }
-  },
+  class SafePromise extends Promise {},
 );
 
 /**
diff --git a/lib/internal/readline/interface.js b/lib/internal/readline/interface.js
@@ -348,10 +348,6 @@ ObjectSetPrototypeOf(InterfaceConstructor.prototype, EventEmitter.prototype);
 ObjectSetPrototypeOf(InterfaceConstructor, EventEmitter);
 
 class Interface extends InterfaceConstructor {
-  // eslint-disable-next-line no-useless-constructor
-  constructor(input, output, completer, terminal) {
-    super(input, output, completer, terminal);
-  }
   get columns() {
     if (this.output?.columns) return this.output.columns;
     return Infinity;
diff --git a/lib/internal/repl/utils.js b/lib/internal/repl/utils.js
@@ -92,10 +92,6 @@ function isRecoverableError(e, code) {
     .extend(
       (Parser) => {
         return class extends Parser {
-          // eslint-disable-next-line no-useless-constructor
-          constructor(options, input, startPos) {
-            super(options, input, startPos);
-          }
           nextToken() {
             super.nextToken();
             if (this.type === tt.eof)
diff --git a/lib/readline/promises.js b/lib/readline/promises.js
@@ -27,10 +27,6 @@ const {
 let addAbortListener;
 
 class Interface extends _Interface {
-  // eslint-disable-next-line no-useless-constructor
-  constructor(input, output, completer, terminal) {
-    super(input, output, completer, terminal);
-  }
   question(query, options = kEmptyObject) {
     return new Promise((resolve, reject) => {
       let cb = resolve;

Original file line number	Diff line number	Diff line change
`@@ -235,11 +235,6 @@ const {`
`235`	`235`	`}`
`236`	`236`
`237`	`237`	`class AsymmetricKeyObject extends KeyObject {`
`238`		`- // eslint-disable-next-line no-useless-constructor`
`239`		`- constructor(type, handle) {`
`240`		`- super(type, handle);`
`241`		`- }`
`242`		`-`
`243`	`238`	`get asymmetricKeyType() {`
`244`	`239`	`return this[kAsymmetricKeyType] \|\|= this[kHandle].getAsymmetricKeyType();`
`245`	`240`	`}`