Author: NikolaRHristov

README.md

@@ -59,8 +59,8 @@ not limited to):**
 - **Denial of Service (DoS):** If an Element can be made unresponsive or crash
   due to specially crafted input or excessive resource consumption.
 - **Path Traversal/Arbitrary File Access:** For Elements interacting with the
-  filesystem (like `Mountain`'s FS handlers), ensuring that
-  path inputs are properly sanitized.
+  filesystem (like `Mountain`'s FS handlers), ensuring that path inputs are
+  properly sanitized.
 - **Insecure IPC/Communication:** For Elements involved in inter-process
   communication (`Vine`, `Track`, `Echo`, `Mist`), vulnerabilities in the
   protocol or handling of messages.

SECURITY.md

@@ -242,49 +242,56 @@ impl CompletionItem {
 /// Completion item kind
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash, Serialize, Deserialize)]
 pub enum CompletionItemKind {
-	/// Text completion
-	Text = 1,
-	/// Method completion
-	Method = 2,
-	/// Function completion
-	Function = 3,
-	/// Constructor completion
-	Constructor = 4,
-	/// Field completion
-	Field = 5,
-	/// Variable completion
-	Variable = 6,
-	/// Class completion
-	Class = 7,
-	/// Interface completion
-	Interface = 8,
-	/// Module completion
-	Module = 9,
-	/// Property completion
-	Property = 10,
-	/// Unit completion
-	Unit = 11,
-	/// Value completion
-	Value = 12,
-	/// Enum completion
-	Enum = 13,
-	/// Keyword completion
-	Keyword = 14,
-	/// Snippet completion
-	Snippet = 15,
-	/// Color completion
-	Color = 16,
-	/// File completion
-	File = 17,
-	/// Reference completion
-	Reference = 18,
-	Folder = 19,
-	EnumMember = 20,
-	Constant = 21,
-	Struct = 22,
-	Event = 23,
-	Operator = 24,
-	TypeParameter = 25,
+/// Text completion
+Text = 1,
+/// Method completion
+Method = 2,
+/// Function completion
+Function = 3,
+/// Constructor completion
+Constructor = 4,
+/// Field completion
+Field = 5,
+/// Variable completion
+Variable = 6,
+/// Class completion
+Class = 7,
+/// Interface completion
+Interface = 8,
+/// Module completion
+Module = 9,
+/// Property completion
+Property = 10,
+/// Unit completion
+Unit = 11,
+/// Value completion
+Value = 12,
+/// Enum completion
+Enum = 13,
+/// Keyword completion
+Keyword = 14,
+/// Snippet completion
+Snippet = 15,
+/// Color completion
+Color = 16,
+/// File completion
+File = 17,
+/// Reference completion
+Reference = 18,
+/// Folder completion
+Folder = 19,
+/// Enum member completion
+EnumMember = 20,
+/// Constant completion
+Constant = 21,
+/// Struct completion
+Struct = 22,
+/// Event completion
+Event = 23,
+/// Operator completion
+Operator = 24,
+/// Type parameter completion
+TypeParameter = 25,
 }
 
 /// Completion item documentation

Source/API/types.rs

@@ -50,7 +50,8 @@ impl Default for VSCodeAPI {
 pub struct Commands;
 
 impl Commands {
-	pub fn new() -> Self { Self }
+/// Create a new Commands instance
+pub fn new() -> Self { Self }
 
 	/// Register a command
 	pub fn register_command(&self, command_id:String, callback:CommandCallback) -> Result<Command, String> {
@@ -74,15 +75,17 @@ pub type CommandCallback = Box<dyn Fn(Vec<serde_json::Value>) -> Result<serde_js
 /// Command representation
 #[derive(Debug, Clone)]
 pub struct Command {
-	pub id:String,
+/// The unique identifier of the command
+pub id:String,
 }
 
 /// Window namespace
 #[derive(Debug, Clone)]
 pub struct Window;
 
 impl Window {
-	pub fn new() -> Self { Self }
+/// Create a new Window instance
+pub fn new() -> Self { Self }
 
 	/// Show an information message
 	pub async fn show_information_message(&self, message:String) -> Result<String, String> {
@@ -109,11 +112,17 @@ impl Window {
 /// Output channel for logging
 #[derive(Debug, Clone)]
 pub struct OutputChannel {
-	name:String,
+/// The name of the output channel
+name:String,
 }
 
 impl OutputChannel {
-	pub fn new(name:String) -> Self { Self { name } }
+/// Create a new output channel
+///
+/// # Arguments
+///
+/// * `name` - The name of the output channel
+pub fn new(name:String) -> Self { Self { name } }
 
 	/// Append a line to the channel
 	pub fn append_line(&self, line:&str) {
@@ -146,7 +155,8 @@ impl OutputChannel {
 pub struct Workspace;
 
 impl Workspace {
-	pub fn new() -> Self { Self }
+/// Create a new Workspace instance
+pub fn new() -> Self { Self }
 
 	/// Get workspace folders
 	pub fn workspace_folders(&self) -> Vec<WorkspaceFolder> {
@@ -176,11 +186,17 @@ pub struct WorkspaceFolder {
 /// Workspace configuration
 #[derive(Debug, Clone)]
 pub struct WorkspaceConfiguration {
-	section:Option<String>,
+/// The configuration section name
+section:Option<String>,
 }
 
 impl WorkspaceConfiguration {
-	pub fn new(section:Option<String>) -> Self { Self { section } }
+/// Create a new workspace configuration
+///
+/// # Arguments
+///
+/// * `section` - Optional section name to retrieve
+pub fn new(section:Option<String>) -> Self { Self { section } }
 
 	/// Get a configuration value
 	pub fn get<T:serde::de::DeserializeOwned>(&self, key:String) -> Result<T, String> {
@@ -203,7 +219,8 @@ impl WorkspaceConfiguration {
 pub struct Languages;
 
 impl Languages {
-	pub fn new() -> Self { Self }
+/// Create a new Languages instance
+pub fn new() -> Self { Self }
 
 	/// Register completion item provider
 	pub async fn register_completion_item_provider<T:CompletionItemProvider>(
@@ -239,13 +256,25 @@ pub type DocumentSelector = Vec<DocumentFilter>;
 
 /// Completion item provider
 pub trait CompletionItemProvider: Send + Sync {
-	fn provide_completion_items(
-		&self,
-		document:TextDocumentIdentifier,
-		position:Position,
-		context:CompletionContext,
-		token:Option<String>,
-	) -> Vec<CompletionItem>;
+/// Provide completion items at the given position
+///
+/// # Arguments
+///
+/// * `document` - The text document identifier
+/// * `position` - The position in the document
+/// * `context` - The completion context
+/// * `token` - Optional cancellation token
+///
+/// # Returns
+///
+/// A vector of completion items
+fn provide_completion_items(
+&self,
+document:TextDocumentIdentifier,
+position:Position,
+context:CompletionContext,
+token:Option<String>,
+) -> Vec<CompletionItem>;
 }
 
 /// Completion context
@@ -279,11 +308,17 @@ pub enum CompletionTriggerKind {
 /// Diagnostic collection
 #[derive(Debug, Clone)]
 pub struct DiagnosticCollection {
-	name:Option<String>,
+/// The name of the diagnostic collection
+name:Option<String>,
 }
 
 impl DiagnosticCollection {
-	pub fn new(name:Option<String>) -> Self { Self { name } }
+/// Create a new diagnostic collection
+///
+/// # Arguments
+///
+/// * `name` - Optional name for the collection
+pub fn new(name:Option<String>) -> Self { Self { name } }
 
 	/// Set diagnostics for a resource
 	pub fn set(&self, uri:String, diagnostics:Vec<Diagnostic>) {
@@ -311,19 +346,22 @@ impl DiagnosticCollection {
 pub struct Disposable;
 
 impl Disposable {
-	pub fn new() -> Self { Self }
+/// Create a new disposable item
+pub fn new() -> Self { Self }
 
-	pub fn dispose(&self) {
-		// Placeholder implementation
-	}
+/// Dispose the resource
+pub fn dispose(&self) {
+// Placeholder implementation
+}
 }
 
 /// Extensions namespace
 #[derive(Debug, Clone)]
 pub struct Extensions;
 
 impl Extensions {
-	pub fn new() -> Self { Self }
+/// Create a new Extensions instance
+pub fn new() -> Self { Self }
 
 	/// Get all extensions
 	pub fn all(&self) -> Vec<Extension> { Vec::new() }
@@ -355,7 +393,8 @@ pub struct Extension {
 pub struct Env;
 
 impl Env {
-	pub fn new() -> Self { Self }
+/// Create a new Env instance
+pub fn new() -> Self { Self }
 
 	/// Get environment variable
 	pub fn get_env_var(&self, name:String) -> Option<String> { std::env::var(name).ok() }