fix duplicate fields in table hover

Author: sumneko

changelog.md

@@ -2,6 +2,7 @@
 
 ## 3.6.1
 * `FIX` wrong diagnostics for `pcall` and `xpcall`
+* `FIX` duplicate fields in table hover
 
 ## 3.6.0
 `2022-11-8`

script/core/hover/table.lua

@@ -94,15 +94,24 @@ local function getVisibleKeyMap(source, fields)
     local uri  = guide.getUri(source)
     local keys = {}
     local map  = {}
+    local ignored = {}
     for _, field in ipairs(fields) do
         local key = vm.viewKey(field, uri)
-        if vm.isVisible(source, field) then
+        local rawKey = guide.getKeyName(field)
+        if rawKey and rawKey ~= key then
+            ignored[rawKey] = true
+            map[rawKey] = nil
+        end
+        if  not ignored[key]
+        and vm.isVisible(source, field) then
             if key and not map[key] then
                 map[key] = true
-                keys[#keys+1] = key
             end
         end
     end
+    for key in pairs(map) do
+        keys[#keys+1] = key
+    end
     table.sort(keys, function (a, b)
         if a == b then
             return false

script/vm/infer.lua

@@ -542,11 +542,17 @@ function vm.viewKey(source, uri)
         end
     end
     if source.type == 'tableindex' then
-        local name = vm.getKeyName(source)
+        local index = source.index
+        local name = vm.getKeyName(index)
         if not name then
             return nil
         end
-        local key = util.viewLiteral(name)
+        local key
+        if index.type == 'string' then
+            key = util.viewString(name, index[2])
+        else
+            key = util.viewLiteral(name)
+        end
         return ('[%s]'):format(key), name
     end
     if source.type == 'tableexp' then

test/hover/init.lua

@@ -1714,7 +1714,7 @@ local <?t?> = {
 local t: {
     x: string = "e",
     y: string = "f",
-    ["z"]: string = "g",
+    ['z']: string = "g",
     [10]: string = "d",
     [1]: string = "a",
     [2]: string = "b",
@@ -2415,3 +2415,18 @@ local t: { [string]: string }[] {
     foo: unknown,
 }
 ]]
+
+TEST [[
+local t = {
+    ['x'] = 1,
+    ['y'] = 2,
+}
+
+print(t.x, <?t?>.y)
+]]
+[[
+local t: {
+    ['x']: integer = 1,
+    ['y']: integer = 2,
+}
+]]