Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TanStack Start: adjust to new streaming method post 1.99 #422

Merged
merged 14 commits into from
Feb 6, 2025

Conversation

phryneas
Copy link
Member

@phryneas phryneas commented Feb 3, 2025

No description provided.

Copy link

changeset-bot bot commented Feb 3, 2025

⚠️ No Changeset found

Latest commit: 3c98038

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Feb 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
apollo__client-integration-react-router ✅ Ready (Inspect) Visit Preview Feb 5, 2025 2:12pm
apollo__client-integration-tanstack-start 🛑 Canceled (Inspect) Feb 5, 2025 2:12pm

Copy link

pkg-pr-new bot commented Feb 3, 2025

npm i https://pkg.pr.new/apollographql/apollo-client-nextjs/@apollo/client-integration-react-router@422
npm i https://pkg.pr.new/apollographql/apollo-client-nextjs/@apollo/client-integration-tanstack-start@422
npm i https://pkg.pr.new/apollographql/apollo-client-nextjs/@apollo/client-react-streaming@422
npm i https://pkg.pr.new/apollographql/apollo-client-nextjs/@apollo/experimental-nextjs-app-support@422

commit: 55624a8

Copy link

relativeci bot commented Feb 3, 2025

#441 Bundle Size — 1.33MiB (+6.75%).

55624a8(current) vs 5293db1 main#409(baseline)

Warning

Bundle contains 1 duplicate package – View duplicate packages

Warning

Bundle introduced one new package: process – View changed packages

Bundle metrics  Change 8 changes Regression 1 regression Improvement 2 improvements
                 Current
#441
     Baseline
#409
Regression  Initial JS 1.05MiB(+5.68%) 1016.91KiB
No change  Initial CSS 70B 70B
Change  Cache Invalidation 81.12% 0.08%
Change  Chunks 37(+8.82%) 34
Change  Assets 63(+6.78%) 59
Change  Modules 676(+6.29%) 636
Improvement  Duplicate Modules 98(-7.55%) 106
Change  Duplicate Code 4.33%(-7.87%) 4.7%
Improvement  Packages 25(-3.85%) 26
No change  Duplicate Packages 1 1
Bundle size by type  Change 3 changes Regression 3 regressions
                 Current
#441
     Baseline
#409
Regression  JS 1.32MiB (+100%) undefined
Regression  Other 10.27KiB (+100%) undefined
Regression  CSS 70B (+100%) undefined

Bundle analysis reportBranch pr/adjust-tanstack-streamingProject dashboard


Generated by RelativeCIDocumentationReport issue

@phryneas phryneas changed the title adjust to new streaming method TanStack Start: adjust to new streaming method post 1.99 Feb 4, 2025
@phryneas
Copy link
Member Author

phryneas commented Feb 4, 2025

This works now, but I'm waiting for a small last change on the TanStack side so we can use hydrate instead of the useRouterState workaround.

@phryneas
Copy link
Member Author

phryneas commented Feb 5, 2025

TanStack/router@a4ae8ec has landed in 1.99.4 so this should be good to go if it successfully deploys.

"@tanstack/start": "patch:@tanstack/start@npm%3A1.97.3#~/.yarn/patches/@tanstack-start-npm-1.97.3-da273c2822.patch",
"@tanstack/start": "1.99.6",
"@tanstack/react-router": "1.99.6",
"@tanstack/start-router-manifest": "1.99.0",
Copy link
Member Author

Choose a reason for hiding this comment

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

This is currently pinned to 1.99.0 to prevent a HMR problem on the TanStack side

// so "loading" is kicked off before the respecitve component is rendered
consumeBackPressure();
if (onQueryEvent) {
registerLateInitializingQueue("__APOLLO_EVENTS__", onQueryEvent);
Copy link
Member Author

Choose a reason for hiding this comment

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

Yay for reusing tools we already have :)

@phryneas
Copy link
Member Author

phryneas commented Feb 5, 2025

Yup, this seems to work nicely. Ready for review!

@phryneas phryneas merged commit c01a77b into next Feb 6, 2025
25 checks passed
@phryneas phryneas deleted the pr/adjust-tanstack-streaming branch February 6, 2025 08:46
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.

2 participants