feat(Mountain): Add per-URI marker payloads to diagnostics events and cap keyring logging

Two changes in Mountain's diagnostic system:

1. **DiagnosticProvider.rs**: Extend the `DiagnosticsChanged` event payload with a new `changedURIs` field containing per-URI marker data alongside the existing `Uris` string array. This enables SkyBridge's `cel:diagnostics:changed` listener to call `IMarkerService.changeOne(owner, uri, markers)` directly without an extra IPC round-trip per diagnostic change. Also adds lowercase `owner` field for consistency. Maintains backward compatibility with listeners expecting the original format.

2. **LoggingPlugin.rs**: Cap the `keyring` crate to Warn level. This dependency (used by Mountain's secret-storage path for license data lookups) emits a 3-line DEBUG block per `get_password` call that fires indefinitely after the workbench paints, similar to the previous `ignore` and `globset` mutes.

Signed-off-by: Nikola Hristov <Nikola@PlayForm.Cloud>

Author: NikolaRHristov

Source/Binary/Build/LoggingPlugin.rs

@@ -90,6 +90,13 @@ pub fn LoggingPlugin<R:tauri::Runtime>(LogLevel:LevelFilter) -> TauriPlugin<R> {
 		.level_for("ignore::walk", LevelFilter::Warn)
 		.level_for("ignore::gitignore", LevelFilter::Warn)
 		.level_for("globset", LevelFilter::Warn)
+		// `keyring` (used by Mountain's secret-storage path on the
+		// `dev1phpTools.license.data` lookup chain) emits a 3-line
+		// DEBUG block per `get_password` call - "creating entry",
+		// "created entry", "get password from entry" - per refresh
+		// tick. After the workbench paints these fire indefinitely.
+		// Cap to Warn alongside the other dependency mutes.
+		.level_for("keyring", LevelFilter::Warn)
 		// Filter out extremely noisy targets
 		.filter(|Metadata| {
 			!Metadata.target().starts_with("polling")

Source/Environment/DiagnosticProvider.rs

@@ -230,28 +230,55 @@ impl DiagnosticManager for MountainEnvironment {
 		let OwnerMap = DiagnosticsMapGuard.entry(Owner.clone()).or_default();
 
 		let mut ChangedURIKeys = Vec::new();
+		// `ChangedEntries` carries the post-update marker set per URI so the
+		// Sky-side `cel:diagnostics:changed` listener can call
+		// `IMarkerService.changeOne(owner, uri, markers)` without an extra
+		// IPC round-trip per change. URIs whose markers were cleared still
+		// appear here with an empty array, so the workbench replaces the
+		// previous owner-set rather than leaving stale red squiggles.
+		let mut ChangedEntries:Vec<serde_json::Value> = Vec::new();
 
 		for (URIComponentsValue, MarkersOption) in DeserializedEntries {
 			let URIKey = Utility::GetURLFromURIComponentsDTO(&URIComponentsValue)?.to_string();
 
 			ChangedURIKeys.push(URIKey.clone());
 
-			if let Some(Markers) = MarkersOption {
-				if Markers.is_empty() {
+			let MarkersForEvent = match MarkersOption {
+				Some(Markers) => {
+					if Markers.is_empty() {
+						OwnerMap.remove(&URIKey);
+						Vec::new()
+					} else {
+						let MarkersClone = Markers.clone();
+						OwnerMap.insert(URIKey.clone(), Markers);
+						MarkersClone
+					}
+				},
+				None => {
 					OwnerMap.remove(&URIKey);
-				} else {
-					OwnerMap.insert(URIKey, Markers);
-				}
-			} else {
-				OwnerMap.remove(&URIKey);
-			}
+					Vec::new()
+				},
+			};
+
+			ChangedEntries.push(json!({
+				"uri": URIKey,
+				"markers": MarkersForEvent,
+			}));
 		}
 
 		drop(DiagnosticsMapGuard);
 
-		// Notify the frontend that diagnostics have changed for specific URIs.
-		// Include both added/cleared URIs so UI can update accurately.
-		let EventPayload = json!({ "Owner": Owner, "Uris": ChangedURIKeys });
+		// Notify the frontend that diagnostics have changed. Both keys are
+		// included for backward compatibility - older listeners read `Uris`
+		// (string-array) while the new SkyBridge marker bridge reads
+		// `changedURIs` (per-URI marker payload) to push directly into
+		// the workbench's `IMarkerService`.
+		let EventPayload = json!({
+			"Owner": Owner,
+			"owner": Owner,
+			"Uris": ChangedURIKeys,
+			"changedURIs": ChangedEntries,
+		});
 
 		if let Err(Error) = self.ApplicationHandle.emit(SkyEvent::DiagnosticsChanged.AsStr(), EventPayload) {
 			dev_log!(
@@ -304,7 +331,19 @@ impl DiagnosticManager for MountainEnvironment {
 				ChangedURIKeys.len()
 			);
 
-			let EventPayload = json!({ "Owner": Owner, "Uris": ChangedURIKeys });
+			// Clear path - every URI's marker set goes to empty so the
+			// SkyBridge listener can wipe them via
+			// `IMarkerService.changeOne(owner, uri, [])`.
+			let ChangedEntries:Vec<serde_json::Value> = ChangedURIKeys
+				.iter()
+				.map(|Uri| json!({ "uri": Uri, "markers": [] }))
+				.collect();
+			let EventPayload = json!({
+				"Owner": Owner,
+				"owner": Owner,
+				"Uris": ChangedURIKeys,
+				"changedURIs": ChangedEntries,
+			});
 
 			if let Err(Error) = self.ApplicationHandle.emit(SkyEvent::DiagnosticsChanged.AsStr(), EventPayload) {
 				dev_log!(