From 0d1fdac09482b8376ea1634869c576ecd88e9d5f Mon Sep 17 00:00:00 2001 From: Neil <4138956+nwparker@users.noreply.github.com> Date: Sun, 31 May 2026 07:57:03 -0700 Subject: [PATCH] perf: move terminal rename cleanup to container ref --- .../src/components/terminal-pane/TerminalPane.tsx | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/renderer/src/components/terminal-pane/TerminalPane.tsx b/src/renderer/src/components/terminal-pane/TerminalPane.tsx index 989470809d5..5b3aa2c5d0c 100644 --- a/src/renderer/src/components/terminal-pane/TerminalPane.tsx +++ b/src/renderer/src/components/terminal-pane/TerminalPane.tsx @@ -1412,8 +1412,14 @@ export default function TerminalPane({ cancelPendingRenameFrames() }, [cancelPendingRenameFrames]) - useEffect( - () => () => { + const setContainerRef = useCallback( + (node: HTMLDivElement | null): void => { + containerRef.current = node + if (node !== null) { + return + } + // Why: inline title rename focus/blur frames are owned by the terminal + // container; invalidate them when that DOM owner detaches. closeRenameSession() }, [closeRenameSession] @@ -1686,7 +1692,7 @@ export default function TerminalPane({ return ( <>