Author: NikolaRHristov

README.md

@@ -83,7 +83,8 @@ _returns a description of that effect_.
 
 ```rust
 async fn read_my_file(fs: &impl FileSystem) -> Result<Vec<u8>, Error> {
-    fs.read("/path/to/file").await // The side effect happens here.
+// The side effect happens here.
+    fs.read("/path/to/file").await
 }
 ```
 
@@ -197,23 +198,27 @@ follows:
     concrete implementation for a `Common` trait.
 
     ```rust
-    // In Mountain/Source/Environment/FileSystemProvider.rs
+// In Mountain/Source/Environment/FileSystemProvider.rs
+   
     use Common::FileSystem::{FileSystemReader, FileSystemWriter};
 
     #[async_trait]
     impl FileSystemReader for MountainEnvironment {
         async fn ReadFile(&self, Path: &PathBuf) -> Result<Vec<u8>, CommonError> {
-            // ... actual `tokio::fs` call ...
+// ... actual `tokio::fs` call ...
+           
         }
-        // ...
+// ...
+       
     }
     ```
 
 2.  **Create and Execute an Effect:** In business logic, create and run an
     effect.
 
     ```rust
-    // In a Mountain service or command
+// In a Mountain service or command
+   
     use Common::FileSystem;
     use Common::Effect::ApplicationRunTime;
 

Source/Command/ExecuteCommand.rs

@@ -19,6 +19,7 @@ use crate::{Effect::ActionEffect::ActionEffect, Error::CommonError::CommonError}
 /// # Parameters
 ///
 /// * `CommandIdentifier`: The unique ID of the command to execute (e.g.,
+
 ///   "FileSystem.ReadFile").
 /// * `Argument`: A `serde_json::Value` containing the arguments for the
 ///   command.
@@ -30,11 +31,14 @@ use crate::{Effect::ActionEffect::ActionEffect, Error::CommonError::CommonError}
 /// during execution.
 pub fn ExecuteCommand(
 	CommandIdentifier:String,
+
 	Argument:Value,
 ) -> ActionEffect<Arc<dyn CommandExecutor>, CommonError, Value> {
 	ActionEffect::New(Arc::new(move |Executor:Arc<dyn CommandExecutor>| {
 		let CommandIdentifierClone = CommandIdentifier.clone();
+
 		let ArgumentClone = Argument.clone();
+
 		Box::pin(async move { Executor.ExecuteCommand(CommandIdentifierClone, ArgumentClone).await })
 	}))
 }

Source/Command/RegisterCommand.rs

@@ -21,18 +21,22 @@ use crate::{Effect::ActionEffect::ActionEffect, Error::CommonError::CommonError}
 /// * `SidecarIdentifier`: The unique ID of the sidecar where the command logic
 ///   resides.
 /// * `CommandIdentifier`: The unique ID of the command itself (e.g.,
+
 ///   "MyExtension.DoSomething").
 ///
 /// # Returns
 ///
 /// An `ActionEffect` that resolves to `()` on success.
 pub fn RegisterCommand(
 	SidecarIdentifier:String,
+
 	CommandIdentifier:String,
 ) -> ActionEffect<Arc<dyn CommandExecutor>, CommonError, ()> {
 	ActionEffect::New(Arc::new(move |Executor:Arc<dyn CommandExecutor>| {
 		let SidecarIdentifierClone = SidecarIdentifier.clone();
+
 		let CommandIdentifierClone = CommandIdentifier.clone();
+
 		Box::pin(async move { Executor.RegisterCommand(SidecarIdentifierClone, CommandIdentifierClone).await })
 	}))
 }

Source/Command/UnregisterCommand.rs

@@ -25,11 +25,14 @@ use crate::{Effect::ActionEffect::ActionEffect, Error::CommonError::CommonError}
 /// An `ActionEffect` that resolves to `()` on success.
 pub fn UnregisterCommand(
 	SidecarIdentifier:String,
+
 	CommandIdentifier:String,
 ) -> ActionEffect<Arc<dyn CommandExecutor>, CommonError, ()> {
 	ActionEffect::New(Arc::new(move |Executor:Arc<dyn CommandExecutor>| {
 		let SidecarIdentifierClone = SidecarIdentifier.clone();
+
 		let CommandIdentifierClone = CommandIdentifier.clone();
+
 		Box::pin(async move { Executor.UnregisterCommand(SidecarIdentifierClone, CommandIdentifierClone).await })
 	}))
 }

Source/Command/mod.rs

@@ -12,6 +12,9 @@ pub mod CommandExecutor;
 
 // --- Effect Constructors ---
 pub mod ExecuteCommand;
+
 pub mod GetAllCommands;
+
 pub mod RegisterCommand;
+
 pub mod UnregisterCommand;

Source/Configuration/ConfigurationInspector.rs

@@ -32,7 +32,9 @@ pub trait ConfigurationInspector: Environment + Send + Sync {
 	/// `None` if the key is not found in any configuration source.
 	async fn InspectConfigurationValue(
 		&self,
+
 		Key:String,
+
 		Overrides:ConfigurationOverridesDTO,
 	) -> Result<Option<InspectResultDataDTO>, CommonError>;
 }

Source/Configuration/ConfigurationProvider.rs

@@ -37,7 +37,9 @@ pub trait ConfigurationProvider: Environment + Send + Sync {
 	/// `serde_json::Value`.
 	async fn GetConfigurationValue(
 		&self,
+
 		Section:Option<String>,
+
 		Overrides:ConfigurationOverridesDTO,
 	) -> Result<Value, CommonError>;
 
@@ -54,10 +56,15 @@ pub trait ConfigurationProvider: Environment + Send + Sync {
 	///   settings.
 	async fn UpdateConfigurationValue(
 		&self,
+
 		Key:String,
+
 		ValueToSet:Value,
+
 		Target:ConfigurationTarget,
+
 		Overrides:ConfigurationOverridesDTO,
+
 		ScopeToLanguage:Option<bool>,
 	) -> Result<(), CommonError>;
 }

Source/Configuration/DTO/ConfigurationScope.rs

@@ -16,18 +16,23 @@ pub enum ConfigurationScope {
 	/// Application-specific configuration, which can only be configured in user
 	/// settings.
 	Application = 1,
+
 	/// Machine-specific configuration, which can only be configured in user
 	/// settings and is not synced.
 	Machine = 2,
+
 	/// Window-specific configuration, which can be configured in user or
 	/// workspace settings.
 	Window = 3,
+
 	/// Resource-specific configuration, which can be configured in all settings
 	/// levels (user, workspace, folder).
 	Resource = 4,
+
 	/// Language-specific configuration, which can be configured in all settings
 	/// levels and can be overridden on a per-language basis.
 	LanguageDefined = 5,
+
 	/// Machine-specific configuration that can be overridden by workspace
 	/// settings.
 	MachineOverridable = 6,

Source/Configuration/DTO/ConfigurationTarget.rs

@@ -15,16 +15,22 @@ use serde::{Deserialize, Serialize};
 pub enum ConfigurationTarget {
 	/// Target the user settings file for the local machine.
 	UserLocal = 1,
+
 	/// Target the user settings, potentially synced across machines.
 	User = 2,
+
 	/// Target the workspace settings file (e.g., `.vscode/settings.json`).
 	WorkSpace = 3,
+
 	/// Target a specific folder's settings in a multi-root workspace.
 	WorkSpaceFolder = 4,
+
 	/// Target the default values (typically a read-only operation).
 	Default = 5,
+
 	/// Target the in-memory configuration for the current session only.
 	Memory = 6,
+
 	/// Target the policy-enforced configuration (read-only).
 	Policy = 7,
 }

Source/Configuration/DTO/InspectResultDataDTO.rs

@@ -17,31 +17,46 @@ use serde_json::Value;
 pub struct InspectResultDataDTO {
 	/// The value from the default configuration.
 	pub DefaultValue:Option<Value>,
+
 	/// The value from the user (global) settings.
 	pub UserValue:Option<Value>,
+
 	/// The value from the local (un-synced) user settings.
 	pub UserLocalValue:Option<Value>,
+
 	/// The value from the remote user settings (if applicable).
 	pub UserRemoteValue:Option<Value>,
+
 	/// The value from the workspace settings.
 	pub WorkSpaceValue:Option<Value>,
+
 	/// The value from a specific workspace folder's settings.
 	pub WorkSpaceFolderValue:Option<Value>,
+
 	/// The value from the in-memory configuration.
 	pub MemoryValue:Option<Value>,
+
 	/// The value from the policy-enforced configuration.
 	pub PolicyValue:Option<Value>,
+
 	/// The final, effective value after all scopes have been merged.
 	pub EffectiveValue:Option<Value>,
 
 	// --- Language-Specific Overrides ---
 	pub DefaultLanguageValue:Option<Value>,
+
 	pub UserLanguageValue:Option<Value>,
+
 	pub UserLocalLanguageValue:Option<Value>,
+
 	pub UserRemoteLanguageValue:Option<Value>,
+
 	pub WorkSpaceLanguageValue:Option<Value>,
+
 	pub WorkSpaceFolderLanguageValue:Option<Value>,
+
 	pub MemoryLanguageValue:Option<Value>,
+
 	pub PolicyLanguageValue:Option<Value>,
 
 	/// A list of language identifiers for which language-specific values were