Skip to content

edit prediction: Drop project state after a project goes away#48191

Merged
osiewicz merged 2 commits intomainfrom
drop-copilot-in-separate-windows
Feb 2, 2026
Merged

edit prediction: Drop project state after a project goes away#48191
osiewicz merged 2 commits intomainfrom
drop-copilot-in-separate-windows

Conversation

@osiewicz
Copy link
Member

@osiewicz osiewicz commented Feb 2, 2026

Closes #48097

Release Notes:

  • Fixed Copilot instances not being cleared up after their window is closed.
  • Copilot edit prediction provider now respects disable_ai setting.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 2, 2026
@osiewicz osiewicz force-pushed the drop-copilot-in-separate-windows branch from 708ea88 to 159c790 Compare February 2, 2026 15:33
@KieranP
Copy link

KieranP commented Feb 2, 2026

Also, if at all possible, ideally Zed would only spin up one Copilot LSP per Zed parent process, otherwise, if I have 5 projects open, then I have 5 Github LSPs, each taking up to 2 GB of RAM :-( As a result, I'm having to limit myself to 2 projects open at a time, otherwise things start to hit swap, and everything slows down.

@osiewicz osiewicz merged commit 9867f16 into main Feb 2, 2026
43 of 45 checks passed
@osiewicz osiewicz deleted the drop-copilot-in-separate-windows branch February 2, 2026 18:47
@osiewicz
Copy link
Member Author

osiewicz commented Feb 2, 2026

/cherry-pick preview

github-actions bot pushed a commit that referenced this pull request Feb 2, 2026
Closes #48097

Release Notes:

- Fixed Copilot instances not being cleared up after their window is
closed.
- Copilot edit prediction provider now respects `disable_ai` setting.
zed-zippy bot added a commit that referenced this pull request Feb 2, 2026
… (cherry-pick to preview) (#48206)

Cherry-pick of #48191 to preview

----
Closes #48097

Release Notes:

- Fixed Copilot instances not being cleared up after their window is
closed.
- Copilot edit prediction provider now respects `disable_ai` setting.

Co-authored-by: Piotr Osiewicz <24362066+osiewicz@users.noreply.github.com>
adb-sh pushed a commit to adb-sh/zed that referenced this pull request Feb 5, 2026
…dustries#48191)

Closes zed-industries#48097

Release Notes:

- Fixed Copilot instances not being cleared up after their window is
closed.
- Copilot edit prediction provider now respects `disable_ai` setting.
@nemeott
Copy link

nemeott commented Feb 21, 2026

I am still getting this problem on NixOS with Zed 0.223.3 for some reason. When I start Zed it shows 3 Copilot instances open, and killing them reduces memory usage. I'll try updating when I can but the issue was marked as fixed in 0.223.3. Might be worth looking into again? I can open another issue if needed.

$ ps aux | grep copilot-language-server | wc -l
3
$ pkill -f copilot-language-server
$ ps aux | grep copilot-language-server | wc -l
1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Zed spinning up multiple copilot-language-server instances and not closing them. Using multiple GB of RAM.

3 participants