allow OnRenderFn be async and use Promises

[revintec]

allow component$(...) to use async functions as callback, thus simplifying server code
this would also align its signature more with useBrowserVisibleTask$(...)(originally useClientEffect$(...)

What is it?

• Feature / enhancement

Description

fixes https://github.com/BuilderIO/qwik/issues/2765

Checklist:

• My code follows the developer guidelines of this project
• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• Added new tests to cover the fix / functionality -- only changes function signature, in a backward compatible way

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[manucorporat]

it's not a practice we recommend! in fact we might error if you detect promises at component$, why do you need to await inside a component$ callback?

[revintec]

@manucorporat sorry for replying late, let's continue discussion here https://github.com/BuilderIO/qwik/issues/2765