eslint-plugin-smarthr 6.10.2 → 6.10.3

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 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.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.3",
   "author": "SmartHR",
   "license": "MIT",
   "description": "A sharable ESLint plugin for SmartHR",
@@ -37,5 +37,5 @@
     "eslintplugin",
     "smarthr"
   ],
-  "gitHead": "662f8876643b981d82a4a5aaf5d9a96d7d7ed58b"
+  "gitHead": "e1a8159fdc6396bb83fdf7f616f52f3c714ff390"
 }

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/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/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 }],
     },
   ],
 })