fix: do validation first before actual business logic, like super()

lib/api/readable.js

@@ -159,13 +159,14 @@ class BodyReadable extends Readable {
   }
 
   async dump (opts) {
-    const limit = Number.isFinite(opts?.limit) ? opts.limit : 128 * 1024
     const signal = opts?.signal
 
     if (signal != null && (typeof signal !== 'object' || !('aborted' in signal))) {
       throw new InvalidArgumentError('signal must be an AbortSignal')
     }
 
+    const limit = Number.isFinite(opts?.limit) ? opts.limit : 128 * 1024
+
     signal?.throwIfAborted()
 
     if (this._readableState.closeEmitted) {

lib/dispatcher/agent.js

@@ -22,8 +22,6 @@ function defaultFactory (origin, opts) {
 
 class Agent extends DispatcherBase {
   constructor ({ factory = defaultFactory, connect, ...options } = {}) {
-    super()
-
     if (typeof factory !== 'function') {
       throw new InvalidArgumentError('factory must be a function.')
     }
@@ -32,6 +30,8 @@ class Agent extends DispatcherBase {
       throw new InvalidArgumentError('connect must be a function or an object')
     }
 
+    super()
+
     if (connect && typeof connect !== 'function') {
       connect = { ...connect }
     }

lib/dispatcher/balanced-pool.js

@@ -50,6 +50,10 @@ function defaultFactory (origin, opts) {
 
 class BalancedPool extends PoolBase {
   constructor (upstreams = [], { factory = defaultFactory, ...opts } = {}) {
+    if (typeof factory !== 'function') {
+      throw new InvalidArgumentError('factory must be a function.')
+    }
+
     super()
 
     this[kOptions] = opts
@@ -63,10 +67,6 @@ class BalancedPool extends PoolBase {
       upstreams = [upstreams]
     }
 
-    if (typeof factory !== 'function') {
-      throw new InvalidArgumentError('factory must be a function.')
-    }
-
     this[kFactory] = factory
 
     for (const upstream of upstreams) {

lib/dispatcher/client.js

@@ -101,8 +101,6 @@ class Client extends DispatcherBase {
     maxConcurrentStreams,
     allowH2
   } = {}) {
-    super()
-
     if (keepAlive !== undefined) {
       throw new InvalidArgumentError('unsupported keepAlive, use pipelining=0 instead')
     }
@@ -187,6 +185,8 @@ class Client extends DispatcherBase {
       throw new InvalidArgumentError('maxConcurrentStreams must be a positive integer, greater than 0')
     }
 
+    super()
+
     if (typeof connect !== 'function') {
       connect = buildConnector({
         ...tls,

lib/dispatcher/pool.js

@@ -37,8 +37,6 @@ class Pool extends PoolBase {
     allowH2,
     ...options
   } = {}) {
-    super()
-
     if (connections != null && (!Number.isFinite(connections) || connections < 0)) {
       throw new InvalidArgumentError('invalid connections')
     }
@@ -51,6 +49,8 @@ class Pool extends PoolBase {
       throw new InvalidArgumentError('connect must be a function or an object')
     }
 
+    super()
+
     if (typeof connect !== 'function') {
       connect = buildConnector({
         ...tls,

lib/dispatcher/proxy-agent.js

@@ -25,8 +25,6 @@ function defaultFactory (origin, opts) {
 
 class ProxyAgent extends DispatcherBase {
   constructor (opts) {
-    super()
-
     if (!opts || (typeof opts === 'object' && !(opts instanceof URL) && !opts.uri)) {
       throw new InvalidArgumentError('Proxy uri is mandatory')
     }
@@ -36,6 +34,8 @@ class ProxyAgent extends DispatcherBase {
       throw new InvalidArgumentError('Proxy opts.clientFactory must be a function.')
     }
 
+    super()
+
     const url = this.#getUrl(opts)
     const { href, origin, port, protocol, username, password, hostname: proxyHostname } = url
 

lib/interceptor/dump.js

@@ -14,12 +14,12 @@ class DumpHandler extends DecoratorHandler {
   #handler = null
 
   constructor ({ maxSize }, handler) {
-    super(handler)
-
     if (maxSize != null && (!Number.isFinite(maxSize) || maxSize < 1)) {
       throw new InvalidArgumentError('maxSize must be a number greater than 0')
     }
 
+    super(handler)
+
     this.#maxSize = maxSize ?? this.#maxSize
     this.#handler = handler
   }

lib/mock/mock-client.js

@@ -21,12 +21,12 @@ const { InvalidArgumentError } = require('../core/errors')
  */
 class MockClient extends Client {
   constructor (origin, opts) {
-    super(origin, opts)
-
     if (!opts || !opts.agent || typeof opts.agent.dispatch !== 'function') {
       throw new InvalidArgumentError('Argument opts.agent must implement Agent')
     }
 
+    super(origin, opts)
+
     this[kMockAgent] = opts.agent
     this[kOrigin] = origin
     this[kDispatches] = []

lib/mock/mock-pool.js

@@ -21,12 +21,12 @@ const { InvalidArgumentError } = require('../core/errors')
  */
 class MockPool extends Pool {
   constructor (origin, opts) {
-    super(origin, opts)
-
     if (!opts || !opts.agent || typeof opts.agent.dispatch !== 'function') {
       throw new InvalidArgumentError('Argument opts.agent must implement Agent')
     }
 
+    super(origin, opts)
+
     this[kMockAgent] = opts.agent
     this[kOrigin] = origin
     this[kDispatches] = []