Switch to onDidStartTerminalShellExecution from the obsolete proposed API terminalData* events - #13605
Switch to onDidStartTerminalShellExecution from the obsolete proposed API terminalData* events#13605Garrett Serack (fearthecowboy) wants to merge 4 commits into
onDidStartTerminalShellExecution from the obsolete proposed API terminalData* events#13605Conversation
…d) API terminalData* events
Bob Brown (bobbrow)
left a comment
There was a problem hiding this comment.
Looks good to me. I'll let someone else on the team approve though.
Sean McManus (sean-mcmanus)
left a comment
There was a problem hiding this comment.
Does anyone know how to test this?
|
|
||
| terminalDataHandlingQueue = terminalDataHandlingQueue.finally(() => void handleTerminalOutput(e)); | ||
| for await (const data of e.execution.read()) { | ||
| void handleTerminalOutput(data); |
There was a problem hiding this comment.
Should this be void handleTerminalOutput(data).catch(logAndReturn.undefined);?
Resolve the DTS check conflict and preserve sequential terminal output handling. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Garrett Serack (@fearthecowboy) please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
|
(copilot) I also addressed the outstanding promise-handling comment by changing the stream loop to There are still compatibility/behavior concerns that I don't think we should overlook before approving the migration:
Before merging, I recommend manually/integration testing at least:
If those cases confirm the concern, the options appear to be retaining the proposed raw-data API as a compatibility fallback, raising the minimum VS Code version and redesigning command launch around |
Fixes #12575
This stops the use of the proposed deprecated API and uses the newer, released
TerminalShellExecutionsupport.