diff options
author | Nayeem Rahman <nayeemrmn99@gmail.com> | 2024-03-07 17:27:24 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-03-07 17:27:24 +0000 |
commit | 8df47882c942d3cf5558fee05f02629025e46ae7 (patch) | |
tree | 95d1eb9f755fece90a379ae52386e196c9bf9910 /cli/lsp/tsc.rs | |
parent | 0fdb33c3aa9f4c75d9e15e8a33d9c00116d9052f (diff) |
fix(lsp): don't apply renames to remote modules (#22765)
Diffstat (limited to 'cli/lsp/tsc.rs')
-rw-r--r-- | cli/lsp/tsc.rs | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/cli/lsp/tsc.rs b/cli/lsp/tsc.rs index ce848c210..104024167 100644 --- a/cli/lsp/tsc.rs +++ b/cli/lsp/tsc.rs @@ -2115,8 +2115,13 @@ impl RenameLocations { LspClientUrl, lsp::TextDocumentEdit, > = HashMap::new(); + let mut includes_non_files = false; for location in self.locations.iter() { let specifier = resolve_url(&location.document_span.file_name)?; + if specifier.scheme() != "file" { + includes_non_files = true; + continue; + } let uri = language_server.url_map.normalize_specifier(&specifier)?; let asset_or_doc = language_server.get_asset_or_document(&specifier)?; @@ -2146,6 +2151,10 @@ impl RenameLocations { })); } + if includes_non_files { + language_server.client.show_message(lsp::MessageType::WARNING, "The renamed symbol had references in non-file schemed modules. These have not been modified."); + } + Ok(lsp::WorkspaceEdit { change_annotations: None, changes: None, |