Fix hydration mismatch for ssr='data-only' with pendingComponent#7266
Fix hydration mismatch for ssr='data-only' with pendingComponent#7266
Conversation
|
@copilot Lets try gpt 5.5 |
|
View your CI Pipeline Execution ↗ for commit 3ff8326
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview1 package(s) bumped directly, 3 bumped as dependents. 🟩 Patch bumps
|
Bundle Size Benchmarks
Trend sparkline is historical gzip bytes ending with this PR measurement; lower is better. |
| const browserErrors: Array<string> = [] | ||
|
|
||
| page.on('pageerror', (error) => { | ||
| browserErrors.push(error.message) | ||
| }) | ||
|
|
||
| page.on('console', (message) => { | ||
| if (message.type() === 'error' || message.type() === 'warning') { | ||
| browserErrors.push(message.text()) | ||
| } | ||
| }) | ||
|
|
||
| page.on('requestfailed', (request) => { | ||
| browserErrors.push( | ||
| `${request.url()} ${request.failure()?.errorText ?? 'failed'}`, | ||
| ) | ||
| }) |
There was a problem hiding this comment.
@copilot do we have all this in a util somewhere ? it seems quite generic
There was a problem hiding this comment.
Added a shared collectBrowserErrors helper in @tanstack/router-e2e-utils and updated this test to use it. Addressed in 92177e3.
There was a problem hiding this comment.
Resolved by the existing 92177e3 follow-up: the generic browser error listener is now collectBrowserErrors in @tanstack/router-e2e-utils, and this spec imports it instead of defining it inline.
|
@copilot what is mre? |
MRE means “minimal reproducible example” — in this PR it’s the small route added to reproduce the hydration mismatch. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Agent-Logs-Url: https://github.com/TanStack/router/sessions/aae41f53-16ad-426b-a9bb-36e4d8c9ffcf Co-authored-by: birkskyum <74932975+birkskyum@users.noreply.github.com>
|
@copilot rename the route from "routes/mre-data-only.tsx" to something more descriptive (don't include MRE, because all routes are practively mre's) |
Merging this PR will not alter performance
Comparing Footnotes
|
Agent-Logs-Url: https://github.com/TanStack/router/sessions/0a1fcc3c-48db-4f47-b20b-72dee8d1ff44 Co-authored-by: birkskyum <74932975+birkskyum@users.noreply.github.com>
Renamed the route to Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
There was a problem hiding this comment.
Nx Cloud has identified a flaky task in your failed CI:
🔂 Since the failure was identified as flaky, we triggered a CI rerun by adding an empty commit to this branch.
🎓 Learn more about Self-Healing CI on nx.dev
Closes #7085
Fixes the Solid Router
resolvedNoSsrcondition sossr: 'data-only'routes with apendingComponenthydrate correctly without duplicating suspense fallback output.Adds Solid Start selective SSR regression coverage for the data-only pending component hydration case, and shares browser error collection through
@tanstack/router-e2e-utils.