fix: add event_callback pattern to Google Ads conversion tracking

- Add event_callback to trackGoogleAdsConversion() to ensure conversion ping completes before navigation
- Update CTA button handlers to prevent default navigation and pass target URL
- Add fallback timeout for reliability in case callback does not fire
- Improve documentation with usage examples

Author: roomote

apps/web-roo-code/src/app/reviewer/ReviewerContent.tsx

@@ -127,7 +127,10 @@ export function ReviewerContent() {
 										href={EXTERNAL_LINKS.CLOUD_APP_SIGNUP_PRO}
 										target="_blank"
 										rel="noopener noreferrer"
-										onClick={trackGoogleAdsConversion}
+										onClick={(e) => {
+											e.preventDefault()
+											trackGoogleAdsConversion(EXTERNAL_LINKS.CLOUD_APP_SIGNUP_PRO)
+										}}
 										className="flex w-full items-center justify-center">
 										Start 14-day Free Trial
 										<ArrowRight className="ml-2" />
@@ -281,7 +284,10 @@ export function ReviewerContent() {
 									href={EXTERNAL_LINKS.CLOUD_APP_SIGNUP_PRO}
 									target="_blank"
 									rel="noopener noreferrer"
-									onClick={trackGoogleAdsConversion}
+									onClick={(e) => {
+										e.preventDefault()
+										trackGoogleAdsConversion(EXTERNAL_LINKS.CLOUD_APP_SIGNUP_PRO)
+									}}
 									className="flex items-center justify-center">
 									Start 14-day Free Trial
 									<ArrowRight className="ml-2 h-4 w-4" />

apps/web-roo-code/src/lib/analytics/google-ads.ts

@@ -3,15 +3,40 @@
  */
 
 /**
- * Track a Google Ads conversion event
- * This should only be called after user consent has been given
+ * Track a Google Ads conversion event with optional navigation callback
+ * This ensures the conversion ping completes before navigation occurs
+ *
+ * @param targetUrl - Optional URL to navigate to after conversion is tracked
+ *
+ * @example
+ * // Track conversion and navigate
+ * trackGoogleAdsConversion("https://app.roocode.com")
+ *
+ * @example
+ * // Track conversion without navigation
+ * trackGoogleAdsConversion()
  */
-export function trackGoogleAdsConversion() {
+export function trackGoogleAdsConversion(targetUrl?: string) {
 	if (typeof window !== "undefined" && window.gtag) {
+		// Callback to handle navigation after conversion tracking
+		const callback = () => {
+			if (targetUrl) {
+				window.location.href = targetUrl
+			}
+		}
+
 		window.gtag("event", "conversion", {
 			send_to: "AW-17391954825/VtOZCJe_77MbEInXkOVA",
 			value: 10.0,
 			currency: "USD",
+			event_callback: callback,
 		})
+
+		// Fallback timeout in case event_callback doesn't fire (network issues, etc.)
+		// This ensures navigation still happens even if tracking fails
+		setTimeout(callback, 1000)
+	} else if (targetUrl) {
+		// If gtag is not available, navigate immediately
+		window.location.href = targetUrl
 	}
 }