perf(Mountain): Deduplicate dev log entries for unregistered commands

Replace dev_log! macro with DebugOnce for unregistered view-action and workbench-internal commands. These commands fire repeatedly (view-action buttons clicked multiple times, getTelemetrySenderObject fires 30+ times per boot extension activation), causing N-line log trails that obscure other diagnostic output.

The new implementation uses a dedup key format (e.g., "view-action-noop:{CommandId}") so the first invocation still logs for debugging probe shape, while subsequent calls to the same command ID are silently suppressed.

This reduces dev log noise in hot IPC paths while preserving diagnostic value for identifying unregistered command patterns.

Author: NikolaRHristov

Source/Environment/CommandProvider.rs

@@ -297,10 +297,19 @@ impl CommandExecutor for MountainEnvironment {
 					|| CommandIdentifier.ends_with(".resetViewLocation")
 					|| CommandIdentifier.ends_with(".removeView")
 				{
-					dev_log!(
+					// Once-per-command-id so the no-op fallback doesn't
+					// generate an N-line trail through the dev log every
+					// time the user clicks a view-action button. The
+					// first occurrence still fires (documents the probe
+					// shape); subsequent invocations of the same command
+					// are silent.
+					crate::IPC::DevLog::DebugOnce(
 						"commands",
-						"[CommandProvider] View-action command '{}' not registered; treating as no-op (auto-generated by view registry in stock VS Code).",
-						CommandIdentifier
+						&format!("view-action-noop:{}", CommandIdentifier),
+						&format!(
+							"[CommandProvider] View-action command '{}' not registered; treating as no-op (auto-generated by view registry in stock VS Code).",
+							CommandIdentifier
+						),
 					);
 					return Ok(Value::Null);
 				}
@@ -329,10 +338,17 @@ impl CommandExecutor for MountainEnvironment {
 					CommandIdentifier.as_str(),
 					"getTelemetrySenderObject" | "testing.clearTestResults"
 				) {
-					dev_log!(
+					// `getTelemetrySenderObject` fires once per extension
+					// activation (~30+ times per boot) - same once-per-id
+					// dedup as the view-action path so the log line
+					// documents the probe but doesn't trail.
+					crate::IPC::DevLog::DebugOnce(
 						"commands",
-						"[CommandProvider] Workbench-internal command '{}' not registered; treating as no-op (Land has no backing service).",
-						CommandIdentifier
+						&format!("workbench-internal-noop:{}", CommandIdentifier),
+						&format!(
+							"[CommandProvider] Workbench-internal command '{}' not registered; treating as no-op (Land has no backing service).",
+							CommandIdentifier
+						),
 					);
 					return Ok(Value::Null);
 				}