Replace execSync with exec and make convertTo async

Author: Nick Barnett

src/cleanup.ts

@@ -1,9 +1,10 @@
-import fs from 'fs';
+import fs from 'fs/promises';
 import del from 'del';
 
 // Removes temp files generated by LibreOffice
-export function cleanupTempFiles(): void {
-  for (const file of fs.readdirSync(`/tmp`)) {
+export async function cleanupTempFiles(): Promise<void> {
+  const files = await fs.readdir(`/tmp`);
+  for (const file of files) {
     if (file.endsWith('.tmp') === true || file.startsWith('OSL_PIPE')) {
       try {
         del.sync([`/tmp/${file}`, `/tmp/${file}/*`], {force: true});

src/convert.ts

@@ -1,7 +1,10 @@
-import {execSync} from 'child_process';
+import childProcess from 'child_process';
+import util from 'util';
 import {cleanupTempFiles} from './cleanup';
 import {getConvertedFilePath} from './logs';
 
+const exec = util.promisify(childProcess.exec);
+
 export const DEFAULT_ARGS = [
   '--headless',
   '--invisible',
@@ -13,8 +16,8 @@ export const DEFAULT_ARGS = [
 ];
 const LO_BINARY_PATH = 'libreoffice7.4';
 
-export function convertTo(filename: string, format: string): string {
-  cleanupTempFiles();
+export async function convertTo(filename: string, format: string): Promise<string> {
+  await cleanupTempFiles();
 
   const argumentsString = DEFAULT_ARGS.join(' ');
   const outputFilename = filename.split(/\\ /).join(' ');
@@ -25,13 +28,13 @@ export function convertTo(filename: string, format: string): string {
 
   // due to an unknown issue, we need to run command twice
   try {
-    logs = execSync(cmd);
+    logs = (await exec(cmd)).stdout;
   } catch (e) {
-    logs = execSync(cmd);
+    logs = (await exec(cmd)).stdout;
   }
 
-  execSync(`rm '/tmp/${outputFilename}'`);
-  cleanupTempFiles();
+  await exec(`rm '/tmp/${outputFilename}'`);
+  await cleanupTempFiles();
 
   return getConvertedFilePath(logs.toString());
 }