Apply suggestions from code review

Co-authored-by: Davis Vaughan <[email protected]>

Author: lionel-

crates/ark/src/interface.rs

@@ -584,18 +584,18 @@ impl RMain {
 
         // In the future we'll also send browser information, see
         // https://github.com/posit-dev/positron/issues/3001. Currently this is
-        // a push model where we send the console scopes at each round. In the
+        // a push model where we send the console inputs at each round. In the
         // future, a pull model would be better, this way the LSP can manage a
-        // cache of scopes and we don't need to retraverse the environments as
-        // often. We'd still push a `DidChangeConsoleScopes` notification from
+        // cache of inputs and we don't need to retraverse the environments as
+        // often. We'd still push a `DidChangeConsoleInputs` notification from
         // here, but only containing high-level information such as `search()`
         // contents and `ls(rho)`.
         if !info.browser && !info.incomplete && !info.input_request {
             match console_inputs() {
                 Ok(inputs) => {
                     self.send_lsp(LspEvent::DidChangeConsoleInputs(inputs));
                 },
-                Err(err) => log::error!("Can't retrieve console scopes: {err:?}"),
+                Err(err) => log::error!("Can't retrieve console inputs: {err:?}"),
             }
         }
 

crates/ark/src/lsp/state.rs

@@ -20,7 +20,7 @@ pub struct WorldState {
     pub workspace: Arc<Mutex<Workspace>>,
 
     /// The scopes for the console. This currently contains a list (outer `Vec`)
-    /// of names (inner `Vec`) of the environments on the search path, starting
+    /// of names (inner `Vec`) within the environments on the search path, starting
     /// from the global environment and ending with the base package. Eventually
     /// this might also be populated with the scope for the current environment
     /// in debug sessions (not implemented yet).