eslint-plugin-smarthr 6.11.0 → 6.12.1

package/.claude/settings.local.json

@@ -0,0 +1,124 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(npm test:*)",
+      "Bash(git checkout:*)",
+      "Bash(git pull:*)",
+      "Bash(git add:*)",
+      "Bash(git commit:*)",
+      "Bash(git push:*)",
+      "Bash(gh pr:*)",
+      "Bash(brew upgrade:*)",
+      "Bash(git fetch:*)",
+      "Bash(git merge:*)",
+      "Bash(git restore:*)",
+      "Bash(git reset:*)",
+      "Bash(grep -l \"node\\) =>\" rules/*/index.js)",
+      "Bash(git branch:*)",
+      "Bash(git revert:*)",
+      "Bash(column -t -s '|')",
+      "Bash(gh api:*)",
+      "Bash(grep -r \"=\\\\{[''''\"\"\\\\`][a-zA-Z0-9]*[''''\"\"\\\\`]\\\\}\" test/*.js)",
+      "Bash(grep -E \"=\\\\{[''''\"\"\\\\`]\" test/*.js)",
+      "Bash(grep \"name={\\\\`ABC\\\\`}\")",
+      "Bash(grep -nE \"=\\\\{\\\\`[^$\\\\`]*\\\\`\\\\}\" test/*.js)",
+      "Read(//Users/atsushi.mizoue/works/tamatebako/**)",
+      "Bash(pnpm install:*)",
+      "Bash(pnpm test:*)",
+      "Bash(grep -v '^master$')",
+      "Bash(grep -E \"FieldSet|BaseColumn|Message$|Fieldsets|FormControls|RadioButtons|RadioButtonPanels|Checkboxes\" test/component-name.js)",
+      "Bash(git stash:*)",
+      "Bash(grep -n \"new RegExp\" rules/*/index.js)",
+      "Bash(grep -nE \"\\\\.replace\\\\\\(\\\\/|\\\\.test\\\\\\(\\\\/|\\\\.match\\\\\\(\\\\/|\\\\.exec\\\\\\(\\\\/|\\\\.search\\\\\\(\\\\/\" rules/*/index.js)",
+      "Bash(grep -nE \"const .* = /\" rules/*/index.js)",
+      "Bash(grep -n \"Object.keys\\(\" rules/*/index.js)",
+      "Bash(grep -n \"const.*=.*Object\\\\.keys\\(\" rules/*/index.js)",
+      "Bash(grep:*)",
+      "Bash(grep -n \"\\\\.forEach\\(\" rules/*/index.js)",
+      "Bash(sort -k1,1 -k3,3)",
+      "Bash(for branch:*)",
+      "Bash(do)",
+      "Bash(echo \"=== $branch ===\")",
+      "Bash(git log:*)",
+      "Bash(done)",
+      "WebSearch",
+      "Bash(git mv:*)",
+      "Bash(node:*)",
+      "Bash(do echo:*)",
+      "Bash(echo \"Checking files...\" git ls-tree -r --name-only $branch packages/eslint-plugin-smarthr/rules/)",
+      "Bash(git ls-tree:*)",
+      "Bash(git config:*)",
+      "Bash(npm run:*)",
+      "Bash(ls rules/*/README.md)",
+      "Bash(find rules:*)",
+      "Bash(cp:*)",
+      "Bash(cd:*)",
+      "Bash(/Users/atsushi.mizoue/works/hanica/client/eslint/rules/best-practice-for-response-message.cjs:*)",
+      "Bash(/Users/atsushi.mizoue/works/hanica/client/eslint/rules/a11y-prohibit-overflow-hidden.cjs:*)",
+      "Bash(/Users/atsushi.mizoue/works/hanica/client/eslint/rules/a11y-scroller-has-tabindex.cjs:*)",
+      "Bash(__NEW_LINE_aaf112b90809b36f__ cat:*)",
+      "Bash(/Users/atsushi.mizoue/works/hanica/client/eslint/rules/best-practice-for-text-component.cjs:*)",
+      "Bash(/Users/atsushi.mizoue/works/hanica/client/eslint/rules/design-system-guideline-prohibit-dialog-button-icon.cjs:*)",
+      "Bash(__NEW_LINE_220f471abdeba11e__ cat:*)",
+      "Bash(for dir:*)",
+      "Bash(do if:*)",
+      "Read(//Users/atsushi.mizoue/works/tamatebako/packages/eslint-plugin-smarthr/$dir/**)",
+      "Bash(then echo:*)",
+      "Bash(fi)",
+      "Bash(pnpm eslint:*)",
+      "Bash(git cherry-pick:*)",
+      "Bash(git rebase:*)",
+      "Bash(GIT_SEQUENCE_EDITOR=\"sed -i '/fix: best-practice-for-text-component/d'\" git rebase -i HEAD~34)",
+      "Bash(gh search:*)",
+      "Bash(gh repo:*)",
+      "Bash(jq)",
+      "Bash(find . -name \"package.json\" -not -path \"*/node_modules/*\" -exec grep -l \"smarthr-ui\" {})",
+      "Bash(mkdir -p /Users/atsushi.mizoue/works/workflow/michi/frontend/eslint/rules/autofixer-smarthr-ui-migration)",
+      "Bash(cp -r /Users/atsushi.mizoue/works/tamatebako/packages/eslint-plugin-smarthr/rules/autofixer-smarthr-ui-migration/* /Users/atsushi.mizoue/works/workflow/michi/frontend/eslint/rules/autofixer-smarthr-ui-migration/)",
+      "Bash(pnpm link:*)",
+      "Bash(pnpm add:*)",
+      "Bash(pnpm run:*)",
+      "Bash(pnpm view:*)",
+      "Bash(pnpm exec:*)",
+      "Bash(pnpm list:*)",
+      "Bash(find /Users/atsushi.mizoue/works/workflow/michi/frontend/node_modules/.pnpm -name eslint-plugin-smarthr* -type d)",
+      "Bash(ls -la /Users/atsushi.mizoue/works/workflow/michi/frontend/node_modules/.pnpm/eslint-config-smarthr@13.10.0_*/node_modules/)",
+      "Read(//Users/atsushi.mizoue/works/workflow/**)",
+      "Bash(rm -rf pnpm-lock.yaml node_modules/.pnpm)",
+      "Bash(ls:*)",
+      "Bash(xargs curl:*)",
+      "Bash(xargs -I {} gh api repos/kufu/smarthr-ui/contents/packages/smarthr-ui/src/components/Dialog/ActionDialog/ActionDialogContentInner.tsx?ref={} --jq '.content')",
+      "Bash(npm view:*)",
+      "Bash(gh run:*)",
+      "Bash(xargs -I {} gh api repos/kufu/tamatebako/contents/packages/eslint-plugin-smarthr/CHANGELOG.md?ref={} --jq '.content')",
+      "Bash(/tmp/new-return-body.js:*)",
+      "Bash(git rm:*)",
+      "Bash(gh release:*)",
+      "Bash(ls -la /Users/atsushi.mizoue/works/workflow/kago/frontend/src/app/\\\\\\(private\\\\\\)/settings/user_roles/_components/UserRolesPage/UserRoleTable/)",
+      "Bash(ls -la \"/Users/atsushi.mizoue/works/workflow/kago/frontend/src/app/\\(private\\)/settings/user_roles/_components/UserRolesPage/UserRoleTable/DeleteUserRoleDialog/\")",
+      "Bash(tee /tmp/eslint-test-output.txt)",
+      "Bash(find /Users/atsushi.mizoue/works/workflow/kago/frontend/src -type d -name \"*\\(*\")",
+      "Bash(test -d \"/Users/atsushi.mizoue/works/workflow/kago/frontend/src/app/\\(private\\)/settings/user_roles/_components/AddUserRoleDialog/hooks/useUsers\")",
+      "Bash(pkill -f eslint)",
+      "Bash(npx eslint:*)",
+      "Bash(/tmp/pr-body.md:*)",
+      "Bash(pkill -9 -f eslint)",
+      "Bash(rm -rf .eslintcache node_modules/.cache)",
+      "Bash(pnpm remove:*)",
+      "Bash(find test:*)",
+      "Bash(git clean:*)",
+      "Bash(open \"https://github.com/kufu/tamatebako/pull/1244\")",
+      "Bash(test-multiple-barrels.tsx:*)",
+      "Bash(src/test-multiple-barrels.tsx:*)",
+      "Bash(src/components/test-multiple-barrels.tsx:*)",
+      "Bash(src/components/pages/routes/route/edit/_components/test-multiple-barrels.tsx:*)",
+      "Bash(src/app/test-both-barrels.tsx:*)",
+      "Bash(pnpm release:*)",
+      "Bash(npm whoami:*)",
+      "Bash(for pr:*)",
+      "Bash(npx lerna:*)",
+      "Bash(npm login:*)",
+      "Bash(curl -s https://raw.githubusercontent.com/kufu/smarthr-ui/main/CHANGELOG.md)"
+    ]
+  }
+}

package/CHANGELOG.md

@@ -2,6 +2,25 @@
 
 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.12.1](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v6.12.0...eslint-plugin-smarthr-v6.12.1) (2026-04-19)
+
+
+### Bug Fixes
+
+* **v91-to-v92:** button/anchorButton系変種コンポーネントのsize属性変換に対応 ([#1251](https://github.com/kufu/tamatebako/issues/1251)) ([14335be](https://github.com/kufu/tamatebako/commit/14335bef82fa38ebe04b0b16fda5e5cd06165cae))
+
+## [6.12.0](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v6.11.0...eslint-plugin-smarthr-v6.12.0) (2026-04-15)
+
+
+### Features
+
+* **require-barrel-import:** additionalBarrelFileNamesオプション追加と同階層・子階層からのバレルimport禁止 ([#1244](https://github.com/kufu/tamatebako/issues/1244)) ([b38a143](https://github.com/kufu/tamatebako/commit/b38a1432a92bd3ee4e52f3a450177e59f65512f0))
+
+
+### Bug Fixes
+
+* **format-import-path:** dirCount関数でmatch結果がnullの場合のエラーを修正 ([#1246](https://github.com/kufu/tamatebako/issues/1246)) ([fc50fba](https://github.com/kufu/tamatebako/commit/fc50fbadc3da3b046a73bbab664c281fc7be847f))
+
 ## [6.11.0](https://github.com/kufu/tamatebako/compare/eslint-plugin-smarthr-v6.10.4...eslint-plugin-smarthr-v6.11.0) (2026-04-14)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-smarthr",
-  "version": "6.11.0",
+  "version": "6.12.1",
   "author": "SmartHR",
   "license": "MIT",
   "description": "A sharable ESLint plugin for SmartHR",
@@ -34,5 +34,5 @@
     "eslintplugin",
     "smarthr"
   ],
-  "gitHead": "17b3e279b5cf8ffeef90d1eccce77101c7b68cba"
+  "gitHead": "3b8ac2fc343826a86b14d05ffc9400c417ce3814"
 }

package/rules/autofixer-smarthr-ui-migration/versions/v91-to-v92/index.js

@@ -39,18 +39,18 @@ const SIZE_VALUE_MAP = {
   m: 'M',
 }
 
-// サイズ指定を持つコンポーネント
-const SIZE_COMPONENTS = [
-  'Button',
-  'AnchorButton',
-  'Select',
-  'SegmentedControl',
-  'SideNav',
-  'SideNavItemButton',
-  'SideNavItemAnchor',
-  'InputFile',
-  'Loader',
-  'LoaderSpinner',
+// サイズ指定を持つコンポーネント（正規表現パターン）
+const SIZE_COMPONENT_PATTERNS = [
+  /Button$/, // XxxxButton (例: Button, PrimaryButton, DangerButton)
+  /AnchorButton$/, // XxxxAnchorButton (例: AnchorButton, PrimaryAnchorButton, DangerAnchorButton)
+  /^Select$/,
+  /^SegmentedControl$/,
+  /^SideNav$/,
+  /^SideNavItemButton$/,
+  /^SideNavItemAnchor$/,
+  /^InputFile$/,
+  /^Loader$/,
+  /^LoaderSpinner$/,
 ]
 
 // 3. decorators属性を持つコンポーネント
@@ -285,8 +285,9 @@ module.exports = {
       'JSXAttribute[name.name="size"][value.type="Literal"]'(node) {
         const componentName = node.parent.name.name
 
-        // 対象コンポーネントかチェック
-        if (!SIZE_COMPONENTS.includes(componentName)) return
+        // 対象コンポーネントかチェック（正規表現パターンマッチング）
+        const isTargetComponent = SIZE_COMPONENT_PATTERNS.some(pattern => pattern.test(componentName))
+        if (!isTargetComponent) return
 
         const sizeValue = node.value.value
         const newValue = SIZE_VALUE_MAP[sizeValue]

package/rules/autofixer-smarthr-ui-migration/versions/v91-to-v92/test.js

@@ -59,8 +59,7 @@ module.exports = {
     { code: `<Button />`, options: v91ToV92Options },
     { code: `<Button size={dynamicSize} />`, options: v91ToV92Options },
 
-    // 対象外のコンポーネント
-    { code: `<CustomButton size="default" />`, options: v91ToV92Options },
+    // 対象外のコンポーネント（Buttonサフィックスを持たないもの）
     { code: `<Text size="s" />`, options: v91ToV92Options },
 
     // smarthrUiAliasオプション: aliasファイル外では置換しない
@@ -176,6 +175,20 @@ const handleAction: ComponentPropsWithoutRef<typeof FormDialog>['onClickClose']
     createSizeConversionTest('Button', 's', 'S'),
     createSizeConversionTest('AnchorButton', 'default', 'M'),
 
+    // Button variants
+    createSizeConversionTest('PrimaryButton', 's', 'S'),
+    createSizeConversionTest('PrimaryButton', 'default', 'M'),
+    createSizeConversionTest('DangerButton', 's', 'S'),
+    createSizeConversionTest('SkeletonButton', 's', 'S'),
+    createSizeConversionTest('TextButton', 's', 'S'),
+
+    // AnchorButton variants
+    createSizeConversionTest('PrimaryAnchorButton', 's', 'S'),
+    createSizeConversionTest('PrimaryAnchorButton', 'default', 'M'),
+    createSizeConversionTest('DangerAnchorButton', 's', 'S'),
+    createSizeConversionTest('SkeletonAnchorButton', 's', 'S'),
+    createSizeConversionTest('TextAnchorButton', 's', 'S'),
+
     // Select
     createSizeConversionTest('Select', 's', 'S'),
     createSizeConversionTest('Select', 'm', 'M'),

package/rules/format-import-path/index.js

@@ -28,7 +28,11 @@ const DIR_SEPARATE_REGEX = /\//g
 const MULTIPLE_DIR_SEPARATE_REGEX =/(\/)+/g
 const TRAILING_SLASH_REGEX = /^(.+?)\/$/
 
-const dirCount = (dir) => dir.match(DIR_SEPARATE_REGEX).length
+const dirCount = (dir) => {
+  if (!dir) return 0
+  const matches = dir.match(DIR_SEPARATE_REGEX)
+  return matches ? matches.length : 0
+}
 
 const convertType = (calcContext, calcDomainNode) => {
   const { option: { format: { all, outside, globalModule, module, domain, lower } } } = calcContext

package/rules/require-barrel-import/README.md

@@ -24,6 +24,49 @@ barrelを経由することで、内部のファイル構成を変更しても
 
 例えば、`Page/parts/Menu/Item` を `Page/parts/Menu` から importすることで、import文がより簡潔で読みやすくなります。
 
+## 同階層・子階層からのバレルimportの禁止
+
+バレルファイル（index.ts、client.ts等）と同じディレクトリまたは子ディレクトリ内からバレルファイルをimportすることを禁止します。
+
+### なぜ禁止する必要があるのか
+
+バレルファイルは**ディレクトリ外部**へのエクスポート専用として設計されています。同じディレクトリ内や子ディレクトリからバレルファイルをimportすると、以下の問題が発生します：
+
+1. **循環参照のリスク**: バレルファイルが内部のファイルをexportし、その内部ファイルがバレルファイルをimportする循環が発生しやすい
+2. **カプセル化の崩壊**: バレルファイルは外部向けAPIであり、内部からimportすると内部実装の境界が曖昧になる
+3. **不要な依存**: 直接相対パスでimportできるものをバレル経由でimportすることは冗長
+
+### ❌ 検出されるエラーケース
+
+```typescript
+// src/components/Button/Button.tsx から同じディレクトリのバレルをimport
+import { ButtonProps } from '.'          // NG
+import { ButtonProps } from './index'    // NG
+import { ButtonProps } from '@/components/Button'  // NG
+
+// src/components/Button/_utils/helper.ts から親のバレルをimport
+import { Button } from '..'              // NG
+import { Button } from '../index'        // NG
+import { Button } from '@/components/Button'  // NG
+
+// client.ts を使用している場合
+import { ButtonPresentation } from './client'  // NG
+```
+
+### ✅ 正しいimport方法
+
+```typescript
+// 同じディレクトリ内では直接相対パスを使用
+import { ButtonProps } from './types'    // OK
+
+// 親ディレクトリのファイルも直接相対パスを使用
+import { buttonUtils } from '../utils'   // OK
+
+// バレルファイルは外部ディレクトリからのみimport
+// src/pages/HomePage.tsx から
+import { Button } from '@/components/Button'  // OK
+```
+
 ## config
 
 ### 必須設定
@@ -43,6 +86,95 @@ tsconfig.json の compilerOptions.pathsに `@/*` もしくは `~/*` としてroo
 
 特定のファイルから特定のimportを許可する設定を記述できます。
 
+### additionalBarrelFileNames
+
+`index` 以外にbarrelファイルとして扱うファイル名を配列で指定します（拡張子なし）。
+
+Next.jsなどで使用される `client.ts` をbarrelファイルとして扱いたい場合に使用します。
+
+- デフォルト: `[]`（`index.*` のみがbarrelファイル）
+- 例: `['client']` を指定すると、`client.ts`, `client.tsx` などもbarrelファイルとして扱われます
+- 複数指定も可能: `['client', 'server']` を指定すると、`server.ts`, `server.tsx` なども追加されます
+
+#### 優先順位とチェックルール
+
+**1. 同じディレクトリ内に複数のbarrelがある場合**
+- `client.ts` と `index.ts` が両方ある場合、どちらからのimportも許容されます
+- 例: `import { Foo } from './api'` も `import { Foo } from './api/client'` もOK
+
+**2. 同じファイル名の場合は親を優先**
+
+探索により同じファイル名のbarrelが複数見つかった場合（例: `client.ts`同士、`index.ts`同士）、より親のbarrelを推奨します。
+
+```typescript
+// 例: 同じファイル名の場合（index同士）
+route/
+  index.ts  ← より親を推奨
+  edit/
+    index.ts
+
+// import { Foo } from './route/edit/Foo'
+// → route/index.ts を推奨（より親のbarrel）
+```
+
+**index.ts経由のre-export対応:**
+
+子で`index.ts`を見つけた場合でも、親方向に`client.ts`があれば、そちらを優先します。
+
+```typescript
+// 子が index.ts、親が client.ts のパターン
+route/
+  client.ts  ← これを推奨
+  edit/
+    index.ts
+
+// import { Foo } from './route/edit/Foo'
+// → route/client.ts を推奨（親のclient.tsが見つかった）
+
+// この場合、route/client.ts で edit/index.ts をre-exportする想定：
+// route/client.ts
+export * from './edit'
+```
+
+**3. エラーメッセージの表示**
+
+`additionalBarrelFileNames`が設定されている場合、エラーメッセージには存在しないbarrelファイルも含めて全ての選択肢が表示されます。
+
+```typescript
+// 例: additionalBarrelFileNames: ['client'] 設定時、index.tsのみ存在
+検出されたバレル: @/components/api/index.ts
+現在のimport:      import { fetchUser } from '@/components/api/user'
+推奨されるimport（以下のいずれか）:
+  - import { fetchUser } from '@/components/api' // index.ts
+  - import { fetchUser } from '@/components/api/client' // client.ts (作成が必要)
+
+※ 存在しないバレルファイルは必要に応じて作成してください。
+```
+
+- `index.ts`が優先的に表示されます（常に最初）
+- 存在しないファイルには `(作成が必要)` マークが表示されます
+- 存在しないファイルがある場合、注意メッセージが追加されます
+- 「検出されたバレル」には実際に存在するファイルのみが表示されます
+
+**4. 階層の一貫性チェック**
+
+子ディレクトリで`client.ts`を使用している場合、親ディレクトリにも同名のbarrelを作成することを促します。これにより、プロジェクト全体でbarrel構造の一貫性を保ちます。
+
+```typescript
+// 例: client.tsパターンの一貫性チェック
+route/
+  index.ts  ← client.tsがない
+  edit/
+    client.ts  ← client.tsを使用
+
+// import { Foo } from './route/edit/client'
+// → エラー: route/client.ts を作成して、edit/client のexportをまとめてください
+
+// この場合、以下のようにroute/client.tsを作成する必要があります：
+// route/client.ts
+export * from './edit/client'
+```
+
 ## rules
 
 ```js
@@ -50,13 +182,16 @@ tsconfig.json の compilerOptions.pathsに `@/*` もしくは `~/*` としてroo
   rules: {
     'smarthr/require-barrel-import': [
       'error',
-      // ignores: ['\\/test\\/'], // 除外したいファイルの正規表現
-      // allowedImports: {
-      //   '/any/path/': { // 正規表現でチェックするファイルを指定
-      //     // import制御するファイル (相対パスを指定する場合、.eslintrc.js を基準とする)
-      //     '@/hoge/fuga': true // ['abc', 'def'] と指定すると個別に指定
-      //   }
-      // },
+      {
+        // ignores: ['\\/test\\/'], // 除外したいファイルの正規表現
+        // allowedImports: {
+        //   '/any/path/': { // 正規表現でチェックするファイルを指定
+        //     // import制御するファイル (相対パスを指定する場合、.eslintrc.js を基準とする)
+        //     '@/hoge/fuga': true // ['abc', 'def'] と指定すると個別に指定
+        //   }
+        // },
+        // additionalBarrelFileNames: ['client'], // Next.jsなどでclient.tsをbarrelとして扱う
+      }
     ],
   },
 }