Avoid unnecessary array copy in virtualizedItems memo

Author: charlesvien

apps/twig/src/renderer/features/sessions/components/ConversationView.tsx

@@ -95,15 +95,15 @@ export function ConversationView({
   const prevItemCountRef = useRef(0);
   const prevPendingCountRef = useRef(0);
 
-  const virtualizedItems = useMemo<VirtualizedItem[]>(() => {
-    const items: VirtualizedItem[] = [...conversationItems];
-
-    for (const msg of queuedMessages) {
-      items.push({ type: "queued", id: msg.id, message: msg });
-    }
+  const queuedItems = useMemo<QueuedItem[]>(
+    () => queuedMessages.map((msg) => ({ type: "queued" as const, id: msg.id, message: msg })),
+    [queuedMessages],
+  );
 
-    return items;
-  }, [conversationItems, queuedMessages]);
+  const virtualizedItems = useMemo<VirtualizedItem[]>(
+    () => (queuedItems.length > 0 ? [...conversationItems, ...queuedItems] : conversationItems),
+    [conversationItems, queuedItems],
+  );
 
   useEffect(() => {
     if (!taskId || hasRestoredScrollRef.current) return;