importing edge-runtime breaks process exit behavior

[EndangeredMassa]

If I import the edge runtime, then my process has an uncaught exception, my process exits with no error message and an exit code of 0.

Reproduction: https://github.com/EndangeredMassa/reproduction-process-exit-behavior

Example:

import { EdgeRuntime } from 'edge-runtime';

throw new Error('intentional break');

It seems that these lines are the cause:

edge-runtime/packages/runtime/src/edge-runtime.ts

Lines 64 to 76 in ccdb203

	/**
	* Define system-level handlers to make sure that we report to the user
	* whenever there is an unhandled rejection or exception before the process crashes.
	*/
	process.on(
	'unhandledRejection',
	function invokeRejectionHandlers(reason, promise) {
	unhandledRejectionHandlers?.forEach((handler) => handler(reason, promise))
	}
	)
	process.on('uncaughtException', function invokeErrorHandlers(error) {
	uncaughtExceptionHandlers?.forEach((handler) => handler(error))
	})

[feugy]

Hey @EndangeredMassa! Thank you for this.

@javivelasco and I did a bit of analysis.

The reason why we caught exceptions & rejections at process level within EdgeRuntime is to avoid exiting the runtime itself. This way, the runtime is resilient to exception/rejections occurring within the executed user code.
Instead, it reports them using events, allowing caller code to gracefully handle them

But this creates that undesirable side you demonstrated: the simple import of the EdgeRuntime swallows everything! Very bad.

Our plan is the following:

• make a distinction between errors happening during user code execution, and at process level
• for the first error group, keep the current behavior (no exit, call event handlers instead)
• for the second error group, propagate the error, so it stops and sets exit code

How does it sounds to you?

[EndangeredMassa]

That sounds good to me, thanks!