fix: gate posthog analytics events

[Mozart299]

Summary by CodeRabbit

• Bug Fixes

  • Added ability to disable analytics tracking via research mode feature flag.
  • Fixed user identity initialization order in authentication flow.

• Refactor

  • Disabled automatic application lifecycle event capturing in analytics.
  • Reordered analytics initialization sequence during user authentication.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2e257ea7-249e-4f52-9b8b-7b7e2e63a2ff

📥 Commits

Reviewing files that changed from the base of the PR and between 750e371 and eec75d9.

📒 Files selected for processing (4)

• src/mobile/lib/main.dart
• src/mobile/lib/src/app/auth/bloc/auth_bloc.dart
• src/mobile/lib/src/app/shared/services/analytics_service.dart
• src/mobile/lib/src/app/shared/services/feature_flag_service.dart

---

📝 Walkthrough

Walkthrough

This PR implements a feature-flag-driven analytics control system for the mobile app. A new isAnalyticsEnabled getter gates event capture in AnalyticsService, auth flows reorder user identity setup before tracking events, and PostHog stops capturing app lifecycle events automatically.

Changes

Analytics Event Control and User Identity Tracking

Layer / File(s)	Summary
Feature flag analytics control src/mobile/lib/src/app/shared/services/feature_flag_service.dart	Added isAnalyticsEnabled getter that returns the state of AppFeatureFlag.researchMode via the existing isEnabled(...) method.
Analytics event capture guard src/mobile/lib/src/app/shared/services/analytics_service.dart	trackEvent now checks FeatureFlagService.instance.isAnalyticsEnabled and returns early without calling Posthog().capture(...) when analytics are disabled.
Auth user identity before tracking src/mobile/lib/src/app/auth/bloc/auth_bloc.dart	In _onLoginUser, _onRegisterUser, and _onLoginWithProvider, reordered calls so AuthHelper.getCurrentUserId(...) and AnalyticsService().setUserIdentity(...) execute before trackUserLoggedIn()/trackUserRegistered().
PostHog lifecycle event configuration src/mobile/lib/main.dart	Changed PostHog setup to set postHogConfig.captureApplicationLifecycleEvents from true to false.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

📊 Events now check before they fly,
User known when tracked on high,
Feature flags become the gate,
App lifecycle meets its fate. 🚀

🚥 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 title accurately summarizes the main objective of the PR: gating PostHog analytics events based on feature flags, which is reflected across all four modified files.
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 fix/posthog-analytics-gating

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)

• JIRA integration encountered authorization issues. Please disconnect and reconnect the integration in the CodeRabbit UI.

---

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.}