fix: remove node_modules files

Author: Baitu Huang

lib/utils.js

@@ -38,15 +38,14 @@ exports.toFunc = function toFunc (exp, useWith = true) {
 const Module = require('module')
 const path = require('path')
 const clearModule = require('clear-module')
+const slash = require('slash')
 exports.getDependencies = function getDependencies (callback, deps) {
   deps = new Set(deps)
   const resolveFilename = Module._resolveFilename
   Module._resolveFilename = function _resolveFilename (req, ...args) {
     const filename = resolveFilename.call(this, req, ...args)
-    if (path.isAbsolute(req) && !/node_modules/.test(req)) {
-      deps.add(filename)
-    }
-    if (/^\.{1,2}\//.test(req)) {
+    const unixFn = slash(filename)
+    if ((path.isAbsolute(req) || /^\.{1,2}\//.test(req)) && !/\/node_modules\//.test(unixFn)) {
       deps.add(filename)
     }
     return filename

package-lock.json

@@ -46,11 +46,12 @@
     "chalk": "^2.4.1",
     "clear-module": "^3.2.0",
     "find-cache-dir": "^3.0.0",
+    "fs-extra": "^7.0.1",
     "loader-utils": "^1.2.0",
     "schema-utils": "^1.0.0",
     "semver": "^6.0.0",
-    "watchpack": "^1.6.0",
-    "fs-extra": "^7.0.1"
+    "slash": "^2.0.0",
+    "watchpack": "^1.6.0"
   },
   "peerDependencies": {
     "webpack": "^4.1.0"

package.json

@@ -10,7 +10,7 @@ module.exports = async function loadCompiler (version = 'latest') {
   if (fse.pathExistsSync(cachePath)) {
     try {
       const compiler = require(cachePath)
-      if ('parseComponent' in compiler) {
+      if (typeof compiler.parseComponent === 'function') {
         return compiler
       }
     } catch (e) {}

test/utils/loadCompiler.js

@@ -2,11 +2,10 @@ const fs = require('fs')
 const path = require('path')
 
 module.exports = (dir) => {
-  const templates = {}
-  fs.readdirSync(dir).map(file => {
+  return fs.readdirSync(dir).reduce((mp, file) => {
     if (path.extname(file) === '.vue' && file[0] !== '_') {
-      templates[file.split('.')[0]] = fs.readFileSync(path.join(dir, file), 'utf8')
+      mp[file.split('.')[0]] = fs.readFileSync(path.join(dir, file), 'utf8')
     }
-  })
-  return templates
+    return mp
+  }, {})
 }

test/utils/loadTemplates.js

@@ -1,4 +1,8 @@
 const Module = require('module')
+const test = require('tape')
+const chalk = require('chalk')
+const clearModule = require('clear-module')
+
 const rfp = Module._resolveFilename
 const mmap = [
   'webpack', 'vue-loader', 'vue-loader/lib/plugin', 'vue-template-compiler/package.json'
@@ -12,10 +16,9 @@ Module._resolveFilename = function _resolveFilename (req, ...args) {
   return rfp.call(this, req, ...args)
 }
 
+test.onFinish(() => { Module._resolveFilename = rfp })
+
 const { build, dev } = require('./webpack-test')
-const test = require('tape')
-const chalk = require('chalk')
-const clearModule = require('clear-module')
 
 const has = (flag, result, watch, useVue = true) => {
   return new RegExp(`(${useVue ? 'template|' : ''}${watch ? '' : 'script|'}style):${flag}{5,}`).test(result)