Preciser Status on each Website (Inactive/active) about codeium-chrome HOT 7 OPEN

I think items 1. 2. and 3. would be appreciated! I think stuff like adding the number of completions accepted we might want our own design team to first make a spec.

from codeium-chrome.

Thanks @fortenforge @pqn
I finished 3 on my side. Considering how to implement 2, if the popup page is immutable, I can add the option on the contextMenu like this:

But I'm blocked by how to disable the functionality of Codeium completion after injection (as the page is already loaded). Two approaches:

1. Write clean-up functions for every possible editor to unregister the completion provider. It's a little complicated and time-consuming, I may need help from you.
2. Ignore the message (completion request) from specific tabs, it sounds like a hack, I don't like it and neither do I know how to do that...

What do you think about it? Do you need me to create a draft PR with current progress and have a discussion?

By the way, I found Colab supports their built-in completion now. It might not be a good way to overwrite the Monaco completion for it.

from codeium-chrome.

Can we write the same code twice, once in each implementations (CodeMirror and Monaco), to be able to read some state change caused by the context menu? Then if Codeium is disabled behave as a noop.

By the way, I found Colab supports their built-in completion now. It might not be a good way to overwrite the Monaco completion for it.

If this is true for everyone then we can disable it.

from codeium-chrome.

If this is true for everyone then we can disable it.

Not for every user, but only some eligible ones: https://blog.google/technology/developers/google-colab-ai-coding-features/

I'm just wondering how to handle this.

from codeium-chrome.

For Colab let's leave it on for now in that case. Not sure how Monaco will choose between them but as long as the user gets some completions it's probably okay either way.

from codeium-chrome.

Hi @pqn @fortenforge, Thanks for your help.

I tried to implement 1 and 3 in this draft #51 I'm not certain whether I understand your idea correctly so I created that PR early for tests and discussion. The code is dirty now, I will clean it up further if needed. Please let me know if I misunderstand anything.

And also, should I also push the implementation of 3 in the same PR, or is it more comfortable to review and handle the code so far? Either is fine to me.

from codeium-chrome.

Thanks! I think it's fine to keep 3 in the same PR, it's not unmanageably sized right now.

from codeium-chrome.