[EXTERNAL] fix: prefix browser-mode detection logs with [RevenueCat] (#1953) via @cpruijsen

[rickvdl]

Thank you for contributing to react-native-purchases. Before pressing the "Create Pull Request" button, please provide the following:

• A description about what and why you are contributing, even if it's trivial.

• The issue number(s) or PR number(s) in the description if you are contributing in response to those.

  • If applicable, unit tests.

Prefixes the three existing console.log calls in shouldUseBrowserMode() with [RevenueCat]. Other TypeScript logs from this package's default handler are tagged [RevenueCat], so the unprefixed web / Expo Go / Rork detection line is hard to filter in a busy console. Detection is unchanged: the log still runs at module import, before configure().

Prefix is [RevenueCat], not [purchases]. The issue asked for [purchases] "as other messages." Native iOS NSLog and purchases-js use [Purchases] / [purchases]. [RevenueCat] is already the TypeScript log tag in this package (default log handler, isConfigured(), etc.).

The detection logs are not gated on setLogLevel. The issue asked whether this line should respect log level. Deferring until configure() / setLogLevel(), or dropping the log, would be a larger behavior change: the log runs at module import, before any log level exists.

Can switch the prefix, or defer or silence the log.

Unit tests in __tests__/environment.test.ts assert the prefixed strings for web, Expo Go, and Rork, and that native iOS does not log.

Fixes #1709.

---

Note

Low Risk
Logging-only change with no impact on browser-mode detection or purchase flows; covered by new unit tests.

Overview
Browser-mode detection messages from shouldUseBrowserMode() now go through a small logBrowserModeDetection() helper that prefixes them with [RevenueCat], matching other TypeScript logs in the package so they are easier to filter in the console. Detection behavior is unchanged—only the log format.

Adds __tests__/environment.test.ts with tests for prefixed web, Expo Go, and Rork messages, and that native iOS does not emit these logs.

Fixes #1709.

^{Reviewed by Cursor Bugbot for commit f5d8a28. Bugbot is set up for automated code reviews on this repo. Configure here.}

Thank you for contributing to react-native-purchases. Before pressing the "Create Pull Request" button, please provide the following:

• A description about what and why you are contributing, even if it's trivial.

• The issue number(s) or PR number(s) in the description if you are contributing in response to those.

• If applicable, unit tests.

[tonidero]

I guess we could potentially build the message with the prefix in one place instead of adding the prefix in 3 places... But TBH, I think its fine as is :)