Set source type to commonjs to enable rule lint rules again (#2240)

Author: FloEdelmann

eslint.config.js

@@ -49,12 +49,16 @@ module.exports = [
     files: ['**/*.js'],
     languageOptions: {
       ecmaVersion: 'latest',
+      sourceType: 'commonjs',
       globals: {
         ...globals.es6,
         ...globals.node,
         ...globals.mocha
       }
     },
+    linterOptions: {
+      reportUnusedDisableDirectives: true
+    },
     rules: {
       'accessor-pairs': 2,
       camelcase: [2, { properties: 'never' }],

lib/rules/array-bracket-newline.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('array-bracket-newline', {
   skipDynamicArguments: true
 })

lib/rules/array-bracket-spacing.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('array-bracket-spacing', {
   skipDynamicArguments: true
 })

lib/rules/array-element-newline.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('array-element-newline', {
   skipDynamicArguments: true
 })

lib/rules/arrow-spacing.js

@@ -5,5 +5,5 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('arrow-spacing')

lib/rules/block-spacing.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('block-spacing', {
   skipDynamicArguments: true
 })

lib/rules/brace-style.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('brace-style', {
   skipDynamicArguments: true
 })

lib/rules/camelcase.js

@@ -5,5 +5,5 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('camelcase')

lib/rules/comma-dangle.js

@@ -5,5 +5,5 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('comma-dangle')

lib/rules/comma-spacing.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('comma-spacing', {
   skipDynamicArguments: true,
   skipDynamicArgumentsReport: true,

lib/rules/comma-style.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('comma-style', {
   create(_context, { coreHandlers }) {
     return {

lib/rules/dot-location.js

@@ -5,5 +5,5 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('dot-location')

lib/rules/dot-notation.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('dot-notation', {
   applyDocument: true
 })

lib/rules/eqeqeq.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('eqeqeq', {
   applyDocument: true
 })

lib/rules/func-call-spacing.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('func-call-spacing', {
   skipDynamicArguments: true,
   applyDocument: true

lib/rules/key-spacing.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('key-spacing', {
   skipDynamicArguments: true
 })

lib/rules/keyword-spacing.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('keyword-spacing', {
   skipDynamicArguments: true
 })

lib/rules/multiline-ternary.js

@@ -6,7 +6,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('multiline-ternary', {
   skipDynamicArguments: true,
   applyDocument: true

lib/rules/no-console.js

@@ -6,7 +6,7 @@
 
 const utils = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = utils.wrapCoreRule('no-console', {
   skipCoreHandlers: true,
   create(context) {

lib/rules/no-constant-condition.js

@@ -7,7 +7,7 @@ const { wrapCoreRule } = require('../utils')
 
 const conditionalDirectiveNames = new Set(['v-show', 'v-if', 'v-else-if'])
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('no-constant-condition', {
   create(_context, { coreHandlers }) {
     return {

lib/rules/no-empty-pattern.js

@@ -5,5 +5,5 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('no-empty-pattern')

lib/rules/no-extra-parens.js

@@ -7,7 +7,7 @@ const { isParenthesized } = require('@eslint-community/eslint-utils')
 const { wrapCoreRule } = require('../utils')
 const { getStyleVariablesContext } = require('../utils/style-variables')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('no-extra-parens', {
   skipDynamicArguments: true,
   applyDocument: true,

lib/rules/no-loss-of-precision.js

@@ -6,7 +6,7 @@
 
 const utils = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = utils.wrapCoreRule('no-loss-of-precision', {
   applyDocument: true
 })

lib/rules/no-restricted-syntax.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('no-restricted-syntax', {
   applyDocument: true
 })

lib/rules/no-sparse-arrays.js

@@ -5,5 +5,5 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('no-sparse-arrays')

lib/rules/no-useless-concat.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('no-useless-concat', {
   applyDocument: true
 })

lib/rules/object-curly-newline.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('object-curly-newline', {
   skipDynamicArguments: true
 })

lib/rules/object-curly-spacing.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('object-curly-spacing', {
   skipDynamicArguments: true
 })

lib/rules/object-property-newline.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('object-property-newline', {
   skipDynamicArguments: true
 })

lib/rules/object-shorthand.js

@@ -6,7 +6,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('object-shorthand', {
   skipDynamicArguments: true
 })

lib/rules/operator-linebreak.js

@@ -5,5 +5,5 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('operator-linebreak')

lib/rules/prefer-template.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('prefer-template', {
   applyDocument: true
 })

lib/rules/quote-props.js

@@ -6,7 +6,7 @@
 
 const { wrapCoreRule, flatten } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('quote-props', {
   skipDynamicArguments: true,
   preprocess(context, { wrapContextToOverrideProperties, defineVisitor }) {

lib/rules/space-in-parens.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('space-in-parens', {
   skipDynamicArguments: true,
   skipDynamicArgumentsReport: true,

lib/rules/space-infix-ops.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('space-infix-ops', {
   skipDynamicArguments: true,
   applyDocument: true

lib/rules/space-unary-ops.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('space-unary-ops', {
   skipDynamicArguments: true,
   applyDocument: true

lib/rules/template-curly-spacing.js

@@ -5,7 +5,7 @@
 
 const { wrapCoreRule } = require('../utils')
 
-// eslint-disable-next-line internal/no-invalid-meta, internal/no-invalid-meta-docs-categories
+// eslint-disable-next-line internal/no-invalid-meta
 module.exports = wrapCoreRule('template-curly-spacing', {
   skipDynamicArguments: true,
   applyDocument: true

lib/utils/indent-ts.js

@@ -229,7 +229,6 @@ function defineVisitor({
     /**
      * @param {ASTNode} node
      */
-    // eslint-disable-next-line complexity -- ignore
     '*[type=/^TS/]'(node) {
       if (!isTypeNode(node)) {
         return