fix(lsp): less agressive completion triggers (#17225)

Closes https://github.com/denoland/deno/issues/17056
Closes https://github.com/denoland/deno/issues/17055

cli/lsp/tsc.rs

@@ -2007,6 +2007,10 @@ impl CompletionEntryDetails {
       documentation,
       command,
       additional_text_edits,
+      // NOTE(bartlomieju): it's not entirely clear to me why we need to do that,
+      // but when `completionItem/resolve` is called, we get a list of commit chars
+      // even though we might have returned an empty list in `completion` request.
+      commit_characters: None,
       ..original_item.clone()
     })
   }

cli/tests/lsp_tests.rs

@@ -3958,12 +3958,6 @@ export class DuckConfig {
             },
             "newText": "import { foo } from \"./b.ts\";\n\n"
           }
-        ],
-        "commitCharacters": [
-          ".",
-          ",",
-          ";",
-          "("
         ]
       }))
     );
@@ -4077,13 +4071,7 @@ export class DuckConfig {
         "sortText": "11",
         "filterText": "type=\"$1\"",
         "insertText": "type=\"$1\"",
-        "insertTextFormat": 2,
-        "commitCharacters": [
-          ".",
-          ",",
-          ";",
-          "("
-        ]
+        "insertTextFormat": 2
       }))
     );
   }