eslint-plugin-smarthr 0.2.0 → 0.2.1

package/rules/prohibit-import/README.md

@@ -0,0 +1,44 @@
+# smarthr/prohibit-import
+
+- 例えば特定の module にバグが発見されたなど、importさせたくない場合に利用するルールです
+
+## rules
+
+```js
+{
+  rules: {
+    'smarthr/prohibit-import': [
+      'error', // 'warn', 'off'
+      {
+        '^.+$': {
+          'smarthr-ui': {
+            imported: ['SecondaryButtonAnchor'],
+            reportMessage: `{{module}}/{{export}} はXxxxxxなので利用せず yyyy/zzzz を利用してください`
+          }, 
+        }
+        '\/pages\/views\/': {
+          'query-string': {
+            imported: true,
+          },
+        },
+      }
+    ]
+  },
+}
+```
+
+## ❌ Incorrect
+
+```js
+// src/pages/views/Page.tsx
+import queryString from 'query-string'
+import { SecondaryButtonAnchor } from 'smarthr-ui'
+```
+
+## ✅ Correct
+
+
+```js
+// src/pages/views/Page.tsx
+import { PrimaryButton, SecondaryButton } from 'smarthr-ui'
+```

package/rules/redundant-name/README.md

@@ -0,0 +1,94 @@
+# smarthr/redundant-name
+
+- ファイル、コードの冗長な部分を取り除くことを提案するruleです
+- ファイルが設置されているディレクトリ構造からキーワードを生成し、取り除く文字列を生成します
+
+## config
+
+- tsconfig.json の compilerOptions.pathsに '@/*' としてroot path を指定する必要があります
+- 以下の設定を行えます。全て省略可能です。
+  - ignoreKeywords
+    - ディレクトリ名から生成されるキーワードに含めたくない文字列を指定します
+  - betterNames
+    - 対象の名前を修正する候補を指定します
+  - allowedNames
+    - 許可する名前を指定します
+  - suffix:
+    - type のみ指定出来ます
+    - type のsuffixを指定します
+
+### ファイル例
+- `@/crews/index/views/page.tsx` の場合
+  - 生成されるキーワードは `['crews', 'crew', 'index', 'page']`
+- `@/crews/index/views/parts/Abc.tsx` の場合
+  - 生成されるキーワードは `['crews', 'crew', 'index', 'Abc']`
+- `@/crews/index/repositories/index.ts` の場合
+  - 生成されるキーワードは `['crews', 'crew', 'index', 'repositories', 'repository']`
+
+
+## rules
+
+```js
+const ignorekeywords = ['views', 'parts']
+const betterNames = {
+  '\/repositories\/': {
+    operator: '-',
+    names: ['repository', 'Repository'],
+  },
+  '\/entities\/': {
+    operator: '+',
+    names: ['entity'],
+  },
+  '\/slices\/': {
+    operator: '=',
+    names: ['index'],
+  },
+}
+// const allowedNames = {
+//   '\/views\/crews\/histories\/': ['crewId'],
+// }
+
+{
+  rules: {
+    'smarthr/redundant-name': [
+      'error', // 'warn', 'off'
+      {
+        type: { ignorekeywords, suffix: ['Props', 'Type'] },
+        file: { ignorekeywords, betternames },
+        // property: { ignorekeywords, allowedNames },
+        // function: { ignorekeywords },
+        // variable: { ignorekeywords },
+        // class: { ignorekeywords },
+      }
+    ]
+  },
+}
+```
+
+## ❌ Incorrect
+
+```js
+// @/crews/index/views/page.tsx
+
+type CrewIndexPage = { hoge: string }
+type CrewsView = { hoge: string }
+```
+```js
+// @/crews/show/repositories/index.tsx
+
+type CrewIndexRepository = { hoge: () => any }
+```
+
+## ✅ Correct
+
+```js
+// @/crews/index/views/page.tsx
+
+type ItemProps = { hoge: string }
+```
+```js
+// @/crews/show/repositories/index.tsx
+
+type IndexProps = { hoge: () => any }
+type ResponseType = { hoge: () => any }
+```

package/rules/{redundant-name.js → redundant-name/index.js}

@@ -1,7 +1,7 @@
 const path = require('path')
 const Inflector = require('inflected')
 
-const { rootPath } = require('../libs/common')
+const { rootPath } = require('../../libs/common')
 
 const uniq = (array) => array.filter((elem, index, self) => self.indexOf(elem) === index)
 

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

@@ -0,0 +1,39 @@
+# smarthr/require-barrel-import
+
+- tsconfig.json の compilerOptions.pathsに '@/*' としてroot path を指定する必要があります
+- importした対象が本来exportされているべきであるbarrel(index.tsなど)が有る場合、import pathの変更を促します
+  - 例: Page/parts/Menu/Item の import は Page/parts/Menu から行わせたい
+- ディレクトリ内のindexファイルを捜査し、対象を決定します
+
+## rules
+
+```js
+{
+  rules: {
+    'smarthr/require-barrel-import': 'error',
+  },
+}
+```
+
+## ❌ Incorrect
+
+```js
+// client/src/views/Page/parts/Menu/index.ts
+export { Menu } from './Menu'
+export { Item } from './Item'
+
+// client/src/App.tsx
+import { Item } from './Page/parts/Menu/Item'
+```
+
+## ✅ Correct
+
+
+```js
+// client/src/views/Page/parts/Menu/index.ts
+export { Menu } from './Menu'
+export { Item } from './Item'
+
+// client/src/App.tsx
+import { Item } from './Page/parts/Menu'
+```

package/rules/{require-barrel-import.js → require-barrel-import/index.js}

@@ -1,6 +1,6 @@
 const path = require('path')
 const fs = require('fs')
-const { replacePaths, rootPath } = require('../libs/common')
+const { replacePaths, rootPath } = require('../../libs/common')
 const calculateAbsoluteImportPath = (source) => {
   if (source[0] === '/') {
     return source

package/rules/require-export/README.md

@@ -0,0 +1,43 @@
+# smarthr/require-export
+
+- 対象ファイルにexportを強制させたい場合に利用します
+  - 例: Page.tsx ではページタイトルを設定させたいので useTitle を必ずexportさせたい
+
+## rules
+
+```js
+{
+  rules: {
+    'smarthr/require-export': [
+      'error',
+      {
+        'adapter\/.+\.ts': ['Props', 'generator'],
+        // slice以下のファイルかつmodulesディレクトリに属さずファイル名にmockを含まないもの
+        '^(?=.*\/slices\/[a-zA-Z0-9]+\.ts)(?!.*(\/modules\/|mock\.)).*$': [ 'default' ],
+      },
+    ]
+  },
+}
+```
+
+## ❌ Incorrect
+
+```js
+// adapter/index.ts
+export type Type = { abc: string }
+
+// slice/index.ts
+export const slice = { method: () => 'any action' }
+```
+
+## ✅ Correct
+
+
+```js
+// adapter/index.ts
+export type Props = { abc: string }
+
+// slice/index.ts
+const slice = { method: () => 'any action' }
+export default slice
+```

package/rules/require-export/index.js

@@ -0,0 +1,90 @@
+const SCHEMA = [{
+  type: 'object',
+  patternProperties: {
+    '.+': {
+      type: 'array',
+      items: { type: 'string' },
+      additionalProperties: true,
+    },
+  },
+  additionalProperties: true,
+}]
+
+const fetchEdgeDeclaration = (node) => {
+  const { declaration } = node
+
+  if (!declaration) {
+    return node
+  }
+
+  return fetchEdgeDeclaration(declaration)
+}
+
+module.exports = {
+  meta: {
+    type: 'suggestion',
+    messages: {
+      'require-export': '{{ message }}',
+    },
+    schema: SCHEMA,
+  },
+  create(context) {
+    const options = context.options[0]
+    const filename = context.getFilename()
+    const targetPathRegexs = Object.keys(options)
+    const targetRequires = targetPathRegexs.filter((regex) => !!filename.match(new RegExp(regex)))
+
+    if (targetRequires.length === 0) {
+      return {}
+    }
+
+    return {
+      Program: (node) => {
+        targetRequires.forEach((requireKey) => {
+          const option = options[requireKey]
+          let existDefault = false
+          const exports =
+            node.body
+              .filter((i) => {
+                if (i.type == 'ExportDefaultDeclaration') {
+                  existDefault = true
+
+                  return false
+                }
+
+                return i.type == 'ExportNamedDeclaration'
+              })
+              .map((i) => {
+                const declaration = fetchEdgeDeclaration(i)
+
+                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))
+                }
+
+                return declaration
+              })
+              .flat(2)
+
+          let notExistsExports = [...(!existDefault && option.includes('default') ? ['default'] : []), ...option.filter((o) => o !== 'default' && !exports.includes(o))]
+
+          if (notExistsExports.length) {
+            context.report({
+              node,
+              messageId: 'require-export',
+              data: {
+                message: `${notExistsExports.join(', ')} をexportしてください`,
+              },
+            })
+          }
+        })
+      },
+    }
+  },
+}
+module.exports.schema = SCHEMA

package/rules/require-import/README.md

@@ -0,0 +1,51 @@
+# smarthr/require-import
+
+- 対象ファイルにimportを強制させたい場合に利用します
+  - 例: Page.tsx ではページタイトルを設定させたいので useTitle を必ずimportさせたい
+
+## rules
+
+```js
+{
+  rules: {
+    'smarthr/require-import': [
+      'error',
+      {
+        'Buttons\/.+\.tsx': {
+          'smarthr-ui': {
+            imported: ['SecondaryButton'],
+            reportMessage: 'Buttons以下のコンポーネントでは {{module}}/{{export}} を拡張するようにしてください',
+          },
+        },
+        'Page.tsx$': {
+          './client/src/hooks/useTitle': {
+            imported: true,
+            reportMessage: '{{module}} を利用してください（ページタイトルを設定するため必要です）',
+          },
+        },
+      },
+    ]
+  },
+}
+```
+
+## ❌ Incorrect
+
+```js
+// client/src/Buttons/SecondaryButton.tsx
+import { SecondaryButtonAnchor } from 'smarthr-ui'
+
+// client/src/Page.tsx
+import { SecondaryButton } from 'smarthr-ui'
+```
+
+## ✅ Correct
+
+
+```js
+// client/src/Buttons/SecondaryButton.tsx
+import { SecondaryButton } from 'smarthr-ui'
+
+// client/src/Page.tsx
+import useTitle from '.hooks/useTitle'
+```

package/test/best-practice-for-date.js

@@ -0,0 +1,31 @@
+const rule = require('../rules/best-practice-for-date')
+const RuleTester = require('eslint').RuleTester
+
+const ruleTester = new RuleTester({
+  parserOptions: {
+    ecmaVersion: 2018,
+    ecmaFeatures: {
+      experimentalObjectRestSpread: true,
+      jsx: true,
+    },
+    sourceType: 'module',
+  },
+})
+
+const errorNewDate = "'new Date(arg)' のように引数一つのみの指定方は実行環境により結果が変わる可能性があるため 'new Date(2022, 12 - 1, 31)' のようにparseするなど他の方法を検討してください。"
+const errorDateParse = 'Date.parse は日付形式の解釈がブラウザによって異なるため、他の手段を検討してください'
+
+ruleTester.run('best-practice-for-date', rule, {
+  valid: [
+    { code: `new Date()` },
+    { code: `new Date(2022, 11, 31)` },
+    { code: `new Date('2022', '11', '31')` },
+    { code: `const year = 2022; const month = 11; const date = 31; new Date(year, month, date)` },
+  ],
+  invalid: [
+    { code: 'new Date("2022/12/31")', errors: [ { message: errorNewDate } ] },
+    { code: 'const arg = "2022/12/31"; new Date(arg)', errors: [ { message: errorNewDate } ] },
+    { code: 'Date.parse("2022/12/31")', errors: [ { message: errorDateParse } ] },
+    { code: 'const arg = "2022/12/31"; Date.parse(arg)', errors: [ { message: errorDateParse } ] },
+  ]
+})

package/test/prohibit-file-name.js

@@ -0,0 +1,45 @@
+const rule = require('../rules/prohibit-file-name')
+const RuleTester = require('eslint').RuleTester
+
+const ruleTester = new RuleTester({
+  parserOptions: {
+    sourceType: 'module',
+    ecmaVersion: 2015
+  },
+})
+
+ruleTester.run('prohibit-file-name', rule, {
+  valid: [
+    {
+      code: 'const any = "code"',
+      filename: 'hoge.js',
+      options: [
+        {
+          'fuga\.js': 'any message.',
+        }
+      ]
+    },
+  ],
+  invalid: [
+    {
+      code: 'const any = "code"',
+      filename: 'hoge.js',
+      options: [
+        {
+          'hoge\.js': 'any message.',
+        }
+      ],
+      errors: [{ message: 'any message.' }]
+    },
+    {
+      code: 'const any = "code"',
+      filename: 'hoge.js',
+      options: [
+        {
+          '(hoge|fuga)\.js': '$1.jsは作成しないで！',
+        }
+      ],
+      errors: [{ message: 'hoge.jsは作成しないで！' }]
+    },
+  ]
+})

package/test/require-export.js

@@ -0,0 +1,83 @@
+const rule = require('../rules/require-export')
+const RuleTester = require('eslint').RuleTester
+
+const ruleTester = new RuleTester({
+  parserOptions: {
+    sourceType: 'module',
+    ecmaVersion: 2015
+  },
+})
+
+ruleTester.run('require-export', rule, {
+  valid: [
+    {
+      code: `const hoge = {}; export { hoge }`,
+      options: [
+        {
+          '^.+$': ['hoge'],
+        }
+      ],
+    },
+    {
+      code: `const hoge = {}, fuga = {}; export { hoge, fuga }`,
+      options: [
+        {
+          '^.+$': ['hoge', 'fuga'],
+        }
+      ],
+    },
+    {
+      code: `const hoge = {}, fuga = {}; export { hoge, fuga }`,
+      options: [
+        {
+          '^.+$': ['fuga'],
+        }
+      ],
+    },
+    {
+      code: `export const hoge = {}`,
+      options: [
+        {
+          '^.+$': ['hoge'],
+        }
+      ],
+    },
+    {
+      code: `const hoge = {}; export default hoge`,
+      options: [
+        {
+          '^.+$': ['default'],
+        }
+      ],
+    },
+  ],
+  invalid: [
+    {
+      code: `const hoge ={}; export { hoge }`,
+      options: [
+        {
+          '^.+$': ['fuga'],
+        }
+      ],
+      errors: [{ message: 'fuga をexportしてください' }],
+    },
+    {
+      code: `export const hoge = {}`,
+      options: [
+        {
+          '^.+$': ['fuga'],
+        }
+      ],
+      errors: [{ message: 'fuga をexportしてください' }],
+    },
+    {
+      code: `const hoge = {}; export { hoge }`,
+      options: [
+        {
+          '^.+$': ['default'],
+        }
+      ],
+      errors: [{ message: 'default をexportしてください' }],
+    },
+  ]
+})