-
Notifications
You must be signed in to change notification settings - Fork 6
test: add tracing for auto, manual update #1003
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.
Pull Request Overview
This PR adds comprehensive Sentry tracing infrastructure to the auto-update system to improve observability and error tracking throughout the entire update lifecycle. The changes enable detailed monitoring of both automatic and manual update flows.
- Added Sentry span tracing for all auto-update phases (initialization, checking, downloading, installation)
- Restructured update handlers into separate functions with proper error handling and tracing
- Added IPC handler for manual update installation with error recovery
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
File | Description |
---|---|
main/src/auto-update.ts | Major refactoring to add comprehensive Sentry tracing throughout the auto-update lifecycle with proper span hierarchy |
main/src/tests/auto-update.test.ts | Added Sentry mocks for testing and updated error message assertions |
main/src/main-window.ts | Removed verbose logging statements |
renderer/src/common/hooks/use-mcp-secrets.ts | Removed test environment check for timeout delay |
Enhanced Auto-Update Tracing with Sentry
Summary
Added comprehensive Sentry tracing for the entire auto-update lifecycle to improve observability, error tracking, and analytics.
Complete traced flow:
Manual update span.op
Auto update span.op
All phases are traced in Sentry with proper span hierarchy