AssistantMessageDeltaEvent and AssistantReasoningEvent not triggered due to unhandled reasoning field from BYOK providers (Copilot CLI)
- Vorherrschende Sprache
- Shell
- Sterne
- 11.2k
- Forks
- 1.9k
- Ø Merge
- 14 Std. 16 Min.
- Gemergte PRs (30 T.)
- 6
Beschreibung
### Describe the bug
When using GitHub Copilot CLI (or SDK) with BYOK and vLLM as the provider, neither AssistantMessageDeltaEvent nor AssistantReasoningEvent are triggered if the reasoning content is returned in the reasoning field. Copilot CLI only checks reasoning_content, so no event is fired or reasoning received. This issue appears with vLLM (and may affect others), and was discovered after inspecting API network responses as detailed in my earlier SDK repo issues. CLI version 1.0.43. SDK version 0.3.0.
### Affected version
1.0.43
### Steps to reproduce the behavior
1. Set BYOK with vLLM as the provider in GitHub Copilot CLI/SDK (Windows or WSL).
2. Trigger completions or events that would yield reasoning.
3. Notice that no AssistantMessageDeltaEvent nor AssistantReasoningEvent is triggered if the response uses the reasoning field instead of reasoning_content.
### Expected behavior
Copilot CLI (and SDK) should process the reasoning field from BYOK providers, not only reasoning_content, so that both events are properly triggered and content received.
### Additional context
- Environment: Windows, WSL
- Copilot CLI version: 1.0.43
- Copilot SDK version: 0.3.0
- BYOK Provider: vLLM
Beitragsleitfaden
Rechercherichtung
Reproduzieren Sie das Problem mit GitHub Copilot CLI oder SDK 0.3.0, BYOK und vLLM und untersuchen Sie anschließend die Ereignisverarbeitung, die reasoning_content liest. Vergleichen Sie die im Bericht beschriebenen Antwortfelder und überprüfen Sie, dass reasoning so verarbeitet wird, dass AssistantMessageDeltaEvent und AssistantReasoningEvent den Inhalt erhalten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- shell
- Bereich
- api, cli
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 55/100