fix: traceroute duplication caused by bypassing deduplication logic (#2387)

meshtasticManager.ts called databaseService.traceroutes.insertTraceroute()
(the repository directly), bypassing the deduplication logic in
databaseService.insertTraceroute() which checks for pending traceroute
requests and updates them instead of inserting duplicates.

This caused every traceroute response to create a new record even when
a pending request record already existed, resulting in duplicate entries
in traceroute history.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Yeraze

src/server/meshtasticManager.ts

@@ -5147,7 +5147,9 @@ class MeshtasticManager {
         createdAt: Date.now()
       };
 
-      await databaseService.traceroutes.insertTraceroute(tracerouteRecord);
+      // Use DatabaseService.insertTraceroute() (not repo directly) for deduplication:
+      // It checks for pending traceroute requests and updates them instead of inserting duplicates
+      databaseService.insertTraceroute(tracerouteRecord);
 
       // Store traceroute hop count as telemetry for Smart Hops tracking
       // Hop count is route.length + 1 (intermediate hops + final hop to destination)