fix: check if item contains brackets before defering to semantic token

Author: Saghen

lua/blink/cmp/accept/brackets.lua

@@ -48,13 +48,6 @@ function brackets.add_brackets(filetype, item)
   if brackets.has_brackets_in_front(text_edit, brackets_for_filetype[1]) then
     return 'skipped', text_edit, #brackets_for_filetype[1]
   end
-  -- check if configuration incidates we should skip
-  if not brackets.should_run_resolution(filetype, 'kind') then return 'check_semantic_token', text_edit, 0 end
-  -- not a function, skip
-  local CompletionItemKind = require('blink.cmp.types').CompletionItemKind
-  if item.kind ~= CompletionItemKind.Function and item.kind ~= CompletionItemKind.Method then
-    return 'check_semantic_token', text_edit, 0
-  end
 
   -- if the item already contains the brackets, conservatively skip adding brackets
   -- todo: won't work for snippets when the brackets_for_filetype is { '{', '}' }
@@ -63,6 +56,14 @@ function brackets.add_brackets(filetype, item)
     return 'skipped', text_edit, 0
   end
 
+  -- check if configuration incidates we should skip
+  if not brackets.should_run_resolution(filetype, 'kind') then return 'check_semantic_token', text_edit, 0 end
+  -- not a function, skip
+  local CompletionItemKind = require('blink.cmp.types').CompletionItemKind
+  if item.kind ~= CompletionItemKind.Function and item.kind ~= CompletionItemKind.Method then
+    return 'check_semantic_token', text_edit, 0
+  end
+
   text_edit = vim.deepcopy(text_edit)
   -- For snippets, we add the cursor position between the brackets as the last placeholder
   if item.insertTextFormat == vim.lsp.protocol.InsertTextFormat.Snippet then