Change `root_dir` to allow function, like `lspconfig`.

[magnusriga]

copilot.nvim currently uses the current working directory at time of loading the plugin, as the root_dir for its language server (vim.loop.cwd()).

I was hoping to make it work like other language server configs defined by nvim-lspconfig, where a function is accepted as root_dir, which runs each time the LSP client attaches to a new buffer.

I tried passing a function to server_opts_overrides.root_dir, but this causes the plugin to crash.

Happy to add a PR, if helpful.

Thanks for the awesome work on this plugin.

[magnusriga]

I added this temporary fix, on my end:

 vim.api.nvim_create_autocmd("BufReadPre", {
  pattern = "*",
  callback = function()
    local root_dir = require("lspconfig.util").root_pattern(".git")(vim.api.nvim_buf_get_name(0))
    local clients = vim.lsp.get_clients()
    for _, client in ipairs(clients) do
      if client.name == "copilot" then
        local _config = vim.deepcopy(client.config)
        _config.root_dir = root_dir
        client:stop()
        vim.lsp.start(_config)
      end
    end
  end,
})

It is a pretty trivial fix to the repo, to ensure a function is exexuted to calculate the root whenever attaching. Just shout out if a PR is useful.

[magnusriga]

@zbirenbaum This is related to #213

[AntoineGS]

Hey I did quite a bit of work on workspaces and root_dir in the past few days and I believe I have resolved the issue that forced a rollback in #213, so I have added the configuration back (though slightly different).
Do you mind testing with this repo and branch?
It does include a couple of other PRs that are not merged yet.

[magnusriga]

Sorry for the delay, looks like it is merged and works now.

Thanks!