add name and message to frontendend exception handler#5585
Conversation
There was a problem hiding this comment.
Greptile Summary
This PR improves frontend error handling by adding more detailed error information throughout the error handling chain. The changes ensure that error name and message are included alongside stack traces, providing better context for debugging. Key changes include:
- Modified
handle_frontend_exceptionin state.py to accept fuller error information - Enhanced JavaScript error handlers in state.js to send error name and message
- Updated error boundary component to display and handle the expanded error format
These changes create a more consistent and informative error reporting system across the frontend-backend boundary, making it easier for developers to diagnose and fix issues.
Confidence score: 5/5
- This PR is very safe to merge as it purely enhances error reporting without modifying core functionality
- The changes are well-coordinated across all relevant files and follow a consistent pattern of improvement
- While all files look good, developers might want to double-check the error formatting in error_boundary.py to ensure the display format meets their expectations
3 files reviewed, no comments
Edit PR Review Bot Settings | Greptile
CodSpeed Performance ReportMerging #5585 will not alter performanceComparing Summary
|
masenf
left a comment
There was a problem hiding this comment.
makes sense, approve when unit tests are fixed up
No description provided.