refactor(Mountain): Align module paths and refine workspace search

- Updated ApplicationState to use concrete `Wry` runtime type for command handlers instead of trait objects, improving type safety and aligning with Tauri best practices
- Corrected import paths for DTOs (`MarkerDataDTO`, `WorkSpaceEditDTO`) to match finalized module structure after Common crate reorganization
- Fixed workspace file search logic by properly handling `IncludeMatcher` as Option in WorkSpaceProvider, preventing potential panics during workspace scans
- Removed obsolete TODO comment after completing path refactoring in ApplicationState

These changes strengthen Mountain's implementation of the `Common` crate abstractions (WorkSpaceProvider, DiagnosticManager) and ensure consistency with the finalized architecture. The workspace search fix specifically enhances the 'Opening Files' workflow reliability.

Author: NikolaRHristov

Source/ApplicationState/ApplicationState.rs

@@ -16,7 +16,7 @@ use std::{
 };
 
 use log::{error, info, warn};
-use tauri::{Manager, Runtime};
+use tauri::{Manager, Wry};
 
 use super::{
 	DTO::{
@@ -35,7 +35,7 @@ use super::{
 	},
 	Internal,
 };
-use crate::Environment::CommandProvider::CommandHandler; // TODO: Fix this path after refactor
+use crate::Environment::CommandProvider::CommandHandler;
 
 /// The central, shared, thread-safe state for the entire Mountain application.
 ///
@@ -58,7 +58,7 @@ pub struct ApplicationState {
 	pub WorkSpaceMementoPath:Arc<StandardMutex<Option<PathBuf>>>,
 
 	// --- Extension & Provider Management ---
-	pub CommandRegistry:Arc<StandardMutex<HashMap<String, CommandHandler<dyn Runtime>>>>,
+	pub CommandRegistry:Arc<StandardMutex<HashMap<String, CommandHandler<Wry>>>>,
 	pub LanguageProviders:Arc<StandardMutex<HashMap<u32, ProviderRegistrationDTO>>>,
 	pub NextProviderHandle:Arc<AtomicU32>,
 	pub ScannedExtensions:Arc<StandardMutex<HashMap<String, ExtensionDescriptionStateDTO>>>,

Source/Environment/DiagnosticProvider.rs

@@ -5,17 +5,14 @@
 //! including storing diagnostics from various sources and notifying the UI of
 //! changes.
 
-use Common::{
-	Diagnostic::DiagnosticManager::DiagnosticManager,
-	Error::CommonError::CommonError,
-	LanguageFeature::DTO::MarkerDataDTO,
-};
+use Common::{Diagnostic::DiagnosticManager::DiagnosticManager, Error::CommonError::CommonError};
 use async_trait::async_trait;
 use log::{debug, error, info};
 use serde_json::{Value, json};
 use tauri::Emitter;
 
 use super::{MountainEnvironment::MountainEnvironment, Utility};
+use crate::ApplicationState::DTO::MarkerDataDTO::MarkerDataDTO;
 
 #[async_trait]
 impl DiagnosticManager for MountainEnvironment {

Source/Environment/WorkSpaceProvider.rs

@@ -8,9 +8,9 @@
 use std::path::PathBuf;
 
 use Common::{
+	DTO::WorkSpaceEditDTO::WorkSpaceEditDTO,
 	Error::CommonError::CommonError,
-	LanguageFeature::DTO::WorkSpaceEditDTO,
-	WorkSpace::{WorkSpaceEditApplier, WorkSpaceProvider},
+	WorkSpace::{WorkSpaceEditApplier::WorkSpaceEditApplier, WorkSpaceProvider::WorkSpaceProvider},
 };
 use async_trait::async_trait;
 use globset::{Glob, GlobMatcher};
@@ -132,7 +132,7 @@ impl WorkSpaceProvider for MountainEnvironment {
 					if Path.is_dir() {
 						continue;
 					}
-					if IncludeMatcher.is_match(Path) {
+					if IncludeMatcher?.is_match(Path) {
 						if let Some(ref Exclude) = ExcludeMatcher {
 							if Exclude.is_match(Path) {
 								continue;