eslint-plugin-smarthr 6.10.1 → 6.10.3

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.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)
+
+
+### Bug Fixes
+
+* **test:** typescript-eslint v8のparserに更新 ([#1217](https://github.com/kufu/tamatebako/issues/1217)) ([e44ac27](https://github.com/kufu/tamatebako/commit/e44ac27e26afc9cbe26244afd38e62bb65aecaa4))
+
 ## [6.10.1](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v6.10.0...eslint-plugin-smarthr-v6.10.1) (2026-04-08)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-smarthr",
-  "version": "6.10.1",
+  "version": "6.10.3",
   "author": "SmartHR",
   "license": "MIT",
   "description": "A sharable ESLint plugin for SmartHR",
@@ -37,5 +37,5 @@
     "eslintplugin",
     "smarthr"
   ],
-  "gitHead": "e2e55eea3e72809fab55082ad6ef4da0c6025ae8"
+  "gitHead": "e1a8159fdc6396bb83fdf7f616f52f3c714ff390"
 }

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

@@ -37,13 +37,25 @@ const REGEX_ROOT_PATH = new RegExp(`^${rootPath}/index\.`)
 const REGEX_INDEX_FILE = /\/index\.(ts|js)x?$/
 const TARGET_EXTS = ['ts', 'tsx', 'js', 'jsx']
 
-// Path aliasの正規表現を事前生成してキャッシュ
-const entriedReplacePathsWithRegex = Object.entries(replacePaths).map(([key, values]) => [
-  key,
-  values,
-  new RegExp(`^${key}(.+)$`),
-  values.map(v => new RegExp(`^${path.resolve(`${CWD}/${v}`)}(.+)$`))
-])
+// Path aliasの情報を事前計算してキャッシュ
+const REPLACE_PATHS_INFO = Object.entries(replacePaths).map(([key, values]) => {
+  const resolvedPaths = values.map(v => path.resolve(`${CWD}/${v.replace(/\/\*$/, '')}`))
+  return {
+    key,
+    values,
+    keyRegex: new RegExp(`^${key}(.+)$`),
+    resolvedPaths,
+    valueRegexes: resolvedPaths.map(p => new RegExp(`^${p}(.+)$`))
+  }
+})
+
+// @/ と ~/ のパスのみをrootとする（READMEの仕様通り）
+const ALL_ROOT_PATHS = (() => {
+  const rootKeys = ['@/', '~/']
+  return REPLACE_PATHS_INFO
+    .filter(info => rootKeys.includes(info.key))
+    .flatMap(info => info.resolvedPaths)
+})()
 
 /**
  * Path aliasを絶対パスに変換する
@@ -55,17 +67,13 @@ const resolvePathAlias = (importPath) => {
     return importPath
   }
 
-  return entriedReplacePathsWithRegex.reduce((result, [key, values, keyRegex]) => {
-    if (result === importPath) {
-      return values.reduce((resolved, value) => {
-        if (resolved === result && keyRegex.test(result)) {
-          return resolved.replace(keyRegex, `${path.resolve(`${CWD}/${value}`)}/$1`)
-        }
-        return resolved
-      }, result)
+  for (const { keyRegex, resolvedPaths } of REPLACE_PATHS_INFO) {
+    if (keyRegex.test(importPath)) {
+      return importPath.replace(keyRegex, `${resolvedPaths[0]}/$1`)
     }
-    return result
-  }, importPath)
+  }
+
+  return importPath
 }
 
 /**
@@ -74,20 +82,15 @@ const resolvePathAlias = (importPath) => {
  * @returns {string} Path alias（例: '@/components/Button'）
  */
 const convertToPathAlias = (absolutePath) => {
-  return entriedReplacePathsWithRegex.reduce((result, [key, values, keyRegex, valueRegexes]) => {
-    if (result === absolutePath) {
-      return values.reduce((converted, value, index) => {
-        if (converted === result) {
-          const regexp = valueRegexes[index]
-          if (regexp.test(converted)) {
-            return converted.replace(regexp, `${key}/$1`).replace(REGEX_UNNECESSARY_SLASH, '/')
-          }
-        }
-        return converted
-      }, result)
+  for (const { key, valueRegexes } of REPLACE_PATHS_INFO) {
+    for (const regexp of valueRegexes) {
+      if (regexp.test(absolutePath)) {
+        return absolutePath.replace(regexp, `${key}/$1`).replace(REGEX_UNNECESSARY_SLASH, '/')
+      }
     }
-    return result
-  }, absolutePath)
+  }
+
+  return absolutePath
 }
 
 /**
@@ -101,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
@@ -164,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()
@@ -172,9 +197,10 @@ const findBarrelFile = (importedPath, importerDir) => {
 
   while (pathSegments.length > 0) {
     // 以下の場合は探索終了
-    // 1. root pathに到達した場合
-    // 2. import先がimport元の内部にある場合（同階層・サブディレクトリからのimport）
-    if (importerDir === rootPath || 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/prohibit-export-array-type.js

@@ -1,9 +1,10 @@
 const rule = require('../rules/prohibit-export-array-type')
 const RuleTester = require('eslint').RuleTester
+const tseslint = require('typescript-eslint')
 
 const ruleTester = new RuleTester({
   languageOptions: {
-    parser: require('@typescript-eslint/parser'),
+    parser: tseslint.parser,
     parserOptions: {
       ecmaFeatures: {
         jsx: true,

package/test/require-barrel-import.js

@@ -0,0 +1,395 @@
+const fs = require('fs')
+const path = require('path')
+const rule = require('../rules/require-barrel-import')
+const RuleTester = require('eslint').RuleTester
+
+const ruleTester = new RuleTester({
+  languageOptions: {
+    parserOptions: {
+      ecmaFeatures: {
+        jsx: true,
+      },
+    },
+  },
+})
+
+// テストフィクスチャのルートディレクトリ
+const fixturesRoot = path.join(__dirname, '..', 'test-fixtures')
+
+/**
+ * テスト用のファイル構造を作成するヘルパー
+ * @param {string} testName - テスト名（ディレクトリ名として使用）
+ * @param {Object} structure - ファイル構造定義
+ * @returns {string} 作成したディレクトリのパス
+ */
+function createFixture(testName, structure) {
+  const fixtureDir = path.join(fixturesRoot, testName)
+
+  // ディレクトリが既に存在する場合は削除
+  if (fs.existsSync(fixtureDir)) {
+    fs.rmSync(fixtureDir, { recursive: true, force: true })
+  }
+
+  // ディレクトリとファイルを再帰的に作成
+  function createStructure(dir, struct) {
+    fs.mkdirSync(dir, { recursive: true })
+
+    for (const [name, content] of Object.entries(struct)) {
+      const fullPath = path.join(dir, name)
+
+      if (typeof content === 'object' && content !== null) {
+        // ディレクトリ
+        createStructure(fullPath, content)
+      } else {
+        // ファイル
+        fs.writeFileSync(fullPath, content || '')
+      }
+    }
+  }
+
+  createStructure(fixtureDir, structure)
+  return fixtureDir
+}
+
+/**
+ * テスト終了後のクリーンアップ
+ */
+function cleanupFixtures() {
+  if (fs.existsSync(fixturesRoot)) {
+    const entries = fs.readdirSync(fixturesRoot)
+    for (const entry of entries) {
+      const fullPath = path.join(fixturesRoot, entry)
+      if (fs.statSync(fullPath).isDirectory()) {
+        fs.rmSync(fullPath, { recursive: true, force: true })
+      }
+    }
+  }
+}
+
+// テスト終了後にクリーンアップ
+afterAll(() => {
+  cleanupFixtures()
+})
+
+ruleTester.run('require-barrel-import', rule, {
+  valid: [
+    // 同階層・サブディレクトリからのimport（エラーにならない）
+    {
+      code: `import { useMenu } from './hooks/useMenu'`,
+      filename: (() => {
+        createFixture('same-level-import', {
+          'Menu': {
+            'MenuItem.tsx': '',
+            'index.tsx': 'export {}',
+            'hooks': {
+              'useMenu.ts': '',
+            },
+          },
+        })
+        return `${fixturesRoot}/same-level-import/Menu/MenuItem.tsx`
+      })(),
+    },
+
+    // Next.js App Router特殊文字パス - 同階層import
+    {
+      code: `import { useUsers } from './hooks/useUsers'`,
+      filename: (() => {
+        createFixture('nextjs-special-chars', {
+          'app': {
+            '(private)': {
+              'settings': {
+                'user_roles': {
+                  '_components': {
+                    'index.tsx': 'export {}',
+                    'AddUserRoleDialog': {
+                      'index.tsx': 'export {}',
+                      'AddUserRoleDialog.tsx': '',
+                      'hooks': {
+                        'useUsers.ts': '',
+                      },
+                    },
+                  },
+                },
+              },
+            },
+          },
+        })
+        return `${fixturesRoot}/nextjs-special-chars/app/(private)/settings/user_roles/_components/AddUserRoleDialog/AddUserRoleDialog.tsx`
+      })(),
+    },
+
+    // Dynamic Routes - [id]パス（同階層）
+    {
+      code: `import { useDetail } from './hooks/useDetail'`,
+      filename: (() => {
+        createFixture('nextjs-dynamic-route', {
+          'app': {
+            'items': {
+              '[id]': {
+                'index.tsx': 'export {}',
+                'DetailPage.tsx': '',
+                'hooks': {
+                  'useDetail.ts': '',
+                },
+              },
+            },
+          },
+        })
+        return `${fixturesRoot}/nextjs-dynamic-route/app/items/[id]/DetailPage.tsx`
+      })(),
+    },
+
+    // barrel が存在しない場合（同階層サブディレクトリ）
+    {
+      code: `import { helper } from './utils/helper'`,
+      filename: (() => {
+        createFixture('no-barrel', {
+          'components': {
+            'Button': {
+              'Button.tsx': '',
+              'utils': {
+                'helper.ts': '',
+              },
+            },
+          },
+        })
+        return `${fixturesRoot}/no-barrel/components/Button/Button.tsx`
+      })(),
+    },
+
+    // 親階層からのimport + barrelなし（エラーにならない）
+    {
+      code: `import { helper } from '../utils/helper'`,
+      filename: (() => {
+        createFixture('parent-import-no-barrel', {
+          // index.tsx なし（barrelなし）
+          'Button': {
+            'Button.tsx': '',
+          },
+          'utils': {
+            'helper.ts': '',
+          },
+        })
+        return `${fixturesRoot}/parent-import-no-barrel/Button/Button.tsx`
+      })(),
+    },
+
+    // Next.js特殊文字パス - 親階層からのimport + barrelなし
+    {
+      code: `import { createUserRole } from '../hooks/createUserRoleAction'`,
+      filename: (() => {
+        createFixture('nextjs-parent-no-barrel', {
+          // index.tsx なし（barrelなし）
+          'AddUserRoleDialog': {
+            'AddUserRoleDialog.tsx': '',
+          },
+          'hooks': {
+            'createUserRoleAction.ts': '',
+          },
+        })
+        return `${fixturesRoot}/nextjs-parent-no-barrel/AddUserRoleDialog/AddUserRoleDialog.tsx`
+      })(),
+    },
+
+    // Dynamic Routes - 親階層からのimport + barrelなし
+    {
+      code: `import { api } from '../api/client'`,
+      filename: (() => {
+        createFixture('dynamic-route-parent-no-barrel', {
+          // index.tsx なし（barrelなし）
+          '[id]': {
+            'DetailPage.tsx': '',
+          },
+          'api': {
+            'client.ts': '',
+          },
+        })
+        return `${fixturesRoot}/dynamic-route-parent-no-barrel/[id]/DetailPage.tsx`
+      })(),
+    },
+
+    // ============================================================
+    // Path alias - 同階層からのimport（エラーにならない）
+    // ============================================================
+    {
+      code: `import { useMenu } from '~/path-alias-same-level/Menu/hooks/useMenu'`,
+      filename: (() => {
+        createFixture('path-alias-same-level', {
+          'Menu': {
+            'MenuItem.tsx': '',
+            'index.tsx': 'export {}',
+            'hooks': {
+              'useMenu.ts': '',
+            },
+          },
+        })
+        return `${fixturesRoot}/path-alias-same-level/Menu/MenuItem.tsx`
+      })(),
+    },
+
+    // Path alias - 同階層サブディレクトリ + barrelなし
+    {
+      code: `import { helper } from '~/path-alias-no-barrel/components/Button/utils/helper'`,
+      filename: (() => {
+        createFixture('path-alias-no-barrel', {
+          'components': {
+            'Button': {
+              'Button.tsx': '',
+              'utils': {
+                'helper.ts': '',
+              },
+            },
+          },
+        })
+        return `${fixturesRoot}/path-alias-no-barrel/components/Button/Button.tsx`
+      })(),
+    },
+
+    // Path alias - 親階層からのimport + barrelなし
+    {
+      code: `import { helper } from '~/path-alias-parent-no-barrel/utils/helper'`,
+      filename: (() => {
+        createFixture('path-alias-parent-no-barrel', {
+          // index.tsx なし（barrelなし）
+          'Button': {
+            'Button.tsx': '',
+          },
+          'utils': {
+            'helper.ts': '',
+          },
+        })
+        return `${fixturesRoot}/path-alias-parent-no-barrel/Button/Button.tsx`
+      })(),
+    },
+
+    // 親階層からのimport（commonParentにbarrelがある場合）
+    {
+      code: `import { createUserRole } from '../hooks/createUserRoleAction'`,
+      filename: (() => {
+        createFixture('parent-import-common-parent-barrel', {
+          'components': {
+            'index.tsx': 'export {}',  // commonParentのbarrelは除外される
+            'AddDialog': {
+              'AddDialog.tsx': '',
+            },
+            'hooks': {
+              'createUserRoleAction.ts': '',
+            },
+          },
+        })
+        return `${fixturesRoot}/parent-import-common-parent-barrel/components/AddDialog/AddDialog.tsx`
+      })(),
+    },
+
+    // 複雑なネスト - commonParentのbarrelは除外される
+    {
+      code: `import type { RequestStepActionNotSkipped } from '../utils/withSkipped'`,
+      filename: (() => {
+        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}/nested-common-parent-barrel/Nodes/StepNodeRequestView/Approvers/ApproverRow/buildUserRowsProps.ts`
+      })(),
+      languageOptions: {
+        parser: require('typescript-eslint').parser,
+      },
+    },
+  ],
+
+  invalid: [
+    // 親階層からのimport（import先の親にbarrelがある場合）
+    {
+      code: `import { api } from '../api/client'`,
+      filename: (() => {
+        createFixture('parent-import-with-barrel', {
+          'components': {
+            'AddDialog': {
+              'AddDialog.tsx': '',
+            },
+            'api': {
+              'index.tsx': 'export {}',  // import先の親にbarrel
+              'client.ts': '',
+            },
+          },
+        })
+        return `${fixturesRoot}/parent-import-with-barrel/components/AddDialog/AddDialog.tsx`
+      })(),
+      errors: [
+        {
+          message: /からimportするか、.*のbarrelファイルを削除して直接import可能にしてください/,
+        },
+      ],
+    },
+
+    // Next.js特殊文字パス - import先の親にbarrel
+    {
+      code: `import { api } from '../api/client'`,
+      filename: (() => {
+        createFixture('nextjs-import-parent-barrel', {
+          'app': {
+            '(private)': {
+              'settings': {
+                'user_roles': {
+                  '_components': {
+                    'AddUserRoleDialog': {
+                      'AddUserRoleDialog.tsx': '',
+                    },
+                    'api': {
+                      'index.tsx': 'export {}',  // import先の親にbarrel
+                      'client.ts': '',
+                    },
+                  },
+                },
+              },
+            },
+          },
+        })
+        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: [
+        {
+          message: /からimportするか、.*のbarrelファイルを削除して直接import可能にしてください/,
+        },
+      ],
+    },
+  ],
+})

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

package/tsconfig.json

@@ -1,8 +1,10 @@
 {
   "compilerOptions": {
     "paths": {
+      // ルール実装で使用
       "@/*": ["./src/*"],
-      "~/*": ["./src/*"]
+      // テスト用
+      "~/*": ["./test-fixtures/*"]
     }
   }
 }