From 9b947b11fac73ac5baac1fee35c19b47afb21f1c Mon Sep 17 00:00:00 2001
From: Toru Nagashima <star.ctor@gmail.com>
Date: Sat, 21 Jul 2018 19:47:00 +0900
Subject: [PATCH] Update: 'parser:false' skip parsing '<script>' (refs
 vuejs/eslint-plugin-vue#446)

---
 README.md              | 12 ++++++++----
 src/index.ts           | 10 ++++++----
 test/parser-options.js | 40 ++++++++++++++++++++++++++++++++++++++++
 3 files changed, 54 insertions(+), 8 deletions(-)
 create mode 100644 test/parser-options.js

diff --git a/README.md b/README.md
index d151bb18..61665615 100644
--- a/README.md
+++ b/README.md
@@ -49,18 +49,19 @@ For example:
     "parser": "vue-eslint-parser",
     "parserOptions": {
         "sourceType": "module",
-        "ecmaVersion": 2017,
+        "ecmaVersion": 2018,
         "ecmaFeatures": {
             "globalReturn": false,
             "impliedStrict": false,
-            "jsx": false,
-            "experimentalObjectRestSpread": false
+            "jsx": false
         }
     }
 }
 ```
 
-Also, you can use `parser` property to specify a custom parser to parse `<script>` tags.
+### parserOptions.parser
+
+You can use `parserOptions.parser` property to specify a custom parser to parse `<script>` tags.
 Other properties than parser would be given to the specified parser.
 For example:
 
@@ -87,6 +88,9 @@ For example:
 - If you use with `babel-eslint`, use `babel-eslint@>=8.1.1`.
 - If you use `typescript-eslint-parser`, the location of original nodes can be wrong. Waiting for `typescript-eslint-parser` to support [parseResult.visitorKeys](https://eslint.org/docs/developer-guide/working-with-plugins#working-with-custom-parsers).
 
+If the `parserOptions.parser` is `false`, the `vue-eslint-parser` skips parsing `<script>` tags completely.
+This is useful for people who use the language ESLint community doesn't provide custom parser implementation.
+
 ## 🎇 Usage for custom rules / plugins
 
 - This parser provides `parserServices` to traverse `<template>`.
diff --git a/src/index.ts b/src/index.ts
index fdd2d066..009700b3 100644
--- a/src/index.ts
+++ b/src/index.ts
@@ -93,6 +93,7 @@ export function parseForESLint(
     if (!isVueFile(code, options)) {
         result = parseScript(code, options)
     } else {
+        const skipParsingScript = options.parser === false
         const tokenizer = new HTMLTokenizer(code)
         const rootAST = new HTMLParser(tokenizer, options).parse()
         const locationCalcurator = new LocationCalculator(
@@ -112,10 +113,11 @@ export function parseForESLint(
                 ? Object.assign(template, concreteInfo)
                 : undefined
 
-        result =
-            script != null
-                ? parseScriptElement(script, locationCalcurator, options)
-                : parseScript("", options)
+        if (skipParsingScript || script == null) {
+            result = parseScript("", options)
+        } else {
+            result = parseScriptElement(script, locationCalcurator, options)
+        }
 
         result.ast.templateBody = templateBody
     }
diff --git a/test/parser-options.js b/test/parser-options.js
new file mode 100644
index 00000000..769dcfe4
--- /dev/null
+++ b/test/parser-options.js
@@ -0,0 +1,40 @@
+/**
+ * @author Toru Nagashima <https://github.com/mysticatea>
+ * See LICENSE file in root directory for full license.
+ */
+"use strict"
+
+const assert = require("assert")
+const { rules } = require("@mysticatea/eslint-plugin")
+const parseForESLint = require("..").parseForESLint
+const eslint = require("./fixtures/eslint")
+const Linter = eslint.Linter
+
+describe.only("parserOptions", () => {
+    describe("parser", () => {
+        const linter = new Linter()
+        linter.defineParser("vue-eslint-parser", { parseForESLint })
+        linter.defineRule(
+            "vue/valid-template-root",
+            rules["vue/valid-template-root"]
+        )
+
+        it("false then skip parsing '<script>'.", () => {
+            const code = `<template>Hello</template>
+<script>This is syntax error</script>`
+            const config = {
+                parser: "vue-eslint-parser",
+                parserOptions: {
+                    parser: false,
+                },
+                rules: {
+                    "vue/valid-template-root": "error",
+                },
+            }
+            const messages = linter.verify(code, config, "test.vue")
+
+            assert.strictEqual(messages.length, 1)
+            assert.strictEqual(messages[0].ruleId, "vue/valid-template-root")
+        })
+    })
+})