Use find_package instead of include to locate "weak-node-api"

Author: kraenhansen

.changeset/quick-poets-greet.md

@@ -0,0 +1,7 @@
+---
+"gyp-to-cmake": minor
+"cmake-rn": minor
+"react-native-node-api": minor
+---
+
+Use `find_package` instead of `include` to locate "weak-node-api"

packages/cmake-rn/README.md

@@ -10,14 +10,14 @@ Android's dynamic linker imposes restrictions on the access to global symbols (s
 
 The implementation of Node-API is split between Hermes and our host package and to avoid addons having to explicitly link against either, we've introduced a `weak-node-api` library (published in `react-native-node-api` package). This library exposes only Node-API and will have its implementation injected by the host.
 
-To link against `weak-node-api` just include the CMake config exposed through `WEAK_NODE_API_CONFIG` and add `weak-node-api` to the `target_link_libraries` of the addon's library target.
+To link against `weak-node-api` just use `find_package` to import the `weak-node-api` target and add it to the `target_link_libraries` of the addon's library target.
 
 ```cmake
 cmake_minimum_required(VERSION 3.15...3.31)
 project(tests-buffers)
 
 # Defines the "weak-node-api" target
-include(${WEAK_NODE_API_CONFIG})
+find_package(weak-node-api REQUIRED CONFIG)
 
 add_library(addon SHARED addon.c)
 target_link_libraries(addon PRIVATE weak-node-api)

packages/cmake-rn/src/weak-node-api.ts

@@ -58,7 +58,9 @@ export function getWeakNodeApiVariables(
   triplet: SupportedTriplet | "apple",
 ): Record<string, string> {
   return {
-    // Expose an includable CMake config file declaring the weak-node-api target
+    // Enable use of `find_package(weak-node-api REQUIRED CONFIG)`
+    "weak-node-api_DIR": path.dirname(weakNodeApiCmakePath),
+    // Enable use of `include(${WEAK_NODE_API_CONFIG})`
     WEAK_NODE_API_CONFIG: weakNodeApiCmakePath,
     WEAK_NODE_API_INC: getNodeApiIncludePaths().join(";"),
     WEAK_NODE_API_LIB: getWeakNodeApiPath(triplet),

packages/gyp-to-cmake/src/transformer.ts

@@ -85,7 +85,7 @@ export function bindingGypToCmakeLists({
   ];
 
   if (weakNodeApi) {
-    lines.push(`include(\${WEAK_NODE_API_CONFIG})`, "");
+    lines.push(`find_package(weak-node-api REQUIRED CONFIG)`, "");
   }
 
   for (const target of gyp.targets) {