bug fix

- Fixed because there was a problem with asynchronous processing
- Stop using the fs-extra module due to a problem that the fs-extra module could not be read in the second execution

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tyranoscript-packager",
-  "version": "0.1.6-beta.2",
+  "version": "0.1.6-beta.3",
   "description": "This is a CLI tool for packaging Typescript apps.",
   "bin": {
     "tyrano": "src/main/index.js"
@@ -26,7 +26,8 @@
   "dependencies": {
     "archiver": "^3.1.1",
     "commander": "^4.0.1",
-    "fs-extra": "^8.1.0",
+    "mkdirp": "^0.5.1",
+    "ncp": "^2.0.0",
     "tar": "^5.0.5"
   },
   "devDependencies": {

src/main/TyranoscriptPackagerForWindows.js

@@ -1,4 +1,4 @@
-const fs = require('fs-extra');
+const fs = require('fs');
 const path = require('path');
 const tar = require('tar');
 const archiver = require('archiver');
@@ -111,9 +111,9 @@ class TyranoscriptPackagerForWindows {
     logger.log('Tyrano app root dir:', tyranoAppRootDir);
     logger.log('NW.js Manifest:', nwjsManifestJson);
     logger.log('Export dir:', destDir);
-    await fs.ensureDir(destDir);
+    await fsUtil.ensureDir(destDir);
     await this.exportGameExe(tyranoAppRootDir, nwjsManifestJson, destDir);
-    await this.copyTyranoBinFiles(destDir)
+    await this.copyTyranoBinFiles(destDir);
   }
 
 
@@ -126,7 +126,7 @@ class TyranoscriptPackagerForWindows {
   }
 
   copyTyranoBinFiles(destDir) {
-    return fs.copy(this.binWinDir, destDir, {filter: (src, dest) => {
+    return fsUtil.copy(this.binWinDir, destDir, {filter: (src, dest) => {
       if (path.basename(src) === 'nw.exe') return false;
       if (path.basename(src) === '.gitignore') return false;
       if (path.basename(src) === '.npmignore') return false;
@@ -141,7 +141,10 @@ class TyranoscriptPackagerForWindows {
    */
   appendNwjsManifest(nwjsManifestJson, archive) {
     return new Promise(resolve => {
-      archive.append(nwjsManifestJson, {name: 'package.json'})
+      archive.append(nwjsManifestJson, {name: 'package.json'});
+      archive.addListener('end', () => {
+        logger.log('Success write Tyrano manifest to archive');
+      });
       resolve();
     })
   }
@@ -157,6 +160,9 @@ class TyranoscriptPackagerForWindows {
       archive.directory(path.join(tyranoAppRootDir, 'data'), 'data');
       archive.directory(path.join(tyranoAppRootDir, 'tyrano'), 'tyrano');
       archive.file(path.join(tyranoAppRootDir, 'index.html'), {name: 'index.html'});
+      archive.addListener('end', () => {
+        logger.log('Success write Tyrano app contents to archive');
+      });
       resolve();
     })
   }
@@ -199,12 +205,11 @@ class TyranoscriptPackagerForWindows {
    */
   writeBinary(ws, binaryPath) {
     return new Promise(resolve => {
-      try {
-        ws.write(new Uint8Array(Buffer.from(fs.readFileSync(binaryPath))));
-      } catch (err) {
-        throw err;
-      }
-      resolve();
+      ws.write(new Uint8Array(Buffer.from(fs.readFileSync(binaryPath))), (err) => {
+        if (err) throw err;
+        logger.log('Success write binary', binaryPath);
+        resolve();
+      });
     });
   }
 }

src/main/fsUtil.js

@@ -1,4 +1,6 @@
-const fs = require('fs-extra');
+const fs = require('fs');
+const ncp = require('ncp');
+const mkdirp = require('mkdirp');
 
 module.exports.isNotExist = (filePath) => {
   try {
@@ -19,3 +21,20 @@ module.exports.canNotRead = (filePath) => {
   }
 };
 
+module.exports.ensureDir = (dir) => {
+  return new Promise(resolve => {
+    mkdirp(dir, (err) => {
+      if (err) throw err;
+      resolve();
+    });
+  });
+}
+
+module.exports.copy = (src, dest, options) => {
+  return new Promise(resolve => {
+    ncp(src, dest, options, (err) => {
+      if (err) throw err;
+      resolve();
+    });
+  });
+}

src/main/index.js

@@ -3,7 +3,7 @@
 'use strict';
 
 const program = require('commander');
-const fs = require('fs-extra');
+const fs = require('fs');
 const path = require('path');
 const logger = require('./logger');
 
@@ -13,7 +13,7 @@ const TyPackagerForWindows = require('./TyranoscriptPackagerForWindows')
 //---------- initialize commander ----------//
 
 program
-  .version(fs.readJsonSync(path.join(__dirname, '../../package.json')).version)
+  .version(JSON.parse(fs.readFileSync(path.join(__dirname, '../../package.json'))).version)
   .option('-a, --app-dir <string>', 'Specify the directory of Typescript application', './')
   .option('-d, --dest-dir <string>', 'Specify the dest directory', './dest')
   .option('-n, --name <string>', 'Specify the name of the game. If omitted, the app-dir name is used')
@@ -50,8 +50,10 @@ program.platforms.forEach((value, index, array) => {
         program.width, program.height, program.maxWidth, program.maxHeight, program.minWidth, program.minHeight
       );
       const destWinDir = path.join(path.resolve(program.destDir), 'win');
-      tyTackager.package(appRootDir, manifest, destWinDir);
-      logger.log('Successful packaging for Windows :', destWinDir);
+      const promise = tyTackager.package(appRootDir, manifest, destWinDir);
+      promise.then(() => {
+        logger.log('Successful packaging for Windows :', destWinDir);
+      });
       break;
     // case 'mac':
       //TODO Implement mac packaging