fix: always handle process rejections

Author: gagik

src/logger.ts

@@ -17,7 +17,7 @@ export const LogId = {
     telemetryEmitFailure: mongoLogId(1_002_002),
     telemetryEmitStart: mongoLogId(1_002_003),
     telemetryEmitSuccess: mongoLogId(1_002_004),
-    telemetryMachineIdFailure: mongoLogId(1_002_005),
+    telemetryDeviceIdFailure: mongoLogId(1_002_005),
 
     toolExecute: mongoLogId(1_003_001),
     toolExecuteFailure: mongoLogId(1_003_002),

src/telemetry/telemetry.ts

@@ -40,7 +40,12 @@ export class Telemetry {
 
     private async start(): Promise<void> {
         this.deviceIdPromise = DeferredPromise.fromPromise(this.getDeviceId());
-        this.commonProperties.device_id = await this.deviceIdPromise;
+        try {
+            this.commonProperties.device_id = await this.deviceIdPromise;
+        } catch (error) {
+            logger.debug(LogId.telemetryDeviceIdFailure, "telemetry", String(error));
+            this.commonProperties.device_id = "unknown";
+        }
 
         this.isBufferingEvents = false;
     }
@@ -72,7 +77,7 @@ export class Telemetry {
             hmac.update(DEVICE_ID_HASH_MESSAGE);
             return hmac.digest("hex");
         } catch (error) {
-            logger.debug(LogId.telemetryMachineIdFailure, "telemetry", String(error));
+            logger.debug(LogId.telemetryDeviceIdFailure, "telemetry", String(error));
             return "unknown";
         }
     }

tests/unit/telemetry.test.ts

@@ -284,7 +284,7 @@ describe("Telemetry", () => {
             expect(telemetry.getCommonProperties().device_id).toBe("unknown");
 
             expect(loggerSpy).toHaveBeenCalledWith(
-                LogId.telemetryMachineIdFailure,
+                LogId.telemetryDeviceIdFailure,
                 "telemetry",
                 "Error: Failed to get device ID"
             );
@@ -315,7 +315,7 @@ describe("Telemetry", () => {
             expect(telemetry.getCommonProperties().device_id).toBe("unknown");
             expect(telemetry["isBufferingEvents"]).toBe(false);
             expect(loggerSpy).toHaveBeenCalledWith(
-                LogId.telemetryMachineIdFailure,
+                LogId.telemetryDeviceIdFailure,
                 "telemetry",
                 "Error: Promise timed out"
             );