fix: 修复一些问题，优化代码

Author: cklwblove

build/index.ts

@@ -1,20 +1,28 @@
-const {run} = require('runjs');
+'use strict';
+
+const { sh, cli } = require('tasksfile');
 const chalk = require('chalk');
 const rawArgv = process.argv.slice(2);
 const args = rawArgv.join(' ');
 
 // 便于捕捉 build 之后的错误，然后进行自定义处理
 // 配合 jenkins 执行 job
-run(`vue-cli-service build ${args}`, {
-  async: true,
-  stdio: 'inherit'
-}).then((output) => {
-  console.log(chalk.cyan(output));
-}).catch((err) => {
-  console.error('\n');
-  console.error(chalk.magenta('编译打包出错了 ~~~~(>_<)~~~~ \n'));
-  console.error(chalk.magenta('具体错误信息如下 \n'));
-  console.error(chalk.red(`${err}.\n`));
-  console.log(chalk.red('  Build failed with errors.\n'));
-  process.exit(1);
-});
+function command() {
+  sh(`vue-cli-service build ${args}`, {
+    async: true,
+    stdio: 'inherit',
+  })
+    .then((output) => {
+      console.log(chalk.cyan(output || ''));
+    })
+    .catch((err) => {
+      console.error('\n');
+      console.error(chalk.magenta('编译打包出错了 ~~~~(>_<)~~~~ \n'));
+      console.error(chalk.magenta('具体错误信息如下 \n'));
+      console.error(chalk.red(`${err}.\n`));
+      console.log(chalk.red('  Build failed with errors.\n'));
+      process.exit(1);
+    });
+}
+
+command();

build/utils.js

@@ -10,12 +10,11 @@ exports.cssLoaders = function (options) {
   const cssLoader = {
     loader: 'css-loader',
     options: {
-      minimize: process.env.NODE_ENV === 'production' || process.env.NODE_ENV === 'dll',
       sourceMap: options.sourceMap
     }
   }
 
-  var postcssLoader = {
+  const postcssLoader = {
     loader: 'postcss-loader',
     options: {
       sourceMap: options.sourceMap
@@ -35,7 +34,7 @@ exports.cssLoaders = function (options) {
           // you can specify a publicPath here
           // by default it use publicPath in webpackOptions.output
           // 解决图片作为背景引入时，路径不对的问题
-          publicPath: '../../'
+          publicPath: './'
         }
       })
     } else {
@@ -109,5 +108,3 @@ exports.createNotifierCallback = function () {
 exports.resolve = function (dir) {
   return path.join(__dirname, '..', dir)
 }
-
-

build/zip.ts

@@ -1,51 +1,57 @@
-'use strict';
-const path = require('path');
+#!/usr/bin/env node
+
 const fs = require('fs');
+const path = require('path');
 const archiver = require('archiver');
-const utils = require('@liwb/cloud-utils');
-const pkg = require('../package.json');
-const outDir = `${path.resolve(__dirname, '../')}/${pkg.name}_${utils.formatDate(new Date(), 'yyyy-MM-dd_HH:mm:ss')}.zip`;
-
-// create a file to stream archive data to.
-const output = fs.createWriteStream(outDir);
-const archive = archiver('zip', {
-  zlib: { level: 9 } // Sets the compression level.
-});
-
-// listen for all archive data to be written
-// 'close' event is fired only when a file descriptor is involved
-output.on('close', () => {
-  console.log(archive.pointer() + ' total bytes');
-  console.log('archiver has been finalized and the output file descriptor has closed.');
-});
-
-// This event is fired when the data source is drained no matter what was the data source.
-// It is not part of this library but rather from the NodeJS Stream API.
-// @see: https://nodejs.org/api/stream.html#stream_event_end
-output.on('end', () => {
-  console.log('Data has been drained');
-});
-
-// good practice to catch warnings (ie stat failures and other non-blocking errors)
-archive.on('warning', (err) => {
-  if (err.code === 'ENOENT') {
-    // log warning
-  } else {
-    // throw error
-    throw err;
+const { formatDate } = require('@liwb/cloud-utils');
+
+const DEST_DIR = path.join(__dirname, '../dist');
+const DEST_ZIP_DIR = path.join(__dirname, '../dist-zip');
+
+const extractExtensionData = () => {
+  const extPackageJson = require('../package.json');
+
+  return {
+    name: extPackageJson.name,
+    version: extPackageJson.version,
+  };
+};
+
+const makeDestZipDirIfNotExists = () => {
+  if (!fs.existsSync(DEST_ZIP_DIR)) {
+    fs.mkdirSync(DEST_ZIP_DIR);
   }
-});
+};
+
+const buildZip = (src, dist, zipFilename) => {
+  console.info(`Building ${zipFilename}...`);
+
+  const archive = archiver('zip', { zlib: { level: 9 } });
+  const stream = fs.createWriteStream(path.join(dist, zipFilename));
+
+  return new Promise((resolve, reject) => {
+    archive
+      .directory(src, false)
+      .on('error', (err) => reject(err))
+      .pipe(stream);
+
+    stream.on('close', () => resolve());
+    archive.finalize();
+  });
+};
 
-// good practice to catch this error explicitly
-archive.on('error', (err) => {
-  throw err;
-});
+const main = () => {
+  const { name, version } = extractExtensionData();
+  const zipFilename = `${name}-v${version}_${formatDate(
+    new Date(),
+    'yyyy-MM-dd_HH-mm-ss'
+  )}.zip`;
 
-// pipe archive data to the file
-archive.pipe(output);
+  makeDestZipDirIfNotExists();
 
-archive.directory('./dist/', false);
+  buildZip(DEST_DIR, DEST_ZIP_DIR, zipFilename)
+    .then(() => console.info('OK'))
+    .catch(console.err);
+};
 
-// finalize the archive (ie we are done appending files but streams have to finish yet)
-// 'close', 'end' or 'finish' may be fired right after calling this method so register to them beforehand
-archive.finalize();
+main();