From 81d0580d1fc9a8cc1aa19d74cbc1d86d06b368d0 Mon Sep 17 00:00:00 2001
From: "K.J. Valencik" <kjvalencik@gmail.com>
Date: Fri, 10 Jan 2025 17:29:16 -0500
Subject: [PATCH 1/2] feat(neon): Allow getting a `&'cx T` from a `Handle<'cx,
 JsBox<T>>`

Resolves https://github.com/neon-bindings/neon/issues/678
---
 crates/neon/src/types_impl/boxed.rs | 31 +++++++++++++++++++++++++++--
 1 file changed, 29 insertions(+), 2 deletions(-)

diff --git a/crates/neon/src/types_impl/boxed.rs b/crates/neon/src/types_impl/boxed.rs
index 02d3557f6..eaac6de51 100644
--- a/crates/neon/src/types_impl/boxed.rs
+++ b/crates/neon/src/types_impl/boxed.rs
@@ -259,12 +259,39 @@ impl<T: Finalize + 'static> JsBox<T> {
     }
 }
 
+impl<T: 'static> JsBox<T> {
+    /// Gets a reference to the inner value of a [`JsBox`]. This method is similar to
+    /// [dereferencing](JsBox::deref) a `JsBox` (e.g., `&*boxed`), but the lifetime
+    /// is _safely_ extended to `'cx`.
+    ///
+    /// See also [`Handle<JsBox>::as_inner`].
+    // N.B.: This would be cleaner with https://github.com/rust-lang/rust/issues/44874
+    pub fn deref<'cx>(v: &Handle<'cx, Self>) -> &'cx T {
+        v.as_inner()
+    }
+}
+
+impl<'cx, T: 'static> Handle<'cx, JsBox<T>> {
+    /// Gets a reference to the inner value of a [`JsBox`]. This method is similar to
+    /// [dereferencing](JsBox::deref) a `JsBox` (e.g., `&*boxed`), but the lifetime
+    /// is _safely_ extended to `'cx`.
+    ///
+    /// See also [`JsBox::deref`].
+    pub fn as_inner(&self) -> &'cx T {
+        // # Safety
+        // JS values associated with an in-scope `Context` *cannot* be garbage collected.
+        // This value is guaranteed to live at least as long as `'cx`.
+        unsafe { &*self.0.raw_data }
+    }
+}
+
 impl<T: 'static> Deref for JsBox<T> {
     type Target = T;
 
     fn deref(&self) -> &Self::Target {
-        // Safety: This depends on a `Handle<'a, JsBox<T>>` wrapper to provide
-        // a proper lifetime.
+        // # Safety
+        // `T` will live at least as long as `JsBox<T>` because it may not be garbage
+        // collected while in scope and only immutable references can be obtained.
         unsafe { &*self.0.raw_data }
     }
 }

From c0f044232793c294b21b7ced84266c035646e4fe Mon Sep 17 00:00:00 2001
From: "K.J. Valencik" <kjvalencik@gmail.com>
Date: Mon, 13 Jan 2025 08:40:57 -0500
Subject: [PATCH 2/2] fix(CI): Allow X server for electron tests

---
 .github/workflows/ci.yml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index f7dd1310d..1eed2558f 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -96,6 +96,10 @@ jobs:
       - name: npm install
         run: npm ci --prefer-offline --no-audit --no-fund
 
+      - name: Allow unprivileged X server
+        if: matrix.os == 'ubuntu-latest'
+        run: sudo sysctl -w kernel.apparmor_restrict_unprivileged_userns=0
+
       - name: Test (Linux)
         if: matrix.os == 'ubuntu-latest'
         run: xvfb-run --auto-servernum npm test -- --nocapture