feat: Consolidating and logging tmpdir cleanup (#26)

src/main.ts

@@ -54,7 +54,7 @@ export async function run(): Promise<void> {
 
     // Create temp directory for downloading non-cached versions
     tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'regctl_'))
-    core.debug(`tmpdir => ${tmpDir}`)
+    core.debug(`Created ${tmpDir}`)
 
     // Check if regctl is already in the tool-cache
     const cache = core.getInput('cache')
@@ -120,17 +120,18 @@ export async function run(): Promise<void> {
         version.substring(1) // remove leading 'v'
       )
     } else core.info('📥 Loaded from runner cache')
+
     // Add the cached regctl to our PATH
     core.addPath(mainCachePath)
-
-    // Cleanup tmpDir
-    fs.rmSync(tmpDir, { recursive: true, force: true })
     core.info('🎉 regctl is ready')
   } catch (error) {
-    // Cleanup tmpDir before terminating during a failure
-    if (tmpDir) fs.rmSync(tmpDir, { recursive: true, force: true })
-
     if (error instanceof Error) core.setFailed(error.message)
     else core.setFailed(error as string)
   }
+
+  // Cleanup tmpDir if it was created at any point
+  if (tmpDir) {
+    core.debug(`Deleting ${tmpDir}`)
+    fs.rmSync(tmpDir, { recursive: true, force: true })
+  }
 }