按ctrl+鼠标左键跳转时，我想吧build目录去掉

[314599782]

如果没有build目录就可以直接跳转过去，现在跳转总是会先弹框，在选择 很难受

[314599782]

是builtin.lua这个文件 ，可不可以不出现他

[tomlau10]

應該是需要改 luals 代碼的了
我試出1種方式，可以排除掉 builtin.lua，並做到你想要的效果
但我不知道是否最佳做法 😕

具體在這個 file 的這個位置

lua-language-server/script/core/definition.lua

Lines 161 to 168 in 2a48dee

	for _, src in ipairs(defs) do
	if src.type == 'global' then
	goto CONTINUE
	end
	local root = guide.getRoot(src)
	if not root then
	goto CONTINUE
	end

line 168 下邊增加

        -- ignore definitions in extenstion's builtin.lua
        if guide.getUri(src):match("extensions/.+/builtin.lua$") then
            goto CONTINUE
        end

• 大概意思是如果對應 source object 的 file uri path，是包含 builtin.lua 就忽略他
• 我額外加上 extensions/.+/ 是想減少誤判，比如說自己 project 剛好又有個 builtin.lua
  如果忽略掉的話，就會出現誤判
• 同時也不能單純忽略整個 extensions/ 文件夾，因為有一些第3方 meta 文件，確實是想要跳轉過去的

---

你可以測試後開 pr 讓插件作者看看，又或者你自己本地先改來用著

[tomlau10]

另外上邊我是在 macos 上測試的
我不確定 windows 上的 uri path separator 是否一樣用 / (windows 下有可能是 \)
你可能得加 print log 看看 😂

又或者 match pattern 中，不要包含 / / \ 以避開跨平台的問題
=> :match("extensions.+builtin.lua$")

[sumneko]

暂时没有想到好的方法标记一个文件有这样的特殊行为

[tomlau10]

issue 作者遇到的情況，是因為他對 data table 加了 table<K, V> 定義
他有在另1個 issue 提過的: #2700
他增加 table<K, V> 是為了 data table 能用上 missing-field 之類的 diagnostics

而這個 table 定義本身是在 builtin.lua 的 ---@class table
=> vm.getDefs() 按某種 logic 就會將 class table 也加到 result list
簡單的重現 test code

---@class Data
---@field name string

---@type table<string, Data>
local Datas = {
    id1 = {
        name = "id1",
    },
    id2 = {
        name = "id2",
    },
}

print(Datas.id1) --< 對 `.id1` 做 get definition，會同時出現 builtin.lua 的定義

---

對於這種 本身是屬於 table field 的字段
會否可以在 core/definition 的 forloop logic 中，假如 任意1個 push 入 results[] 的 src.type == 'field'
=> 就將 所有 res.type ~= 'field' 的都移除掉呢 🤔 ?

• 相等於如果 defs list 存在 table field，就忽略任何其他的 annotation 所引伸的 def 了
• 這個方式也 不需要 針對指定文件做任何標記

基於 #2839 的測試，這 comment 提出做法會引起其他問題 => 不可行。。。

具體來說是在這個位置:

lua-language-server/script/core/definition.lua

Lines 161 to 227 in 2fe2ff3

	for _, src in ipairs(defs) do
	if src.type == 'global' then
	goto CONTINUE
	end
	local root = guide.getRoot(src)
	if not root then
	goto CONTINUE
	end
	if src.type == 'self' then
	goto CONTINUE
	end
	src = src.field or src.method or src
	if src.type == 'getindex'
	or src.type == 'setindex'
	or src.type == 'tableindex' then
	src = src.index
	if not src then
	goto CONTINUE
	end
	if not guide.isLiteral(src) then
	goto CONTINUE
	end
	else
	if guide.isLiteral(src)
	and src.type ~= 'function'
	and src.type ~= 'doc.type.function'
	and src.type ~= 'doc.type.table' then
	goto CONTINUE
	end
	end
	if src.type == 'doc.class' then
	src = src.class
	end
	if src.type == 'doc.alias' then
	src = src.alias
	end
	if src.type == 'doc.enum' then
	src = src.enum
	end
	if src.type == 'doc.type.field' then
	src = src.name
	end
	if src.type == 'doc.class.name'
	or src.type == 'doc.alias.name'
	or src.type == 'doc.enum.name' then
	if source.type ~= 'doc.type.name'
	and source.type ~= 'doc.extends.name'
	and source.type ~= 'doc.see.name'
	and source.type ~= 'doc.class.name'
	and source.type ~= 'doc.alias.name' then
	goto CONTINUE
	end
	end
	if src.type == 'doc.generic.name' then
	goto CONTINUE
	end
	if src.type == 'doc.param' then
	goto CONTINUE
	end
	results[#results+1] = {
	target = src,
	uri = root.uri,
	source = source,
	}
	::CONTINUE::
	end

改成類似以下

    local hasField = false
    for _, src in ipairs(defs) do
        ...
        hasField = hasField or src.type == 'field'
        results[#results+1] = {
            target = src,
            uri    = root.uri,
            source = source,
        }
        ::CONTINUE::
    end

    if #results == 0 then
        return nil
    end

    if hasField then
        -- keep only results that are 'field'
        for i = #results, 1, -1 do
            local res = results[i].target
            if res.type ~= 'field' then
                table.remove(results, i)
            end
        end
    end

試了一下能解決 issue 作者遇到的問題，但我不確定會否引起其他問題

[sumneko]

那看起来是个bug

[tomlau10]

新提議的方式: #2839 (comment)

在處理 goto definition 時，如果

• src 的 root uri 是 meta directory 內的 builtin.lua
• && src type 不是 doc.class
• => 忽略

lua-language-server/script/core/definition.lua

Lines 161 to 171 in c9d8193

	for _, src in ipairs(defs) do
	if src.type == 'global' then
	goto CONTINUE
	end
	local root = guide.getRoot(src)
	if not root then
	goto CONTINUE
	end
	if src.type == 'self' then
	goto CONTINUE
	end

改成

    local metaUriPrefix = furi.encode(fs.absolute(fs.path(METAPATH)):string())
    for _, src in ipairs(defs) do
        ...
        -- after line 171
        -- ignore meta builtin.lua definitions if not doc.class
        if src.type ~= 'doc.class'
        and root.uri:find(metaUriPrefix, 1, true)
        and root.uri:match("/builtin.lua$")
        then
            goto CONTINUE
        end

• 這裡的目標主要還是想忽略 builtin.lua 那個 table<K, V> 的 popup
• 但同時想允許 ctrl click string / number 等 built-in class 可以跳轉到 builtin.lua