From 63d9991be66ab9b4ba0bfca5d63c1ac03baea4d9 Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Thu, 31 Jan 2019 23:43:18 +0900
Subject: [PATCH 1/4] =?UTF-8?q?=E2=AD=90=EF=B8=8FNew:=20Add=20`vue/space-i?=
 =?UTF-8?q?n-parens`=20rule?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 docs/rules/space-in-parens.md      |  23 ++++
 lib/configs/no-layout-rules.js     |   1 +
 lib/rules/space-in-parens.js       |  12 ++
 tests/lib/rules/space-in-parens.js | 195 +++++++++++++++++++++++++++++
 4 files changed, 231 insertions(+)
 create mode 100644 docs/rules/space-in-parens.md
 create mode 100644 lib/rules/space-in-parens.js
 create mode 100644 tests/lib/rules/space-in-parens.js

diff --git a/docs/rules/space-in-parens.md b/docs/rules/space-in-parens.md
new file mode 100644
index 000000000..3b39305dc
--- /dev/null
+++ b/docs/rules/space-in-parens.md
@@ -0,0 +1,23 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/space-in-parens
+description: enforce consistent spacing inside parentheses
+---
+# vue/space-in-parens
+> enforce consistent spacing inside parentheses
+
+- :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.
+
+This rule is the same rule as core [space-in-parens] rule but it applies to the expressions in `<template>`.
+
+## :books: Further reading
+
+- [space-in-parens]
+
+[space-in-parens]: https://eslint.org/docs/rules/space-in-parens
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/space-in-parens.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/space-in-parens.js)
diff --git a/lib/configs/no-layout-rules.js b/lib/configs/no-layout-rules.js
index 3b26ae1be..5a9487caa 100644
--- a/lib/configs/no-layout-rules.js
+++ b/lib/configs/no-layout-rules.js
@@ -32,6 +32,7 @@ module.exports = {
     'vue/padding-line-between-blocks': 'off',
     'vue/script-indent': 'off',
     'vue/singleline-html-element-content-newline': 'off',
+    'vue/space-in-parens': 'off',
     'vue/space-infix-ops': 'off',
     'vue/space-unary-ops': 'off',
     'vue/template-curly-spacing': 'off'
diff --git a/lib/rules/space-in-parens.js b/lib/rules/space-in-parens.js
new file mode 100644
index 000000000..f3b03e22f
--- /dev/null
+++ b/lib/rules/space-in-parens.js
@@ -0,0 +1,12 @@
+/**
+ * @author Yosuke Ota
+ */
+'use strict'
+
+const { wrapCoreRule } = require('../utils')
+
+// eslint-disable-next-line no-invalid-meta, no-invalid-meta-docs-categories
+module.exports = wrapCoreRule(
+  require('eslint/lib/rules/space-in-parens'),
+  { skipDynamicArguments: true, skipDynamicArgumentsReport: true }
+)
diff --git a/tests/lib/rules/space-in-parens.js b/tests/lib/rules/space-in-parens.js
new file mode 100644
index 000000000..3b02d48f8
--- /dev/null
+++ b/tests/lib/rules/space-in-parens.js
@@ -0,0 +1,195 @@
+/**
+ * @author Yosuke Ota
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/space-in-parens')
+
+const tester = new RuleTester({
+  parser: require.resolve('vue-eslint-parser'),
+  parserOptions: { ecmaVersion: 2015 }
+})
+
+tester.run('space-in-parens', rule, {
+  valid: [
+    `<template>
+      <button
+        @click="foo(arg)"
+      />
+    </template>`,
+    {
+      code: `
+      <template>
+        <button
+          @click="foo( arg )"
+        />
+      </template>`,
+      options: ['always']
+    },
+    `
+    <template>
+      <button
+        :[foo(arg)]="foo(arg)"
+      />
+    </template>`,
+    {
+      code: `
+      <template>
+        <button
+          :[foo(arg)]="foo( arg )"
+        />
+      </template>`,
+      options: ['always']
+    }
+  ],
+  invalid: [
+    {
+      code: `
+      <template>
+        <button
+          @click="foo( arg )"
+        />
+      </template>`,
+      output: `
+      <template>
+        <button
+          @click="foo(arg)"
+        />
+      </template>`,
+      errors: [
+        {
+          messageId: 'rejectedOpeningSpace',
+          line: 4
+        },
+        {
+          messageId: 'rejectedClosingSpace',
+          line: 4
+        }
+      ]
+    },
+    {
+      code: `
+      <template>
+        <button
+          @click="foo(arg)"
+        />
+      </template>`,
+      options: ['always'],
+      output: `
+      <template>
+        <button
+          @click="foo( arg )"
+        />
+      </template>`,
+      errors: [
+        {
+          messageId: 'missingOpeningSpace',
+          line: 4
+        },
+        {
+          messageId: 'missingClosingSpace',
+          line: 4
+        }
+      ]
+    },
+    {
+      code: `
+      <template>
+        <input
+          :value="( 1 + 2 ) + 3"
+        >
+      </template>`,
+      output: `
+      <template>
+        <input
+          :value="(1 + 2) + 3"
+        >
+      </template>`,
+      errors: [
+        {
+          messageId: 'rejectedOpeningSpace',
+          line: 4
+        },
+        {
+          messageId: 'rejectedClosingSpace',
+          line: 4
+        }
+      ]
+    },
+    {
+      code: `
+      <template>
+        <input
+          :value="(1 + 2) + 3"
+        >
+      </template>`,
+      options: ['always'],
+      output: `
+      <template>
+        <input
+          :value="( 1 + 2 ) + 3"
+        >
+      </template>`,
+      errors: [
+        {
+          messageId: 'missingOpeningSpace',
+          line: 4
+        },
+        {
+          messageId: 'missingClosingSpace',
+          line: 4
+        }
+      ]
+    },
+    {
+      code: `
+      <template>
+        <input
+          :[(1+2)]="( 1 + 2 ) + 3"
+        >
+      </template>`,
+      output: `
+      <template>
+        <input
+          :[(1+2)]="(1 + 2) + 3"
+        >
+      </template>`,
+      errors: [
+        {
+          messageId: 'rejectedOpeningSpace',
+          line: 4
+        },
+        {
+          messageId: 'rejectedClosingSpace',
+          line: 4
+        }
+      ]
+    },
+    {
+      code: `
+      <template>
+        <input
+          :[(1+2)]="(1 + 2) + 3"
+        >
+      </template>`,
+      options: ['always'],
+      output: `
+      <template>
+        <input
+          :[(1+2)]="( 1 + 2 ) + 3"
+        >
+      </template>`,
+      errors: [
+        {
+          messageId: 'missingOpeningSpace',
+          line: 4
+        },
+        {
+          messageId: 'missingClosingSpace',
+          line: 4
+        }
+      ]
+    }
+  ]
+})

From 143bf68ee3480f929343413d3b37db4b41eba1f0 Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Sat, 23 May 2020 11:25:32 +0900
Subject: [PATCH 2/4] update

---
 lib/utils/index.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/lib/utils/index.js b/lib/utils/index.js
index 698a9a047..4181e7521 100644
--- a/lib/utils/index.js
+++ b/lib/utils/index.js
@@ -117,7 +117,9 @@ function wrapContextToOverrideReportMethodToSkipDynamicArgument (context) {
     report (descriptor, ...args) {
       let range = null
       if (descriptor.loc) {
-        range = [sourceCode.getIndexFromLoc(descriptor.loc.start), sourceCode.getIndexFromLoc(descriptor.loc.end)]
+        const startLoc = descriptor.loc.start
+        const endLoc = descriptor.loc.end || startLoc
+        range = [sourceCode.getIndexFromLoc(startLoc), sourceCode.getIndexFromLoc(endLoc)]
       } else if (descriptor.node) {
         range = descriptor.node.range
       }

From 39121092e3a2399ff0ae3261dc33d555baeb00d1 Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Sat, 23 May 2020 11:34:39 +0900
Subject: [PATCH 3/4] update

---
 lib/utils/index.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/lib/utils/index.js b/lib/utils/index.js
index 4181e7521..0f68bf11f 100644
--- a/lib/utils/index.js
+++ b/lib/utils/index.js
@@ -117,7 +117,7 @@ function wrapContextToOverrideReportMethodToSkipDynamicArgument (context) {
     report (descriptor, ...args) {
       let range = null
       if (descriptor.loc) {
-        const startLoc = descriptor.loc.start
+        const startLoc = isLoc(descriptor.loc.start) ? descriptor.loc.start : descriptor.loc
         const endLoc = descriptor.loc.end || startLoc
         range = [sourceCode.getIndexFromLoc(startLoc), sourceCode.getIndexFromLoc(endLoc)]
       } else if (descriptor.node) {
@@ -133,6 +133,10 @@ function wrapContextToOverrideReportMethodToSkipDynamicArgument (context) {
       context.report(descriptor, ...args)
     }
   }
+
+  function isLoc (loc) {
+    return loc && typeof loc === 'object' && typeof loc.line === 'number' && typeof loc.column === 'number'
+  }
 }
 
 // ------------------------------------------------------------------------------

From ff721785be35115cff875f32cda2a9af2bc7e9c2 Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Sat, 23 May 2020 11:46:25 +0900
Subject: [PATCH 4/4] update

---
 tests/lib/rules/space-in-parens.js | 60 +++++++++++++++++-------------
 1 file changed, 35 insertions(+), 25 deletions(-)

diff --git a/tests/lib/rules/space-in-parens.js b/tests/lib/rules/space-in-parens.js
index 3b02d48f8..11e83591d 100644
--- a/tests/lib/rules/space-in-parens.js
+++ b/tests/lib/rules/space-in-parens.js
@@ -3,9 +3,19 @@
  */
 'use strict'
 
-const RuleTester = require('eslint').RuleTester
+const { RuleTester, CLIEngine } = require('eslint')
+const semver = require('semver')
 const rule = require('../../../lib/rules/space-in-parens')
 
+const errorMessage = semver.lt(CLIEngine.version, '6.4.0')
+  ? (obj) => {
+    const messageId = obj.messageId
+    delete obj.messageId
+    obj.message = messageId.startsWith('missing') ? 'There must be a space inside this paren.' : 'There should be no spaces inside this paren.'
+    return obj
+  }
+  : obj => obj
+
 const tester = new RuleTester({
   parser: require.resolve('vue-eslint-parser'),
   parserOptions: { ecmaVersion: 2015 }
@@ -58,14 +68,14 @@ tester.run('space-in-parens', rule, {
         />
       </template>`,
       errors: [
-        {
+        errorMessage({
           messageId: 'rejectedOpeningSpace',
           line: 4
-        },
-        {
+        }),
+        errorMessage({
           messageId: 'rejectedClosingSpace',
           line: 4
-        }
+        })
       ]
     },
     {
@@ -83,14 +93,14 @@ tester.run('space-in-parens', rule, {
         />
       </template>`,
       errors: [
-        {
+        errorMessage({
           messageId: 'missingOpeningSpace',
           line: 4
-        },
-        {
+        }),
+        errorMessage({
           messageId: 'missingClosingSpace',
           line: 4
-        }
+        })
       ]
     },
     {
@@ -107,14 +117,14 @@ tester.run('space-in-parens', rule, {
         >
       </template>`,
       errors: [
-        {
+        errorMessage({
           messageId: 'rejectedOpeningSpace',
           line: 4
-        },
-        {
+        }),
+        errorMessage({
           messageId: 'rejectedClosingSpace',
           line: 4
-        }
+        })
       ]
     },
     {
@@ -132,14 +142,14 @@ tester.run('space-in-parens', rule, {
         >
       </template>`,
       errors: [
-        {
+        errorMessage({
           messageId: 'missingOpeningSpace',
           line: 4
-        },
-        {
+        }),
+        errorMessage({
           messageId: 'missingClosingSpace',
           line: 4
-        }
+        })
       ]
     },
     {
@@ -156,14 +166,14 @@ tester.run('space-in-parens', rule, {
         >
       </template>`,
       errors: [
-        {
+        errorMessage({
           messageId: 'rejectedOpeningSpace',
           line: 4
-        },
-        {
+        }),
+        errorMessage({
           messageId: 'rejectedClosingSpace',
           line: 4
-        }
+        })
       ]
     },
     {
@@ -181,14 +191,14 @@ tester.run('space-in-parens', rule, {
         >
       </template>`,
       errors: [
-        {
+        errorMessage({
           messageId: 'missingOpeningSpace',
           line: 4
-        },
-        {
+        }),
+        errorMessage({
           messageId: 'missingClosingSpace',
           line: 4
-        }
+        })
       ]
     }
   ]