Added implementation of V1 async_work runtime functions

[paradowstack]

This PR brings in:

• Implementation of following V1 async work runtime functions using facebook::react::CallInvoker:
  • napi_create_async_work
  • napi_queue_async_work
  • napi_delete_async_work
  • napi_cancel_async_work
• Introduces test addon, adopted from https://github.com/nodejs/node/blob/main/test/node-api/test_async
• Adds a proper assertion to addon tests
• Fixed an bug in implementation of napi_create_external_buffer

[changeset-bot]

🦋 Changeset detected

Latest commit: 5ce6a02

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Copilot]

Pull Request Overview

This PR implements the V1 async work runtime functions (napi_create_async_work, napi_queue_async_work, napi_delete_async_work, napi_cancel_async_work) using facebook::react::CallInvoker, adds a new async test suite (C and JS), refactors buffer tests to use shared macros, and fixes a bug in napi_create_external_buffer.

• Wire up and expose async work APIs in the host runtime
• Add end-to-end async tests under tests/async
• Refactor buffer tests to use RuntimeNodeApiTestsCommon.h and proper assert calls

Reviewed Changes

Copilot reviewed 19 out of 20 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
packages/node-addon-examples/tests/buffers/addon.js	Swapped console.log for assert.strictEqual calls
packages/node-addon-examples/tests/buffers/addon.c	Replaced inlined macros with RuntimeNodeApiTestsCommon.h
packages/node-addon-examples/tests/async/…	Added new async test package (JS and C), gyp, CMake setup
packages/node-addon-examples/index.js	Registered the new async test in the examples index
packages/host/scripts/generate-weak-node-api-injector.ts	Included RuntimeNodeApiAsync.hpp and extended function list
packages/host/ios/… and packages/host/android/…	Imported async header and set the CallInvoker
packages/host/cpp/RuntimeNodeApiAsync.hpp/.cpp	Implemented the async work registry and API entrypoints
packages/host/cpp/RuntimeNodeApi.hpp/.cpp	Fixed napi_create_external_buffer implementation
apps/test-app/App.tsx	Updated test runner to handle async test functions
.changeset/rich-donuts-mix.md	Added changelog entry for async work support

Comments suppressed due to low confidence (2)

packages/node-addon-examples/tests/buffers/addon.js:21

• This call triggers an invalid buffer conversion but lacks an assertion. Consider wrapping it in assert.throws to verify that the correct error is thrown.

  addon.invalidObjectAsBuffer({});

packages/host/cpp/RuntimeNodeApiAsync.hpp:19

• [nitpick] The use of node_api_basic_env is inconsistent with the other APIs using napi_env. For consistency and to prevent confusion, consider using napi_env for all async work functions.

napi_status napi_queue_async_work(node_api_basic_env env, napi_async_work work);

[kraenhansen]

In any case, it might make more sense to move this call into the CxxNodeApiHostModule constructor?

[paradowstack]

I thought that having all node api initialisations (callstack::nodeapihost::injectIntoWeakNodeApi() + callstack::nodeapihost::setCallInvoker(caller)) in one place would be more correct. But now when revisiting it, code is duplicated because of Android/iOS anyway, so maybe moving it to the constructor makes more sense - I will change that.

[kraenhansen]

Seems like the tests are now legitimately failing.