Make NativeModules.foo also load turbo modules - Githubissues

facebook / react-native

A framework for building native applications using React

https://reactnative.dev

MIT License

119.28k stars 24.35k forks source link

Make NativeModules.foo also load turbo modules #47598

Open RSNara opened 3 days ago

RSNara commented 3 days ago

Summary:

Changes

Now:

in bridgeless, NativeModules.foo will also return turbo modules. And, global.__turboModuleProxy no longer exists.
in bridge, nothing changes.

| JS API | Bridge | [Before] Bridgeless | [Before] Bridgeless w/ Interop| [After] Bridgeless | global.__turboModuleProxy | turbo modules | turbo modules | turbo modules |deleted | global.nativeModuleProxy | legacy modules | error | legacy modules | turbo + legacy modules

Justification

This reduces the cost for adopting the new architecture:

Prior, you had to migrate the module itself, and all its callsites: NativeModules.foo -> NativeFoo
Now, you have to migrate the module itself only.

This simplifies the interop layer logic in bridgeless: all modules come from the same thing.

Changelog: [General][Breaking] Bridgeless: Make NativeModules.foo load turbomodules (unset turboModuleProxy in bridgeless).

Differential Revision: D65896934

facebook-github-bot commented 3 days ago

This pull request was exported from Phabricator. Differential Revision: D65896934

facebook-github-bot commented 1 day ago

This pull request was exported from Phabricator. Differential Revision: D65896934

facebook-github-bot commented 1 day ago

This pull request was exported from Phabricator. Differential Revision: D65896934

facebook-github-bot commented 1 day ago

This pull request was exported from Phabricator. Differential Revision: D65896934

facebook-github-bot commented 1 day ago

This pull request was exported from Phabricator. Differential Revision: D65896934