From 057da6ed4d76148ff58a5763dfd008b81050b07c Mon Sep 17 00:00:00 2001
From: Debadree Chatterjee <debadree333@gmail.com>
Date: Tue, 13 Dec 2022 14:53:24 +0530
Subject: [PATCH] lib,src: update exit codes as per todos

Refs: https://github.com/nodejs/node/pull/44746
---
 lib/internal/debugger/inspect.js | 3 ++-
 lib/internal/main/repl.js        | 4 ++--
 src/node.cc                      | 6 +++---
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/lib/internal/debugger/inspect.js b/lib/internal/debugger/inspect.js
index 039d33811d1a9d..aa20304611d211 100644
--- a/lib/internal/debugger/inspect.js
+++ b/lib/internal/debugger/inspect.js
@@ -45,6 +45,7 @@ const {
   exitCodes: {
     kGenericUserError,
     kNoFailure,
+    kInvalidCommandLineArgument,
   },
 } = internalBinding('errors');
 
@@ -340,7 +341,7 @@ function startInspect(argv = ArrayPrototypeSlice(process.argv, 2),
                          `       ${invokedAs} --port=<port>\n` +
                          `       ${invokedAs} -p <pid>\n`);
     // TODO(joyeecheung): should be kInvalidCommandLineArgument.
-    process.exit(kGenericUserError);
+    process.exit(kInvalidCommandLineArgument);
   }
 
   const options = parseArgv(argv);
diff --git a/lib/internal/main/repl.js b/lib/internal/main/repl.js
index c371706b352c82..c77dfede89f49f 100644
--- a/lib/internal/main/repl.js
+++ b/lib/internal/main/repl.js
@@ -16,7 +16,7 @@ const console = require('internal/console/global');
 
 const { getOptionValue } = require('internal/options');
 
-const { exitCodes: { kGenericUserError } } = internalBinding('errors');
+const { exitCodes: { kInvalidCommandLineArgument } } = internalBinding('errors');
 
 prepareMainThreadExecution();
 
@@ -33,7 +33,7 @@ if (process.env.NODE_REPL_EXTERNAL_MODULE) {
     // so use console.error.
     console.error('Cannot specify --input-type for REPL');
     // TODO(joyeecheung): should be kInvalidCommandLineArgument.
-    process.exit(kGenericUserError);
+    process.exit(kInvalidCommandLineArgument);
   }
 
   const esmLoader = require('internal/process/esm_loader');
diff --git a/src/node.cc b/src/node.cc
index f79f4421f8e347..698bf0be57589f 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -1133,7 +1133,7 @@ ExitCode GenerateAndWriteSnapshotData(const SnapshotData** snapshot_data_ptr,
               "entry point but Node.js was built without embedded "
               "snapshot.\n");
       // TODO(joyeecheung): should be kInvalidCommandLineArgument instead.
-      exit_code = ExitCode::kGenericUserError;
+      exit_code = ExitCode::kInvalidCommandLineArgument;
       return exit_code;
     }
   } else {
@@ -1167,7 +1167,7 @@ ExitCode GenerateAndWriteSnapshotData(const SnapshotData** snapshot_data_ptr,
             "Cannot open %s for writing a snapshot.\n",
             snapshot_blob_path.c_str());
     // TODO(joyeecheung): should be kStartupSnapshotFailure.
-    exit_code = ExitCode::kGenericUserError;
+    exit_code = ExitCode::kStartupSnapshotFailure;
   }
   return exit_code;
 }
@@ -1193,7 +1193,7 @@ ExitCode LoadSnapshotDataAndRun(const SnapshotData** snapshot_data_ptr,
     if (!ok) {
       // If we fail to read the customized snapshot, simply exit with 1.
       // TODO(joyeecheung): should be kStartupSnapshotFailure.
-      exit_code = ExitCode::kGenericUserError;
+      exit_code = ExitCode::kStartupSnapshotFailure;
       return exit_code;
     }
     *snapshot_data_ptr = read_data.release();