-
Notifications
You must be signed in to change notification settings - Fork 2.9k
feat: add current file context automatically to chat #5670
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
Conversation
✅ Deploy Preview for continuedev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great! We've gone back and forth on this feature in the past but I think it's worth experimenting with again.
That being said, I think this should be configured as a UI setting, not an IDE setting.
@RomneyDa recently added a new section for experimental settings in https://github.com/continuedev/continue/blob/main/gui/src/pages/config/UserSettingsForm.tsx, so let's move this in there and then it should be good to go 👍
My other comments were just a few small nitpicks.
gui/src/components/mainInput/TipTapEditor/useMainEditorWebviewListeners.ts
Outdated
Show resolved
Hide resolved
gui/src/components/mainInput/TipTapEditor/useMainEditorWebviewListeners.ts
Outdated
Show resolved
Hide resolved
gui/src/components/mainInput/TipTapEditor/utils/insertContent.ts
Outdated
Show resolved
Hide resolved
- rename to `addFileContext` to `useCurrentFileAsContext`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making those updates! It looks like you moved the setting to the UI, but I was thinking specifically of placing it under the new "Experimental Settings" section. I think you just need to merge main and move the toggle to that section and then we're good to go 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thanks for cleaning up the UI while you were editing the experimental section! Appreciate your patience on the back and forth requests here, looking forward to getting this in next pre-release 🚀
Description
This PR adds the ability to have the current file context added to a new chat.
I really like this feature from when I used Augment and wanted it in Continue.
In this PR:
addFileContext
for the extensionCloses #5457
Checklist
Screenshots
feat.mp4
Tests
[ What tests were added or updated to ensure the changes work as expected? ]
Summary by mrge
Added a setting to automatically include the current file context as a removable mention when starting a new chat.
addFileContext
setting in VS Code extension.