simplify(landing): drop unnecessary Suspense from HubSpot pageview tracker

usePathname() alone doesn't require a Suspense boundary to preserve static
rendering — only useSearchParams() does. The tracker only needs the path,
not the query string, so drop useSearchParams and the Suspense wrapper
entirely. Simpler, and no risk to the landing site's static rendering/LCP.

Author: waleedlatif1

apps/sim/app/(landing)/hubspot-page-view-tracker.tsx

@@ -1,7 +1,7 @@
 'use client'
 
 import { useEffect, useRef } from 'react'
-import { usePathname, useSearchParams } from 'next/navigation'
+import { usePathname } from 'next/navigation'
 
 declare global {
   interface Window {
@@ -18,7 +18,6 @@ declare global {
  */
 export function HubspotPageViewTracker() {
   const pathname = usePathname()
-  const searchParams = useSearchParams()
   const isInitialRender = useRef(true)
 
   useEffect(() => {
@@ -27,13 +26,10 @@ export function HubspotPageViewTracker() {
       return
     }
 
-    const query = searchParams.toString()
-    const path = query ? `${pathname}?${query}` : pathname
-
     window._hsq = window._hsq || []
-    window._hsq.push(['setPath', path])
+    window._hsq.push(['setPath', pathname])
     window._hsq.push(['trackPageView'])
-  }, [pathname, searchParams])
+  }, [pathname])
 
   return null
 }

apps/sim/app/(landing)/layout.tsx

@@ -1,5 +1,4 @@
 import type { ReactNode } from 'react'
-import { Suspense } from 'react'
 import type { Metadata } from 'next'
 import Script from 'next/script'
 import { isHosted } from '@/lib/core/config/env-flags'
@@ -37,9 +36,7 @@ export default function LandingLayout({ children }: { children: ReactNode }) {
       {isHosted && (
         <>
           <Script id='hs-script-loader' src={HUBSPOT_SCRIPT_SRC} strategy='afterInteractive' />
-          <Suspense fallback={null}>
-            <HubspotPageViewTracker />
-          </Suspense>
+          <HubspotPageViewTracker />
         </>
       )}
     </LandingShell>