feat: add GitHub Copilot provider with per-user token authentication #137

ssncferreira · 2026-01-21T13:37:30Z

Description

Adds support for GitHub Copilot as an AI provider. Unlike other providers that use a global API key, Copilot uses per-user keys passed in the Authorization header. Copilot's API is OpenAI-compatible, so the provider reuses the existing OpenAI interceptors.

Changes

Add Copilot provider in with support for:
- /chat/completions endpoint
- /responses endpoint
- Passthrough routes: /models, /agents/, /mcp/
InjectAuthHeader is a no-op since Copilot uses per-user keys from the original request
Add Copilot specific tests

Related to: coder/internal#1235

ssncferreira · 2026-01-21T13:37:50Z

This stack of pull requests is managed by Graphite. Learn more about stacking.

## Description Fixes a nil pointer dereference when extracting Actor ID for trace attributes. Discovered while testing the Copilot provider implementation (implemented upstack #137), where requests may not always have an actor set in the context. ## Changes Adds `ActorIDFromContext` helper that safely returns an empty string if no actor is present, and updates the interceptors to use it.

provider/copilot.go

…e and non-zero indices

…client timeout during tool invocation

provider/copilot.go

provider/copilot_test.go

ssncferreira · 2026-01-27T10:29:28Z

Merge activity

Jan 27, 10:29 AM UTC: A user started a stack merge that includes this pull request via Graphite.
Jan 27, 10:32 AM UTC: Graphite couldn't merge this PR because it had merge conflicts.
Jan 27, 10:44 AM UTC: A user started a stack merge that includes this pull request via Graphite.
Jan 27, 10:44 AM UTC: @ssncferreira merged this pull request with Graphite.

pawbana

LGTM
some tests nits that can be ignored

pawbana · 2026-01-27T10:29:11Z

provider/copilot_test.go

+
+		body := `{"model": "gpt-4", "messages": [{"role": "user", "content": "hello"}], "stream": false}`
+		req := httptest.NewRequest(http.MethodPost, routeCopilotChatCompletions, bytes.NewBufferString(body))
+		req.Header.Set("Authorization", "Bearer test-token")


nit: storing headers in map would be more clear when checking equal later.

ssncferreira mentioned this pull request Jan 21, 2026

fix: handle nil actor from context in trace attributes #136

Merged

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch 2 times, most recently from ca6355f to 0b199a7 Compare January 22, 2026 10:47

ssncferreira force-pushed the ssncferreira/fix-actor-nil branch from b99f59b to 7bd40c9 Compare January 22, 2026 10:47

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch 2 times, most recently from b0be741 to 68e00ca Compare January 23, 2026 15:51

ssncferreira force-pushed the ssncferreira/fix-actor-nil branch from 9182ea4 to 6a9968e Compare January 23, 2026 15:51

ssncferreira changed the base branch from ssncferreira/fix-actor-nil to graphite-base/137 January 23, 2026 15:54

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from 68e00ca to ea722ff Compare January 23, 2026 16:00

ssncferreira force-pushed the graphite-base/137 branch from 6a9968e to a127009 Compare January 23, 2026 16:00

graphite-app bot changed the base branch from graphite-base/137 to main January 23, 2026 16:00

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from ea722ff to ca7e288 Compare January 23, 2026 16:00

ssncferreira changed the base branch from main to graphite-base/137 January 23, 2026 18:14

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from ca7e288 to 77dea81 Compare January 23, 2026 18:14

ssncferreira changed the base branch from graphite-base/137 to ssncferreira/fix-chatcompletions-streaming-toolcall January 23, 2026 18:14

ssncferreira mentioned this pull request Jan 23, 2026

fix: handle chatcompletions streaming tool calls with no text preamble and non-zero indices #147

Merged

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch 2 times, most recently from 3744945 to c1d483a Compare January 23, 2026 18:51

ssncferreira force-pushed the ssncferreira/fix-chatcompletions-streaming-toolcall branch from b4479ce to 6de1225 Compare January 23, 2026 18:52

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch 3 times, most recently from f3cc0dd to bb51264 Compare January 23, 2026 19:24

ssncferreira force-pushed the ssncferreira/fix-chatcompletions-streaming-toolcall branch from 6de1225 to 82de4d9 Compare January 23, 2026 20:43

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch 2 times, most recently from 01ca570 to 905e2b5 Compare January 23, 2026 20:45

ssncferreira force-pushed the ssncferreira/fix-chatcompletions-streaming-toolcall branch from 82de4d9 to d02fd35 Compare January 23, 2026 20:45

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from 905e2b5 to 274b1ba Compare January 23, 2026 20:53

ssncferreira force-pushed the ssncferreira/fix-chatcompletions-streaming-toolcall branch from d02fd35 to 9e54caf Compare January 23, 2026 20:53

ssncferreira marked this pull request as ready for review January 26, 2026 08:38

ssncferreira requested review from dannykopping and pawbana January 26, 2026 08:38

dannykopping approved these changes Jan 26, 2026

View reviewed changes

provider/copilot.go Show resolved Hide resolved

provider/copilot.go Outdated Show resolved Hide resolved

provider/copilot.go Outdated Show resolved Hide resolved

provider/copilot.go Outdated Show resolved Hide resolved

fix: handle chatcompletions streaming tool calls with no text preambl…

e9bac3b

…e and non-zero indices

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from 274b1ba to 1646733 Compare January 26, 2026 10:57

ssncferreira force-pushed the ssncferreira/fix-chatcompletions-streaming-toolcall branch from 9e54caf to e9bac3b Compare January 26, 2026 10:57

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from 1646733 to a2eb2d3 Compare January 26, 2026 12:09

chore: add newlines at fixture files

734ddda

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from a2eb2d3 to cad2669 Compare January 26, 2026 12:15

ssncferreira force-pushed the ssncferreira/fix-chatcompletions-streaming-toolcall branch from efa7ba4 to 734ddda Compare January 26, 2026 12:15

fix: initiate SSE stream synchronously to prevent race condition and …

61ea53a

…client timeout during tool invocation

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from cad2669 to 79e2524 Compare January 26, 2026 18:56

dannykopping approved these changes Jan 27, 2026

View reviewed changes

provider/copilot.go Outdated Show resolved Hide resolved

matifali reviewed Jan 27, 2026

View reviewed changes

provider/copilot.go Show resolved Hide resolved

ssncferreira added 3 commits January 27, 2026 10:15

chore: address comments

8e62b06

feat: add GitHub Copilot provider with per-user token authentication

84c49e6

chore: address comments

8fec1c2

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from ce1e93a to 8fec1c2 Compare January 27, 2026 10:15

chore: address comments

ecd5e50

ssncferreira force-pushed the ssncferreira/feat-copilot-provider branch from bf4405e to ecd5e50 Compare January 27, 2026 10:21

pawbana reviewed Jan 27, 2026

View reviewed changes

provider/copilot_test.go Show resolved Hide resolved

pawbana reviewed Jan 27, 2026

View reviewed changes

provider/copilot_test.go Show resolved Hide resolved

ssncferreira changed the base branch from ssncferreira/fix-chatcompletions-streaming-toolcall to graphite-base/137 January 27, 2026 10:29

ssncferreira changed the base branch from graphite-base/137 to main January 27, 2026 10:30

pawbana approved these changes Jan 27, 2026

View reviewed changes

ssncferreira added 2 commits January 27, 2026 10:36

Merge branch 'main' into ssncferreira/feat-copilot-provider

d195932

chore: fix fmt

2ef6458

ssncferreira merged commit 602942b into main Jan 27, 2026
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add GitHub Copilot provider with per-user token authentication #137

feat: add GitHub Copilot provider with per-user token authentication #137

ssncferreira commented Jan 21, 2026 •

edited

Loading

Uh oh!

ssncferreira commented Jan 21, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ssncferreira commented Jan 27, 2026 •

edited

Loading

Uh oh!

pawbana left a comment

Uh oh!

pawbana Jan 27, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

feat: add GitHub Copilot provider with per-user token authentication #137

feat: add GitHub Copilot provider with per-user token authentication #137

Conversation

ssncferreira commented Jan 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Changes

Uh oh!

ssncferreira commented Jan 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ssncferreira commented Jan 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

pawbana left a comment

Choose a reason for hiding this comment

Uh oh!

pawbana Jan 27, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

ssncferreira commented Jan 21, 2026 •

edited

Loading

ssncferreira commented Jan 21, 2026 •

edited

Loading

ssncferreira commented Jan 27, 2026 •

edited

Loading