eslint-plugin-smarthr 1.4.2 → 1.5.1

package/rules/best-practice-for-data-test-attribute/index.js

@@ -1,9 +1,6 @@
 const SCHEMA = []
 
-const prohibitAttributies = [
-  "data-spec",
-  "data-testid"
-]
+const PROHIBIT_ATTR_REGEX = /^(data-(spec|testid))$/
 
 /**
  * @type {import('@typescript-eslint/utils').TSESLint.RuleModule<''>}
@@ -16,12 +13,12 @@ module.exports = {
   create(context) {
     return {
       JSXAttribute: (node) => {
-        const hit = prohibitAttributies.find((attr) => attr === node.name.name)
+        const hit = node.name.name.match(PROHIBIT_ATTR_REGEX)
 
         if (hit) {
           context.report({
             node,
-            message: `テストのために要素を指定するために、${hit} 属性を利用するのではなく、他の方法で要素を指定することを検討してください。
+            message: `テストのために要素を指定するために、${hit[1]} 属性を利用するのではなく、他の方法で要素を指定することを検討してください。
  - 方法1: click_link, click_button等を利用したりすることで、利用しているテスト環境に準じた方法で要素を指定することを検討してください。
    - 参考(Testing Library): https://testing-library.com/docs/queries/about
    - 参考(Capybara): https://rubydoc.info/github/jnicklas/capybara/Capybara/Node/Finders

package/rules/best-practice-for-layouts/index.js

@@ -1,25 +1,12 @@
-const { generateTagFormatter } = require('../../libs/format_styled_components')
+const MULTI_CHILDREN_REGEX = /(Cluster|Stack)$/
 
-const MULTI_CHILDREN_EXPECTED_NAMES = {
-  'Cluster$': '(Cluster)$',
-  'Stack$': '(Stack)$',
-}
-const EXPECTED_NAMES = {
-  ...MULTI_CHILDREN_EXPECTED_NAMES,
-  'Center$': '(Center)$',
-
-}
-
-const UNEXPECTED_NAMES = EXPECTED_NAMES
-
-const MULTI_CHILDREN_REGEX = new RegExp(`(${Object.keys(MULTI_CHILDREN_EXPECTED_NAMES).join('|')})`)
 const REGEX_NLSP = /^\s*\n+\s*$/
 const FLEX_END_REGEX = /^(flex-)?end$/
 
 const filterFalsyJSXText = (cs) => cs.filter(checkFalsyJSXText)
 const checkFalsyJSXText = (c) => (
   !(
-    c.type === 'JSXText' && c.value.match(REGEX_NLSP) ||
+    c.type === 'JSXText' && REGEX_NLSP.test(c.value) ||
     c.type === 'JSXEmptyExpression'
   )
 )
@@ -63,7 +50,6 @@ module.exports = {
   },
   create(context) {
     return {
-      ...generateTagFormatter({ context, EXPECTED_NAMES, UNEXPECTED_NAMES }),
       JSXOpeningElement: (node) => {
         const nodeName = node.name.name;
 

package/rules/best-practice-for-remote-trigger-dialog/index.js

@@ -1,11 +1,4 @@
-const { generateTagFormatter } = require('../../libs/format_styled_components');
-
-const EXPECTED_NAMES = {
-  'RemoteDialogTrigger$': 'RemoteDialogTrigger$',
-  'RemoteTrigger(.+)Dialog$': 'RemoteTrigger(.+)Dialog$',
-}
-
-const REGEX_REMOTE_TRIGGER_DIALOG = /RemoteTrigger(Action|Message|Modeless)Dialog$/
+const REGEX_REMOTE_TRIGGER_DIALOG = /RemoteTrigger(Action|Form|Message|Modeless)Dialog$/
 const REGEX_REMOTE_DIALOG_TRIGGER = /RemoteDialogTrigger$/
 
 /**
@@ -18,13 +11,12 @@ module.exports = {
   },
   create(context) {
     return {
-      ...generateTagFormatter({ context, EXPECTED_NAMES }),
       JSXOpeningElement: (node) => {
         const nodeName = node.name.name || '';
 
-        const regexRemoteTriggerDialog = nodeName.match(REGEX_REMOTE_TRIGGER_DIALOG)
+        const regexRemoteTriggerDialog = REGEX_REMOTE_TRIGGER_DIALOG.test(nodeName)
 
-        if (regexRemoteTriggerDialog || nodeName.match(REGEX_REMOTE_DIALOG_TRIGGER)) {
+        if (regexRemoteTriggerDialog || REGEX_REMOTE_DIALOG_TRIGGER.test(nodeName)) {
           const attrName = regexRemoteTriggerDialog ? 'id' : 'targetId'
           const id = node.attributes.find((a) => a.name?.name === attrName)
 

package/rules/best-practice-for-tailwind-prohibit-root-margin/index.js

@@ -3,6 +3,8 @@ const { AST_NODE_TYPES } = require('@typescript-eslint/utils')
 const SCHEMA = []
 const MARGIN_CLASS_PATTERNS = /shr-m[trbl]?-/ // mt-, mr-, mb-, ml-, m-
 
+const findClassNameAttr = (attr) => attr.type === AST_NODE_TYPES.JSXAttribute && attr.name.name === 'className'
+
 /**
  * コンポーネントのルート要素を渡し、該当の余白クラスが存在すればそれを、なければNULLを返す
  * @param {import('@typescript-eslint/utils').TSESTree.Node} node
@@ -12,17 +14,18 @@ const findSpacingClassInRootElement = (node) => {
   // JSX でなければ対象外
   if (node.type !== AST_NODE_TYPES.JSXElement) return null
 
-  // className属性がなければ対象外
-  const classNameAttr = node.openingElement.attributes.find(
-    (attr) => attr.type === AST_NODE_TYPES.JSXAttribute && attr.name.name === 'className',
-  )
-  if (!classNameAttr) return null
+  const classNameAttr = node.openingElement.attributes.find(findClassNameAttr)
 
-  // className属性の値がリテラルでなければ対象外
-  if (classNameAttr?.value?.type !== AST_NODE_TYPES.Literal || typeof classNameAttr.value.value !== 'string') return null
+  if (
+    classNameAttr &&
+    // className属性の値がリテラル、かつ余白クラスの場合
+    classNameAttr.value?.type === AST_NODE_TYPES.Literal && typeof classNameAttr.value.value === 'string' &&
+    MARGIN_CLASS_PATTERNS.test(classNameAttr.value.value)
+  ) {
+    return classNameAttr.value
+  }
 
-  // className属性の値に余白クラスが含まれていればそれを返す
-  return MARGIN_CLASS_PATTERNS.test(classNameAttr.value.value) ? classNameAttr.value : null
+  return null
 }
 
 /**
@@ -56,30 +59,37 @@ module.exports = {
      * 関数本体をチェックし、ルート要素で余白クラスが設定されたJSXを返している場合、エラーを報告する
      * @param {import('@typescript-eslint/utils').TSESTree.Node} body
      */
-    const checkFunctionBody = (body) => {
-      // 関数がブロックを持たずに直接JSXを返すパターン
-      if (body.type === AST_NODE_TYPES.JSXElement) {
-        const spacingClass = findSpacingClassInRootElement(body)
-        if (spacingClass) {
-          context.report({
-            node: spacingClass,
-            messageId: 'noRootSpacing',
-          })
-        }
-        return
-      }
+    const checkFunctionBody = (n) => {
+      const body = n.body
+
+      switch (body.type) {
+        // 関数がブロックを持たずに直接JSXを返すパターン
+        case AST_NODE_TYPES.JSXElement: {
+          const spacingClass = findSpacingClassInRootElement(body)
 
-      // 関数がブロック内で JSX を return するパターン
-      if (body.type === AST_NODE_TYPES.BlockStatement) {
-        const returnStatement = findJSXReturnStatement(body)
-        if (returnStatement?.argument) {
-          const spacingClass = findSpacingClassInRootElement(returnStatement.argument)
           if (spacingClass) {
             context.report({
               node: spacingClass,
               messageId: 'noRootSpacing',
             })
           }
+
+          break
+        }
+        // 関数がブロック内で JSX を return するパターン
+        case AST_NODE_TYPES.BlockStatement: {
+          const returnStatement = findJSXReturnStatement(body)
+          if (returnStatement?.argument) {
+            const spacingClass = findSpacingClassInRootElement(returnStatement.argument)
+            if (spacingClass) {
+              context.report({
+                node: spacingClass,
+                messageId: 'noRootSpacing',
+              })
+            }
+          }
+
+          break
         }
       }
     }
@@ -88,14 +98,12 @@ module.exports = {
       // アロー関数式のコンポーネントをチェック
       ArrowFunctionExpression: (node) => {
         if (node.parent?.type === AST_NODE_TYPES.VariableDeclarator) {
-          checkFunctionBody(node.body)
+          checkFunctionBody(node)
         }
       },
 
       // function宣言のコンポーネントをチェック
-      FunctionDeclaration: (node) => {
-        checkFunctionBody(node.body)
-      },
+      FunctionDeclaration: checkFunctionBody,
     }
   },
 }

package/rules/best-practice-for-tailwind-variants/index.js

@@ -6,8 +6,6 @@ const TV_RESULT_CONST_NAME_REGEX = /(C|c)lassNameGenerator$/
 
 const findValidImportNameNode = (s) => s.type === 'ImportSpecifier' && s.imported.name === TV_COMPONENTS_METHOD && s.local.name !== TV_COMPONENTS_METHOD
 
-const checkImportTailwindVariants = (node, context) => {
-}
 const findNodeHasId = (node) => {
   if (node.id) {
     return node
@@ -42,13 +40,11 @@ module.exports = {
   create(context) {
     return {
       ImportDeclaration: (node) => {
-        if (node.source.value === TV_COMPONENTS) {
-          if (node.specifiers.some(findValidImportNameNode)) {
-            context.report({
-              node,
-              message: `${TV_COMPONENTS} をimportする際は、名称が"${TV_COMPONENTS_METHOD}" となるようにしてください。例: "import { ${TV_COMPONENTS_METHOD} } from '${TV_COMPONENTS}'"`,
-            });
-          }
+        if (node.source.value === TV_COMPONENTS && node.specifiers.some(findValidImportNameNode)) {
+          context.report({
+            node,
+            message: `${TV_COMPONENTS} をimportする際は、名称が"${TV_COMPONENTS_METHOD}" となるようにしてください。例: "import { ${TV_COMPONENTS_METHOD} } from '${TV_COMPONENTS}'"`,
+          });
         }
       },
       CallExpression: (node) => {
@@ -61,15 +57,11 @@ module.exports = {
               message: `${TV_COMPONENTS_METHOD}の実行結果を格納する変数名は "${idNode.id.name}" ではなく "${TV_RESULT_CONST_NAME_REGEX}"にmatchする名称に統一してください。`,
             });
           }
-        } else if (TV_RESULT_CONST_NAME_REGEX.test(node.callee.name)) {
-          const useMemoNode = findNodeUseMemo(node.parent)
-
-          if (!useMemoNode) {
-            context.report({
-              node,
-              message: `"${node.callee.name}" を実行する際、useMemoでラップし、メモ化してください`,
-            });
-          }
+        } else if (TV_RESULT_CONST_NAME_REGEX.test(node.callee.name) && !findNodeUseMemo(node.parent)) {
+          context.report({
+            node,
+            message: `"${node.callee.name}" を実行する際、useMemoでラップし、メモ化してください`,
+          });
         }
       },
     }

package/rules/component-name/README.md

@@ -0,0 +1,44 @@
+# smarthr/component-name
+
+- styled-componentなどでコンポーネントを作成する際の命名規則を設定するルールです
+- 特定のタグ、smarthr-uiが提供するコンポーネントなどを拡張する際、元の要素がなんであるか？がわかる名称になるようにチェックします
+- a11y系ルールなどの前提になるチェックのため、基本的にoffにすることは推奨されません
+
+## rules
+
+```js
+{
+  rules: {
+    'smarthr/component-name': 'error', // 'warn', 'off',
+  },
+}
+```
+
+## ❌ Incorrect
+
+```jsx
+// import 時のasをチェック
+import { HogeSelect as Fuga } from 'any'
+
+// selectと勘違いしてしまうような名称はNG
+const HogeSelect = styled.div
+const HogeSelect = styled(Hoge)
+
+// selectとわからないような名称はNG
+const Hoge = styled.select
+const Hoge = styled(FugaSelect)
+```
+
+## ✅ Correct
+
+```jsx
+// import 時のasをチェック
+import { HogeSelect as FugaSelect } from 'any'
+// typeを設定すれば命名チェックから除外される
+import { type HogeSelect as Fuga } from 'any'
+import type { HogeSelect as Fuga } from 'any'
+
+// 継承元がselectであることがわかるためOK
+const HogeSelect = styled.select
+const HogeSelect = styled(FugaSelect)
+```

package/rules/component-name/index.js

@@ -0,0 +1,139 @@
+const { generateTagFormatter } = require('../../libs/format_styled_components')
+
+const EXPECTED_NAMES = {
+  '(A|^a)rticle$': 'Article$',
+  '(A|^a)side$': 'Aside$',
+  '(B|^b)utton$': 'Button$',
+  '(Date|Wareki)Picker$': '(Date|Wareki)Picker$',
+  '(F|^f)ieldset$': 'Fieldset$',
+  '(F|^f)orm$': 'Form$',
+  '(Heading|^h(2|3|4|5|6))$': 'Heading$',
+  '(I|^i)nput$': 'Input$',
+  '(L|^l)abel$': 'Label$',
+  '(N|^n)av$': 'Nav$',
+  '(Ordered(.*)List|^ol)$': 'Ordered(.*)List$',
+  '(PageHeading|^h1)$': 'PageHeading$',
+  '(S|^s)ection$': 'Section$',
+  '(S|^s)elect$': 'Select$',
+  '(T|^t)extarea$': 'Textarea$',
+  'AccordionPanel$': 'AccordionPanel$',
+  'ActionDialogWithTrigger$': 'ActionDialogWithTrigger$',
+  'Anchor$': 'Anchor$',
+  'AnchorButton$': 'AnchorButton$',
+  'Base$': 'Base$',
+  'BaseColumn$': 'BaseColumn$',
+  'Center$': 'Center$',
+  'Check(B|b)ox$': 'Checkbox$',
+  'Check(B|b)ox(e)?s$': 'Checkboxes$',
+  'Cluster$': 'Cluster$',
+  'Combo(B|b)ox$': 'Combobox$',
+  'DialogTrigger$': 'DialogTrigger$',
+  'DropZone$': 'DropZone$',
+  'DropdownTrigger$': 'DropdownTrigger$',
+  'FieldSet$': 'FieldSet$',
+  'Fieldsets$': 'Fieldsets$',
+  'FilterDropdown$': 'FilterDropdown$',
+  'FormControl$': 'FormControl$',
+  'FormControls$': 'FormControls$',
+  'FormDialog$': 'FormDialog$',
+  'FormGroup$': 'FormGroup$',
+  'Icon$': 'Icon$',
+  'Image$': 'Image$',
+  'Img$': 'Img$',
+  'IndexNav$': 'IndexNav$',
+  'InputFile$': 'InputFile$',
+  'Link$': 'Link$',
+  'Message$': 'Message$',
+  'ModelessDialog$': 'ModelessDialog$',
+  'Pagination$': 'Pagination$',
+  'RadioButton$': 'RadioButton$',
+  'RadioButtonPanel$': 'RadioButtonPanel$',
+  'RadioButtonPanels$': 'RadioButtonPanels$',
+  'RadioButtons$': 'RadioButtons$',
+  'Reel$': 'Reel$',
+  'RemoteDialogTrigger$': 'RemoteDialogTrigger$',
+  'RemoteTrigger(.*)FormDialog$': 'RemoteTrigger(.*)FormDialog$',
+  'RemoteTrigger(.+)Dialog$': 'RemoteTrigger(.+)Dialog$',
+  'RightFixedNote$': 'RightFixedNote$',
+  'SearchInput$': 'SearchInput$',
+  'SegmentedControl$': 'SegmentedControl$',
+  'SideNav$': 'SideNav$',
+  'Sidebar$': 'Sidebar$',
+  'SmartHRLogo$': 'SmartHRLogo$',
+  'Stack$': 'Stack$',
+  'Switch$': 'Switch$',
+  'TabItem$': 'TabItem$',
+  'Text$': 'Text$',
+  'TimePicker$': 'TimePicker$',
+  '^(img|svg)$': '(Img|Image|Icon)$',
+  '^a$': '(Anchor|Link)$',
+}
+
+const unexpectedMessageTemplate = `{{extended}} は smarthr-ui/{{expected}} をextendすることを期待する名称になっています
+ - childrenにHeadingを含まない場合、コンポーネントの名称から"{{expected}}"を取り除いてください
+ - childrenにHeadingを含み、アウトラインの範囲を指定するためのコンポーネントならば、smarthr-ui/{{expected}}をexendしてください
+   - "styled(Xxxx)" 形式の場合、拡張元であるXxxxコンポーネントの名称の末尾に"{{expected}}"を設定し、そのコンポーネント内でsmarthr-ui/{{expected}}を利用してください`
+const UNEXPECTED_NAMES = {
+  '(Anchor|^a)$': '(Anchor)$',
+  '(A|^a)rticle$': ['(Article)$', unexpectedMessageTemplate ],
+  '(A|^a)side$': ['(Aside)$', unexpectedMessageTemplate ],
+  '(B|^b)utton$': '(Button)$',
+  '(Date|Wareki)Picker$': '((Date|Wareki)Picker)$',
+  '(F|^f)ieldset$': '(Fieldset)$',
+  '(F|^f)orm$': '(Form)$',
+  '(Heading|^h(1|2|3|4|5|6))$': '(Heading)$',
+  '(Icon|^(img|svg))$': '(Icon)$',
+  '(Image|^(img|svg))$': '(Image)$',
+  '(Img|^(img|svg))$': '(Img)$',
+  '(I|^i)nput$': '(Input)$',
+  '(Link|^a)$': '(Link)$',
+  '(L|^l)abel$': '(Label)$',
+  '(N|^n)av$': ['(Nav)$', unexpectedMessageTemplate ],
+  '(Ordered(.*)List|^ol)$': '(Ordered(.*)List)$',
+  '(S|^s)ection$': ['(Section)$', unexpectedMessageTemplate ],
+  '(S|^s)elect$': '(Select)$',
+  '(T|^t)extarea$': '(Textarea)$',
+  'Base$': '(Base)$',
+  'BaseColumn$': '(BaseColumn)$',
+  'Center$': '(Center)$',
+  'Check(B|b)ox$': '(Checkbox)$',
+  'Check(B|b)ox(e)?s$': '(Checkboxes)$',
+  'Cluster$': '(Cluster)$',
+  'Combo(B|b)ox$': '(Combobox)$',
+  'Fieldsets$': '(Fieldsets)$',
+  'FilterDropdown$': '(FilterDropdown)$',
+  'FormControl$': '(FormControl)$',
+  'FormControls$': '(FormControls)$',
+  'FormDialog$': '(FormDialog)$',
+  'FormGroup$': '(FormGroup)$',
+  'IndexNav$': '(IndexNav)$',
+  'InputFile$': '(InputFile)$',
+  'RadioButton$': '(RadioButton)$',
+  'RadioButtonPanel$': '(RadioButtonPanel)$',
+  'RadioButtonPanels$': '(RadioButtonPanels)$',
+  'RadioButtons$': '(RadioButtons)$',
+  'Reel$': '(Reel)$',
+  'RemoteTrigger(.*)FormDialog$': '(RemoteTrigger(.*)FormDialog)$',
+  'SearchInput$': '(SearchInput)$',
+  'SideNav$': '(SideNav)$',
+  'Sidebar$': '(Sidebar)$',
+  'Stack$': '(Stack)$',
+  'TimePicker$': '(TimePicker)$',
+}
+
+
+const SCHEMA = []
+
+/**
+ * @type {import('@typescript-eslint/utils').TSESLint.RuleModule<''>}
+ */
+module.exports = {
+  meta: {
+    type: 'problem',
+    schema: SCHEMA,
+  },
+  create(context) {
+    return generateTagFormatter({ context, EXPECTED_NAMES, UNEXPECTED_NAMES })
+  },
+}
+module.exports.schema = SCHEMA

package/rules/design-system-guideline-prohibit-double-icons/index.js

@@ -1,15 +1,6 @@
-const { generateTagFormatter } = require('../../libs/format_styled_components')
-
-const EXPECTED_NAMES = {
-  '(Button)$': '(Button)$',
-  '(Link)$': '(Link)$',
-}
-
-const UNEXPECTED_NAMES = EXPECTED_NAMES
-
 const SCHEMA = []
 
-const REGEX_PATTERN = new RegExp(`(${Object.keys(EXPECTED_NAMES).join('|')})`)
+const REGEX_PATTERN = /(Button|Link)$/
 
 /**
  * @type {import('@typescript-eslint/utils').TSESLint.RuleModule<''>}
@@ -21,7 +12,6 @@ module.exports = {
   },
   create(context) {
     return {
-      ...generateTagFormatter({ context, EXPECTED_NAMES, UNEXPECTED_NAMES }),
       JSXOpeningElement: (node) => {
         const nodeName = node.name.name
 

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

@@ -21,6 +21,14 @@ const SCHEMA = [
   }
 ]
 
+const CWD = process.cwd()
+const REPLACE_PATH_ENTRIES = Object.entries(replacePaths)
+const REPLACE_PATH_KEY_REGEX = new RegExp(`^(${Object.keys(replacePaths).join('|')})`)
+const DIR_SEPARATE_REGEX = /\//g
+const MULTIPLE_DIR_SEPARATE_REGEX =/(\/)+/g
+
+const dirCount = (dir) => dir.match(DIR_SEPARATE_REGEX).length
+
 const convertType = (calcContext, calcDomainNode) => {
   const { option: { format: { all, outside, globalModule, module, domain, lower } } } = calcContext
   const { isGlobalModuleImport, isModuleImport, isDomainImport, isLowerImport } = calcDomainNode
@@ -49,14 +57,14 @@ const calculateAbsoluteImportPath = ({ importPath, resolvedImportPath }) => {
     return importPath
   }
 
-  return Object.entries(replacePaths).reduce((prev, [key, values]) => {
+  return REPLACE_PATH_ENTRIES.reduce((prev, [key, values]) => {
     if (resolvedImportPath === prev) {
       return values.reduce((p, v) => {
         if (prev === p) {
-          const regexp = new RegExp(`^${path.resolve(`${process.cwd()}/${v}`)}(.+)$`)
+          const regexp = new RegExp(`^${path.resolve(`${CWD}/${v}`)}(.+)$`)
 
-          if (prev.match(regexp)) {
-            return p.replace(regexp, `${key}/$1`).replace(/(\/)+/g, '/')
+          if (regexp.test(prev)) {
+            return p.replace(regexp, `${key}/$1`).replace(MULTIPLE_DIR_SEPARATE_REGEX, '/')
           }
         }
 
@@ -71,7 +79,7 @@ const calculateRelativeImportPath = ({ importPath, filteredDirs, filteredPaths }
   // HINT: 相対パスの場合でも、余計にさかのぼっていたりする場合もあるので修正対象とする
   if (
     importPath[0] !== '.' &&
-    !importPath.match(new RegExp(`^(${Object.keys(replacePaths).join('|')})`))
+    !REPLACE_PATH_KEY_REGEX.test(importPath)
   ) {
     return importPath
   }
@@ -113,7 +121,7 @@ module.exports = {
 
               // HINT: 記述するdirの数でより近い方を選択する。
               // 相対・絶対記法が同一の場合、おそらくimport元から距離はあるため、たどりやすくするために絶対記法を選択する
-              return (absoluted.split('/').length <= relatived.split('/').length) ? absoluted : relatived
+              return (dirCount(absoluted) <= dirCount(relatived)) ? absoluted : relatived
           }
 
           return importPath

package/rules/format-translate-component/index.js

@@ -13,6 +13,8 @@ const SCHEMA = [
   }
 ]
 
+const NOOP = () => {}
+
 /**
  * @type {import('@typescript-eslint/utils').TSESLint.RuleModule<''>}
  */
@@ -23,7 +25,7 @@ module.exports = {
   },
   create(context) {
     const { componentPath, componentName, prohibitAttributies } = context.options[0]
-    let JSXAttribute = () => {}
+    let JSXAttribute = NOOP
 
     if (prohibitAttributies) {
       JSXAttribute = (node) => {

package/rules/no-import-other-domain/index.js

@@ -46,7 +46,7 @@ module.exports = {
     const calcContext = calculateDomainContext(context)
 
     // 対象外ファイル
-    if (!calcContext.isTarget || calcContext.option.ignores && calcContext.option.ignores.some((i) => !!calcContext.filename.match(new RegExp(i)))) {
+    if (!calcContext.isTarget || calcContext.option.ignores && calcContext.option.ignores.some((i) => (new RegExp(i)).test(calcContext.filename))) {
       return {}
     }
 
@@ -55,8 +55,8 @@ module.exports = {
       humanizeParentDir,
     } = calcContext
 
-    const targetPathRegexs = Object.keys(option?.allowedImports || {})
-    const targetAllowedImports = targetPathRegexs.filter((regex) => !!calcContext.filename.match(new RegExp(regex)))
+    const targetPathRegexs = option?.allowedImports ? Object.keys(option.allowedImports) : []
+    const targetAllowedImports = targetPathRegexs.filter((regex) => (new RegExp(regex)).test(calcContext.filename))
 
     return {
       ImportDeclaration: (node) => {
@@ -90,11 +90,11 @@ module.exports = {
           })
         })
 
-        if (isDenyPath && deniedModules[0] === true) {
-          return
-        }
-
-        if (!isDenyPath && deniedModules.length === 1 && deniedModules[0].length === 0) {
+        if (isDenyPath) {
+          if (deniedModules[0] === true) {
+            return
+          }
+        } else if (deniedModules.length === 1 && deniedModules[0].length === 0) {
           return
         }
 

package/rules/prohibit-file-name/index.js

@@ -19,7 +19,7 @@ module.exports = {
   },
   create(context) {
     const options = context.options[0]
-    const targetPaths = Object.keys(options).filter((regex) => !!context.filename.match(new RegExp(regex)))
+    const targetPaths = Object.keys(options).filter((regex) => (new RegExp(regex)).test(context.filename))
 
 
     if (targetPaths.length === 0) {

package/rules/prohibit-import/index.js

@@ -1,4 +1,5 @@
 const path = require('path')
+const { getParentDir } = require('../../libs/util')
 
 const SCHEMA = [{
   type: 'object',
@@ -30,6 +31,8 @@ const SCHEMA = [{
   additionalProperties: true,
 }]
 
+const CWD = process.cwd()
+
 const defaultReportMessage = (moduleName, exportName) => `${moduleName}${typeof exportName == 'string' ? `/${exportName}`: ''} は利用しないでください`
 
 /**
@@ -42,14 +45,9 @@ module.exports = {
   },
   create(context) {
     const options = context.options[0]
-    const parentDir = (() => {
-      const dir = context.filename.match(/^(.+?)\..+?$/)[1].split('/')
-      dir.pop()
-
-      return dir.join('/')
-    })()
+    const parentDir = getParentDir(context.filename)
     const targetPathRegexs = Object.keys(options)
-    const targetProhibits = targetPathRegexs.filter((regex) => !!context.filename.match(new RegExp(regex)))
+    const targetProhibits = targetPathRegexs.filter((regex) => (new RegExp(regex)).test(context.filename))
 
     if (targetProhibits.length === 0) {
       return {}
@@ -63,32 +61,32 @@ module.exports = {
 
           targetModules.forEach((targetModule) => {
             const { imported, reportMessage } = Object.assign({imported: true}, option[targetModule])
-            const actualTarget = targetModule[0] !== '.' ? targetModule : path.resolve(`${process.cwd()}/${targetModule}`)
+            const actualTarget = targetModule[0] !== '.' ? targetModule : path.resolve(`${CWD}/${targetModule}`)
             let sourceValue = node.source.value
 
             if (actualTarget[0] === '/') {
               sourceValue = path.resolve(`${parentDir}/${sourceValue}`)
             }
 
-            if (actualTarget !== sourceValue) {
-              return
-            }
+            if (actualTarget === sourceValue) {
+              let useImported = false
 
-            const useImported = (() => {
               if (!Array.isArray(imported)) {
-                return !!imported
-              }
+                useImported = !!imported
+              } else {
+                const specifier = node.specifiers.find((s) => s.imported && imported.includes(s.imported.name))
 
-              const specifier = node.specifiers.find((s) => s.imported && imported.includes(s.imported.name))
-
-              return specifier ? specifier.imported.name : false
-            })()
+                if (specifier) {
+                  useImported = specifier.imported.name
+                }
+              }
 
-            if (useImported) {
-              context.report({
-                node,
-                message: reportMessage ? reportMessage.replaceAll('{{module}}', node.source.value).replaceAll('{{export}}', useImported) : defaultReportMessage(node.source.value, useImported)
-              });
+              if (useImported) {
+                context.report({
+                  node,
+                  message: reportMessage ? reportMessage.replaceAll('{{module}}', node.source.value).replaceAll('{{export}}', useImported) : defaultReportMessage(node.source.value, useImported)
+                });
+              }
             }
           })
         })

package/rules/prohibit-path-within-template-literal/index.js

@@ -13,7 +13,7 @@ const recursiveFetchName = (obj, chained = '') => {
 const recursiveFetchRootNameIsPath = (obj, regex) => {
   const [name, chained] = recursiveFetchName(obj, '')
 
-  return name.match(regex) ? chained : null
+  return regex.test(name) ? chained : null
 }
 
 const SCHEMA = [