feat(tools/spxls): implement textDocument/completion

[aofei]

Updates #1059

[qiniu-prow]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[aofei]

当时在 https://github.com/goplus/builder/pull/1162/files#diff-0c6691e1482b8fb7fab79c84814456d80895bd823bc74fee320c1dc82c70ea9f 添加这个处理，是因为有语法错误时 astFile 为 nil 对吧？

这个 pr 调整了 compile 的逻辑，现在有语法错误时也能正常拿到 astFile 了。

[nighca]

添加这个处理，是因为有语法错误时 astFile 为 nil 对吧？

是的

这个 pr 调整了 compile 的逻辑，现在有语法错误时也能正常拿到 astFile 了。

棒 👍

Copilot reviewed 6 out of 11 changed files in this pull request and generated 2 comments.

Files not reviewed (5)

• tools/spxls/README.md: Evaluated as low risk
• tools/spxls/internal/server/command.go: Evaluated as low risk
• tools/spxls/internal/server/command_test.go: Evaluated as low risk
• tools/spxls/internal/server/compile.go: Evaluated as low risk
• tools/spxls/internal/server/completion_test.go: Evaluated as low risk

Comments suppressed due to low confidence (3)

tools/spxls/internal/server/util.go:148

• The function name expandGopOverloadedMethod is now consistent with the naming convention used elsewhere in the codebase.

func expandGopOverloadedMethod(method *types.Func) []*types.Func {

tools/spxls/internal/server/util.go:3

• The bitwise OR operation to convert the first character to lowercase might not work correctly for non-ASCII characters. Consider using a more robust method for handling Unicode characters.

return string(s[0]|32) + s[1:]

tools/spxls/internal/server/server.go:83

• Ensure that the CompletionItem struct is properly initialized and used within the textDocumentCompletion function.

return s.textDocumentCompletion(&params)

Copilot reviewed 6 out of 11 changed files in this pull request and generated 1 comment.

Files not reviewed (5)

• tools/spxls/README.md: Evaluated as low risk
• tools/spxls/internal/server/command.go: Evaluated as low risk
• tools/spxls/internal/server/server.go: Evaluated as low risk
• tools/spxls/internal/server/util.go: Evaluated as low risk
• tools/spxls/internal/server/compile.go: Evaluated as low risk

[qiniu-ci]

This PR has been deployed to the preview environment. You can explore it using the preview URL.

Warning

Please note that deployments in the preview environment are temporary and will be automatically cleaned up after a certain period. Make sure to explore it before it is removed. For any questions, contact the Go+ Builder team.