refactor: refactor shared utils + move openBrowser

Author: yyx990803

packages/@vue/cli-service/lib/commands/serve.js

@@ -2,6 +2,7 @@ const {
   info,
   error,
   hasYarn,
+  openBrowser,
   clearConsole
 } = require('@vue/cli-shared-utils')
 
@@ -37,7 +38,6 @@ module.exports = (api, options) => {
     const webpack = require('webpack')
     const WebpackDevServer = require('webpack-dev-server')
     const portfinder = require('portfinder')
-    const openBrowser = require('../util/openBrowser')
     const prepareURLs = require('../util/prepareURLs')
     const prepareProxy = require('../util/prepareProxy')
     const launchEditorMiddleware = require('launch-editor-middleware')

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

@@ -42,7 +42,6 @@
     "javascript-stringify": "^1.6.0",
     "launch-editor-middleware": "^2.2.0",
     "minimist": "^1.2.0",
-    "opn": "^5.2.0",
     "optimize-css-assets-webpack-plugin": "^3.2.0",
     "ora": "^1.3.0",
     "portfinder": "^1.0.13",

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

@@ -1,4 +1,10 @@
-Object.assign(exports, require('./env'))
-Object.assign(exports, require('./logger'))
-Object.assign(exports, require('./validate'))
-Object.assign(exports, require('./linkBin'))
+[
+  'env',
+  'linkBin',
+  'logger',
+  'spinner',
+  'validate',
+  'openBrowser'
+].forEach(m => {
+  Object.assign(exports, require(`./lib/${m}`))
+})

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

@@ -0,0 +1,12 @@
+module.exports = function silence (exports) {
+  const logs = {}
+  Object.keys(exports).forEach(key => {
+    if (key !== 'error') {
+      exports[key] = (...args) => {
+        if (!logs[key]) logs[key] = []
+        logs[key].push(args)
+      }
+    }
+  })
+  exports.logs = logs
+}

packages/@vue/cli-shared-utils/env.js renamed to packages/@vue/cli-shared-utils/lib/env.js

@@ -1,10 +1,7 @@
 const chalk = require('chalk')
-const spinner = require('./spinner')
 const readline = require('readline')
 const padStart = require('string.prototype.padstart')
 
-Object.assign(exports, spinner)
-
 const format = (label, msg) => {
   return msg.split('\n').map((line, i) => {
     return i === 0
@@ -48,14 +45,5 @@ exports.clearConsole = title => {
 
 // silent all logs except errors during tests and keep record
 if (process.env.VUE_CLI_TEST) {
-  const logs = {}
-  Object.keys(exports).forEach(key => {
-    if (key !== 'error') {
-      exports[key] = (...args) => {
-        if (!logs[key]) logs[key] = []
-        logs[key].push(args)
-      }
-    }
-  })
-  exports.logs = logs
+  require('./_silence')(exports)
 }

packages/@vue/cli-shared-utils/linkBin.js renamed to packages/@vue/cli-shared-utils/lib/linkBin.js

@@ -6,10 +6,10 @@
  * https://github.com/facebookincubator/create-react-app/blob/master/LICENSE
  */
 
+const opn = require('opn')
+const execa = require('execa')
 const chalk = require('chalk')
 const execSync = require('child_process').execSync
-const execa = require('execa')
-const opn = require('opn')
 
 // https://github.com/sindresorhus/opn#app
 const OSX_CHROME = 'google chrome'
@@ -107,7 +107,7 @@ function startBrowserProcess (browser, url) {
  * Reads the BROWSER evironment variable and decides what to do with it. Returns
  * true if it opened a browser or ran a node.js script, otherwise false.
  */
-function openBrowser (url) {
+exports.openBrowser = function (url) {
   const { action, value } = getBrowserEnv()
   switch (action) {
   case Actions.NONE:
@@ -121,5 +121,3 @@ function openBrowser (url) {
     throw new Error('Not implemented.')
   }
 }
-
-module.exports = openBrowser

packages/@vue/cli-shared-utils/logger.js renamed to packages/@vue/cli-shared-utils/lib/logger.js

@@ -42,3 +42,8 @@ exports.pauseSpinner = () => {
 exports.resumeSpinner = () => {
   spinner.start()
 }
+
+// silent all logs except errors during tests and keep record
+if (process.env.VUE_CLI_TEST) {
+  require('./_silence')(exports)
+}