@zendeskgarden/eslint-config 43.0.0 → 44.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zendeskgarden/eslint-config",
-  "version": "43.0.0",
+  "version": "44.0.0",
   "description": "Garden ESLint config",
   "license": "Apache-2.0",
   "author": "Zendesk Garden <garden@zendesk.com>",
@@ -20,37 +20,35 @@
     "lint": "eslint eslint.config.js index.js plugins/*.js rules/*.js --max-warnings 0",
     "prepare": "husky",
     "start": "npm exec @eslint/config-inspector",
-    "tag": "[ `git rev-parse --abbrev-ref HEAD` = 'main' ] && standard-version --no-verify",
+    "tag": "[ `git rev-parse --abbrev-ref HEAD` = 'main' ] && commit-and-tag-version --no-verify --npmPublishHint 'true'",
     "test": "npm run format && npm run lint && git diff --quiet"
   },
   "dependencies": {
-    "@babel/eslint-parser": "7.24.7",
-    "@eslint/compat": "1.1.0",
-    "eslint-plugin-deprecation": "3.0.0",
-    "eslint-plugin-jest": "28.6.0",
-    "eslint-plugin-jsx-a11y": "6.9.0",
-    "eslint-plugin-n": "17.9.0",
+    "@babel/eslint-parser": "7.25.1",
+    "eslint-plugin-jest": "28.8.3",
+    "eslint-plugin-jsx-a11y": "6.10.0",
+    "eslint-plugin-n": "17.10.2",
     "eslint-plugin-notice": "1.0.0",
-    "eslint-plugin-react": "7.34.3",
+    "eslint-plugin-react": "7.36.0",
     "eslint-plugin-react-hooks": "4.6.2",
-    "globals": "15.6.0",
-    "typescript-eslint": "7.13.1"
+    "globals": "15.9.0",
+    "typescript-eslint": "8.5.0"
   },
   "peerDependencies": {
     "eslint": "^9.0.0"
   },
   "devDependencies": {
-    "@babel/core": "7.24.7",
-    "@eslint/config-inspector": "0.5.0",
-    "@zendeskgarden/scripts": "2.3.0",
+    "@babel/core": "7.25.2",
+    "@eslint/config-inspector": "0.5.4",
+    "@zendeskgarden/scripts": "2.4.2",
+    "commit-and-tag-version": "12.4.3",
     "envalid": "8.0.0",
-    "eslint": "9.5.0",
-    "husky": "9.0.11",
+    "eslint": "9.10.0",
+    "husky": "9.1.6",
     "jest": "29.7.0",
     "prettier-package-json": "2.8.0",
     "react": "18.3.1",
-    "standard-version": "9.5.0",
-    "typescript": "5.4.5"
+    "typescript": "5.6.2"
   },
   "keywords": [
     "eslint",

package/plugins/react.js

@@ -5,7 +5,6 @@
  * found at http://www.apache.org/licenses/LICENSE-2.0.
  */
 
-import { fixupPluginRules } from '@eslint/compat';
 import jsxA11yPlugin from 'eslint-plugin-jsx-a11y';
 import reactHooksPlugin from 'eslint-plugin-react-hooks';
 import reactPlugin from 'eslint-plugin-react';
@@ -13,8 +12,7 @@ import reactPlugin from 'eslint-plugin-react';
 export default {
   plugins: {
     react: reactPlugin,
-    // TODO remove fixup when eslint-plugin-react-hooks 5.x is released
-    'react-hooks': fixupPluginRules(reactHooksPlugin),
+    'react-hooks': reactHooksPlugin,
     'jsx-a11y': jsxA11yPlugin
   },
   languageOptions: {
@@ -58,6 +56,8 @@ export default {
     'react/forbid-foreign-prop-types': 0,
     // forbid certain propTypes
     'react/forbid-prop-types': 0,
+    // require all forwardRef components include a ref parameter
+    'react/forward-ref-uses-ref': 2,
     // standardize the way function components get defined
     'react/function-component-definition': [
       1,

package/plugins/typescript-type-checked.js

@@ -5,11 +5,9 @@
  * found at http://www.apache.org/licenses/LICENSE-2.0.
  */
 
-import deprecationPlugin from 'eslint-plugin-deprecation';
 import eslintLayoutFormatting from '../rules/layout-formatting.js';
 import eslintPossibleProblems from '../rules/possible-problems.js';
 import eslintSuggestions from '../rules/suggestions.js';
-import { fixupPluginRules } from '@eslint/compat';
 import tseslint from 'typescript-eslint';
 
 const eslintRules = {
@@ -20,8 +18,7 @@ const eslintRules = {
 
 export default {
   plugins: {
-    '@typescript-eslint': tseslint.plugin,
-    deprecation: fixupPluginRules(deprecationPlugin)
+    '@typescript-eslint': tseslint.plugin
   },
   languageOptions: {
     parser: tseslint.parser,
@@ -71,6 +68,8 @@ export default {
     '@typescript-eslint/no-base-to-string': 2,
     // requires expressions of type void to appear in statement position
     '@typescript-eslint/no-confusing-void-expression': 2,
+    // disallow using code marked as `@deprecated`
+    '@typescript-eslint/no-deprecated': 1,
     // disallows duplicate union or intersection type members
     '@typescript-eslint/no-duplicate-type-constituents': 2,
     // requires Promise-like values to be handled appropriately
@@ -100,6 +99,8 @@ export default {
     '@typescript-eslint/no-unnecessary-type-arguments': 2,
     // warns if a type assertion does not change the type of an expression
     '@typescript-eslint/no-unnecessary-type-assertion': 2,
+    // disallow type parameters that aren't used multiple times
+    '@typescript-eslint/no-unnecessary-type-parameters': 1,
     // disallows calling an function with an any type value
     '@typescript-eslint/no-unsafe-argument': 2,
     // disallows assigning any to variables and properties
@@ -163,8 +164,6 @@ export default {
     // enforces unbound methods are called with their expected scope
     '@typescript-eslint/unbound-method': 2,
     // requires type annotation of catch() parameter remain unknown
-    '@typescript-eslint/use-unknown-in-catch-callback-variable': 2,
-    // disallow usage of deprecated APIs
-    'deprecation/deprecation': 'warn'
+    '@typescript-eslint/use-unknown-in-catch-callback-variable': 2
   }
 };

package/plugins/typescript.js

@@ -23,7 +23,7 @@ export default {
   languageOptions: {
     parser: tseslint.parser,
     parserOptions: {
-      project: true
+      projectService: true
     }
   },
   rules: {
@@ -44,7 +44,6 @@ export default {
     'no-import-assign': 0,
     'no-invalid-this': 0,
     'no-loop-func': 0,
-    'no-loss-of-precision': 0,
     'no-magic-numbers': 0,
     'no-obj-calls': 0,
     'no-redeclare': 0,
@@ -69,8 +68,6 @@ export default {
     '@typescript-eslint/ban-ts-comment': 2,
     // bans `// tslint:<rule-flag>` comments from being used
     '@typescript-eslint/ban-tslint-comment': 1,
-    // bans specific types from being used
-    '@typescript-eslint/ban-types': 2,
     // ensures that literals on classes are exposed in a consistent style
     '@typescript-eslint/class-literal-property-style': 2,
     // enforce that class methods utilize `this`
@@ -119,8 +116,8 @@ export default {
     '@typescript-eslint/no-dynamic-delete': 2,
     // disallow empty functions
     '@typescript-eslint/no-empty-function': eslintRules['no-empty-function'],
-    // disallow the declaration of empty interfaces
-    '@typescript-eslint/no-empty-interface': 2,
+    // disallow accidentally using the "empty object" type
+    '@typescript-eslint/no-empty-object-type': 2,
     // disallow usage of the `any` type
     '@typescript-eslint/no-explicit-any': 2,
     // disallow extra non-null assertion
@@ -137,9 +134,6 @@ export default {
     '@typescript-eslint/no-invalid-void-type': 2,
     // disallow function declarations that contain unsafe references inside loop statements
     '@typescript-eslint/no-loop-func': eslintRules['no-loop-func'],
-    // disallow literal numbers that lose precision
-    '@typescript-eslint/no-loss-of-precision':
-      eslintRules['no-loss-of-precision'],
     // disallow magic numbers
     '@typescript-eslint/no-magic-numbers': eslintRules['no-magic-numbers'],
     // enforce valid definition of `new` and `constructor`
@@ -159,14 +153,20 @@ export default {
     // restrict usage of specified node imports
     '@typescript-eslint/no-restricted-imports':
       eslintRules['no-restricted-imports'],
+    // disallow certain types
+    '@typescript-eslint/no-restricted-types': 2,
     // disallow variable declarations from shadowing variables declared in the outer scope
     '@typescript-eslint/no-shadow': eslintRules['no-shadow'],
     // disallow aliasing `this`
     '@typescript-eslint/no-this-alias': 2,
+    // disallow unnecessary assignment of constructor property parameter
+    '@typescript-eslint/no-unnecessary-parameter-property-assignment': 2,
     // disallows unnecessary constraints on generic type
     '@typescript-eslint/no-unnecessary-type-constraint': 2,
     // disallows erging between classes and interfaces
     '@typescript-eslint/no-unsafe-declaration-merging': 2,
+    // disallow using the unsafe built-in Function type
+    '@typescript-eslint/no-unsafe-function-type': 2,
     // disallow unused expressions
     '@typescript-eslint/no-unused-expressions':
       eslintRules['no-unused-expressions'],
@@ -180,8 +180,8 @@ export default {
       eslintRules['no-useless-constructor'],
     // disallow unnecessary empty export
     '@typescript-eslint/no-useless-empty-export': 2,
-    // disallows the use of require statements except in import statements
-    '@typescript-eslint/no-var-requires': 2,
+    // disallow using confusing built-in primitive class wrappers
+    '@typescript-eslint/no-wrapper-object-types': 2,
     // require or disallow parameter properties in class constructors
     '@typescript-eslint/parameter-properties': 0,
     // prefer usage of `as const` over literal type