-
Notifications
You must be signed in to change notification settings - Fork 28.8k
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
Move cmd start escape to end of prompt #210443
Conversation
Since the mode prompt 'appears to the left of the regular prompt', declaring that the start of the cmd causes the rest of the prompt to be part of the command.
Seems to be a more recent issue than #185907, implying #209946 is what actually broke it? Appears that before #209946, escape E was successfully specifying the command, but now C is before E the E no longer takes effect. However disregarding the change to E, #185907 still needs fixing in my opinion to place C in the right place. @Tyriar what testing steps took place for #209946 that I can use to reproduce what swapping the ordering of C and E did (when |
@grgar the swapping of C and E makes the full command line available to the upcoming shellIntegration extension API. You can test it by uncommenting this part and then inspect the event object in devtools: It should show the correct command line with a confidence of I didn't do much testing outside of powershell and I tried just now to add a fish mode prompt but I don't think I could get it working. So if you can verify that your proposal works then we're good to merge 👌 |
Move cmd start escape to end of prompt
Thanks @Tyriar, that was indeed very convenient test code! Much appreciated showing me these additional test steps and thanks for merging. |
@grgar awesome! Thanks a lot for the contribution |
#185907 moves the cmd start escape to the end of the mode prompt, but since the mode prompt ‘appears to the left of the regular prompt’ that seems too early: declaring that the start of the cmd causes the rest of the prompt to be part of the command in the recent commands list.
Moves
__vsc_fish_cmd_start
to the end offish_prompt
so the prompt is not part of the command. Tested that the prompt no longer appears in the recent commands and running the command from the recent list no longer tries to execute the prompt as the command.