Microsoft/vscode

Explore file watcher handling of verbose VS Code CLI commands

Open

#141.488 aperta il 25 gen 2022

Vedi su GitHub
 (2 commenti) (0 reazioni) (1 assegnatario)TypeScript (10.221 fork)batch import
debtengineeringhelp wantedmacosmacos-big-surworkbench-cli

Metriche repository

Star
 (74.848 star)
Metriche merge PR
 (Merge medio 11h 43m) (1000 PR mergiate in 30 g)

Descrizione

Ref https://github.com/microsoft/vscode/issues/141053

Currently:

  • The CLI process on macOS (Big Sur and above) spawns the main process using the open command so that VS Code is launched with the correct entitlements.
  • the open command can only redirect stdout and stderr to files, so we currently create temp files for the open command, and then set up file watchers on the CLI process to detect when those files are being written to.
  • code-insiders --status and code-insiders --verbose both require the setup with the file watchers.
  • :bug: When the main process exits, the CLI process cancels the filewatching, but the watcher might not have finished reading through the file yet. Therefore, some of the expected output is cut off early at the end.

A mitigation is to add in a timeout so that there is a delay between the main process exiting and the filewatcher being cancelled, but there should be a better way to detect when to stop watching. Actually, the issue reminds me a bit of flush() methods in other contexts.

Guida contributor

Direzione di ricerca
Indaga sul meccanismo del file watcher per l'output CLI in VS Code su macOS. Il problema fa riferimento a #141053 e descrive una condizione di competizione tra l'uscita del processo principale e la cancellazione del file watcher. Esamina il codice di generazione del processo CLI e l'implementazione del file watching nel processo principale. C'è un assegnatario; verifica se eventuali lavori in corso o commenti forniscono ulteriore contesto. Concentrati sul trovare un modo affidabile per rilevare quando tutto l'output è stato letto prima di annullare il watcher.
Tech stack
typescript
Dominio
clideveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jsGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60