eslint-plugin-smarthr 0.2.25 → 0.3.0

package/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 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.3.0](https://github.com/kufu/eslint-plugin-smarthr/compare/v0.2.25...v0.3.0) (2023-04-14)
+
+
+### ⚠ BREAKING CHANGES
+
+* best-practice-for-remote-trigger-dialog にリネーム & 特定パターンでバグが起きる問題を修正する (#61)
+
+### Bug Fixes
+
+* best-practice-for-remote-trigger-dialog にリネーム & 特定パターンでバグが起きる問題を修正する ([#61](https://github.com/kufu/eslint-plugin-smarthr/issues/61)) ([0ad3601](https://github.com/kufu/eslint-plugin-smarthr/commit/0ad3601caa941946a0f30aacc2f010cd478b6133))
+
 ### [0.2.25](https://github.com/kufu/eslint-plugin-smarthr/compare/v0.2.24...v0.2.25) (2023-04-04)
 
 

package/package.json

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

package/rules/{best-practice-for-remote-trigger-action-dialog → best-practice-for-remote-trigger-dialog}/README.md

@@ -1,6 +1,6 @@
-# smarthr/best-practice-for-remote-trigger-action-dialog
+# smarthr/best-practice-for-remote-trigger-dialog
 
-- RemoteDialogTrigger, RemoteTriggerActionDialog のベストプラクティスをチェックするルールです
+- RemoteDialogTrigger, RemoteTriggerXxxxDialog のベストプラクティスをチェックするルールです
   - `id` もしくは `targetId` にリテラルな文字列以外が設定できなくなります
   - 変数なども使えません
   - これは対応するTriggerとDialogをわかりやすくするためです（検索などが容易になります）
@@ -10,7 +10,7 @@
 ```js
 {
   rules: {
-    'smarthr/best-practice-for-remote-trigger-action-dialog': 'error', // 'warn', 'off'
+    'smarthr/best-practice-for-remote-trigger-dialog': 'error', // 'warn', 'off'
   },
 }
 ```

package/rules/{best-practice-for-remote-trigger-action-dialog → best-practice-for-remote-trigger-dialog}/index.js

@@ -5,7 +5,7 @@ const EXPECTED_NAMES = {
   'RemoteTrigger(.+)Dialog$': 'RemoteTrigger(.+)Dialog$',
 }
 
-const REGEX_REMOTE_TRIGGER_DIALOG = /RemoteTrigger(.+)Dialog$/
+const REGEX_REMOTE_TRIGGER_DIALOG = /RemoteTrigger(Action|Message|Modeless)Dialog$/
 const REGEX_REMOTE_DIALOG_TRIGGER = /RemoteDialogTrigger$/
 
 module.exports = {
@@ -25,7 +25,7 @@ module.exports = {
           const attrName = regexRemoteTriggerDialog ? 'id' : 'targetId'
           const id = node.attributes.find((a) => a.name?.name === attrName)
 
-          if (id.value.type !== 'Literal') {
+          if (id && id.value.type !== 'Literal') {
             context.report({
               node: id,
               message: `${nodeName}の${attrName}属性には直接文字列を指定してください。

package/test/{best-practice-for-remote-trigger-action-dialog.js → best-practice-for-remote-trigger-dialog.js}

@@ -1,4 +1,4 @@
-const rule = require('../rules/best-practice-for-remote-trigger-action-dialog')
+const rule = require('../rules/best-practice-for-remote-trigger-dialog')
 const RuleTester = require('eslint').RuleTester
 
 const ruleTester = new RuleTester({
@@ -12,7 +12,7 @@ const ruleTester = new RuleTester({
   },
 })
 
-ruleTester.run('best-practice-for-remote-trigger-action-dialog', rule, {
+ruleTester.run('best-practice-for-remote-trigger-dialog', rule, {
   valid: [
     { code: `import styled from 'styled-components'` },
     { code: 'const HogeRemoteDialogTrigger = styled(RemoteDialogTrigger)``' },
@@ -33,10 +33,6 @@ ruleTester.run('best-practice-for-remote-trigger-action-dialog', rule, {
     { code: '<StyledRemoteDialogTrigger targetId={"fuga"}>open.</StyledRemoteDialogTrigger>', errors: [ { message: `StyledRemoteDialogTriggerのtargetId属性には直接文字列を指定してください。
   - 変数などは利用できません（これは関連するTriggerとDialogを検索しやすくするためです）
   - RemoteTriggerActionDialogはループやDropdown内にTriggerが存在する場合に利用してください
-  - ループやDropdown以外にTriggerが設定されている場合、TriggerAndActionDialogを利用してください` } ] },
-    { code: '<RemoteTriggerHogeDialog {...args} id={hoge}>content.</RemoteTriggerHogeDialog>', errors: [ { message: `RemoteTriggerHogeDialogのid属性には直接文字列を指定してください。
-  - 変数などは利用できません（これは関連するTriggerとDialogを検索しやすくするためです）
-  - RemoteTriggerActionDialogはループやDropdown内にTriggerが存在する場合に利用してください
   - ループやDropdown以外にTriggerが設定されている場合、TriggerAndActionDialogを利用してください` } ] },
     { code: '<StyldRemoteTriggerActionDialog {...args} id={"fuga"}>content.</StyldRemoteTriggerActionDialog>', errors: [ { message: `StyldRemoteTriggerActionDialogのid属性には直接文字列を指定してください。
   - 変数などは利用できません（これは関連するTriggerとDialogを検索しやすくするためです）