-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Improve GitHub issue reporting with pre-filled error details in StreamError dialog #6882
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.
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cubic analysis
No issues found across 1 file. Review in cubic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one nitpick
Co-authored-by: Patrick Erichsen <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's also add an auto-release workflow for this 👍 the new errors
package has a build
and test
command so should integrate fine with the new CI logic for building/testing all packages
automatically.
* Standard error class for Continue application with enhanced metadata support | ||
*/ | ||
export class ContinueError extends Error { | ||
public readonly code?: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add additional data from ideInfo
here? I think with some of the centralized error logging we are doing with Sentry we could do this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After discussing, yes, wouldn't hurt to add a few more optional properties from ideInfo
Summary by cubic
Improved the GitHub issue reporting flow in the StreamError dialog by pre-filling error details and environment info when users report a bug.