refactor(Mountain): organize module structure and fix RPC error handling

- Uncommented and activated core Mountain modules in Library.rs including `app_state`, `environment`, handlers submodules, and IPC-related components to establish proper code structure for backend implementation
- Fixed error handling in `track::map_common_error_to_rpc_error_string` to correctly reference `error_utils` module, ensuring proper error translation for IPC communication with Cocoon
- Updated Vine IPC layer to use absolute `crate::track` path when dispatching sidecar requests, resolving potential module resolution issues
- Removed redundant line breaks in documentation comments for better code readability
- Activated handler modules for config management, diagnostics, and workspace operations as part of MVP architecture setup

These changes improve code organization in the Mountain backend while strengthening RPC error handling reliability - critical for stable communication with the Cocoon extension host. The module activation aligns with Path A goals by enabling foundational components for extension interoperability and state management.

Author: NikolaRHristov

Source/Library.rs

@@ -8,60 +8,60 @@ fn main() { Binary::Fn::Fn(); }
 pub mod Binary;
 
 // NEW:
-// pub mod app_state;
+pub mod app_state;
 
-// pub mod environment;
+pub mod environment;
 
-// pub mod handlers {
-// 	pub mod commands;
+pub mod handlers {
+	pub mod commands;
 
-// 	pub mod config;
+	pub mod config;
 
-// 	pub mod diagnostics;
+	pub mod diagnostics;
 
-// 	pub mod documents;
+	pub mod documents;
 
-// 	pub mod enablement;
+	pub mod enablement;
 
-// 	pub mod native_fs;
+	pub mod error_utils;
 
-// 	pub mod output;
+	pub mod extension_status;
 
-// 	pub mod process_mgmt;
+	pub mod native_fs;
 
-// 	pub mod protocol;
+	pub mod output;
 
-// 	pub mod proxy;
+	pub mod process_mgmt;
 
-// 	pub mod registry;
+	pub mod protocol;
 
-// 	pub mod secrets;
+	pub mod proxy;
 
-// 	pub mod storage;
+	pub mod registry;
 
-// 	pub mod terminal;
+	pub mod secrets;
 
-// 	pub mod ui;
+	pub mod sky_ui_responses;
 
-// 	pub mod workspace;
+	pub mod storage;
 
-// 	pub mod workspace_fs_api;
+	pub mod terminal;
 
-// 	pub mod error_utils;
+	pub mod ui;
 
-// 	pub mod extension_status;
+	pub mod workspace_fs_api;
 
-// 	pub mod sky_ui_responses;
-// }
+	pub mod workspace;
+}
 
-// pub mod Entry;
+pub mod Entry;
 
-// pub mod mist;
+pub mod mist;
 
-// pub mod track;
+pub mod track;
 
-// pub mod rpc;
+pub mod rpc;
 
-// pub mod runtime;
+pub mod runtime;
 
-// pub mod vine;
+pub mod vine;

Source/handlers/sky_ui_responses.rs

@@ -302,9 +302,8 @@ pub async fn sky_resolves_ui_request(
 				Ok(successful_data)
 			},
 
-			// Case 3: Success with no specific data (e.g., user cancellation of a dialog,
-
-			//         simple ack for a message box), or data was explicitly `null`.
+			// Case 3: Success with no specific data (e.g., user cancellation of a dialog, simple ack for a message
+			// box), or data was explicitly `null`.
 			(None, None) => {
 				debug!(
 					"[Sky UI Resp Handler] ReqID '{}' resolved by Sky with no data and no error. Interpreting as user \

Source/track.rs

@@ -141,7 +141,6 @@ fn create_parameter_parse_error_string(
 }
 
 /// Maps a `CommonError` (from effect execution) to a JSON-RPC error string.
-///
 /// Delegates to `error_utils::map_common_error_to_rpc_string`.
 fn map_common_error_to_rpc_error_string(e:CommonError, operation_context:&str) -> String {
 	error_utils::map_common_error_to_rpc_string(e, operation_context)
@@ -161,14 +160,8 @@ fn map_common_error_to_rpc_error_string(e:CommonError, operation_context:&str) -
 /// * `window` - The Tauri `Window` context.
 /// * `runtime` - Managed `Arc<AppRuntime>` for executing effects.
 /// * `command` - The string identifier of the command to dispatch (e.g.,
-///
-///
-///
 ///   `Land_Echo::REQUEST_READ_FILE`).
 /// * `args` - A `serde_json::Value` containing the arguments for the command,
-///
-///
-///
 ///   typically an object.
 ///
 /// # Returns
@@ -230,9 +223,6 @@ pub async fn dispatch_command<R:TauriRuntime>(
 // --- Sidecar Request/Notification Dispatcher (Called by Vine) ---
 
 /// Dispatches requests and notifications received from a sidecar process (e.g.,
-///
-///
-///
 /// Cocoon) via the Vine IPC layer.
 ///
 /// This function tries to:
@@ -700,6 +690,7 @@ pub async fn dispatch_sidecar_request<R:TauriRuntime>(
 ///
 ///
 ///
+///
 /// `$dispose`.
 fn is_output_method_fallback_candidate(method_name:&str) -> bool {
 	matches!(method_name, "$register" | "$append" | "$replace" | "$reveal" | "$close")

Source/vine.rs

@@ -122,7 +122,7 @@ use tokio::{
 };
 
 // For dispatching incoming requests from sidecars
-use crate::{runtime::AppRuntime, track};
+use crate::runtime::AppRuntime;
 
 // --- Vine Error Type ---
 /// Defines errors specific to the Vine IPC layer.
@@ -883,7 +883,7 @@ async fn process_incoming_line_from_sidecar<R:Runtime>(
                             (Some(main_window), Some(app_runtime_state)) if app_runtime_state.inner().is_some() => {
 
 								// Dispatch the request to Track.
-                                match track::dispatch_sidecar_request(
+                                match crate::track::dispatch_sidecar_request(
                                     app_handle.clone(),
 
                                     main_window,