fix: remove setTimeout race and use debug-level logging

- Remove the 100ms setTimeout that tried to sync enabled state to backend
  (unnecessary - frontend guards trackEvent with isTelemetryEnabled())
- Change all telemetry logs from log/warn to debug level to avoid
  giving users a reason to disable telemetry

Author: ammar-agent

src/common/telemetry/client.ts

@@ -38,13 +38,13 @@ export function setTelemetryEnabled(enabled: boolean): void {
   if (typeof window === "undefined") return;
 
   localStorage.setItem(TELEMETRY_ENABLED_KEY, enabled.toString());
-  console.log(`[Telemetry] ${enabled ? "Enabled" : "Disabled"}`);
+  console.debug(`[Telemetry] ${enabled ? "Enabled" : "Disabled"}`);
 
   // Notify backend of preference change
   const client = window.__ORPC_CLIENT__;
   if (client) {
     client.telemetry.setEnabled({ enabled }).catch((err: unknown) => {
-      console.warn("[Telemetry] Failed to sync enabled state to backend:", err);
+      console.debug("[Telemetry] Failed to sync enabled state to backend:", err);
     });
   }
 }
@@ -80,23 +80,14 @@ export function initTelemetry(): void {
   }
 
   if (isInitialized) {
-    console.warn("Telemetry already initialized");
+    console.debug("[Telemetry] Already initialized");
     return;
   }
 
   isInitialized = true;
   console.debug("[Telemetry] Initialized (backend mode)");
-
-  // Sync enabled state to backend once client is available
-  // Use a small delay to allow ORPC connection to establish
-  setTimeout(() => {
-    const client = window.__ORPC_CLIENT__;
-    if (client) {
-      client.telemetry.setEnabled({ enabled: true }).catch((err: unknown) => {
-        console.warn("[Telemetry] Failed to sync initial enabled state:", err);
-      });
-    }
-  }, 100);
+  // No need to sync enabled state - backend defaults to enabled,
+  // and frontend guards all trackEvent calls with isTelemetryEnabled()
 }
 
 /**
@@ -129,7 +120,7 @@ export function trackEvent(payload: TelemetryEventPayload): void {
 
   // Fire and forget - don't block on telemetry
   client.telemetry.track(payload).catch((err: unknown) => {
-    console.warn("[Telemetry] Failed to track event:", err);
+    console.debug("[Telemetry] Failed to track event:", err);
   });
 }
 

src/node/services/telemetryService.ts

@@ -68,7 +68,7 @@ export class TelemetryService {
     }
 
     if (this.client) {
-      console.warn("[TelemetryService] Already initialized");
+      console.debug("[TelemetryService] Already initialized");
       return;
     }
 
@@ -109,7 +109,7 @@ export class TelemetryService {
       await fs.mkdir(this.muxHome, { recursive: true });
       await fs.writeFile(idPath, newId, "utf-8");
     } catch (err) {
-      console.warn("[TelemetryService] Failed to persist distinct ID:", err);
+      console.debug("[TelemetryService] Failed to persist distinct ID:", err);
     }
 
     return newId;
@@ -127,7 +127,7 @@ export class TelemetryService {
    */
   setEnabled(enabled: boolean): void {
     this.enabled = enabled;
-    console.log(`[TelemetryService] ${enabled ? "Enabled" : "Disabled"}`);
+    console.debug(`[TelemetryService] ${enabled ? "Enabled" : "Disabled"}`);
   }
 
   /**
@@ -185,7 +185,7 @@ export class TelemetryService {
     try {
       await this.client.shutdown();
     } catch (err) {
-      console.warn("[TelemetryService] Error during shutdown:", err);
+      console.debug("[TelemetryService] Error during shutdown:", err);
     }
 
     this.client = null;