Skip to content

feat/sentry-enhancebreadcrumbs #856

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.

Sign up for GitHub

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

Jump to bottom
Open
bagusnl wants to merge 7 commits into
base: main
Choose a base branch
from
Open

feat/sentry-enhancebreadcrumbs #856

bagusnl wants to merge 7 commits into from

Conversation

@bagusnl
Copy link
Member

@bagusnl bagusnl commented Jan 6, 2026 •
edited
Loading

Main Goal

Enhance Sentry breadcrumbs data in aid for troubleshooting

New fields:

1. Plugin Info

List all imported plugins with its version and the standard version (pluginVersion-standardVersion)

2. App Configuration

Displays some configuration data used by Collapse

AppThread: httpDownload, extractionCpu
DownloadPreallocated: bool
ExternalDNS: bool
HttpCache: bool
HttpOpts: Redirect//Cookie//Insecure
Proxy: cl//+sys
SophonThread: httpDownload,extractionCpu
Waifu2x: bool
image

PR Status :

  • Overall Status : Done
  • Commits : Done
  • Synced to base (Collapse:main) : Yes
  • Build status : OK
  • Crashing : No
  • Bug found caused by PR : 0

Templates

Changelog Prefixes 
  **[New]**
  **[Imp]**
  **[Fix]**
  **[Loc]**
  **[Doc]**

bagusnl added 5 commits January 6, 2026 22:14
@bagusnl
[Sentry] Add loaded plugins information
51d7c85 
Signed-off-by: Bagus Nur Listiyono <dzakibagus@gmail.com>
@bagusnl
[Sentry] Move breadcrumb definitions off main SentryHelper.cs
43d419a 
Signed-off-by: Bagus Nur Listiyono <dzakibagus@gmail.com>
@bagusnl
[Sentry] Add app config breadcrumbs
3019bb9 
Signed-off-by: Bagus Nur Listiyono <dzakibagus@gmail.com>
@bagusnl
[Sentry] Fix hard crash on proxy breadcrumb getter
3064c62 
Signed-off-by: Bagus Nur Listiyono <dzakibagus@gmail.com>
@bagusnl
[Sentry] Simplify PluginList breadcrumb builder
2c08dba 
Signed-off-by: Bagus Nur Listiyono <dzakibagus@gmail.com>
@bagusnl bagusnl requested a review from a team January 6, 2026 17:16
@bagusnl bagusnl self-assigned this Jan 6, 2026
@bagusnl bagusnl added Enhancement New feature or request Area: Runtime Issue labeled for runtime, Hi3Helper.Core and other libraries used by Collapse Priority: Low Size: Small labels Jan 6, 2026
bagusnl added 2 commits January 7, 2026 00:25
@bagusnl
[Sentry] Wrap GetSystemProxy in trycatch
f9b0176 
Signed-off-by: Bagus Nur Listiyono <dzakibagus@gmail.com>
@bagusnl
[Sentry] Add proxy type to breadcrumb
3c32eb3 
Signed-off-by: Bagus Nur Listiyono <dzakibagus@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@sentry sentry[bot] sentry[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@bagusnl bagusnl

Labels

Area: Runtime Issue labeled for runtime, Hi3Helper.Core and other libraries used by Collapse Enhancement New feature or request Priority: Low Size: Small

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bagusnl