Fix user input commands being echoed twice in terminal #9959
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.
End-user friendly description of the problem this fixes or functionality this introduces.
Fixed an issue where user-entered bash commands would appear twice in the terminal. Now, when you type a command in the terminal, it will only be displayed once, providing a cleaner and more consistent experience.
Summarize what the PR does, explaining any non-trivial design decisions.
This PR fixes a bug where user input bash commands were being echoed twice in the terminal:
The solution modifies the
renderCommand
function inuse-terminal.ts
to skip rendering user input commands that have already been displayed as the user typed them. This is done by adding anisUserInput
parameter to the function that, when true, will skip rendering commands of type "input".Design decisions:
Link of any specific issues this addresses:
Fixes #6661