Exit code isn't shown for the last run command in Python's shell integration enabled REPL

[Tyriar]

Testing #22878

For this state:

Run recent command shows this:

🐛 c should say exit code 1, it's also odd that it doesn't show the path with a ~

[Tyriar]

This is happening because it's considered executing at the time:

https://github.com/microsoft/vscode/blob/5a9a7ac5892fdfacbe9150511ee43197b6c3faa7/src/vs/workbench/contrib/terminal/browser/terminalRunRecentQuickPick.ts#L71-L75

[Tyriar]

The reason it's executing is because the E sequence is in the next command:

�]633;D;1��]633;A�>>> �]633;B��]633;C��]633;E;b�" [[13,10,84,114,97,99,101,98,97,99,107,32,40,109,111,115,116,32,114,101,99,101,110,116,32,99,97,108,108,32,108,97,115,116,41,58,13,10,32,32,70,105,108,101,32,34,60,115,116,100,105,110,62,34,44,32,108,105,110,101,32,49,44,32,105,110,32,60,109,111,100,117,108,101,62,13,10,78,97,109,101,69,114,114,111,114,58,32,110,97,109,101,32,39,98,39,32,105,115,32,110,111,116,32,100,101,102,105,110,101,100,13,10,27,93,54,51,51,59,68,59,49,7,27,93,54,51,51,59,65,7,62,62,62,32,27,93,54,51,51,59,66,7,27,93,54,51,51,59,67,7,27,93,54,51,51,59,69,59,98,7]]

E should come after C (executing) of the command, but it's part of the following command because it's after D (command finished). Since Python adds C early, this should go just before the D sequence.

[Tyriar]

This is also why the command would be 1 off; since the command line for the proceeding command is used.

[anthonykim1]

Hello, Daniel Imms (@Tyriar). Thank you so much for looking into this!!

I followed your recommendation and injected E sequence right before D sequence, and it solved both the exit code being back up issue , and the rerun command being backed up issue as well :)
Thank you!

Screen.Recording.2024-03-01.at.2.02.56.PM.mov