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

feat(chat-ui): add smart apply to chat panel #3112

Merged
merged 47 commits into from
Oct 26, 2024

Conversation

Sma1lboy
Copy link
Collaborator

@Sma1lboy Sma1lboy commented Sep 10, 2024

Demo (last update: 10/17/2024):

Screen.Recording.2024-10-17.at.03.19.27.mov

Three general way using smart apply

  • adding tests
  • adding comments
  • insert similar code

Steps

  • Implement a smart apply button in the chat view
  • Use fuzzy search to get line range for insertion or replacement
  • If fuzzy search cannot find an appropriate line range, use the Tabby backend to prompt for a range to insert
  • Refactor chat useful functions
  • Refactor Workspace operation requests into WorkspaceFeature in VSCode
  • Keep provideSmartApplyEdit, consider still using it in situations where the current smartApply is not working
  • Change the prompt for provideSmartApplyLineRange for the Tabby backend

@Sma1lboy Sma1lboy marked this pull request as ready for review September 19, 2024 00:49
@Sma1lboy Sma1lboy requested a review from wsxiaoys September 24, 2024 02:05
also adding default data config

fix: remove test
@wsxiaoys wsxiaoys requested a review from icycodes October 21, 2024 02:34
@icycodes
Copy link
Member

@Sma1lboy, could you please rebase this pull request on the main branch and resolve conflicts?

@Sma1lboy
Copy link
Collaborator Author

@Sma1lboy, could you please rebase this pull request on the main branch and resolve conflicts?

Hi @icycodes, I just rebased the code.

@icycodes
Copy link
Member

Hi @Sma1lboy, I have created a PR at Sma1lboy#2 that makes some updates to the code style. Please check it out. Thank you.

@icycodes icycodes merged commit 7656965 into TabbyML:main Oct 26, 2024
4 checks passed
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

Successfully merging this pull request may close these issues.

3 participants