refactor(Mountain/Environment): Improve command handling and workspace search logic

- Added manual `Clone` implementation for `CommandHandler` to support function pointers, ensuring command handlers can be properly cloned during registration in `Mountain`'s command system
- Fixed state retrieval in `CommandProvider` by adding explicit type annotation when accessing `ApplicationRunTime` from Tauri state
- Enhanced error handling for proxied commands by mapping gRPC errors to `CommonError::IPCError`
- Simplified type usage in `TreeViewProvider` by leveraging `TreeViewOptionsDTO` alias
- Corrected file search logic in `WorkSpaceProvider` by fixing include/exclude pattern matching and path handling
- Updated `Cargo.toml` to explicitly include build assets for packaging

These changes strengthen the command execution pipeline between `Mountain` and `Cocoon`, fix file search pattern handling for workspace operations, and ensure proper build artifact inclusion. The improvements directly support implemented workflows like Command Palette execution and file search functionality.

Author: NikolaRHristov

Cargo.toml

@@ -82,4 +82,17 @@ edition = "2024"
 license-file = "LICENSE"
 name = "Mountain"
 publish = false
+include = [
+	"build.rs",
+	"capabilities/**/*",
+	"Cargo.toml",
+	"CHANGELOG.md",
+	"icons/**/*",
+	"LICENSE",
+	"Mountain.key.pub",
+	"Proto/**/*",
+	"README.md",
+	"Source/**/*",
+	"tauri.conf.json",
+]
 version = "0.0.1"

Source/Environment/CommandProvider.rs

@@ -21,7 +21,6 @@ use super::MountainEnvironment::MountainEnvironment;
 use crate::{RunTime::ApplicationRunTime::ApplicationRunTime, Vine::Client};
 
 /// An enum representing the different ways a command can be handled.
-#[derive(Clone)]
 pub enum CommandHandler<R:Runtime + 'static> {
 	/// A command handled by a native, asynchronous Rust function.
 	Native(
@@ -36,6 +35,21 @@ pub enum CommandHandler<R:Runtime + 'static> {
 	Proxied { SidecarIdentifier:String, CommandIdentifier:String },
 }
 
+// Manually implement Clone because fn pointers are not automatically Clone
+impl<R:Runtime> Clone for CommandHandler<R> {
+	fn clone(&self) -> Self {
+		match self {
+			Self::Native(f) => Self::Native(*f),
+			Self::Proxied { SidecarIdentifier, CommandIdentifier } => {
+				Self::Proxied {
+					SidecarIdentifier:SidecarIdentifier.clone(),
+					CommandIdentifier:CommandIdentifier.clone(),
+				}
+			},
+		}
+	}
+}
+
 #[async_trait]
 impl CommandExecutor for MountainEnvironment {
 	/// Executes a registered command by dispatching it to the appropriate
@@ -52,7 +66,8 @@ impl CommandExecutor for MountainEnvironment {
 		match HandlerInfoOption {
 			Some(CommandHandler::Native(Function)) => {
 				debug!("[CommandProvider] Executing NATIVE command '{}'.", CommandIdentifier);
-				let RunTime:Arc<ApplicationRunTime> = self.ApplicationHandle.state().inner().clone();
+				let RunTime:Arc<ApplicationRunTime> =
+					self.ApplicationHandle.state::<Arc<ApplicationRunTime>>().inner().clone();
 				let MainWindow = self.ApplicationHandle.get_webview_window("main").ok_or_else(|| {
 					CommonError::UserInterfaceInteraction {
 						Reason:"Main window not found for command execution".into(),
@@ -72,7 +87,9 @@ impl CommandExecutor for MountainEnvironment {
 				);
 				let RPCParameters = json!([ProxiedCommandIdentifier, Argument]);
 				let RPCMethod = format!("{}$ExecuteContributedCommand", ProxyTarget::ExtHostCommands.GetTargetPrefix());
-				Client::SendRequest(&SidecarIdentifier, RPCMethod, RPCParameters, 30000).await
+				Client::SendRequest(&SidecarIdentifier, RPCMethod, RPCParameters, 30000)
+					.await
+					.map_err(|e| CommonError::IPCError { Description:e.to_string() })
 			},
 			None => {
 				error!("[CommandProvider] Command '{}' not found in registry.", CommandIdentifier);

Source/Environment/TreeViewProvider.rs

@@ -4,7 +4,10 @@
 //! provider manages the lifecycle of custom tree views and orchestrates the
 //! data flow between the extension host (`Cocoon`) and the UI (`Sky`).
 
-use Common::{Error::CommonError::CommonError, TreeView::TreeViewProvider::TreeViewProvider};
+use Common::{
+	Error::CommonError::CommonError,
+	TreeView::{DTO::TreeViewOptionsDTO::TreeViewOptionsDTO, TreeViewProvider::TreeViewProvider},
+};
 use async_trait::async_trait;
 use log::{info, warn};
 use serde_json::{Value, json};
@@ -18,7 +21,7 @@ impl TreeViewProvider for MountainEnvironment {
 	/// Registers a new tree data provider from Cocoon.
 	async fn RegisterTreeDataProvider(&self, ViewIdentifier:String, Options:Value) -> Result<(), CommonError> {
 		info!("[TreeViewProvider] Registering data provider for view: {}", ViewIdentifier);
-		let OptionsDTO:Common::TreeView::DTO::TreeViewOptionsDTO::TreeViewOptionsDTO = serde_json::from_value(Options)
+		let OptionsDTO:TreeViewOptionsDTO = serde_json::from_value(Options)
 			.map_err(|e| CommonError::InvalidArgument { ArgumentName:"Options".into(), Reason:e.to_string() })?;
 
 		let NewState = TreeViewStateDTO {

Source/Environment/WorkSpaceProvider.rs

@@ -132,14 +132,16 @@ impl WorkSpaceProvider for MountainEnvironment {
 					if Path.is_dir() {
 						continue;
 					}
-					if IncludeMatcher?.is_match(Path) {
-						if let Some(ref Exclude) = ExcludeMatcher {
-							if Exclude.is_match(Path) {
-								continue;
+					if let Some(ref include) = IncludeMatcher {
+						if include.is_match(Path) {
+							if let Some(ref exclude) = ExcludeMatcher {
+								if exclude.is_match(Path) {
+									continue;
+								}
+							}
+							if let Ok(URL) = Url::from_file_path(Path) {
+								Results.push(URL);
 							}
-						}
-						if let Ok(URL) = Url::from_file_path(Path) {
-							Results.push(URL);
 						}
 					}
 				}