fix: clean up some node-api details (#24178)

- fix a few napi_* types
- clean up env hooks
- implement blocking queue in tsfn
- reduce transmutes
- use new `DataView::new` api from rusty_v8

1 files changed, 1 insertions, 3 deletions

diff --git a/ext/napi/value.rs b/ext/napi/value.rs
index 6fb96758c..71beac07e 100644
--- a/ext/napi/value.rs
+++ b/ext/napi/value.rs
@@ -31,9 +31,7 @@ where
   v8::Local<'s, T>: Into<v8::Local<'s, v8::Value>>,
 {
   fn from(v: v8::Local<'s, T>) -> Self {
-    // SAFETY: It is safe to cast v8::Local<T> that implements Into<v8::Local<v8::Value>>.
-    //         `fn into(self)` transmutes anyways.
-    Self(unsafe { transmute(v) }, std::marker::PhantomData)
+    Self(Some(NonNull::from(&*v.into())), std::marker::PhantomData)
   }
 }