style(Common): Remove non_snake_case allowance across Telemetry and Transport modules

Strip the `#![allow(non_snake_case)]` attribute from all files under `Source/Telemetry/` and `Source/Transport/` to enforce strict PascalCase naming compliance across the Core library. Also remove this allowance from `Source/IPC/Channel.rs` and `Source/IPC/SkyEvent.rs` where it was previously suppressing warnings for event types. Correct an escaped character in `SourceControlCreateDTO.rs` comments.

This aligns the Common crate with the project-wide naming convention where all functions, variables, and structs must use PascalCase, removing a long-standing lint suppression that was masking naming violations in critical subsystems.

Author: NikolaRHristov

Source/IPC/Channel.rs

@@ -31,7 +31,7 @@
 //!     offline-gallery stubs).
 //!   - `Normal`: everything else.
 
-#![allow(non_snake_case, non_camel_case_types)]
+#![allow(non_camel_case_types)]
 
 /// Lane selector for Echo scheduler dispatch.
 ///

Source/IPC/SkyEvent.rs

@@ -26,7 +26,7 @@
 //! `FromStr` in one pass so adding an event is a single-line change that
 //! compilers can't forget.
 
-#![allow(non_snake_case, non_camel_case_types)]
+#![allow(non_camel_case_types)]
 
 macro_rules! DefineSkyEvents {
 

Source/SourceControlManagement/DTO/SourceControlCreateDTO.rs

@@ -32,7 +32,7 @@ pub struct SourceControlCreateDTO {
 	/// resolve - allocating a fresh handle here keys the
 	/// `SourceControlManagementProviders` map under a value Cocoon will
 	/// never reference, and every later `update_scm_group` warns
-	/// "Received group update for unknown provider handle: <H>" while the
+	/// "Received group update for unknown provider handle: \<H\>" while the
 	/// SCM viewlet stays empty. Optional so callers without an external
 	/// handle (none currently) still work via Mountain-side allocation.
 	#[serde(default, skip_serializing_if = "Option::is_none")]

Source/Telemetry/CaptureError.rs

@@ -1,4 +1,3 @@
-#![allow(non_snake_case)]
 
 //! Emit `land:<tier>:error` with `error_tag` + `error_message`. The
 //! Errors & Reliability dashboard rolls these up across Mountain /

Source/Telemetry/CaptureEvent.rs

@@ -1,4 +1,3 @@
-#![allow(non_snake_case)]
 
 //! Emit a named PostHog event. Stamps the calling `Tier` plus the
 //! standard Land identity (`$app`, `$app_version`, `$build_mode`,

Source/Telemetry/CaptureSession.rs

@@ -1,4 +1,3 @@
-#![allow(non_snake_case)]
 
 //! Emit `land:<tier>:session:start` with pid / OS / arch. Called once
 //! by `Initialize::Fn` so the Boot & Startup Performance dashboard

Source/Telemetry/Client.rs

@@ -1,4 +1,3 @@
-#![allow(non_snake_case)]
 
 //! Process-wide PostHog client singleton. Populated once by
 //! `Initialize::Fn`; every `Capture*::Fn` reads through this static.

Source/Telemetry/Configuration.rs

@@ -1,4 +1,3 @@
-#![allow(non_snake_case)]
 
 //! Runtime read of `.env.Land.PostHog`. Sidecars don't have their own
 //! `build.rs` env-bake (Mountain does, for compile-time tree-shake) -

Source/Telemetry/DistinctId.rs

@@ -1,4 +1,3 @@
-#![allow(non_snake_case)]
 
 //! Stable distinct ID for the dev session. Mirrors Mountain's
 //! `Binary/Build/PostHogPlugin/DistinctId` so a single dev run merges

Source/Telemetry/EmitOTLPSpan.rs

@@ -1,4 +1,3 @@
-#![allow(non_snake_case)]
 
 //! Fire-and-forget OTLP span exporter. Lifted from Mountain's
 //! `IPC/DevLog/EmitOTLPSpan` so Air / Echo / Rest / Grove / Mist /