refactor(Common): Enforce PascalCase naming convention across codebase

Align all trait method names, function names, and code examples with the PascalCase naming standard defined in CLAUDE.md. This includes:

- Rename trait methods: `read_file` → `ReadFile`, `write_file` → `WriteFile`, `get_configuration` → `GetConfiguration`, `update_configuration` → `UpdateConfiguration`, `get_workspace_folders` → `UpdateWorkspaceFolders`
- Rename effect constructors: `read_file_effect` → `ReadFile_effect`, `get_configuration_effect` → `GetConfiguration_effect`
- Reformat struct field declarations to remove spaces before colons (consistent with Rust style)
- Reorganize import statements to use grouped `super::{}` syntax
- Update doc comments with proper line wrapping for readability
- Fix code examples in DeepDive.md documentation to reflect new naming

These are purely cosmetic refactoring changes that improve code consistency without altering any runtime behavior.

Author: NikolaRHristov

Documentation/GitHub/DeepDive.md

@@ -206,13 +206,13 @@ graph LR
 // Service trait definition
 #[async_trait]
 pub trait FileSystemReader: Send + Sync {
-    async fn read_file(&self, path: &PathBuf) -> Result<Vec<u8>, CommonError>;
+    async fn ReadFile(&self, path: &PathBuf) -> Result<Vec<u8>, CommonError>;
 }
 
 // Effect requiring the trait
-pub fn read_file_effect(path: PathBuf) -> ActionEffect<Arc<dyn FileSystemReader>, CommonError, Vec<u8>> {
+pub fn ReadFile_effect(path: PathBuf) -> ActionEffect<Arc<dyn FileSystemReader>, CommonError, Vec<u8>> {
     ActionEffect::new(move |fs: Arc<dyn FileSystemReader>| {
-        Box::pin(async move { fs.read_file(&path).await })
+        Box::pin(async move { fs.ReadFile(&path).await })
     })
 }
 ```
@@ -333,7 +333,7 @@ sequenceDiagram
 #[async_trait]
 pub trait WorkspaceService: Send + Sync {
     async fn get_workspace_folders(&self) -> Result<Vec<WorkspaceFolder>, CommonError>;
-    async fn update_workspace_folders(&self, folders: Vec<WorkspaceFolder>) -> Result<(), CommonError>;
+    async fn UpdateWorkspaceFolders(&self, folders: Vec<WorkspaceFolder>) -> Result<(), CommonError>;
 }
 ```
 
@@ -368,15 +368,15 @@ pub struct WorkspaceFolderDTO {
 // VSCode file system service interface
 #[async_trait]
 pub trait FileSystemService: Send + Sync {
-    async fn read_file(&self, uri: &str) -> Result<Vec<u8>, CommonError>;
-    async fn write_file(&self, uri: &str, content: &[u8]) -> Result<(), CommonError>;
+    async fn ReadFile(&self, uri: &str) -> Result<Vec<u8>, CommonError>;
+    async fn WriteFile(&self, uri: &str, content: &[u8]) -> Result<(), CommonError>;
     async fn stat(&self, uri: &str) -> Result<FileStat, CommonError>;
 }
 
 // File system effect constructors
-pub fn read_file_effect(uri: String) -> ActionEffect<Arc<dyn FileSystemService>, CommonError, Vec<u8>> {
+pub fn ReadFile_effect(uri: String) -> ActionEffect<Arc<dyn FileSystemService>, CommonError, Vec<u8>> {
     ActionEffect::new(move |fs: Arc<dyn FileSystemService>| {
-        Box::pin(async move { fs.read_file(&uri).await })
+        Box::pin(async move { fs.ReadFile(&uri).await })
     })
 }
 ```
@@ -387,14 +387,14 @@ pub fn read_file_effect(uri: String) -> ActionEffect<Arc<dyn FileSystemService>,
 // VSCode configuration service interface
 #[async_trait]
 pub trait ConfigurationService: Send + Sync {
-    async fn get_configuration(&self, section: Option<&str>) -> Result<Value, CommonError>;
-    async fn update_configuration(&self, key: &str, value: Value) -> Result<(), CommonError>;
+    async fn GetConfiguration(&self, section: Option<&str>) -> Result<Value, CommonError>;
+    async fn UpdateConfiguration(&self, key: &str, value: Value) -> Result<(), CommonError>;
 }
 
 // Configuration effect constructors
-pub fn get_configuration_effect(section: Option<String>) -> ActionEffect<Arc<dyn ConfigurationService>, CommonError, Value> {
+pub fn GetConfiguration_effect(section: Option<String>) -> ActionEffect<Arc<dyn ConfigurationService>, CommonError, Value> {
     ActionEffect::new(move |config: Arc<dyn ConfigurationService>| {
-        Box::pin(async move { config.get_configuration(section.as_deref()).await })
+        Box::pin(async move { config.GetConfiguration(section.as_deref()).await })
     })
 }
 ```
@@ -527,12 +527,12 @@ use Common::Effect::ActionEffect;
 use std::sync::Arc;
 
 // Advanced effect composition
-let complex_effect = FileSystem::read_file(path.clone())
-    .and_then(|content| FileSystem::write_file(other_path, content))
+let complex_effect = FileSystem::ReadFile(path.clone())
+    .and_then(|content| FileSystem::WriteFile(other_path, content))
     .map(|_| println!("File operation completed successfully"));
 
 // Effect with custom error handling
-let resilient_effect = FileSystem::read_file(path)
+let resilient_effect = FileSystem::ReadFile(path)
     .recover_with(|error| {
         log::warn!("File read failed: {}", error);
         ActionEffect::pure(Vec::new()) // Fallback to empty content

Source/LanguageFeature/ProvideCallHierarchy.rs

@@ -1,7 +1,7 @@
 //! # ProvideCallHierarchy Effect
 //!
-//! Defines the `ActionEffect` for requesting call hierarchy incoming calls from a language feature
-//! provider.
+//! Defines the `ActionEffect` for requesting call hierarchy incoming calls from
+//! a language feature provider.
 
 use std::sync::Arc;
 
@@ -10,7 +10,8 @@ use serde_json::Value;
 use super::LanguageFeatureProviderRegistry::LanguageFeatureProviderRegistry;
 use crate::{Effect::ActionEffect::ActionEffect, Error::CommonError::CommonError};
 
-/// Creates an effect that, when executed, will request call hierarchy incoming calls.
+/// Creates an effect that, when executed, will request call hierarchy incoming
+/// calls.
 pub fn ProvideCallHierarchy(
 	ItemDTO:Value,
 ) -> ActionEffect<Arc<dyn LanguageFeatureProviderRegistry>, CommonError, Option<Value>> {

Source/LanguageFeature/ProvideCodeActions.rs

@@ -1,7 +1,7 @@
 //! # ProvideCodeActions Effect
 //!
-//! Defines the `ActionEffect` for requesting code actions from a language feature
-//! provider.
+//! Defines the `ActionEffect` for requesting code actions from a language
+//! feature provider.
 
 use std::sync::Arc;
 
@@ -24,6 +24,10 @@ pub fn ProvideCodeActions(
 		let RangeOrSelectionDTOClone = RangeOrSelectionDTO.clone();
 		let ContextDTOClone = ContextDTO.clone();
 
-		Box::pin(async move { Registry.ProvideCodeActions(DocumentURIClone, RangeOrSelectionDTOClone, ContextDTOClone).await })
+		Box::pin(async move {
+			Registry
+				.ProvideCodeActions(DocumentURIClone, RangeOrSelectionDTOClone, ContextDTOClone)
+				.await
+		})
 	}))
 }

Source/LanguageFeature/ProvideCodeLenses.rs

@@ -1,7 +1,7 @@
 //! # ProvideCodeLenses Effect
 //!
-//! Defines the `ActionEffect` for requesting code lenses from a language feature
-//! provider.
+//! Defines the `ActionEffect` for requesting code lenses from a language
+//! feature provider.
 
 use std::sync::Arc;
 

Source/LanguageFeature/ProvideDefinition.rs

@@ -1,15 +1,16 @@
 //! # ProvideDefinition Effect
 //!
-//! Defines the `ActionEffect` for requesting definition locations from a language feature
-//! provider.
+//! Defines the `ActionEffect` for requesting definition locations from a
+//! language feature provider.
 
 use std::sync::Arc;
 
-use super::DTO::LocationDTO::LocationDTO;
-use super::DTO::PositionDTO::PositionDTO;
 use url::Url;
 
-use super::LanguageFeatureProviderRegistry::LanguageFeatureProviderRegistry;
+use super::{
+	DTO::{LocationDTO::LocationDTO, PositionDTO::PositionDTO},
+	LanguageFeatureProviderRegistry::LanguageFeatureProviderRegistry,
+};
 use crate::{Effect::ActionEffect::ActionEffect, Error::CommonError::CommonError};
 
 /// Creates an effect that, when executed, will request definition locations.

Source/LanguageFeature/ProvideDocumentFormatting.rs

@@ -1,18 +1,18 @@
 //! # ProvideDocumentFormatting Effect
 //!
-//! Defines the `ActionEffect` for requesting document formatting edits from a language feature
-//! provider.
+//! Defines the `ActionEffect` for requesting document formatting edits from a
+//! language feature provider.
 
 use std::sync::Arc;
 
 use serde_json::Value;
-use super::DTO::TextEditDTO::TextEditDTO;
 use url::Url;
 
-use super::LanguageFeatureProviderRegistry::LanguageFeatureProviderRegistry;
+use super::{DTO::TextEditDTO::TextEditDTO, LanguageFeatureProviderRegistry::LanguageFeatureProviderRegistry};
 use crate::{Effect::ActionEffect::ActionEffect, Error::CommonError::CommonError};
 
-/// Creates an effect that, when executed, will request document formatting edits.
+/// Creates an effect that, when executed, will request document formatting
+/// edits.
 pub fn ProvideDocumentFormatting(
 	DocumentURI:Url,
 

Source/LanguageFeature/ProvideDocumentHighlights.rs

@@ -1,15 +1,14 @@
 //! # ProvideDocumentHighlights Effect
 //!
-//! Defines the `ActionEffect` for requesting document highlights from a language feature
-//! provider.
+//! Defines the `ActionEffect` for requesting document highlights from a
+//! language feature provider.
 
 use std::sync::Arc;
 
 use serde_json::Value;
-use super::DTO::PositionDTO::PositionDTO;
 use url::Url;
 
-use super::LanguageFeatureProviderRegistry::LanguageFeatureProviderRegistry;
+use super::{DTO::PositionDTO::PositionDTO, LanguageFeatureProviderRegistry::LanguageFeatureProviderRegistry};
 use crate::{Effect::ActionEffect::ActionEffect, Error::CommonError::CommonError};
 
 /// Creates an effect that, when executed, will request document highlights.

Source/LanguageFeature/ProvideDocumentSymbols.rs

@@ -1,7 +1,7 @@
 //! # ProvideDocumentSymbols Effect
 //!
-//! Defines the `ActionEffect` for requesting document symbols from a language feature
-//! provider.
+//! Defines the `ActionEffect` for requesting document symbols from a language
+//! feature provider.
 
 use std::sync::Arc;
 

Source/LanguageFeature/ProvideFoldingRanges.rs

@@ -1,7 +1,7 @@
 //! # ProvideFoldingRanges Effect
 //!
-//! Defines the `ActionEffect` for requesting folding ranges from a language feature
-//! provider.
+//! Defines the `ActionEffect` for requesting folding ranges from a language
+//! feature provider.
 
 use std::sync::Arc;
 

Source/LanguageFeature/ProvideInlayHints.rs

@@ -1,7 +1,7 @@
 //! # ProvideInlayHints Effect
 //!
-//! Defines the `ActionEffect` for requesting inlay hints from a language feature
-//! provider.
+//! Defines the `ActionEffect` for requesting inlay hints from a language
+//! feature provider.
 
 use std::sync::Arc;