admin/metatron-open-seo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Ignore TanStack Query CancelledError in client error tracking (#524)

Browse Source
This commit is contained in:
Ben Senescu 2026-08-26 09:33:43 -04:00 committed by GitHub
parent fcb35a8145
commit 044d712580
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/client/lib/posthog.ts
View File

@ -20,12 +20,16 @@ type ExceptionEntry = {
stacktrace?: { frames?: unknown[] }; stacktrace?: { frames?: unknown[] };
}; };
// Unactionable exceptions we don't want polluting error tracking. They share // Unactionable exceptions we don't want polluting error tracking. Most share
// the trait of not being our code: browser extensions inject promise rejections // the trait of not being our code: browser extensions inject promise rejections
// and cross-origin scripts surface as a detail-less "Script error.", while the // and cross-origin scripts surface as a detail-less "Script error.", while the
// global onerror handler synthesizes a stackless "undefined" when it fires // global onerror handler synthesizes a stackless "undefined" when it fires
// without a real Error object. Real app errors always carry a stack, so the // without a real Error object. Real app errors always carry a stack, so the
// "undefined" rule is gated on synthetic + no frames to avoid false drops. // "undefined" rule is gated on synthetic + no frames to avoid false drops.
// The rest are expected cancellations: TanStack Query throws "CancelledError"
// when a route load is abandoned mid-flight (navigating away), which is normal
// behavior, not a failure. Matched exactly so a real error that merely mentions
// cancellation still gets captured.
function isIgnorableException( function isIgnorableException(
properties: Record<string, unknown> | undefined, properties: Record<string, unknown> | undefined,
): boolean { ): boolean {
@ -36,6 +40,7 @@ function isIgnorableException(
if (value.includes("Object Not Found Matching Id")) return true; if (value.includes("Object Not Found Matching Id")) return true;
if (value === "Script error.") return true; if (value === "Script error.") return true;
if (value.includes("signal is aborted without reason")) return true; if (value.includes("signal is aborted without reason")) return true;
if (value === "CancelledError") return true;
const frames = entry?.stacktrace?.frames; const frames = entry?.stacktrace?.frames;
return ( return (
value === "undefined" && value === "undefined" &&