eslint-config-gristow 2.0.20 → 3.0.0-alpha.1

package/rules/airbnb-imports.js

@@ -0,0 +1,185 @@
+/**
+ * Airbnb Import Rules
+ * Extracted from eslint-config-airbnb-base for ESLint 9 compatibility
+ * @see https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/imports.js
+ */
+export default {
+  // Ensure imports point to files/modules that can be resolved
+  'import/no-unresolved': ['error', { commonjs: true, caseSensitive: true }],
+
+  // Ensure named imports coupled with named exports
+  'import/named': 'error',
+
+  // Ensure default import coupled with default export
+  'import/default': 'off',
+
+  // Ensure imported namespaces contain dereferenced properties as they are dereferenced
+  'import/namespace': 'off',
+
+  // Report any invalid exports, i.e. re-export of the same name
+  'import/export': 'error',
+
+  // Do not allow a default import name to match a named export
+  'import/no-named-as-default': 'error',
+
+  // Warn on accessing default export property names that are also named exports
+  'import/no-named-as-default-member': 'error',
+
+  // Disallow use of jsdoc-marked-deprecated imports
+  'import/no-deprecated': 'off',
+
+  // Forbid the use of extraneous packages
+  'import/no-extraneous-dependencies': ['error', {
+    devDependencies: [
+      'test/**', // tape, common npm pattern
+      'tests/**', // also common npm pattern
+      'spec/**', // mocha, rspec-like pattern
+      '**/__tests__/**', // jest pattern
+      '**/__mocks__/**', // jest pattern
+      'test.{js,jsx}', // repos with a single test file
+      'test-*.{js,jsx}', // repos with multiple top-level test files
+      '**/*{.,_}{test,spec}.{js,jsx}', // tests where the weights is suffixed
+      '**/jest.config.js', // jest config
+      '**/jest.setup.js', // jest setup
+      '**/vue.config.js', // vue-cli config
+      '**/webpack.config.js', // webpack config
+      '**/webpack.config.*.js', // webpack config
+      '**/rollup.config.js', // rollup config
+      '**/rollup.config.*.js', // rollup config
+      '**/gulpfile.js', // gulp config
+      '**/gulpfile.*.js', // gulp config
+      '**/Gruntfile{,.js}', // grunt config
+      '**/protractor.conf.js', // protractor config
+      '**/protractor.conf.*.js', // protractor config
+      '**/karma.conf.js', // karma config
+      '**/.eslintrc.js', // eslint config
+    ],
+    optionalDependencies: false,
+  }],
+
+  // Forbid mutable exports
+  'import/no-mutable-exports': 'error',
+
+  // Disallow require()
+  'import/no-commonjs': 'off',
+
+  // Disallow AMD require/define
+  'import/no-amd': 'error',
+
+  // Disallow Node.js builtin modules
+  'import/no-nodejs-modules': 'off',
+
+  // Ensure all imports appear before other statements
+  'import/first': 'error',
+
+  // Disallow non-import statements appearing before import statements
+  'import/imports-first': 'off',
+
+  // Disallow duplicate imports
+  'import/no-duplicates': 'error',
+
+  // Disallow namespace imports
+  'import/no-namespace': 'off',
+
+  // Ensure consistent use of file extension within the import path
+  'import/extensions': ['error', 'ignorePackages', {
+    js: 'never',
+    mjs: 'never',
+    jsx: 'never',
+  }],
+
+  // Ensure absolute imports are above relative imports
+  'import/order': ['error', { groups: [['builtin', 'external', 'internal']] }],
+
+  // Require a newline after the last import/require in a group
+  'import/newline-after-import': 'error',
+
+  // Prefer a default export if module exports a single name
+  'import/prefer-default-export': 'error',
+
+  // Restrict which files can be imported in a given folder
+  'import/no-restricted-paths': 'off',
+
+  // Limit the maximum number of dependencies a module can have
+  'import/max-dependencies': ['off', { max: 10 }],
+
+  // Forbid import of modules using absolute paths
+  'import/no-absolute-path': 'error',
+
+  // Forbid require() calls with expressions
+  'import/no-dynamic-require': 'error',
+
+  // Prevent importing the submodules of other modules
+  'import/no-internal-modules': ['off', { allow: [] }],
+
+  // Warn if a module could be mistakenly parsed as a script by a consumer
+  'import/unambiguous': 'off',
+
+  // Forbid Webpack loader syntax in imports
+  'import/no-webpack-loader-syntax': 'error',
+
+  // Prevent unassigned imports
+  'import/no-unassigned-import': 'off',
+
+  // Prevent importing the default as if it were named
+  'import/no-named-default': 'error',
+
+  // Reports if a module's default export is unnamed
+  'import/no-anonymous-default-export': ['off', {
+    allowArray: false,
+    allowArrowFunction: false,
+    allowAnonymousClass: false,
+    allowAnonymousFunction: false,
+    allowLiteral: false,
+    allowObject: false,
+  }],
+
+  // Prefer named exports to be grouped together in a single export declaration
+  'import/exports-last': 'off',
+
+  // Reports when named exports are not grouped together
+  'import/group-exports': 'off',
+
+  // Forbid default exports
+  'import/no-default-export': 'off',
+
+  // Forbid named exports
+  'import/no-named-export': 'off',
+
+  // Forbid a module from importing itself
+  'import/no-self-import': 'error',
+
+  // Forbid cyclical dependencies between modules
+  'import/no-cycle': ['error', { maxDepth: '∞', ignoreExternal: true }],
+
+  // Ensures that there are no useless path segments
+  'import/no-useless-path-segments': ['error', { commonjs: true }],
+
+  // Enforce a leading comment with the webpackChunkName for dynamic imports
+  'import/dynamic-import-chunkname': ['off', {
+    importFunctions: [],
+    webpackChunknameFormat: '[0-9a-zA-Z-_/.]+',
+  }],
+
+  // Forbid importing modules from parent directories
+  'import/no-relative-parent-imports': 'off',
+
+  // Reports modules without any exports, or with unused exports
+  'import/no-unused-modules': ['off', {
+    ignoreExports: [],
+    missingExports: true,
+    unusedExports: true,
+  }],
+
+  // Reports the use of import declarations with CommonJS exports
+  'import/no-import-module-exports': ['error', { exceptions: [] }],
+
+  // Forbid importing packages through relative paths
+  'import/no-relative-packages': 'error',
+
+  // Enforce a consistent style for type specifiers (inline vs top-level)
+  'import/consistent-type-specifier-style': ['off', 'prefer-inline'],
+
+  // Forbid empty named import blocks
+  'import/no-empty-named-blocks': 'off',
+};

package/rules/airbnb-style.js

@@ -0,0 +1,350 @@
+/**
+ * Airbnb Style Rules
+ * Extracted from eslint-config-airbnb-base for ESLint 9 compatibility
+ *
+ * NOTE: Formatting rules are disabled as they are handled by Prettier.
+ * Only semantic/code-quality rules are enabled.
+ *
+ * @see https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js
+ */
+export default {
+  // ============================================================
+  // DISABLED - Handled by Prettier
+  // ============================================================
+
+  // Enforce line breaks after opening and before closing array brackets
+  'array-bracket-newline': 'off',
+
+  // Enforce line breaks between array elements
+  'array-element-newline': 'off',
+
+  // Enforce spacing inside array brackets
+  'array-bracket-spacing': 'off',
+
+  // Enforce spacing inside single-line blocks
+  'block-spacing': 'off',
+
+  // Enforce one true brace style
+  'brace-style': 'off',
+
+  // Require trailing commas in multiline object literals
+  'comma-dangle': 'off',
+
+  // Enforce spacing before and after comma
+  'comma-spacing': 'off',
+
+  // Enforce one true comma style
+  'comma-style': 'off',
+
+  // Disallow padding inside computed properties
+  'computed-property-spacing': 'off',
+
+  // Enforce newline at the end of file, with no multiple empty lines
+  'eol-last': 'off',
+
+  // Enforce line breaks between arguments of a function call
+  'function-call-argument-newline': 'off',
+
+  // Enforce spacing between functions and their invocations
+  'func-call-spacing': 'off',
+
+  // Enforce consistent line breaks inside function parentheses
+  'function-paren-newline': 'off',
+
+  // Enforce the location of arrow function bodies with implicit returns
+  'implicit-arrow-linebreak': 'off',
+
+  // This option sets a specific tab width for your code
+  'indent': 'off',
+
+  // Specify whether double or single quotes should be used in JSX attributes
+  'jsx-quotes': 'off',
+
+  // Enforces spacing between keys and values in object literal properties
+  'key-spacing': 'off',
+
+  // Require a space before & after certain keywords
+  'keyword-spacing': 'off',
+
+  // Disallow mixed 'LF' and 'CRLF' as linebreaks
+  'linebreak-style': 'off',
+
+  // Specify the maximum length of a line in your program
+  'max-len': 'off',
+
+  // Require multiline ternary
+  'multiline-ternary': 'off',
+
+  // Disallow the omission of parentheses when invoking a constructor with no arguments
+  'new-parens': 'off',
+
+  // Enforces new line after each method call in the chain to make it more readable
+  'newline-per-chained-call': 'off',
+
+  // Disallow mixed spaces and tabs for indentation
+  'no-mixed-spaces-and-tabs': 'off',
+
+  // Disallow multiple empty lines, only one newline at the end
+  'no-multiple-empty-lines': 'off',
+
+  // Disallow use of tabs
+  'no-tabs': 'off',
+
+  // Disallow trailing whitespace at the end of lines
+  'no-trailing-spaces': 'off',
+
+  // Disallow whitespace before properties
+  'no-whitespace-before-property': 'off',
+
+  // Enforce the location of single-line statements
+  'nonblock-statement-body-position': 'off',
+
+  // Require padding inside curly braces
+  'object-curly-spacing': 'off',
+
+  // Enforce line breaks between braces
+  'object-curly-newline': 'off',
+
+  // Enforce "same line" or "multiple line" on object properties
+  'object-property-newline': 'off',
+
+  // Require a newline around variable declaration
+  'one-var-declaration-per-line': 'off',
+
+  // Requires operator at the beginning of the line in multiline statements
+  'operator-linebreak': 'off',
+
+  // Disallow padding within blocks
+  'padded-blocks': 'off',
+
+  // Require quotes around object literal property names
+  'quote-props': 'off',
+
+  // Specify whether double or single quotes should be used
+  'quotes': 'off',
+
+  // Require or disallow use of semicolons instead of ASI
+  'semi': 'off',
+
+  // Enforce spacing before and after semicolons
+  'semi-spacing': 'off',
+
+  // Enforce location of semicolons
+  'semi-style': 'off',
+
+  // Require or disallow space before blocks
+  'space-before-blocks': 'off',
+
+  // Require or disallow space before function opening parenthesis
+  'space-before-function-paren': 'off',
+
+  // Require or disallow spaces inside parentheses
+  'space-in-parens': 'off',
+
+  // Require spaces around operators
+  'space-infix-ops': 'off',
+
+  // Require or disallow spaces before/after unary operators
+  'space-unary-ops': 'off',
+
+  // Enforce spacing around colons of switch statements
+  'switch-colon-spacing': 'off',
+
+  // Require or disallow spacing between template tags and their literals
+  'template-tag-spacing': 'off',
+
+  // Require regex literals to be wrapped in parentheses
+  'wrap-regex': 'off',
+
+  // ============================================================
+  // ENABLED - Semantic/Code Quality Rules
+  // ============================================================
+
+  // Require camel case names
+  'camelcase': ['error', { properties: 'never', ignoreDestructuring: false }],
+
+  // Enforce or disallow capitalization of the first letter of a comment
+  'capitalized-comments': 'off',
+
+  // Enforces consistent naming when capturing the current execution context
+  'consistent-this': 'off',
+
+  // Requires function names to match the name of the variable or property
+  'func-name-matching': 'off',
+
+  // Require function expressions to have a name
+  'func-names': 'warn',
+
+  // Enforces use of function declarations or expressions
+  'func-style': 'off',
+
+  // Disallow specified identifiers
+  'id-denylist': 'off',
+
+  // This option enforces minimum and maximum identifier lengths
+  'id-length': 'off',
+
+  // Require identifiers to match the provided regular expression
+  'id-match': 'off',
+
+  // Enforce position of line comments
+  'line-comment-position': 'off',
+
+  // Require or disallow an empty line between class members
+  'lines-between-class-members': ['error', 'always', { exceptAfterSingleLine: false }],
+
+  // Enforces empty lines around comments
+  'lines-around-comment': 'off',
+
+  // Require or disallow logical assignment logical operator shorthand
+  'logical-assignment-operators': 'off',
+
+  // Specify the maximum depth that blocks can be nested
+  'max-depth': 'off',
+
+  // Specify the max number of lines in a file
+  'max-lines': 'off',
+
+  // Enforce a maximum function length
+  'max-lines-per-function': 'off',
+
+  // Specify the maximum depth callbacks can be nested
+  'max-nested-callbacks': 'off',
+
+  // Limits the number of parameters that can be used in the function declaration
+  'max-params': 'off',
+
+  // Specify the maximum number of statement allowed in a function
+  'max-statements': 'off',
+
+  // Restrict the number of statements per line
+  'max-statements-per-line': 'off',
+
+  // Enforce a particular style for multiline comments
+  'multiline-comment-style': 'off',
+
+  // Require a capital letter for constructors
+  'new-cap': ['error', {
+    newIsCap: true,
+    newIsCapExceptions: [],
+    capIsNew: false,
+    capIsNewExceptions: ['Immutable.Map', 'Immutable.Set', 'Immutable.List'],
+  }],
+
+  // Disallow use of the Array constructor
+  'no-array-constructor': 'error',
+
+  // Disallow use of bitwise operators
+  'no-bitwise': 'error',
+
+  // Disallow use of the continue statement
+  'no-continue': 'error',
+
+  // Disallow comments inline after code
+  'no-inline-comments': 'off',
+
+  // Disallow if as the only statement in an else block
+  'no-lonely-if': 'error',
+
+  // Disallow un-paren'd mixes of different operators
+  'no-mixed-operators': ['error', {
+    groups: [
+      ['%', '**'],
+      ['%', '+'],
+      ['%', '-'],
+      ['%', '*'],
+      ['%', '/'],
+      ['/', '*'],
+      ['&', '|', '<<', '>>', '>>>'],
+      ['==', '!=', '===', '!=='],
+      ['&&', '||'],
+    ],
+    allowSamePrecedence: false,
+  }],
+
+  // Disallow use of chained assignment expressions
+  'no-multi-assign': ['error'],
+
+  // Disallow negated conditions
+  'no-negated-condition': 'off',
+
+  // Disallow nested ternary expressions
+  'no-nested-ternary': 'error',
+
+  // Disallow use of the Object constructor
+  'no-new-object': 'error',
+
+  // Disallow use of unary operators, ++ and --
+  'no-plusplus': 'error',
+
+  // Disallow certain syntax forms
+  'no-restricted-syntax': ['error',
+    {
+      selector: 'ForInStatement',
+      message: 'for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array.',
+    },
+    {
+      selector: 'ForOfStatement',
+      message: 'iterators/generators require regenerator-runtime, which is too heavyweight for this guide to allow them. Separately, loops should be avoided in favor of array iterations.',
+    },
+    {
+      selector: 'LabeledStatement',
+      message: 'Labels are a form of GOTO; using them makes code confusing and hard to maintain and understand.',
+    },
+    {
+      selector: 'WithStatement',
+      message: '`with` is disallowed in strict mode because it makes code impossible to predict and optimize.',
+    },
+  ],
+
+  // Disallow the use of ternary operators
+  'no-ternary': 'off',
+
+  // Disallow dangling underscores in identifiers
+  'no-underscore-dangle': ['error', {
+    allow: [],
+    allowAfterThis: false,
+    allowAfterSuper: false,
+    enforceInMethodNames: true,
+  }],
+
+  // Disallow the use of Boolean literals in conditional expressions
+  'no-unneeded-ternary': ['error', { defaultAssignment: false }],
+
+  // Allow just one var statement per function
+  'one-var': ['error', 'never'],
+
+  // Require assignment operator shorthand where possible or prohibit it entirely
+  'operator-assignment': ['error', 'always'],
+
+  // Require or disallow padding lines between statements
+  'padding-line-between-statements': 'off',
+
+  // Prefer use of an exponentiation operator over Math.pow
+  'prefer-exponentiation-operator': 'error',
+
+  // Prefer use of an object spread over Object.assign
+  'prefer-object-spread': 'error',
+
+  // Requires object keys to be sorted
+  'sort-keys': 'off',
+
+  // Sort variables within the same declaration block
+  'sort-vars': 'off',
+
+  // Require or disallow a space immediately following the // or /* in a comment
+  'spaced-comment': ['error', 'always', {
+    line: {
+      exceptions: ['-', '+'],
+      markers: ['=', '!', '/'],
+    },
+    block: {
+      exceptions: ['-', '+'],
+      markers: ['=', '!', ':', '::'],
+      balanced: true,
+    },
+  }],
+
+  // Require or disallow the Unicode Byte Order Mark
+  'unicode-bom': ['error', 'never'],
+};

package/rules/airbnb-variables.js

@@ -0,0 +1,118 @@
+/**
+ * Airbnb Variables Rules
+ * Extracted from eslint-config-airbnb-base for ESLint 9 compatibility
+ * @see https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/variables.js
+ */
+
+// List of confusing browser globals that should be explicitly referenced from window
+const restrictedGlobals = [
+  'addEventListener',
+  'blur',
+  'close',
+  'closed',
+  'confirm',
+  'defaultStatus',
+  'defaultstatus',
+  'event',
+  'external',
+  'find',
+  'focus',
+  'frameElement',
+  'frames',
+  'history',
+  'innerHeight',
+  'innerWidth',
+  'length',
+  'location',
+  'locationbar',
+  'menubar',
+  'moveBy',
+  'moveTo',
+  'name',
+  'onblur',
+  'onerror',
+  'onfocus',
+  'onload',
+  'onresize',
+  'onunload',
+  'open',
+  'opener',
+  'opera',
+  'outerHeight',
+  'outerWidth',
+  'pageXOffset',
+  'pageYOffset',
+  'parent',
+  'print',
+  'removeEventListener',
+  'resizeBy',
+  'resizeTo',
+  'screen',
+  'screenLeft',
+  'screenTop',
+  'screenX',
+  'screenY',
+  'scroll',
+  'scrollbars',
+  'scrollBy',
+  'scrollTo',
+  'scrollX',
+  'scrollY',
+  'self',
+  'status',
+  'statusbar',
+  'stop',
+  'toolbar',
+  'top',
+];
+
+export default {
+  // Enforce or disallow variable initializations at definition
+  'init-declarations': 'off',
+
+  // Disallow the catch clause parameter name being the same as a variable in the outer scope
+  'no-catch-shadow': 'off',
+
+  // Disallow deletion of variables
+  'no-delete-var': 'error',
+
+  // Disallow labels that share a name with a variable
+  'no-label-var': 'error',
+
+  // Disallow specific globals
+  'no-restricted-globals': ['error',
+    {
+      name: 'isFinite',
+      message: 'Use Number.isFinite instead https://github.com/airbnb/javascript#standard-library--isfinite',
+    },
+    {
+      name: 'isNaN',
+      message: 'Use Number.isNaN instead https://github.com/airbnb/javascript#standard-library--isnan',
+    },
+    ...restrictedGlobals.map((g) => ({
+      name: g,
+      message: `Use window.${g} instead. https://github.com/facebook/create-react-app/blob/HEAD/packages/confusing-browser-globals/README.md`,
+    })),
+  ],
+
+  // Disallow declaration of variables already declared in the outer scope
+  'no-shadow': 'error',
+
+  // Disallow shadowing of names such as arguments
+  'no-shadow-restricted-names': 'error',
+
+  // Disallow use of undeclared variables unless mentioned in a /*global */ block
+  'no-undef': 'error',
+
+  // Disallow use of undefined when initializing variables
+  'no-undef-init': 'error',
+
+  // Disallow use of undefined variable
+  'no-undefined': 'off',
+
+  // Disallow declaration of variables that are not used in the code
+  'no-unused-vars': ['error', { vars: 'all', args: 'after-used', ignoreRestSiblings: true }],
+
+  // Disallow use of variables before they are defined
+  'no-use-before-define': ['error', { functions: true, classes: true, variables: true }],
+};

package/rules/import-rules.js

@@ -1,4 +1,7 @@
-module.exports = {
+/**
+ * Import plugin rules and overrides
+ */
+export default {
   'no-duplicate-imports': 'off',
   'import/no-duplicates': ['error', { 'prefer-inline': false }],
   'import/no-deprecated': 'warn',

package/rules/{naming-convention.cjs → naming-convention.js}

@@ -1,4 +1,7 @@
-module.exports = {
+/**
+ * TypeScript Naming Convention Rules
+ */
+export default {
   '@typescript-eslint/naming-convention': [
     'error',
     // By default, all should be camelCase

package/rules/{shared-rules.cjs → shared-rules.js}

@@ -1,9 +1,9 @@
-const namingConvention = require('./naming-convention.cjs');
+import namingConvention from './naming-convention.js';
 
 /**
  * These rules are shared by both .js and .ts
  */
-module.exports = {
+export default {
   // We disable eslint's no-unused vars, and enable typescript's because
   // otherwise local vars listed in function overloads are flagged.
   'no-unused-vars': 'off',
@@ -104,4 +104,6 @@ module.exports = {
   camelcase: 'off',
   ...namingConvention,
   '@typescript-eslint/no-unnecessary-condition': ['warn'],
+  // Allow default exports (Airbnb restricts this, but it's a valid pattern)
+  'no-restricted-exports': 'off',
 };