PowerShell / PowerShell/PowerShellEditorServices
Verify the main runspace reset method in the PSES pipeline thread consumer
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C#
- Sterne
- 767
- Forks
- 266
- Ø Merge
- 3 T. 16 Std.
- Gemergte PRs (30 T.)
- 1
Beschreibung
We have a kind of primitive stab at a method to clean up a faulty runspace state in the PSES pipeline thread consumer here:
We need to induce a fault and see if this works as expected, as well as ensure it doesn't run at other times...
My suspicion is that it will need some refinement and could have some significant bugs in it. But it's currently a corner case and a worse-case scenario handler anyway (if it didn't exist, we are essentially crashed).
So we need to investigate this and see how we can improve:
- detection of bad runspace state
- restoration of good runspace state
- how our threading mechanisms work in this scenario...
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit src/PowerShellEditorServices/Services/PowerShell/Host/PsesInternalHost.cs ungefähr in den Zeilen 784–826 und verfolge dann den Consumer des Pipeline-Threads sowie seine Behandlung des Runspace-Zustands. Provoziere die im Issue beschriebene fehlerhafte Runspace-Bedingung und überprüfe die Erkennung, Wiederherstellung sowie, dass der Reset während des normalen Betriebs nicht ausgeführt wird; abgeschlossen ist die Arbeit, wenn das Verhalten bestätigt wurde oder die erforderlichen Verfeinerungen dokumentiert und getestet sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- csharp, powershell
- Bereich
- devtools
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100