Support locally installed docker-langserver (or configure option for specifying docker lsp binary)

[rubin55]

I noticed this extension blindly downloads docker-langserver to the extensions/work folder. In my case, I have docker-langserver installed, and I don't want extenstions downloading their own language server binaries.

Could you implement one of two things:

1: Search $PATH for docker-langserver, don't install if found
or
2: A configuration option which allows you to specify a local binary path, which then disables downloading docker-langserver, indicating you want to use some other binary as lsp

[rubin55]

For posterity, I was able to use my own docker-langserver LSP with the following settings in settings.json:

  "file_types": {
    "Dockerfile": ["Dockerfile", "Dockerfile.*", "*.Dockerfile"]
  },

  "languages": {
    "Dockerfile": {
      "language_servers": ["dockerfile-language-server"]
    },
  }

  "lsp": {
    "dockerfile-language-server": {
      "binary": {
        "path": "/usr/bin/docker-langserver",
        "arguments": ["--stdio"]
      }
    }
  }

[rcjsuen]

For posterity, I was able to use my own docker-langserver LSP with the following settings in settings.json

Thank you for sharing this, @rubin55. Zed will still download the language server even if it detects this in the settings or...?

[rubin55]

No, it will not download:

ls "$HOME/.local/share/zed/extensions/work/dockerfile"
# empty dir

[d1y]

Welcome PR, I don't have this requirement 👀
You can refer to a similar approach to customize a binary

https://github.com/zed-extensions/dart/blob/4b93c2e7e13cac173ee66ac06650369fcb46cc2d/src/dart.rs#L14-L43