fix: completion of bdd prefixes optimized

- If you press CTRL+SPACE after a bdd prefix the completion list is shown without typing any other key.
- if the cursor is at the bdd prefix, other bdd prefix are on the top of the completion list and if you select a bdd prefix only the old prefix is overwritten

Author: d-biehl

packages/language_server/src/robotcode/language_server/robotframework/parts/completion.py

@@ -578,7 +578,9 @@ async def create_testcase_settings_completion_items(self, range: Optional[Range]
             for setting in settings
         ]
 
-    async def create_bdd_prefix_completion_items(self, range: Optional[Range]) -> List[CompletionItem]:
+    async def create_bdd_prefix_completion_items(
+        self, range: Optional[Range], at_top: bool = False, with_space: bool = True
+    ) -> List[CompletionItem]:
         prefixes = {"Given", "When", "Then", "And", "But"}
 
         if self.namespace.languages is not None:
@@ -589,9 +591,11 @@ async def create_bdd_prefix_completion_items(self, range: Optional[Range]) -> Li
                 label=prefix,
                 kind=CompletionItemKind.UNIT,
                 detail="BDD Prefix",
-                sort_text=f"080_{prefix}",
+                sort_text=f"000_{prefix}" if at_top else f"080_{prefix}",
                 insert_text_format=InsertTextFormat.PLAIN_TEXT,
-                text_edit=TextEdit(range=range, new_text=f"{prefix} ") if range is not None else None,
+                text_edit=TextEdit(range=range, new_text=prefix + (" " if with_space else ""))
+                if range is not None
+                else None,
             )
             for prefix in prefixes
         ]
@@ -674,18 +678,29 @@ async def create_keyword_completion_items(
 
         has_bdd = False
         bdd_token = None
+        only_bdd = False
 
         if token is not None:
             old_token = token
             bdd_token, token = self.split_bdd_prefix(self.namespace, token)
 
+            if (
+                bdd_token is None
+                and token is not None
+                and self.is_bdd_token(self.namespace, token)
+                and position.character > range_from_token(token).end.character
+            ):
+                bdd_token = token
+                token = None
+                only_bdd = True
+
             if token is not None and token.value == "":
                 token = None
 
             if bdd_token is not None and position.character > range_from_token(bdd_token).end.character:
                 has_bdd = True
 
-            if not has_bdd and token is None:
+            if not has_bdd and token is not None:
                 token = old_token
 
         if token is not None:
@@ -877,12 +892,20 @@ def enumerate_indexes(s: str, c: str) -> Iterator[int]:
                 )
             )
 
-        if add_bdd_prefixes and not has_bdd:
-            result += await self.create_bdd_prefix_completion_items(
-                range_from_token(token) if token is not None else None
-            )
+        if add_bdd_prefixes and not (has_bdd or only_bdd):
+            bdd_range = r
+            at_top = False
+            with_space = True
+            if bdd_token is not None and (
+                position in range_from_token(bdd_token) or position == range_from_token(bdd_token).end.character
+            ):
+                at_top = True
+                with_space = False
+                bdd_range = range_from_token(bdd_token)
+
+            result += await self.create_bdd_prefix_completion_items(bdd_range, at_top, with_space)
 
-        if add_reserverd:
+        if add_reserverd and not (has_bdd or only_bdd):
             for k in get_reserved_keywords():
                 result.append(
                     CompletionItem(

packages/language_server/src/robotcode/language_server/robotframework/parts/model_helper.py

@@ -613,11 +613,9 @@ def is_bdd_token(cls, namespace: Namespace, token: Token) -> bool:
             return bool(bdd_match)
 
         parts = token.value.split()
-        if len(parts) < 2:
-            return False
 
-        for index in range(1, len(parts)):
-            prefix = " ".join(parts[:index]).title()
+        for index in range(0, len(parts)):
+            prefix = " ".join(parts[: index + 1]).title()
 
             if prefix.title() in (
                 namespace.languages.bdd_prefixes if namespace.languages is not None else DEFAULT_BDD_PREFIXES