refactor: launch + more helpful error message

Author: Guillaume Chau

packages/@vue/cli-plugin-eslint/package.json

@@ -26,7 +26,6 @@
     "babel-eslint": "^8.2.5",
     "eslint": "^4.19.1",
     "eslint-loader": "^2.0.0",
-    "eslint-plugin-vue": "^4.5.0",
-    "launch-editor": "^2.2.1"
+    "eslint-plugin-vue": "^4.5.0"
   }
 }

packages/@vue/cli-plugin-eslint/ui/index.js

@@ -22,7 +22,7 @@ module.exports = api => {
         label: 'org.vue.eslint.suggestions.open-eslintrc.label',
         handler () {
           const file = config.foundFiles.eslint.path
-          const launch = require('launch-editor')
+          const { launch } = require('@vue/cli-shared-utils')
           launch(file)
           return {
             keep: true

packages/@vue/cli-plugin-pwa/package.json

@@ -22,7 +22,7 @@
     "access": "public"
   },
   "dependencies": {
-    "launch-editor": "^2.2.1",
+    "@vue/cli-shared-utils": "^3.0.0-rc.10",
     "workbox-webpack-plugin": "^3.3.1"
   },
   "devDependencies": {

packages/@vue/cli-plugin-pwa/ui.js

@@ -131,7 +131,7 @@ module.exports = api => {
           label: 'org.vue.pwa.suggestions.open-vue.label',
           handler () {
             const file = config.foundFiles.vue.path
-            const launch = require('launch-editor')
+            const { launch } = require('@vue/cli-shared-utils')
             launch(file)
             return {
               keep: true
@@ -148,7 +148,7 @@ module.exports = api => {
           label: 'org.vue.pwa.suggestions.open-manifest.label',
           handler () {
             const file = config.foundFiles.manifest.path
-            const launch = require('launch-editor')
+            const { launch } = require('@vue/cli-shared-utils')
             launch(file)
             return {
               keep: true

packages/@vue/cli-shared-utils/index.js

@@ -7,6 +7,7 @@
   'object',
   'openBrowser',
   'pluginResolution',
+  'launch',
   'request',
   'spinner',
   'validate'

packages/@vue/cli-shared-utils/lib/launch.js

@@ -0,0 +1,16 @@
+const launch = require('launch-editor')
+
+exports.launch = (...args) => {
+  const file = args[0]
+  console.log(`Opening ${file}...`)
+  let cb = args[args.length - 1]
+  if (typeof cb !== 'function') {
+    cb = null
+  }
+  launch(...args, (fileName, errorMessage) => {
+    console.error(`Unable to open '${fileName}'`, errorMessage)
+    console.log(`Try setting the EDITOR env variable. More info: https://github.com/yyx990803/launch-editor`)
+
+    if (cb) cb(fileName, errorMessage)
+  })
+}

packages/@vue/cli-shared-utils/package.json

@@ -25,6 +25,7 @@
     "node-ipc": "^9.1.1",
     "opn": "^5.3.0",
     "ora": "^2.1.0",
+    "launch-editor": "^2.2.1",
     "request": "^2.87.0",
     "request-promise-native": "^1.0.5",
     "semver": "^5.5.0",

packages/@vue/cli-ui/apollo-server/connectors/files.js

@@ -1,4 +1,4 @@
-const launch = require('launch-editor')
+const { launch } = require('@vue/cli-shared-utils')
 const path = require('path')
 // Connectors
 const cwd = require('./cwd')
@@ -22,9 +22,7 @@ async function openInEditor (input, context) {
     message: `Opening file '${query}' in code editor...`,
     type: 'info'
   }, context)
-  launch(query, (fileName, errorMessage) => {
-    console.error(`Unable to open '${fileName}'`, errorMessage)
-  })
+  launch(query)
   return true
 }
 

packages/@vue/cli-ui/package.json

@@ -41,7 +41,6 @@
     "graphql-type-json": "^0.2.1",
     "javascript-stringify": "^1.6.0",
     "js-yaml": "^3.12.0",
-    "launch-editor": "^2.2.1",
     "lodash.merge": "^4.6.1",
     "lowdb": "^1.0.0",
     "lru-cache": "^4.1.3",

packages/@vue/cli-ui/ui-defaults/suggestions.js

@@ -52,7 +52,7 @@ module.exports = api => {
           handler () {
             const file = config.foundFiles.vue.path
             console.log('open', file)
-            const launch = require('launch-editor')
+            const { launch } = require('@vue/cli-shared-utils')
             launch(file)
             return {
               keep: true