Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CoPilot should be decoupled from zed #6708

Closed
1 task done
assapir opened this issue Jan 25, 2024 · 6 comments
Closed
1 task done

CoPilot should be decoupled from zed #6708

assapir opened this issue Jan 25, 2024 · 6 comments
Labels
ai Improvement related to Assistant, Copilot, or other AI features copilot AI feedback for Copilot enhancement [core label] inline completion Umbrella label for Copilot, Supermaven, etc. completions

Comments

@assapir
Copy link

assapir commented Jan 25, 2024

Check for existing issues

  • Completed

Describe the feature

As of now, GitHub CoPilot is coupled into Zed. This makes Zed very opinionated about what AI tool for code to use instead of using some extension mechanism (I know this is WIP) to allow other alternatives such as Tabnine.
This also aligns with your public roadmap about AI support in the editor - it will not require an OpenAI key but expose the requirement component.

I can help with integrating Tabnine if you need me.

This is related to #4504 #4726 and others

If applicable, add mockups / screenshots to help present your vision of the feature

No response

@assapir assapir added admin read Pending admin review enhancement [core label] triage Maintainer needs to classify the issue labels Jan 25, 2024
@raystriker
Copy link

I think it would also be nice to have the option to use a locally run LLM, they're getting pretty good and do not rely on the internet.

@YurySolovyov
Copy link

https://github.com/ollama/ollama Could be a great candidate/target that provides common api to run local LLMs.

@JosephTLyons JosephTLyons added ai Improvement related to Assistant, Copilot, or other AI features copilot AI feedback for Copilot and removed triage Maintainer needs to classify the issue admin read Pending admin review labels Jan 25, 2024
@TwistingTwists
Copy link

TwistingTwists commented Mar 1, 2024

Also being discussed in #4424

@patgd
Copy link

patgd commented Mar 24, 2024

Does Zed have a roadmap I can check for this FR?

@JosephTLyons JosephTLyons added the inline completion Umbrella label for Copilot, Supermaven, etc. completions label Jun 12, 2024
@rgbkrk
Copy link
Member

rgbkrk commented Jun 12, 2024

There's now an InlineCompletionProvider trait that can be implemented in a PR to add more providers, like has been done for Supermaven.

@notpeter
Copy link
Member

notpeter commented Jun 18, 2024

I think this issue has been resolved. We now support multiple providers. Discussion (and 👍 voting) should be done in issues for each provider:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ai Improvement related to Assistant, Copilot, or other AI features copilot AI feedback for Copilot enhancement [core label] inline completion Umbrella label for Copilot, Supermaven, etc. completions
Projects
None yet
Development

No branches or pull requests

8 participants