fix: use temp in grammar-utils

Author: aminya

lib/grammar-utils.js

@@ -2,9 +2,10 @@
 
 // Require some libs used for creating temporary files
 import os from "os"
-import fs from "fs"
+import * as fs from "fs"
 import path from "path"
-import { v1 as uuidv1 } from "uuid"
+import * as temp from "temp"
+temp.track()
 
 // Public: GrammarUtils - utilities for determining how to run code
 const GrammarUtils = {
@@ -17,17 +18,10 @@ const GrammarUtils = {
   // Returns the {String} filepath of the new file
   createTempFileWithCode(code, extension = "") {
     try {
-      if (!fs.existsSync(this.tempFilesDir)) {
-        fs.mkdirSync(this.tempFilesDir)
-      }
-
-      const tempFilePath = this.tempFilesDir + path.sep + uuidv1() + extension
-
-      const file = fs.openSync(tempFilePath, "w")
-      fs.writeSync(file, code)
-      fs.closeSync(file)
-
-      return tempFilePath
+      const tempFile = temp.openSync({ dir: this.tempFilesDir, suffix: extension })
+      fs.writeSync(tempFile.fd, code)
+      fs.closeSync(tempFile.fd)
+      return tempFile.path
     } catch (error) {
       throw new Error(`Error while creating temporary file (${error})`)
     }
@@ -37,14 +31,7 @@ const GrammarUtils = {
   // {GrammarUtils::createTempFileWithCode}
   deleteTempFiles() {
     try {
-      if (fs.existsSync(this.tempFilesDir)) {
-        const files = fs.readdirSync(this.tempFilesDir)
-        if (files.length) {
-          files.forEach((file) => fs.unlinkSync(this.tempFilesDir + path.sep + file))
-        }
-        return fs.rmdirSync(this.tempFilesDir)
-      }
-      return null
+      return temp.cleanupSync()
     } catch (error) {
       throw new Error(`Error while deleting temporary files (${error})`)
     }