Upstreaming the Node-API host into React Native core ☂️
還沒有人認領這個 Issue。
評估
- 難度
- 5/5
- 預估耗時
- 一週以上
- 新手友好度
- 25/100
- Issue 類型
- 功能
- 描述清晰度
- 需要釐清
- 活躍度
- 冷清
- 技術堆疊
- cpp, react-native
研究方向
先閱讀 cpp/HermesNapiHost.cpp 和 cpp/RuntimeNodeApi.cpp,接著檢視 #222、#91、#368、#412 以及 weak-node-api 討論。只有當最小 upstream 邊界、前置條件,以及關於排程、環境和 addon 連結的未解決問題都在一份提案中確定下來後,此 issue 才算完成。
由索引模型根據 Issue 內容生成。
描述
A tracking issue for a long-term ambition: propose the core of this project — creating a Node-API environment and loading an addon into it — for inclusion in React Native itself, and keep the build tooling here.
Nothing here is committed to. The point is to have one place to argue about the seam, and to record what has to be true before a proposal is worth writing.
Why the cut is worth making
The repo currently spans two quite different kinds of work:
- A runtime capability. A TurboModule (
CxxNodeApiHostModule) that creates anapi_envper addon, loads a dynamic library into it, and supplies the host primitives Node-API needs but React Native has no equivalent of — async work, thread-safe function dispatch, fatal-error routing (HostContextincpp/HermesNapiHost.cpp,cpp/RuntimeNodeApi.cpp). - A build system.
cmake-rn,ferric,gyp-to-cmake, the autolinking CLI, the Babel plugin, the prebuild naming and packaging conventions.
(1) is small, general, and belongs anywhere React Native runs. (2) is opinionated, iterates fast, and is a much bigger ask to hand to another project. Asking core to adopt both is likely to sink the whole proposal; asking for (1) alone is close to a one-sentence pitch.
The seam: loading an addon
The natural boundary is the loading primitive itself — the thing #222 describes as a process.dlopen analogue. Core would own "given a reference to an addon, load it and give me its exports"; everything that decides the reference stays here.
Two things to get right in a proposal:
It cannot honestly be process.dlopen(module, filename). Node's takes an absolute filesystem path. On neither of our platforms is there one: on Android the library is lib<name>.so resolved by the dynamic linker out of the APK, on Apple it is @rpath/<name>.framework/<name>. A dlopen-named API that does not accept a path will cost the proposal a long argument about its name. Something honest — load a Node-API addon by library name, resolved the platform's way — is both defensible and a smaller API surface, and the path-shaped case can be an extension later if it ever makes sense.
The naming must not leak across the seam. Whatever core accepts has to be free of our library-name mangling (see #222, #91 and the manifest discussion in #368). If the strategy leaks upstream, we can never change it.
What already argues in favor
- Hermes implements Node-API itself. The engine at our pinned commit ships the whole surface, plus
hermes_napi_create_envandhermes_napi_load_module, which we now use directly (#445). The upstream ask reduces to "expose what the engine already does", not "implement Node-API". - #412 — once React Native ships a Hermes with Node-API, we stop vendoring, and the remaining host code is small enough to read in one sitting.
- The frameworks are no longer preloaded (#351), so addons load on demand and the loading path is ordinary, not a startup-order special case.
The weak-node-api problem
This is the main obstacle, and it is not build tooling, so it does not fall cleanly on our side of the cut.
Addons are compiled independently of any app, so they cannot link against the app's Hermes. Today they link weak-node-api: generated forwarding stubs for every Node-API symbol, backed by a function table the host injects at startup. If the host moves to core, addons still need something to link, and "core gains one TurboModule method" turns into "core gains a method and takes on a shim package" — a much harder sell.
The long-term dissolution is nodejs/node#60916 ("node-api: use v-table to reverse module dependencies", @vmoroz), which inverts the dependency: the runtime exposes its API through vtables reachable from napi_env, and an addon built with NODE_API_MODULE_USE_VTABLE links against nothing at all. That removes the reason weak-node-api exists, and with it the awkward part of the cut. It is upstream, open, and not something to plan a schedule around — but it is the shape of the end state, and worth tracking here.
Until then, options worth thinking through:
- Keep
weak-node-apias a peer package here and propose only the host to core, accepting that addons still need our shim to build. - Have addons link the Hermes/React Native framework directly where the platform allows it, and keep the shim only where it does not.
- Wait for the vtable work and propose the whole thing once addons link nothing.
Prerequisites on our side
Roughly in order, and all independently useful:
- A raw, naming-free loading API in JS — the shape core would eventually own (#222).
- A friendly resolver layered on top of it, taking
{ packageName, path }(#222, #91). - Make the library name data rather than a shared algorithm — a manifest emitted by the linking step (
getLibraryMap()already computes exactly this map) so JS and the linker cannot disagree, and so per-library overrides (#368) have somewhere to live. - Stop vendoring Hermes (#412).
- Settle the addon-declared Node-API version question, which needs an upstream Hermes change either way (#4).
Open questions
- Does
HostContext's threading model (a process-global worker pool plusCallInvokerdispatch) survive contact with core's own scheduling, or would core want to back those primitives differently? - Is the per-addon
napi_env(as in Node) the right model for core, given a React Native app can have several runtimes over its lifetime? - What is the minimum viable proposal — the TurboModule alone, or does it only make sense together with autolinking, so that an app can actually get an addon into its bundle?
Related
#222, #91, #368, #412, #4, #3, #104
- 主要語言
- TypeScript
- 星號
- 188
- 分支
- 10
- 平均合併
- 2 天 17 小時
- 30 天內合併 PR
- 3
貢獻指南
這個儲存庫沒有索引到貢獻指南
從這裡開始
- 先讀完整個 Issue,再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫,在一個分支上完成修改。
- 送出 Pull Request,並在描述裡引用這個 Issue 編號。
callstackincubator/react-native-node-api 的其他 Issue
-
Android 🤖
難度 2/5 1-3 小時 新手友好度 66/100
-
Sanity check of Hermes source directory via environment variable, when building Android projects 未關閉Android 🤖 good first issue
難度 2/5 1-3 小時 新手友好度 78/100
callstackincubator/react-native-node-api#163 · 1 則留言 ·
-
enhancement good first issue gyp-to-cmake
難度 2/5 1-3 小時 新手友好度 72/100
callstackincubator/react-native-node-api#161 · 3 則留言 · 1 個 reaction ·
-
Automatable good first issue
難度 3/5 1-2 天 新手友好度 15/100
callstackincubator/react-native-node-api#424 · 1 則留言 ·
-
gyp-to-cmake
難度 4/5 3-5 天 新手友好度 55/100
callstackincubator/react-native-node-api#423 · 1 則留言 ·
查看 callstackincubator/react-native-node-api 的全部 Issue
相似的 Issue
-
comp/desktop P3 type/bug
難度 1/5 1 小時以內 新手友好度 92/100
NousResearch/hermes-agent#118866 ·
-
Browser Waiting for: Product Owner
難度 2/5 1-3 小時 新手友好度 85/100
getsentry/sentry-javascript#24577 · 1 則留言 ·
-
難度 2/5 1-3 小時 新手友好度 78/100
agilepathway/label-checker#640 ·
-
Plugin stuck at "loading" on DSH 0.1.6-alpha.2 — turnTail list slot registration missing options.id 未關閉
難度 2/5 1-3 小時 新手友好度 88/100
-
難度 2/5 1-3 小時 新手友好度 68/100
chrisparsons83/flexspotff#153 ·