feat: add fingerprinted errors, source tagging, and error capture#2000
Merged
feat: add fingerprinted errors, source tagging, and error capture#2000
Conversation
Wrap non-Error arguments in a new Error() so Sentry receives a proper stack trace instead of a flat message string. This ensures all log.error() calls produce stack frames regardless of argument type. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add dual-mode log.error that supports tagged templates for error
fingerprinting. When called as log.error`message ${value}`, the static
template becomes the Sentry grouping key while dynamic values are
preserved as context metadata. This prevents errors with different
dynamic values from creating separate Sentry issues.
Also uses package.json version for the Sentry release tag instead of
the backend-provided version.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add log.error to critical paths that were silently swallowing errors: - WebSocket onerror handlers for editor, code-editor, and launch realtime connections - .catch() for engine type definitions fetch in code-editor - Replace console.error with log.error in intellisense script fetch and launch sourcefiles loading Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add response.ok validation to upload REST API, store fetches, and texture download - Replace console.error with log.error in realtime error handler - Add .catch() to autofill script fetch promise chain - Auto-categorize errors with a source tag derived from the top stack frame's file path (e.g. source: realtime, source: pickers/version-control) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
What's Changed
log.errorthat supports tagged templates for Sentry fingerprinting —log.error`missing asset ${id}`` groups all errors with the same template as one issue regardless of interpolated valuessourcetag derived from the top stack frame's file path (e.g.realtime,assets,pickers/version-control)user_idtag to all Sentry eventspackage.jsonversion for the Sentry release taglog.errorcalls by wrapping innew Error()onerrorhandlers for editor, code-editor, and launch realtime connectionsresponse.okvalidation to upload REST API, store fetches, and texture download.catch()to engine type definitions and autofill script fetch chainsconsole.errorwithlog.errorin realtime error handler, intellisense, and sourcefiles loadingallowTaggedTemplates)Checks