fix: cache posthog user count

Author: kevinjosethomas

src/pages/api/active-users.ts

@@ -6,6 +6,10 @@ type Data = {
   error?: string
 }
 
+// In-memory cache
+let cachedUsers: { value: number; timestamp: number } | null = null
+const CACHE_DURATION = 60 * 1000 // 1 minute
+
 /**
  * API endpoint to get active user count from PostHog
  * This keeps the PostHog API key secure on the server side
@@ -23,19 +27,38 @@ export default async function handler(
   }
 
   try {
+    const now = Date.now()
+
+    // Serve from cache if not expired
+    if (cachedUsers && now - cachedUsers.timestamp < CACHE_DURATION) {
+      return res.status(200).json({
+        activeUsers: cachedUsers.value,
+        success: true,
+      })
+    }
+
+    // Fetch fresh data
     const activeUsers = await fetchActiveUsersFromPostHog()
 
     if (activeUsers !== null) {
+      cachedUsers = {
+        value: activeUsers,
+        timestamp: now,
+      }
+
       return res.status(200).json({
         activeUsers,
         success: true,
       })
     }
+
+    throw new Error('Failed to retrieve active users')
   } catch (error) {
     console.error('Error in active-users API:', error)
     return res.status(500).json({
       activeUsers: 0,
       success: false,
+      error: 'Internal server error',
     })
   }
 }