Add defaultOptions to all rules

Author: FloEdelmann

lib/rules/consistent-output.js

@@ -32,6 +32,7 @@ module.exports = {
         default: 'consistent',
       },
     ],
+    defaultOptions: ['consistent'],
     messages: {
       missingOutput: 'This test case should have an output assertion.',
     },

lib/rules/meta-property-ordering.js

@@ -6,6 +6,18 @@
 
 const { getKeyName, getRuleInfo } = require('../utils');
 
+const defaultOrder = [
+  'type',
+  'docs',
+  'fixable',
+  'hasSuggestions',
+  'deprecated',
+  'replacedBy',
+  'schema',
+  'defaultOptions', // https://github.com/eslint/rfcs/tree/main/designs/2023-rule-options-defaults
+  'messages',
+];
+
 // ------------------------------------------------------------------------------
 // Rule Definition
 // ------------------------------------------------------------------------------
@@ -28,6 +40,7 @@ module.exports = {
         elements: { type: 'string' },
       },
     ],
+    defaultOptions: [defaultOrder],
     messages: {
       inconsistentOrder:
         'The meta properties should be placed in a consistent order: [{{order}}].',
@@ -41,17 +54,7 @@ module.exports = {
       return {};
     }
 
-    const order = context.options[0] || [
-      'type',
-      'docs',
-      'fixable',
-      'hasSuggestions',
-      'deprecated',
-      'replacedBy',
-      'schema',
-      'defaultOptions', // https://github.com/eslint/rfcs/tree/main/designs/2023-rule-options-defaults
-      'messages',
-    ];
+    const order = context.options[0] || defaultOrder;
 
     const orderMap = new Map(order.map((name, i) => [name, i]));
 

lib/rules/no-property-in-node.js

@@ -70,6 +70,7 @@ module.exports = {
         additionalProperties: false,
       },
     ],
+    defaultOptions: [{ additionalNodeTypeFiles: [] }],
     messages: {
       in: 'Prefer checking specific node properties instead of a broad `in`.',
     },

lib/rules/report-message-format.js

@@ -29,6 +29,7 @@ module.exports = {
         type: 'string',
       },
     ],
+    defaultOptions: [''],
     messages: {
       noMatch: "Report message does not match the pattern '{{pattern}}'.",
     },

lib/rules/require-meta-docs-description.js

@@ -35,6 +35,7 @@ module.exports = {
         additionalProperties: false,
       },
     ],
+    defaultOptions: [{ pattern: '^(enforce|require|disallow)' }],
     messages: {
       extraWhitespace:
         '`meta.docs.description` must not have leading nor trailing whitespace.',

lib/rules/require-meta-docs-recommended.js

@@ -28,6 +28,7 @@ module.exports = {
         additionalProperties: false,
       },
     ],
+    defaultOptions: [{ allowNonBoolean: false }],
     messages: {
       incorrect: '`meta.docs.recommended` is required to be a boolean.',
       missing: '`meta.docs.recommended` is required.',

lib/rules/require-meta-docs-url.js

@@ -40,6 +40,7 @@ module.exports = {
         additionalProperties: false,
       },
     ],
+    defaultOptions: [{}],
     messages: {
       mismatch: '`meta.docs.url` property must be `{{expectedUrl}}`.',
       missing: '`meta.docs.url` property is missing.',

lib/rules/require-meta-fixable.js

@@ -36,6 +36,7 @@ module.exports = {
         additionalProperties: false,
       },
     ],
+    defaultOptions: [{ catchNoFixerButFixableProperty: false }],
     messages: {
       invalid: '`meta.fixable` must be either `code`, `whitespace`, or `null`.',
       missing:

lib/rules/require-meta-schema.js

@@ -31,6 +31,7 @@ module.exports = {
         additionalProperties: false,
       },
     ],
+    defaultOptions: [{ requireSchemaPropertyWhenOptionless: true }],
     messages: {
       addEmptySchema: 'Add empty schema indicating the rule has no options.',
       foundOptionsUsage:

lib/rules/test-case-property-ordering.js

@@ -7,6 +7,19 @@
 
 const utils = require('../utils');
 
+const defaultOrder = [
+  'filename',
+  'code',
+  'output',
+  'options',
+  'parser',
+  'languageOptions', // flat-mode only
+  'parserOptions', // eslintrc-mode only
+  'globals', // eslintrc-mode only
+  'env', // eslintrc-mode only
+  'errors',
+];
+
 // ------------------------------------------------------------------------------
 // Rule Definition
 // ------------------------------------------------------------------------------
@@ -30,6 +43,7 @@ module.exports = {
         elements: { type: 'string' },
       },
     ],
+    defaultOptions: [defaultOrder],
     messages: {
       inconsistentOrder:
         'The properties of a test case should be placed in a consistent order: [{{order}}].',
@@ -40,18 +54,7 @@ module.exports = {
     // ----------------------------------------------------------------------
     // Public
     // ----------------------------------------------------------------------
-    const order = context.options[0] || [
-      'filename',
-      'code',
-      'output',
-      'options',
-      'parser',
-      'languageOptions', // flat-mode only
-      'parserOptions', // eslintrc-mode only
-      'globals', // eslintrc-mode only
-      'env', // eslintrc-mode only
-      'errors',
-    ];
+    const order = context.options[0] || defaultOrder;
     const sourceCode = context.sourceCode || context.getSourceCode(); // TODO: just use context.sourceCode when dropping eslint < v9
 
     return {