eslint-plugin-smarthr 3.3.2 → 3.4.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.4.0](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v3.3.2...eslint-plugin-smarthr-v3.4.0) (2025-12-24)
+
+
+### Features
+
+* best-practice-for-rest-parameters の残余引数の命名規則をrestもしくはxxxRestにすることを促すように修正 ([#973](https://github.com/kufu/tamatebako/issues/973)) ([3824818](https://github.com/kufu/tamatebako/commit/3824818a68d0ab6b2b2dc04ff5292f9d8be03d9b))
+
 ## [3.3.2](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v3.3.1...eslint-plugin-smarthr-v3.3.2) (2025-12-23)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-smarthr",
-  "version": "3.3.2",
+  "version": "3.4.0",
   "author": "SmartHR",
   "license": "MIT",
   "description": "A sharable ESLint plugin for SmartHR",
@@ -37,5 +37,5 @@
     "eslintplugin",
     "smarthr"
   ],
-  "gitHead": "6b6b572bcc35d4b23cb20ef6b29bedf011e3a881"
+  "gitHead": "25b508c89653cc0e754e091f83d8b31346afb08e"
 }

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

@@ -7,11 +7,12 @@
     - コンポーネントが受け取れる属性を定義する際、多用される "Props" 型と勘違いされる可能性を減らすためです
     - 残余引数の時点でコンポーネントが受け取れる属性全てではないことが確定するためpropsの利用を禁止しています
     - `rest` という名称に揃えることで可読性を向上させることが出来ます
-- 残余引数以外でrestという名称と完全一致する設定することを禁止します
+  - restがすでに利用されており設定出来ない場合 `xxxRest` というフォーマットが利用出来ます
+- 残余引数以外でrest、もしくはxxxRestというフォーマットの名称を設定することを禁止します
   - restは rest parametersから命名されたjsのイディオムのため、残余引数以外の箇所で利用すると混乱を招くためです
-    - `xxxRest` のように他単語と組合されている場合はエラーになりません
 - 残余引数内の属性を直接参照することを禁止します
   - 例: `const hoge = rest.fuga`
+  - 例: `const { hoge } = rest`
   - この条件を守る場合、残余引数がそのスコープ内で関心が薄い引数の集まりになり、可読性が向上します
 
 ## rules
@@ -27,7 +28,7 @@
 ## ❌ Incorrect
 
 ```js
-// 残余引数にrest以外の名称が利用されているためNG
+// 残余引数にrest、xxxRest以外の名称が利用されているためNG
 const hoge = (a, b, ...props) => {
   // any
 }
@@ -37,15 +38,15 @@ const hoge = ({ a, b, ...props }) => {
   // any
 }
 
-// 残余引数ではない箇所でrestという文字列と完全一致する場合NG
-const hoge = (a, rest, b) => {
+// 残余引数ではない箇所で残余引数と勘違いしかねない命名がされているためNG
+const hoge = (a, anyRest, b) => {
   // any
 }
 // 引数以外の場合でも混乱するためNG
-const rest = { /* any */ }
+const hogeRest = { /* any */ }
 
 // 残余引数内の属性を参照しているためNG
-const Component = ({ a, b, ...rest }) => {
+const ComponentA = ({ a, b, ...rest }) => {
   ...
 
   if (rest.abc) {
@@ -54,6 +55,18 @@ const Component = ({ a, b, ...rest }) => {
 
   ...
 }
+const ComponentB = ({ a, b, ...rest }) => {
+  ...
+
+  // 残余引数を構造分解することもNG
+  const { abc } = rest
+
+  if (abc) {
+    return <Children {...rest} />
+  }
+
+  ...
+}
 ```
 
 ## ✅ Correct
@@ -64,6 +77,9 @@ const Component = ({ a, b, ...rest }) => {
 const hoge = (a, b, ...rest) => {
   // any
 }
+const fuga = ({ a, b, ...anyRest }) => {
+  // any
+}
 
 // 残余引数ではない場合、rest以外の名称を許容
 const hoge = (props) => {

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

@@ -1,6 +1,7 @@
 const SCHEMA = []
 
-const MEMBER_EXPRESSION_REST_REGEX = /^rest\./
+const REST_REGEX = /(^r|R)est$/
+const MEMBER_EXPRESSION_REST_REGEX = /^(r|[a-zA-Z0-9_]+R)est\./
 const DETAIL_LINK = `
  - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-rest-parameters`
 
@@ -16,9 +17,9 @@ module.exports = {
     const actionNotRest = (node) => {
       context.report({
         node,
-        message: `残余引数以外に 'rest' という名称を利用しないでください${DETAIL_LINK}
+        message: `残余引数以外に ${REST_REGEX} とマッチする名称を利用しないでください${DETAIL_LINK}
  - 残余引数(rest parameters)と混同する可能性があるため別の名称に修正してください`,
-      });
+      })
     }
     const actionMemberExpressionName = (node) => {
       if (node.parent.type === 'MemberExpression') {
@@ -29,20 +30,32 @@ module.exports = {
         context.report({
           node,
           message: `残余引数内の属性を参照しないでください${DETAIL_LINK}`,
-        });
+        })
       }
     }
 
     return {
-      [`RestElement:not([argument.name='rest'])`]: (node) => {
+      [`ObjectPattern[properties.length=1]>RestElement`]: (node) => {
         context.report({
           node,
-          message: `残余引数には 'rest' という名称を指定してください${DETAIL_LINK}`,
-        });
+          message: `意味のない残余引数のため、単一の引数に変更してください${DETAIL_LINK}`,
+        })
+      },
+      [`RestElement:not([argument.name=${REST_REGEX}])`]: (node) => {
+        context.report({
+          node,
+          message: `残余引数には ${REST_REGEX} とマッチする名称を指定してください${DETAIL_LINK}`,
+        })
+      },
+      [`:not(:matches(RestElement,JSXSpreadAttribute,JSXSpreadAttribute>TSAsExpression,SpreadElement,SpreadElement>TSAsExpression,MemberExpression,VariableDeclarator,ArrayExpression,CallExpression,ObjectPattern>Property,ObjectExpression>Property))>Identifier[name=${REST_REGEX}]`]: actionNotRest,
+      [`:matches(VariableDeclarator[id.name=${REST_REGEX}],ObjectPattern>Property[value.name=${REST_REGEX}],ObjectExpression>Property[key.name=${REST_REGEX}])`]: actionNotRest,
+      [`MemberExpression[object.name=${REST_REGEX}]`]: actionMemberExpressionName,
+      [`VariableDeclarator[id.type='ObjectPattern'][init.name=${REST_REGEX}]`]: (node) => {
+        context.report({
+          node,
+          message: `残余引数内の属性を参照しないでください${DETAIL_LINK}`,
+        })
       },
-      [`:not(:matches(RestElement,JSXSpreadAttribute,JSXSpreadAttribute>TSAsExpression,SpreadElement,SpreadElement>TSAsExpression,MemberExpression,VariableDeclarator,ArrayExpression,CallExpression,ObjectPattern>Property,ObjectExpression>Property))>Identifier[name='rest']`]: actionNotRest,
-      [`:matches(VariableDeclarator[id.name='rest'],ObjectPattern>Property[value.name='rest'],ObjectExpression>Property[key.name='rest'])`]: actionNotRest,
-      [`MemberExpression[object.name='rest']`]: actionMemberExpressionName,
     }
   },
 }

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

@@ -13,37 +13,40 @@ const ruleTester = new RuleTester({
 
 const DETAIL_LINK = `
  - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/best-practice-for-rest-parameters`
-const ERROR_REST_NAME = `残余引数には 'rest' という名称を指定してください${DETAIL_LINK}`
-const ERROR_NOT_REST_NAME = `残余引数以外に 'rest' という名称を利用しないでください${DETAIL_LINK}
+const ERROR_REST_NAME = `残余引数には /(^r|R)est$/ とマッチする名称を指定してください${DETAIL_LINK}`
+const ERROR_NOT_REST_NAME = `残余引数以外に /(^r|R)est$/ とマッチする名称を利用しないでください${DETAIL_LINK}
  - 残余引数(rest parameters)と混同する可能性があるため別の名称に修正してください`
 const ERROR_REST_CHILD_REF = `残余引数内の属性を参照しないでください${DETAIL_LINK}`
 
 ruleTester.run('best-practice-for-rest-parameters', rule, {
   valid: [
+    { code: `const hoge = (...rest) => {}` },
     { code: `const hoge = (a, b, ...rest) => {}` },
-    { code: `const hoge = ({ a, b, ...rest }) => {}` },
+    { code: `const hoge = ({ a, b, ...anyRest }) => {}` },
     { code: `const hoge = (props) => {}` },
     { code: `const props = {}` },
-    { code: `const hogeRest = {}` },
     { code: `const hoge = { fuga: rest }` },
-    { code: `const hoge = rest` },
+    { code: `const hoge = hogeRest` },
     { code: `const hoge = hoge.rest.fuga` },
-    { code: `const hoge = { ...rest }` },
+    { code: `const hoge = { ...anyRest }` },
     { code: `const hoge = [rest]` },
-    { code: `hoge(rest)` },
+    { code: `hoge(fugaRest)` },
     { code: `<Any {...rest} />` },
   ],
   invalid: [
-    { code: `const hoge = (a, b, ...props) => {}`, errors: [ { message: ERROR_REST_NAME } ] },
-    { code: `const hoge = ({ a, b, ...props }) => {}`, errors: [ { message: ERROR_REST_NAME } ] },
+    { code: `const hoge = ({ ...rest }) => {}`, errors: [ { message: `意味のない残余引数のため、単一の引数に変更してください${DETAIL_LINK}` } ] },
+    { code: `const hoge = (a, b, ...any) => {}`, errors: [ { message: ERROR_REST_NAME } ] },
+    { code: `const hoge = ({ a, b, ...restHoge }) => {}`, errors: [ { message: ERROR_REST_NAME } ] },
     { code: `const hoge = (rest) => {}`, errors: [ { message: ERROR_NOT_REST_NAME } ] },
+    { code: `const hogeRest = {}`, errors: [ { message: ERROR_NOT_REST_NAME } ] },
     { code: `const hoge = (a, b, rest) => {}`, errors: [ { message: ERROR_NOT_REST_NAME } ] },
-    { code: `const hoge = ({ a: rest, b }) => {}`, errors: [ { message: ERROR_NOT_REST_NAME } ] },
+    { code: `const hoge = ({ a: anyRest, b }) => {}`, errors: [ { message: ERROR_NOT_REST_NAME } ] },
     { code: `const rest = {}`, errors: [ { message: ERROR_NOT_REST_NAME } ] },
-    { code: `const hoge = { rest }`, errors: [ { message: ERROR_NOT_REST_NAME } ] },
+    { code: `const hoge = { hogeRest }`, errors: [ { message: ERROR_NOT_REST_NAME } ] },
     { code: `const hoge = { rest: fuga }`, errors: [ { message: ERROR_NOT_REST_NAME } ] },
     { code: `const hoge = rest.hoge`, errors: [ { message: ERROR_REST_CHILD_REF } ] },
-    { code: `const hoge = rest.hoge.fuga`, errors: [ { message: ERROR_REST_CHILD_REF } ] },
+    { code: `const hoge = anyRest.hoge.fuga`, errors: [ { message: ERROR_REST_CHILD_REF } ] },
+    { code: `const { any } = rest`, errors: [ { message: ERROR_REST_CHILD_REF } ] },
   ]
 })