fix: all sdkReplayConfig should have a default value

[marandaneto]

Problem

https://posthog.com/questions/installing-session-replay-and-updating-posthog-crashes-the-app

Changes

Release info Sub-libraries affected

Bump level

• [ ] Major
• [ ] Minor
• [X] Patch

Libraries affected

• [ ] All of them
• [ ] posthog-web
• [ ] posthog-node
• [X] posthog-react-native

Changelog notes

• Added support for X

[github-actions[bot]]

Size Change: +162 B (+0.15%)

Total Size: 106 kB

Filename	Size	Change
posthog-react-native/lib/posthog-react-native/src/posthog-rn.js	3.97 kB	+162 B (+4.26%)

ℹ️ View Unchanged

| Filename | Size | | :--- | :---: | | `posthog-node/lib/index.cjs.js` | 20 kB | | `posthog-node/lib/index.esm.js` | 20 kB | | `posthog-react-native/lib/posthog-core/src/eventemitter.js` | 1.08 kB | | `posthog-react-native/lib/posthog-core/src/index.js` | 9.72 kB | | `posthog-react-native/lib/posthog-core/src/lz-string.js` | 1.42 kB | | `posthog-react-native/lib/posthog-core/src/types.js` | 365 B | | `posthog-react-native/lib/posthog-core/src/utils.js` | 822 B | | `posthog-react-native/lib/posthog-core/src/vendor/uuidv7.js` | 2.04 kB | | `posthog-react-native/lib/posthog-react-native/index.js` | 485 B | | `posthog-react-native/lib/posthog-react-native/src/autocapture.js` | 1.78 kB | | `posthog-react-native/lib/posthog-react-native/src/frameworks/wix-navigation.js` | 651 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useFeatureFlag.js` | 437 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useFeatureFlags.js` | 362 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useLifecycleTracker.js` | 416 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useNavigationTracker.js` | 628 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/usePostHog.js` | 249 B | | `posthog-react-native/lib/posthog-react-native/src/legacy.js` | 810 B | | `posthog-react-native/lib/posthog-react-native/src/native-deps.js` | 1.17 kB | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalAsyncStorage.js` | 183 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoApplication.js` | 215 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoDevice.js` | 211 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoFileSystem.js` | 224 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoLocalization.js` | 216 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalReactNativeDeviceInfo.js` | 220 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalReactNativeNavigation.js` | 218 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalReactNativeNavigationWix.js` | 222 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalSessionReplay.js` | 231 B | | `posthog-react-native/lib/posthog-react-native/src/PostHogContext.js` | 210 B | | `posthog-react-native/lib/posthog-react-native/src/PostHogProvider.js` | 1.49 kB | | `posthog-react-native/lib/posthog-react-native/src/storage.js` | 1.09 kB | | `posthog-react-native/lib/posthog-react-native/src/types.js` | 90 B | | `posthog-react-native/lib/posthog-react-native/src/version.js` | 123 B | | `posthog-web/lib/index.cjs.js` | 17.1 kB | | `posthog-web/lib/index.esm.js` | 17.1 kB |

_{compressed-size-action}

[marandaneto]

Not fluent in js but I think we could also use the spread operator to merge defaults with params.

LGTM though

Yes, the spread operator would merge but it'd have the wrong result if someone passes eg maskAllTextInputs: undefined, the undefined would have precedence over the default value, with the current form, no, the default value would have precedence.

[ioannisj]

Yeah, super valid thinking. Awesome :)