feat: trigger parameter hints instead of suggest on pos args (#243)

* dev: disable aggressive positional arguments completion

* feat: trigger parameter hints instead of suggest on pos args

* dev: update snapshot

crates/tinymist-query/src/upstream/complete.rs

@@ -1029,7 +1029,9 @@ impl<'a, 'w> CompletionContext<'a, 'w> {
             // command on a suggestion to "manually" retrigger suggest after inserting one
             //
             // todo: only vscode and neovim (0.9.1) support this
-            command: Some("editor.action.triggerSuggest"),
+            command: snippet
+                .contains("${")
+                .then_some("editor.action.triggerSuggest"),
             ..Completion::default()
         });
     }
@@ -1181,7 +1183,7 @@ impl<'a, 'w> CompletionContext<'a, 'w> {
         let mut command = None;
         if parens && matches!(value, Value::Func(_)) {
             if let Value::Func(func) = value {
-                command = Some("editor.action.triggerSuggest");
+                command = Some("editor.action.triggerParameterHints");
                 if func
                     .params()
                     .is_some_and(|params| params.iter().all(|param| param.name == "self"))

crates/tinymist-query/src/upstream/complete/ext.rs

@@ -313,7 +313,7 @@ impl<'a, 'w> CompletionContext<'a, 'w> {
                     kind: kind.clone(),
                     label_detail: ty_detail,
                     // todo: only vscode and neovim (0.9.1) support this
-                    command: Some("editor.action.triggerSuggest"),
+                    command: Some("editor.action.triggerParameterHints"),
                     ..Default::default()
                 };
 
@@ -641,13 +641,7 @@ pub fn param_completions<'a>(
                 apply: Some(eco_format!("{}: ${{}}", param.name)),
                 detail: Some(plain_docs_sentence(&param.docs)),
                 label_detail: None,
-                // todo: only vscode and neovim (0.9.1) support this
-                //
-                // VS Code doesn't do that... Auto triggering suggestion only happens on typing
-                // (word starts or trigger characters). However, you can use
-                // editor.action.triggerSuggest as command on a suggestion to
-                // "manually" retrigger suggest after inserting one
-                command: Some("editor.action.triggerSuggest"),
+                command: Some("tinymist.triggerNamedCompletion"),
                 ..Completion::default()
             };
             match param.infer_type {
@@ -742,8 +736,7 @@ fn type_completion(
                 label: f.clone(),
                 apply: Some(eco_format!("{}: ${{}}", f)),
                 detail: docs.map(Into::into),
-                // todo: only vscode and neovim (0.9.1) support this
-                command: Some("editor.action.triggerSuggest"),
+                command: Some("tinymist.triggerNamedCompletion"),
                 ..Completion::default()
             });
         }

crates/tinymist/src/server/lsp_init.rs

@@ -434,9 +434,7 @@ impl Init {
                     // setting.
                     trigger_characters: Some(vec![
                         String::from("#"),
-                        String::from("("),
                         String::from("."),
-                        String::from(","),
                         String::from(":"),
                         String::from("/"),
                         String::from("\""),

editors/vscode/src/extension.ts

@@ -157,6 +157,9 @@ async function startClient(context: ExtensionContext): Promise<void> {
         commands.registerCommand("tinymist.showSummary", () => commandShowSummary(context)),
         commands.registerCommand("tinymist.showSymbolView", () => commandShowSymbolView(context)),
         commands.registerCommand("tinymist.profileCurrentFile", () => commandShowTrace(context)),
+        // We would like to define it at the server side, but it is not possible for now.
+        // https://github.com/microsoft/language-server-protocol/issues/1117
+        commands.registerCommand("tinymist.triggerNamedCompletion", triggerNamedCompletion),
         commands.registerCommand("tinymist.showLog", () => {
             if (client) {
                 client.outputChannel.show();
@@ -668,6 +671,11 @@ function determineVscodeTheme(): any {
     }
 }
 
+function triggerNamedCompletion() {
+    vscode.commands.executeCommand("editor.action.triggerSuggest");
+    vscode.commands.executeCommand("editor.action.triggerParameterHints");
+}
+
 // "tinymist.hoverPeriscope": {
 //     "title": "Show preview document in periscope mode on hovering",
 //     "description": "In VSCode, enable compile status meaning that the extension will show the compilation status in the status bar. Since neovim and helix don't have a such feature, it is disabled by default at the language server lebel.",

tests/e2e/main.rs

@@ -374,7 +374,7 @@ fn e2e() {
         });
 
         let hash = replay_log(&tinymist_binary, &root.join("neovim"));
-        insta::assert_snapshot!(hash, @"siphash128_13:8a283a742935063715fbc8af6bff2b88");
+        insta::assert_snapshot!(hash, @"siphash128_13:69e46cf64247dc4715e1bd46b33906df");
     }
 
     {
@@ -385,7 +385,7 @@ fn e2e() {
         });
 
         let hash = replay_log(&tinymist_binary, &root.join("vscode"));
-        insta::assert_snapshot!(hash, @"siphash128_13:ab459e29e562f52a4594eb784b592341");
+        insta::assert_snapshot!(hash, @"siphash128_13:f6fc76ff19ea6754c8e8a47cc721c929");
     }
 }