Merge 06bc7bac6694d63459b39844782aca50ddee8d6e into 275012d7c624be85173315286752888e20996072

crates/typst-ide/src/analyze.rs

@@ -1,5 +1,3 @@
-use std::collections::HashSet;
-
 use comemo::Track;
 use ecow::{eco_vec, EcoString, EcoVec};
 use typst::foundations::{Label, Styles, Value};
@@ -68,22 +66,14 @@ pub fn analyze_import(world: &dyn IdeWorld, source: &LinkedNode) -> Option<Value
 /// - All labels and descriptions for them, if available
 /// - A split offset: All labels before this offset belong to nodes, all after
 ///   belong to a bibliography.
-///
-/// Note: When multiple labels in the document have the same identifier,
-/// this only returns the first one.
 pub fn analyze_labels(
     document: &PagedDocument,
 ) -> (Vec<(Label, Option<EcoString>)>, usize) {
     let mut output = vec![];
-    let mut seen_labels = HashSet::new();
 
     // Labels in the document.
     for elem in document.introspector.all() {
         let Some(label) = elem.label() else { continue };
-        if !seen_labels.insert(label) {
-            continue;
-        }
-
         let details = elem
             .to_packed::<FigureElem>()
             .and_then(|figure| match figure.caption.as_option() {

crates/typst-ide/src/complete.rs

@@ -76,7 +76,7 @@ pub struct Completion {
 }
 
 /// A kind of item that can be completed.
-#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
+#[derive(Debug, Clone, Serialize, Deserialize)]
 #[serde(rename_all = "kebab-case")]
 pub enum CompletionKind {
     /// A syntactical structure.
@@ -130,14 +130,7 @@ fn complete_markup(ctx: &mut CompletionContext) -> bool {
         return true;
     }
 
-    // Start of a reference: "@|".
-    if ctx.leaf.kind() == SyntaxKind::Text && ctx.before.ends_with("@") {
-        ctx.from = ctx.cursor;
-        ctx.label_completions();
-        return true;
-    }
-
-    // An existing reference: "@he|".
+    // Start of a reference: "@|" or "@he|".
     if ctx.leaf.kind() == SyntaxKind::RefMarker {
         ctx.from = ctx.leaf.offset() + 1;
         ctx.label_completions();
@@ -1571,7 +1564,7 @@ mod tests {
 
     use typst::layout::PagedDocument;
 
-    use super::{autocomplete, Completion, CompletionKind};
+    use super::{autocomplete, Completion};
     use crate::tests::{FilePos, TestWorld, WorldLike};
 
     /// Quote a string.
@@ -1651,19 +1644,6 @@ mod tests {
         test_with_doc(world, pos, doc.as_ref())
     }
 
-    #[track_caller]
-    fn test_with_addition(
-        initial_text: &str,
-        addition: &str,
-        pos: impl FilePos,
-    ) -> Response {
-        let mut world = TestWorld::new(initial_text);
-        let doc = typst::compile(&world).output.ok();
-        let end = world.main.text().len();
-        world.main.edit(end..end, addition);
-        test_with_doc(&world, pos, doc.as_ref())
-    }
-
     #[track_caller]
     fn test_with_doc(
         world: impl WorldLike,
@@ -1729,30 +1709,6 @@ mod tests {
             .must_exclude(["bib"]);
     }
 
-    #[test]
-    fn test_autocomplete_ref_function() {
-        test_with_addition("x<test>", " #ref(<)", -2).must_include(["test"]);
-    }
-
-    #[test]
-    fn test_autocomplete_ref_shorthand() {
-        test_with_addition("x<test>", " @", -1).must_include(["test"]);
-    }
-
-    #[test]
-    fn test_autocomplete_ref_shorthand_with_partial_identifier() {
-        test_with_addition("x<test>", " @te", -1).must_include(["test"]);
-    }
-
-    #[test]
-    fn test_autocomplete_ref_identical_labels_returns_single_completion() {
-        let result = test_with_addition("x<test> y<test>", " @t", -1);
-        let completions = result.completions();
-        let label_count =
-            completions.iter().filter(|c| c.kind == CompletionKind::Label).count();
-        assert_eq!(label_count, 1);
-    }
-
     /// Test what kind of brackets we autocomplete for function calls depending
     /// on the function and existing parens.
     #[test]