Skip to content

lsp: Open updated imports in multibuffer after file rename#45110

Merged
gaauwe merged 2 commits intomainfrom
open-project-transaction-for-lsp-edits
Dec 17, 2025
Merged

lsp: Open updated imports in multibuffer after file rename#45110
gaauwe merged 2 commits intomainfrom
open-project-transaction-for-lsp-edits

Conversation

@gaauwe
Copy link
Contributor

@gaauwe gaauwe commented Dec 17, 2025

Fixes an issue where we would update the imports after a file rename in TypeScript, but those changes wouldn't surface anywhere until those buffers were manually opened (#35930 (comment)). In #36681 we already added support for opening a multibuffer with edits, but vtsls has a different flow for renames.

Release Notes:

  • Files with updated imports now open in a multibuffer when renaming or moving TypeScript or JavaScript files

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 17, 2025
@zed-industries-bot
Copy link
Contributor

zed-industries-bot commented Dec 17, 2025

Messages
📖

This PR includes links to the following GitHub Issues: #35930
If this PR aims to close an issue, please include a Closes #ISSUE line at the top of the PR body.

Generated by 🚫 dangerJS against d994a22

@github-project-automation github-project-automation bot moved this to Community Champion PRs in Quality Week – December 2025 Dec 17, 2025
@gaauwe gaauwe merged commit c186877 into main Dec 17, 2025
23 checks passed
@gaauwe gaauwe deleted the open-project-transaction-for-lsp-edits branch December 17, 2025 14:29
@github-project-automation github-project-automation bot moved this from Community Champion PRs to Done in Quality Week – December 2025 Dec 17, 2025
@gaauwe gaauwe self-assigned this Dec 17, 2025
HactarCE pushed a commit that referenced this pull request Dec 17, 2025
Fixes an issue where we would update the imports after a file rename in
TypeScript, but those changes wouldn't surface anywhere until those
buffers were manually opened
(#35930 (comment)).
In #36681 we already added
support for opening a multibuffer with edits, but vtsls has a different
flow for renames.

Release Notes:

- Files with updated imports now open in a multibuffer when renaming or
moving TypeScript or JavaScript files
rtfeldman pushed a commit that referenced this pull request Jan 5, 2026
Fixes an issue where we would update the imports after a file rename in
TypeScript, but those changes wouldn't surface anywhere until those
buffers were manually opened
(#35930 (comment)).
In #36681 we already added
support for opening a multibuffer with edits, but vtsls has a different
flow for renames.

Release Notes:

- Files with updated imports now open in a multibuffer when renaming or
moving TypeScript or JavaScript files
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
…tries#45110)

Fixes an issue where we would update the imports after a file rename in
TypeScript, but those changes wouldn't surface anywhere until those
buffers were manually opened
(zed-industries#35930 (comment)).
In zed-industries#36681 we already added
support for opening a multibuffer with edits, but vtsls has a different
flow for renames.

Release Notes:

- Files with updated imports now open in a multibuffer when renaming or
moving TypeScript or JavaScript files
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
…tries#45110)

Fixes an issue where we would update the imports after a file rename in
TypeScript, but those changes wouldn't surface anywhere until those
buffers were manually opened
(zed-industries#35930 (comment)).
In zed-industries#36681 we already added
support for opening a multibuffer with edits, but vtsls has a different
flow for renames.

Release Notes:

- Files with updated imports now open in a multibuffer when renaming or
moving TypeScript or JavaScript files
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

Development

Successfully merging this pull request may close these issues.

2 participants