From f16be32ba68b92794d1e3df3604fbdfdb8d24620 Mon Sep 17 00:00:00 2001
From: Tessa Walsh <tessa@bitarchivist.net>
Date: Tue, 11 Feb 2025 15:04:38 -0500
Subject: [PATCH] Make sure all exit calls use ExitCodes enum (#767)

Quick follow-up to #584 to make sure enum is used everywhere in profile editing mode:
- profile browser exits with ExitCodes.SignalInterrupted in response to signal
- use ExitCodes.Success or GenericError for other exit codes
---
 src/create-login-profile.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/create-login-profile.ts b/src/create-login-profile.ts
index 6063e7f1..07aba7c4 100755
--- a/src/create-login-profile.ts
+++ b/src/create-login-profile.ts
@@ -147,7 +147,7 @@ function getDefaultWindowSize() {
 
 function handleTerminate(signame: string) {
   logger.info(`Got signal ${signame}, exiting`);
-  process.exit(ExitCodes.GenericError);
+  process.exit(ExitCodes.SignalInterrupted);
 }
 
 async function main() {
@@ -299,7 +299,7 @@ async function automatedProfile(
     }
     logger.debug("Login form could not be found");
     await page.close();
-    process.exit(1);
+    process.exit(ExitCodes.GenericError);
     return;
   }
 
@@ -318,7 +318,7 @@ async function automatedProfile(
 
   await createProfile(params, browser, page, cdp);
 
-  process.exit(0);
+  process.exit(ExitCodes.Success);
 }
 
 async function createProfile(