Repository navigation
fix: require solid-js 1.9.13 or later - #543
Conversation
The Solid compiler emits `use` from solid-js/web for every `ref`, and devtools-ui 0.7.0 added a ref to ThemeContextProvider. The server build of solid-js exported `use` only from 1.9.13, so SSR apps with solid-js 1.9.12 or earlier in their lockfile failed with `"use" is not exported`. The dependency and peer ranges were ">=1.9.7" and "^1.9.9", so resolvers kept the old versions. They now start at 1.9.13. The lockfile moves to solid-js 1.9.15 and no other package changes. 1.9.11+ also has the seroval security fix from #410. Fixes #511 Fixes #410
|
Caution Review failedThe pull request is closed. No actionable comments were generated in the recent review. ℹ️ Recent review info
Walkthrough
Merge Risk:
|
| Check name | Status | Explanation |
|---|---|---|
| Linked Issues check | ✅ Passed | The PR raises the solid-js dependency and peer-dependency minimum to ^1.9.13 or >=1.9.13 in packages/devtools and packages/devtools-ui, which addresses the SSR use export failure in [#511]… |
| Out of Scope Changes check | ✅ Passed | The changeset and the Solid version updates in the Solid examples and e2e app support the dependency-floor change. The additional package manifest updates keep related workspace packages consistent wi… |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Title check | ✅ Passed | The title clearly and concisely identifies the main change: requiring SolidJS 1.9.13 or later. |
| Description check | ✅ Passed | The description is complete and relevant. It explains the SSR failure, dependency changes, release impact, testing performed, linked issues, risks, rollback, and public API impact. It also records tha… |
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Autopilot is currently an internal CodeRabbit preview.
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
|
View your CI Pipeline Execution ↗ for commit 345187f
☁️ Nx Cloud last updated this comment at |
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx affected --targets=test:eslint,test:sherif,t... |
❌ Failed | 4m 54s | View ↗ |
nx run-many --target=test:e2e --parallel=1 --pr... |
✅ Succeeded | 1m 16s | View ↗ |
nx run-many --targets=build --exclude=examples/... |
✅ Succeeded | 48s | View ↗ |
☁️ Nx Cloud last updated this comment at 2026-10-02 13:33:48 UTC
More templates
@tanstack/angular-devtools
@tanstack/devtools
@tanstack/devtools-a11y
@tanstack/devtools-bundler-core
@tanstack/devtools-client
@tanstack/devtools-rspack
@tanstack/devtools-ui
@tanstack/devtools-utils
@tanstack/devtools-vite
@tanstack/devtools-webmcp
@tanstack/devtools-event-bus
@tanstack/devtools-event-client
@tanstack/preact-devtools
@tanstack/react-devtools
@tanstack/solid-devtools
@tanstack/svelte-devtools
@tanstack/vue-devtools
commit: |
sherif requires one range per dependency across the workspace. The examples and the Solid e2e app still asked for ^1.9.9. The lockfile already resolved them to 1.9.15, so only the specifiers change.
# Conflicts: # packages/devtools-a11y/package.json # packages/devtools-ui/package.json # packages/devtools-utils/package.json # packages/devtools/package.json # packages/solid-devtools/package.json # pnpm-lock.yaml
🦋 Changeset detectedLatest commit: a708c24 The changes in this PR will be included in the next version bump. This PR includes changesets to release 14 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
🎉 This PR has been released!
Thank you for your contribution! |

SSR apps with solid-js 1.9.12 or earlier in their lockfile cannot start the dev server:
"use" is not exported by solid-js/web/dist/server.js. This PR raises the solid-js minimum to 1.9.13, the first version whose server build exportsuse.🎯 Changes
usefromsolid-js/webfor everyref. Sincedevtools-ui0.7.0,ThemeContextProviderhas aref, anddevtoolsanddevtools-utilshave refs too.useto its server build (as anotSupstub) in 1.9.13. Our ranges were>=1.9.7and^1.9.9, so lockfiles with 1.9.12 stayed valid and then failed.devtools,devtools-ui,devtools-utils,devtools-a11y, andsolid-devtoolsnow start at1.9.13. The peer range is^1.9.13, so it keeps the Solid 2 cap that main added in the meantime.^1.9.13, becausesherifrequires one range per dependency in the workspace.serovalsecurity fix that bump solid-js to avoid security vulnerability #410 asks for.✅ Checklist
pnpm test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Testing
Commands run
vitest runfordevtools-ui(64),devtools(370),devtools-utils(18), anddevtools-a11y(31): all pass on solid-js 1.9.15.pnpm install --frozen-lockfile: the lockfile is consistent.pnpm run test:sherif: no errors.packages/devtools-ui,solid-js/webresolves to the server build under Node, andtypeof useisfunction.pnpm test:pr.Manual test
solid-js@1.9.12in its lockfile and@tanstack/devtools@0.14.x.vite devfails withMISSING_EXPORT "use".vite devstarts.How this PR makes testing easy
No new tests. The existing Solid package suites now run on solid-js 1.9.15.
Linked issues
Fixes #511
Fixes #410
Risk / rollback
Low. Apps that pin solid-js below 1.9.13 get a peer warning and must update solid-js. To undo, revert this PR.
Public API change
Before
After
🤖 Generated with Claude Code
Summary by CodeRabbit