Skip to content
Snippets Groups Projects

Add cloneable error object for error handlers insights providers

Merged Warren Gifford requested to merge code-insights/fix-error-cloneable-problem into main

Created by: vovakulikov

Closes https://github.com/sourcegraph/sourcegraph/issues/20336

This PR adds a cloneable error object in order to support error handling for browsers that don't support native Error objects as a cloneable object from the web worker thread.

As a result of this problem we had failed loading for insight which returned an error from insight extension and also that ruined all loading insight pipeline which means if you have one error-producing insight, it blocks the others.

You could see this problem only in Safari and Firefox but not in Chrome. Now all browsers should work as expected (don't block the loading of other insights in case if they got an error from the web worker thread)

Merge request reports

Merged by avatar (Jun 23, 2025 10:29am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading