Fix/rn 0.78 compatibility

[jvsena42]

Description

Fixes Android build errors when using react-native-ldk with React Native 0.78.

Problem

The package fails to compile on RN 0.78 with errors:

• Unresolved reference 'reactContext'
• Unresolved reference 'runOnUiThread'

Solution

1. Added getReactContext() getter method to LdkEventEmitter object
2. Updated LdkPersister.kt to use LdkEventEmitter.getReactContext() instead of LdkModule.reactContext
3. Replaced deprecated runOnUiThread() with runOnUiQueueThread()

Testing

1. Before linking, the TypeScript source must be compiled:

cd /path/to/react-native-ldk/lib
yarn install  # This runs prepublish script which builds the dist/ folder

2. Link the local package:

# In the consuming project's package.json
"@synonymdev/react-native-ldk": "file:/path/to/react-native-ldk/lib"

# Then install
yarn install

3. Verify the local version is linked:

ls -la node_modules/@synonymdev/react-native-ldk/
grep "getReactContext" node_modules/@synonymdev/react-native-ldk/android/src/main/java/com/reactnativeldk/LdkModule.kt
grep "runOnUiQueueThread" node_modules/@synonymdev/react-native-ldk/android/src/main/java/com/reactnativeldk/classes/LdkPersister.kt

4. Clean previous builds:

cd android
./gradlew clean
cd ..

5. Build and run:

yarn start
# Open another terminal
yarn android

• Builds successfully on Android with RN 0.78
• App runs without crashes
• LDK functionality works as expected

[jvsena42]

Worked locally, trying to fix the CI

[Copilot]

Pull Request Overview

This PR fixes Android build compatibility issues with React Native 0.78 by updating deprecated APIs and improving context access patterns in the react-native-ldk package.

Key changes:

• Added a getter method for ReactContext in LdkEventEmitter to resolve reference errors
• Replaced deprecated runOnUiThread() with runOnUiQueueThread() for RN 0.78 compatibility
• Added Metro config blocklist to prevent nested dependency conflicts

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
lib/package.json	Version bump to 0.0.162
lib/android/src/main/java/com/reactnativeldk/classes/LdkPersister.kt	Updated to use new context getter and RN 0.78 thread method
lib/android/src/main/java/com/reactnativeldk/LdkModule.kt	Added getReactContext() getter method to LdkEventEmitter
example/tests/eclair.ts	Added missing trailing comma
example/metro.config.js	Added blocklist to exclude nested node_modules

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[jvsena42]

The CI failures will be fixed in other branch

[ovitrif]

Approved with some nit comments for cleanup as feasible.

[ovitrif]

nit: this change doesn't make a lot of sense to me. Why would we change what was already there?!

[jvsena42]

The method runOnUiThread doesn't exist on ReactContext class, that is why I assumed it was removed
https://github.com/facebook/react-native/blob/0ee4a2d01a7abac197851d648260a7a34648c59d/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContext.java#L369

Here you can see that ReactContextBaseJavaModule is a child of BaseJavaModule
https://github.com/facebook/react-native/blob/main/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContextBaseJavaModule.kt

And here you can see that the mReactApplicationContext is a private property
https://github.com/facebook/react-native/blob/main/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/BaseJavaModule.java

[jvsena42]

the ReactNative version of the project was updated

[ovitrif]

nit: this change doesn't make a lot of sense to me. Why would we change what was already there?!