-
Notifications
You must be signed in to change notification settings - Fork 49
feat: experimental inline buffer chat #142
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
Merged
Merged
Changes from all commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
4e3bda1
WIP: very early ground break
sudo-tee bac518f
feat: new buffer context types
sudo-tee d9b8076
feat: quick chat ask for input if no message
sudo-tee b70e379
fix: git_diff not returning value
sudo-tee 1ec3cd5
fix: range transformation for selection
sudo-tee 32245ba
fix: promise with nil arguments
sudo-tee 752f595
refactor(context): modularize context gathering and add search/replac…
sudo-tee a8179c0
refactor(quick_chat): extract search/replace and spinner into separat…
sudo-tee 8e6bac6
feat(search_replace): support empty SEARCH block as insert operation
sudo-tee dc5e376
refactor(quick_chat): improve prompt structure and formatting
sudo-tee d5ec089
fix(quick_chat): remove duplicated instructions
sudo-tee d6ffb06
feat: optimize SEARCH/REPLACE quick chat for concise context and outp…
sudo-tee f7a70fc
feat(quick_chat): cheat for position of window for snacks
sudo-tee cdd55e4
feat(quick_chat): update default conf
sudo-tee bd19f34
feat(ui): use floating window for CursorSpinner
sudo-tee e76825f
fix(quick_chat): session deletion
sudo-tee f742006
feat: add cancel keymap for cancelling quick_chat session
sudo-tee 6a08f8c
feat: update prompt and configs
sudo-tee ca5065d
feat: update docs toi add images of quick chat
sudo-tee ed7ff03
chore:remove test file
sudo-tee 6478340
chore: fix typo
sudo-tee 7e8eade
fix: typo in readme
sudo-tee af4e12f
fix(quick_chat): rename default_prompt to instructions
sudo-tee cbf6c07
fix: typo
sudo-tee a421014
feat: refactor context system for modularity and quick chat improvements
sudo-tee 22de13c
chore: remove rogue import
sudo-tee eb54371
chore: remove duplicated line
sudo-tee ce0a37c
fix(quick_chat_context): don't re-indent selection
sudo-tee 56bfccc
fix(context): don't trim the current line in context
sudo-tee 867e430
fix: typo
sudo-tee 990776c
fix(quick_chat): improve patch block parsing and application logic
sudo-tee 9e31367
feat: narrower cursor context works better
sudo-tee 6609853
fix(api): show selection/line context in Quick Chat input prompt
sudo-tee ed2e6ae
feat: replace search/replace with raw code generation in quick chat
sudo-tee 7f92287
fix(quick_chat): improve code extraction and raw replace instructions
sudo-tee 1fdfb40
fix: wrong property name in README
sudo-tee 893efbd
chore: remove .values for accessing config
sudo-tee 1477681
fix: message condition
sudo-tee f612caa
fix: delta context not using current context from state
sudo-tee cf88ab6
feat(context): add sent_at/sent_at_mtime file context state tracking
sudo-tee b5d86fd
fix: send message not working in last merge
sudo-tee 08bf3f9
refactor(context): remove unused legacy context management and test f…
sudo-tee 27db2e4
feat: add support for multi-range diagnostics with selection-aware fi…
sudo-tee ff6c9d1
feat: better errors in promise finally
sudo-tee c8863b0
perf: debounce context loading to improve performance
sudo-tee a2305a2
fix: return type
sudo-tee File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
The
quick_chatfunction inapi.luaon lines 126-128 concatenates an array of message parts withtable.concat(message, ' '). However, this assumes thatmessageis always a table when it's passed as an array. If a user accidentally passes a string that looks like an array or has unexpected structure, this could fail. Consider adding type validation or documenting that message must be a table when passed as multiple arguments.