Add new completion API

LSP.sublime-settings

@@ -58,17 +58,6 @@
   // Show symbol action links in hover popup if available
   "show_symbol_action_links": false,
 
-  // Request completions for all characters if set to true,
-  // or just after trigger characters only otherwise.
-  "complete_all_chars": true,
-
-  // Controls which hints the completion panel displays
-  // "auto": completion details if available or kind otherwise
-  // "detail": completion details if available
-  // "kind": completion kind if available
-  // "none": completion item label only
-  "completion_hint_type": "auto",
-
   // Disable Sublime Text's explicit and word completion.
   "only_show_lsp_completions": false,
 

docs/features.md

@@ -70,9 +70,7 @@ Add these settings to your Sublime settings, Syntax-specific settings and/or in
 
 ### Package settings (LSP)
 
-* `complete_all_chars` `true` *request completions for all characters, not just trigger characters*
 * `only_show_lsp_completions` `false` *disable sublime word completion and snippets from autocomplete lists*
-* `completion_hint_type` `"auto"` *override automatic completion hints with "detail", "kind" or "none"*
 * `show_references_in_quick_panel` `false` *show symbol references in Sublime's quick panel instead of the bottom panel*
 * `show_view_status` `true` *show permanent language server status in the status bar*
 * `auto_show_diagnostics_panel` `always` (`never`, `saved`) *open the diagnostics panel automatically if there are diagnostics*

plugin/core/completion.py

@@ -1,76 +1,63 @@
-from .protocol import CompletionItemKind, Range
-from .types import Settings
-from .logging import debug
+import sublime
+from .restore_lines import RestoreLines
 from .typing import Tuple, Optional, Dict, List, Union
 
 
-completion_item_kind_names = {v: k for k, v in CompletionItemKind.__dict__.items()}
-
-
-def get_completion_hint(item: dict, settings: Settings) -> Optional[str]:
-    # choose hint based on availability and user preference
-    hint = None
-    if settings.completion_hint_type == "auto":
-        hint = item.get("detail")
-        if not hint:
-            kind = item.get("kind")
-            if kind:
-                hint = completion_item_kind_names[kind]
-    elif settings.completion_hint_type == "detail":
-        hint = item.get("detail")
-    elif settings.completion_hint_type == "kind":
-        kind = item.get("kind")
-        if kind:
-            hint = completion_item_kind_names.get(kind)
-    return hint
-
-
-def format_completion(item: dict, word_col: int, settings: Settings) -> Tuple[str, str]:
-    # Sublime handles snippets automatically, so we don't have to care about insertTextFormat.
-    trigger = item["label"]
-
-    hint = get_completion_hint(item, settings)
-
-    # label is an alternative for insertText if neither textEdit nor insertText is provided
-    replacement = text_edit_text(item, word_col) or item.get("insertText") or trigger
-
-    if replacement[0] != trigger[0]:
-        # fix some common cases when server sends different start on label and replacement.
-        if replacement[0] == '$':
-            trigger = '$' + trigger  # add missing $
-        elif replacement[0] == '-':
-            trigger = '-' + trigger  # add missing -
-        elif trigger[0] == ':':
-            replacement = ':' + replacement  # add missing :
-        elif trigger[0] == '$':
-            trigger = trigger[1:]  # remove leading $
-        elif trigger[0] == ' ' or trigger[0] == '•':
-            trigger = trigger[1:]  # remove clangd insertion indicator
-        else:
-            debug("WARNING: Replacement prefix does not match trigger '{}'".format(trigger))
-
-    if len(replacement) > 0 and replacement[0] == '$':  # sublime needs leading '$' escaped.
-        replacement = '\\$' + replacement[1:]
-    # only return trigger with a hint if available
-    return "\t  ".join((trigger, hint)) if hint else trigger, replacement
-
-
-def text_edit_text(item: dict, word_col: int) -> Optional[str]:
-    text_edit = item.get('textEdit')
-    if text_edit:
-        edit_range, edit_text = text_edit.get("range"), text_edit.get("newText")
-        if edit_range and edit_text:
-            edit_range = Range.from_lsp(edit_range)
-
-            # debug('textEdit from col {}, {} applied at col {}'.format(
-            #     edit_range.start.col, edit_range.end.col, word_col))
-
-            if edit_range.start.col <= word_col:
-                # if edit starts at current word, we can use it.
-                # if edit starts before current word, use the whole thing and we'll fix it up later.
-                return edit_text
-
-    return None
+completion_kinds = {
+    1: (sublime.KIND_ID_MARKUP, "Ξ", "Text"),
+    2: (sublime.KIND_ID_FUNCTION, "λ", "Method"),
+    3: (sublime.KIND_ID_FUNCTION, "λ", "Function"),
+    4: (sublime.KIND_ID_FUNCTION, "c", "Constructor"),
+    5: (sublime.KIND_ID_VARIABLE, "f", "Field"),
+    6: (sublime.KIND_ID_VARIABLE, "v", "Variable"),
+    7: (sublime.KIND_ID_TYPE, "c", "Class"),
+    8: (sublime.KIND_ID_TYPE, "i", "Interface"),
+    9: (sublime.KIND_ID_NAMESPACE, "◪", "Module"),
+    10: (sublime.KIND_ID_VARIABLE, "ρ", "Property"),
+    11: (sublime.KIND_ID_VARIABLE, "u", "Unit"),
+    12: (sublime.KIND_ID_VARIABLE, "ν", "Value"),
+    13: (sublime.KIND_ID_TYPE, "ε", "Enum"),
+    14: (sublime.KIND_ID_KEYWORD, "κ", "Keyword"),
+    15: (sublime.KIND_ID_SNIPPET, "s", "Snippet"),
+    16: (sublime.KIND_ID_AMBIGUOUS, "c", "Color"),
+    17: (sublime.KIND_ID_AMBIGUOUS, "#", "File"),
+    18: (sublime.KIND_ID_AMBIGUOUS, "⇢", "Reference"),
+    19: (sublime.KIND_ID_AMBIGUOUS, "ƒ", "Folder"),
+    20: (sublime.KIND_ID_TYPE, "ε", "EnumMember"),
+    21: (sublime.KIND_ID_VARIABLE, "π", "Constant"),
+    22: (sublime.KIND_ID_TYPE, "s", "Struct"),
+    23: (sublime.KIND_ID_FUNCTION, "e", "Event"),
+    24: (sublime.KIND_ID_KEYWORD, "ο", "Operator"),
+    25: (sublime.KIND_ID_TYPE, "τ", "Type Parameter")
+}
+
+
+def format_completion(item: dict, restore_lines: RestoreLines) -> sublime.CompletionItem:
+    trigger = item.get('label') or ""
+    annotation = item.get('detail') or ""
+    kind = sublime.KIND_AMBIGUOUS
+
+    item_kind = item.get("kind")
+    if item_kind:
+        kind = completion_kinds.get(item_kind, sublime.KIND_AMBIGUOUS)
+
+    is_deprecated = item.get("deprecated", False)
+    if is_deprecated:
+        list_kind = list(kind)
+        list_kind[1] = '⚠'
+        list_kind[2] = "⚠ {} - Deprecated".format(list_kind[2])
+        kind = tuple(list_kind)  # type: ignore
+
+    return sublime.CompletionItem.command_completion(
+        trigger,
+        command="lsp_select_completion_item",
+        args={
+            "item": item,
+            "restore_lines_dict": restore_lines.to_dict()
+        },
+        annotation=annotation,
+        kind=kind
+    )
 
 
 def parse_completion_response(response: Optional[Union[Dict, List]]) -> Tuple[List[Dict], bool]:

plugin/core/protocol.py

@@ -79,6 +79,11 @@ class CompletionItemKind(object):
 completion_item_kinds = list(range(CompletionItemKind.Text, CompletionItemKind.TypeParameter + 1))
 
 
+class InsertTextFormat:
+    PlainText = 1
+    Snippet = 2
+
+
 class DocumentHighlightKind(object):
     Unknown = 0
     Text = 1

plugin/core/restore_lines.py

@@ -0,0 +1,51 @@
+import sublime
+from .typing import List
+
+
+class RestoreLines:
+    def __init__(self):
+        self.saved_lines = []  # type: List[dict]
+
+    def save_lines(self, locations: List[int], view: sublime.View) -> None:
+        change_id = view.change_id()
+
+        for point in locations:
+            line = view.line(point)
+            change_region = (line.begin(), line.end())
+            text = view.substr(line)
+
+            self.saved_lines.append({
+                "change_id": change_id,
+                "change_region": change_region,
+                "text": text,
+                # cursor will be use retore the cursor the te exact position
+                "cursor": point
+            })
+
+    def to_dict(self):
+        return {
+            "saved_lines": self.saved_lines
+        }
+
+    @staticmethod
+    def from_dict(dictionary):
+        restore_lines = RestoreLines()
+        restore_lines.saved_lines = dictionary["saved_lines"]
+        return restore_lines
+
+    def restore_lines(self, edit: sublime.Edit, view: sublime.View) -> None:
+        # restore lines contents
+        # insert back lines from the bottom to top
+        for saved_line in reversed(self.saved_lines):
+            change_id = saved_line['change_id']
+            begin, end = saved_line['change_region']
+            change_region = sublime.Region(begin, end)
+
+            transform_region = view.transform_region_from(change_region, change_id)
+            view.erase(edit, transform_region)
+            view.insert(edit, transform_region.begin(), saved_line['text'])
+
+        # restore old cursor position
+        view.sel().clear()
+        for saved_line in self.saved_lines:
+            view.sel().add(saved_line["cursor"])

plugin/core/sessions.py

@@ -32,7 +32,8 @@ def get_initialize_params(workspace_folders: List[WorkspaceFolder], config: Clie
                 },
                 "completion": {
                     "completionItem": {
-                        "snippetSupport": True
+                        "snippetSupport": True,
+                        "deprecatedSupport": True
                     },
                     "completionItemKind": {
                         "valueSet": completion_item_kinds

plugin/core/settings.py

@@ -75,8 +75,6 @@ def update_settings(settings: Settings, settings_obj: sublime.Settings) -> None:
     settings.show_code_actions_bulb = read_bool_setting(settings_obj, "show_code_actions_bulb", False)
     settings.show_symbol_action_links = read_bool_setting(settings_obj, "show_symbol_action_links", False)
     settings.only_show_lsp_completions = read_bool_setting(settings_obj, "only_show_lsp_completions", False)
-    settings.complete_all_chars = read_bool_setting(settings_obj, "complete_all_chars", True)
-    settings.completion_hint_type = read_str_setting(settings_obj, "completion_hint_type", "auto")
     settings.show_references_in_quick_panel = read_bool_setting(settings_obj, "show_references_in_quick_panel", False)
     settings.disabled_capabilities = read_array_setting(settings_obj, "disabled_capabilities", [])
     settings.log_debug = read_bool_setting(settings_obj, "log_debug", False)

plugin/core/types.py

@@ -22,8 +22,6 @@ def __init__(self) -> None:
         self.diagnostics_gutter_marker = "dot"
         self.show_code_actions_bulb = False
         self.show_symbol_action_links = False
-        self.complete_all_chars = False
-        self.completion_hint_type = "auto"
         self.show_references_in_quick_panel = False
         self.disabled_capabilities = []  # type: List[str]
         self.log_debug = True

stubs/sublime.pyi

@@ -2,7 +2,7 @@
 #
 # NOTE: This dynamically typed stub was automatically generated by stubgen.
 
-from typing import Any, Optional, Callable, Sequence, Tuple, Union, List, Sized
+from typing import Any, Optional, Callable, Sequence, Tuple, Union, List, Sized, Iterator
 
 
 class _LogWriter:
@@ -55,6 +55,7 @@ CLASS_LINE_END = ...  # type: int
 CLASS_EMPTY_LINE = ...  # type: int
 INHIBIT_WORD_COMPLETIONS = ...  # type: int
 INHIBIT_EXPLICIT_COMPLETIONS = ...  # type: int
+DYNAMIC_COMPLETIONS = ...  # type: int
 DIALOG_CANCEL = ...  # type: int
 DIALOG_YES = ...  # type: int
 DIALOG_NO = ...  # type: int
@@ -67,6 +68,24 @@ UI_ELEMENT_OPEN_FILES = ...  # type: int
 LAYOUT_INLINE = ...  # type: int
 LAYOUT_BELOW = ...  # type: int
 LAYOUT_BLOCK = ...  # type: int
+KIND_ID_AMBIGUOUS = ...  # type: int
+KIND_ID_KEYWORD = ...  # type: int
+KIND_ID_TYPE = ...  # type: int
+KIND_ID_FUNCTION = ...  # type: int
+KIND_ID_NAMESPACE = ...  # type: int
+KIND_ID_NAVIGATION = ...  # type: int
+KIND_ID_MARKUP = ...  # type: int
+KIND_ID_VARIABLE = ...  # type: int
+KIND_ID_SNIPPET = ...  # type: int
+KIND_AMBIGUOUS = ...  # type: Tuple[int, str, str]
+KIND_KEYWORD = ...  # type: Tuple[int, str, str]
+KIND_TYPE = ...  # type: Tuple[int, str, str]
+KIND_FUNCTION = ...  # type: Tuple[int, str, str]
+KIND_NAMESPACE = ...  # type: Tuple[int, str, str]
+KIND_NAVIGATION = ...  # type: Tuple[int, str, str]
+KIND_MARKUP = ...  # type: Tuple[int, str, str]
+KIND_VARIABLE = ...  # type: Tuple[int, str, str]
+KIND_SNIPPET = ...  # type: Tuple[int, str, str]
 
 
 class Settings:
@@ -238,6 +257,22 @@ def get_macro() -> Sequence[dict]:
     ...
 
 
+class CompletionItem:
+    @classmethod
+    def command_completion(cls,
+                           trigger: str,
+                           command: str,
+                           args: dict = {},
+                           annotation: str = "",
+                           kind: Tuple[int, str, str] = KIND_AMBIGUOUS
+                           ) -> 'CompletionItem':
+        ...
+
+class CompletionList:
+    def set_completions(self, completions: List[CompletionItem], flags: int = 0) -> None:
+        ...
+
+
 class Window:
     window_id = ...  # type: int
     settings_object = ...  # type: Settings
@@ -478,6 +513,12 @@ class Selection(Sized):
     def __len__(self) -> int:
         ...
 
+    def __iter__(self) -> Iterator[Region]:
+        ...
+
+    def __next__(self) -> Region:
+        ...
+
     def __getitem__(self, index: int) -> Region:
         ...
 
@@ -491,7 +532,7 @@ class Selection(Sized):
     def clear(self) -> None:
         ...
 
-    def add(self, x: Region) -> None:
+    def add(self, x: Union[Region, int]) -> None:
         ...
 
     def add_all(self, regions: Sequence[Region]) -> None: