feat: full test coverage on macos & ios, v8 14 (#28)

- Adds full test coverage 100% success for iOS, and introduces macOS test support too
- Introduces some built-in Node.js APIs like node:path, etc. A process global is also added with minimal API surface
- Bunch of fixes in the FFI layer, related to correctness & performance
- Introduces Generated Signature Dispatch: compile time generated trampolines that boost performance of native calls by 20% on average through eliminating libffi layer
- Improved memory management, less leaks. Introduces tests for reliable memory management
- Removes types, build artifacts and metadata binaries from the repo
- Cleans up old runtime related stuff
- Fetches prebuilt V8 binaries now. To be added for Hermes soon.

Author: DjDeveloperr

.github/workflows/ci.yml

@@ -1,12 +1,12 @@
 name: CI
 on:
   push:
-    branches: '*'
+    branches: [main]
   pull_request:
 
 jobs:
   build:
-    runs-on: macos-latest
+    runs-on: macos-26
     steps:
       - uses: actions/checkout@v5
         with:
@@ -20,6 +20,9 @@ jobs:
       - name: Install Dependencies
         run: npm install
 
+      - name: Download V8
+        run: ./download_v8.sh
+
       - name: Install libffi build tools
         run: brew install autoconf automake libtool texinfo
 
@@ -28,6 +31,22 @@ jobs:
 
       - name: Build metadata generator
         run: npm run build-metagen
+
+      - name: Generate macOS metadata
+        run: npm run metagen macos
 
       - name: Build NativeScript
-        run: npm run build --quickjs
+        run: npm run build
+
+      - name: Build CLI
+        run: npm run build:macos-cli
+
+      - name: Run Tests on macOS
+        run: npm run test:macos
+
+      - name: Run Tests on iOS
+        env:
+          IOS_BUILD_TIMEOUT_MS: "600000"
+          IOS_TEST_TIMEOUT_MS: "600000"
+          IOS_TEST_INACTIVITY_TIMEOUT_MS: "180000"
+        run: npm run test:ios

.gitignore

@@ -59,4 +59,10 @@ v8_build
 
 .cache/
 
+dist
+packages/*/types
+
 SwiftBindgen
+
+# Generated Objective-C/C dispatch wrappers
+NativeScript/ffi/GeneratedSignatureDispatch.inc

NativeScript/CMakeLists.txt

@@ -78,7 +78,6 @@ elseif(TARGET_ENGINE STREQUAL "hermes")
   set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -stdlib=libc++ -std=c++20 -DTARGET_ENGINE_HERMES")
 elseif(TARGET_ENGINE STREQUAL "v8")
   set(TARGET_ENGINE_V8 TRUE)
-  add_link_options("-fuse-ld=lld")
   set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fno-rtti -stdlib=libc++ -std=c++20 -DTARGET_ENGINE_V8")
 elseif(TARGET_ENGINE STREQUAL "quickjs")
   set(TARGET_ENGINE_QUICKJS TRUE)
@@ -152,10 +151,16 @@ if(ENABLE_JS_RUNTIME)
     runtime/modules/worker/WorkerImpl.mm
     runtime/modules/worker/WorkerImpl.mm
     runtime/modules/module/ModuleInternal.cpp
+    runtime/modules/node/Node.cpp
+    runtime/modules/node/FS.cpp
+    runtime/modules/node/Path.cpp
+    runtime/modules/node/Process.cpp
     runtime/modules/performance/Performance.cpp
+    runtime/ThreadSafeFunction.mm
     runtime/Bundle.mm
     runtime/modules/timers/Timers.mm
     runtime/modules/app/App.mm
+    runtime/modules/web/Web.mm
     runtime/NativeScript.mm
     runtime/RuntimeConfig.cpp
     runtime/modules/url/ada/ada.cpp
@@ -166,7 +171,6 @@ if(ENABLE_JS_RUNTIME)
   if(TARGET_ENGINE_V8)
     include_directories(
       napi/v8
-      napi/v8/include
       napi/v8/v8_inspector
     )
 
@@ -292,6 +296,17 @@ target_sources(
   "NativeScript.h"
 )
 
+if(TARGET_ENGINE_V8 AND TARGET_PLATFORM_IOS)
+  # iOS V8 slices are built with pointer compression enabled. Keep embedder
+  # build flags in sync to satisfy V8::Initialize() build config checks.
+  target_compile_definitions(
+    ${NAME}
+    PRIVATE
+    V8_COMPRESS_POINTERS
+    V8_31BIT_SMIS_ON_64BIT_ARCH
+  )
+endif()
+
 set(FRAMEWORK_VERSION_VALUE "${VERSION}")
 if(TARGET_PLATFORM_MACOS)
   # macOS framework consumers (including Xcode's copy/sign phases) expect
@@ -392,27 +407,42 @@ if(TARGET_ENGINE_JSC)
 endif()
 
 if(TARGET_ENGINE_V8)
-  if(TARGET_PLATFORM_MACOS)
-    target_link_directories(
-      ${NAME}
-      PRIVATE
-      ${CMAKE_CURRENT_SOURCE_DIR}/../Frameworks/v8_macos
-    )
+  add_library(v8::monolith STATIC IMPORTED GLOBAL)
+
+  set(V8_XCFRAMEWORK "${CMAKE_SOURCE_DIR}/../Frameworks/libv8_monolith.xcframework")
+
+  # Decide platform + slice
+  if(APPLE)
+    if(TARGET_PLATFORM STREQUAL "ios-sim")
+        # Prefer universal sim slice if present
+        set(V8_SLICE_DIR "${V8_XCFRAMEWORK}/ios-arm64-simulator/libv8_monolith.framework")
+        if(NOT EXISTS "${V8_SLICE_DIR}")
+          set(V8_SLICE_DIR "${V8_XCFRAMEWORK}/ios-arm64-simulator/libv8_monolith.framework") # fallback
+        endif()
+    elseif(TARGET_PLATFORM STREQUAL "ios")
+        # Prefer universal sim slice if present
+        set(V8_SLICE_DIR "${V8_XCFRAMEWORK}/ios-arm64/libv8_monolith.framework")
+        if(NOT EXISTS "${V8_SLICE_DIR}")
+          set(V8_SLICE_DIR "${V8_XCFRAMEWORK}/ios-arm64/libv8_monolith.framework") # fallback
+        endif()
+    else()
+      # macOS
+      set(V8_SLICE_DIR "${V8_XCFRAMEWORK}/macos-arm64/libv8_monolith.framework")
+      if(NOT EXISTS "${V8_SLICE_DIR}")
+        set(V8_SLICE_DIR "${V8_XCFRAMEWORK}/macos-arm64/libv8_monolith.framework") # keep same, just explicit
+      endif()
+    endif()
   else()
-    target_link_directories(
-      ${NAME}
-      PRIVATE
-      ${CMAKE_CURRENT_SOURCE_DIR}/../Frameworks/v8_ios
-    )
+    message(FATAL_ERROR "This example is for Apple platforms only.")
   endif()
 
-  target_link_libraries(
-    ${NAME}
-    PRIVATE
-    "v8_monolith"
-    "v8_libbase"
-    "v8_libplatform"
+  # Point imported lib at the slice
+  set_target_properties(v8::monolith PROPERTIES
+    IMPORTED_LOCATION "${V8_SLICE_DIR}/libv8_monolith"
+    INTERFACE_INCLUDE_DIRECTORIES "${V8_SLICE_DIR}/Headers"
   )
+
+  target_link_libraries(${NAME} PRIVATE v8::monolith)
 endif()
 
 target_link_libraries(

NativeScript/cli/main.cpp

@@ -61,6 +61,13 @@ void bootFromModuleSpec(std::string baseDir, std::string spec) {
 
 int main(int argc, char** argv) {
   RuntimeConfig.LogToSystemConsole = true;
+  RuntimeConfig.Arguments.clear();
+  if (argv != nullptr && argc > 0) {
+    RuntimeConfig.Arguments.reserve(static_cast<size_t>(argc));
+    for (int i = 0; i < argc; i++) {
+      RuntimeConfig.Arguments.emplace_back(argv[i] != nullptr ? argv[i] : "");
+    }
+  }
 
 #ifdef __APPLE__
   std::string bytecodePath = getBytecodePathFromBundle();

NativeScript/ffi/Block.h

@@ -14,16 +14,22 @@ class FunctionPointer {
   void* function;
   metagen::MDSectionOffset offset;
   Cif* cif;
+  bool ownsCif = false;
 
   static napi_value wrap(napi_env env, void* function,
                          metagen::MDSectionOffset offset, bool isBlock);
+  static napi_value wrapWithEncoding(napi_env env, void* function,
+                                     const char* encoding, bool isBlock);
   static void finalize(napi_env env, void* finalize_data, void* finalize_hint);
 
   static napi_value jsCallAsCFunction(napi_env env, napi_callback_info cbinfo);
   static napi_value jsCallAsBlock(napi_env env, napi_callback_info cbinfo);
 };
 
 id registerBlock(napi_env env, Closure* closure, napi_value callback);
+napi_value getCachedBlockCallback(napi_env env, void* blockPtr);
+bool isObjCBlockObject(id obj);
+const char* getObjCBlockSignature(void* blockPtr);
 
 NAPI_FUNCTION(registerBlock);