Wire RemoteConfigManager refresh on first foreground and identity changes (off by default)

[tonidero]

Motivation

Final piece of the initial remote-config wiring: hook RemoteConfigManager into the SDK's actual lifecycle so the topic-fetch pipeline runs on real devices. The earlier PRs in the stack added the network scaffolding (#3435), the manager + topic fetcher (#3437), and the unreferenced-topic cleanup (#3439). None of them invoke the manager — this PR does, behind a developer-only build flag so production traffic is unaffected.

Description

• Constructs RemoteConfigManager (with a TopicFetcher) inside PurchasesFactory and passes it into PurchasesOrchestrator.
• Adds a private refreshRemoteConfigIfEnabled(appUserID) helper on PurchasesOrchestrator that calls remoteConfigManager.updateRemoteConfigIfNeeded(...). The helper is invoked from four places, each of which already runs only when something meaningful changed:
  • onAppForegrounded() — only when state.firstTimeInForeground is true (i.e. first foreground after Purchases.configure(...), exactly once per Purchases instance).
  • logIn(...) onSuccess — only reached when the new appUserID differs from the current one.
  • logOut(...) success branch — the appUserID has just been replaced with a fresh anonymous id.
  • switchUser(...) — the early-return on the same-id path filters out no-ops before we get here.
• Every call site is gated on the new compile-time flag BuildConfig.ENABLE_REMOTE_CONFIG, fed from local.properties (ENABLE_REMOTE_CONFIG=true). Default: false, so disabling the flag is a no-op at every call site. Mirrors the existing ENABLE_EXTRA_REQUEST_LOGGING pattern.
• Existing orchestrator tests are updated to pass a relaxed mock for the new constructor parameter. No tests are added for the flag-on path: BuildConfig.ENABLE_REMOTE_CONFIG is false in CI unit-test builds, so that branch is unreachable from JUnit. Same precedent as ENABLE_EXTRA_REQUEST_LOGGING. The on-path is exercised manually via purchase-tester against a localhost backend (combine with REMOTE_CONFIG_BASE_URL from Add network scaffolding for remote config endpoint #3435).

All new types remain internal. No public API surface change.

Stack

• ⬇ Add remote-config network scaffolding: Add network scaffolding for remote config endpoint #3435
• ⬇ Add RemoteConfigManager and TopicFetcher: Add RemoteConfigManager and TopicFetcher #3437
• ⬇ Clean up unreferenced topic files after successful remote-config refresh: Clean up unreferenced topic files after successful remote-config refresh #3439
• This PR

Checklist

• Existing unit tests pass (orchestrator tests pick up the new relaxed mock; flag-on path is dev-only, tested manually).
• Follow-up issues for purchases-ios / hybrids (deferred — feature is not yet wired to any consumer).

[tonidero]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Use product_entitlement_mapping topic blob for offline entitlements when remote config enabled #3455
• Wire RemoteConfigManager refresh on first foreground and identity changes (off by default) #3450 👈 (View in Graphite)
• Fall back across blob sources when topic downloads fail #3466
• Select blob source by priority and weighted random #3458
• Clean up unreferenced topic files after successful remote-config refresh #3439
• Cache remote config response in memory with TTL and persist to disk #3457
• Add RemoteConfigManager and TopicFetcher #3437
• Add network scaffolding for remote config endpoint #3435
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov]

Codecov Report

❌ Patch coverage is 92.85714% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 79.98%. Comparing base (4650c71) to head (bf72a8a).

Files with missing lines	Patch %	Lines
.../com/revenuecat/purchases/PurchasesOrchestrator.kt	85.71%	1 Missing ⚠️

Additional details and impacted files

@@                              Coverage Diff                               @@
##           toniricodiez/fall-back-across-blob-sources    #3450      +/-   ##
==============================================================================
+ Coverage                                       79.96%   79.98%   +0.02%     
==============================================================================
  Files                                             369      369              
  Lines                                           14937    14951      +14     
  Branches                                         2068     2069       +1     
==============================================================================
+ Hits                                            11944    11959      +15     
+ Misses                                           2154     2153       -1     
  Partials                                          839      839              

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

[emerge-tools]

📸 Snapshot Test

591 unchanged

Name	Added	Removed	Modified	Renamed	Unchanged	Errored	Approval
TestPurchasesUIAndroidCompatibility Paparazzi com.revenuecat.testpurchasesuiandroidcompatibility.paparazzi	0	0	0	0	257	0	N/A
TestPurchasesUIAndroidCompatibility com.revenuecat.testpurchasesuiandroidcompatibility	0	0	0	0	334	0	N/A

---

🛸 Powered by Emerge Tools