Question: Better highlighter command support for Dockerfile with treesitter about sonokai HOT 6 CLOSED

Here is the video, you could see wget is highlighted as green and then becoming white

from sonokai.

Oh, you have the LSP server for Dockerfile active apparently, and that LSP server has semantic tokens enabled. LSP semantic token are providing extra highlighting in addition to Tree-sitter.

Support for LSP semantic tokens was added in Sonokai in #86.

In sainnhe/everforest#119, I tried to determine whether we could customize those color schemes ever further to improve the experience with LSP semantic tokens, but my conclusion was that LSP semantic tokens highlights sometimes conflict with Tree-sitter pretty badly. From my experience, this feature degrades the highlights more than it improves them. For instance, I wasn't able to enhance highlights for the Go language due to the priority that the LSP semantic token highlights were given over Tree-sitter's.

Thanks a lot for the video 👍 I will check whether we can improve the Dockerfile file type specifically for LSP semantic tokens. In case that's not possible, it's probably best to disable that feature.

from sonokai.

I was able to disable this feature in dockerls with the following lspconfig:

After disable tthat feature, it's working well now, thanks

from sonokai.

Yes, you simply need to enable Tree-sitter for the "Dockerfile" filetype with :TSInstall dockerfile.

Here is how it looks for me:

from sonokai.

from sonokai.

I just investigated the issue, and unfortunately LSP semantic highlights override Tree-sitter entirely in this case.

Tree-sitter has smart queries which allows detecting and highlighting sh/bash inside Dockerfiles. You can see it in the screenshot below as:

Treesitter
  - @variable.bash links to @variable bash    <!-- bash detected -->
  - @constant.bash links to @constant bash

Unfortunately, as soon as LSP semantic tokens are enabled, everything becomes a "parameter" instead, and the embedded highlights are overridden:

Semantic Tokens
  - @lsp.type.parameter.dockerfile links to @parameter priority: 125  <!-- higher priority than Tree-sitter -->

Screenshot:

That's why all the colors disappear in your video. I see the exact same behavior in my Neovim with the dockerls LSP server enabled.

I was able to disable this feature in dockerls with the following lspconfig:

local lspconfig = require "lspconfig"

lspconfig.dockerls.setup {
  on_attach = function(client, bufnr)
    client.server_capabilities.semanticTokensProvider = nil
  end,
}

from sonokai.