From e1993691516bad237cffd703cc143d1ace8ba959 Mon Sep 17 00:00:00 2001 From: Alistair Smith Date: Thu, 5 Jun 2025 15:26:56 -0700 Subject: [PATCH] mv --- src/bun.js/node/node_process.zig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bun.js/node/node_process.zig b/src/bun.js/node/node_process.zig index 728f0bec7b..1bce74fffb 100644 --- a/src/bun.js/node/node_process.zig +++ b/src/bun.js/node/node_process.zig @@ -255,7 +255,7 @@ fn setCwd_(globalObject: *JSC.JSGlobalObject, to: *JSC.ZigString) bun.JSError!JS // TODO(@190n) this may need to be noreturn pub fn exit(globalObject: *JSC.JSGlobalObject, code: u8) callconv(.c) void { var vm = globalObject.bunVM(); - vm.exit_handler.exit_code = code; // TODO(@alii) Is setting this multiple times incorrect? There's a regression node test for this (https://linear.app/oven/issue/BUN-13917/nodejs-test-test-worker-process-exit-async-modulejs) + vm.exit_handler.exit_code = code; // TODO(@alii): https://github.com/oven-sh/bun/pull/20213 if (vm.worker) |worker| { // TODO(@190n) we may need to use requestTerminate or throwTerminationException // instead to terminate the worker sooner