Skip to content

feat(mobile): optional crash reporting#2142

Open
brow wants to merge 6 commits into
mainfrom
tomb/mobile-sentry-crash-reporting
Open

feat(mobile): optional crash reporting#2142
brow wants to merge 6 commits into
mainfrom
tomb/mobile-sentry-crash-reporting

Conversation

@brow

@brow brow commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

What changed?

Adds default-on Sentry crash reporting to Buzz mobile with an explicit opt-out.

Reporting initializes only in builds that inject a Sentry key at build time (e.g. our app store builds) and stops immediately when the preference is revoked.

The SDK is configured to omit default PII, screenshots, view hierarchy, replay, tracing, logs, breadcrumbs, request bodies, and user text.

Native Android auto-init providers are removed so no SDK code runs before Dart applies consent.

The change will be inactive until our release build pipeline is updated to inject the Sentry key.

How is it tested?

Added tests:

Flutter analysis and the full mobile test suite pass.

Before / After

Before After
Settings before crash-report controls Settings with consent-gated crash-report controls

Co-authored-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
@brow

brow commented Jul 19, 2026

Copy link
Copy Markdown
Contributor Author

@codex review

@brow

brow commented Jul 19, 2026

Copy link
Copy Markdown
Contributor Author

@builderbot review

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

Comment thread mobile/lib/main.dart Outdated
Comment thread mobile/pubspec.lock
@brow

brow commented Jul 19, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Review summary: I found one blocking startup issue and one dependency compatibility concern on fb375ddf. Sentry initialization must not prevent runApp() when persisted consent is present, and the resolver changes should be acknowledged because Sentry 9.24.0 explicitly pins jni 0.14.2. Details are in the inline review comments. I did not approve this head.

Co-authored-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
Comment thread mobile/lib/shared/diagnostics/diagnostics_controller.dart
Comment thread mobile/lib/features/settings/settings_page.dart Outdated
Co-authored-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
@brow
brow marked this pull request as ready for review July 19, 2026 22:43
@brow
brow requested a review from a team as a code owner July 19, 2026 22:43
Co-authored-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
Comment thread mobile/lib/shared/diagnostics/diagnostics_controller.dart
Co-authored-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
@brow brow changed the title Add consent-gated mobile crash reporting feat(mobile): optional crash reporting Jul 20, 2026
Co-authored-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1ft62tztwwm2x9xamk25smmuaj4sfckdkldksruf2x2jwqalffkrq0g7arr <4af4a5896e76d4629bbbb2a90def9d95609c59b6fb6d01f12a32a4e077e94d86@sprout-oss.stage.blox.sqprod.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant