Make plugins async

[MarijnMensinga]

For the benefit of bootstrapping plugins it is usefull to make plugins load async.

nitro/src/runtime/app.ts

Lines 89 to 91 in ed3a480

	for (const plugin of plugins) {
	plugin(app);
	}

Can someone explain to me if there is a good reason to load plugins synchronously?

[pi0]

This issue is on track and I might have some ideas to support it in a nonbreaking change version with slight behavior-only changes. In the meantime, please consider issue 👍🏼 on main issue if you also want this feature 🙏🏼

[robertpatrick]

It all seems relatively straightforward. The last step is the only tricky one

• Convert createNitroApp into an async function that returns Promise<NitroApp>
• Change the plugin invocations to catch the return value from the plugin
• If the return value is a Promise, call await Promise.all([pluginPromise])
• change the export for nitroApp so that it is initialized when the promise is resolved.

This will ensure that the semantics of order and waiting for one plugin to finish before the next runs are retained. I guess I don't understand why this is complicated...

[pi0]

Sadly, it is not that straightforward without behavior changes / overhead because not all nitro presets are compatible with an async entry today but thanks for your feedback dear @robertpatrick.