Matthieu Coudron

but with lua I am not sure you need to go through the Plug route anyway, you could straightaway document: `vim.keymap.set("n", "gs", function () require'llm'.accept_suggestion() end)`

> Not sure about checking the PATH for llm-ls though. I believe most programs dont hardcode the paths to their software though, so they look in PATH, that's what it's...

I feel like it could even deprecate the lsp.bin_path config option such that the change doesn't increase the code maintainance. Instead of setting lsp.bin_path, A user could just update the...

testing this branch I get on startup: ``` Gp: response from the config.providers.copilot.secret command { "bash", "-c", "cat ~/.config/github-copilot/hosts.json | sed -e 's/.*oauth_token...//;s/\".*//'" } is empty ``` because I dont...

answering to myself, my agents got silently discarded because I had removed the model and/or prompt. It would be best to warn the user rather than discard the agent IMO...

I've commented here https://github.com/Robitx/gp.nvim/issues/120 about the agents getting removed not to pollute this PR. I noticed that endpoints for image and whisper are hardcoded to openai. I have whisper working...

@mecattaf not ollama but local-ai. Should work the same though you can read about my experience at https://github.com/Robitx/gp.nvim/issues/122

@Robitx can we merge this ? I would like to contribute to gp.nvim a lot more but while I can deal with stacked PRs, this is a pretty fundamental change...

I would rather make the plugin more approachable with things such as https://github.com/Robitx/gp.nvim/pull/125, more logging etc. The merge of the multiple providers support is a very good news. Now it...

@mecattaf that would be awesome. We talked a bit about it at https://github.com/Robitx/gp.nvim/pull/125#issuecomment-2231291915 . It's important to discuss the implementation with @Robitx first.