Ben McCann

Hmm. What wasn't this an issue with Svelte 4 though?

Does `jsdom` mock browser globals like `window` and `document`? > why do we rely on BROWSER instead of requestAnimationFrame ?? noop though ? One reason is for smaller bundle size....

Really what we want is a way to say that we're running browser code in Node. What if we had testing library svelte specify `resolve: { conditions: ['browser', 'node'] }`...

Can you share an example of what the HTML output looks like now vs what it's supposed to look like?

I merged the other one. Please feel free to rebase and send changes if there are additional features you'd like

I believe this happens specifically when you want to install a specific branch, sub-directory, etc. E.g. I ran `pnpm add -D "github:sveltejs/svelte#path:packages/svelte&sibling-dce"` followed by `pnpm build` on https://github.com/benmccann/interactive-blogs-benchmark/tree/upgrade/sveltekit_mdsvex because I...

I filed an issue in the pnpm repo asking if they might be able to change this on their side: https://github.com/pnpm/pnpm/issues/8437

I wanted to put in a note of support for this PR. I filed an issue (https://github.com/vitejs/vite/issues/17421) a couple of months ago that this would solve, which is that `connect`...

Yeah, sadly there's been no commits to `connect` for three years, so I don't feel very optimistic we could get a change in.

[Express 5 has added support](https://expressjs.com/en/guide/migrating-5.html) for this: >Request middleware and handlers that return rejected promises are now handled by forwarding the rejected value as an Error to the error handling...