Skip to content

fix: Sentry reporting handled chunk load errors - #1193

Closed
sentry[bot] wants to merge 2 commits into
mainfrom
seer/fix/sentry-ignore-chunk-load-errors
Closed

sentry[bot] wants to merge 2 commits into
mainfrom
seer/fix/sentry-ignore-chunk-load-errors

Conversation

@sentry

@sentry sentry Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Dynamic import failures (chunk load errors) due to version skew are correctly handled by stale-app-reload.ts, which forces a page reload. However, Sentry was still capturing these as unhandled exceptions, creating noise in error monitoring.

This PR exports staleAppErrorPatterns from src/utils/stale-app-reload.ts and passes it as the ignoreErrors array to Sentry.init in src/router.tsx. This ensures that Sentry will now silently drop these expected and already-handled errors, preventing them from being reported.

Fixes TANSTACK-COM-4AP

This PR was automatically generated by Sentry. You can adjust this setting at any time.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 29, 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 Preview URL Updated (UTC)
✅ Deployment successful!
View logs
tanstack-com 8cd3cda Commit Preview URL

Branch Preview URL
Aug 29 2026, 11:04 PM

@tannerlinsley tannerlinsley added the source-audit Tracked by the automated source audit label Aug 30, 2026
@tannerlinsley

Copy link
Copy Markdown
Member

This catches the deployment noise, but it also hides failures when the reload cooldown prevents recovery. I added a narrower fix and regression coverage in #1327. Keeping this open until that replacement passes its checks and lands.

AI review notes: ignoreErrors matches the message regardless of whether reloadOnStaleAppError returned true. The SDK regression in #1327 verifies that the first handled error is suppressed and a new error during the 10-second cooldown is still reported, using an in-memory Sentry transport. No deployed-browser recovery is claimed.

@tannerlinsley

Copy link
Copy Markdown
Member

Thanks for flagging the stale-chunk noise. #1327 has now landed with the narrower recovery check, so this implementation is superseded.

AI review notes: The merged change suppresses the exact error that triggered a reload and retains reporting for new errors during the cooldown. This replaces the blanket ignoreErrors proposal here. The original Sentry incident link and proposal remain in this PR for reference. This is a verified merge, not a claim that production deployment has finished.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

source-audit Tracked by the automated source audit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant