🧵 fix: Prevent Unnecessary Re-renders when Loading Chats

[danny-avila]

Summary

I optimized to prevent unnecessary re-renders when loading conversations initially

• Refactored the message selection logic using useCallback to prevent buildTree from unnecessarily invoking more than once on initial conversation load.
• Improved message scrolling logic to handle empty message trees and reference checks.
• Optimized icon rendering and chat properties to enhance rendering performance.
• Integrated queryClient to reset message data when initiating a new conversation.
• Updated data-testid attributes for consistency and improved code readability.
• Removed unnecessary console logs in useQueryParams cleanup.

Closes #5185

Change Type

• Bug fix (non-breaking change which fixes an issue)

Testing

I tested the changes by loading conversations with over 1000 messages and verified that the application does not re-render unnecessarily or crash with recursion error. I ensured that starting new conversations resets message data correctly and that icon rendering works as expected. I also checked that data-testid attributes are consistent and that there are no console errors or warnings.

Checklist

• My code adheres to this project's style guidelines
• I have performed a self-review of my own code
• My changes do not introduce new warnings
• Local unit tests pass with my changes