Skip to content

bug: clarify docs that copilot LSP is only a requirement if NES is enabled #331

Description

@dwf

Did you check docs and existing issues?

  • I have read all the sidekick.nvim docs
  • I have updated the plugin to the latest version before submitting this issue
  • I have searched the existing issues of sidekick.nvim
  • I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)

0.12.3

Operating system/version

Linux

Describe the bug

This is deducible from the docs but not stated explicitly under "Requirements": Copilot LSP installation is not required with NES disabled, i.e. to only use the CLI tool functionality. The docs do state that the CLI functionality does not depend on Copilot at all, and is usable with NES disabled, but this is inconsistent with "Requirements".

As it stands, this is causing nixpkgs to list copilot-language-server as a hard dependency, which requires one to then globally allow unfree packages or allowlist copilot-language-server.

Steps To Reproduce

N/A, it's a documentation issue

Expected Behavior

The documentation should clearly state under what circumstances/usages the Copilot LSP is required.

Repro

vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    { "folke/sidekick.nvim", opts = {} },
    -- add any other plugins here
  },
})

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions