fix: use exit code

Signed-off-by: Marty Jones <[email protected]>
bloomberg · Dec 19, 2024 · b71559b · b71559b
1 parent 4190189
commit b71559b
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 6 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/core/src/context.ts b/packages/core/src/context.ts
@@ -46,9 +46,9 @@ export interface StricliProcess extends WritableStreams {
      */
     readonly env?: Readonly<Partial<Record<string, string>>>;
     /**
-     * Set the exit code and end the current process.
+     * A number which will be the process exit code.
      */
-    readonly exit?: (code: number) => void;
+    exitCode?: number | string;
 }
 
 /**

diff --git a/packages/core/src/index.ts b/packages/core/src/index.ts
@@ -76,5 +76,8 @@ export async function run<CONTEXT extends CommandContext>(
     context: StricliDynamicCommandContext<CONTEXT>,
 ): Promise<void> {
     const exitCode = await runApplication(app, inputs, context);
-    context.process.exit?.(exitCode);
+
+    // We set the exit code instead of calling exit() so as not
+    // to cancel any pending tasks (e.g. stdout writes)
+    context.process.exitCode = exitCode;
 }
diff --git a/packages/core/tests/fakes/context.ts b/packages/core/tests/fakes/context.ts
@@ -16,7 +16,6 @@ interface FakeProcess extends StricliProcess {
     readonly stdout: FakeWritable;
     readonly stderr: FakeWritable;
     readonly exit: (code: number) => void;
-    readonly exitCode: number;
 }
 
 export type FakeContext = StricliDynamicCommandContext<CommandContext> & {