Skip to content

refactor: report errors to Bugsnag when using error boundary (ErrorPage fallback) #1696

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
morgan-wowk wants to merge 2 commits into
base: 01-26-feat_add_error_normalization_and_integrate
Choose a base branch
from
Open

refactor: report errors to Bugsnag when using error boundary (ErrorPage fallback) #1696

morgan-wowk wants to merge 2 commits into from

Conversation

@morgan-wowk
Copy link

@morgan-wowk morgan-wowk commented Jan 26, 2026 •
edited
Loading

Description

Note: Some of these changes are in preparation for work that is upstack. We wouldn't normally do this but don't wish to change it at this point.

Enhanced error handling by moving ErrorPage from routes to shared components and adding Bugsnag integration. The component now automatically reports errors to Bugsnag when configured and includes the current pathname as metadata. Added support for error boundary reset functionality, allowing the component to properly reset when used with React error boundaries. Improved error message handling to better display different error types.

Type of Change

  • Bug fix
  • New feature
  • Improvement
  • Cleanup/Refactor
  • Breaking change
  • Documentation update

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Test Instructions

  1. Test the error display by navigating to a non-existent route
  2. Verify that the error message is displayed correctly
  3. Test the "Try Again" and "Go Home" buttons to ensure they work as expected
  4. Confirm that errors are properly reported to Bugsnag when enabled

@morgan-wowk morgan-wowk mentioned this pull request Jan 26, 2026
Closed
4 tasks
@morgan-wowk Graphite App
Copy link
Author

morgan-wowk commented Jan 26, 2026 •
edited
Loading

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.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Jan 26, 2026
Closed
Closed
Closed
Open
Closed
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from 5002cbc to dc253c8 Compare January 26, 2026 21:02
@morgan-wowk 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 319066c to 80defd3 Compare January 26, 2026 21:02
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from dc253c8 to 9be3ab2 Compare January 26, 2026 21:07
@morgan-wowk 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 9688389 to c3cfc88 Compare January 26, 2026 21:10
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from 9be3ab2 to d2ad4cd Compare January 26, 2026 21:10
@morgan-wowk morgan-wowk marked this pull request as ready for review January 26, 2026 21:13
@morgan-wowk morgan-wowk requested a review from a team as a code owner January 26, 2026 21:13
@morgan-wowk 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 January 28, 2026 18:33
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from d2ad4cd to bedad4c Compare January 28, 2026 18:33
@morgan-wowk 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 January 28, 2026 18:48
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from bedad4c to 43e186c Compare January 28, 2026 18:48
@morgan-wowk morgan-wowk changed the base branch from 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 to graphite-base/1696 January 28, 2026 19:20
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from 43e186c to a8636d6 Compare January 28, 2026 19:21
@morgan-wowk morgan-wowk changed the base branch from graphite-base/1696 to 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 January 28, 2026 19:21
@morgan-wowk Graphite App
Copy link
Author

morgan-wowk commented Jan 28, 2026

Note: I am actively working on implementing a refactor to simplify the contents of this PR

@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from 9322f42 to 43d5e44 Compare January 28, 2026 19:57
@morgan-wowk 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 62bc345 to 58af5dc Compare January 28, 2026 21:27
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from 43d5e44 to dcab8f2 Compare January 28, 2026 21:27
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from dcab8f2 to d58f11b Compare January 28, 2026 21:30
@morgan-wowk morgan-wowk changed the title refactor: rename ErrorPage to FullPageError and extract ErrorDisplay refactor: report errors to Bugsnag when using error boundary (ErrorPage fallback) Jan 28, 2026
@morgan-wowk morgan-wowk changed the base branch from 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 to graphite-base/1696 January 28, 2026 21:48
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from d58f11b to 82614d3 Compare January 28, 2026 21:49
@morgan-wowk morgan-wowk changed the base branch from graphite-base/1696 to 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 January 28, 2026 21:49
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from 82614d3 to fccc289 Compare January 28, 2026 21:58
@morgan-wowk 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 January 28, 2026 21:58
@morgan-wowk morgan-wowk changed the base branch from 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 to graphite-base/1696 January 28, 2026 22:06
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from fccc289 to 1d5e2eb Compare January 28, 2026 22:06
@morgan-wowk morgan-wowk changed the base branch from graphite-base/1696 to 01-26-feat_add_error_normalization_and_integrate January 28, 2026 22:06
This was referenced Jan 28, 2026
Open
Open
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from 1d5e2eb to e18dc07 Compare January 28, 2026 22:12
@morgan-wowk morgan-wowk force-pushed the 01-26-feat_add_error_normalization_and_integrate branch 2 times, most recently from b29beaf to 7dd0dc9 Compare January 28, 2026 22:16
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch 2 times, most recently from 844c73a to 5bcb71f Compare January 28, 2026 22:28
@morgan-wowk morgan-wowk force-pushed the 01-26-feat_add_error_normalization_and_integrate branch from 7dd0dc9 to ca3c44a Compare January 28, 2026 22:28
@morgan-wowk
feat: add error message normalization and integration
7406e59 
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 like Bugsnag.

Integrate normalization into Bugsnag error handler with custom grouping hash.
@morgan-wowk
refactor: improve error handling components
15c6748 
- Rename ErrorPage to FullPageError for clarity
- Extract ErrorDisplay component for reusable error UI
- Add unified ErrorHandler component for consistent error handling
- Integrate ErrorHandler with FullPageError
@morgan-wowk morgan-wowk changed the base branch from 01-26-feat_add_error_normalization_and_integrate to graphite-base/1696 January 28, 2026 22:33
@morgan-wowk morgan-wowk force-pushed the 01-26-refactor-error-component-names branch from 5bcb71f to 15c6748 Compare January 28, 2026 22:34
@morgan-wowk morgan-wowk changed the base branch from graphite-base/1696 to 01-26-feat_add_error_normalization_and_integrate January 28, 2026 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@camielvs camielvs Awaiting requested review from camielvs

@maxy-shpfy maxy-shpfy Awaiting requested review from maxy-shpfy

@Mbeaulne Mbeaulne Awaiting requested review from Mbeaulne

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@morgan-wowk