fix(cli): ensure temp files are always removed

Even in cases when the CLI has errored, ensure that any temp files
created are removed when shutting down.

src/index.ts

@@ -95,7 +95,7 @@ export const cli = async (options: ICliOptions) => {
     }
     complete('Finished');
   } catch (err) {
-    bug(err);
     await clean(options);
+    bug(err);
   }
 };

src/log.ts

@@ -1,4 +1,6 @@
 import chalk from 'chalk';
+import { IOptions } from '.';
+import { clean } from './tmpdir';
 
 let color = new chalk.constructor({ enabled: true });
 
@@ -16,8 +18,9 @@ export const bug = (err: Error): void => {
   process.exit(1);
 };
 
-export const panic = (value: string): void => {
+export const panic = async (value: string, options: IOptions): Promise<void> => {
   console.log(color.red('! %s'), value);
+  await clean(options);
   process.exit(1);
 };
 

src/run-imagealpha.ts

@@ -11,7 +11,7 @@ export const runImageAlpha: AppRunner = async (options: IOptions) => {
     .map((file) => file.tmp)
     .filter(isSupported(IMAGEALPHA.supports));
   if (!(await pathExists(PNGQUANT_BIN_PATH))) {
-    return panic(`ImageAlpha.app is not installed (${IMAGEALPHA_URL})`);
+    return panic(`ImageAlpha.app is not installed (${IMAGEALPHA_URL})`, options);
   }
   await pngquant(pngFilePaths, options);
   verbose(`${IMAGEALPHA.name} has finished`);

src/run-imageoptim.ts

@@ -7,7 +7,7 @@ import { info, panic, verbose } from './log';
 export const runImageOptim: AppRunner = async (options) => {
   info(`Running ${IMAGEOPTIM.name}...`);
   if (!(await pathExists(IMAGEOPTIM_BIN_PATH))) {
-    return panic(`ImageOptim.app is not installed (${IMAGEOPTIM_URL})`);
+    return panic(`ImageOptim.app is not installed (${IMAGEOPTIM_URL})`, options);
   }
   await execa(IMAGEOPTIM_BIN_PATH, [options.tmpDir]);
   verbose(`${IMAGEOPTIM.name} has finished`);

src/run-jpegmini.ts

@@ -18,11 +18,11 @@ export const runJpegMini: AppRunner = async (options) => {
   const [app, canAutomate] = await Promise.all([jpegMini, assistiveDeviceSupport]);
 
   if (!app) {
-    return panic(`JPEGmini is not installed (${JPEG_MINI_URL})`);
+    return panic(`JPEGmini is not installed (${JPEG_MINI_URL})`, options);
   }
 
   if (!canAutomate) {
-    return panic(`Support for assistive devices needed, see ${ASSISTIVE_DEVICES_URL}`);
+    return panic(`Support for assistive devices needed, see ${ASSISTIVE_DEVICES_URL}`, options);
   }
 
   info(`Running ${app.name}...`);

src/tmpdir.ts

@@ -5,7 +5,10 @@ import { verbose } from './log';
 const sourceToTmp = ({ source, tmp }: IFile) => copy(source, tmp);
 const tmpToSource = ({ source, tmp }: IFile) => copy(tmp, source);
 
-export const clean = (options: { tmpDir: string }) => remove(options.tmpDir);
+export const clean = (options: { tmpDir: string }) => {
+  verbose(`Deleting temp directory ${options.tmpDir}`);
+  return remove(options.tmpDir);
+};
 
 export const setup = async (options: IOptions) => {
   verbose(`Creating temp directory ${options.tmpDir}`);
@@ -17,6 +20,5 @@ export const setup = async (options: IOptions) => {
 export const tearDown = async (options: IOptions) => {
   verbose(`Copying ${options.filePaths.length} files back to original location`);
   await Promise.all(options.filePaths.map(tmpToSource));
-  verbose(`Deleting temp directory ${options.tmpDir}`);
   await clean(options);
 };