Skip to content

refactor(solid-query): solid 2 reactivity compliance - #10316

Merged
birkskyum merged 4 commits into
TanStack:solid-query-v6-prefrom
2wheeh:refactor/v2-teaks
Mar 23, 2026
Merged

refactor(solid-query): solid 2 reactivity compliance#10316
birkskyum merged 4 commits into
TanStack:solid-query-v6-prefrom
2wheeh:refactor/v2-teaks

Conversation

@2wheeh

@2wheeh2wheeh commented Mar 22, 2026

Copy link
Copy Markdown
Contributor

🎯 Changes

From RFC: Reactivity, batching, and effects,

  • 2.0 introduces strict top-level access - explicit untrack is necessary for the top-level reactive reads in component body.
  • No writes under owned scope - signals that sync external state (e.g., query cache subscriptions) use { pureWrite: true } to opt out of the dev warning, as these are one-way writes from external callbacks, not reactive feedback loops.
  • And warning for no destructuring reactive proxies: destructuring a store proxy (const { status } = useQuery(...)) evaluates the property immediately, losing reactivity. Accessing via state.status preserves tracking.

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm run test:pr.

🚀 Release Impact

  • This change affects published code, and I have generated a changeset.

@coderabbitai

coderabbitaiBot commented Mar 22, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e4be1e9b-f372-4d41-9235-6a64ba222229

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

nx-cloudBot commented Mar 22, 2026•
edited
Loading

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 72ddf97

CommandStatusDurationResult
nx affected --targets=test:sherif,test:knip,tes...✅ Succeeded2m 15sView ↗
nx run-many --target=build --exclude=examples/*...✅ Succeeded42sView ↗

☁️ Nx Cloud last updated this comment at 2026-03-23 06:07:02 UTC

@pkg-pr-new

pkg-pr-newBot commented Mar 22, 2026

Copy link
Copy Markdown
More templates

@tanstack/angular-query-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-experimental@10316

@tanstack/eslint-plugin-query

npm i https://pkg.pr.new/@tanstack/eslint-plugin-query@10316

@tanstack/preact-query

npm i https://pkg.pr.new/@tanstack/preact-query@10316

@tanstack/preact-query-devtools

npm i https://pkg.pr.new/@tanstack/preact-query-devtools@10316

@tanstack/preact-query-persist-client

npm i https://pkg.pr.new/@tanstack/preact-query-persist-client@10316

@tanstack/query-async-storage-persister

npm i https://pkg.pr.new/@tanstack/query-async-storage-persister@10316

@tanstack/query-broadcast-client-experimental

npm i https://pkg.pr.new/@tanstack/query-broadcast-client-experimental@10316

@tanstack/query-core

npm i https://pkg.pr.new/@tanstack/query-core@10316

@tanstack/query-devtools

npm i https://pkg.pr.new/@tanstack/query-devtools@10316

@tanstack/query-persist-client-core

npm i https://pkg.pr.new/@tanstack/query-persist-client-core@10316

@tanstack/query-sync-storage-persister

npm i https://pkg.pr.new/@tanstack/query-sync-storage-persister@10316

@tanstack/react-query

npm i https://pkg.pr.new/@tanstack/react-query@10316

@tanstack/react-query-devtools

npm i https://pkg.pr.new/@tanstack/react-query-devtools@10316

@tanstack/react-query-next-experimental

npm i https://pkg.pr.new/@tanstack/react-query-next-experimental@10316

@tanstack/react-query-persist-client

npm i https://pkg.pr.new/@tanstack/react-query-persist-client@10316

@tanstack/solid-query

npm i https://pkg.pr.new/@tanstack/solid-query@10316

@tanstack/solid-query-devtools

npm i https://pkg.pr.new/@tanstack/solid-query-devtools@10316

@tanstack/solid-query-persist-client

npm i https://pkg.pr.new/@tanstack/solid-query-persist-client@10316

@tanstack/svelte-query

npm i https://pkg.pr.new/@tanstack/svelte-query@10316

@tanstack/svelte-query-devtools

npm i https://pkg.pr.new/@tanstack/svelte-query-devtools@10316

@tanstack/svelte-query-persist-client

npm i https://pkg.pr.new/@tanstack/svelte-query-persist-client@10316

@tanstack/vue-query

npm i https://pkg.pr.new/@tanstack/vue-query@10316

@tanstack/vue-query-devtools

npm i https://pkg.pr.new/@tanstack/vue-query-devtools@10316

commit: 2641790

@2wheeh
2wheeh marked this pull request as ready for review March 22, 2026 16:41
@2wheeh

2wheeh commented Mar 23, 2026

Copy link
Copy Markdown
ContributorAuthor

@brenelz@birkskyum Hi, I added some tweaks to reduce warnings

@birkskyum
birkskyum requested a review from brenelzMarch 23, 2026 11:32
@birkskyum
birkskyum merged commit 76fa68e into TanStack:solid-query-v6-preMar 23, 2026
5 of 7 checks passed
@2wheeh
2wheeh deleted the refactor/v2-teaks branch March 23, 2026 12:51
@github-actionsgithub-actionsBot mentioned this pull request Mar 23, 2026
Merged
@ryansolidryansolid mentioned this pull request Aug 11, 2026
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@birkskyumbirkskyumbirkskyum approved these changes
@brenelzbrenelzAwaiting requested review from brenelz

Assignees

No one assigned

Labels

package: solid-query

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@2wheeh@birkskyum