fix: restore early GA tracking

tannerlinsley · tannerlinsley · commit 56a5f9013691 · 2026-03-30T16:35:10.000-06:00
Load GA from the document head so initial visits are tracked before hydration and keep SPA pageviews from double-counting the first load. Remove the stale Rollup output flag that broke typechecking with the current locked Vite version.
diff --git a/src/routes/__root.tsx b/src/routes/__root.tsx
@@ -36,6 +36,10 @@ import { Navbar } from '~/components/Navbar'
 import { THEME_COLORS } from '~/utils/utils'
 import { useHubSpotChat } from '~/hooks/useHubSpotChat'
 
+const GOOGLE_ANALYTICS_ID = 'G-JMT1Z50SPS'
+const GOOGLE_ANALYTICS_SCRIPT_SRC = `https://www.googletagmanager.com/gtag/js?id=${GOOGLE_ANALYTICS_ID}`
+const GOOGLE_ANALYTICS_BOOTSTRAP = `window.dataLayer = window.dataLayer || [];window.gtag = window.gtag || function(){window.dataLayer.push(arguments);};window.gtag('js', new Date());window.gtag('config', '${GOOGLE_ANALYTICS_ID}');`
+
 declare global {
   interface Window {
     dataLayer: unknown[] | undefined
@@ -101,6 +105,13 @@ export const Route = createRootRouteWithContext<{
       {
         children: `(function(){try{var t=localStorage.getItem('theme')||'auto';var v=['light','dark','auto'].includes(t)?t:'auto';if(v==='auto'){var a=matchMedia('(prefers-color-scheme: dark)').matches?'dark':'light';document.documentElement.classList.add(a,'auto')}else{document.documentElement.classList.add(v)}}catch(e){var a=matchMedia('(prefers-color-scheme: dark)').matches?'dark':'light';document.documentElement.classList.add(a,'auto')}})()`,
       },
+      {
+        async: true,
+        src: GOOGLE_ANALYTICS_SCRIPT_SRC,
+      },
+      {
+        children: GOOGLE_ANALYTICS_BOOTSTRAP,
+      },
     ],
   }),
   beforeLoad: async (ctx) => {
@@ -189,7 +200,7 @@ function ShellComponent({ children }: { children: React.ReactNode }) {
       <body className="overflow-x-hidden">
         <LoginModalProvider>
           <ToastProvider>
-            <IdleGtmLoader />
+            <GoogleAnalyticsTracker />
             {hideNavbar ? children : <Navbar>{children}</Navbar>}
             {showDevtools ? (
               <LazyRouterDevtools position="bottom-right" />
@@ -260,7 +271,7 @@ function SearchHotkeyController() {
   )
 }
 
-function IdleGtmLoader() {
+function GoogleAnalyticsTracker() {
   const pagePath = useRouterState({
     select: (s) => {
       const pathname = s.resolvedLocation?.pathname || '/'
@@ -269,38 +280,15 @@ function IdleGtmLoader() {
       return `${pathname}${search}`
     },
   })
+  const hasTrackedInitialPage = React.useRef(false)
 
   React.useEffect(() => {
-    const gaId = 'G-JMT1Z50SPS'
-    const existingScript = document.querySelector<HTMLScriptElement>(
-      `script[src*="googletagmanager.com/gtag/js?id=${gaId}"]`,
-    )
-
-    if (!window.dataLayer) {
-      window.dataLayer = []
-    }
-
     if (!window.gtag) {
-      window.gtag = (...args: unknown[]) => {
-        window.dataLayer?.push(args)
-      }
-    }
-
-    window.gtag('js', new Date())
-    window.gtag('config', gaId, {
-      send_page_view: false,
-    })
-
-    if (!existingScript) {
-      const script = document.createElement('script')
-      script.async = true
-      script.src = `https://www.googletagmanager.com/gtag/js?id=${gaId}`
-      document.head.appendChild(script)
+      return
     }
-  }, [])
 
-  React.useEffect(() => {
-    if (!window.gtag) {
+    if (!hasTrackedInitialPage.current) {
+      hasTrackedInitialPage.current = true
       return
     }
 
diff --git a/vite.config.ts b/vite.config.ts
@@ -59,7 +59,6 @@ export default defineConfig({
         return id.includes('postgres')
       },
       output: {
-        onlyExplicitManualChunks: true,
         manualChunks: (id) => {
           // Vendor chunk splitting for better caching
           if (id.includes('node_modules')) {
