fix(ui): stop terminal auto-select from stealing copilot input focus

[waleedlatif1]

Summary

• Remove focusTerminal() call from terminal's auto-select effect so new console entries don't steal focus from the copilot textarea
• Terminal still visually selects the latest entry — focus is only moved on explicit user interaction (clicking an entry or keyboard nav)

Type of Change

• Bug fix

Testing

Tested manually

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 16, 2026 7:41pm

[cursor]

PR Summary

Medium Risk
Terminal change is straightforward, but the duplicated callback declarations and removal of the catch wrapper around stopGeneration() could introduce build errors or unhandled promise rejections depending on stopGeneration’s return type.

Overview
Prevents the terminal’s auto-select-on-new-logs effect from calling focusTerminal(), so selecting the newest log entry no longer steals focus from the Copilot textarea.

Also updates the chat stop-generation callbacks in home.tsx and the workflow panel.tsx (including duplicated handle*StopGeneration declarations in the diff) to call stopGeneration() directly rather than wrapping it in void ...catch(...).

^{Reviewed by Cursor Bugbot for commit 4677f01. Configure here.}

[greptile-apps]

Greptile Summary

Removes focusTerminal() from the terminal's auto-select useEffect, preventing the terminal from stealing keyboard focus from the copilot textarea whenever new console entries arrive. The terminal still visually highlights the latest entry; focus is only moved on explicit user interaction (clicking an entry or keyboard navigation). The focusTerminal reference is correctly removed from the effect's dependency array as well.

Confidence Score: 5/5

Safe to merge — minimal, targeted change with no functional regression risk.

The change removes a single focusTerminal() call and its corresponding dep from an auto-select effect. All explicit user-interaction paths (handleSelectEntry, navigateToEntry) still call focusTerminal(), preserving keyboard navigation. No logic, state, or rendering is otherwise affected.

No files require special attention.

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/terminal/terminal.tsx	Removes focusTerminal() call and its dep from the auto-select effect — surgical, correct fix for the focus-stealing bug.
apps/sim/app/workspace/[workspaceId]/home/home.tsx	Included via a secondary commit in the PR branch (v0.6.29 login/posthog improvements); no issues identified.
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/panel.tsx	Included via a secondary commit in the PR branch (v0.6.29 login/posthog improvements); no issues identified.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[New console entry arrives] --> B{autoSelectEnabled?}
    B -- No --> Z[No-op]
    B -- Yes --> C[Set selectedEntryId to latest entry]
    C --> D[Expand parent nodes if needed]
    D --> E[Terminal visually highlights entry]
    E --> F{Before fix: focusTerminal called?}
    F -- Yes BEFORE --> G[terminalRef.focus - Steals focus from copilot textarea]
    F -- No AFTER --> H[Focus stays on copilot textarea]
    I[User clicks entry] --> J[handleSelectEntry]
    J --> K[focusTerminal called explicitly]
    K --> L[Terminal receives focus for keyboard nav]

Loading

_{Reviews (1): Last reviewed commit: "fix(ui): stop terminal auto-select from ..." | Re-trigger Greptile}