eslint-plugin-smarthr 3.2.0 → 3.3.0

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+## [3.3.0](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v3.2.0...eslint-plugin-smarthr-v3.3.0) (2025-12-18)
+
+
+### Features
+
+* best-practice-for-rest-parametersを追加 ([#953](https://github.com/kufu/tamatebako/issues/953)) ([093f8f2](https://github.com/kufu/tamatebako/commit/093f8f258b65a746cc10ae90696827600ca45dfc))
+
 ## [3.2.0](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v3.1.0...eslint-plugin-smarthr-v3.2.0) (2025-12-15)
 
 

package/README.md

@@ -23,6 +23,7 @@
 - [best-practice-for-nested-attributes-array-index](https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-nested-attributes-array-index)
 - [best-practice-for-prohibit-import-smarthr-ui-local](https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-prohibit-import-smarthr-ui-local)
 - [best-practice-for-remote-trigger-dialog](https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-remote-trigger-dialog)
+- [best-practice-for-rest-parameters](https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-rest-parameters)
 - [best-practice-for-tailwind-prohibit-root-margin](https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-tailwind-prohibit-root-margin)
 - [best-practice-for-tailwind-variants](https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-tailwind-variants)
 - [design-system-guideline-prohibit-double-icons](https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/design-system-guideline-prohibit-double-icons)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-smarthr",
-  "version": "3.2.0",
+  "version": "3.3.0",
   "author": "SmartHR",
   "license": "MIT",
   "description": "A sharable ESLint plugin for SmartHR",
@@ -37,5 +37,5 @@
     "eslintplugin",
     "smarthr"
   ],
-  "gitHead": "c6e4fb54bb2ff5d8421c656213dc29f3bc7237ac"
+  "gitHead": "61261c2334b39d607b47e1f5be64fdb74f4ac83f"
 }

package/rules/best-practice-for-rest-parameters/README.md

@@ -0,0 +1,63 @@
+# smarthr/best-practice-for-rest-parameters
+
+- 残余引数(rest parameters)の命名規則を設定するルールです
+  - https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Functions/rest_parameters
+- 残余引数にはrestという名称を設定することを推奨します
+  - よく利用される `props` という名称と完全一致する場合、エラーになります
+    - コンポーネントが受け取れる属性を定義する際、多用される "Props" 型と勘違いされる可能性を減らすためです
+    - 残余引数の時点でコンポーネントが受け取れる属性全てではないことが確定するためpropsの利用を禁止しています
+    - `xxxProps` のように他単語と組合されている場合はエラーになりません
+- 残余引数以外でrestという名称と完全一致する設定することを禁止します
+  - restは rest parametersから命名されたjsのイディオムのため、残余引数以外の箇所で利用すると混乱を招くためです
+    - `xxxRest` のように他単語と組合されている場合はエラーになりません
+
+## rules
+
+```js
+{
+  rules: {
+    'smarthr/best-practice-for-rest-parameters': 'error', // 'warn', 'off'
+  },
+}
+```
+
+## ❌ Incorrect
+
+```js
+// 残余引数にpropsという名称が設定されているためNG
+const hoge = (a, b, ...props) => {
+  // any
+}
+
+// オブジェクトの残余引数の場合も同様のチェックを行うためNG
+const hoge = ({ a, b, ...props }) => {
+  // any
+}
+
+// 残余引数ではない箇所でrestという文字列と完全一致する場合NG
+const hoge = (a, rest, b) => {
+  // any
+}
+// 引数以外の場合でも混乱するためNG
+const rest = { /* any */ }
+```
+
+## ✅ Correct
+
+
+```js
+// 残余引数にrestという名称が設定されておりOK
+const hoge = (a, b, ...rest) => {
+  // any
+}
+// 残余引数でもprops以外は許容
+const hoge = ({ a, b, ...actionButtonProps }) => {
+  // any
+}
+
+// 残余引数以外の場合はpropsを許容
+const hoge = (props) => {
+  // any
+}
+const props = { /* any */ }
+```

package/rules/best-practice-for-rest-parameters/index.js

@@ -0,0 +1,32 @@
+const SCHEMA = []
+
+/**
+ * @type {import('@typescript-eslint/utils').TSESLint.RuleModule<''>}
+ */
+module.exports = {
+  meta: {
+    type: 'suggestion',
+    schema: SCHEMA,
+  },
+  create(context) {
+    return {
+      [`RestElement[argument.name='props']`]: (node) => {
+        context.report({
+          node,
+          message: `残余引数には 'props' という名称を利用しないでください
+ - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-rest-parameters
+ - 'rest' という名称を推奨します`,
+        });
+      },
+      [`:not(RestElement)>Identifier[name='rest']`]: (node) => {
+        context.report({
+          node,
+          message: `残余引数以外に 'rest' という名称を利用しないでください
+ - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-rest-parameters
+ - 残余引数(rest parameters)と混同する可能性があるため別の名称に修正してください`,
+        });
+      },
+    }
+  },
+}
+module.exports.schema = SCHEMA

package/test/best-practice-for-rest-parameters.js

@@ -0,0 +1,39 @@
+const rule = require('../rules/best-practice-for-rest-parameters')
+const RuleTester = require('eslint').RuleTester
+
+const ruleTester = new RuleTester({
+  languageOptions: {
+    parserOptions: {
+      ecmaFeatures: {
+        jsx: true,
+      },
+    },
+  },
+})
+
+const ERROR_PROPS = `残余引数には 'props' という名称を利用しないでください
+ - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-rest-parameters
+ - 'rest' という名称を推奨します`
+const ERROR_REST = `残余引数以外に 'rest' という名称を利用しないでください
+ - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-rest-parameters
+ - 残余引数(rest parameters)と混同する可能性があるため別の名称に修正してください`
+
+ruleTester.run('best-practice-for-rest-parameters', rule, {
+  valid: [
+    { code: `const hoge = (a, b, ...rest) => {}` },
+    { code: `const hoge = ({ a, b, ...rest }) => {}` },
+    { code: `const hoge = (a, b, ...hoge) => {}` },
+    { code: `const hoge = ({ a, b, ...xxxProps }) => {}` },
+    { code: `const hoge = (props) => {}` },
+    { code: `const props = {}` },
+    { code: `const hogeRest = {}` },
+  ],
+  invalid: [
+    { code: `const hoge = (a, b, ...props) => {}`, errors: [ { message: ERROR_PROPS } ] },
+    { code: `const hoge = ({ a, b, ...props }) => {}`, errors: [ { message: ERROR_PROPS } ] },
+    { code: `const hoge = (rest) => {}`, errors: [ { message: ERROR_REST } ] },
+    { code: `const hoge = (a, b, rest) => {}`, errors: [ { message: ERROR_REST } ] },
+    { code: `const rest = {}`, errors: [ { message: ERROR_REST } ] },
+  ]
+})
+