Updates based on code review

Author: alex72508

src/components/HallAnalytics.astro

@@ -28,9 +28,10 @@ const HALL_API_KEY = import.meta.env.HALL_API_KEY;
           'Referer': document.referrer,
           'Accept-Language': navigator.language,
           'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8',
-          'Sec-Ch-Ua': navigator.userAgent.includes('Chrome') ? '"Not_A Brand";v="8", "Chromium";v="120"' : '',
-          'Sec-Ch-Ua-Mobile': /Mobile|Android|iPhone|iPad/.test(navigator.userAgent) ? '?1' : '?0',
-          'Sec-Ch-Ua-Platform': `"${navigator.platform}"`
+          ...(navigator.userAgentData ? {
+            'Sec-Ch-Ua-Mobile': navigator.userAgentData.mobile ? '?1' : '?0',
+            'Sec-Ch-Ua-Platform': `"${navigator.userAgentData.platform}"`
+          } : {})
         };
 
         const analyticsData = {
@@ -67,6 +68,9 @@ const HALL_API_KEY = import.meta.env.HALL_API_KEY;
         trackPageView();
       }
 
+      // Store observers for potential cleanup
+      const observers = [];
+      
       // Also track when body becomes available (fallback)
       if (!document.body) {
         const bodyObserver = new MutationObserver(() => {
@@ -75,6 +79,7 @@ const HALL_API_KEY = import.meta.env.HALL_API_KEY;
             trackPageView();
           }
         });
+        observers.push(bodyObserver);
         bodyObserver.observe(document.documentElement, {
           childList: true,
           subtree: true
@@ -90,12 +95,18 @@ const HALL_API_KEY = import.meta.env.HALL_API_KEY;
             setTimeout(trackPageView, 100); // Small delay to ensure new page is loaded
           }
         });
+        observers.push(observer);
 
         observer.observe(document.body, {
           childList: true,
           subtree: true
         });
       }
+      
+      // Cleanup on page unload
+      window.addEventListener('beforeunload', () => {
+        observers.forEach(observer => observer.disconnect());
+      });
 
     })();
   </script>