cloudflare / vinext

fix: eliminate react-dom/server.edge dual-import Rollup warning in SSR entry (#41)
* fix: use static import for renderToStaticMarkup to eliminate dual-import warning The generated SSR entry imported renderToReadableStream statically from react-dom/server.edge at the top level, but later used a dynamic import() for renderToStaticMarkup from the same module. Rollup detects this mixed static+dynamic import pattern and emits a warning for every build: "react-dom/server.edge" is imported as both a static and dynamic import in the same module Fix: add renderToStaticMarkup to the existing static import, removing the redundant dynamic import(). The module was already loaded — the dynamic import added no lazy-loading benefit since it runs unconditionally during SSR. * Remove comment on renderToStaticMarkup import Removed comment about static import of renderToStaticMarkup. * Remove unneccesary test --------- Co-authored-by: Fred K. Schott <622227+FredKSchott@users.noreply.github.com>
nianyi(likai) nianyi(likai) committed on Feb 27, 2026, 07:13 AM
Showing 2 changed files +2 additions -2 deletions
Browse files at this commit →