fix: capture OS.networkInterfaces() once in getDesktopAdvertisedEndpoints

Avoid calling OS.networkInterfaces() twice with an await in between,
which could yield inconsistent snapshots if network state changes.
A single local variable is now shared by both resolveDesktopServerExposure
and resolveTailscaleAdvertisedEndpoints.

Author: cursoragent

apps/desktop/src/main.ts

@@ -315,10 +315,11 @@ function getDesktopServerExposureState(): DesktopServerExposureState {
 }
 
 async function getDesktopAdvertisedEndpoints() {
+  const networkInterfaces = OS.networkInterfaces();
   const exposure = resolveDesktopServerExposure({
     mode: desktopServerExposureMode,
     port: backendPort,
-    networkInterfaces: OS.networkInterfaces(),
+    networkInterfaces,
     ...(backendAdvertisedHost ? { advertisedHostOverride: backendAdvertisedHost } : {}),
   });
   const coreEndpoints = resolveDesktopCoreAdvertisedEndpoints({
@@ -328,7 +329,7 @@ async function getDesktopAdvertisedEndpoints() {
   });
   const tailscaleEndpoints = await resolveTailscaleAdvertisedEndpoints({
     port: backendPort,
-    networkInterfaces: OS.networkInterfaces(),
+    networkInterfaces,
   });
   return [...coreEndpoints, ...tailscaleEndpoints];
 }