From 975caa75b11b87362d69c2e677708c97c0695ee4 Mon Sep 17 00:00:00 2001 From: Alistair Smith Date: Thu, 5 Jun 2025 14:27:22 -0700 Subject: [PATCH] dont miss this --- 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 07966abedd..0c0d8a82cd 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; + 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) if (vm.worker) |worker| { // TODO(@190n) we may need to use requestTerminate or throwTerminationException // instead to terminate the worker sooner