Don't expose debug_env

Author: lionel-

crates/ark/src/interface.rs

@@ -259,7 +259,7 @@ pub struct RMain {
     /// Current topmost environment on the stack while waiting for input in the
     /// debugger. This is `Some()` only when R is idle and in a `browser()`
     /// prompt.
-    pub(crate) debug_env: Option<RObject>,
+    debug_env: Option<RObject>,
 }
 
 /// Represents the currently active execution request from the frontend. It
@@ -2072,6 +2072,11 @@ impl RMain {
             StdInRpcReply::Interrupt => Ok(RObject::null()),
         }
     }
+
+    #[cfg(not(test))] // Avoid warnings in unit test
+    pub(crate) fn debug_env(&self) -> Option<RObject> {
+        self.debug_env.clone()
+    }
 }
 
 /// Report an incomplete request to the frontend

crates/ark/src/lsp/completions/sources/composite/search_path.rs

@@ -65,7 +65,7 @@ fn completions_from_search_path(
         #[cfg(not(test))] // Unit tests do not have an `RMain`
         {
             use crate::interface::RMain;
-            if let Some(debug_env) = &RMain::get().debug_env {
+            if let Some(debug_env) = &RMain::get().debug_env() {
                 // Mem-Safety: Object protected by `RMain` for the duration of the `r_task()`
                 env = debug_env.sexp;
             }