You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Fix missing-chunk recovery with the released renderer - #1501
Missing route chunks could trap the renderer in repeated error-boundary handling before the site could recover. Upgrade Redact from 0.1.4 to 0.2.5, which includes the upstream nested error-boundary fix in TanStack/redact#109.
Align Cloudflare Vite with Redact's supported 1.65.0 peer and Wrangler with that plugin's required version. Use Vite 8's native default minifier, because the Cloudflare plugin's ES2026 worker target is rejected by esbuild, including 0.28.2.
Related to #734. Keep the issue open for deployed verification.
AI review notes: pnpm test passed on the aligned dependencies. Both focused recovery/Sentry tests passed, including reporting failures during the reload cooldown. Production build passed after removing the esbuild override.
In a local production preview of this candidate, removing the Blog chunk and navigating from Tenets reached the error UI with two captured missing-module errors, without the prior runaway loop. Restoring the chunk and manually reloading rendered the Blog. Exact automatic reload count and live Sentry delivery were not measured.
Against main 477d87b, total gzip across all 680 client JavaScript assets decreased from 6,572,757 to 6,455,610 bytes, a reduction of 117,147 bytes. This is total emitted JavaScript, not initial page transfer size.
Required CI and protected dependency/config review remain before merge. No production deployment has been verified.
Summary by CodeRabbit
Chores
Updated the versions of the project’s build and development tools.
Adjusted the production build configuration. These changes do not add or remove any user-facing features, and no changes to app functionality are described in this update.
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.
No merge-blocking issue is identified. Normal CI and deployment verification remain appropriate.
Pre-merge checks: 5
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
The title clearly describes the main change: upgrading the renderer to fix missing-chunk recovery.
Docstring Coverage
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Finishing Touches
Generate docstrings
Commit to this branch
Create a new PR
Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · 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.
File: No configuration file from this repository applied
Commit: Unavailable
Layers: defaults
Review profile: CHILL
Plan: Advanced
Run ID: a81610b4-d37e-4109-b890-6243f81fe5df
Commits
Reviewing files that changed from the base of the PR and between 477d87b and 6ec489e.
Files ignored due to path filters: 1
pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
Files selected for processing: 2
package.json
vite.config.ts
Files with no reviewable changes: 1
vite.config.ts
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Missing route chunks could trap the renderer in repeated error-boundary handling before the site could recover. Upgrade Redact from 0.1.4 to 0.2.5, which includes the upstream nested error-boundary fix in TanStack/redact#109.
Align Cloudflare Vite with Redact's supported 1.65.0 peer and Wrangler with that plugin's required version. Use Vite 8's native default minifier, because the Cloudflare plugin's ES2026 worker target is rejected by esbuild, including 0.28.2.
Related to #734. Keep the issue open for deployed verification.
Summary by CodeRabbit