Fix bug when script tags are missing and multiple parsers are used (#126)

Author: ota-meshi

scripts/update-fixtures-ast.js

@@ -223,6 +223,7 @@ for (const name of TARGETS) {
     const tokenRangesPath = path.join(ROOT, `${name}/token-ranges.json`)
     const treePath = path.join(ROOT, `${name}/tree.json`)
     const scopePath = path.join(ROOT, `${name}/scope.json`)
+    const servicesPath = path.join(ROOT, `${name}/services.json`)
     const source = fs.readFileSync(sourcePath, "utf8")
     const options = Object.assign(
         { filePath: sourcePath },
@@ -249,4 +250,10 @@ for (const name of TARGETS) {
             scopeToJSON(actual.scopeManager || analyze(actual.ast, options))
         )
     }
+    if (fs.existsSync(servicesPath)) {
+        fs.writeFileSync(
+            servicesPath,
+            JSON.stringify(Object.keys(actual.services).sort(), null, 4)
+        )
+    }
 }

src/index.ts

@@ -115,6 +115,7 @@ export function parseForESLint(
             result = parseScript("", {
                 ...options,
                 ecmaVersion: options.ecmaVersion || DEFAULT_ECMA_VERSION,
+                parser: scriptParser,
             })
         } else if (
             scripts.length === 2 &&

test/ast.js

@@ -175,13 +175,17 @@ describe("Template AST", () => {
         const sourcePath = path.join(ROOT, `${name}/source.vue`)
         const optionsPath = path.join(ROOT, `${name}/parser-options.json`)
         const requirementsPath = path.join(ROOT, `${name}/requirements.json`)
+        const servicesPath = path.join(ROOT, `${name}/services.json`)
         const source = fs.readFileSync(sourcePath, "utf8")
         const parserOptions = fs.existsSync(optionsPath)
             ? JSON.parse(fs.readFileSync(optionsPath, "utf8"))
             : {}
         const requirements = fs.existsSync(requirementsPath)
             ? JSON.parse(fs.readFileSync(requirementsPath, "utf8"))
             : {}
+        const services = fs.existsSync(servicesPath)
+            ? JSON.parse(fs.readFileSync(servicesPath, "utf8"))
+            : null
         const options = Object.assign(
             { filePath: sourcePath },
             PARSER_OPTIONS,
@@ -293,6 +297,15 @@ describe("Template AST", () => {
             it("should have correct parent properties.", () => {
                 validateParent(source, parserOptions)
             })
+
+            if (services) {
+                it("should have correct services.", () => {
+                    assert.deepStrictEqual(
+                        Object.keys(actual.services).sort(),
+                        services
+                    )
+                })
+            }
         })
     }
 })

test/fixtures/ast/parser-option-multiple-parsers-1/services.json

@@ -0,0 +1,11 @@
+[
+    "defineCustomBlocksVisitor",
+    "defineDocumentVisitor",
+    "defineTemplateBodyVisitor",
+    "esTreeNodeToTSNodeMap",
+    "getDocumentFragment",
+    "getTemplateBodyTokenStore",
+    "hasFullTypeInformation",
+    "program",
+    "tsNodeToESTreeNodeMap"
+]

test/fixtures/ast/parser-option-multiple-parsers-2/services.json

@@ -0,0 +1,7 @@
+[
+    "defineCustomBlocksVisitor",
+    "defineDocumentVisitor",
+    "defineTemplateBodyVisitor",
+    "getDocumentFragment",
+    "getTemplateBodyTokenStore"
+]

test/fixtures/ast/parser-option-multiple-parsers-3/services.json

@@ -0,0 +1,11 @@
+[
+    "defineCustomBlocksVisitor",
+    "defineDocumentVisitor",
+    "defineTemplateBodyVisitor",
+    "esTreeNodeToTSNodeMap",
+    "getDocumentFragment",
+    "getTemplateBodyTokenStore",
+    "hasFullTypeInformation",
+    "program",
+    "tsNodeToESTreeNodeMap"
+]