Skip to content

Conversation

@birkskyum
Copy link
Member

@birkskyum birkskyum commented Oct 22, 2025

solid-js@1.9.5 uses seroval@1.2.0
TanStack Router/Start uses seroval@1.3.2

Upgrading to solid-js@1.9.9 makes solid compatible with seroval ~1.3

Summary by CodeRabbit

  • Chores
    • Upgraded SolidJS dependency across the repository from 1.9.5 → 1.9.9 to align examples, tests, and packages.
    • Added a global package override/pinning to ensure a consistent SolidJS version across development, examples, and CI, reducing version drift and easing reproducible builds.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 22, 2025

Walkthrough

Updated solid-js from ^1.9.5 → ^1.9.9 across examples, e2e suites, and core package.json files and added a top-level pnpm override pinning solid-js to 1.9.9.

Changes

Cohort / File(s) Summary
E2E Solid Router Tests
e2e/solid-router/basic-esbuild-file-based/package.json, e2e/solid-router/basic-file-based-code-splitting/package.json, e2e/solid-router/basic-file-based/package.json, e2e/solid-router/basic-scroll-restoration/package.json, e2e/solid-router/basic-solid-query-file-based/package.json, e2e/solid-router/basic-solid-query/package.json, e2e/solid-router/basic-virtual-file-based/package.json, e2e/solid-router/basic-virtual-named-export-config-file-based/package.json, e2e/solid-router/basic/package.json, e2e/solid-router/rspack-basic-file-based/package.json, e2e/solid-router/rspack-basic-virtual-named-export-config-file-based/package.json, e2e/solid-router/scroll-restoration-sandbox-vite/package.json
Bumped solid-js from ^1.9.5^1.9.9 in dependencies
E2E Solid Start Tests
e2e/solid-start/basic-auth/package.json, e2e/solid-start/basic-solid-query/package.json, e2e/solid-start/basic-tsr-config/package.json, e2e/solid-start/basic/package.json, e2e/solid-start/custom-basepath/package.json, e2e/solid-start/query-integration/package.json, e2e/solid-start/scroll-restoration/package.json, e2e/solid-start/selective-ssr/package.json, e2e/solid-start/server-functions/package.json, e2e/solid-start/server-routes/package.json, e2e/solid-start/spa-mode/package.json, e2e/solid-start/virtual-routes/package.json, e2e/solid-start/website/package.json
Bumped solid-js from ^1.9.5^1.9.9 in dependencies
Solid Example Projects
examples/solid/basic-devtools-panel/package.json, examples/solid/basic-file-based/package.json, examples/solid/basic-non-nested-devtools/package.json, examples/solid/basic-solid-query-file-based/package.json, examples/solid/basic-solid-query/package.json, examples/solid/basic-ssr-streaming-file-based/package.json, examples/solid/basic/package.json, examples/solid/kitchen-sink-file-based/package.json, examples/solid/kitchen-sink-solid-query-file-based/package.json, examples/solid/kitchen-sink-solid-query/package.json, examples/solid/quickstart-file-based/package.json, examples/solid/start-bare/package.json, examples/solid/start-basic-auth/package.json, examples/solid/start-basic-cloudflare/package.json, examples/solid/start-basic-netlify/package.json, examples/solid/start-basic-nitro/package.json, examples/solid/start-basic-solid-query/package.json, examples/solid/start-basic-static/package.json, examples/solid/start-basic/package.json
Bumped solid-js from ^1.9.5^1.9.9 in dependencies
Core Packages
packages/router-devtools-core/package.json, packages/solid-router-devtools/package.json, packages/solid-router-ssr-query/package.json, packages/solid-router/package.json, packages/solid-start-server/package.json
Bumped solid-js from ^1.9.5^1.9.9 in devDependencies and/or peerDependencies
Repository root
package.json
Added top-level pnpm.overrides pinning solid-js to 1.9.9

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • lachlancollins
  • schiller-manuel

Poem

"🐇 I hopped through package.json, neat and spry,
I nudged Solid from five to nine with a try,
Across examples, e2e, and core I did roam,
Pinned it at 1.9.9 — now it can go home! 🥕"

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "chore(solid): bump solid-js to ^1.9.9 to align seroval versions" accurately summarizes the main change in the changeset. The PR updates the solid-js dependency from ^1.9.5 to ^1.9.9 across all e2e test examples, example projects, and core packages in the repository, along with an override in the root package.json. The title uses a conventional commit format, is concise and clear, and includes both the what (bump solid-js) and the why (align seroval versions), which provides helpful context for developers reviewing the history.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bump-solid-1.9.9

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 293a1f3 and 186e670.

📒 Files selected for processing (1)
  • package.json (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • package.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Test
  • GitHub Check: 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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nx-cloud
Copy link

nx-cloud bot commented Oct 22, 2025

View your CI Pipeline Execution ↗ for commit 186e670

Command Status Duration Result
nx affected --targets=test:eslint,test:unit,tes... ✅ Succeeded 6m 33s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 1m 42s View ↗

☁️ Nx Cloud last updated this comment at 2025-10-23 00:05:29 UTC

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
examples/solid/quickstart-file-based/package.json (1)

13-14: Use workspace:* protocol for internal @TanStack dependencies per coding guidelines.

Internal dependencies should use the workspace protocol to ensure consistent versions across the monorepo. Update lines 13–14 and 22 to reference workspace versions.

  "dependencies": {
    "@tailwindcss/postcss": "^4.1.15",
-   "@tanstack/solid-router": "^1.133.24",
-   "@tanstack/solid-router-devtools": "^1.133.24",
+   "@tanstack/solid-router": "workspace:*",
+   "@tanstack/solid-router-devtools": "workspace:*",
    "postcss": "^8.5.1",
    "redaxios": "^0.5.1",
    "solid-js": "^1.9.9",
    "tailwindcss": "^4.1.15",
    "zod": "^3.24.2"
  },
  "devDependencies": {
-   "@tanstack/router-plugin": "^1.133.22",
+   "@tanstack/router-plugin": "workspace:*",
    "typescript": "^5.7.2",
    "vite": "^7.1.7",
    "vite-plugin-solid": "^2.11.10"
  }

As per coding guidelines.

Also applies to: 22-22

🧹 Nitpick comments (1)
examples/solid/start-bare/package.json (1)

12-14: Consider using workspace:* protocol for internal dependencies.

Per the coding guidelines, internal dependencies should use the workspace:* protocol instead of pinned versions. This applies to the @tanstack-scoped packages.

Apply this diff to align with the coding guidelines:

  "dependencies": {
-   "@tanstack/solid-router": "^1.133.24",
-   "@tanstack/solid-router-devtools": "^1.133.24",
-   "@tanstack/solid-start": "^1.133.24",
+   "@tanstack/solid-router": "workspace:*",
+   "@tanstack/solid-router-devtools": "workspace:*",
+   "@tanstack/solid-start": "workspace:*",
    "solid-js": "^1.9.9",

This change should be applied consistently across all internal package.json files in the monorepo if not already done. As per coding guidelines.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 88b6a8c and b779d95.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (49)
  • e2e/solid-router/basic-esbuild-file-based/package.json (1 hunks)
  • e2e/solid-router/basic-file-based-code-splitting/package.json (1 hunks)
  • e2e/solid-router/basic-file-based/package.json (1 hunks)
  • e2e/solid-router/basic-scroll-restoration/package.json (1 hunks)
  • e2e/solid-router/basic-solid-query-file-based/package.json (1 hunks)
  • e2e/solid-router/basic-solid-query/package.json (1 hunks)
  • e2e/solid-router/basic-virtual-file-based/package.json (1 hunks)
  • e2e/solid-router/basic-virtual-named-export-config-file-based/package.json (1 hunks)
  • e2e/solid-router/basic/package.json (1 hunks)
  • e2e/solid-router/rspack-basic-file-based/package.json (1 hunks)
  • e2e/solid-router/rspack-basic-virtual-named-export-config-file-based/package.json (1 hunks)
  • e2e/solid-router/scroll-restoration-sandbox-vite/package.json (1 hunks)
  • e2e/solid-start/basic-auth/package.json (1 hunks)
  • e2e/solid-start/basic-solid-query/package.json (1 hunks)
  • e2e/solid-start/basic-tsr-config/package.json (1 hunks)
  • e2e/solid-start/basic/package.json (1 hunks)
  • e2e/solid-start/custom-basepath/package.json (1 hunks)
  • e2e/solid-start/query-integration/package.json (1 hunks)
  • e2e/solid-start/scroll-restoration/package.json (1 hunks)
  • e2e/solid-start/selective-ssr/package.json (1 hunks)
  • e2e/solid-start/server-functions/package.json (1 hunks)
  • e2e/solid-start/server-routes/package.json (1 hunks)
  • e2e/solid-start/spa-mode/package.json (1 hunks)
  • e2e/solid-start/virtual-routes/package.json (1 hunks)
  • e2e/solid-start/website/package.json (1 hunks)
  • examples/solid/basic-devtools-panel/package.json (1 hunks)
  • examples/solid/basic-file-based/package.json (1 hunks)
  • examples/solid/basic-non-nested-devtools/package.json (1 hunks)
  • examples/solid/basic-solid-query-file-based/package.json (1 hunks)
  • examples/solid/basic-solid-query/package.json (1 hunks)
  • examples/solid/basic-ssr-streaming-file-based/package.json (1 hunks)
  • examples/solid/basic/package.json (1 hunks)
  • examples/solid/kitchen-sink-file-based/package.json (1 hunks)
  • examples/solid/kitchen-sink-solid-query-file-based/package.json (1 hunks)
  • examples/solid/kitchen-sink-solid-query/package.json (1 hunks)
  • examples/solid/quickstart-file-based/package.json (1 hunks)
  • examples/solid/start-bare/package.json (1 hunks)
  • examples/solid/start-basic-auth/package.json (1 hunks)
  • examples/solid/start-basic-cloudflare/package.json (1 hunks)
  • examples/solid/start-basic-netlify/package.json (1 hunks)
  • examples/solid/start-basic-nitro/package.json (1 hunks)
  • examples/solid/start-basic-solid-query/package.json (1 hunks)
  • examples/solid/start-basic-static/package.json (1 hunks)
  • examples/solid/start-basic/package.json (1 hunks)
  • packages/router-devtools-core/package.json (1 hunks)
  • packages/solid-router-devtools/package.json (1 hunks)
  • packages/solid-router-ssr-query/package.json (1 hunks)
  • packages/solid-router/package.json (1 hunks)
  • packages/solid-start-server/package.json (1 hunks)
🧰 Additional context used
📓 Path-based instructions (5)
**/package.json

📄 CodeRabbit inference engine (AGENTS.md)

Use workspace:* protocol for internal dependencies in package.json files

Files:

  • e2e/solid-start/basic-auth/package.json
  • examples/solid/start-basic-auth/package.json
  • examples/solid/start-basic-cloudflare/package.json
  • e2e/solid-router/basic-file-based/package.json
  • e2e/solid-router/basic-solid-query-file-based/package.json
  • e2e/solid-start/scroll-restoration/package.json
  • e2e/solid-router/basic-virtual-file-based/package.json
  • examples/solid/start-bare/package.json
  • examples/solid/basic-non-nested-devtools/package.json
  • e2e/solid-router/basic/package.json
  • e2e/solid-router/basic-file-based-code-splitting/package.json
  • examples/solid/basic-file-based/package.json
  • e2e/solid-router/basic-solid-query/package.json
  • examples/solid/basic-devtools-panel/package.json
  • examples/solid/start-basic-nitro/package.json
  • packages/router-devtools-core/package.json
  • packages/solid-start-server/package.json
  • e2e/solid-start/spa-mode/package.json
  • e2e/solid-router/rspack-basic-file-based/package.json
  • e2e/solid-start/website/package.json
  • examples/solid/basic-solid-query-file-based/package.json
  • examples/solid/basic/package.json
  • e2e/solid-start/basic-tsr-config/package.json
  • e2e/solid-start/query-integration/package.json
  • examples/solid/kitchen-sink-solid-query/package.json
  • examples/solid/quickstart-file-based/package.json
  • e2e/solid-start/virtual-routes/package.json
  • examples/solid/kitchen-sink-solid-query-file-based/package.json
  • examples/solid/kitchen-sink-file-based/package.json
  • e2e/solid-start/server-routes/package.json
  • e2e/solid-router/basic-esbuild-file-based/package.json
  • e2e/solid-start/custom-basepath/package.json
  • examples/solid/basic-ssr-streaming-file-based/package.json
  • e2e/solid-start/server-functions/package.json
  • packages/solid-router-devtools/package.json
  • packages/solid-router-ssr-query/package.json
  • examples/solid/start-basic/package.json
  • e2e/solid-start/selective-ssr/package.json
  • examples/solid/start-basic-static/package.json
  • e2e/solid-router/basic-virtual-named-export-config-file-based/package.json
  • e2e/solid-router/rspack-basic-virtual-named-export-config-file-based/package.json
  • examples/solid/start-basic-solid-query/package.json
  • examples/solid/basic-solid-query/package.json
  • e2e/solid-start/basic/package.json
  • e2e/solid-router/scroll-restoration-sandbox-vite/package.json
  • e2e/solid-start/basic-solid-query/package.json
  • examples/solid/start-basic-netlify/package.json
  • e2e/solid-router/basic-scroll-restoration/package.json
  • packages/solid-router/package.json
e2e/**

📄 CodeRabbit inference engine (AGENTS.md)

Store end-to-end tests under the e2e/ directory

Files:

  • e2e/solid-start/basic-auth/package.json
  • e2e/solid-router/basic-file-based/package.json
  • e2e/solid-router/basic-solid-query-file-based/package.json
  • e2e/solid-start/scroll-restoration/package.json
  • e2e/solid-router/basic-virtual-file-based/package.json
  • e2e/solid-router/basic/package.json
  • e2e/solid-router/basic-file-based-code-splitting/package.json
  • e2e/solid-router/basic-solid-query/package.json
  • e2e/solid-start/spa-mode/package.json
  • e2e/solid-router/rspack-basic-file-based/package.json
  • e2e/solid-start/website/package.json
  • e2e/solid-start/basic-tsr-config/package.json
  • e2e/solid-start/query-integration/package.json
  • e2e/solid-start/virtual-routes/package.json
  • e2e/solid-start/server-routes/package.json
  • e2e/solid-router/basic-esbuild-file-based/package.json
  • e2e/solid-start/custom-basepath/package.json
  • e2e/solid-start/server-functions/package.json
  • e2e/solid-start/selective-ssr/package.json
  • e2e/solid-router/basic-virtual-named-export-config-file-based/package.json
  • e2e/solid-router/rspack-basic-virtual-named-export-config-file-based/package.json
  • e2e/solid-start/basic/package.json
  • e2e/solid-router/scroll-restoration-sandbox-vite/package.json
  • e2e/solid-start/basic-solid-query/package.json
  • e2e/solid-router/basic-scroll-restoration/package.json
examples/{react,solid}/**

📄 CodeRabbit inference engine (AGENTS.md)

Keep example applications under examples/react/ and examples/solid/

Files:

  • examples/solid/start-basic-auth/package.json
  • examples/solid/start-basic-cloudflare/package.json
  • examples/solid/start-bare/package.json
  • examples/solid/basic-non-nested-devtools/package.json
  • examples/solid/basic-file-based/package.json
  • examples/solid/basic-devtools-panel/package.json
  • examples/solid/start-basic-nitro/package.json
  • examples/solid/basic-solid-query-file-based/package.json
  • examples/solid/basic/package.json
  • examples/solid/kitchen-sink-solid-query/package.json
  • examples/solid/quickstart-file-based/package.json
  • examples/solid/kitchen-sink-solid-query-file-based/package.json
  • examples/solid/kitchen-sink-file-based/package.json
  • examples/solid/basic-ssr-streaming-file-based/package.json
  • examples/solid/start-basic/package.json
  • examples/solid/start-basic-static/package.json
  • examples/solid/start-basic-solid-query/package.json
  • examples/solid/basic-solid-query/package.json
  • examples/solid/start-basic-netlify/package.json
packages/{router-devtools,*-router-devtools}/**

📄 CodeRabbit inference engine (AGENTS.md)

Keep router devtools packages in packages/router-devtools/ and packages/*-router-devtools/

Files:

  • packages/solid-router-devtools/package.json
packages/{react-router,solid-router}/**

📄 CodeRabbit inference engine (AGENTS.md)

Implement React and Solid bindings/components only in packages/react-router/ and packages/solid-router/

Files:

  • packages/solid-router/package.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Preview
  • GitHub Check: Test
🔇 Additional comments (47)
examples/solid/start-basic-nitro/package.json (1)

15-15: LGTM!

The solid-js dependency is correctly updated to ^1.9.9, aligning with the PR objective to bump versions across the monorepo.

examples/solid/basic-ssr-streaming-file-based/package.json (1)

23-23: Version bump aligns with PR objective.

The solid-js dependency has been correctly updated from ^1.9.5 to ^1.9.9, maintaining backward compatibility while allowing the newer patch versions as intended.

examples/solid/start-basic-netlify/package.json (1)

15-15: LGTM!

The solid-js version bump from ^1.9.5 to ^1.9.9 is a straightforward patch-level update within the minor version range (1.9.x), ensuring backward compatibility. The change aligns with the PR objective to bump solid-js and align seroval versions across the repository.

examples/solid/start-bare/package.json (1)

15-15: Solid-js dependency bump looks good!

The version update from ^1.9.5 to ^1.9.9 aligns with the PR objective.

examples/solid/basic-file-based/package.json (1)

17-17: ✓ Solid-js version bump is correct and aligned with PR objectives.

The dependency has been updated from ^1.9.5 to ^1.9.9 as intended. The file location adheres to the examples/solid/ directory structure, and no other modifications have affected build behavior or configuration.

examples/solid/start-basic-cloudflare/package.json (1)

18-18: Approve solid-js version bump.

The version upgrade from ^1.9.5 to ^1.9.9 is correctly specified and aligns with the PR objectives to align seroval versions. The semver range (^1.9.9) is appropriate for a library bump.

e2e/solid-router/basic/package.json (1)

19-19: LGTM!

Version bump from ^1.9.5 to ^1.9.9 is applied correctly, and internal dependencies correctly use the workspace protocol.

e2e/solid-router/basic-file-based/package.json (1)

21-21: LGTM!

Version bump applied correctly with proper workspace protocol for internal dependencies.

e2e/solid-router/basic-virtual-file-based/package.json (1)

21-21: LGTM!

Version bump applied correctly with proper workspace protocol for internal dependencies.

examples/solid/basic/package.json (1)

17-17: LGTM!

Solid-js version bump applied correctly.

e2e/solid-start/spa-mode/package.json (1)

17-17: LGTM!

Version bump applied correctly with proper workspace protocol for internal dependencies.

examples/solid/basic-devtools-panel/package.json (1)

17-17: LGTM!

Solid-js version bump applied correctly.

examples/solid/start-basic/package.json (1)

16-16: LGTM!

Solid-js version bump applied correctly.

examples/solid/start-basic-solid-query/package.json (1)

19-19: LGTM!

Solid-js version bump applied correctly.

examples/solid/basic-solid-query/package.json (1)

19-19: Dependency version bump looks good.

Updating solid-js from ^1.9.5 to ^1.9.9 is a safe minor version bump that aligns with the PR objectives.

examples/solid/start-basic-static/package.json (1)

16-16: Dependency version bump is consistent across the PR.

packages/solid-router-devtools/package.json (1)

73-73: Dependency updates maintain consistency in both dev and peer dependencies.

Both devDependencies and peerDependencies are updated to ^1.9.9, ensuring the devtools package and its consumers use the aligned version. The use of workspace:^ for internal dependencies follows the coding guidelines.

Also applies to: 78-78

examples/solid/basic-solid-query-file-based/package.json (1)

20-20: Dependency version bump is consistent.

e2e/solid-router/basic-solid-query-file-based/package.json (1)

22-22: Dependency version bump is consistent and workspace protocol usage is correct.

e2e/solid-router/scroll-restoration-sandbox-vite/package.json (1)

24-24: Dependency version bump is consistent.

examples/solid/kitchen-sink-solid-query-file-based/package.json (1)

21-21: Dependency version bump is consistent.

e2e/solid-router/basic-scroll-restoration/package.json (1)

20-20: Dependency version bump is consistent.

e2e/solid-router/basic-virtual-named-export-config-file-based/package.json (1)

21-21: LGTM!

Straightforward patch version bump from ^1.9.5 to ^1.9.9, with all internal dependencies correctly using the workspace:^ protocol per coding guidelines.

e2e/solid-start/basic-solid-query/package.json (1)

21-21: LGTM!

Patch version update (^1.9.5 → ^1.9.9) with all internal dependencies correctly using the workspace:^ protocol.

e2e/solid-start/virtual-routes/package.json (1)

19-19: LGTM!

Patch version update (^1.9.5 → ^1.9.9) with internal dependencies correctly using the workspace:^ protocol.

e2e/solid-start/selective-ssr/package.json (1)

16-16: LGTM!

Patch version update (^1.9.5 → ^1.9.9) with internal dependencies correctly using the workspace:^ protocol.

packages/solid-start-server/package.json (1)

69-69: LGTM!

Patch version update (^1.9.5 → ^1.9.9) in devDependencies. Internal dependencies correctly use the workspace:* protocol. The peerDependencies requirement (^1.0.0) remains satisfied.

e2e/solid-start/website/package.json (1)

18-18: LGTM!

Patch version update (^1.9.5 → ^1.9.9) with internal dependencies correctly using the workspace:^ protocol.

e2e/solid-start/basic-auth/package.json (1)

21-21: LGTM!

Patch version update (^1.9.5 → ^1.9.9) with internal dependencies correctly using the workspace:^ protocol.

examples/solid/kitchen-sink-solid-query/package.json (1)

20-20: LGTM!

Patch version update (^1.9.5 → ^1.9.9) in this example project. Dependencies are using published versions, which is appropriate for example applications.

e2e/solid-start/basic-tsr-config/package.json (1)

17-17: Clean dependency version bump.

The solid-js update from ^1.9.5 to ^1.9.9 is consistent across the repository and aligns with the PR objective to align seroval versions. Internal dependencies correctly use the workspace:* protocol.

e2e/solid-router/basic-file-based-code-splitting/package.json (1)

21-21: Clean dependency version bump.

The solid-js update is consistent and properly applied. Internal @tanstack/* dependencies correctly use workspace:* protocol.

examples/solid/basic-non-nested-devtools/package.json (1)

17-17: Dependency update applied correctly.

The solid-js version is updated from ^1.9.5 to ^1.9.9. Note: This example file uses absolute version constraints (^1.133.24) for internal @tanstack/* dependencies rather than workspace:*, which differs from e2e test conventions but may be intentional for released examples.

e2e/solid-router/basic-solid-query/package.json (1)

21-21: Clean dependency version bump.

The solid-js update is applied consistently. Internal dependencies correctly use workspace:* protocol.

e2e/solid-router/basic-esbuild-file-based/package.json (1)

18-18: Clean dependency version bump.

The solid-js update from ^1.9.5 to ^1.9.9 is applied consistently. Internal @tanstack/* dependencies and workspace protocols are correct.

e2e/solid-start/custom-basepath/package.json (1)

18-18: Clean dependency version bump.

The solid-js update is consistent and properly applied across this file. Internal dependencies correctly use workspace:* protocol.

e2e/solid-start/server-routes/package.json (1)

19-19: Clean dependency version bump.

The solid-js update is applied consistently with the repository changes. Internal dependencies correctly use workspace:* protocol.

examples/solid/start-basic-auth/package.json (1)

19-19: Dependency update applied correctly.

The solid-js version is updated from ^1.9.5 to ^1.9.9. As with other example files in this PR, this uses absolute version constraints for internal @tanstack/* dependencies rather than workspace:*.

e2e/solid-router/rspack-basic-virtual-named-export-config-file-based/package.json (1)

16-16: Version bump is consistent with PR objective.

The solid-js dependency update from ^1.9.5 to ^1.9.9 aligns with the PR goal to align seroval versions. No other changes to dependencies or configuration.

examples/solid/kitchen-sink-file-based/package.json (1)

18-18: Version bump applied correctly.

The solid-js dependency is updated from ^1.9.5 to ^1.9.9 as intended. No other changes.

e2e/solid-router/rspack-basic-file-based/package.json (1)

16-16: Version bump is appropriate.

solid-js dependency updated from ^1.9.5 to ^1.9.9 to align with seroval versions, as intended.

e2e/solid-start/scroll-restoration/package.json (1)

19-19: Version bump consistent with PR objective.

solid-js dependency updated from ^1.9.5 to ^1.9.9. No other modifications.

e2e/solid-start/basic/package.json (1)

24-24: Version bump aligns with PR intent.

solid-js dependency updated from ^1.9.5 to ^1.9.9 to align seroval versions. Build and test scripts remain unchanged.

e2e/solid-start/query-integration/package.json (1)

20-20: Version bump appropriate for this e2e suite.

solid-js dependency updated from ^1.9.5 to ^1.9.9 without affecting other dependencies or scripts.

e2e/solid-start/server-functions/package.json (1)

19-19: Version bump is correct.

solid-js dependency updated from ^1.9.5 to ^1.9.9 as part of the PR's seroval alignment effort.

packages/solid-router-ssr-query/package.json (1)

73-73: Version bump correctly applied to both devDependencies and peerDependencies.

solid-js updated from ^1.9.5 to ^1.9.9 in both scopes (lines 73 and 80), ensuring consistency for this core package. Consumers will correctly understand the supported solid-js version range.

Also applies to: 80-80

packages/solid-router/package.json (1)

116-116: Consistent version bump across devDependencies and peerDependencies.

Both constraints updated to ^1.9.9 uniformly, ensuring package consumers and developers have aligned solid-js versions.

Also applies to: 121-121

},
"devDependencies": {
"solid-js": "^1.9.5",
"solid-js": "^1.9.9",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Update peerDependencies to match devDependencies version constraint.

The devDependencies are bumped to ^1.9.9, but peerDependencies remain at >=1.9.5. This inconsistency allows consumers to install incompatible solid-js versions and creates potential runtime mismatches. Align peerDependencies to ^1.9.9 for the intended upgrade.

  "peerDependencies": {
    "@tanstack/router-core": "workspace:^",
-   "solid-js": ">=1.9.5",
+   "solid-js": "^1.9.9",
    "tiny-invariant": "^1.3.3"
  },

Also applies to: 76-76

🤖 Prompt for AI Agents
packages/router-devtools-core/package.json lines 70 and 76: peerDependencies for
solid-js are still set to ">=1.9.5" while devDependencies were bumped to
"^1.9.9"; update the peerDependencies entries to use the same version constraint
"^1.9.9" so peer and dev constraints align and consumers cannot install an
incompatible solid-js version.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 22, 2025

More templates

@tanstack/arktype-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/arktype-adapter@5586

@tanstack/directive-functions-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/directive-functions-plugin@5586

@tanstack/eslint-plugin-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/eslint-plugin-router@5586

@tanstack/history

npm i https://pkg.pr.new/TanStack/router/@tanstack/history@5586

@tanstack/nitro-v2-vite-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/nitro-v2-vite-plugin@5586

@tanstack/react-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router@5586

@tanstack/react-router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router-devtools@5586

@tanstack/react-router-ssr-query

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router-ssr-query@5586

@tanstack/react-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start@5586

@tanstack/react-start-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-client@5586

@tanstack/react-start-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-server@5586

@tanstack/router-cli

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-cli@5586

@tanstack/router-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-core@5586

@tanstack/router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-devtools@5586

@tanstack/router-devtools-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-devtools-core@5586

@tanstack/router-generator

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-generator@5586

@tanstack/router-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-plugin@5586

@tanstack/router-ssr-query-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-ssr-query-core@5586

@tanstack/router-utils

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-utils@5586

@tanstack/router-vite-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-vite-plugin@5586

@tanstack/server-functions-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/server-functions-plugin@5586

@tanstack/solid-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router@5586

@tanstack/solid-router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router-devtools@5586

@tanstack/solid-router-ssr-query

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router-ssr-query@5586

@tanstack/solid-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start@5586

@tanstack/solid-start-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-client@5586

@tanstack/solid-start-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-server@5586

@tanstack/start-client-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-client-core@5586

@tanstack/start-plugin-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-plugin-core@5586

@tanstack/start-server-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-core@5586

@tanstack/start-static-server-functions

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-static-server-functions@5586

@tanstack/start-storage-context

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-storage-context@5586

@tanstack/valibot-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/valibot-adapter@5586

@tanstack/virtual-file-routes

npm i https://pkg.pr.new/TanStack/router/@tanstack/virtual-file-routes@5586

@tanstack/zod-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/zod-adapter@5586

commit: 186e670

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b779d95 and 293a1f3.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**/package.json

📄 CodeRabbit inference engine (AGENTS.md)

Use workspace:* protocol for internal dependencies in package.json files

Files:

  • package.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Test
  • GitHub Check: Preview

@birkskyum birkskyum merged commit 47bc1c3 into main Oct 23, 2025
6 checks passed
@birkskyum birkskyum deleted the bump-solid-1.9.9 branch October 23, 2025 00:09
@birkskyum birkskyum added this to the catch up solid to react milestone Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants