Description

• Closes Use renderToPipeableStream instead of renderToString in ssr-react playground #359

I was revising React streaming and also the discussion of whether we should add async: true to preamble script #352.

My conclusion is that it's fine to always add async: true since otherwise inline module script <script type="module">console.log("foo")</script> is always blocked until full html load and React preamble script doesn't need to wait. (For this one, I'll raise a separate PR.)

However, regardless of having async or not, relying on transformIndexHtml for ssr streaming is likely not a good idea because in order to fully utilize streaming feature, bootstrapModules of streaming API is recommended, but then, to avoid race condition of execution order between react hmr injectIntoGlobalHook and bootstrapModules, it's required to have an additional logic around it. In this PR, I added playground test as a demo to show one way to achieve this (which is using virtual client entry as wrapper). It's rather verbose, so maybe not necessary to have it here (and it's more of SSR framework territory), but I can show it as a reference to close the issue at least 🙂