Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jan 25, 2025

Pull in cline/cline#1294


Important

Fixes unnecessary terminal creation in TerminalManager.ts by reusing available terminals with matching cwd.

  • Behavior:
    • Fixes issue in getOrCreateTerminal() in TerminalManager.ts where unnecessary new terminals were created.
    • Now checks for a matching terminal with the same cwd and reuses it if available.
    • If no matching terminal, finds any non-busy terminal and navigates to the desired cwd.
    • Only creates a new terminal if all existing ones are busy.

This description was created by Ellipsis for 68712ea. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Jan 25, 2025

🦋 Changeset detected

Latest commit: 68712ea

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
roo-cline Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mrubens mrubens merged commit 2dfe2c5 into main Jan 25, 2025
6 checks passed
@mrubens mrubens deleted the fix_multiple_terminals_created branch January 25, 2025 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants