lib: enforce use of trailing commas in more files

lib/.eslintrc.yaml

@@ -261,3 +261,52 @@ globals:
  module: false
  internalBinding: false
  primordials: false
+overrides:
+ - files:
+ - ./*/promises.js
+ - ./_stream_*.js
+ - ./_tls_common.js
+ - ./assert/*.js
+ - ./cluster.js
+ - ./console.js
+ - ./constants.js
+ - ./internal/assert.js
+ - ./internal/child_process/*.js
+ - ./internal/cli_table.js
+ - ./internal/debugger/inspect.js
+ - ./internal/events/*.js
+ - ./internal/fixed_queue.js
+ - ./internal/freelist.js
+ - ./internal/heap_utils.js
+ - ./internal/http.js
+ - ./internal/idna.js
+ - ./internal/js_stream_socket.js
+ - ./internal/mime.js
+ - ./internal/modules/*.js
+ - ./internal/per_context/messageport.js
+ - ./internal/policy/*.js
+ - ./internal/priority_queue.js
+ - ./internal/readline/*.js
+ - ./internal/readme.md
+ - ./internal/repl/history.js
+ - ./internal/source_map/prepare_stack_trace.js
+ - ./internal/structured_clone.js
+ - ./internal/test/*.js
+ - ./internal/test_runner/**/*.js
+ - ./internal/tls/secure-context.js
+ - ./internal/util/parse_args/*.js
+ - ./internal/v8_prof_processor.js
+ - ./internal/vm.js
+ - ./internal/watch_mode/*.js
+ - ./internal/webidl.js
+ - ./module.js
+ - ./path/*.js
+ - ./process.js
+ - ./punycode.js
+ - ./stream/*.js
+ - ./sys.js
+ - ./test.js
+ - ./url.js
+ - ./util/*.js
+ rules:
+ comma-dangle: [error, always-multiline]

lib/internal/modules/helpers.js

@@ -15,7 +15,7 @@ const {
 } = primordials;
 const {
  ERR_MANIFEST_DEPENDENCY_MISSING,
- ERR_UNKNOWN_BUILTIN_MODULE
+ ERR_UNKNOWN_BUILTIN_MODULE,
 } = require('internal/errors').codes;
 const { BuiltinModule } = require('internal/bootstrap/loaders');
 
@@ -203,7 +203,7 @@ function addBuiltinLibsToObject(object, dummyModuleName) {
  },
  set: setReal,
  configurable: true,
- enumerable: false
+ enumerable: false,
  });
  });
 }

lib/internal/policy/sri.js

@@ -14,7 +14,7 @@ const {
 } = primordials;
 
 const {
- ERR_SRI_PARSE
+ ERR_SRI_PARSE,
 } = require('internal/errors').codes;
 const kWSP = '[\\x20\\x09]';
 const kVCHAR = '[\\x21-\\x7E]';
@@ -55,7 +55,7 @@ const parse = (str) => {
  algorithm: match[2],
  value: BufferFrom(match[3], 'base64'),
  options: match[4] === undefined ? null : match[4],
- })
+ }),
  });
  prevIndex += match[0].length;
  }

lib/internal/readline/utils.js

@@ -93,7 +93,7 @@ function* emitKeys(stream) {
  name: undefined,
  ctrl: false,
  meta: false,
- shift: false
+ shift: false,
  };
 
  if (ch === kEscape) {
@@ -388,5 +388,5 @@ module.exports = {
  commonPrefix,
  emitKeys,
  kSubstringSearch,
- CSI
+ CSI,
 };

lib/internal/test/transfer.js

@@ -30,7 +30,7 @@ class F extends E {
  [kClone]() {
  return {
  data: { b: this.b },
- deserializeInfo: 'internal/test/transfer:F'
+ deserializeInfo: 'internal/test/transfer:F',
  };
  }
 

lib/internal/test_runner/harness.js

@@ -113,7 +113,7 @@ function setup(root) {
  },
  destroy(asyncId) {
  testResources.delete(asyncId);
- }
+ },
  });
 
  hook.enable();

lib/internal/test_runner/yaml_to_js.js

@@ -2,7 +2,7 @@
 const {
  codes: {
  ERR_TEST_FAILURE,
- }
+ },
 } = require('internal/errors');
 const AssertionError = require('internal/assert/assertion_error');
 const {
@@ -37,7 +37,7 @@ function reConstructError(parsedYaml) {
  message: parsedYaml.error,
  actual: parsedYaml.actual,
  expected: parsedYaml.expected,
- operator: parsedYaml.operator
+ operator: parsedYaml.operator,
  });
  } else {
  // eslint-disable-next-line no-restricted-syntax

lib/timers/promises.js

@@ -12,7 +12,7 @@ const {
 const {
  Timeout,
  Immediate,
- insert
+ insert,
 } = require('internal/timers');
 const {
  clearImmediate,
@@ -26,7 +26,7 @@ const {
  ERR_ILLEGAL_CONSTRUCTOR,
  ERR_INVALID_ARG_TYPE,
  ERR_INVALID_THIS,
- }
+ },
 } = require('internal/errors');
 
 const {