fix: Make native modules built with Neon work in Bun

Author: kjvalencik

crates/neon/src/sys/bindings/functions.rs

@@ -371,19 +371,19 @@ pub(super) unsafe fn load(env: Env) -> Result<(), libloading::Error> {
     // with `Error: Module did not self-register` if N-API does not exist.
     let version = get_version(&host, env).expect("Failed to find N-API version");
 
-    napi1::load(&host, version, 1)?;
+    napi1::load(&host, version, 1);
 
     #[cfg(feature = "napi-4")]
-    napi4::load(&host, version, 4)?;
+    napi4::load(&host, version, 4);
 
     #[cfg(feature = "napi-5")]
-    napi5::load(&host, version, 5)?;
+    napi5::load(&host, version, 5);
 
     #[cfg(feature = "napi-6")]
-    napi6::load(&host, version, 6)?;
+    napi6::load(&host, version, 6);
 
     #[cfg(feature = "napi-8")]
-    napi8::load(&host, version, 8)?;
+    napi8::load(&host, version, 8);
 
     Ok(())
 }

crates/neon/src/sys/bindings/mod.rs

@@ -118,7 +118,7 @@ macro_rules! generate {
 
         #[inline(never)]
         fn panic_load<T>() -> T {
-            panic!("Must load N-API bindings")
+            panic!("Node-API symbol has not been loaded")
         }
 
         static mut NAPI: Napi = {
@@ -139,21 +139,29 @@ macro_rules! generate {
             host: &libloading::Library,
             actual_napi_version: u32,
             expected_napi_version: u32,
-        ) -> Result<(), libloading::Error> {
+        ) {
             assert!(
                 actual_napi_version >= expected_napi_version,
-                "Minimum required N-API version {}, found {}.",
+                "Minimum required Node-API version {}, found {}.",
                 expected_napi_version,
                 actual_napi_version,
             );
 
             NAPI = Napi {
                 $(
-                    $name: *host.get(napi_name!($name).as_bytes())?,
+                    $name: match host.get(napi_name!($name).as_bytes()) {
+                        Ok(f) => *f,
+                        // Node compatible runtimes may not have full coverage of Node-API
+                        // (e.g., bun). Instead of failing to start, warn on start and
+                        // panic when the API is called.
+                        // https://github.com/Jarred-Sumner/bun/issues/158
+                        Err(err) => {
+                            eprintln!("WARN: {}", err);
+                            NAPI.$name
+                        },
+                    },
                 )*
             };
-
-            Ok(())
         }
 
         $(