Skip to content

chore: require preact-render-to-string >=6.7.0 as the optional peer - #5231

Merged
JoviDeCroock merged 2 commits into
mainfrom
JoviDeCroock/rts-peer-6-7
Sep 2, 2026
Merged

chore: require preact-render-to-string >=6.7.0 as the optional peer#5231
JoviDeCroock merged 2 commits into
mainfrom
JoviDeCroock/rts-peer-6-7

Conversation

@JoviDeCroock

Copy link
Copy Markdown
Member

Note

This PR was authored by an LLM (Claude Fable 5.1) during the v11 release-readiness audit; Jovi reviewed the change and the tests.

Summary

Bumps the optional preact-render-to-string peer range from >=5 to >=6.7.0.

Preact 11 is ESM-only, so preact/compat/server is now imported through compat/server.mjs, which does import … from 'preact-render-to-string/stream-node'. In preact-render-to-string ≤ 6.6 the import condition of that subpath points at a dist/stream-node.mjs that is not published, so the import fails with ERR_MODULE_NOT_FOUND at startup. 6.7.0 publishes the entry (dist/stream/node/index.mjs) and its peer range already allows Preact 11.

The devDependency is left at 6.5.0 for now: 6.7.0's dist/stream-node.d.ts imports WritableStream from node:stream, which no @types/node exports, and that fails this repo's tsc -p jsconfig-lint.json. That is a one-line upstream fix (import Writable); the devDependency can follow once it ships.

@coveralls

coveralls commented Sep 2, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 99.534%. remained the same — JoviDeCroock/rts-peer-6-7 into main

@JoviDeCroock
JoviDeCroock merged commit 66e19fd into main Sep 2, 2026
4 checks passed
@JoviDeCroock
JoviDeCroock deleted the JoviDeCroock/rts-peer-6-7 branch September 2, 2026 17:20
@JoviDeCroock JoviDeCroock mentioned this pull request Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants