feat: add Bugsnag error reporting service #1695
Closed
Conversation
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
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Jan 26, 2026
morgan-wowk
force-pushed
the
01-26-feat_add_bugsnag_error_reporting_service_initialize_bugsnag_sdk_with_custom_error_normalization_and_grouping._reads_configuration_from_environment_variables_and_applies_normalization_to_all_reported_errors
branch
3 times, most recently
from
9688389 to
c3cfc88
Compare
Mbeaulne
reviewed
Jan 26, 2026
Collaborator
There was a problem hiding this comment.
we typically don't do barrel files and we just directly import from the files where we need. I'd recommend using @.cursorrules to catch these sort of "style" errors.
| }; | ||
|
|
||
| // Initialize Bugsnag on module load | ||
| initializeBugsnag(); |
Collaborator
There was a problem hiding this comment.
Lets export this function and Initialize where we are using it (maybe src/index.tsx)
| return; | ||
| } | ||
|
|
||
| Bugsnag.start({ |
Collaborator
There was a problem hiding this comment.
lets wrap this in a try catch:
try {
Bugsnag.start({ ... });
} catch (error) {
console.error("Failed to initialize Bugsnag:", error);
}
| onError: (event) => { | ||
| // Only normalize and enhance generic "Error" instances | ||
| // Custom error classes (NetworkError, ValidationError, etc.) are preserved as-is | ||
| if (event.errors[0].errorClass === "Error") { |
Collaborator
There was a problem hiding this comment.
lets abstract this into constants:
const GENERIC_ERROR_CLASS = "Error";
if (event.errors[0].errorClass === GENERIC_ERROR_CLASS) {
Comment on lines
52
to
83
| onError: (event) => { | ||
| // Only normalize and enhance generic "Error" instances | ||
| // Custom error classes (NetworkError, ValidationError, etc.) are preserved as-is | ||
| if (event.errors[0].errorClass === "Error") { | ||
| const errorMessage = event.errors[0].errorMessage; | ||
| const { normalizedMessage, extractedValues } = | ||
| normalizeErrorMessage(errorMessage); | ||
|
|
||
| // Set custom grouping hash for better error grouping | ||
| event.groupingHash = normalizedMessage; | ||
|
|
||
| // Update errorClass to show the normalized message in Observe UI | ||
| event.errors[0].errorClass = normalizedMessage; | ||
|
|
||
| // Add custom grouping key to metadata if configured | ||
| if (config.customGroupingKey) { | ||
| event.addMetadata("custom", { | ||
| [config.customGroupingKey]: normalizedMessage, | ||
| }); | ||
| } | ||
|
|
||
| // Add extracted values as metadata for debugging | ||
| if (Object.keys(extractedValues).length > 0) { | ||
| event.addMetadata("extracted_values", extractedValues); | ||
| } | ||
| } | ||
|
|
||
| // Add pathname for context (for all errors) | ||
| event.addMetadata("context", { pathname: window.location.pathname }); | ||
| }, | ||
| }); | ||
| }; |
Collaborator
There was a problem hiding this comment.
This is fairly complex. Can we abstract it into a function?
const handleGenericError = (event: Bugsnag.Event, config: BugsnagConfig) => { ... };
Author
There was a problem hiding this comment.
For sure. Added some unit tests too.
morgan-wowk
changed the base branch from
01-26-feat_add_error_message_normalization_utility_add_utility_to_normalize_error_messages_by_extracting_dynamic_values_uuids_ids_hashes_etc._and_replacing_them_with_placeholders._this_enables_better_error_grouping_in_monitoring_tools
to
graphite-base/1695
morgan-wowk
force-pushed
the
01-26-feat_add_bugsnag_error_reporting_service_initialize_bugsnag_sdk_with_custom_error_normalization_and_grouping._reads_configuration_from_environment_variables_and_applies_normalization_to_all_reported_errors
branch
from
c3cfc88 to
6cf38f9
Compare
morgan-wowk
force-pushed
the
graphite-base/1695
branch
from
956fbba to
3e8c8af
Compare
morgan-wowk
changed the base branch from
graphite-base/1695
to
01-26-feat_add_error_message_normalization_utility_add_utility_to_normalize_error_messages_by_extracting_dynamic_values_uuids_ids_hashes_etc._and_replacing_them_with_placeholders._this_enables_better_error_grouping_in_monitoring_tools
morgan-wowk
changed the base branch from
01-26-feat_add_error_message_normalization_utility_add_utility_to_normalize_error_messages_by_extracting_dynamic_values_uuids_ids_hashes_etc._and_replacing_them_with_placeholders._this_enables_better_error_grouping_in_monitoring_tools
to
graphite-base/1695
morgan-wowk
force-pushed
the
01-26-feat_add_bugsnag_error_reporting_service_initialize_bugsnag_sdk_with_custom_error_normalization_and_grouping._reads_configuration_from_environment_variables_and_applies_normalization_to_all_reported_errors
branch
from
6cf38f9 to
ad2ef91
Compare
morgan-wowk
force-pushed
the
graphite-base/1695
branch
from
3e8c8af to
1119b6f
Compare
morgan-wowk
changed the base branch from
graphite-base/1695
to
01-26-feat_add_error_message_normalization_utility_add_utility_to_normalize_error_messages_by_extracting_dynamic_values_uuids_ids_hashes_etc._and_replacing_them_with_placeholders._this_enables_better_error_grouping_in_monitoring_tools
morgan-wowk
force-pushed
the
01-26-feat_add_bugsnag_error_reporting_service_initialize_bugsnag_sdk_with_custom_error_normalization_and_grouping._reads_configuration_from_environment_variables_and_applies_normalization_to_all_reported_errors
branch
from
ad2ef91 to
62bc345
Compare
morgan-wowk
force-pushed
the
01-26-feat_add_error_message_normalization_utility_add_utility_to_normalize_error_messages_by_extracting_dynamic_values_uuids_ids_hashes_etc._and_replacing_them_with_placeholders._this_enables_better_error_grouping_in_monitoring_tools
branch
from
1119b6f to
7ab4019
Compare
morgan-wowk
force-pushed
the
01-26-feat_add_bugsnag_error_reporting_service_initialize_bugsnag_sdk_with_custom_error_normalization_and_grouping._reads_configuration_from_environment_variables_and_applies_normalization_to_all_reported_errors
branch
2 times, most recently
from
58af5dc to
0217c23
Compare
Initialize Bugsnag SDK with custom error normalization and grouping. Reads configuration from environment variables and applies normalization to all reported errors.
morgan-wowk
force-pushed
the
01-26-feat_add_bugsnag_error_reporting_service_initialize_bugsnag_sdk_with_custom_error_normalization_and_grouping._reads_configuration_from_environment_variables_and_applies_normalization_to_all_reported_errors
branch
from
0217c23 to
e60eb29
Compare
morgan-wowk
force-pushed
the
01-26-feat_add_error_message_normalization_utility_add_utility_to_normalize_error_messages_by_extracting_dynamic_values_uuids_ids_hashes_etc._and_replacing_them_with_placeholders._this_enables_better_error_grouping_in_monitoring_tools
branch
from
7ab4019 to
d7fc62b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Added Bugsnag error management service with custom error normalization and grouping capabilities. The implementation includes:
Type of Change
Checklist
Test Instructions
.envfile:Additional Comments
The error normalization helps group similar errors together even when they contain different dynamic values, improving our error tracking and debugging capabilities.