feat: Add the ability to include cleared prompts in the history, toggled by a KV-persisted command palette item (resolves #11489)#11710
Open
ariane-emory wants to merge 21 commits intoanomalyco:devfrom
Conversation
When users press Ctrl+C to clear the input field, the current prompt is now saved to history before clearing. This allows users to navigate back to cleared prompts using arrow keys, preventing loss of work. Addresses anomalyco#11489
Adds a toggle command in the System category that allows users to enable
or disable saving cleared prompts to history. The feature is disabled by
default to preserve existing behavior.
When enabled via the command palette ("Include cleared prompts in history"),
pressing Ctrl+C will save the current prompt to history before clearing it,
allowing users to navigate back with arrow keys.
The setting persists in kv.json.
Contributor
|
The following comment was made by an LLM, it may be inaccurate: No duplicate PRs found |
ariane-emory
changed the title
ariane-emory
changed the title
ariane-emory
changed the title
ariane-emory
changed the title
ariane-emory
changed the title
…emory/opencode into feat/canceled-prompts-in-history
Avoid adding duplicate entries to prompt history when the same input is appended multiple times (e.g., clearing with ctrl+c then restoring via history navigation and clearing again).
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
What does this PR do?
I saw @sai-sy's feature request and decided to take a stab at it, figuring that it would likely be fairly straightforward forwards. Sure enough it was, and here it is:
This PR a toggling command in the System category of the
command_listthat allows users to enable or disable saving cleared prompts to history. The feature is disabled by default to preserve existing behaviour that our current users may have learned to expect.When enabled via the
command_list(using a menu item listed therein as either "Include cleared prompts in history" / "Don't include cleared prompts in history", depending on it's current state),pressing Ctrl+C will save the current prompt to history before clearing it, allowing users to navigate back with arrow keys.The current of the state of the setting is persistent to the
kv.jsonfile, and so no structural change to the configuration files whatsoever is required.Resolves #11489
How did you verify your code works?
Manual testing,
bun type check, andbun test.