Skip to content

fix(router): filter out non-actionable errors from third-party scripts#771

Merged
LadyBluenotes merged 2 commits intomainfrom
feature/fix-TANSTACK-COM-J2
Mar 17, 2026
Merged

fix(router): filter out non-actionable errors from third-party scripts#771
LadyBluenotes merged 2 commits intomainfrom
feature/fix-TANSTACK-COM-J2

Conversation

@TkDodo
Copy link
Contributor

@TkDodo TkDodo commented Mar 17, 2026

Summary by CodeRabbit

  • Chores
    • Improved error tracking reliability by filtering out noise from third-party ad-tech sources.

@netlify
Copy link

netlify bot commented Mar 17, 2026

Deploy Preview for tanstack ready!

Name Link
🔨 Latest commit f4eb44d
🔍 Latest deploy log https://app.netlify.com/projects/tanstack/deploys/69b9a6e82e55f900086d4eca
😎 Deploy Preview https://deploy-preview-771--tanstack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 33
Accessibility: 90
Best Practices: 83
SEO: 97
PWA: 70
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d8084a35-aedb-470c-a7ec-ac2d2f76b115

📥 Commits

Reviewing files that changed from the base of the PR and between 145dfab and ddbf402.

📒 Files selected for processing (1)
  • src/router.tsx

📝 Walkthrough

Walkthrough

A beforeSend hook is added to Sentry initialization to filter out noisy third-party ad-tech errors. The hook inspects exception stack traces and discards events where all frames originate from ftUtils.js files or domains under fuseplatform.net, while preserving other events.

Changes

Cohort / File(s) Summary
Sentry Error Filtering
src/router.tsx
Added beforeSend hook to Sentry initialization that drops stack traces containing only ad-tech errors from ftUtils.js and fuseplatform.net domains, filtering out noisy third-party signals.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 With whiskers twitching, we filter the spam,
Dropping those ad-tech errors, thank you ma'am!
ftUtils and fuseplatform fade to the night,
Sentry reports now pristinely bright—
A rabbit's refactor, precise and so keen! 🎯

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: filtering out non-actionable errors from third-party scripts in the Sentry initialization.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/fix-TANSTACK-COM-J2
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

You can get early access to new features in CodeRabbit.

Enable the early_access setting to enable early access features such as new models, tools, and more.

@LadyBluenotes LadyBluenotes merged commit fb4dc25 into main Mar 17, 2026
7 of 8 checks passed
@LadyBluenotes LadyBluenotes deleted the feature/fix-TANSTACK-COM-J2 branch March 17, 2026 19:09
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