From 5719568ec03fe9c7e8ddbd511c3fc7d84fd82b4f 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 0c0d8a82cd..7b898e93a7 100644 --- a/src/bun.js/node/node_process.zig +++ b/src/bun.js/node/node_process.zig @@ -282,7 +282,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