refactor(tools/spxls): centralize source location handling in compileResult

[aofei]

No description provided.

[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

Copilot reviewed 5 out of 10 changed files in this pull request and generated no comments.

Files not reviewed (5)

• tools/spxls/internal/server/compile.go: Evaluated as low risk
• tools/spxls/internal/server/document.go: Evaluated as low risk
• tools/spxls/internal/server/server.go: Evaluated as low risk
• tools/spxls/internal/server/highlight.go: Evaluated as low risk
• tools/spxls/internal/server/reference.go: Evaluated as low risk

Comments suppressed due to low confidence (1)

tools/spxls/internal/server/rename.go:35

• [nitpick] The function util.ToPtr is used here, but it is not clear what it does from the context. Consider inlining the logic for clarity if it is specific to this use case.

return util.ToPtr(result.rangeForNode(ident)), nil

[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.