fix(components): address review findings

- MarkdownView: add useEffect to sync content prop changes
- UserMultiSelect: remove unnecessary array copy in filteredUsers
- UserMultiSelect: remove dead email guard (email is always a string)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: hendrikebbers

src/components/markdown-view.tsx

@@ -1,5 +1,6 @@
 "use client";
 
+import { useEffect } from "react";
 import { useEditor, EditorContent } from "@tiptap/react";
 import StarterKit from "@tiptap/starter-kit";
 import Link from "@tiptap/extension-link";
@@ -34,5 +35,11 @@ export function MarkdownView({ content }: MarkdownViewProps) {
     },
   });
 
+  useEffect(() => {
+    if (editor) {
+      editor.commands.setContent(content);
+    }
+  }, [content, editor]);
+
   return <EditorContent editor={editor} />;
 }

src/components/user-multi-select.tsx

@@ -34,7 +34,7 @@ export function UserMultiSelect({
           u.name.toLowerCase().includes(search.toLowerCase()) ||
           u.email.toLowerCase().includes(search.toLowerCase()),
       )
-    : [...users];
+    : users;
 
   return (
     <Popover open={open} onOpenChange={setOpen}>
@@ -119,9 +119,7 @@ export function UserMultiSelect({
                 <UserAvatar user={user} size="md" />
                 <div className="flex flex-col items-start overflow-hidden">
                   <span className="truncate font-medium">{user.name}</span>
-                  {user.email && (
-                    <span className="text-muted-foreground truncate text-xs">{user.email}</span>
-                  )}
+                  <span className="text-muted-foreground truncate text-xs">{user.email}</span>
                 </div>
               </button>
             );