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 remains active despite being turned off in local settings #9335

Open
1 task done
lum1n opened this issue Mar 14, 2024 · 10 comments
Open
1 task done

Copilot remains active despite being turned off in local settings #9335

lum1n opened this issue Mar 14, 2024 · 10 comments
Labels
ai Improvement related to Assistant, Copilot, or other AI features bug [core label] copilot AI feedback for Copilot inline completion Umbrella label for Copilot, Supermaven, etc. completions regression A bug that was introduced in a recent release setting Feedback for preferences, configuration, etc

Comments

@lum1n
Copy link

lum1n commented Mar 14, 2024

Check for existing issues

  • Completed

Describe the bug / provide steps to reproduce it

When opening an existing project with local zed settings and copilot turned off, it does not recognize being turned off before opening local settings.

.zed/settings.json:

{
    "features": {
        "copilot": false
    }
}

~/.config/zed/setting.json:

{
    "features": {
        "copilot": true
    }
}

Environment

Zed: 0.127.1 (preview)
MacOs: 14.4
Macbook M1 Max

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

No response

If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.

No response

@lum1n lum1n added admin read Pending admin review bug [core label] triage Maintainer needs to classify the issue labels Mar 14, 2024
@Moshyfawn Moshyfawn added regression A bug that was introduced in a recent release setting Feedback for preferences, configuration, etc copilot AI feedback for Copilot ai Improvement related to Assistant, Copilot, or other AI features labels Mar 14, 2024
@Moshyfawn
Copy link
Member

Reproduction steps:

  1. Open a project.
    1.1. Make sure that Copilot is enabled in your global settings.
  2. Create .zed/settings.json.
{
  "show_copilot_suggestions": false
}
  1. Close the settings.json file.
  2. Restart Zed.
  3. See that the Copilot icon is enabled.

@JosephTLyons JosephTLyons removed triage Maintainer needs to classify the issue admin read Pending admin review labels Mar 14, 2024
@JosephTLyons
Copy link
Collaborator

cc @SomeoneToIgnore, since this issue was closed out:

@JosephTLyons
Copy link
Collaborator

@lum1n, does the global setting to turn copilot suggestions off work? I'm curious if this is a bug with the logic, or if maybe the local settings just don't support this setting yet. I think at the moment, unless something has changed since I last knew, local settings don't include all global settings, and maybe this copilot one is one of those not supported.

@SomeoneToIgnore
Copy link
Contributor

I fail to reproduce it on the latest Preview with default settings and show_copilot_suggestions:
https://github.com/zed-industries/zed/assets/2690773/241e5d8a-d744-45fd-a81f-45a1c9a9243f

so wonder what am I doing wrong?

@Moshyfawn
Copy link
Member

Moshyfawn commented Mar 14, 2024

so wonder what am I doing wrong?

It looks like you left the settings.json file open before restarting the editor.

@SomeoneToIgnore
Copy link
Contributor

I have tried a few more times, closing all files did not help to repro this either:

Screen.Recording.2024-03-14.at.22.10.41.mov

also, tried cmd-q instead of the restart action, but that also did not change much.

What I did notice, is that despite currently opened project files had the disabled copilot, opening "internal" Zed files like settings.json had copilot enabled, switching tabs changed the copilot icon:

Untitled.mov

So, there seems to be some worktree-related oddity with Copilot and settings.

But this issue seems to be about files from the same worktree ("project")?

@lum1n
Copy link
Author

lum1n commented Mar 14, 2024

@lum1n, does the global setting to turn copilot suggestions off work? I'm curious if this is a bug with the logic, or if maybe the local settings just don't support this setting yet. I think at the moment, unless something has changed since I last knew, local settings don't include all global settings, and maybe this copilot one is one of those not supported.

I actually haven't tried disabling suggestions only. In my case, i am not allowed to use AI on this specific codebase, so i find it a bit unsettling that i suddenly get suggestions even though i have disabled copilot locally. If i open .zed/settings.json once, it gets disabled. But i have to remember doing that every time. And today it suddenly activated mid session as well. So, have to disable it globally.

Also, disabling suggestions only, i suppose that will still leave copilot "activated" for the project?

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

I am facing the same issue.

Before a few updates ago, the following setting would disable copilot and hide its icon from the status bar:

{
  "features": {
    // Show Copilot icon in status bar
    "copilot": false
  },
}

This seems not to have an effect anymore as it no longer exists in Zed's Default Settings json file.

@notpeter
Copy link
Member

notpeter commented Jun 19, 2024

Good morning all, could you please alter your settings to use the following values and report back.

rename features.copilot to inline_completions

{
  "features": {
    "inline_completions": "none"
  },
}

rename show_copilot_suggestions to show_inline_completions

{
  "show_inline_completions": false
}

We attempted to rename these values in a backwards compatible way (leveraging a serde alias feature) and ultimately that caused a host of new issues for users so we've decided to just deprecate the old settings values altogether. This may or may not solve your issue, but you should migrate your settings and cease usage of features.copiolot, show_copilot_suggestions as they will be no-ops going forward.

Similarly, please move any language_overrides into languages.

We understand this a PITA and are using this as a learning opportunity to better for how we handle migrations and settings compatibility going forward. Please bear with us.

Context:

@tv42
Copy link

tv42 commented Jun 30, 2024

To get rid of the copilot icon in status bar, try

"features": {
  "inline_completion_provider": "none"
}

from #7121 (comment)

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 bug [core label] copilot AI feedback for Copilot inline completion Umbrella label for Copilot, Supermaven, etc. completions regression A bug that was introduced in a recent release setting Feedback for preferences, configuration, etc
Projects
None yet
Development

No branches or pull requests

7 participants