From 1ed81132de22ab635fb8910cac27777c508a7e60 Mon Sep 17 00:00:00 2001
From: Pig Fang <g-plane@hotmail.com>
Date: Wed, 9 Feb 2022 17:54:00 +0800
Subject: [PATCH 1/7] Add `prefer-true-attribute-shorthand` rule

---
 docs/rules/README.md                          |   1 +
 docs/rules/prefer-true-attribute-shorthand.md |  68 +++++++++
 lib/index.js                                  |   1 +
 lib/rules/prefer-true-attribute-shorthand.js  |  69 +++++++++
 .../rules/prefer-true-attribute-shorthand.js  | 133 ++++++++++++++++++
 5 files changed, 272 insertions(+)
 create mode 100644 docs/rules/prefer-true-attribute-shorthand.md
 create mode 100644 lib/rules/prefer-true-attribute-shorthand.js
 create mode 100644 tests/lib/rules/prefer-true-attribute-shorthand.js

diff --git a/docs/rules/README.md b/docs/rules/README.md
index b74075445..a612dca83 100644
--- a/docs/rules/README.md
+++ b/docs/rules/README.md
@@ -353,6 +353,7 @@ For example:
 | [vue/no-v-text](./no-v-text.md) | disallow use of v-text |  |
 | [vue/padding-line-between-blocks](./padding-line-between-blocks.md) | require or disallow padding lines between blocks | :wrench: |
 | [vue/prefer-separate-static-class](./prefer-separate-static-class.md) | require static class names in template to be in a separate `class` attribute | :wrench: |
+| [vue/prefer-true-attribute-shorthand](./prefer-true-attribute-shorthand.md) | require shorthand form attribute when `v-bind` value is `true`. | :bulb: |
 | [vue/require-direct-export](./require-direct-export.md) | require the component to be directly exported |  |
 | [vue/require-emit-validator](./require-emit-validator.md) | require type definitions in emits | :bulb: |
 | [vue/require-expose](./require-expose.md) | require declare public properties using `expose` | :bulb: |
diff --git a/docs/rules/prefer-true-attribute-shorthand.md b/docs/rules/prefer-true-attribute-shorthand.md
new file mode 100644
index 000000000..130df55b5
--- /dev/null
+++ b/docs/rules/prefer-true-attribute-shorthand.md
@@ -0,0 +1,68 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/prefer-true-attribute-shorthand
+description: require shorthand form attribute when `v-bind` value is `true`.
+---
+# vue/prefer-true-attribute-shorthand
+
+> require shorthand form attribute when `v-bind` value is `true`.
+
+- :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> ***This rule has not been released yet.*** </badge>
+- :bulb: Some problems reported by this rule are manually fixable by editor [suggestions](https://eslint.org/docs/developer-guide/working-with-rules#providing-suggestions).
+
+## :book: Rule Details
+
+`v-bind` attribute with `true` value usually can be written in shorthand form. This can reduce verbosity.
+
+<eslint-code-block :rules="{'vue/prefer-true-attribute-shorthand': ['error']}">
+
+```vue
+<template>
+  <!-- ✗ BAD -->
+  <MyComponent v-bind:show="true" />
+  <MyComponent :show="true" />
+
+  <!-- ✓ GOOD -->
+  <MyComponent show />
+  <MyComponent another-prop="true" />
+</template>
+```
+
+</eslint-code-block>
+
+However, those two representations are not always equivalent.
+This case will be occurred if the definition of a prop include `String`:
+
+```vue
+<template>
+  <pre>{{ prop }}</pre>
+</template>
+
+<script>
+export default {
+  name: 'MyComponent',
+  props: {
+    prop: [String, Boolean]
+  }
+}
+</script>
+```
+
+```vue
+<template>
+  <MyComponent prop />
+  <MyComponent :prop="true"/>
+</template>
+```
+
+Those two calls will introduce different render result. See [this demo](https://sfc.vuejs.org/#eyJBcHAudnVlIjoiPHNjcmlwdCBzZXR1cD5cbmltcG9ydCBNeUNvbXBvbmVudCBmcm9tICcuL015Q29tcG9uZW50LnZ1ZSdcbjwvc2NyaXB0PlxuXG48dGVtcGxhdGU+XG4gIDxNeUNvbXBvbmVudCBhIGIvPlxuICA8TXlDb21wb25lbnQgOmE9XCJ0cnVlXCIgOmI9XCJ0cnVlXCIvPlxuPC90ZW1wbGF0ZT4iLCJpbXBvcnQtbWFwLmpzb24iOiJ7XG4gIFwiaW1wb3J0c1wiOiB7XG4gICAgXCJ2dWVcIjogXCJodHRwczovL3NmYy52dWVqcy5vcmcvdnVlLnJ1bnRpbWUuZXNtLWJyb3dzZXIuanNcIlxuICB9XG59IiwiTXlDb21wb25lbnQudnVlIjoiPHNjcmlwdD5cbmV4cG9ydCBkZWZhdWx0IHtcbiAgcHJvcHM6IHtcbiAgICBhOiBCb29sZWFuLFxuICAgIGI6IFtTdHJpbmcsIEJvb2xlYW5dXG4gIH1cbn1cbjwvc2NyaXB0PlxuXG48dGVtcGxhdGU+XG4gIDxwcmU+XG5hOiB7e2F9fVxuYjoge3tifX1cbiAgPC9wcmU+XG48L3RlbXBsYXRlPiJ9).
+
+## :wrench: Options
+
+Nothing.
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/prefer-true-attribute-shorthand.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/prefer-true-attribute-shorthand.js)
diff --git a/lib/index.js b/lib/index.js
index 5d770e8b2..941de0b56 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -159,6 +159,7 @@ module.exports = {
     'padding-line-between-blocks': require('./rules/padding-line-between-blocks'),
     'prefer-separate-static-class': require('./rules/prefer-separate-static-class'),
     'prefer-template': require('./rules/prefer-template'),
+    'prefer-true-attribute-shorthand': require('./rules/prefer-true-attribute-shorthand'),
     'prop-name-casing': require('./rules/prop-name-casing'),
     'quote-props': require('./rules/quote-props'),
     'require-component-is': require('./rules/require-component-is'),
diff --git a/lib/rules/prefer-true-attribute-shorthand.js b/lib/rules/prefer-true-attribute-shorthand.js
new file mode 100644
index 000000000..69d1797ae
--- /dev/null
+++ b/lib/rules/prefer-true-attribute-shorthand.js
@@ -0,0 +1,69 @@
+/**
+ * @author Pig Fang
+ * See LICENSE file in root directory for full license.
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const utils = require('../utils')
+
+// ------------------------------------------------------------------------------
+// Rule Definition
+// ------------------------------------------------------------------------------
+
+module.exports = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description:
+        'require shorthand form attribute when `v-bind` value is `true`.',
+      categories: undefined,
+      url: 'https://eslint.vuejs.org/rules/prefer-true-attribute-shorthand.html'
+    },
+    fixable: null,
+    hasSuggestions: true,
+    schema: [],
+    messages: {
+      report:
+        "Boolean prop with 'true' value can be written in shorthand form.",
+      fix: 'Rewrite this prop into shorthand form.'
+    }
+  },
+  /** @param {RuleContext} context */
+  create(context) {
+    // ...
+
+    return utils.defineTemplateBodyVisitor(context, {
+      /** @param {VDirective} node */
+      "VAttribute[directive=true][key.name.name='bind']"(node) {
+        const { argument } = node.key
+        if (
+          !argument ||
+          !node.value ||
+          !node.value.expression ||
+          node.value.expression.type !== 'Literal' ||
+          node.value.expression.value !== true
+        ) {
+          return
+        }
+
+        context.report({
+          node,
+          messageId: 'report',
+          suggest: [
+            {
+              messageId: 'fix',
+              fix: (fixer) => {
+                const sourceCode = context.getSourceCode()
+                return fixer.replaceText(node, sourceCode.getText(argument))
+              }
+            }
+          ]
+        })
+      }
+    })
+  }
+}
diff --git a/tests/lib/rules/prefer-true-attribute-shorthand.js b/tests/lib/rules/prefer-true-attribute-shorthand.js
new file mode 100644
index 000000000..002b0d004
--- /dev/null
+++ b/tests/lib/rules/prefer-true-attribute-shorthand.js
@@ -0,0 +1,133 @@
+/**
+ * @author Pig Fang
+ * See LICENSE file in root directory for full license.
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/prefer-true-attribute-shorthand')
+
+const tester = new RuleTester({
+  parser: require.resolve('vue-eslint-parser'),
+  parserOptions: {
+    ecmaVersion: 2020,
+    sourceType: 'module'
+  }
+})
+
+tester.run('prefer-true-attribute-shorthand', rule, {
+  valid: [
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp v-if="true" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp v-bind="true" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp v-loading="true" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp show="true" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp v-bind:show="value" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp :show="value" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp v-bind:show="false" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp :show="false" />
+      </template>
+      `
+    }
+  ],
+  invalid: [
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp v-bind:show="true" />
+      </template>`,
+      errors: [
+        {
+          messageId: 'report',
+          line: 3,
+          column: 17,
+          suggestions: [
+            {
+              messageId: 'fix',
+              output: `
+      <template>
+        <MyComp show />
+      </template>`
+            }
+          ]
+        }
+      ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp :show="true" />
+      </template>`,
+      errors: [
+        {
+          messageId: 'report',
+          line: 3,
+          column: 17,
+          suggestions: [
+            {
+              messageId: 'fix',
+              output: `
+      <template>
+        <MyComp show />
+      </template>`
+            }
+          ]
+        }
+      ]
+    }
+  ]
+})

From 62119d57154019b00d0a6fc2d77f1a587fb64487 Mon Sep 17 00:00:00 2001
From: Pig Fang <g-plane@hotmail.com>
Date: Wed, 9 Feb 2022 18:08:26 +0800
Subject: [PATCH 2/7] fix typo

---
 docs/rules/prefer-true-attribute-shorthand.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docs/rules/prefer-true-attribute-shorthand.md b/docs/rules/prefer-true-attribute-shorthand.md
index 130df55b5..f67a3beee 100644
--- a/docs/rules/prefer-true-attribute-shorthand.md
+++ b/docs/rules/prefer-true-attribute-shorthand.md
@@ -32,7 +32,7 @@ description: require shorthand form attribute when `v-bind` value is `true`.
 </eslint-code-block>
 
 However, those two representations are not always equivalent.
-This case will be occurred if the definition of a prop include `String`:
+This case will be occurred if the definition of a prop includes `String`:
 
 ```vue
 <template>

From dbfe89cc921333cccc0c723772bd8da03633a9af Mon Sep 17 00:00:00 2001
From: Pig Fang <g-plane@hotmail.com>
Date: Thu, 10 Feb 2022 14:09:46 +0800
Subject: [PATCH 3/7] accept suggestions

---
 docs/rules/README.md                          |  2 +-
 docs/rules/prefer-true-attribute-shorthand.md | 44 +++++++++++++------
 lib/rules/prefer-true-attribute-shorthand.js  |  6 +--
 3 files changed, 33 insertions(+), 19 deletions(-)

diff --git a/docs/rules/README.md b/docs/rules/README.md
index a612dca83..b2f19b512 100644
--- a/docs/rules/README.md
+++ b/docs/rules/README.md
@@ -353,7 +353,7 @@ For example:
 | [vue/no-v-text](./no-v-text.md) | disallow use of v-text |  |
 | [vue/padding-line-between-blocks](./padding-line-between-blocks.md) | require or disallow padding lines between blocks | :wrench: |
 | [vue/prefer-separate-static-class](./prefer-separate-static-class.md) | require static class names in template to be in a separate `class` attribute | :wrench: |
-| [vue/prefer-true-attribute-shorthand](./prefer-true-attribute-shorthand.md) | require shorthand form attribute when `v-bind` value is `true`. | :bulb: |
+| [vue/prefer-true-attribute-shorthand](./prefer-true-attribute-shorthand.md) | require shorthand form attribute when `v-bind` value is `true` | :bulb: |
 | [vue/require-direct-export](./require-direct-export.md) | require the component to be directly exported |  |
 | [vue/require-emit-validator](./require-emit-validator.md) | require type definitions in emits | :bulb: |
 | [vue/require-expose](./require-expose.md) | require declare public properties using `expose` | :bulb: |
diff --git a/docs/rules/prefer-true-attribute-shorthand.md b/docs/rules/prefer-true-attribute-shorthand.md
index f67a3beee..a7871af9f 100644
--- a/docs/rules/prefer-true-attribute-shorthand.md
+++ b/docs/rules/prefer-true-attribute-shorthand.md
@@ -2,11 +2,11 @@
 pageClass: rule-details
 sidebarDepth: 0
 title: vue/prefer-true-attribute-shorthand
-description: require shorthand form attribute when `v-bind` value is `true`.
+description: require shorthand form attribute when `v-bind` value is `true`
 ---
 # vue/prefer-true-attribute-shorthand
 
-> require shorthand form attribute when `v-bind` value is `true`.
+> require shorthand form attribute when `v-bind` value is `true`
 
 - :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> ***This rule has not been released yet.*** </badge>
 - :bulb: Some problems reported by this rule are manually fixable by editor [suggestions](https://eslint.org/docs/developer-guide/working-with-rules#providing-suggestions).
@@ -31,32 +31,48 @@ description: require shorthand form attribute when `v-bind` value is `true`.
 
 </eslint-code-block>
 
-However, those two representations are not always equivalent.
-This case will be occurred if the definition of a prop includes `String`:
+::: warning Warning
+The shorthand form is not always equivalent! If a prop accepts multiple types, but Boolean is not the first one, a shorthand prop won't pass `true`.
+:::
 
 ```vue
-<template>
-  <pre>{{ prop }}</pre>
-</template>
-
 <script>
 export default {
   name: 'MyComponent',
   props: {
-    prop: [String, Boolean]
+    bool: Boolean,
+    boolOrString: [Boolean, String],
+    stringOrBool: [String, Boolean],
   }
 }
 </script>
 ```
 
+**Shorthand form:**
+
 ```vue
-<template>
-  <MyComponent prop />
-  <MyComponent :prop="true"/>
-</template>
+<MyComponent bool bool-or-string string-or-bool />
+```
+
+```
+bool: true (boolean)
+boolOrString: true (boolean)
+stringOrBool: "" (string)
+```
+
+**Longhand form:**
+
+```vue
+<MyComponent :bool="true" :bool-or-string="true" :string-or-bool="true" />
+```
+
+```
+bool: true (boolean)
+boolOrString: true (boolean)
+stringOrBool: true (boolean)
 ```
 
-Those two calls will introduce different render result. See [this demo](https://sfc.vuejs.org/#eyJBcHAudnVlIjoiPHNjcmlwdCBzZXR1cD5cbmltcG9ydCBNeUNvbXBvbmVudCBmcm9tICcuL015Q29tcG9uZW50LnZ1ZSdcbjwvc2NyaXB0PlxuXG48dGVtcGxhdGU+XG4gIDxNeUNvbXBvbmVudCBhIGIvPlxuICA8TXlDb21wb25lbnQgOmE9XCJ0cnVlXCIgOmI9XCJ0cnVlXCIvPlxuPC90ZW1wbGF0ZT4iLCJpbXBvcnQtbWFwLmpzb24iOiJ7XG4gIFwiaW1wb3J0c1wiOiB7XG4gICAgXCJ2dWVcIjogXCJodHRwczovL3NmYy52dWVqcy5vcmcvdnVlLnJ1bnRpbWUuZXNtLWJyb3dzZXIuanNcIlxuICB9XG59IiwiTXlDb21wb25lbnQudnVlIjoiPHNjcmlwdD5cbmV4cG9ydCBkZWZhdWx0IHtcbiAgcHJvcHM6IHtcbiAgICBhOiBCb29sZWFuLFxuICAgIGI6IFtTdHJpbmcsIEJvb2xlYW5dXG4gIH1cbn1cbjwvc2NyaXB0PlxuXG48dGVtcGxhdGU+XG4gIDxwcmU+XG5hOiB7e2F9fVxuYjoge3tifX1cbiAgPC9wcmU+XG48L3RlbXBsYXRlPiJ9).
+Those two calls will introduce different render result. See [this demo](https://sfc.vuejs.org/#eyJBcHAudnVlIjoiPHNjcmlwdCBzZXR1cD5cbmltcG9ydCBNeUNvbXBvbmVudCBmcm9tICcuL015Q29tcG9uZW50LnZ1ZSdcbjwvc2NyaXB0PlxuXG48dGVtcGxhdGU+XG4gIFNob3J0aGFuZCBmb3JtOlxuICA8TXlDb21wb25lbnQgYm9vbCBib29sLW9yLXN0cmluZyBzdHJpbmctb3ItYm9vbCAvPlxuICBcbiAgTG9uZ2hhbmQgZm9ybTpcbiAgPE15Q29tcG9uZW50IDpib29sPVwidHJ1ZVwiIDpib29sLW9yLXN0cmluZz1cInRydWVcIiA6c3RyaW5nLW9yLWJvb2w9XCJ0cnVlXCIgLz5cbjwvdGVtcGxhdGU+IiwiaW1wb3J0LW1hcC5qc29uIjoie1xuICBcImltcG9ydHNcIjoge1xuICAgIFwidnVlXCI6IFwiaHR0cHM6Ly9zZmMudnVlanMub3JnL3Z1ZS5ydW50aW1lLmVzbS1icm93c2VyLmpzXCJcbiAgfVxufSIsIk15Q29tcG9uZW50LnZ1ZSI6IjxzY3JpcHQ+XG5leHBvcnQgZGVmYXVsdCB7XG4gIHByb3BzOiB7XG4gICAgYm9vbDogQm9vbGVhbixcbiAgICBib29sT3JTdHJpbmc6IFtCb29sZWFuLCBTdHJpbmddLFxuICAgIHN0cmluZ09yQm9vbDogW1N0cmluZywgQm9vbGVhbl0sXG4gIH1cbn1cbjwvc2NyaXB0PlxuXG48dGVtcGxhdGU+XG4gIDxwcmU+XG5ib29sOiB7e2Jvb2x9fSAoe3sgdHlwZW9mIGJvb2wgfX0pXG5ib29sT3JTdHJpbmc6IHt7Ym9vbE9yU3RyaW5nfX0gKHt7IHR5cGVvZiBib29sT3JTdHJpbmcgfX0pXG5zdHJpbmdPckJvb2w6IHt7c3RyaW5nT3JCb29sfX0gKHt7IHR5cGVvZiBzdHJpbmdPckJvb2wgfX0pXG4gIDwvcHJlPlxuPC90ZW1wbGF0ZT4ifQ==).
 
 ## :wrench: Options
 
diff --git a/lib/rules/prefer-true-attribute-shorthand.js b/lib/rules/prefer-true-attribute-shorthand.js
index 69d1797ae..9da953c77 100644
--- a/lib/rules/prefer-true-attribute-shorthand.js
+++ b/lib/rules/prefer-true-attribute-shorthand.js
@@ -16,10 +16,10 @@ const utils = require('../utils')
 
 module.exports = {
   meta: {
-    type: 'problem',
+    type: 'suggestion',
     docs: {
       description:
-        'require shorthand form attribute when `v-bind` value is `true`.',
+        'require shorthand form attribute when `v-bind` value is `true`',
       categories: undefined,
       url: 'https://eslint.vuejs.org/rules/prefer-true-attribute-shorthand.html'
     },
@@ -34,8 +34,6 @@ module.exports = {
   },
   /** @param {RuleContext} context */
   create(context) {
-    // ...
-
     return utils.defineTemplateBodyVisitor(context, {
       /** @param {VDirective} node */
       "VAttribute[directive=true][key.name.name='bind']"(node) {

From 83e4311c7670432d3ccec75c5f33477dbe288098 Mon Sep 17 00:00:00 2001
From: Pig Fang <g-plane@hotmail.com>
Date: Thu, 10 Feb 2022 14:43:13 +0800
Subject: [PATCH 4/7] accept option `"always"` or `"never"`

---
 docs/rules/README.md                          |   2 +-
 docs/rules/prefer-true-attribute-shorthand.md |  31 ++++-
 lib/rules/prefer-true-attribute-shorthand.js  |  44 +++++--
 .../rules/prefer-true-attribute-shorthand.js  | 118 +++++++++++++++++-
 4 files changed, 175 insertions(+), 20 deletions(-)

diff --git a/docs/rules/README.md b/docs/rules/README.md
index b2f19b512..cbba3dada 100644
--- a/docs/rules/README.md
+++ b/docs/rules/README.md
@@ -353,7 +353,7 @@ For example:
 | [vue/no-v-text](./no-v-text.md) | disallow use of v-text |  |
 | [vue/padding-line-between-blocks](./padding-line-between-blocks.md) | require or disallow padding lines between blocks | :wrench: |
 | [vue/prefer-separate-static-class](./prefer-separate-static-class.md) | require static class names in template to be in a separate `class` attribute | :wrench: |
-| [vue/prefer-true-attribute-shorthand](./prefer-true-attribute-shorthand.md) | require shorthand form attribute when `v-bind` value is `true` | :bulb: |
+| [vue/prefer-true-attribute-shorthand](./prefer-true-attribute-shorthand.md) | require shorthand form attribute when `v-bind` value is `true` | :wrench::bulb: |
 | [vue/require-direct-export](./require-direct-export.md) | require the component to be directly exported |  |
 | [vue/require-emit-validator](./require-emit-validator.md) | require type definitions in emits | :bulb: |
 | [vue/require-expose](./require-expose.md) | require declare public properties using `expose` | :bulb: |
diff --git a/docs/rules/prefer-true-attribute-shorthand.md b/docs/rules/prefer-true-attribute-shorthand.md
index a7871af9f..4780b45f5 100644
--- a/docs/rules/prefer-true-attribute-shorthand.md
+++ b/docs/rules/prefer-true-attribute-shorthand.md
@@ -9,13 +9,14 @@ description: require shorthand form attribute when `v-bind` value is `true`
 > require shorthand form attribute when `v-bind` value is `true`
 
 - :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> ***This rule has not been released yet.*** </badge>
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
 - :bulb: Some problems reported by this rule are manually fixable by editor [suggestions](https://eslint.org/docs/developer-guide/working-with-rules#providing-suggestions).
 
 ## :book: Rule Details
 
 `v-bind` attribute with `true` value usually can be written in shorthand form. This can reduce verbosity.
 
-<eslint-code-block :rules="{'vue/prefer-true-attribute-shorthand': ['error']}">
+<eslint-code-block fix :rules="{'vue/prefer-true-attribute-shorthand': ['error']}">
 
 ```vue
 <template>
@@ -76,7 +77,33 @@ Those two calls will introduce different render result. See [this demo](https://
 
 ## :wrench: Options
 
-Nothing.
+Default options is `"always"`.
+
+```json
+{
+  "vue/prefer-true-attribute-shorthand": ["error", "always" | "never"]
+}
+```
+
+- `"always"` (default) ... requires shorthand form.
+- `"never"` ... requires long form.
+
+### `"never"`
+
+<eslint-code-block fix :rules="{'vue/prefer-true-attribute-shorthand': ['error', 'never']}">
+
+```vue
+<template>
+  <!-- ✗ BAD -->
+  <MyComponent show />
+
+  <!-- ✓ GOOD -->
+  <MyComponent :show="true" />
+  <MyComponent v-bind:show="true" />
+</template>
+```
+
+</eslint-code-block>
 
 ## :mag: Implementation
 
diff --git a/lib/rules/prefer-true-attribute-shorthand.js b/lib/rules/prefer-true-attribute-shorthand.js
index 9da953c77..b27a8a711 100644
--- a/lib/rules/prefer-true-attribute-shorthand.js
+++ b/lib/rules/prefer-true-attribute-shorthand.js
@@ -23,23 +23,40 @@ module.exports = {
       categories: undefined,
       url: 'https://eslint.vuejs.org/rules/prefer-true-attribute-shorthand.html'
     },
-    fixable: null,
+    fixable: 'code',
     hasSuggestions: true,
-    schema: [],
+    schema: [{ enum: ['always', 'never'] }],
     messages: {
-      report:
-        "Boolean prop with 'true' value can be written in shorthand form.",
-      fix: 'Rewrite this prop into shorthand form.'
+      expectShort:
+        "Boolean prop with 'true' value should be written in shorthand form.",
+      expectLong:
+        "Boolean prop with 'true' value should be written in long form.",
+      suggestion: 'Rewrite this prop into shorthand form.'
     }
   },
   /** @param {RuleContext} context */
   create(context) {
+    /** @type {'always' | 'never'} */
+    const option = context.options[0] || 'always'
+
     return utils.defineTemplateBodyVisitor(context, {
-      /** @param {VDirective} node */
-      "VAttribute[directive=true][key.name.name='bind']"(node) {
-        const { argument } = node.key
+      VAttribute(node) {
+        if (option === 'never' && !node.directive && !node.value) {
+          context.report({
+            node,
+            messageId: 'expectLong',
+            fix: (fixer) =>
+              fixer.replaceText(node, `:${node.key.rawName}="true"`)
+          })
+          return
+        }
+
+        if (option !== 'always') {
+          return
+        }
+
         if (
-          !argument ||
+          !node.directive ||
           !node.value ||
           !node.value.expression ||
           node.value.expression.type !== 'Literal' ||
@@ -48,12 +65,17 @@ module.exports = {
           return
         }
 
+        const { argument } = node.key
+        if (!argument) {
+          return
+        }
+
         context.report({
           node,
-          messageId: 'report',
+          messageId: 'expectShort',
           suggest: [
             {
-              messageId: 'fix',
+              messageId: 'suggestion',
               fix: (fixer) => {
                 const sourceCode = context.getSourceCode()
                 return fixer.replaceText(node, sourceCode.getText(argument))
diff --git a/tests/lib/rules/prefer-true-attribute-shorthand.js b/tests/lib/rules/prefer-true-attribute-shorthand.js
index 002b0d004..8fd74281a 100644
--- a/tests/lib/rules/prefer-true-attribute-shorthand.js
+++ b/tests/lib/rules/prefer-true-attribute-shorthand.js
@@ -80,6 +80,41 @@ tester.run('prefer-true-attribute-shorthand', rule, {
         <MyComp :show="false" />
       </template>
       `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp show />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp show />
+      </template>
+      `,
+      options: ['always']
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp :show="true" />
+      </template>
+      `,
+      options: ['never']
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp v-bind:show="true" />
+      </template>
+      `,
+      options: ['never']
     }
   ],
   invalid: [
@@ -91,12 +126,12 @@ tester.run('prefer-true-attribute-shorthand', rule, {
       </template>`,
       errors: [
         {
-          messageId: 'report',
+          messageId: 'expectShort',
           line: 3,
           column: 17,
           suggestions: [
             {
-              messageId: 'fix',
+              messageId: 'suggestion',
               output: `
       <template>
         <MyComp show />
@@ -104,7 +139,8 @@ tester.run('prefer-true-attribute-shorthand', rule, {
             }
           ]
         }
-      ]
+      ],
+      output: null
     },
     {
       filename: 'test.vue',
@@ -114,12 +150,12 @@ tester.run('prefer-true-attribute-shorthand', rule, {
       </template>`,
       errors: [
         {
-          messageId: 'report',
+          messageId: 'expectShort',
           line: 3,
           column: 17,
           suggestions: [
             {
-              messageId: 'fix',
+              messageId: 'suggestion',
               output: `
       <template>
         <MyComp show />
@@ -127,7 +163,77 @@ tester.run('prefer-true-attribute-shorthand', rule, {
             }
           ]
         }
-      ]
+      ],
+      output: null
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp v-bind:show="true" />
+      </template>`,
+      options: ['always'],
+      errors: [
+        {
+          messageId: 'expectShort',
+          line: 3,
+          column: 17,
+          suggestions: [
+            {
+              messageId: 'suggestion',
+              output: `
+      <template>
+        <MyComp show />
+      </template>`
+            }
+          ]
+        }
+      ],
+      output: null
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp :show="true" />
+      </template>`,
+      options: ['always'],
+      errors: [
+        {
+          messageId: 'expectShort',
+          line: 3,
+          column: 17,
+          suggestions: [
+            {
+              messageId: 'suggestion',
+              output: `
+      <template>
+        <MyComp show />
+      </template>`
+            }
+          ]
+        }
+      ],
+      output: null
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <MyComp show />
+      </template>`,
+      options: ['never'],
+      errors: [
+        {
+          messageId: 'expectLong',
+          line: 3,
+          column: 17
+        }
+      ],
+      output: `
+      <template>
+        <MyComp :show="true" />
+      </template>`
     }
   ]
 })

From ca411322e88161314dfcba6b3da5e42745041e47 Mon Sep 17 00:00:00 2001
From: Pig Fang <g-plane@hotmail.com>
Date: Fri, 11 Feb 2022 15:35:12 +0800
Subject: [PATCH 5/7] ignore native HTML elements

---
 lib/rules/prefer-true-attribute-shorthand.js  |  4 +++
 .../rules/prefer-true-attribute-shorthand.js  | 33 +++++++++++++++++++
 2 files changed, 37 insertions(+)

diff --git a/lib/rules/prefer-true-attribute-shorthand.js b/lib/rules/prefer-true-attribute-shorthand.js
index b27a8a711..cb2ed9645 100644
--- a/lib/rules/prefer-true-attribute-shorthand.js
+++ b/lib/rules/prefer-true-attribute-shorthand.js
@@ -41,6 +41,10 @@ module.exports = {
 
     return utils.defineTemplateBodyVisitor(context, {
       VAttribute(node) {
+        if (!utils.isCustomComponent(node.parent.parent)) {
+          return
+        }
+
         if (option === 'never' && !node.directive && !node.value) {
           context.report({
             node,
diff --git a/tests/lib/rules/prefer-true-attribute-shorthand.js b/tests/lib/rules/prefer-true-attribute-shorthand.js
index 8fd74281a..fcf801935 100644
--- a/tests/lib/rules/prefer-true-attribute-shorthand.js
+++ b/tests/lib/rules/prefer-true-attribute-shorthand.js
@@ -115,6 +115,39 @@ tester.run('prefer-true-attribute-shorthand', rule, {
       </template>
       `,
       options: ['never']
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <input v-bind:checked="true" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <input :checked="true" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <input checked="checked" />
+      </template>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <template>
+        <input checked />
+      </template>
+      `,
+      options: ['never']
     }
   ],
   invalid: [

From 2188af9edd3159d81c1da72b3f61894089bbd86d Mon Sep 17 00:00:00 2001
From: Pig Fang <g-plane@hotmail.com>
Date: Fri, 11 Feb 2022 15:43:37 +0800
Subject: [PATCH 6/7] provide suggestions instead of auto fix

---
 lib/rules/prefer-true-attribute-shorthand.js  | 25 ++++++++++++++---
 .../rules/prefer-true-attribute-shorthand.js  | 27 +++++++++++++------
 2 files changed, 40 insertions(+), 12 deletions(-)

diff --git a/lib/rules/prefer-true-attribute-shorthand.js b/lib/rules/prefer-true-attribute-shorthand.js
index cb2ed9645..19a98a351 100644
--- a/lib/rules/prefer-true-attribute-shorthand.js
+++ b/lib/rules/prefer-true-attribute-shorthand.js
@@ -31,7 +31,11 @@ module.exports = {
         "Boolean prop with 'true' value should be written in shorthand form.",
       expectLong:
         "Boolean prop with 'true' value should be written in long form.",
-      suggestion: 'Rewrite this prop into shorthand form.'
+      rewriteIntoShort: 'Rewrite this prop into shorthand form.',
+      rewriteIntoLongVueProp:
+        'Rewrite this prop into long-form Vue component prop.',
+      rewriteIntoLongHtmlAttr:
+        'Rewrite this prop into long-form HTML attribute.'
     }
   },
   /** @param {RuleContext} context */
@@ -49,8 +53,21 @@ module.exports = {
           context.report({
             node,
             messageId: 'expectLong',
-            fix: (fixer) =>
-              fixer.replaceText(node, `:${node.key.rawName}="true"`)
+            suggest: [
+              {
+                messageId: 'rewriteIntoLongVueProp',
+                fix: (fixer) =>
+                  fixer.replaceText(node, `:${node.key.rawName}="true"`)
+              },
+              {
+                messageId: 'rewriteIntoLongHtmlAttr',
+                fix: (fixer) =>
+                  fixer.replaceText(
+                    node,
+                    `${node.key.rawName}="${node.key.rawName}"`
+                  )
+              }
+            ]
           })
           return
         }
@@ -79,7 +96,7 @@ module.exports = {
           messageId: 'expectShort',
           suggest: [
             {
-              messageId: 'suggestion',
+              messageId: 'rewriteIntoShort',
               fix: (fixer) => {
                 const sourceCode = context.getSourceCode()
                 return fixer.replaceText(node, sourceCode.getText(argument))
diff --git a/tests/lib/rules/prefer-true-attribute-shorthand.js b/tests/lib/rules/prefer-true-attribute-shorthand.js
index fcf801935..e7d52ef7f 100644
--- a/tests/lib/rules/prefer-true-attribute-shorthand.js
+++ b/tests/lib/rules/prefer-true-attribute-shorthand.js
@@ -164,7 +164,7 @@ tester.run('prefer-true-attribute-shorthand', rule, {
           column: 17,
           suggestions: [
             {
-              messageId: 'suggestion',
+              messageId: 'rewriteIntoShort',
               output: `
       <template>
         <MyComp show />
@@ -188,7 +188,7 @@ tester.run('prefer-true-attribute-shorthand', rule, {
           column: 17,
           suggestions: [
             {
-              messageId: 'suggestion',
+              messageId: 'rewriteIntoShort',
               output: `
       <template>
         <MyComp show />
@@ -213,7 +213,7 @@ tester.run('prefer-true-attribute-shorthand', rule, {
           column: 17,
           suggestions: [
             {
-              messageId: 'suggestion',
+              messageId: 'rewriteIntoShort',
               output: `
       <template>
         <MyComp show />
@@ -238,7 +238,7 @@ tester.run('prefer-true-attribute-shorthand', rule, {
           column: 17,
           suggestions: [
             {
-              messageId: 'suggestion',
+              messageId: 'rewriteIntoShort',
               output: `
       <template>
         <MyComp show />
@@ -260,13 +260,24 @@ tester.run('prefer-true-attribute-shorthand', rule, {
         {
           messageId: 'expectLong',
           line: 3,
-          column: 17
-        }
-      ],
-      output: `
+          column: 17,
+          suggestions: [
+            {
+              output: `
       <template>
         <MyComp :show="true" />
       </template>`
+            },
+            {
+              output: `
+      <template>
+        <MyComp show="show" />
+      </template>`
+            }
+          ]
+        }
+      ],
+      output: null
     }
   ]
 })

From df75affd6888112837795a567458546d9ae90046 Mon Sep 17 00:00:00 2001
From: Pig Fang <g-plane@hotmail.com>
Date: Fri, 11 Feb 2022 23:06:49 +0800
Subject: [PATCH 7/7] meta update

---
 docs/rules/README.md                          | 2 +-
 docs/rules/prefer-true-attribute-shorthand.md | 5 ++---
 lib/rules/prefer-true-attribute-shorthand.js  | 2 +-
 3 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/docs/rules/README.md b/docs/rules/README.md
index cbba3dada..b2f19b512 100644
--- a/docs/rules/README.md
+++ b/docs/rules/README.md
@@ -353,7 +353,7 @@ For example:
 | [vue/no-v-text](./no-v-text.md) | disallow use of v-text |  |
 | [vue/padding-line-between-blocks](./padding-line-between-blocks.md) | require or disallow padding lines between blocks | :wrench: |
 | [vue/prefer-separate-static-class](./prefer-separate-static-class.md) | require static class names in template to be in a separate `class` attribute | :wrench: |
-| [vue/prefer-true-attribute-shorthand](./prefer-true-attribute-shorthand.md) | require shorthand form attribute when `v-bind` value is `true` | :wrench::bulb: |
+| [vue/prefer-true-attribute-shorthand](./prefer-true-attribute-shorthand.md) | require shorthand form attribute when `v-bind` value is `true` | :bulb: |
 | [vue/require-direct-export](./require-direct-export.md) | require the component to be directly exported |  |
 | [vue/require-emit-validator](./require-emit-validator.md) | require type definitions in emits | :bulb: |
 | [vue/require-expose](./require-expose.md) | require declare public properties using `expose` | :bulb: |
diff --git a/docs/rules/prefer-true-attribute-shorthand.md b/docs/rules/prefer-true-attribute-shorthand.md
index 4780b45f5..b2b521b6a 100644
--- a/docs/rules/prefer-true-attribute-shorthand.md
+++ b/docs/rules/prefer-true-attribute-shorthand.md
@@ -9,14 +9,13 @@ description: require shorthand form attribute when `v-bind` value is `true`
 > require shorthand form attribute when `v-bind` value is `true`
 
 - :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> ***This rule has not been released yet.*** </badge>
-- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
 - :bulb: Some problems reported by this rule are manually fixable by editor [suggestions](https://eslint.org/docs/developer-guide/working-with-rules#providing-suggestions).
 
 ## :book: Rule Details
 
 `v-bind` attribute with `true` value usually can be written in shorthand form. This can reduce verbosity.
 
-<eslint-code-block fix :rules="{'vue/prefer-true-attribute-shorthand': ['error']}">
+<eslint-code-block :rules="{'vue/prefer-true-attribute-shorthand': ['error']}">
 
 ```vue
 <template>
@@ -90,7 +89,7 @@ Default options is `"always"`.
 
 ### `"never"`
 
-<eslint-code-block fix :rules="{'vue/prefer-true-attribute-shorthand': ['error', 'never']}">
+<eslint-code-block :rules="{'vue/prefer-true-attribute-shorthand': ['error', 'never']}">
 
 ```vue
 <template>
diff --git a/lib/rules/prefer-true-attribute-shorthand.js b/lib/rules/prefer-true-attribute-shorthand.js
index 19a98a351..00fe95489 100644
--- a/lib/rules/prefer-true-attribute-shorthand.js
+++ b/lib/rules/prefer-true-attribute-shorthand.js
@@ -23,7 +23,7 @@ module.exports = {
       categories: undefined,
       url: 'https://eslint.vuejs.org/rules/prefer-true-attribute-shorthand.html'
     },
-    fixable: 'code',
+    fixable: null,
     hasSuggestions: true,
     schema: [{ enum: ['always', 'never'] }],
     messages: {