fix: should not throw when a plugin listed in optionalDependencies is not installed

haoqunjiang · haoqunjiang · commit 7ea080b9d1b9 · 2018-10-13T00:25:42.000+08:00
This bug is due to `normalize-package-data` (required by `read-pkg`)
adding `optionalDependencies` to `dependencies`.
diff --git a/packages/@vue/cli-service/lib/Service.js b/packages/@vue/cli-service/lib/Service.js
@@ -159,7 +159,23 @@ module.exports = class Service {
       const projectPlugins = Object.keys(this.pkg.devDependencies || {})
         .concat(Object.keys(this.pkg.dependencies || {}))
         .filter(isPlugin)
-        .map(idToPlugin)
+        .map(id => {
+          if (
+            this.pkg.optionalDependencies &&
+            id in this.pkg.optionalDependencies
+          ) {
+            let apply = () => {}
+            try {
+              apply = require(id)
+            } catch (e) {
+              warn(`Optional dependency ${id} is not installed.`)
+            }
+
+            return { id, apply }
+          } else {
+            return idToPlugin(id)
+          }
+        })
       plugins = builtInPlugins.concat(projectPlugins)
     }
 
