chore(samples): replace Sentry with Firebase Crashlytics

[xsahil03x]

Summary

• Drop sentry_flutter from the sample app: we no longer have access to the Sentry account, so the DSN, init call, log-handler capture, and runZonedGuarded wrapper around main() are removed.
• Wire Firebase Crashlytics in its place. Firebase Core is already initialized for messaging, so the only pubspec/melos additions are:
  • firebase_crashlytics: ^4.0.0 in sample_app/pubspec.yaml and melos.yaml
• Dart wiring in main.dart (modern post-Flutter-3.3 pattern, no runZonedGuarded):
  • FlutterError.onError → FirebaseCrashlytics.instance.recordFlutterFatalError (release) / dumpErrorToConsole (debug)
  • PlatformDispatcher.instance.onError → recordError(..., fatal: true) (release) / debugPrint (debug) — catches every uncaught async error, replacing the old zone-based capture
  • Crashlytics collection disabled in debug builds via setCrashlyticsCollectionEnabled(!kDebugMode)
• _sampleAppLogHandler in app.dart now forwards Stream chat client errors to Crashlytics, attaching the log message as reason.

Platform setup

flutterfire configure --project=stream-chat-internal --platforms=android,ios,macos,web was run as part of this PR. The CLI:

• added the com.google.firebase.crashlytics Gradle plugin to android/settings.gradle and android/app/build.gradle (inside the auto-managed FlutterFire Configuration block);
• added the flutterfire upload-crashlytics-symbols run-script phase to the iOS and macOS Runner Xcode targets so dSYMs upload automatically on release builds;
• regenerated google-services.json, GoogleService-Info.plist, firebase_options.dart, and firebase.json against the current stream-chat-internal Firebase project state.

The remaining Pods/Sentry/* directory and Sentry framework references in ios/Runner.xcodeproj/project.pbxproj are auto-generated by CocoaPods and will disappear on the next pod install.

Test plan

• flutter pub get resolves cleanly in sample_app/ (verified locally)
• dart analyze passes (verified locally — "No issues found!")
• flutter run (debug) on Android + iOS — app boots, no Crashlytics reports expected with collection disabled
• flutter run --release on Android — trigger a test crash via FirebaseCrashlytics.instance.crash() and confirm it lands in the Firebase console
• Same release smoke test on iOS, with dSYM upload confirmed in the build log

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features

  • Automatic upload of debug symbols for iOS and macOS to improve crash report accuracy.

• Chores

  • Switched runtime crash reporting from Sentry to Firebase Crashlytics across the sample app.
  • Added Crashlytics build/plugin integration and symbol-upload build steps for Android, iOS, and macOS.
  • Updated Firebase app/configuration entries to align with Crashlytics.
  • Enforced a minimum Android compile SDK level (36) for sample app builds.

[coderabbitai]

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

• @coderabbitai resume to resume automatic reviews.
• @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

• ▶️ Resume reviews
• 🔍 Trigger review

📝 Walkthrough

Walkthrough

Replaces Sentry with Firebase Crashlytics across dependencies, Android Gradle plugin and compileSdk floor, app initialization and error handlers, log reporting, native symbol-upload build phases, and Firebase configuration files.

Changes

Sentry to Crashlytics Migration

Layer / File(s)	Summary
Dependency updates melos.yaml, sample_app/pubspec.yaml	firebase_crashlytics (^4.0.0) is added and sentry_flutter is removed from monorepo bootstrap and sample app dependencies.
Android plugin & compileSdk floor sample_app/android/app/build.gradle, sample_app/android/build.gradle, sample_app/android/settings.gradle	Adds com.google.firebase.crashlytics plugin declarations and sets android.compileSdkVersion = Math.max(flutter.compileSdkVersion, 36) to enforce a minimum compile SDK.
Android google-services config sample_app/android/app/google-services.json	Removed an existing Android client entry and added a new client entry for another Android package.
App initialization and error handler wiring sample_app/lib/main.dart	Initializes Firebase directly, toggles Crashlytics collection by kDebugMode, wires FlutterError.onError and PlatformDispatcher.instance.onError to record fatal errors to Crashlytics, and launches the app without the prior Sentry zone wrapper.
Log handler & config cleanup sample_app/lib/app.dart, sample_app/lib/utils/app_config.dart	_sampleAppLogHandler now calls FirebaseCrashlytics.instance.recordError(...) (with reason: record.message) and the unused sentryDsn constant is removed.
iOS/macOS Crashlytics upload build phases sample_app/ios/Runner.xcodeproj/project.pbxproj, sample_app/macos/Runner.xcodeproj/project.pbxproj, sample_app/firebase.json	Added flutterfire upload-crashlytics-symbols shell script build phases to iOS and macOS projects, updated iOS embed frameworks to include Firebase frameworks and remove Sentry frameworks, and enabled uploadDebugSymbols for iOS/macos in firebase.json.
Firebase options and macOS plist sample_app/lib/firebase_options.dart, sample_app/macos/Runner/GoogleService-Info.plist	Reformatted DefaultFirebaseOptions.ios args, removed iosClientId from macos options, and removed CLIENT_ID/REVERSED_CLIENT_ID from the macOS GoogleService-Info.plist.

Sequence Diagram

sequenceDiagram
  participant App as StreamChatSampleApp
  participant Firebase as Firebase.initializeApp
  participant Crashlytics as FirebaseCrashlytics.instance
  participant Platform as PlatformDispatcher.instance

  App->>Firebase: initializeApp()
  Firebase->>Crashlytics: configure collection (enable/disable by kDebugMode)
  App->>Crashlytics: FlutterError.onError -> recordError(fatal)
  Platform->>Crashlytics: onError -> recordError(fatal) and return true

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I hopped from Sentry's hill to Crashlytics' den,
Plugins and scripts now hum with a pen,
Firebase wakes and handles each fall,
Errors recorded — big and small,
A rabbit smiles: "Crash logs safe again."

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title accurately summarizes the primary change: migrating error reporting from Sentry to Firebase Crashlytics across the sample app.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/replace-sentry-with-firebase

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@sample_app/lib/main.dart`:
- Around line 17-18: Add native Crashlytics defaults to prevent debug uploads on
first launch and only enable collection at runtime for non-debug builds: add the
Android meta-data key firebase_crashlytics_collection_enabled with value false
to the Android <application> manifest entry, add the Info.plist key
FirebaseCrashlyticsCollectionEnabled with value false to iOS app metadata, and
in main.dart only call
FirebaseCrashlytics.instance.setCrashlyticsCollectionEnabled(true) when not in
debug (use kDebugMode to conditionally enable) instead of unconditionally
calling setCrashlyticsCollectionEnabled(!kDebugMode).
- Around line 35-45: The handler assigned to PlatformDispatcher.instance.onError
currently always returns true, suppressing platform errors in debug; update the
onError closure so that when kDebugMode is true you either call
FlutterError.reportError(FlutterErrorDetails(exception: error, stack: stack,
library: 'platform')) and then return false, or simply return false after
debugPrint so the error is not marked handled by the embedder; keep the existing
FirebaseCrashlytics.instance.recordError(..., fatal: true) for the non-debug
branch and continue returning true there.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: bfc3ef2c-e998-4c4b-9bcf-0473a04fcc01

📥 Commits

Reviewing files that changed from the base of the PR and between d9d4ff4 and 2a16222.

📒 Files selected for processing (7)

• melos.yaml
• sample_app/android/app/build.gradle
• sample_app/android/settings.gradle
• sample_app/lib/app.dart
• sample_app/lib/main.dart
• sample_app/lib/utils/app_config.dart
• sample_app/pubspec.yaml

💤 Files with no reviewable changes (1)

• sample_app/lib/utils/app_config.dart

[coderabbitai]

🧹 Nitpick comments (1)

sample_app/android/app/build.gradle (1)

5-5: 💤 Low value

Minor: Use double quotes for consistency.

All other plugin declarations in this file use double quotes, but this line uses single quotes.

🔧 Proposed fix

-    id 'com.google.firebase.crashlytics'
+    id "com.google.firebase.crashlytics"

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@sample_app/android/app/build.gradle` at line 5, Replace the single-quoted
plugin identifier id 'com.google.firebase.crashlytics' with double quotes to
match the file's quoting style; locate the plugin declaration line containing
com.google.firebase.crashlytics in sample_app/android/app/build.gradle and
update the quotes around the plugin id to use double quotes.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@sample_app/android/app/build.gradle`:
- Line 5: Replace the single-quoted plugin identifier id
'com.google.firebase.crashlytics' with double quotes to match the file's quoting
style; locate the plugin declaration line containing
com.google.firebase.crashlytics in sample_app/android/app/build.gradle and
update the quotes around the plugin id to use double quotes.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 72a5a96c-bf35-4831-b0b7-e18319289f9c

📥 Commits

Reviewing files that changed from the base of the PR and between 9c774a3 and 4dad8f3.

📒 Files selected for processing (2)

• sample_app/android/app/build.gradle
• sample_app/android/build.gradle

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.29%. Comparing base (1f4eae9) to head (07ec98f).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2665   +/-   ##
=======================================
  Coverage   65.29%   65.29%           
=======================================
  Files         423      423           
  Lines       26635    26635           
=======================================
  Hits        17390    17390           
  Misses       9245     9245           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.