Remove redundant else branches (#86)

Else branches are an anti pattern especially if you can easily return from the previous branch. Over time, else branches cause deep nesting and make code unreadable and unmaintainable. Remove elses where possible.

1 files changed, 14 insertions, 13 deletions

diff --git a/packages/cli/src/gemini.ts b/packages/cli/src/gemini.ts
index 60863c6a..97502399 100644
--- a/packages/cli/src/gemini.ts
+++ b/packages/cli/src/gemini.ts
@@ -54,20 +54,21 @@ process.on('unhandledRejection', (reason, _promise) => {
     );
     console.warn('-----------------------------------------');
     console.warn('Reason:', reason);
-    // No process.exit(1);
-  } else {
-    // Log other unexpected unhandled rejections as critical errors
-    console.error('=========================================');
-    console.error('CRITICAL: Unhandled Promise Rejection!');
-    console.error('=========================================');
-    console.error('Reason:', reason);
-    console.error('Stack trace may follow:');
-    if (!(reason instanceof Error)) {
-      console.error(reason);
-    }
-    // Exit for genuinely unhandled errors
-    process.exit(1);
+    return;
+    // No process.exit(1); Don't exit.
+  }
+
+  // Log other unexpected unhandled rejections as critical errors
+  console.error('=========================================');
+  console.error('CRITICAL: Unhandled Promise Rejection!');
+  console.error('=========================================');
+  console.error('Reason:', reason);
+  console.error('Stack trace may follow:');
+  if (!(reason instanceof Error)) {
+    console.error(reason);
   }
+  // Exit for genuinely unhandled errors
+  process.exit(1);
 });
 
 // --- Global Entry Point ---