fix: address PR review suggestions for #439

- Wrap FadeIn in memo() to prevent unnecessary re-renders from parent
  state changes that don't affect the component's own props
- Log suppressed extension noise errors to console.debug in development
  so they remain visible locally while staying off Sentry in production

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: 2 people

website/src/components/ui/FadeIn.tsx

@@ -1,8 +1,8 @@
 'use client';
 import { motion, useReducedMotion } from 'framer-motion';
-import { ReactNode, useState, useEffect } from 'react';
+import { ReactNode, useState, useEffect, memo } from 'react';
 
-export function FadeIn({ children, delay = 0, className = '' }: { children: ReactNode; delay?: number; className?: string }) {
+export const FadeIn = memo(function FadeIn({ children, delay = 0, className = '' }: { children: ReactNode; delay?: number; className?: string }) {
   const shouldReduce = useReducedMotion();
   const [isMounted, setIsMounted] = useState(false);
 
@@ -29,4 +29,4 @@ export function FadeIn({ children, delay = 0, className = '' }: { children: Reac
       {children}
     </motion.div>
   );
-}
+});

website/src/instrumentation-client.ts

@@ -24,10 +24,14 @@ Sentry.init({
     // 2. "Object Not Found Matching Id:N, MethodName:update" — Chrome DevTools
     //    Protocol messages from extensions interacting with CodeMirror/Monaco.
     //    The error originates outside our code and only affects a single session.
-    if (
+    const isExtensionNoise =
       msg.includes("Cannot assign to read only property 'pushState'") ||
-      msg.includes("Object Not Found Matching Id:")
-    ) {
+      msg.includes("Object Not Found Matching Id:");
+
+    if (isExtensionNoise) {
+      if (process.env.NODE_ENV === "development") {
+        console.debug("[Sentry] Suppressed extension noise:", msg);
+      }
       return null;
     }