From 25287758153046809e4daaf868179314225bc70f Mon Sep 17 00:00:00 2001
From: Roo Code <roomote@roocode.com>
Date: Wed, 28 Jan 2026 00:44:40 +0000
Subject: [PATCH] fix(blog): prevent duplicate blog_post_time_spent events

- Add trackedTimeSpent ref to track if time-spent event already fired
- Check flag in visibilitychange handler and cleanup function
- Remove duplicate interface declaration in BlogPostAnalyticsProps
- Remove duplicate BlogPostAnalytics component opening tag
---
 apps/web-roo-code/src/app/blog/[slug]/page.tsx     |  1 -
 .../src/components/blog/blog-analytics.tsx         | 14 +++++++++-----
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/apps/web-roo-code/src/app/blog/[slug]/page.tsx b/apps/web-roo-code/src/app/blog/[slug]/page.tsx
index bc88f81ff1..2ed25ff34b 100644
--- a/apps/web-roo-code/src/app/blog/[slug]/page.tsx
+++ b/apps/web-roo-code/src/app/blog/[slug]/page.tsx
@@ -89,7 +89,6 @@ export default async function BlogPostPage({ params }: BlogPostPageProps) {
 			<script type="application/ld+json" dangerouslySetInnerHTML={{ __html: JSON.stringify(breadcrumbSchema) }} />
 
 			{/* PostHog Analytics */}
-			<BlogPostAnalytics
 			<BlogPostAnalytics
 				post={{
 					slug: post.slug,
diff --git a/apps/web-roo-code/src/components/blog/blog-analytics.tsx b/apps/web-roo-code/src/components/blog/blog-analytics.tsx
index 00c11bcdd5..781134b264 100644
--- a/apps/web-roo-code/src/components/blog/blog-analytics.tsx
+++ b/apps/web-roo-code/src/components/blog/blog-analytics.tsx
@@ -25,7 +25,6 @@ export function BlogIndexAnalytics({ postCount }: BlogIndexAnalyticsProps) {
 	return null
 }
 
-interface BlogPostAnalyticsProps {
 interface BlogPostAnalyticsProps {
 	post: {
 		slug: string
@@ -44,6 +43,7 @@ interface BlogPostAnalyticsProps {
 export function BlogPostAnalytics({ post }: BlogPostAnalyticsProps) {
 	const trackedView = useRef(false)
 	const trackedDepths = useRef<Set<25 | 50 | 75 | 100>>(new Set())
+	const trackedTimeSpent = useRef(false)
 	const startTime = useRef<number>(Date.now())
 
 	useEffect(() => {
@@ -74,7 +74,8 @@ export function BlogPostAnalytics({ post }: BlogPostAnalyticsProps) {
 
 		// Track time spent on page when leaving
 		const handleVisibilityChange = () => {
-			if (document.visibilityState === "hidden") {
+			if (document.visibilityState === "hidden" && !trackedTimeSpent.current) {
+				trackedTimeSpent.current = true
 				const timeSpent = Date.now() - effectStartTime
 				trackBlogPostTimeSpent(post as BlogPost, timeSpent)
 			}
@@ -90,9 +91,12 @@ export function BlogPostAnalytics({ post }: BlogPostAnalyticsProps) {
 			window.removeEventListener("scroll", handleScroll)
 			document.removeEventListener("visibilitychange", handleVisibilityChange)
 
-			// Track time spent when component unmounts
-			const timeSpent = Date.now() - effectStartTime
-			trackBlogPostTimeSpent(post as BlogPost, timeSpent)
+			// Track time spent when component unmounts (only if not already tracked)
+			if (!trackedTimeSpent.current) {
+				trackedTimeSpent.current = true
+				const timeSpent = Date.now() - effectStartTime
+				trackBlogPostTimeSpent(post as BlogPost, timeSpent)
+			}
 		}
 		// eslint-disable-next-line react-hooks/exhaustive-deps
 	}, [post.slug])