Skip to content

Add GitHub Copilot provider support#369

Open
jasnsy wants to merge 4 commits intoOpenKnots:mainfrom
jasnsy:feat/github-copilot-provider
Open

Add GitHub Copilot provider support#369
jasnsy wants to merge 4 commits intoOpenKnots:mainfrom
jasnsy:feat/github-copilot-provider

Conversation

@jasnsy
Copy link
Copy Markdown

@jasnsy jasnsy commented Apr 9, 2026

Summary

  • add GitHub Copilot as a first-class provider across shared contracts, server adapters, health checks, and web settings
  • expose Copilot in the provider/model picker with built-in model options and setup guidance
  • register the Copilot adapter and add the SDK dependency plus adapter registry coverage

Verification

  • bun fmt
  • bun lint
  • bun typecheck

Screenshot

image

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 9, 2026

@jasnsy is attempting to deploy a commit to the 0xBuns Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added size:XXL vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Apr 9, 2026
@BunsDev BunsDev requested a review from Copilot April 9, 2026 02:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@jasnsy jasnsy force-pushed the feat/github-copilot-provider branch from b303511 to 2aae280 Compare April 9, 2026 10:27
@jasnsy jasnsy requested review from BunsDev and Copilot April 9, 2026 10:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@BunsDev
Copy link
Copy Markdown
Contributor

BunsDev commented Apr 9, 2026

@copilot apply changes based on the comments in this thread

@fuller-stack-dev
Copy link
Copy Markdown

fuller-stack-dev commented Apr 9, 2026

@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

@jasnsy jasnsy force-pushed the feat/github-copilot-provider branch from 67233ae to 36537fa Compare April 10, 2026 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants