feat(provider): add GitHub Copilot context tier - #3282
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
⏳ DRAFT
What to do
Review readiness checklist
0/4 boxes ticked. This pull request was already a draft. Its draft status will be preserved after every issue above is resolved. |
리뷰 · 우선순위 58 / 80이 PR는 이슈 #3281을 그대로 구현한다. GitHub Copilot 쪽에서 모델마다 구성도 한 바퀴 돌린다. 새 막힌 곳은 게이트와 몇 가지 날카로운 모서리다. hygiene이 라인 - 메인테이너의 판단이 필요한 지점
너의 추천 이 댓글은 grok-bot이 작성했습니다 |
a3fc5df to
4f7132b
Compare
1ff88a7 to
691f338
Compare
Summary
modelContextTierssetting withdefaultandlong_contextvalues.ocx provider edit --model-context-tier.contextTierthrough both OpenAI Chat Completions and Responses adapters.No credential, OAuth-flow, or unrelated-provider behavior changes are included. Each model defaults to the normal tier;
long_contextremains explicitly labeled for models that support it.UI evidence
Compatibility evidence
GitHub documents
contextTieras an official Copilot setting withdefaultandlong_contextvalues:--context TIERfor tiered-pricing models.contextTiersetting.This proves the field is not Luna-specific, while the upstream eligibility remains model-conditional. The UI therefore covers every detected/configured id without claiming that every Copilot model accepts
long_context; unsupported models stay ondefaultunless an operator explicitly opts in.Validation
bun test tests/github-copilot-context-tier.test.tsbun test tests/cli-headless-parity.test.tsbun test tests/management-provider-validation.test.ts --test-name-pattern 'exposes and persists context-window|hand-edited context|rejects runtime metadata'bun test tests/oauth-upsert-preserves-api-key.test.ts --test-name-pattern 'Copilot|github-copilot'bun test ./gui/tests/provider-settings-live-models-provenance.test.tsxbun run build:guibun install --frozen-lockfile && bun run buildfromdocs-site/bun run privacy:scanbun run skill:surface:checkThe changed-mode sweep also exercised 15,030 tests: 14,909 passed, 10 skipped, and 111 existing environment/platform-sensitive failures outside this feature. The repository typecheck currently stops on pre-existing Bun
fetch(..., { timeout })typing errors insrc/server/claude-messages.tsandsrc/server/responses/fetch-helpers.ts; no feature-related TypeScript error was reported. GUI lint is currently blocked by the repository oxlint plugin loader rejecting the TypeScript local plugin in this environment.The branch is rebased onto the latest
dev(81a1fc1cc) and the current head is691f338c0.References
Refs #3281
This is intentionally separate from #3156/#3163, which covered reading Copilot context-window metadata rather than selecting and forwarding the upstream context tier.
Review readiness checklist
This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:
All CI tests are green on my local testing.
I pushed my PR to the latest dev commit.
I resolved all correct Codex and CodeRabbit findings.
My PR is ready for review.