Skip to content

fix: preserve route context during hydration recovery - #1341

Open
tannerlinsley wants to merge 1 commit into
mainfrom
taren/adopt-redact-0-1-4
Open

tannerlinsley wants to merge 1 commit into
mainfrom
taren/adopt-redact-0-1-4

Conversation

@tannerlinsley

@tannerlinsley tannerlinsley commented Oct 4, 2026 •

Copy link
Copy Markdown
Member

Client navigation can render an obsolete route outlet after a hydration mismatch, causing RenderErrorCapture and a missing route-match exception. Pin @tanstack/redact to the published 0.1.4 repair, which preserves component and context-provider ancestry during recovery.

Only the package manifest and renderer lockfile entries change. Charts remains on 0.16.1.

Validation: frozen install, full pnpm test, production build, repeated docs-to-home navigation, homepage and catalog rendering, logged-out chat access, and defined HTTP 200 feedback responses against the published package without a local renderer override. Renderer fix and release: TanStack/redact#40 and TanStack/redact#41.

Summary by CodeRabbit

  • Chores
    • Updated internal project tooling configuration. No changes to the app’s features or behavior are included in this update.

@tannerlinsley
tannerlinsley requested a review from a team October 4, 2026 19:57
@changeset-bot

changeset-bot Bot commented Oct 4, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 92ff961

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.

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

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
tanstack-com 92ff961 Oct 04 2026, 07:59 PM

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​tanstack/​redact@​0.1.2 ⏵ 0.1.485 +410074 +197 +1100

View full report

@coderabbitai

coderabbitai Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 9331f5cc-58a0-4b1e-bfcd-384c0989c2dd
📥 Commits

Reviewing files that changed from the base of the PR and between 0113dd2 and 92ff961.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

The package manifest now pins the @tanstack/redact devDependency to version 0.1.4 instead of using the ^0.1.2 version range.

Changes

Redact dependency pin

Layer / File(s) Summary
Pin the dependency version
package.json
@tanstack/redact now uses the exact version 0.1.4 instead of the ^0.1.2 range.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Suggested reviewers: ladybluenotes

Merge Risk: ⚪ Minimal · up to 92ff9

The dependency is pinned to 0.1.4, matching the lockfile. No concrete issue remains that should block merging.

Architecture Summary

Architecture risk: 🔵 Low · up to 92ff9

The change affects 1 system.

Changed systems: package.json

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — package.json (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in package.json: @tanstack/redact changes from the ^0.1.2 version range to the exact version 0.1.4.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title describes the main objective: preserve route context during hydration recovery by updating @tanstack/redact to its repair release.
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…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ 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

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.

This branch has not been deployed

No deployments
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.

1 participant