Disallow renaming of non-local structs

Author: alibektas

crates/ide-db/src/rename.rs

@@ -22,7 +22,7 @@
 //! Our current behavior is ¯\_(ツ)_/¯.
 use std::fmt;
 
-use base_db::{AnchoredPathBuf, FileId, FileRange};
+use base_db::{AnchoredPathBuf, CrateOrigin, FileId, FileRange};
 use either::Either;
 use hir::{FieldSource, HasSource, InFile, ModuleSource, Semantics};
 use stdx::never;
@@ -77,6 +77,15 @@ impl Definition {
                 bail!("Cannot rename builtin type")
             }
             Definition::SelfType(_) => bail!("Cannot rename `Self`"),
+            Definition::Adt(hir::Adt::Struct(strukt)) => {
+                if !matches!(
+                    strukt.module(sema.db).krate().origin(sema.db),
+                    CrateOrigin::Local { .. }
+                ) {
+                    bail!("Cannot rename a non-local struct.")
+                }
+                rename_reference(sema, Definition::Adt(hir::Adt::Struct(strukt)), new_name)
+            }
             def => rename_reference(sema, def, new_name),
         }
     }

crates/ide/src/rename.rs

@@ -2487,4 +2487,18 @@ fn main() {
 ",
         )
     }
+
+    #[test]
+    fn disallow_renaming_for_non_local_struct() {
+        check(
+            "Baz",
+            r#"
+//- /lib.rs crate:lib new_source_root:library
+pub struct S$0;
+//- /main.rs crate:main deps:lib new_source_root:local
+use lib::S;
+"#,
+            "error: Cannot rename a non-local struct.",
+        );
+    }
 }