eslint-plugin-smarthr 6.10.2 → 6.10.4

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 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.
 
+## [6.10.4](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v6.10.3...eslint-plugin-smarthr-v6.10.4) (2026-04-13)
+
+
+### Bug Fixes
+
+* **require-i18n-text:** 許容する記号の拡充と、テストコードでの除外設定を追加 ([#1225](https://github.com/kufu/tamatebako/issues/1225)) ([705113f](https://github.com/kufu/tamatebako/commit/705113f9b599bb77bec35b2c9b28825603d581ea))
+
+## [6.10.3](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v6.10.2...eslint-plugin-smarthr-v6.10.3) (2026-04-10)
+
+
+### Bug Fixes
+
+* **require-barrel-import:** commonParentのbarrelを除外して同一ツリー内の相対importを許可 ([#1222](https://github.com/kufu/tamatebako/issues/1222)) ([fe5a68c](https://github.com/kufu/tamatebako/commit/fe5a68cd50a6f76dde0c403dd1ec33e5b1dd5d57))
+* trim-propsのセレクターをesquery互換へ修正し、ESLint 9.xでのクラッシュを解消 ([#1214](https://github.com/kufu/tamatebako/issues/1214)) ([d2345a0](https://github.com/kufu/tamatebako/commit/d2345a0623ebbf0f805c7c8a6b2b957ac21b0ffd))
+
 ## [6.10.2](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v6.10.1...eslint-plugin-smarthr-v6.10.2) (2026-04-09)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-smarthr",
-  "version": "6.10.2",
+  "version": "6.10.4",
   "author": "SmartHR",
   "license": "MIT",
   "description": "A sharable ESLint plugin for SmartHR",
@@ -37,5 +37,5 @@
     "eslintplugin",
     "smarthr"
   ],
-  "gitHead": "662f8876643b981d82a4a5aaf5d9a96d7d7ed58b"
+  "gitHead": "5e89902850d7084a12d50400dad9d1b0e5764e00"
 }

package/rules/require-barrel-import/index.js

@@ -104,6 +104,24 @@ const isImportedInsideImporter = (importerDir, importedPath) => {
   return importedPath === importerDir || importedPath.startsWith(importerDir + '/')
 }
 
+/**
+ * 2つのパスの共通の親ディレクトリを見つける
+ * @param {string} path1 - パス1
+ * @param {string} path2 - パス2
+ * @returns {string} 共通の親ディレクトリの絶対パス
+ */
+const findCommonParent = (path1, path2) => {
+  const segments1 = path1.split('/')
+  const segments2 = path2.split('/')
+
+  let i = 0
+  while (i < segments1.length && i < segments2.length && segments1[i] === segments2[i]) {
+    i++
+  }
+
+  return segments1.slice(0, i).join('/')
+}
+
 /**
  * allowedImportsオプションに基づいて、特定のimportが許可されているかチェックする
  * @param {object} node - ImportDeclaration node
@@ -167,6 +185,10 @@ const findBarrelFile = (importedPath, importerDir) => {
   let currentPath = importedPath
   let barrel = undefined
 
+  // import元とimport先の共通の親ディレクトリを見つける
+  // 共通の親のbarrelファイルは除外する（同じディレクトリツリー内の相対importには適用されない）
+  const commonParent = findCommonParent(importerDir, importedPath)
+
   // ディレクトリ指定の場合、そのindex.tsを指していることは自明なので一階層上から探索
   if (fs.existsSync(currentPath) && fs.statSync(currentPath).isDirectory()) {
     pathSegments.pop()
@@ -175,9 +197,10 @@ const findBarrelFile = (importedPath, importerDir) => {
 
   while (pathSegments.length > 0) {
     // 以下の場合は探索終了
-    // 1. いずれかのreplacePathsのルートに到達した場合
-    // 2. import先がimport元の内部にある場合（同階層・サブディレクトリからのimport）
-    if (ALL_ROOT_PATHS.includes(currentPath) || isImportedInsideImporter(importerDir, currentPath)) {
+    // 1. 共通の親ディレクトリに到達した場合（commonParent自体のbarrelは除外）
+    // 2. いずれかのreplacePathsのルートに到達した場合
+    // 3. import先がimport元の内部にある場合（同階層・サブディレクトリからのimport）
+    if (currentPath === commonParent || ALL_ROOT_PATHS.includes(currentPath) || isImportedInsideImporter(importerDir, currentPath)) {
       break
     }
 

package/rules/require-i18n-text/index.js

@@ -28,8 +28,9 @@ const generateAttributeSelector = (attributes) =>
 const generateTemplateLiteralSelector = (attributes) =>
   `JSXAttribute[name.name=/^(${attributes.join('|')})$/][value.type="JSXExpressionContainer"][value.expression.type="TemplateLiteral"]`
 
-const IGNORE_TEXT_REGEX = /^ *(\.|\+|\-|\*|\/|[0-9]+) *$/
-const checkIgnoreText = (text) => !IGNORE_TEXT_REGEX.test(text)
+const REGEX_IGNORE_FILENAME = /\.(spec|test|stories)\./
+const REGEX_IGNORE_TEXT = /^\s*(\.|\+|\-|〜|：|:|（|）|\(|\)|,|\*|\/|[0-9]+)\s*$/
+const checkIgnoreText = (text) => !REGEX_IGNORE_TEXT.test(text)
 
 const someReportTemplateLiteralError = (quasi) => quasi.value.cooked && quasi.value.cooked.trim() !== '' && checkIgnoreText(quasi.value.cooked)
 
@@ -42,6 +43,10 @@ module.exports = {
     schema: SCHEMA,
   },
   create(context) {
+    if (REGEX_IGNORE_FILENAME.test(context.getFilename())) {
+      return {}
+    }
+
     const elementsObj = (context.options[0] || {}).elements || {}
     // ユーザーが'*'を設定していない場合のみデフォルトを適用
     const wildcardAttributes = elementsObj['*'] || DEFAULT_WILDCARD_ATTRIBUTES
@@ -57,7 +62,7 @@ module.exports = {
       if (checkIgnoreText(node.value.value)) {
         context.report({
           node,
-          message: `${node.parent.name.name}の${node.name.name}属性に文字列リテラルが指定されています。多言語化対応のため、翻訳関数を使用してください
+          message: `${node.parent.name.name}の${node.name.name}属性に文字列リテラル "${node.value.value.trim()}" が指定されています。多言語化対応のため、翻訳関数を使用してください
  - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/require-i18n-text`,
         })
       }
@@ -101,7 +106,7 @@ module.exports = {
       if (checkIgnoreText(node.value)) {
         context.report({
           node,
-          message: `子要素に文字列リテラルが指定されています。多言語化対応のため、翻訳関数を使用してください
+          message: `子要素に文字列リテラル "${node.value.trim()}" が指定されています。多言語化対応のため、翻訳関数を使用してください
  - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/require-i18n-text`,
         })
       }

package/rules/trim-props/index.js

@@ -12,15 +12,23 @@ module.exports = {
     fixable: 'whitespace',
   },
   create(context) {
-    return {
-      ':matches(JSXAttribute > Literal[value=/(^ | $)/], JSXAttribute > JSXExpressionContainer > TemplateLiteral:has(> TemplateElement:matches(:first-child[value.raw=/^ /],:last-child[value.raw=/ $/])))': (node) => {
-        context.report({
-          node,
-          message: `属性に設定している文字列から先頭、末尾の空白文字を削除してください
+    const checker = (node) => {
+      context.report({
+        node,
+        message: `属性に設定している文字列から先頭、末尾の空白文字を削除してください
  - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/trim-props`,
-          fix: (fixer) => fixer.replaceText(node, context.sourceCode.getText(node).replace(TRIM_REGEX, '$1$2$3')),
-        })
-      },
+        fix: (fixer) => fixer.replaceText(node, context.sourceCode.getText(node).replace(TRIM_REGEX, '$1$2$3')),
+      })
+    }
+
+    return {
+      // esquery は :matches()・:has() 内での > (子結合子) を解釈できないため、
+      // 1つの複合セレクターではなく個別のセレクターに分割している
+      'JSXAttribute > Literal[value=/(^ | $)/]': checker,
+      'JSXAttribute > JSXExpressionContainer > TemplateLiteral > TemplateElement:matches(:first-child[value.raw=/^ /],:last-child[value.raw=/ $/])':
+        (node) => {
+          checker(node.parent)
+        },
     }
   },
 }

package/test/require-barrel-import.js

@@ -261,16 +261,14 @@ ruleTester.run('require-barrel-import', rule, {
         return `${fixturesRoot}/path-alias-parent-no-barrel/Button/Button.tsx`
       })(),
     },
-  ],
 
-  invalid: [
-    // 親階層からのimport（barrelが存在する場合）
+    // 親階層からのimport（commonParentにbarrelがある場合）
     {
       code: `import { createUserRole } from '../hooks/createUserRoleAction'`,
       filename: (() => {
-        createFixture('parent-import-with-barrel', {
+        createFixture('parent-import-common-parent-barrel', {
           'components': {
-            'index.tsx': 'export {}',
+            'index.tsx': 'export {}',  // commonParentのbarrelは除外される
             'AddDialog': {
               'AddDialog.tsx': '',
             },
@@ -279,91 +277,57 @@ ruleTester.run('require-barrel-import', rule, {
             },
           },
         })
-        return `${fixturesRoot}/parent-import-with-barrel/components/AddDialog/AddDialog.tsx`
+        return `${fixturesRoot}/parent-import-common-parent-barrel/components/AddDialog/AddDialog.tsx`
       })(),
-      errors: [
-        {
-          message: /からimportするか、.*のbarrelファイルを削除して直接import可能にしてください/,
-        },
-      ],
     },
 
-    // Next.js特殊文字パス - 親階層からのimport
+    // 複雑なネスト - commonParentのbarrelは除外される
     {
-      code: `import { createUserRole } from '../hooks/createUserRoleAction'`,
+      code: `import type { RequestStepActionNotSkipped } from '../utils/withSkipped'`,
       filename: (() => {
-        createFixture('nextjs-parent-import', {
-          'app': {
-            '(private)': {
-              'settings': {
-                'user_roles': {
-                  '_components': {
-                    'index.tsx': 'export {}',
-                    'AddUserRoleDialog': {
-                      'AddUserRoleDialog.tsx': '',
-                    },
-                    'hooks': {
-                      'createUserRoleAction.ts': '',
-                    },
+        createFixture('nested-common-parent-barrel', {
+          'Nodes': {
+            'index.tsx': 'export {}',  // commonParentより上のbarrel（除外される）
+            'StepNodeRequestView': {
+              'index.tsx': 'export {}',  // commonParent（除外される）
+              'Approvers': {
+                'ApproverRow': {
+                  'buildUserRowsProps.ts': '',
+                },
+                'utils': {
+                  'withSkipped': {
+                    'index.ts': 'export {}',  // import先のbarrel（valid）
                   },
                 },
               },
             },
           },
         })
-        return `${fixturesRoot}/nextjs-parent-import/app/(private)/settings/user_roles/_components/AddUserRoleDialog/AddUserRoleDialog.tsx`
+        return `${fixturesRoot}/nested-common-parent-barrel/Nodes/StepNodeRequestView/Approvers/ApproverRow/buildUserRowsProps.ts`
       })(),
-      errors: [
-        {
-          message: /からimportするか、.*のbarrelファイルを削除して直接import可能にしてください/,
-        },
-      ],
+      languageOptions: {
+        parser: require('typescript-eslint').parser,
+      },
     },
+  ],
 
-    // [id] Dynamic Routes - 親階層からのimport
+  invalid: [
+    // 親階層からのimport（import先の親にbarrelがある場合）
     {
       code: `import { api } from '../api/client'`,
       filename: (() => {
-        createFixture('dynamic-route-parent-import', {
-          'app': {
-            'items': {
-              'index.tsx': 'export {}',
-              '[id]': {
-                'DetailPage.tsx': '',
-              },
-              'api': {
-                'client.ts': '',
-              },
-            },
-          },
-        })
-        return `${fixturesRoot}/dynamic-route-parent-import/app/items/[id]/DetailPage.tsx`
-      })(),
-      errors: [
-        {
-          message: /からimportするか、.*のbarrelファイルを削除して直接import可能にしてください/,
-        },
-      ],
-    },
-
-    // ============================================================
-    // Path alias - 親階層からのimport（barrelあり）
-    // ============================================================
-    {
-      code: `import { createUserRole } from '~/path-alias-parent-import-with-barrel/components/hooks/createUserRoleAction'`,
-      filename: (() => {
-        createFixture('path-alias-parent-import-with-barrel', {
+        createFixture('parent-import-with-barrel', {
           'components': {
-            'index.tsx': 'export {}',
             'AddDialog': {
               'AddDialog.tsx': '',
             },
-            'hooks': {
-              'createUserRoleAction.ts': '',
+            'api': {
+              'index.tsx': 'export {}',  // import先の親にbarrel
+              'client.ts': '',
             },
           },
         })
-        return `${fixturesRoot}/path-alias-parent-import-with-barrel/components/AddDialog/AddDialog.tsx`
+        return `${fixturesRoot}/parent-import-with-barrel/components/AddDialog/AddDialog.tsx`
       })(),
       errors: [
         {
@@ -372,22 +336,22 @@ ruleTester.run('require-barrel-import', rule, {
       ],
     },
 
-    // Path alias + Next.js特殊文字パス - 親階層からのimport
+    // Next.js特殊文字パス - import先の親にbarrel
     {
-      code: `import { createUserRole } from '~/path-alias-nextjs-parent/app/(private)/settings/user_roles/_components/hooks/createUserRoleAction'`,
+      code: `import { api } from '../api/client'`,
       filename: (() => {
-        createFixture('path-alias-nextjs-parent', {
+        createFixture('nextjs-import-parent-barrel', {
           'app': {
             '(private)': {
               'settings': {
                 'user_roles': {
                   '_components': {
-                    'index.tsx': 'export {}',
                     'AddUserRoleDialog': {
                       'AddUserRoleDialog.tsx': '',
                     },
-                    'hooks': {
-                      'createUserRoleAction.ts': '',
+                    'api': {
+                      'index.tsx': 'export {}',  // import先の親にbarrel
+                      'client.ts': '',
                     },
                   },
                 },
@@ -395,7 +359,31 @@ ruleTester.run('require-barrel-import', rule, {
             },
           },
         })
-        return `${fixturesRoot}/path-alias-nextjs-parent/app/(private)/settings/user_roles/_components/AddUserRoleDialog/AddUserRoleDialog.tsx`
+        return `${fixturesRoot}/nextjs-import-parent-barrel/app/(private)/settings/user_roles/_components/AddUserRoleDialog/AddUserRoleDialog.tsx`
+      })(),
+      errors: [
+        {
+          message: /からimportするか、.*のbarrelファイルを削除して直接import可能にしてください/,
+        },
+      ],
+    },
+
+    // Path alias - import先の親にbarrel
+    {
+      code: `import { api } from '~/path-alias-import-parent-barrel/components/api/client'`,
+      filename: (() => {
+        createFixture('path-alias-import-parent-barrel', {
+          'components': {
+            'AddDialog': {
+              'AddDialog.tsx': '',
+            },
+            'api': {
+              'index.tsx': 'export {}',  // import先の親にbarrel
+              'client.ts': '',
+            },
+          },
+        })
+        return `${fixturesRoot}/path-alias-import-parent-barrel/components/AddDialog/AddDialog.tsx`
       })(),
       errors: [
         {

package/test/require-i18n-text.js

@@ -11,9 +11,9 @@ const ruleTester = new RuleTester({
   },
 })
 
-const attributeError = (element, attr) => `${element}の${attr}属性に文字列リテラルが指定されています。多言語化対応のため、翻訳関数を使用してください
+const attributeError = (element, attr, text) => `${element}の${attr}属性に文字列リテラル "${text}" が指定されています。多言語化対応のため、翻訳関数を使用してください
  - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/require-i18n-text`
-const childTextError = `子要素に文字列リテラルが指定されています。多言語化対応のため、翻訳関数を使用してください
+const childTextError = (text) => `子要素に文字列リテラル "${text}" が指定されています。多言語化対応のため、翻訳関数を使用してください
  - 詳細: https://github.com/kufu/tamatebako/tree/master/packages/eslint-plugin-smarthr/rules/require-i18n-text`
 
 const options = [
@@ -61,6 +61,14 @@ ruleTester.run('require-i18n-text', rule, {
     { code: `<img alt="-" />` },
     { code: `<i>*</i>` },
     { code: `<i>/</i>` },
+    { code: `<div>〜</div>` },
+    { code: `<div>：</div>` },
+    { code: `<div>:</div>` },
+    { code: `<div>（</div>` },
+    { code: `<div>）</div>` },
+    { code: `<div>(</div>` },
+    { code: `<div>)</div>` },
+    { code: `<div>,</div>` },
 
     // ワイルドカード - 空配列で除外
     {
@@ -96,41 +104,41 @@ ruleTester.run('require-i18n-text', rule, {
     // 属性エラー: デフォルト設定
     {
       code: `<img alt="Profile picture" />`,
-      errors: [{ message: attributeError('img', 'alt') }],
+      errors: [{ message: attributeError('img', 'alt', 'Profile picture') }],
     },
     {
       code: `<CustomComponent aria-label="Label" />`,
-      errors: [{ message: attributeError('CustomComponent', 'aria-label') }],
+      errors: [{ message: attributeError('CustomComponent', 'aria-label', 'Label') }],
     },
     {
       code: `<DefinitionListItem term="Label" />`,
-      errors: [{ message: attributeError('DefinitionListItem', 'term') }],
+      errors: [{ message: attributeError('DefinitionListItem', 'term', 'Label') }],
     },
     {
       code: `<button title="Click me" />`,
-      errors: [{ message: attributeError('button', 'title') }],
+      errors: [{ message: attributeError('button', 'title', 'Click me') }],
     },
 
     // 数値、.と演算記号の場合でも他の文字列が含まれていればエラー
-    { code: `<Any aria-label="1234 あ" />`, errors: [{ message: attributeError('Any', 'aria-label') }] },
-    { code: `<div>a.</div>`, errors: [{ message: childTextError }] },
-    { code: `<a> + b</a>`, errors: [{ message: childTextError }] },
-    { code: `<img alt="-zod" />`, errors: [{ message: attributeError('img', 'alt') }] },
-    { code: `<i>*1</i>`, errors: [{ message: childTextError }] },
-    { code: `<i>a/</i>`, errors: [{ message: childTextError }] },
+    { code: `<Any aria-label="1234 あ" />`, errors: [{ message: attributeError('Any', 'aria-label', '1234 あ') }] },
+    { code: `<div>a.</div>`, errors: [{ message: childTextError('a.') }] },
+    { code: `<a> + b</a>`, errors: [{ message: childTextError('+ b') }] },
+    { code: `<img alt="-zod" />`, errors: [{ message: attributeError('img', 'alt', '-zod') }] },
+    { code: `<i>*1</i>`, errors: [{ message: childTextError('*1') }] },
+    { code: `<i>a/</i>`, errors: [{ message: childTextError('a/') }] },
 
     // 属性エラー: カスタムオプション
     {
       code: `<img alt="Profile picture" />`,
       options,
-      errors: [{ message: attributeError('img', 'alt') }],
+      errors: [{ message: attributeError('img', 'alt', 'Profile picture') }],
     },
 
     // 属性エラー: 同一要素の複数属性
     {
       code: `<img alt="Profile" title="User profile" />`,
       options,
-      errors: [{ message: attributeError('img', 'alt') }, { message: attributeError('img', 'title') }],
+      errors: [{ message: attributeError('img', 'alt', 'Profile') }, { message: attributeError('img', 'title', 'User profile') }],
     },
 
     // 属性エラー: ワイルドカード
@@ -143,7 +151,7 @@ ruleTester.run('require-i18n-text', rule, {
           },
         },
       ],
-      errors: [{ message: attributeError('CustomComponent', 'label') }],
+      errors: [{ message: attributeError('CustomComponent', 'label', 'Text') }],
     },
 
     // 属性エラー: 個別設定がワイルドカードより優先
@@ -157,20 +165,20 @@ ruleTester.run('require-i18n-text', rule, {
           },
         },
       ],
-      errors: [{ message: attributeError('Button', 'label') }, { message: attributeError('Button', 'helperText') }],
+      errors: [{ message: attributeError('Button', 'label', 'Submit') }, { message: attributeError('Button', 'helperText', 'Help') }],
     },
 
     // 子要素エラー（オプション未設定時でもチェックされる）
     {
       code: `<div>Hello World</div>`,
-      errors: [{ message: childTextError }],
+      errors: [{ message: childTextError('Hello World') }],
     },
 
     // 複合エラー: 属性と子要素
     {
       code: `<Button label="Submit">Click here</Button>`,
       options,
-      errors: [{ message: attributeError('Button', 'label') }, { message: childTextError }],
+      errors: [{ message: attributeError('Button', 'label', 'Submit') }, { message: childTextError('Click here') }],
     },
 
     // 複合エラー: 入れ子構造
@@ -178,22 +186,10 @@ ruleTester.run('require-i18n-text', rule, {
       code: `<div title="Parent"><Button label="Child">Grandchild text</Button></div>`,
       options,
       errors: [
-        { message: attributeError('div', 'title') },
-        { message: attributeError('Button', 'label') },
-        { message: childTextError },
+        { message: attributeError('div', 'title', 'Parent') },
+        { message: attributeError('Button', 'label', 'Child') },
+        { message: childTextError('Grandchild text') },
       ],
     },
-
-    // TemplateLiteral - 変数を含み、文字列リテラル部分がある
-    {
-      code: `<img alt={\`Profile \${name}\`} />`,
-      options,
-      errors: [{ message: attributeError('img', 'alt') }],
-    },
-    {
-      code: `<div title={\`\${prefix} title\`} />`,
-      options,
-      errors: [{ message: attributeError('div', 'title') }],
-    },
   ],
 })

package/test/trim-props.js

@@ -55,12 +55,12 @@ ruleTester.run('trim-props', rule, {
     {
       code: '<div data-spec={` a${b} c `}>....</div>',
       output: '<div data-spec={`a${b} c`}>....</div>',
-      errors: [{ message: ERROR_MESSAGE }],
+      errors: [{ message: ERROR_MESSAGE }, { message: ERROR_MESSAGE }],
     },
     {
       code: '<div data-spec={` a${b ? ` ${c} ` : "  "} d `}>....</div>',
       output: '<div data-spec={`a${b ? ` ${c} ` : "  "} d`}>....</div>',
-      errors: [{ message: ERROR_MESSAGE }],
+      errors: [{ message: ERROR_MESSAGE }, { message: ERROR_MESSAGE }],
     },
   ],
 })