eslint-plugin-smarthr 1.4.1 → 1.5.0

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 = [

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

@@ -47,10 +47,8 @@ const calculateAbsoluteImportPath = (source) => {
       const regexp = new RegExp(`^${key}(.+)$`)
 
       return values.reduce((p, v) => {
-        if (prev === p) {
-          if (prev.match(regexp)) {
-            return p.replace(regexp, `${path.resolve(`${CWD}/${v}`)}/$1`)
-          }
+        if (prev === p && regexp.test(regexp)) {
+          return p.replace(regexp, `${path.resolve(`${CWD}/${v}`)}/$1`)
         }
 
         return p
@@ -67,7 +65,7 @@ const calculateReplacedImportPath = (source) => {
         if (prev === p) {
           const regexp = new RegExp(`^${path.resolve(`${CWD}/${v}`)}(.+)$`)
 
-          if (prev.match(regexp)) {
+          if (regexp.test(prev)) {
             return p.replace(regexp, `${key}/$1`).replace(REGEX_UNNECESSARY_SLASH, '/')
           }
         }
@@ -94,15 +92,15 @@ module.exports = {
   create(context) {
     const option = context.options[0] || {}
 
-    if (option.ignores && option.ignores.some((i) => !!context.filename.match(new RegExp(i)))) {
+    if (option.ignores && option.ignores.some((i) => (new RegExp(i)).test(context.filename))) {
       return {}
     }
 
     let d = context.filename.split('/')
     d.pop()
     const dir = d.join('/')
-    const targetPathRegexs = Object.keys(option?.allowedImports || {})
-    const targetAllowedImports = targetPathRegexs.filter((regex) => !!context.filename.match(new RegExp(regex)))
+    const targetPathRegexs = option?.allowedImports ? Object.keys(option.allowedImports) || []
+    const targetAllowedImports = targetPathRegexs.filter((regex) => (new RegExp(regex)).test(context.filename))
 
     return {
       ImportDeclaration: (node) => {

package/rules/require-declaration/index.js

@@ -33,6 +33,8 @@ const SCHEMA = [
   },
 ]
 
+const DECLARATION_REGEX = /Declaration$/
+
 const find = (type, ds, rd) => ds.find((d) => d.type === type && d.id.name === rd)
 const codeSeparator = '[^a-zA-Z0-1_$]'
 const useRegex = (use) => {
@@ -51,7 +53,7 @@ module.exports = {
   create(context) {
     const options = context.options[0]
     const targetPathRegexs = Object.keys(options)
-    const targetRequires = targetPathRegexs.filter((regex) => !!context.filename.match(new RegExp(regex)))
+    const targetRequires = targetPathRegexs.filter((regex) => (new RegExp(regex)).test(context.filename))
 
     if (targetRequires.length === 0) {
       return {}
@@ -59,7 +61,7 @@ module.exports = {
 
     return {
       Program: (node) => {
-        const declarations = node.body.filter((i) => i.type.match(/Declaration$/)).map((d) => d.declaration || d)
+        const declarations = node.body.filter((i) => DECLARATION_REGEX.test(i.type)).map((d) => d.declaration || d)
 
         if (declarations.length === 0) {
           return
@@ -108,7 +110,7 @@ module.exports = {
               let reported = false
 
               localOption.use.forEach((u) => {
-                if (!code.match(useRegex(u)) && (!localOption.reportMessage || !reported)) {
+                if (!useRegex(u).test(code) && (!localOption.reportMessage || !reported)) {
                   context.report({
                     node: hit,
                     message: localOption.reportMessage || `${localOption.type} ${requireDeclaration} では ${u} を利用してください`,

package/rules/require-export/index.js

@@ -13,11 +13,7 @@ const SCHEMA = [{
 const fetchEdgeDeclaration = (node) => {
   const { declaration } = node
 
-  if (!declaration) {
-    return node
-  }
-
-  return fetchEdgeDeclaration(declaration)
+  return declaration ? fetchEdgeDeclaration(declaration) : node
 }
 
 /**
@@ -31,7 +27,7 @@ module.exports = {
   create(context) {
     const options = context.options[0]
     const targetPathRegexs = Object.keys(options)
-    const targetRequires = targetPathRegexs.filter((regex) => !!context.filename.match(new RegExp(regex)))
+    const targetRequires = targetPathRegexs.filter((regex) => (new RegExp(regex)).test(context.filename))
 
     if (targetRequires.length === 0) {
       return {}
@@ -42,35 +38,43 @@ module.exports = {
         targetRequires.forEach((requireKey) => {
           const option = options[requireKey]
           let existDefault = false
-          const exports =
-            node.body
-              .filter((i) => {
-                if (i.type == 'ExportDefaultDeclaration') {
-                  existDefault = true
 
-                  return false
-                }
+          const exports = []
+
+          for (let i of node.body) {
+            if (i.type == 'ExportDefaultDeclaration') {
+              existDefault = true
 
-                return i.type == 'ExportNamedDeclaration'
-              })
-              .map((i) => {
-                const declaration = fetchEdgeDeclaration(i)
+              continue
+            }
 
-                if (declaration.id) {
-                  return declaration.id.name
-                }
-                if (declaration.specifiers) {
-                  return declaration.specifiers.map((s) => s.exported.name)
-                }
-                if (declaration.declarations) {
-                  return declaration.declarations.map((d) => d.id.name || d.id.properties.map((p) => p.key.name))
-                }
+            if (i.type === 'ExportNamedDeclaration') {
+              const declaration = fetchEdgeDeclaration(i)
 
-                return declaration
-              })
-              .flat(2)
+              if (declaration.id) {
+                exports.push(declaration.id.name)
+              }
+              if (declaration.specifiers) {
+                declaration.specifiers.forEach((s) => {
+                  exports.push(s.exported.name)
+                })
+              }
+              if (declaration.declarations) {
+                declaration.declarations.forEach((d) => {
+                  if (d.id.name) {
+                    exports.push(d.id.name)
+                  } else {
+                    d.id.properties.forEach((p) => {
+                      exports.push(p.key.name)
+                    })
+                  }
+                })
+              }
+            }
+          }
 
-          let notExistsExports = [...(!existDefault && option.includes('default') ? ['default'] : []), ...option.filter((o) => o !== 'default' && !exports.includes(o))]
+          const exportsRegex = new RegExp(`^(default|${exports.join('|')})$`)
+          const notExistsExports = (!existDefault && option.includes('default') ? ['default'] : []).concat(option.filter((o) => !exportsRegex.test(o)))
 
           if (notExistsExports.length) {
             context.report({

package/rules/require-import/index.js

@@ -1,4 +1,6 @@
 const path = require('path')
+const { getParentDir } = require('../../libs/util')
+
 const SCHEMA = [{
   type: 'object',
   patternProperties: {
@@ -31,6 +33,7 @@ const SCHEMA = [{
 }]
 
 const defaultReportMessage = (moduleName, exportName) => `${moduleName}${typeof exportName == 'string' ? `/${exportName}`: ''} をimportしてください`
+const filterImportDeclaration = (item) => item.type === 'ImportDeclaration'
 
 /**
  * @type {import('@typescript-eslint/utils').TSESLint.RuleModule<''>}
@@ -43,21 +46,18 @@ module.exports = {
   create(context) {
     const options = context.options[0]
     const targetPathRegexs = Object.keys(options)
-    const targetRequires = targetPathRegexs.filter((regex) => !!context.filename.match(new RegExp(regex)))
+    const targetRequires = targetPathRegexs.filter((regex) => (new RegExp(regex)).test(context.filename))
 
     if (targetRequires.length === 0) {
       return {}
     }
 
+    const CWD = process.cwd()
+
     return {
       Program: (node) => {
-        const importDeclarations = node.body.filter((item) => item.type === 'ImportDeclaration')
-        const parentDir = (() => {
-          const dir = context.filename.match(/^(.+?)\..+?$/)[1].split('/')
-          dir.pop()
-
-          return dir.join('/')
-        })()
+        const importDeclarations = node.body.filter(filterImportDeclaration)
+        const parentDir = getParentDir(context.filename)
 
         targetRequires.forEach((requireKey) => {
           const option = options[requireKey]
@@ -65,11 +65,11 @@ module.exports = {
           Object.keys(option).forEach((targetModule) => {
             const { imported, reportMessage, targetRegex } = Object.assign({imported: true}, option[targetModule])
 
-            if (targetRegex && !context.filename.match(new RegExp(targetRegex))) {
+            if (targetRegex && !(new RegExp(targetRegex)).test(context.filename)) {
               return
             }
 
-            const actualTarget = targetModule[0] !== '.' ? targetModule : path.resolve(`${process.cwd()}/${targetModule}`)
+            const actualTarget = targetModule[0] !== '.' ? targetModule : path.resolve(`${CWD}/${targetModule}`)
             const importDeclaration = importDeclarations.find(
               actualTarget[0] !== '/' ? (
                 (id) => id.source.value === actualTarget

package/rules/trim-props/index.js

@@ -1,22 +1,23 @@
+const SCHEMA = []
+const EXTRA_SPACE_REGES = /(^\s+|\s+$)/
+
 /**
  * @type {import('@typescript-eslint/utils').TSESLint.RuleModule<''>}
  */
 module.exports = {
   meta: {
     type: 'suggestion',
-    schema: [],
+    schema: SCHEMA,
     fixable: 'whitespace',
   },
   create(context) {
     return {
       JSXOpeningElement: (node) =>
-        node.attributes.reduce((prev, current) => {
+        node.attributes.forEach((current) => {
           const attribute = current.value?.type === 'JSXExpressionContainer' ? current.value.expression : current.value
           const props = attribute?.value
 
-          if (typeof props !== 'string') return prev
-
-          if (props.match(/(^\s+|\s+$)/)) {
+          if (typeof props === 'string' && EXTRA_SPACE_REGES.test(props)) {
             return context.report({
               node,
               loc: current.loc,
@@ -26,9 +27,9 @@ module.exports = {
               },
             })
           }
-          return prev
-        }, []),
+        })
     }
   },
 }
-module.exports.schema = []
+
+module.exports.schema = SCHEMA

package/test/a11y-anchor-has-href-attribute.js

@@ -20,15 +20,6 @@ const generateErrorText = (name) => `${name} に href 属性を正しく設定
 
 ruleTester.run('a11y-anchor-has-href-attribute', rule, {
   valid: [
-    { code: `import styled from 'styled-components'` },
-    { code: `import styled, { css } from 'styled-components'` },
-    { code: `import { css } from 'styled-components'` },
-    { code: `import { HogeAnchor as FugaAnchor } from './hoge'` },
-    { code: `import { Link as FugaLink } from './hoge'` },
-    { code: 'const HogeAnchor = styled.a``' },
-    { code: 'const HogeLink = styled.a``' },
-    { code: 'const HogeAnchor = styled(Anchor)``' },
-    { code: 'const HogeLink = styled(Link)``' },
     { code: `<a href="hoge">ほげ</a>` },
     { code: `<a href={hoge}>ほげ</a>` },
     { code: `<a href={undefined}>ほげ</a>` },
@@ -38,26 +29,6 @@ ruleTester.run('a11y-anchor-has-href-attribute', rule, {
     { code: '<AnyAnchor {...args1} />', options: [{ checkType: 'allow-spread-attributes' }] },
   ],
   invalid: [
-    { code: `import hoge from 'styled-components'`, errors: [ { message: `styled-components をimportする際は、名称が"styled" となるようにしてください。例: "import styled from 'styled-components'"` } ] },
-    { code: `import { Anchor as AnchorHoge } from './hoge'`, errors: [ { message: `AnchorHogeを正規表現 "/Anchor$/" がmatchする名称に変更してください。
- - Anchorが型の場合、'import type { Anchor as AnchorHoge }' もしくは 'import { type Anchor as AnchorHoge }' のように明示的に型であることを宣言してください。名称変更が不要になります` } ] },
-    { code: `import { HogeLink as HogeLinkFuga } from './hoge'`, errors: [ { message: `HogeLinkFugaを正規表現 "/Link$/" がmatchする名称に変更してください。
- - HogeLinkが型の場合、'import type { HogeLink as HogeLinkFuga }' もしくは 'import { type HogeLink as HogeLinkFuga }' のように明示的に型であることを宣言してください。名称変更が不要になります` } ] },
-    { code: 'const Hoge = styled.a``', errors: [ { message: `Hogeを正規表現 "/(Anchor|Link)$/" がmatchする名称に変更してください。` } ] },
-    { code: 'const Hoge = styled(Anchor)``', errors: [ { message: `Hogeを正規表現 "/Anchor$/" がmatchする名称に変更してください。` } ] },
-    { code: 'const Hoge = styled(Link)``', errors: [ { message: `Hogeを正規表現 "/Link$/" がmatchする名称に変更してください。` } ] },
-    { code: 'const FugaAnchor = styled.div``', errors: [ { message: `FugaAnchor は /(Anchor|^a)$/ にmatchする名前のコンポーネントを拡張することを期待している名称になっています
- - FugaAnchor の名称の末尾が"Anchor" という文字列ではない状態にしつつ、"div"を継承していることをわかる名称に変更してください
- - もしくは"div"を"FugaAnchor"の継承元であることがわかるような適切なタグや別コンポーネントに差し替えてください
-   - 修正例1: const FugaXxxx = styled.div
-   - 修正例2: const FugaAnchorXxxx = styled.div
-   - 修正例3: const FugaAnchor = styled(XxxxAnchor)` } ] },
-    { code: 'const FugaLink = styled.p``', errors: [ { message: `FugaLink は /(Link|^a)$/ にmatchする名前のコンポーネントを拡張することを期待している名称になっています
- - FugaLink の名称の末尾が"Link" という文字列ではない状態にしつつ、"p"を継承していることをわかる名称に変更してください
- - もしくは"p"を"FugaLink"の継承元であることがわかるような適切なタグや別コンポーネントに差し替えてください
-   - 修正例1: const FugaXxxx = styled.p
-   - 修正例2: const FugaLinkXxxx = styled.p
-   - 修正例3: const FugaLink = styled(XxxxLink)` } ] },
     { code: `<a></a>`, errors: [{ message: generateErrorText('a') }] },
     { code: `<a>hoge</a>`, errors: [{ message: generateErrorText('a') }] },
     { code: `<Anchor>hoge</Anchor>`, errors: [{ message: generateErrorText('Anchor') }] },

package/test/a11y-clickable-element-has-text.js

@@ -17,28 +17,6 @@ const defaultErrorMessage = `a, buttonなどのクリッカブルな要素内に
 
 ruleTester.run('a11y-clickable-element-has-text', rule, {
   valid: [
-    { code: `import styled from 'styled-components'` },
-    { code: `import styled, { css } from 'styled-components'` },
-    { code: `import { css } from 'styled-components'` },
-    { code: `import { SmartHRLogo as HogeSmartHRLogo } from './hoge'` },
-    { code: `import { AbcButton as StyledAbcButton } from './hoge'` },
-    { code: `import { HogeAnchor as FugaAnchor } from './hoge'` },
-    { code: `import { Link as FugaLink } from './hoge'` },
-    { code: `import { FugaText as HogeFugaText } from './hoge'` },
-    { code: `import { FugaMessage as HogeFugaMessage } from './hoge'` },
-    { code: 'const HogeAnchor = styled.a``' },
-    { code: 'const HogeLink = styled.a``' },
-    { code: 'const HogeButton = styled.button``' },
-    { code: 'const HogeAnchor = styled(Anchor)``' },
-    { code: 'const HogeLink = styled(Link)``' },
-    { code: 'const HogeButton = styled(Button)``' },
-    { code: 'const FugaAnchor = styled(HogeAnchor)``' },
-    { code: 'const FugaSmartHRLogo = styled(SmartHRLogo)``' },
-    { code: 'const HogeAnchor = styled.a(() => ``)' },
-    { code: 'const HogeAnchor = styled("a")(() => ``)' },
-    { code: 'const HogeAnchor = styled(Anchor)(() => ``)' },
-    { code: 'const FugaText = styled(HogeText)(() => ``)' },
-    { code: 'const FugaMessage = styled(HogeMessage)(() => ``)' },
     {
       code: `<a>ほげ</a>`,
     },
@@ -128,50 +106,6 @@ ruleTester.run('a11y-clickable-element-has-text', rule, {
     },
   ],
   invalid: [
-    { code: `import hoge from 'styled-components'`, errors: [ { message: `styled-components をimportする際は、名称が"styled" となるようにしてください。例: "import styled from 'styled-components'"` } ] },
-    { code: `import { SmartHRLogo as SmartHRLogoHoge } from './hoge'`, errors: [ { message: `SmartHRLogoHogeを正規表現 "/SmartHRLogo$/" がmatchする名称に変更してください。
- - SmartHRLogoが型の場合、'import type { SmartHRLogo as SmartHRLogoHoge }' もしくは 'import { type SmartHRLogo as SmartHRLogoHoge }' のように明示的に型であることを宣言してください。名称変更が不要になります` } ] },
-    { code: `import { AbcButton as AbcButtonFuga } from './hoge'`, errors: [ { message: `AbcButtonFugaを正規表現 "/Button$/" がmatchする名称に変更してください。
- - AbcButtonが型の場合、'import type { AbcButton as AbcButtonFuga }' もしくは 'import { type AbcButton as AbcButtonFuga }' のように明示的に型であることを宣言してください。名称変更が不要になります` } ] },
-    { code: `import { Anchor as AnchorHoge } from './hoge'`, errors: [ { message: `AnchorHogeを正規表現 "/Anchor$/" がmatchする名称に変更してください。
- - Anchorが型の場合、'import type { Anchor as AnchorHoge }' もしくは 'import { type Anchor as AnchorHoge }' のように明示的に型であることを宣言してください。名称変更が不要になります` } ] },
-    { code: `import { HogeLink as HogeLinkFuga } from './hoge'`, errors: [ { message: `HogeLinkFugaを正規表現 "/Link$/" がmatchする名称に変更してください。
- - HogeLinkが型の場合、'import type { HogeLink as HogeLinkFuga }' もしくは 'import { type HogeLink as HogeLinkFuga }' のように明示的に型であることを宣言してください。名称変更が不要になります` } ] },
-    { code: `import { FugaText as FugaTextFuga } from './hoge'`, errors: [ { message: `FugaTextFugaを正規表現 "/Text$/" がmatchする名称に変更してください。
- - FugaTextが型の場合、'import type { FugaText as FugaTextFuga }' もしくは 'import { type FugaText as FugaTextFuga }' のように明示的に型であることを宣言してください。名称変更が不要になります` } ] },
-    { code: `import { FugaMessage as FugaMessageFuga } from './hoge'`, errors: [ { message: `FugaMessageFugaを正規表現 "/Message$/" がmatchする名称に変更してください。
- - FugaMessageが型の場合、'import type { FugaMessage as FugaMessageFuga }' もしくは 'import { type FugaMessage as FugaMessageFuga }' のように明示的に型であることを宣言してください。名称変更が不要になります` } ] },
-    { code: 'const Hoge = styled.a``', errors: [ { message: `Hogeを正規表現 "/(Anchor|Link)$/" がmatchする名称に変更してください。` } ] },
-    { code: 'const Hoge = styled.button``', errors: [ { message: `Hogeを正規表現 "/Button$/" がmatchする名称に変更してください。` } ]  },
-    { code: 'const Hoge = styled(Anchor)``', errors: [ { message: `Hogeを正規表現 "/Anchor$/" がmatchする名称に変更してください。` } ]  },
-    { code: 'const Hoge = styled(Link)``', errors: [ { message: `Hogeを正規表現 "/Link$/" がmatchする名称に変更してください。` } ]  },
-    { code: 'const Hoge = styled(Button)``', errors: [ { message: `Hogeを正規表現 "/Button$/" がmatchする名称に変更してください。` } ]  },
-    { code: 'const Fuga = styled(HogeAnchor)``', errors: [ { message: `Fugaを正規表現 "/Anchor$/" がmatchする名称に変更してください。` } ]  },
-    { code: 'const Fuga = styled(SmartHRLogo)``', errors: [ { message: `Fugaを正規表現 "/SmartHRLogo$/" がmatchする名称に変更してください。` } ]  },
-    { code: 'const Piyo = styled.a(() => ``)', errors: [ { message: `Piyoを正規表現 "/(Anchor|Link)$/" がmatchする名称に変更してください。` } ] },
-    { code: 'const Piyo = styled("a")(() => ``)', errors: [ { message: `Piyoを正規表現 "/(Anchor|Link)$/" がmatchする名称に変更してください。` } ] },
-    { code: 'const Piyo = styled("a")``', errors: [ { message: `Piyoを正規表現 "/(Anchor|Link)$/" がmatchする名称に変更してください。` } ] },
-    { code: 'const Piyo = styled(Anchor)(() => ``)', errors: [ { message: `Piyoを正規表現 "/Anchor$/" がmatchする名称に変更してください。` } ] },
-    { code: 'const Hoge = styled(Text)``', errors: [ { message: `Hogeを正規表現 "/Text$/" がmatchする名称に変更してください。` } ]  },
-    { code: 'const Hoge = styled(HogeMessage)``', errors: [ { message: `Hogeを正規表現 "/Message$/" がmatchする名称に変更してください。` } ]  },
-    { code: 'const StyledButton = styled.div``', errors: [ { message: `StyledButton は /(B|^b)utton$/ にmatchする名前のコンポーネントを拡張することを期待している名称になっています
- - StyledButton の名称の末尾が"Button" という文字列ではない状態にしつつ、"div"を継承していることをわかる名称に変更してください
- - もしくは"div"を"StyledButton"の継承元であることがわかるような適切なタグや別コンポーネントに差し替えてください
-   - 修正例1: const StyledXxxx = styled.div
-   - 修正例2: const StyledButtonXxxx = styled.div
-   - 修正例3: const StyledButton = styled(XxxxButton)` } ]  },
-    { code: 'const HogeAnchor = styled(Fuga)``', errors: [ { message: `HogeAnchor は /(Anchor|^a)$/ にmatchする名前のコンポーネントを拡張することを期待している名称になっています
- - HogeAnchor の名称の末尾が"Anchor" という文字列ではない状態にしつつ、"Fuga"を継承していることをわかる名称に変更してください
- - もしくは"Fuga"を"HogeAnchor"の継承元であることがわかるような名称に変更するか、適切な別コンポーネントに差し替えてください
-   - 修正例1: const HogeXxxx = styled(Fuga)
-   - 修正例2: const HogeAnchorXxxx = styled(Fuga)
-   - 修正例3: const HogeAnchor = styled(XxxxAnchor)` } ]  },
-    { code: 'const HogeLink = styled.p``', errors: [ { message: `HogeLink は /(Link|^a)$/ にmatchする名前のコンポーネントを拡張することを期待している名称になっています
- - HogeLink の名称の末尾が"Link" という文字列ではない状態にしつつ、"p"を継承していることをわかる名称に変更してください
- - もしくは"p"を"HogeLink"の継承元であることがわかるような適切なタグや別コンポーネントに差し替えてください
-   - 修正例1: const HogeXxxx = styled.p
-   - 修正例2: const HogeLinkXxxx = styled.p
-   - 修正例3: const HogeLink = styled(XxxxLink)` } ]  },
     {
       code: `<a><img src="hoge.jpg" /></a>`,
       errors: [{ message: defaultErrorMessage }]

package/test/{a11y-delegate-element-has-role-presantation.js → a11y-delegate-element-has-role-presentation.js}

@@ -11,7 +11,8 @@ const ruleTester = new RuleTester({
   },
 })
 
-const defaultInteractiveRegex = '/((i|I)nput$|(t|T)extarea$|(s|S)elect$|InputFile$|RadioButtonPanel$|Check(b|B)ox$|Combo(b|B)ox$|(Date|Wareki)Picker$|TimePicker$|DropZone$|Switch$|SegmentedControl$|RightFixedNote$|FieldSet$|Fieldset$|FormControl$|FormGroup$|(b|B)utton$|Anchor$|Link$|TabItem$|^a$|(f|F)orm$|ActionDialogWithTrigger$|RemoteDialogTrigger$|RemoteTrigger(.+)Dialog$|FormDialog$|Pagination$|SideNav$|AccordionPanel$|FilterDropdown$)/'
+const defaultInteractiveMessage = '((B|b)utton|(Check|Combo)(B|b)ox|(Date(timeLocal)?|Time|Month|Wareki)Picker|(I|i)nput(File)?|(S|s)elect|(T|t)extarea|(ActionDialogWith|RemoteDialog)Trigger|AccordionPanel|^a|Anchor|Link|DropZone|Field(S|s)et|FilterDropdown|(F|f)orm(Control|Group|Dialog)?|Pagination|RadioButton(Panel)?|RemoteTrigger(.+)Dialog|RightFixedNote|SegmentedControl|SideNav|Switch|TabItem)$'
+const defaultInteractiveRegex = `/(${defaultInteractiveMessage})/`
 const messageNonInteractiveEventHandler = (nodeName, onAttrs, interactiveComponentRegex = defaultInteractiveRegex) => {
   const onAttrsText = onAttrs.join(', ')
 
@@ -66,7 +67,7 @@ ruleTester.run('smarthr/a11y-delegate-element-has-role-presentation', rule, {
     { code: '<div onClick={any} onSubmit={any2} role="presentation"><Hoge /></div>', errors: [ { message: messageRolePresentationNotHasInteractive('div', ['onClick', 'onSubmit']) } ] },
     { code: '<div onClick={any}><Link /></div>', errors: [ { message: messageNonInteractiveEventHandler('div', ['onClick']) } ] },
     { code: '<Wrapper onClick={any}><Link /></Wrapper>', errors: [ { message: messageNonInteractiveEventHandler('Wrapper', ['onClick']) } ] },
-    { code: '<Wrapper onSubmit={any}><Hoge /></Wrapper>', options: [{ additionalInteractiveComponentRegex: ['^Hoge$'] }], errors: [ { message: messageNonInteractiveEventHandler('Wrapper', ['onSubmit'], '/((i|I)nput$|(t|T)extarea$|(s|S)elect$|InputFile$|RadioButtonPanel$|Check(b|B)ox$|Combo(b|B)ox$|(Date|Wareki)Picker$|TimePicker$|DropZone$|Switch$|SegmentedControl$|RightFixedNote$|FieldSet$|Fieldset$|FormControl$|FormGroup$|(b|B)utton$|Anchor$|Link$|TabItem$|^a$|(f|F)orm$|ActionDialogWithTrigger$|RemoteDialogTrigger$|RemoteTrigger(.+)Dialog$|FormDialog$|Pagination$|SideNav$|AccordionPanel$|FilterDropdown$|^Hoge$)/') } ] },
+    { code: '<Wrapper onSubmit={any}><Hoge /></Wrapper>', options: [{ additionalInteractiveComponentRegex: ['^Hoge$'] }], errors: [ { message: messageNonInteractiveEventHandler('Wrapper', ['onSubmit'], `/(${defaultInteractiveMessage}|^Hoge$)/`) } ] },
     { code: '<Wrapper onClick={any} onChange={anyany}><any><Link /></any></Wrapper>', errors: [ { message: messageNonInteractiveEventHandler('Wrapper', ['onClick', 'onChange']) } ] },
     { code: '<Wrapper onClick={any}>{any ? null : (hoge ? <AnyLink /> : null)}</Wrapper>', errors: [ { message: messageNonInteractiveEventHandler('Wrapper', ['onClick']) } ] },
   ],

package/test/a11y-form-control-in-form.js

@@ -14,7 +14,7 @@ const generateErrorText = (elementName) => `${elementName}をform要素で囲む
  - form要素で囲むことでスクリーンリーダーに入力フォームであることが正しく伝わる、入力要素にfocusした状態でEnterを押せばsubmitできる、inputのpattern属性を利用できるなどのメリットがあります
  - 以下のいずれかの方法で修正をおこなってください
    - 方法1: form要素で ${elementName} を囲んでください。smarthr-ui/ActionDialog、もしくはsmarthr-ui/RemoteTriggerActionDialogを利用している場合、smarthr-ui/FormDialog、smarthr-ui/RemoteTriggerFormDialogに置き換えてください
-   - 方法2: ${elementName} がコンポーネント内の一要素であり、かつその親コンポーネントがFormControl、もしくはFieldsetを表現するものである場合、親コンポーネント名を "((Fieldset)$|(Fieldsets)$|(FormGroup)$|(FormControl)$|(FormControls)$)" とマッチするものに変更してください`
+   - 方法2: ${elementName} がコンポーネント内の一要素であり、かつその親コンポーネントがFormControl、もしくはFieldsetを表現するものである場合、親コンポーネント名を "/(Fieldset|Form(Group|Control))(s)?$/" とマッチするものに変更してください`
 
 ruleTester.run('a11y-form-control-in-form', rule, {
   valid: [