Errors in `ctx.waitUntil` are not captured (Cloudflare)

[samuelcolvin]

I'm using the following, but surely it should happen by default?

function runAfter(ctx: ExecutionContext, promise: Promise<unknown>) {
  ctx.waitUntil(wrapLogfire(promise))
}

async function wrapLogfire(promise: Promise<unknown>): Promise<void> {
  try {
    await promise
  } catch (error) {
    logfire.reportError('Error in ctx.waitUntil', error as Error)
    throw error
  }
}

[petyosi]

The weird thing is that the global event handlers are not triggered:

		addEventListener('unhandledrejection', (event) => {
			console.log('UNCAUGHT:', event.promise); // The promise that was rejected.
			console.log('UNCAUGHT:', event.reason); // The value or Error with which the promise was rejected.
		});

		addEventListener('rejectionhandled', (event) => {
			console.log('CAUGHT:', event.promise); // The promise that was rejected.
			console.log('CAUGHT:', event.reason); // The value or Error with which the promise was rejected.
		});

Leaving this aside, our SDK does not do any automatic error/promise rejection subscription by default. In this particular case, this would also introduce an additional dependency - @pydantic/logfire-cf-workers would depend on @pydantic/logfire-api so that it can report the error. This is possible, but it's not the case at the moment.

[petyosi]

I need your input on this, so I'm creating PYD-2276.