-
Notifications
You must be signed in to change notification settings - Fork 43
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
Clarify transmission of directory listings? #36
Comments
Hey! Thanks for sharing feedback with the community! |
Good point, we should probably make this optional as some form of config setting. |
For what it's worth, for now I'm using terminal-copilot via a script that always runs it from an empty directory in It's a very useful command. Thank you again. |
@rafpaf Want to try to make a small PR to make it more like you would prefer it to be? Eg. making the directory listing optional? or asking the user the first time to include it or not? |
@rafpaf Any other thoughts on how we can make the tool even better? |
Thank you for this fantastic piece of software!
I appreciated seeing the thoughtful note in your README about sensitive information. I saw this line:
This made me think that no sensitive information would be transmitted to OpenAI if I avoided those flags. However, when I run terminal-copilot with the
-v
flag, I can see that it transmits a listing of the current directory as part of the prompt. You could clarify that this transmission will happen, even if no special flags are used. It might also be useful to let users disable this transmission. Another reason it is sometimes useful to disable this feature is that if the directory listing is very long, it can lead to an unexpectedopenai.error.InvalidRequestError
.The text was updated successfully, but these errors were encountered: