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

Is it bug or feature? #4

Open
0fuz opened this issue Apr 8, 2024 · 3 comments
Open

Is it bug or feature? #4

0fuz opened this issue Apr 8, 2024 · 3 comments

Comments

@0fuz
Copy link

0fuz commented Apr 8, 2024

Hello, question about dropdown list of allowed files for chat context. When using claude-3 and opened .gitlab-ci.yml but not selected on dropdown menu, those chat tells how to edit .yml to do what needed. But im not allowed such file for it.

Only one idea why this happens: any currently active file tab automatically inserted into chat context.

Tried to start new chat and ask about .yml and it writes "what now" and "what to edit"

@julian-shalaby
Copy link
Owner

This is a feature. Finding the right balance between automatically including context and leaving context as a manual selection process has been a bit non trivial.

If I leave all unselected by default, this strays away from the standard of code assistants, its a bit annoying having to constantly select the file you're on when you ask questions, and it could be confusing to new users who pick this up and just want to start seeing results right away. If I select all by default, its also quite annoying having to deselect files all the time, and inexperienced users seem to basically just flood the context all the time by not deselecting files.

I figured since this is a code assistant, you will almost always be referencing some kind of code file, and that will likely be the file you have currently focused at the very least. I am open to suggestions here on making this experience better if you have any

@0fuz
Copy link
Author

0fuz commented Apr 9, 2024

Its good feature in general. But were a bit not obvious.

Seems configuration files, like semi-prod .env might be opened and shared with cloud models. I understand that having production .env on dev project not correct but anyway its will present, depending on team size and development flow.

What about one-time notice about "currently opened file is auto added to chat context, so dont chat when opened files which might have confidential data like .env and etc"?

And what about mark current file on dropdown menu of files with icon like "auto added to chat context". Currently its just gray font, maybe "(i) with a notice"

@julian-shalaby
Copy link
Owner

Agreed .env and known sensitive files have no reason to be read in as defaults. Will add this to a list of files to not just read in by default.

A 1 time notice to the user is also a good idea. I find getting all the "here's how it works" aspects of chatwindow to the user is a bit challenging, but I do want to find a way to easily get these kind of points to the user without having them open my user guide. If find there is too much to just throw at the user in 1 intro paragraph, but there's a lot the user really should know before/when using the plugin. I am adding a Q&A slash command to my next version which runs RAG on my documentation. May also add something similar to a "take a tour" like feature, if I can think of a fairly presentable way to.

Agreed the grey overlay is not intuitive. Its another 1 of those features where I just couldn't think of a great solution so far. An (i) icon I will consider and probably mess with in dev bit. Good idea, but I want to make sure it won't get annoying seeing it all the time after using the plugin for a long time.

Great points and I appreciate the feedback btw. Helps clean up the plugin a lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants