Skip to content

Fix Go semantic token initialization in Neovim - #25

Closed
user3301 wants to merge 2 commits into
masterfrom
fix/gopls-semantic-token-legend
Closed

user3301 wants to merge 2 commits into
masterfrom
fix/gopls-semantic-token-legend

Conversation

@user3301

@user3301 user3301 commented Oct 5, 2026

Copy link
Copy Markdown
Owner

Enable semantic tokens during gopls initialization so it advertises its own token legend instead of relying on LazyVim's incompatible fallback.

Enable semantic tokens during gopls initialization so it advertises its own token legend instead of relying on LazyVim's incompatible fallback.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 343d3984-6409-428d-8816-7df7961af61c

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The added option duplicates existing LazyVim Go-extra configuration, so the PR does not implement its stated fix.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Adds documentation and configuration intended to enable gopls semantic tokens during initialization.

Changes:

  • Adds semanticTokens to gopls initialization options.
  • Documents the semantic-token behavior.
File Description
nvim/​.config/​nvim/​README.md Documents gopls semantic-token initialization.
nvim/​.config/​nvim/​lua/​plugins/​lsp.lua Adds gopls semantic-token initialization options.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread nvim/.config/nvim/lua/plugins/lsp.lua Outdated
Comment on lines +63 to +65
init_options = {
semanticTokens = true,
},
The installed LazyVim checkout was 28db03f9, not the locked 99970099 revision. The locked Go extra already supplies semanticTokens in init_options. Remove the duplicate configuration and document restoring the locked plugin instead.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 343d3984-6409-428d-8816-7df7961af61c
@user3301 user3301 closed this Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants