Conversation
|
@jasnsy is attempting to deploy a commit to the 0xBuns Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Pull request overview
Adds GitHub Copilot as a first-class provider across the contracts, server adapter + health checks, and the web UI settings/picker so sessions can be started and monitored using the Copilot CLI/SDK.
Changes:
- Extend shared/contracts provider + model abstractions to include
copilot(start options, model options, runtime event sources). - Add server-side Copilot adapter + registry wiring and health/doctor probes via
@github/copilot-sdk. - Expose Copilot in the web app: provider picker, setup card guidance, settings for binary path/config dir, and custom models.
Reviewed changes
Copilot reviewed 22 out of 23 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/shared/src/model.ts | Adds Copilot model option normalization and model/provider helpers. |
| packages/contracts/src/providerRuntime.ts | Registers Copilot runtime raw event source. |
| packages/contracts/src/orchestration.ts | Adds copilot to ProviderKind and defines Copilot start options. |
| packages/contracts/src/model.ts | Introduces Copilot reasoning effort/model options + built-in model/alias tables. |
| bun.lock | Adds @github/copilot-sdk (and transitive deps) to the lockfile. |
| apps/web/src/store.ts | Allows legacy provider mapping to include Copilot/OpenClaw. |
| apps/web/src/session-logic.ts | Adds Copilot to provider option list. |
| apps/web/src/routes/_chat.settings.tsx | Adds Copilot install settings + custom model handling. |
| apps/web/src/components/home/home-utils.ts | Adds Copilot/OpenClaw provider labels. |
| apps/web/src/components/chat/providerStatusPresentation.ts | Adds Copilot provider label for status UI. |
| apps/web/src/components/chat/ProviderSetupCard.tsx | Adds Copilot install/auth commands. |
| apps/web/src/components/chat/ProviderModelPicker.tsx | Adds Copilot icon + styling in provider/model picker. |
| apps/web/src/components/chat/ProviderModelPicker.browser.tsx | Updates browser test fixture to include Copilot provider models. |
| apps/web/src/components/chat/composerProviderRegistry.tsx | Adds Copilot modelOptions mapping for dispatch. |
| apps/web/src/appSettings.ts | Adds Copilot settings keys, custom models, and provider start options mapping. |
| apps/server/src/serverLayers.ts | Wires Copilot adapter layer into the server provider layer. |
| apps/server/src/provider/Services/CopilotAdapter.ts | Defines the Copilot adapter service interface. |
| apps/server/src/provider/Layers/ProviderHealth.ts | Adds Copilot health probe using CopilotClient. |
| apps/server/src/provider/Layers/ProviderAdapterRegistry.ts | Registers Copilot adapter in the default registry set. |
| apps/server/src/provider/Layers/ProviderAdapterRegistry.test.ts | Extends registry test coverage to include Copilot. |
| apps/server/src/provider/Layers/CopilotAdapter.ts | Implements the Copilot adapter using @github/copilot-sdk sessions/events. |
| apps/server/src/doctor.ts | Adds Copilot to doctor health checks + install guidance. |
| apps/server/package.json | Adds @github/copilot-sdk dependency. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
b303511 to
2aae280
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 22 out of 23 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot apply changes based on the comments in this thread |
the comments were addressed in 67233ae, the changes were commented in the resolved comments <3 |
67233ae to
36537fa
Compare
Summary
Verification
Screenshot