Global TextEncoder initialization prevents tree-shaking

[waiter]

I'm using React Router as a...

library

Reproduction

react-router/packages/react-router/lib/server-runtime/crypto.ts

Line 1 in 796e9ae

const encoder = new TextEncoder();

System Info

no need

Used Package Manager

yarn

Expected Behavior

after build, no TextEncoder

Actual Behavior

still has new TextEncoder

[markdalgleish]

I've just merged #13504 which marks the line you noted as pure. I've confirmed that when building with Rollup (via Vite), this is now removed from the final build if this module isn't used.