dotnet/command-line-api

GetCompletions should check exit code of invoked application

Open

#2.137 geöffnet am 31. März 2023

Auf GitHub ansehen
 (1 Kommentar) (3 Reaktionen) (0 zugewiesene Personen)C# (3.665 Stars) (421 Forks)auto 404
Area-Completionsbughelp wanted

Beschreibung

This code should probably check the exit code and not consider standard output to be completions when it is non-zero:

https://github.com/dotnet/command-line-api/blob/403234a4c4247151d09073bc07ccc13530ce62c6/src/System.CommandLine.Suggest/SuggestionStore.cs#L47-L50

cc @jonsequitur @adamsitnik

Contributor Guide

Tech Stack
csharp
Domain
cli
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
C# basicsunderstanding of exit codes
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90
Research-Richtung
The issue is in SuggestionStore.cs at lines 47-50 (linked in the issue). Currently, the code invokes the application and reads stdout without checking the exit code. The fix is to add a check after invocation: if exit code is non zero, ignore stdout. A maintainer has confirmed this is a valid bug. No other files need changes.