eslint-config-moneyforward 4.0.0 → 4.1.0

package/README.md

@@ -45,6 +45,7 @@ We also provide various other rule sets that you can configure to suit your proj
 import {
   essentials,
   jsdoc,
+  next,
   node,
   react,
   storybook,

package/configs/flat/essentials.mjs

@@ -1,4 +1,5 @@
 // @ts-check
+import globals from 'globals';
 // @ts-ignore
 import importPlugin from 'eslint-plugin-import';
 import promisePlugin from 'eslint-plugin-promise';
@@ -33,6 +34,16 @@ const promiseRuleSet = {
 };
 
 export default [
+  {
+    languageOptions: {
+      parserOptions: {
+        ecmaVersion: 'latest',
+        sourceType: 'module',
+      },
+      globals: globals.builtin,
+    },
+  },
+
   bestPracticesRuleSet,
   errorsRuleSet,
   es6RuleSet,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-moneyforward",
-  "version": "4.0.0",
+  "version": "4.1.0",
   "description": "Money Forward's ESLint rules as an extensible shared config.",
   "exports": {
     "./flat": {
@@ -54,7 +54,8 @@
     "eslint-plugin-react": "^7.34.0",
     "eslint-plugin-react-hooks": "^4.6.0",
     "eslint-plugin-storybook": "^0.8.0",
-    "eslint-plugin-testing-library": "^6.2.2"
+    "eslint-plugin-testing-library": "^6.2.2",
+    "globals": "^15.9.0"
   },
   "devDependencies": {
     "@semantic-release/changelog": "6.0.3",
@@ -62,10 +63,10 @@
     "eslint": "8.57.0",
     "eslint-config-prettier": "9.1.0",
     "jest": "29.7.0",
-    "prettier": "3.2.5",
+    "prettier": "3.3.3",
     "prettier-plugin-organize-imports": "3.2.4",
     "semantic-release": "23.1.1",
-    "typescript": "5.4.5"
+    "typescript": "5.5.4"
   },
   "peerDependencies": {
     "eslint": "^8.56.0",

package/rules/imports.js

@@ -18,13 +18,13 @@ module.exports = {
   settings: {
     'import/resolver': {
       node: {
-        extensions: ['.js', '.mjs', '.json', 'ts', 'mts'],
+        extensions: ['.js', '.mjs', '.jsx', '.ts', '.mts', '.tsx'],
       },
       // Resolve the problem of incorrect recognition of alias paths by TypeScript compiler options.
       // https://github.com/import-js/eslint-plugin-import/issues/1485#issuecomment-535351922
       typescript: {},
     },
-    'import/extensions': ['.js', '.mjs', '.jsx'],
+    'import/extensions': ['.js', '.mjs', '.jsx', '.ts', '.mts', '.tsx'],
     'import/core-modules': [],
     'import/ignore': [
       'node_modules',
@@ -33,7 +33,7 @@ module.exports = {
     // TODO: Remove this once eslint-plugin-import supports Flat Config completely.
     // https://github.com/import-js/eslint-plugin-import/issues/2556#issuecomment-1419518561
     'import/parsers': {
-      espree: ['.js', '.mjs', '.jsx', 'ts', 'mts', 'tsx'],
+      espree: ['.js', 'cjs', '.mjs', '.jsx'],
     },
   },
 

package/rules/react.js

@@ -96,7 +96,13 @@ module.exports = {
     // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-pascal-case.md
     'react/jsx-pascal-case': ['error', { allowAllCaps: true }],
 
-    // Enforce props alphabetical sorting
+    // Enforce props sorting
+    // The order of props is as follows:
+    //   1. React reserved props: e.g. key, ref
+    //   2. Shorthand props: e.g. disabled
+    //   3. Others
+    //   4. Callbacks e.g. onClick
+    // The props will no be sorted alphabetically.
     // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-sort-props.md
     'react/jsx-sort-props': [
       'error',