fix(favicon): record an icon that failed before hydration

onError never fires for a failure React missed, so the fallback never showed and any remount asked
for the icon again: 204 answers are not reusable within a document, which is what Links' first paint
shows as every favicon read twice. The ref now records the failure at hydration; decode separates a
broken image from an SVG with no intrinsic size.

Author: waleedlatif1

apps/sim/components/favicon.tsx

@@ -32,21 +32,25 @@ interface FaviconProps {
 export function Favicon({ host, fallback, className, onLoad }: FaviconProps) {
   const [, rerender] = useReducer((failures: number) => failures + 1, 0)
   if (!host || knownMisses.has(host)) return fallback
+  const recordMiss = () => {
+    knownMisses.set(host, true)
+    rerender()
+  }
   return (
     <img
       ref={(image) => {
-        if (onLoad && image?.complete && image.naturalWidth > 0) onLoad()
+        if (!image?.complete) return
+        if (image.naturalWidth > 0) onLoad?.()
+        // A failure before hydration fires no onError, and a remount would ask for the icon again.
+        else void image.decode().catch(recordMiss)
       }}
       src={faviconSrc(host)}
       alt=''
       loading='lazy'
       decoding='async'
       className={cn('rounded-sm', className)}
       onLoad={onLoad}
-      onError={() => {
-        knownMisses.set(host, true)
-        rerender()
-      }}
+      onError={recordMiss}
     />
   )
 }