This repository was archived by the owner on May 4, 2026. It is now read-only.
WIP: feat: GitHub Copilot adapter#1
Merged
Conversation
Wire GitHub Copilot into the provider stack, shared contracts, and chat settings so the app can start and configure Copilot sessions alongside Codex.
Use live Copilot model metadata to validate session configuration, drive reasoning-effort support, and keep the web model picker aligned with server-reported capabilities.
Resolve a bundled Copilot CLI by default, update the Copilot SDK dependency, and avoid duplicating assistant message text when providers send both deltas and completed-message fallbacks.
Skip missing workspace roots when starting sessions or capturing checkpoints, avoid duplicate or empty assistant message finalization, and add adapter coverage through an injectable Copilot client factory.
Keep a stable orchestration turn id across Copilot tool, approval, and user-input events while preserving provider turn refs and surfacing better tool titles from session events.
Keep approval and user-input resolution tied to their originating turn and let the chat work log keep showing the most relevant visible turn while a newer turn is still waiting for output.
Filter visible work-log entries by activities that happened after the latest user message instead of tracking a separate visible turn heuristic, which keeps follow-up Copilot work visible across provider turns with simpler session logic.
Auto-approve Copilot permission requests in full-access mode and align the chat completion summary plus composer effort persistence with the simplified post-user-message work-log model.
Route git commit and PR generation through the selected provider so Git quick actions can use Copilot models and provider-specific options instead of always falling back to Codex.
Expose Copilot billing multipliers and quota snapshots from provider health so the model picker can show per-model cost hints and remaining usage directly in the chat UI.
Keep Copilot billing-multiplier badges from shrinking or breaking label alignment in the provider picker so quota and model metadata stay readable in tighter layouts.
Use the real current workspace root in the checkpoint diff test so it stays compatible with the stricter existing-workspace validation path instead of relying on a missing fake tmp directory.
Resolve platform-specific bundled Copilot binaries more reliably, ignore no-op CLI overrides, package the bundled binaries into desktop artifacts, and reconnect websocket requests after pre-open errors.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Context
Notes