refactor(Mountain): Standardize sidecar naming to SideCar and implement Binary staging

- Renamed `SidecarIdentifier` to `SideCarIdentifier` in all DTOs (CustomDocumentStateDTO, ProviderRegistrationDTO, WebViewStateDTO) and provider implementations (CommandProvider, DebugProvider, DocumentProvider, IPCProvider, LanguageFeatureProvider, StatusBarProvider, TerminalProvider, WebViewProvider) to ensure consistent naming across the Mountain backend
- Updated README.md diagram to change "CocoonSidecar" to "CocoonSideCar"
- Added new `Binary/` directory with README.md explaining its role in staging sidecar executables during build, and .gitignore to exclude contents from version control
- Regenerated gRPC code (vine_ipc.rs) to reflect naming changes in protocol
- Deleted build artifacts (executables/installers) as they should not be version controlled

This refactoring enhances consistency in the IPC layer (Vine) and state management (AppState), while the Binary staging directory streamlines Tauri bundling by providing a transient location for sidecar executables, as described in the project's build strategy. The changes maintain compatibility with existing workflows like extension registration and command execution.

Author: NikolaRHristov

Binary/.gitignore

@@ -0,0 +1,3 @@
+/*
+!/.gitkeep
+!/README.md

Binary/.gitkeep

@@ -0,0 +1,54 @@
+<table><tr>
+<td colspan="1"> <h3 align="center"> <picture>
+<source media="(prefers-color-scheme: dark)" srcset="https://PlayForm.Cloud/Dark/Image/GitHub/Land.svg">
+<source media="(prefers-color-scheme: light)" srcset="https://PlayForm.Cloud/Image/GitHub/Land.svg">
+<img width="28" alt="Land Logo" src="https://PlayForm.Cloud/Image/GitHub/Land.svg">
+</picture> </h3> </td> <td colspan="3" valign="top"> <h3 align="center"> Mountain Binaries 📦
+</h3> </td>
+</tr></table>
+
+---
+
+# **Mountain Binaries** 📦 Staged Executables for Bundling
+
+This directory, `Binary/`, serves as a temporary staging area for executables
+that are dynamically selected and prepared by the `Build.rs` orchestrator. It is
+essential for managing different application "flavours" that bundle specific
+sidecar runtimes.
+
+## Purpose and Workflow
+
+The Land Code Editor is designed to support different underlying runtimes for
+its extension host, such as various versions of Node.js. It would be inefficient
+and create bloated installers to bundle every possible runtime into a single
+application.
+
+To solve this, the `Binary/` directory is used as part of a "just-in-time"
+bundling process:
+
+1.  **Selection:** During the build process, the `Build.rs` script determines
+    which specific sidecar version is needed (e.g., `Node.js v22`) based on the
+    build arguments (`--node-version`).
+2.  **Staging:** The script copies the selected executable from the main
+    `SideCar/` repository (where all versions are stored) into this `Binary/`
+    directory. The copied file is always given a consistent, predictable name
+    (e.g., `node.exe` on Windows or `node` on Unix).
+3.  **Bundling:** The `tauri.conf.json` is dynamically configured to point its
+    `bundle.externalBin` to this staged file (e.g., `bin/node`). The Tauri
+    bundler then includes this single executable in the final application
+    installer.
+4.  **Cleanup:** After the Tauri build is complete, the `Build.rs` script
+    automatically cleans up this directory, removing the temporary binary.
+
+This ensures that the final application installer is lean and contains only the
+one required sidecar runtime, while the source code remains clean and doesn't
+need to be polluted with temporary files.
+
+> [!IMPORTANT] This directory is managed automatically by the build system. Its
+> contents are transient and **should not be committed to version control**. It
+> should be included in your project's `.gitignore` file.
+
+---
+
+**Parent Project**: [`Mountain`](https://github.com/CodeEditorLand/Mountain/tree/Current/README.md) | **Related Directory**:
+[`SideCar`](https://github.com/CodeEditorLand/SideCar/tree/Current/README.md)

Binary/README.md

@@ -111,14 +111,14 @@ graph LR
 
     subgraph "Clients"
         WindUI["Wind/Sky User Interface (WebView)"]:::wind
-        CocoonSidecar["Cocoon Extension Host (Node.js)"]:::cocoon
+        CocoonSideCar["Cocoon Extension Host (Node.js)"]:::cocoon
     end
 
     TauriRuntime -- Hosts --> WindUI
     WindUI -- Tauri Command --> TrackDispatcher
     TrackDispatcher -- Tauri Events --> WindUI
 
-    VinegRPC -- gRPC Protocol <--> CocoonSidecar; class VinegRPC,CocoonSidecar IPC
+    VinegRPC -- gRPC Protocol <--> CocoonSideCar; class VinegRPC,CocoonSideCar IPC
     VinegRPC -- Forwards requests to --> TrackDispatcher
 
     NativeHandlers -- Implements traits from --> CommonCrate

README.md

@@ -27,7 +27,7 @@ pub struct CustomDocumentStateDTO {
 
 	/// The identifier of the sidecar process where the custom editor provider
 	/// lives.
-	pub SidecarIdentifier:String,
+	pub SideCarIdentifier:String,
 
 	/// A flag indicating if the document is currently editable by the user.
 	pub IsEditable:bool,

Source/ApplicationState/DTO/CustomDocumentStateDTO.rs

@@ -26,7 +26,7 @@ pub struct ProviderRegistrationDTO {
 	pub Selector:Value,
 
 	/// The identifier of the sidecar process that hosts this provider's logic.
-	pub SidecarIdentifier:String,
+	pub SideCarIdentifier:String,
 
 	/// The identifier of the extension that contributed this provider.
 	pub ExtensionIdentifier:Value,

Source/ApplicationState/DTO/ProviderRegistrationDTO.rs

@@ -33,7 +33,7 @@ pub struct WebViewStateDTO {
 	pub PanelOptions: Value,
 
 	/// The identifier of the sidecar process that owns this WebView.
-	pub SidecarIdentifier:String,
+	pub SideCarIdentifier:String,
 
 	/// The identifier of the extension that owns this WebView.
 	pub ExtensionIdentifier:String,

Source/ApplicationState/DTO/WebViewStateDTO.rs

@@ -43,17 +43,17 @@ pub enum CommandHandler<R:Runtime + 'static> {
 	),
 
 	/// A command implemented in an extension and proxied to a sidecar.
-	Proxied { SidecarIdentifier:String, CommandIdentifier:String },
+	Proxied { SideCarIdentifier:String, CommandIdentifier:String },
 }
 
 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, CommandIdentifier } => {
 				Self::Proxied {
-					SidecarIdentifier:SidecarIdentifier.clone(),
+					SideCarIdentifier:SideCarIdentifier.clone(),
 
 					CommandIdentifier:CommandIdentifier.clone(),
 				}
@@ -95,17 +95,17 @@ impl CommandExecutor for MountainEnvironment {
 					})
 			},
 
-			Some(CommandHandler::Proxied { SidecarIdentifier, CommandIdentifier: ProxiedCommandIdentifier }) => {
+			Some(CommandHandler::Proxied { SideCarIdentifier, CommandIdentifier: ProxiedCommandIdentifier }) => {
 				debug!(
 					"[CommandProvider] Executing PROXIED command '{}' on sidecar '{}'.",
-					CommandIdentifier, SidecarIdentifier
+					CommandIdentifier, SideCarIdentifier
 				);
 
 				let RPCParameters = json!([ProxiedCommandIdentifier, Argument]);
 
 				let RPCMethod = format!("{}$ExecuteContributedCommand", ProxyTarget::ExtHostCommands.GetTargetPrefix());
 
-				Client::SendRequest(&SidecarIdentifier, RPCMethod, RPCParameters, 30000)
+				Client::SendRequest(&SideCarIdentifier, RPCMethod, RPCParameters, 30000)
 					.await
 					.map_err(|e| CommonError::IPCError { Description:e.to_string() })
 			},
@@ -119,10 +119,10 @@ impl CommandExecutor for MountainEnvironment {
 	}
 
 	/// Registers a command contributed by a sidecar process.
-	async fn RegisterCommand(&self, SidecarIdentifier:String, CommandIdentifier:String) -> Result<(), CommonError> {
+	async fn RegisterCommand(&self, SideCarIdentifier:String, CommandIdentifier:String) -> Result<(), CommonError> {
 		info!(
 			"[CommandProvider] Registering PROXY command '{}' from sidecar '{}'",
-			CommandIdentifier, SidecarIdentifier
+			CommandIdentifier, SideCarIdentifier
 		);
 
 		let mut Registry = self
@@ -133,14 +133,14 @@ impl CommandExecutor for MountainEnvironment {
 
 		Registry.insert(
 			CommandIdentifier.clone(),
-			CommandHandler::Proxied { SidecarIdentifier, CommandIdentifier },
+			CommandHandler::Proxied { SideCarIdentifier, CommandIdentifier },
 		);
 
 		Ok(())
 	}
 
 	/// Unregisters a previously registered command.
-	async fn UnregisterCommand(&self, _SidecarIdentifier:String, CommandIdentifier:String) -> Result<(), CommonError> {
+	async fn UnregisterCommand(&self, _SideCarIdentifier:String, CommandIdentifier:String) -> Result<(), CommonError> {
 		info!("[CommandProvider] Unregistering command '{}'", CommandIdentifier);
 
 		self.ApplicationState

Source/Environment/CommandProvider.rs

@@ -42,7 +42,7 @@ impl DebugService for MountainEnvironment {
 
 		_ProviderHandle:u32,
 
-		_SidecarIdentifier:String,
+		_SideCarIdentifier:String,
 	) -> Result<(), CommonError> {
 		// TODO: Store this registration in ApplicationState
 		info!(
@@ -60,7 +60,7 @@ impl DebugService for MountainEnvironment {
 
 		_FactoryHandle:u32,
 
-		_SidecarIdentifier:String,
+		_SideCarIdentifier:String,
 	) -> Result<(), CommonError> {
 		// TODO: Store this registration in ApplicationState
 		info!(
@@ -87,7 +87,7 @@ impl DebugService for MountainEnvironment {
 		info!("[DebugProvider] Resolving debug configuration...");
 
 		let ResolvedConfig = IPCProvider
-			.SendRequestToSidecar(
+			.SendRequestToSideCar(
 				"cocoon-main".into(),
 				"$resolveDebugConfiguration".into(),
 				json!([DebugType, Configuration]),
@@ -99,7 +99,7 @@ impl DebugService for MountainEnvironment {
 		info!("[DebugProvider] Creating debug adapter descriptor...");
 
 		let Descriptor = IPCProvider
-			.SendRequestToSidecar(
+			.SendRequestToSideCar(
 				"cocoon-main".into(),
 				"$createDebugAdapterDescriptor".into(),
 				json!([DebugType, &ResolvedConfig]),

Source/Environment/DebugProvider.rs

@@ -94,7 +94,7 @@ impl DocumentProvider for MountainEnvironment {
 			let IPCProvider:Arc<dyn IPCProvider> = self.Require();
 
 			let RpcResult = IPCProvider
-				.SendRequestToSidecar(
+				.SendRequestToSideCar(
 					// In a multi-host world, we'd look this up
 					"cocoon-main".to_string(),
 					"$provideTextDocumentContent".to_string(),
@@ -305,7 +305,7 @@ async fn NotifyModelAdded(Environment:&MountainEnvironment, DocumentStateDTO:&Va
 	let IPCProvider:Arc<dyn IPCProvider> = Environment.Require();
 
 	if let Err(e) = IPCProvider
-		.SendNotificationToSidecar("cocoon-main".to_string(), "$acceptModelAdded".to_string(), Payload)
+		.SendNotificationToSideCar("cocoon-main".to_string(), "$acceptModelAdded".to_string(), Payload)
 		.await
 	{
 		error!("[DocumentProvider] Failed to send $acceptModelAdded for {}: {}", URIString, e);
@@ -325,7 +325,7 @@ async fn NotifyModelChanged(Environment:&MountainEnvironment, URI:&Url, NewVersi
 	let IPCProvider:Arc<dyn IPCProvider> = Environment.Require();
 
 	if let Err(e) = IPCProvider
-		.SendNotificationToSidecar("cocoon-main".to_string(), "$acceptModelChanged".to_string(), Payload)
+		.SendNotificationToSideCar("cocoon-main".to_string(), "$acceptModelChanged".to_string(), Payload)
 		.await
 	{
 		error!("[DocumentProvider] Failed to send $acceptModelChanged for {}: {}", URI, e);
@@ -343,7 +343,7 @@ async fn NotifyModelSaved(Environment:&MountainEnvironment, URI:&Url) {
 	let IPCProvider:Arc<dyn IPCProvider> = Environment.Require();
 
 	if let Err(e) = IPCProvider
-		.SendNotificationToSidecar("cocoon-main".to_string(), "$acceptModelSaved".to_string(), Payload)
+		.SendNotificationToSideCar("cocoon-main".to_string(), "$acceptModelSaved".to_string(), Payload)
 		.await
 	{
 		error!("[DocumentProvider] Failed to send $acceptModelSaved for {}: {}", URI, e);
@@ -361,7 +361,7 @@ pub async fn NotifyModelRemoved(Environment:&MountainEnvironment, URI:&Url) {
 	let IPCProvider:Arc<dyn IPCProvider> = Environment.Require();
 
 	if let Err(e) = IPCProvider
-		.SendNotificationToSidecar("cocoon-main".to_string(), "$acceptModelRemoved".to_string(), Payload)
+		.SendNotificationToSideCar("cocoon-main".to_string(), "$acceptModelRemoved".to_string(), Payload)
 		.await
 	{
 		error!("[DocumentProvider] Failed to send $acceptModelRemoved for {}: {}", URI, e);