fix: backup log if has error

src/main/app/LauncherApp.ts

@@ -160,7 +160,15 @@ export abstract class LauncherApp extends EventEmitter {
     /**
      * Quit the app gentally.
      */
-    abstract quit(): void;
+    quit() {
+        Promise.all(this.managers.map(m => m.beforeQuit()))
+            .then(() => this.quitApp());
+    }
+
+    /**
+     * Quit the app gentally.
+     */
+    protected abstract quitApp(): void;
 
     /**
      * Force exit the app with exit code
@@ -337,7 +345,7 @@ export abstract class LauncherApp extends EventEmitter {
         this
             .on('window-all-closed', () => {
                 if (this.parking) return;
-                if (process.platform !== 'darwin') { this.quit(); }
+                if (process.platform !== 'darwin') { this.quitApp(); }
             })
             .on('minecraft-start', () => { this.parking = true; })
             .on('minecraft-exit', () => { this.parking = false; });

src/main/app/updater.ts

@@ -124,7 +124,7 @@ export async function quitAndInstallAsar(this: ElectronLauncherApp) {
         await promisify(unlink)(appAsarPath);
         await promisify(rename)(updateAsarPath, appAsarPath);
     }
-    this.quit();
+    this.quitApp();
 }
 
 

src/main/electron/ElectronLauncherApp.ts

@@ -90,7 +90,7 @@ export default class ElectronLauncherApp extends LauncherApp {
 
     showItemInFolder = shell.showItemInFolder;
 
-    quit = app.quit;
+    quitApp = app.quit;
 
     exit = app.exit;
 

src/main/manager/LogManager.ts

@@ -1,11 +1,14 @@
 import LauncherApp from '@main/app/LauncherApp';
 import { IS_DEV } from '@main/constant';
 import { createWriteStream, WriteStream } from 'fs';
-import { ensureDir } from 'fs-extra';
-import { resolve } from 'path';
+import { ensureDir, copy, readFile, writeFile } from 'fs-extra';
+import { resolve, join } from 'path';
 import { PassThrough, pipeline, Transform } from 'stream';
 import { format } from 'util';
 import { Manager } from '.';
+import { createGzip } from 'zlib';
+import filenamify from 'filenamify';
+import { gzip } from '@main/util/zip';
 
 function formatMsg(message: any, options: any[]) { return options.length !== 0 ? format(message, options) : format(message); }
 function baseTransform(tag: string) { return new Transform({ transform(c, e, cb) { cb(undefined, `[${tag}] [${new Date().toLocaleString()}] ${c}\n`); } }); }
@@ -25,13 +28,19 @@ export default class LogManager extends Manager {
 
     private openedStream: { [name: string]: WriteStream } = {};
 
+    private hasError = false;
+
     constructor(app: LauncherApp) {
         super(app);
 
         pipeline(this.loggerEntries.log, this.output, () => { });
         pipeline(this.loggerEntries.warn, this.output, () => { });
         pipeline(this.loggerEntries.error, this.output, () => { });
 
+        this.loggerEntries.error.once('data', () => {
+            this.hasError = true;
+        });
+
         process.on('uncaughtException', (err) => {
             this.error('Uncaught Exception');
             this.error(err);
@@ -76,21 +85,26 @@ export default class LogManager extends Manager {
     }
 
     async redirectLogPipeline(root: string) {
-        try {
-            await ensureDir(resolve(root, 'logs'));
-        } catch (e) {
-            if (e.code !== 'EEXIST') {
-                throw e;
-            }
-        }
         this.logRoot = resolve(root, 'logs');
-        let mainLog = resolve(root, 'logs', 'main.log');
-        this.output.pipe(createWriteStream(mainLog, { encoding: 'utf-8', flags: 'w+' }));
+        await ensureDir(this.logRoot);
+        const mainLog = join(this.logRoot, 'main.log');
+        const stream = createWriteStream(mainLog, { encoding: 'utf-8', flags: 'w+' });
+        this.output.pipe(stream);
+        this.openedStream.MAIN_LOG = stream;
     }
 
     // SETUP CODE
 
     setup() {
-        this.redirectLogPipeline(this.app.root);
+        return this.redirectLogPipeline(this.app.root);
+    }
+
+    async beforeQuit() {
+        const mainLog = this.openedStream.MAIN_LOG;
+        mainLog.close();
+        const mainLogPath = join(this.logRoot, 'main.log');
+        if (this.hasError) {
+            await writeFile(join(this.logRoot, filenamify(new Date().toJSON())), await gzip(await readFile(mainLogPath)));
+        }
     }
 }

src/main/manager/index.ts

@@ -21,6 +21,8 @@ export abstract class Manager {
     storeReady(store: StaticStore<any>): Promise<void> | void { }
     /* eslint-enable */
 
+    beforeQuit(): Promise<void> | void { }
+
     log(m: any, ...args: any[]) { this.app.logManager.log(`[${this.name}] ${m}`, ...args); }
 
     warn(m: any, ...args: any[]) { this.app.logManager.warn(`[${this.name}] ${m}`, ...args); }

src/main/service/BaseService.ts

@@ -85,7 +85,7 @@ export default class BaseService extends Service {
         this.commit('updateStatus', 'ready');
     }
 
-    quit = this.app.quit;
+    quit = this.app.quit.bind(this.app);
 
     exit = this.app.exit;
 }