fix: plugin sub node_module directory should have higher priority i…

…n `resolveLoader.modules` in case of interference of unwanted top level dependency hoisting
vuejs · Sep 28, 2018 · 47a28e0 · 47a28e0
1 parent fed948a
commit 47a28e0
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/packages/@vue/cli-plugin-babel/index.js b/packages/@vue/cli-plugin-babel/index.js
@@ -5,7 +5,7 @@ module.exports = (api, options) => {
   const cliServicePath = require('path').dirname(require.resolve('@vue/cli-service'))
 
   api.chainWebpack(webpackConfig => {
-    webpackConfig.resolveLoader.modules.add(path.join(__dirname, 'node_modules'))
+    webpackConfig.resolveLoader.modules.prepend(path.join(__dirname, 'node_modules'))
 
     const jsRule = webpackConfig.module
       .rule('js')

diff --git a/packages/@vue/cli-plugin-eslint/index.js b/packages/@vue/cli-plugin-eslint/index.js
@@ -24,7 +24,7 @@ module.exports = (api, options) => {
     )
 
     api.chainWebpack(webpackConfig => {
-      webpackConfig.resolveLoader.modules.add(path.join(__dirname, 'node_modules'))
+      webpackConfig.resolveLoader.modules.prepend(path.join(__dirname, 'node_modules'))
 
       webpackConfig.module
         .rule('eslint')

diff --git a/packages/@vue/cli-plugin-typescript/index.js b/packages/@vue/cli-plugin-typescript/index.js
@@ -5,7 +5,7 @@ module.exports = (api, options) => {
   const useThreads = process.env.NODE_ENV === 'production' && options.parallel
 
   api.chainWebpack(config => {
-    config.resolveLoader.modules.add(path.join(__dirname, 'node_modules'))
+    config.resolveLoader.modules.prepend(path.join(__dirname, 'node_modules'))
 
     if (!options.pages) {
       config.entry('app')