lib: add trailing commas to more internal files

PR-URL: #46811
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

lib/.eslintrc.yaml

@@ -251,10 +251,8 @@ globals:
   primordials: false
 overrides:
   - files:
-      - ./*/promises.js
-      - ./_stream_*.js
-      - ./_tls_common.js
-      - ./assert/*.js
+      - ./*/*.js
+      - ./_*.js
       - ./child_process.js
       - ./cluster.js
       - ./console.js
@@ -276,7 +274,7 @@ overrides:
       - ./internal/js_stream_socket.js
       - ./internal/mime.js
       - ./internal/modules/*.js
-      - ./internal/per_context/messageport.js
+      - ./internal/per_context/*.js
       - ./internal/perf/*.js
       - ./internal/policy/*.js
       - ./internal/priority_queue.js
@@ -285,7 +283,7 @@ overrides:
       - ./internal/readme.md
       - ./internal/repl.js
       - ./internal/repl/*.js
-      - ./internal/source_map/prepare_stack_trace.js
+      - ./internal/source_map/*.js
       - ./internal/streams/*.js
       - ./internal/structured_clone.js
       - ./internal/test/*.js
@@ -298,15 +296,12 @@ overrides:
       - ./internal/webidl.js
       - ./internal/webstreams/*.js
       - ./module.js
-      - ./path/*.js
       - ./process.js
       - ./punycode.js
       - ./repl.js
-      - ./stream/*.js
       - ./sys.js
       - ./test.js
       - ./tls.js
       - ./url.js
-      - ./util/*.js
     rules:
       comma-dangle: [error, always-multiline]

lib/_http_agent.js

@@ -244,7 +244,7 @@ Agent.prototype.addRequest = function addRequest(req, options, port/* legacy */,
       __proto__: null,
       host: options,
       port,
-      localAddress
+      localAddress,
     };
   }
 
@@ -533,5 +533,5 @@ function asyncResetHandle(socket) {
 
 module.exports = {
   Agent,
-  globalAgent: new Agent()
+  globalAgent: new Agent(),
 };

lib/_http_client.js

@@ -58,7 +58,7 @@ const {
 const {
   kUniqueHeaders,
   parseUniqueHeadersOption,
-  OutgoingMessage
+  OutgoingMessage,
 } = require('_http_outgoing');
 const Agent = require('_http_agent');
 const { Buffer } = require('buffer');
@@ -78,7 +78,7 @@ const {
   ERR_INVALID_ARG_TYPE,
   ERR_INVALID_HTTP_TOKEN,
   ERR_INVALID_PROTOCOL,
-  ERR_UNESCAPED_CHARACTERS
+  ERR_UNESCAPED_CHARACTERS,
 } = codes;
 const {
   validateInteger,
@@ -87,7 +87,7 @@ const {
 const { getTimerDuration } = require('internal/timers');
 const {
   DTRACE_HTTP_CLIENT_REQUEST,
-  DTRACE_HTTP_CLIENT_RESPONSE
+  DTRACE_HTTP_CLIENT_RESPONSE,
 } = require('internal/dtrace');
 
 const {
@@ -650,7 +650,7 @@ function parserOnIncomingClient(res, shouldKeepAlive) {
       httpVersionMajor: res.httpVersionMajor,
       httpVersionMinor: res.httpVersionMinor,
       headers: res.headers,
-      rawHeaders: res.rawHeaders
+      rawHeaders: res.rawHeaders,
     });
 
     return 1;  // Skip body but don't treat as Upgrade.
@@ -977,5 +977,5 @@ ClientRequest.prototype.clearTimeout = function clearTimeout(cb) {
 };
 
 module.exports = {
-  ClientRequest
+  ClientRequest,
 };

lib/_http_common.js

@@ -37,7 +37,7 @@ const incoming = require('_http_incoming');
 const {
   IncomingMessage,
   readStart,
-  readStop
+  readStop,
 } = incoming;
 
 const kIncomingMessage = Symbol('IncomingMessage');

lib/_http_incoming.js

@@ -27,7 +27,7 @@ const {
   StringPrototypeCharCodeAt,
   StringPrototypeSlice,
   StringPrototypeToLowerCase,
-  Symbol
+  Symbol,
 } = primordials;
 
 const { Readable, finished } = require('stream');
@@ -55,7 +55,7 @@ function IncomingMessage(socket) {
 
   if (socket) {
     streamOptions = {
-      highWaterMark: socket.readableHighWaterMark
+      highWaterMark: socket.readableHighWaterMark,
     };
   }
 
@@ -104,7 +104,7 @@ ObjectDefineProperty(IncomingMessage.prototype, 'connection', {
   },
   set: function(val) {
     this.socket = val;
-  }
+  },
 });
 
 ObjectDefineProperty(IncomingMessage.prototype, 'headers', {
@@ -124,7 +124,7 @@ ObjectDefineProperty(IncomingMessage.prototype, 'headers', {
   },
   set: function(val) {
     this[kHeaders] = val;
-  }
+  },
 });
 
 ObjectDefineProperty(IncomingMessage.prototype, 'headersDistinct', {
@@ -144,7 +144,7 @@ ObjectDefineProperty(IncomingMessage.prototype, 'headersDistinct', {
   },
   set: function(val) {
     this[kHeadersDistinct] = val;
-  }
+  },
 });
 
 ObjectDefineProperty(IncomingMessage.prototype, 'trailers', {
@@ -164,7 +164,7 @@ ObjectDefineProperty(IncomingMessage.prototype, 'trailers', {
   },
   set: function(val) {
     this[kTrailers] = val;
-  }
+  },
 });
 
 ObjectDefineProperty(IncomingMessage.prototype, 'trailersDistinct', {
@@ -184,7 +184,7 @@ ObjectDefineProperty(IncomingMessage.prototype, 'trailersDistinct', {
   },
   set: function(val) {
     this[kTrailersDistinct] = val;
-  }
+  },
 });
 
 IncomingMessage.prototype.setTimeout = function setTimeout(msecs, callback) {
@@ -452,5 +452,5 @@ function onError(self, error, cb) {
 module.exports = {
   IncomingMessage,
   readStart,
-  readStop
+  readStop,
 };

lib/_http_outgoing.js

@@ -53,7 +53,7 @@ const {
 } = require('_http_common');
 const {
   defaultTriggerAsyncIdScope,
-  symbols: { async_id_symbol }
+  symbols: { async_id_symbol },
 } = require('internal/async_hooks');
 const {
   codes: {
@@ -70,9 +70,9 @@ const {
     ERR_STREAM_ALREADY_FINISHED,
     ERR_STREAM_WRITE_AFTER_END,
     ERR_STREAM_NULL_VALUES,
-    ERR_STREAM_DESTROYED
+    ERR_STREAM_DESTROYED,
   },
-  hideStackFrames
+  hideStackFrames,
 } = require('internal/errors');
 const { validateString } = require('internal/validators');
 const { isUint8Array } = require('internal/util/types');
@@ -177,36 +177,36 @@ ObjectDefineProperty(OutgoingMessage.prototype, 'writableFinished', {
       this.outputSize === 0 &&
       (!this.socket || this.socket.writableLength === 0)
     );
-  }
+  },
 });
 
 ObjectDefineProperty(OutgoingMessage.prototype, 'writableObjectMode', {
   __proto__: null,
   get() {
     return false;
-  }
+  },
 });
 
 ObjectDefineProperty(OutgoingMessage.prototype, 'writableLength', {
   __proto__: null,
   get() {
     return this.outputSize + (this.socket ? this.socket.writableLength : 0);
-  }
+  },
 });
 
 ObjectDefineProperty(OutgoingMessage.prototype, 'writableHighWaterMark', {
   __proto__: null,
   get() {
     return this.socket ? this.socket.writableHighWaterMark : HIGH_WATER_MARK;
-  }
+  },
 });
 
 ObjectDefineProperty(OutgoingMessage.prototype, 'writableCorked', {
   __proto__: null,
   get() {
     const corked = this.socket ? this.socket.writableCorked : 0;
     return corked + this[kCorked];
-  }
+  },
 });
 
 ObjectDefineProperty(OutgoingMessage.prototype, '_headers', {
@@ -227,7 +227,7 @@ ObjectDefineProperty(OutgoingMessage.prototype, '_headers', {
         headers[StringPrototypeToLowerCase(name)] = [name, val[name]];
       }
     }
-  }, 'OutgoingMessage.prototype._headers is deprecated', 'DEP0066')
+  }, 'OutgoingMessage.prototype._headers is deprecated', 'DEP0066'),
 });
 
 ObjectDefineProperty(OutgoingMessage.prototype, 'connection', {
@@ -237,7 +237,7 @@ ObjectDefineProperty(OutgoingMessage.prototype, 'connection', {
   },
   set: function(val) {
     this.socket = val;
-  }
+  },
 });
 
 ObjectDefineProperty(OutgoingMessage.prototype, '_headerNames', {
@@ -272,7 +272,7 @@ ObjectDefineProperty(OutgoingMessage.prototype, '_headerNames', {
           header[0] = val[keys[i]];
       }
     }
-  }, 'OutgoingMessage.prototype._headerNames is deprecated', 'DEP0066')
+  }, 'OutgoingMessage.prototype._headerNames is deprecated', 'DEP0066'),
 });
 
 
@@ -368,7 +368,7 @@ OutgoingMessage.prototype._send = function _send(data, encoding, callback, byteL
       this.outputData.unshift({
         data: header,
         encoding: 'latin1',
-        callback: null
+        callback: null,
       });
       this.outputSize += header.length;
       this._onPendingData(header.length);
@@ -419,7 +419,7 @@ function _storeHeader(firstLine, headers) {
     date: false,
     expect: false,
     trailer: false,
-    header: firstLine
+    header: firstLine,
   };
 
   if (headers) {
@@ -806,19 +806,19 @@ ObjectDefineProperty(OutgoingMessage.prototype, 'headersSent', {
   __proto__: null,
   configurable: true,
   enumerable: true,
-  get: function() { return !!this._header; }
+  get: function() { return !!this._header; },
 });
 
 ObjectDefineProperty(OutgoingMessage.prototype, 'writableEnded', {
   __proto__: null,
-  get: function() { return this.finished; }
+  get: function() { return this.finished; },
 });
 
 ObjectDefineProperty(OutgoingMessage.prototype, 'writableNeedDrain', {
   __proto__: null,
   get: function() {
     return !this.destroyed && !this.finished && this[kNeedDrain];
-  }
+  },
 });
 
 const crlf_buf = Buffer.from('\r\n');
@@ -1171,5 +1171,5 @@ module.exports = {
   parseUniqueHeadersOption,
   validateHeaderName,
   validateHeaderValue,
-  OutgoingMessage
+  OutgoingMessage,
 };