lib: enforce use of trailing commas

PR-URL: #46881 Reviewed-By: Debadree Chatterjee <debadree333@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Jacob Smith <jacob@frende.me>
nodejs · Mar 14, 2023 · 05614f8 · 05614f8
1 parent debc0ad
commit 05614f8
Show file tree

Hide file tree

Showing 67 changed files with 251 additions and 285 deletions.
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
@@ -2,13 +2,6 @@ env:
   es6: true
 
 rules:
-  comma-dangle: [error, {
-    arrays: always-multiline,
-    exports: always-multiline,
-    functions: always-multiline,
-    imports: always-multiline,
-    objects: only-multiline,
-  }]
   prefer-object-spread: error
   no-buffer-constructor: error
   no-mixed-operators:
@@ -261,30 +254,3 @@ globals:
   module: false
   internalBinding: false
   primordials: false
-overrides:
-  - files:
-      - ./*/*.js
-      - ./*.js
-      - ./internal/child_process/*.js
-      - ./internal/cluster/*.js
-      - ./internal/debugger/*.js
-      - ./internal/events/*.js
-      - ./internal/fs/*.js
-      - ./internal/modules/*.js
-      - ./internal/per_context/*.js
-      - ./internal/perf/*.js
-      - ./internal/policy/*.js
-      - ./internal/process/*.js
-      - ./internal/readline/*.js
-      - ./internal/readme.md
-      - ./internal/repl/*.js
-      - ./internal/source_map/*.js
-      - ./internal/streams/*.js
-      - ./internal/test/*.js
-      - ./internal/test_runner/**/*.js
-      - ./internal/tls/*.js
-      - ./internal/util/parse_args/*.js
-      - ./internal/watch_mode/*.js
-      - ./internal/webstreams/*.js
-    rules:
-      comma-dangle: [error, always-multiline]
diff --git a/lib/internal/assert/assertion_error.js b/lib/internal/assert/assertion_error.js
@@ -38,7 +38,7 @@ const kReadableOperator = {
     'Expected "actual" not to be reference-equal to "expected":',
   notDeepEqual: 'Expected "actual" not to be loosely deep-equal to:',
   notIdentical: 'Values have same structure but are not reference-equal:',
-  notDeepEqualUnequal: 'Expected values not to be loosely deep-equal:'
+  notDeepEqualUnequal: 'Expected values not to be loosely deep-equal:',
 };
 
 // Comparing short primitives should just show === / !== instead of using the
@@ -330,11 +330,11 @@ class AssertionError extends Error {
       stackStartFn,
       details,
       // Compatibility with older versions.
-      stackStartFunction
+      stackStartFunction,
     } = options;
     let {
       actual,
-      expected
+      expected,
     } = options;
 
     const limit = Error.stackTraceLimit;
@@ -429,7 +429,7 @@ class AssertionError extends Error {
       value: 'AssertionError [ERR_ASSERTION]',
       enumerable: false,
       writable: true,
-      configurable: true
+      configurable: true,
     });
     this.code = 'ERR_ASSERTION';
     if (details) {
@@ -478,7 +478,7 @@ class AssertionError extends Error {
     const result = inspect(this, {
       ...ctx,
       customInspect: false,
-      depth: 0
+      depth: 0,
     });
 
     // Reset the properties after inspection.

diff --git a/lib/internal/assert/calltracker.js b/lib/internal/assert/calltracker.js
@@ -63,7 +63,7 @@ class CallTrackerContext {
         actual: this.#calls.length,
         expected: this.#expected,
         operator: this.#name,
-        stack: this.#stackTrace
+        stack: this.#stackTrace,
       };
     }
   }
@@ -110,7 +110,7 @@ class CallTracker {
       expected,
       // eslint-disable-next-line no-restricted-syntax
       stackTrace: new Error(),
-      name: fn.name || 'calls'
+      name: fn.name || 'calls',
     });
     const tracked = new Proxy(fn, {
       __proto__: null,

diff --git a/lib/internal/bootstrap/browser.js b/lib/internal/bootstrap/browser.js
@@ -94,7 +94,7 @@ function exposeNamespace(target, name, namespaceObject) {
     writable: true,
     enumerable: false,
     configurable: true,
-    value: namespaceObject
+    value: namespaceObject,
   });
 }
 

diff --git a/lib/internal/bootstrap/loaders.js b/lib/internal/bootstrap/loaders.js
@@ -68,7 +68,7 @@ ObjectDefineProperty(process, 'moduleLoadList', {
   value: moduleLoadList,
   configurable: true,
   enumerable: true,
-  writable: false
+  writable: false,
 });
 
 
@@ -181,7 +181,7 @@ let internalBinding;
 const loaderId = 'internal/bootstrap/loaders';
 const {
   builtinIds,
-  compileFunction
+  compileFunction,
 } = internalBinding('builtins');
 
 const getOwn = (target, property, receiver) => {
@@ -349,7 +349,7 @@ class BuiltinModule {
 const loaderExports = {
   internalBinding,
   BuiltinModule,
-  require: requireBuiltin
+  require: requireBuiltin,
 };
 
 function requireBuiltin(id) {

diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js
@@ -197,7 +197,7 @@ internalBinding('async_wrap').setupHooks(nativeHooks);
 
 const {
   setupTaskQueue,
-  queueMicrotask
+  queueMicrotask,
 } = require('internal/process/task_queues');
 
 // Non-standard extensions:
@@ -236,11 +236,11 @@ ObjectDefineProperty(process, 'allowedNodeEnvironmentFlags', {
       value,
       configurable: true,
       enumerable: true,
-      writable: true
+      writable: true,
     });
   },
   enumerable: true,
-  configurable: true
+  configurable: true,
 });
 
 // process.assert
@@ -265,22 +265,22 @@ const features = {
   // code cache even if the binary is built with embedded code cache.
   get cached_builtins() {
     return binding.hasCachedBuiltins();
-  }
+  },
 };
 
 ObjectDefineProperty(process, 'features', {
   __proto__: null,
   enumerable: true,
   writable: false,
   configurable: false,
-  value: features
+  value: features,
 });
 
 {
   const {
     onGlobalUncaughtException,
     setUncaughtExceptionCaptureCallback,
-    hasUncaughtExceptionCaptureCallback
+    hasUncaughtExceptionCaptureCallback,
   } = require('internal/process/execution');
 
   // For legacy reasons this is still called `_fatalException`, even
@@ -327,14 +327,14 @@ process.emitWarning = emitWarning;
 function setupPrepareStackTrace() {
   const {
     setEnhanceStackForFatalException,
-    setPrepareStackTraceCallback
+    setPrepareStackTraceCallback,
   } = internalBinding('errors');
   const {
     prepareStackTrace,
     fatalExceptionStackEnhancers: {
       beforeInspector,
-      afterInspector
-    }
+      afterInspector,
+    },
   } = require('internal/errors');
   // Tell our PrepareStackTraceCallback passed to the V8 API
   // to call prepareStackTrace().
@@ -353,7 +353,7 @@ function setupProcessObject() {
     enumerable: false,
     writable: true,
     configurable: false,
-    value: 'process'
+    value: 'process',
   });
 
   // Create global.process as getters so that we have a
@@ -379,7 +379,7 @@ function setupGlobalProxy() {
     value: 'global',
     writable: false,
     enumerable: false,
-    configurable: true
+    configurable: true,
   });
   globalThis.global = globalThis;
 }

diff --git a/lib/internal/bootstrap/switches/does_not_own_process_state.js b/lib/internal/bootstrap/switches/does_not_own_process_state.js
@@ -23,7 +23,7 @@ if (credentials.implementsPosixCredentials) {
 // ----              compare the setups side-by-side                    -----
 
 const {
-  codes: { ERR_WORKER_UNSUPPORTED_OPERATION }
+  codes: { ERR_WORKER_UNSUPPORTED_OPERATION },
 } = require('internal/errors');
 
 function wrappedUmask(mask) {

diff --git a/lib/internal/bootstrap/switches/does_own_process_state.js b/lib/internal/bootstrap/switches/does_own_process_state.js
@@ -25,18 +25,18 @@ if (credentials.implementsPosixCredentials) {
 const {
   parseFileMode,
   validateArray,
-  validateString
+  validateString,
 } = require('internal/validators');
 
 function wrapPosixCredentialSetters(credentials) {
   const {
     codes: {
       ERR_INVALID_ARG_TYPE,
-      ERR_UNKNOWN_CREDENTIAL
-    }
+      ERR_UNKNOWN_CREDENTIAL,
+    },
   } = require('internal/errors');
   const {
-    validateUint32
+    validateUint32,
   } = require('internal/validators');
 
   const {
@@ -45,7 +45,7 @@ function wrapPosixCredentialSetters(credentials) {
     setegid: _setegid,
     seteuid: _seteuid,
     setgid: _setgid,
-    setuid: _setuid
+    setuid: _setuid,
   } = credentials;
 
   function initgroups(user, extraGroup) {
@@ -99,7 +99,7 @@ function wrapPosixCredentialSetters(credentials) {
     setegid: wrapIdSetter('Group', _setegid),
     seteuid: wrapIdSetter('User', _seteuid),
     setgid: wrapIdSetter('Group', _setgid),
-    setuid: wrapIdSetter('User', _setuid)
+    setuid: wrapIdSetter('User', _setuid),
   };
 }
 

diff --git a/lib/internal/bootstrap/switches/is_main_thread.js b/lib/internal/bootstrap/switches/is_main_thread.js
@@ -5,7 +5,7 @@ const rawMethods = internalBinding('process_methods');
 const {
   namespace: {
     addSerializeCallback,
-    isBuildingSnapshot
+    isBuildingSnapshot,
   },
 } = require('internal/v8/startup_snapshot');
 // TODO(joyeecheung): deprecate and remove these underscore methods
@@ -23,7 +23,7 @@ function defineStream(name, getter) {
     __proto__: null,
     configurable: true,
     enumerable: true,
-    get: getter
+    get: getter,
   });
 }
 
@@ -34,7 +34,7 @@ defineStream('stderr', getStderr);
 // Worker threads don't receive signals.
 const {
   startListeningIfSignal,
-  stopListeningIfSignal
+  stopListeningIfSignal,
 } = require('internal/process/signal');
 process.on('newListener', startListeningIfSignal);
 process.on('removeListener', stopListeningIfSignal);
@@ -74,13 +74,13 @@ function createWritableStdioStream(fd) {
         stream = new net.Socket({
           handle: process[kChannelHandle],
           readable: false,
-          writable: true
+          writable: true,
         });
       } else {
         stream = new net.Socket({
           fd,
           readable: false,
-          writable: true
+          writable: true,
         });
       }
 
@@ -95,7 +95,7 @@ function createWritableStdioStream(fd) {
       stream = new Writable({
         write(buf, enc, cb) {
           cb();
-        }
+        },
       });
     }
   }
@@ -218,14 +218,14 @@ function getStdin() {
           handle: process.channel,
           readable: true,
           writable: false,
-          manualStart: true
+          manualStart: true,
         });
       } else {
         stdin = new net.Socket({
           fd: fd,
           readable: true,
           writable: false,
-          manualStart: true
+          manualStart: true,
         });
       }
       // Make sure the stdin can't be `.end()`-ed

diff --git a/lib/internal/bootstrap/switches/is_not_main_thread.js b/lib/internal/bootstrap/switches/is_not_main_thread.js
@@ -10,7 +10,7 @@ function defineStream(name, getter) {
     __proto__: null,
     configurable: true,
     enumerable: true,
-    get: getter
+    get: getter,
   });
 }
 
@@ -21,7 +21,7 @@ defineStream('stderr', getStderr);
 // Worker threads don't receive signals.
 const {
   startListeningIfSignal,
-  stopListeningIfSignal
+  stopListeningIfSignal,
 } = require('internal/process/signal');
 process.removeListener('newListener', startListeningIfSignal);
 process.removeListener('removeListener', stopListeningIfSignal);
@@ -30,7 +30,7 @@ process.removeListener('removeListener', stopListeningIfSignal);
 // ----              compare the setups side-by-side                    -----
 
 const {
-  createWorkerStdio
+  createWorkerStdio,
 } = require('internal/worker/io');
 
 let workerStdio;