PR feedback

Author: mrubens

package-lock.json

@@ -23,6 +23,7 @@
     "@docusaurus/preset-classic": "^3.8.1",
     "@easyops-cn/docusaurus-search-local": "^0.48.5",
     "@mdx-js/react": "^3.0.0",
+    "@roo-code/types": "^1.79.0",
     "@vscode/codicons": "^0.0.36",
     "clsx": "^2.0.0",
     "posthog-docusaurus": "^2.0.4",

package.json

@@ -46,18 +46,6 @@ export function PostHogProvider({ children }: { children: React.ReactNode }) {
     if (typeof window !== 'undefined' && window.posthog) {
       window.posthog.opt_out_capturing();
       window.posthog.stop_session_recording();
-      if (document) {
-        const cookies = document.cookie.split(';');
-        cookies.forEach(cookie => {
-          const eqPos = cookie.indexOf('=');
-          const name = eqPos > -1 ? cookie.substr(0, eqPos).trim() : cookie.trim();
-          if (name.startsWith('ph_') || name.startsWith('posthog')) {
-            // Delete PostHog related cookies
-            document.cookie = `${name}=; expires=Thu, 01 Jan 1970 00:00:00 UTC; path=/; domain=.roocode.com`;
-            document.cookie = `${name}=; expires=Thu, 01 Jan 1970 00:00:00 UTC; path=/`;
-          }
-        });
-      }
     }
   };
 

src/components/PostHogProvider/index.tsx

@@ -28,7 +28,4 @@ export const OPEN_VSX_URL = 'https://open-vsx.org/extension/RooVeterinaryInc/roo
 export const CONTACT_EMAIL = 'mailto:[email protected]';
 export const CAREERS_URL = 'https://careers.roocode.com';
 export const WEBSITE_PRIVACY_URL = 'https://roocode.com/privacy';
-export const EXTENSION_PRIVACY_URL = `${GITHUB_MAIN_REPO_URL}/blob/main/PRIVACY.md`;
-
-// Cookies
-export const CONSENT_COOKIE_NAME = "roo-code-cookie-consent";
+export const EXTENSION_PRIVACY_URL = `${GITHUB_MAIN_REPO_URL}/blob/main/PRIVACY.md`;

src/constants.ts

@@ -1,4 +1,5 @@
-import { CONSENT_COOKIE_NAME } from '../../constants';
+import { CONSENT_COOKIE_NAME } from '@roo-code/types';
+import { getCookieConsentValue } from 'react-cookie-consent';
 
 // Custom event names for consent tracking
 export const CONSENT_GRANTED_EVENT = 'cookieConsentGranted';
@@ -31,20 +32,12 @@ export function dispatchConsentEvent(granted: boolean): void {
 }
 
 /**
- * Checks if the user has already provided consent
- * @returns boolean indicating if consent cookie exists and is set to 'true'
+ * Check if user has given consent for analytics cookies
+ * Uses react-cookie-consent's built-in function
  */
 export function hasConsent(): boolean {
   if (typeof window === 'undefined') return false;
-  
-  const cookies = document.cookie.split(';');
-  for (let cookie of cookies) {
-    const [name, value] = cookie.trim().split('=');
-    if (name === CONSENT_COOKIE_NAME) {
-      return value === 'true';
-    }
-  }
-  return false;
+  return getCookieConsentValue(CONSENT_COOKIE_NAME) === 'true';
 }
 
 /**