fix: remove unused state and simplify image rendering logic in DidAvatar component

Author: benjinus

packages/webapp/src/components/shared/DidAvatar.tsx

@@ -1,5 +1,3 @@
-import { useState } from "react"
-
 import { Avatar, AvatarFallback, AvatarImage } from "@/components/ui/avatar"
 import { cn } from "@/lib/utils"
 import { useConnectionStore } from "@/stores/connection"
@@ -34,23 +32,19 @@ interface DidAvatarProps {
 
 export function DidAvatar({ did, avatarUrl, className }: DidAvatarProps) {
   const nodeUrl = useConnectionStore((state) => state.nodeUrl)
-  const [imgError, setImgError] = useState(false)
 
   // Resolve relative paths (e.g. /api/v1/profile/avatar) against the node URL
   const resolvedUrl =
     avatarUrl && avatarUrl.startsWith("/") && nodeUrl
       ? `${nodeUrl.replace(/\/$/, "")}${avatarUrl}`
       : avatarUrl
 
-  const showImage = Boolean(resolvedUrl) && !imgError
-
   return (
     <Avatar className={cn("size-9", className)}>
-      {showImage && (
+      {resolvedUrl && (
         <AvatarImage
           src={resolvedUrl}
           alt={did}
-          onError={() => setImgError(true)}
         />
       )}
       <AvatarFallback style={{ backgroundColor: colorForDid(did), color: "white" }}>