Conversation
There was a problem hiding this comment.
Pull Request Overview
This pull request adds support for Sentry error monitoring and reporting to the application. The change integrates the Sentry SDK to enable error tracking and performance monitoring.
- Adds Sentry DSN configuration parameter to enable Sentry monitoring
- Creates reusable initialization functions for both Pyroscope and Sentry
- Integrates Sentry initialization into the application startup flow
Reviewed Changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| internal/pkg/config/config.go | Adds SentryDSN configuration field |
| go.mod | Adds sentry-go dependency |
| cmd/utils.go | Creates shared initialization functions for Pyroscope and Sentry |
| cmd/get_url.go | Integrates Sentry initialization into get_url command |
| cmd/get_hq.go | Refactors to use shared Pyroscope function and adds Sentry initialization |
| cmd/get.go | Adds sentry-dsn command line flag |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #456 +/- ##
==========================================
- Coverage 55.86% 55.66% -0.21%
==========================================
Files 128 129 +1
Lines 8053 8084 +31
==========================================
+ Hits 4499 4500 +1
- Misses 3182 3210 +28
- Partials 372 374 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Also fixes pyroscope for URL-scoped crawl. Closes #453