eslint-plugin-smarthr 0.1.1 → 0.1.2

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.
 
+### [0.1.2](https://github.com/kufu/eslint-plugin-smarthr/compare/v0.1.1...v0.1.2) (2022-03-09)
+
+
+### Bug Fixes
+
+* require-barrel-import修正（barrelファイルが複数存在する場合、一番親に当たるファイルを検知する） ([#14](https://github.com/kufu/eslint-plugin-smarthr/issues/14)) ([87a6724](https://github.com/kufu/eslint-plugin-smarthr/commit/87a67240f31c9408faad6784741bbf6a2f7ef47b))
+
 ### [0.1.1](https://github.com/kufu/eslint-plugin-smarthr/compare/v0.1.0...v0.1.1) (2022-03-08)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-smarthr",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "author": "SmartHR",
   "license": "MIT",
   "description": "A sharable ESLint plugin for SmartHR",

package/rules/require-barrel-import.js

@@ -43,6 +43,7 @@ const calculateReplacedImportPath = (source) => {
     return prev
   }, source)
 }
+const TARGET_EXTS = ['ts', 'tsx', 'js', 'jsx']
 
 module.exports = {
   meta: {
@@ -55,11 +56,6 @@ module.exports = {
   create(context) {
     const filename = context.getFilename()
 
-    // HINT: indexファイルがある == barrelであるとする
-    if (filename.match(/\/index\.(js|ts)x?$/)) {
-      return {}
-    }
-
     const dir = (() => {
       const d = filename.split('/')
       d.pop()
@@ -82,39 +78,40 @@ module.exports = {
         }
 
         const sources = sourceValue.split('/')
-        let joinedSources = sourceValue
-        let ext = undefined
+
+        // HINT: directoryの場合、indexファイルからimportしていることは自明であるため、一階層上からチェックする
+        if (fs.existsSync(sourceValue) && fs.statSync(sourceValue).isDirectory()) {
+          sources.pop()
+          sourceValue = sources.join('/')
+        }
+
+        let barrel = undefined
 
         while (sources.length > 0) {
           // HINT: 以下の場合は即終了
           // - import元以下のimportだった場合
           // - rootまで捜索した場合
-          if (dir === joinedSources || dir === rootPath) {
-            return
-          }
-
-          ext = ['ts', 'tsx', 'js', 'jsx'].find((e) => fs.existsSync(`${sources.join('/')}/index.${e}`))
-
-          if (ext) {
+          if (
+            dir === rootPath ||
+            dir.match(new RegExp(`^${sourceValue}`))
+          ) {
             break
           }
 
+          barrel = TARGET_EXTS.map((e) => `${sourceValue}/index.${e}`).find((p) => fs.existsSync(p)) || barrel
+
           sources.pop()
-          joinedSources = sources.join('/')
+          sourceValue = sources.join('/')
         }
 
-        if (
-          joinedSources &&
-          sourceValue !== joinedSources &&
-          !dir.match(new RegExp(`^${joinedSources}/`))
-        ) {
-          const replacedSources = calculateReplacedImportPath(joinedSources)
+        if (barrel) {
+          barrel = calculateReplacedImportPath(barrel)
 
           context.report({
             node,
             messageId: 'require-barrel-import',
             data: {
-              message: `${replacedSources}からimportするか、${replacedSources}/index.${ext}を削除してください`,
+              message: `${barrel.replace(/\/index\.(ts|js)x?$/, '')} からimportするか、${barrel} を削除してください`,
             },
           });
         }