merge pls msg is anoying

thank you!

@hrsh7th @lewis6991 This iterates while adding new keys, which doesn't seem to cause any issues but is technically undefined afaik. vim.tbl_add_reverse_lookup uses vim.tbl_keys to get around this issue.

Yes, I just realised that. Hopefully this doesn't cause an issue but fixing should be easy if it does.

I could confirm this causes some kv pairs not added on my end. I added a print statement just after the loop assignments:

diff --git a/lua/cmp/types/lsp.lua b/lua/cmp/types/lsp.lua
index 65d6301..2dfa192 100644
--- a/lua/cmp/types/lsp.lua
+++ b/lua/cmp/types/lsp.lua
@@ -205,6 +205,7 @@ lsp.CompletionItemKind = {
 for k, v in pairs(lsp.CompletionItemKind) do
   lsp.CompletionItemKind[v] = k
 end
+vim.print(lsp.CompletionItemKind)
 
 ---@class lsp.internal.CompletionItemDefaults
 ---@field public commitCharacters? string[]

And occasionally it shows that some pairs are not correctly added:

Reference at index 18 is missing this time. The missing kind is random, and I've seen kinds Function and Text could also disappear in completion window.