ros/automaker
1
0
Fork 0
mirror of https://github.com/AutoMaker-Org/automaker.git synced 2026-02-04 09:13:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add cleanup function to setTimeout in useEffect to prevent memory leaks

Browse Source 
Co-authored-by: GTheMachine <156854865+GTheMachine@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot]
2025-12-10 22:42:17 +00:00
parent 7886a29089
commit a4dc21fd84
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/src/components/views/interview-view.tsx
View File

@@ -100,9 +100,10 @@ export function InterviewView() {
// Auto-scroll to bottom when messages change // Auto-scroll to bottom when messages change
useEffect(() => { useEffect(() => {
let timeoutId: NodeJS.Timeout | undefined;
if (messagesContainerRef.current) { if (messagesContainerRef.current) {
// Use a small delay to ensure DOM is updated // Use a small delay to ensure DOM is updated
setTimeout(() => { timeoutId = setTimeout(() => {
if (messagesContainerRef.current) { if (messagesContainerRef.current) {
messagesContainerRef.current.scrollTo({ messagesContainerRef.current.scrollTo({
top: messagesContainerRef.current.scrollHeight, top: messagesContainerRef.current.scrollHeight,
@@ -111,6 +112,11 @@ export function InterviewView() {
} }
}, 100); }, 100);
} }
return () => {
if (timeoutId) {
clearTimeout(timeoutId);
}
};
}, [messages]); }, [messages]);
// Auto-focus input // Auto-focus input