mirror of
https://github.com/EKKOLearnAI/hermes-web-ui.git
synced 2026-05-25 21:40:13 +00:00
610f3eb9d0
* feat(copilot): integrate GitHub Copilot provider with dynamic model list 集成 GitHub Copilot provider 与动态模型列表 EN: - New copilot-models service: fetch live model list from GitHub /models API - Filter noise IDs (accounts/, text-embedding, rerank prefixes) - Pass through preview/disabled metadata to frontend - Cache isolated per OAuth token (FNV-1a hash key) to prevent cross-account leak - Multi-source token resolution: env > apps.json > gh CLI - ModelSelector renders PREVIEW (orange) and UNAVAILABLE (gray, non-selectable) badges with tooltips - ProviderFormModal exposes Copilot OAuth login entry - New CopilotLoginModal component: guides gh auth login device flow - ProviderCard hides delete button for OAuth-only builtin providers (copilot/codex/nous) since their credentials live outside auth.json ZH: - 新增 copilot-models 服务:从 GitHub /models live API 拉取模型列表 - 噪音 ID 过滤(accounts/、text-embedding、rerank 前缀) - preview/disabled 元数据透传至前端 - 缓存按 OAuth token 隔离(FNV-1a hash key),避免切换 profile 串账号 - 多源 token 解析优先级:env > apps.json > gh CLI - ModelSelector 渲染 PREVIEW(橙色)/ UNAVAILABLE(灰色、不可选)badge,附 tooltip - ProviderFormModal 提供 Copilot OAuth 登录入口 - 新增 CopilotLoginModal 组件:引导 gh auth login 设备流程 - ProviderCard 对 OAuth-only builtin(copilot/codex/nous)隐藏删除按钮 其凭证不在 auth.json,删除按钮原本无效 Tests / 测试: new copilot-models suite (cache isolation, noise filter, preview/disabled passthrough) + copilot-login-modal — 24/24 passed. Pre-existing sessions-db-lineage failure on upstream/main is unrelated. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * refactor(copilot): switch to explicit opt-in per maintainer feedback 回应 PR #239 review:上一版会自动把系统级 GitHub OAuth 凭证(VS Code Copilot 插件、gh CLI 登录态)当作 hermes provider 拉到列表里,对未在 hermes 中注册过 Copilot 的用户造成困扰。本次改为显式 opt-in:用户必须通过 Add Provider 主动添加, 删除时按 token 来源决定是否清 ~/.hermes/.env,并避免误清理 VS Code / gh CLI 用户的 全局凭证。 Address PR #239 review feedback. Previously Copilot would silently appear in the provider list whenever the host had any GitHub OAuth token (VS Code plugin, gh CLI login). This caused confusion for users who never explicitly registered Copilot in hermes. Now Copilot requires explicit opt-in via Add Provider; on delete we only clear ~/.hermes/.env when the token actually originated there, leaving VS Code / gh CLI credentials untouched. What changed - 新增 ~/.hermes-web-ui/config.json 的 copilotEnabled flag 控制可见性 - 即便能解析到 token,未启用时也不在列表中显示 - resolveCopilotOAuthTokenWithSource 区分 token 来源(env / gh-cli / apps-json) - ProviderFormModal 增加 GitHub Copilot 入口;无 token 时进 device flow modal - CopilotLoginModal 重写为 in-app device flow 状态机(不再要求用户在终端跑 gh) - 删除 Copilot 时仅 source='env' 才清 ~/.hermes/.env,并自动 fallback 默认模型 - 老用户升级兼容:若 default 仍指向已禁用的 copilot,后端清空 default 让前端兜底 API - POST /api/hermes/copilot-auth/check-token - POST /api/hermes/copilot-auth/enable - POST /api/hermes/copilot-auth/disable - POST /api/hermes/copilot-auth/start (device flow) - POST /api/hermes/copilot-auth/poll (device flow) Tests - tests/server/copilot-auth-controller.test.ts (11 cases) - tests/server/copilot-device-flow.test.ts (12 cases) - tests/client/copilot-login-modal.test.ts 重写覆盖状态机 Follow-ups (留作后续 PR) - device flow session 未绑定 profile,登录中切 profile 会写到错的 .env - copilot device-code 接口的 expires_in 字段未使用,硬编码 15 分钟超时 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
352 lines
13 KiB
TypeScript
352 lines
13 KiB
TypeScript
import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
|
||
|
||
// Mock os.homedir before imports so file path resolution is stable.
|
||
vi.mock('os', async () => {
|
||
const actual = await vi.importActual<typeof import('os')>('os')
|
||
return { ...actual, homedir: () => '/fake/home' }
|
||
})
|
||
|
||
const { mockReadFile, mockExecFile } = vi.hoisted(() => ({
|
||
mockReadFile: vi.fn(),
|
||
mockExecFile: vi.fn(),
|
||
}))
|
||
|
||
vi.mock('fs/promises', () => ({ readFile: mockReadFile }))
|
||
vi.mock('child_process', () => ({ execFile: mockExecFile }))
|
||
|
||
import {
|
||
resolveCopilotOAuthToken,
|
||
getCopilotModels,
|
||
getCopilotModelsDetailed,
|
||
COPILOT_FALLBACK_MODELS,
|
||
__resetCopilotModelsCacheForTest,
|
||
} from '../../packages/server/src/services/hermes/copilot-models'
|
||
|
||
const ORIGINAL_ENV = { ...process.env }
|
||
const ORIGINAL_FETCH = global.fetch
|
||
|
||
function clearTokenEnv() {
|
||
delete process.env.COPILOT_GITHUB_TOKEN
|
||
delete process.env.GH_TOKEN
|
||
delete process.env.GITHUB_TOKEN
|
||
}
|
||
|
||
beforeEach(() => {
|
||
__resetCopilotModelsCacheForTest()
|
||
vi.clearAllMocks()
|
||
clearTokenEnv()
|
||
// Default: apps.json read fails (ENOENT)
|
||
mockReadFile.mockRejectedValue(new Error('ENOENT'))
|
||
// Default: gh CLI fails
|
||
mockExecFile.mockImplementation((_cmd: any, _args: any, _opts: any, cb: any) => {
|
||
cb(new Error('gh not installed'), { stdout: '', stderr: '' })
|
||
})
|
||
})
|
||
|
||
afterEach(() => {
|
||
process.env = { ...ORIGINAL_ENV }
|
||
global.fetch = ORIGINAL_FETCH
|
||
})
|
||
|
||
describe('resolveCopilotOAuthToken', () => {
|
||
it('优先级:COPILOT_GITHUB_TOKEN > GH_TOKEN > GITHUB_TOKEN', async () => {
|
||
process.env.COPILOT_GITHUB_TOKEN = 'gho_copilot'
|
||
process.env.GH_TOKEN = 'gho_gh'
|
||
process.env.GITHUB_TOKEN = 'gho_github'
|
||
expect(await resolveCopilotOAuthToken('')).toBe('gho_copilot')
|
||
|
||
delete process.env.COPILOT_GITHUB_TOKEN
|
||
expect(await resolveCopilotOAuthToken('')).toBe('gho_gh')
|
||
|
||
delete process.env.GH_TOKEN
|
||
expect(await resolveCopilotOAuthToken('')).toBe('gho_github')
|
||
})
|
||
|
||
it('跳过 classic PAT (ghp_),回退到下一来源', async () => {
|
||
process.env.GH_TOKEN = 'ghp_classic_pat'
|
||
process.env.GITHUB_TOKEN = 'gho_oauth_token'
|
||
expect(await resolveCopilotOAuthToken('')).toBe('gho_oauth_token')
|
||
})
|
||
|
||
it('从 .env 读取并去掉两端引号', async () => {
|
||
expect(await resolveCopilotOAuthToken('GH_TOKEN="gho_quoted"\n')).toBe('gho_quoted')
|
||
expect(await resolveCopilotOAuthToken("GH_TOKEN='gho_single'\n")).toBe('gho_single')
|
||
expect(await resolveCopilotOAuthToken('GH_TOKEN=gho_plain\n')).toBe('gho_plain')
|
||
})
|
||
|
||
it('忽略 .env 中以 # 开头的注释行', async () => {
|
||
expect(await resolveCopilotOAuthToken('GH_TOKEN=# comment\n')).toBe('')
|
||
})
|
||
|
||
it('回退到 ~/.config/github-copilot/apps.json 的 oauth_token', async () => {
|
||
mockReadFile.mockImplementation(async (p: string) => {
|
||
if (p.includes('apps.json')) {
|
||
return JSON.stringify({
|
||
'github.com:abc': { oauth_token: 'gho_from_apps_json', user: 'me' },
|
||
})
|
||
}
|
||
throw new Error('ENOENT')
|
||
})
|
||
expect(await resolveCopilotOAuthToken('')).toBe('gho_from_apps_json')
|
||
})
|
||
|
||
it('apps.json 中的 ghp_ token 也应跳过', async () => {
|
||
mockReadFile.mockImplementation(async (p: string) => {
|
||
if (p.includes('apps.json')) {
|
||
return JSON.stringify({ 'github.com:a': { oauth_token: 'ghp_pat_in_apps' } })
|
||
}
|
||
throw new Error('ENOENT')
|
||
})
|
||
expect(await resolveCopilotOAuthToken('')).toBe('')
|
||
})
|
||
|
||
it('最后回退到 `gh auth token`', async () => {
|
||
mockExecFile.mockImplementation((_cmd: any, _args: any, _opts: any, cb: any) => {
|
||
cb(null, { stdout: 'gho_from_gh_cli\n', stderr: '' })
|
||
})
|
||
expect(await resolveCopilotOAuthToken('')).toBe('gho_from_gh_cli')
|
||
})
|
||
|
||
it('所有来源都失败时返回空字符串', async () => {
|
||
expect(await resolveCopilotOAuthToken('')).toBe('')
|
||
})
|
||
})
|
||
|
||
describe('getCopilotModels', () => {
|
||
function mockFetchSequence(responses: Array<Partial<Response> | Error>) {
|
||
let i = 0
|
||
global.fetch = vi.fn(async () => {
|
||
const r = responses[i++]
|
||
if (r instanceof Error) throw r
|
||
return r as Response
|
||
}) as any
|
||
}
|
||
|
||
it('成功路径:返回 chat type 且 supports /chat/completions 的模型 id', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
mockFetchSequence([
|
||
{ ok: true, json: async () => ({ token: 'tok_copilot' }) } as any,
|
||
{
|
||
ok: true,
|
||
json: async () => ({
|
||
data: [
|
||
{ id: 'gpt-5.4', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
{ id: 'claude-opus-4.7', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions', '/v1/messages'] },
|
||
{ id: 'embedding-1', capabilities: { type: 'embeddings' }, supported_endpoints: ['/embeddings'] },
|
||
{ id: 'completion-only', capabilities: { type: 'chat' }, supported_endpoints: ['/completions'] },
|
||
{ id: 'no-endpoints', capabilities: { type: 'chat' } },
|
||
],
|
||
}),
|
||
} as any,
|
||
])
|
||
const ids = await getCopilotModels('')
|
||
expect(ids).toContain('gpt-5.4')
|
||
expect(ids).toContain('claude-opus-4.7')
|
||
expect(ids).toContain('no-endpoints') // endpoints 缺省时允许
|
||
expect(ids).not.toContain('embedding-1')
|
||
expect(ids).not.toContain('completion-only')
|
||
})
|
||
|
||
it('不再强制 model_picker_enabled —— picker_enabled=false 的模型也返回', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
mockFetchSequence([
|
||
{ ok: true, json: async () => ({ token: 'tok' }) } as any,
|
||
{
|
||
ok: true,
|
||
json: async () => ({
|
||
data: [
|
||
{ id: 'a', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'], model_picker_enabled: false },
|
||
{ id: 'b', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'], model_picker_enabled: true },
|
||
],
|
||
}),
|
||
} as any,
|
||
])
|
||
const ids = await getCopilotModels('')
|
||
expect(ids).toEqual(expect.arrayContaining(['a', 'b']))
|
||
})
|
||
|
||
it('无 token 时返回 fallback 列表', async () => {
|
||
const ids = await getCopilotModels('')
|
||
expect(ids).toEqual(COPILOT_FALLBACK_MODELS.map(m => m.id))
|
||
})
|
||
|
||
it('token exchange 失败返回 fallback', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
mockFetchSequence([{ ok: false, status: 401 } as any])
|
||
const ids = await getCopilotModels('')
|
||
expect(ids).toEqual(COPILOT_FALLBACK_MODELS.map(m => m.id))
|
||
})
|
||
|
||
it('models endpoint 失败返回 fallback', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
mockFetchSequence([
|
||
{ ok: true, json: async () => ({ token: 'tok' }) } as any,
|
||
{ ok: false, status: 503 } as any,
|
||
])
|
||
const ids = await getCopilotModels('')
|
||
expect(ids).toEqual(COPILOT_FALLBACK_MODELS.map(m => m.id))
|
||
})
|
||
|
||
it('网络错误(如超时)返回 fallback', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
mockFetchSequence([new Error('AbortError: timeout')])
|
||
const ids = await getCopilotModels('')
|
||
expect(ids).toEqual(COPILOT_FALLBACK_MODELS.map(m => m.id))
|
||
})
|
||
|
||
it('正缓存命中:第二次调用不再发请求', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
const fetchMock = vi.fn()
|
||
.mockResolvedValueOnce({ ok: true, json: async () => ({ token: 'tok' }) })
|
||
.mockResolvedValueOnce({
|
||
ok: true,
|
||
json: async () => ({ data: [{ id: 'm1', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] }] }),
|
||
})
|
||
global.fetch = fetchMock as any
|
||
const a = await getCopilotModels('')
|
||
const b = await getCopilotModels('')
|
||
expect(a).toEqual(['m1'])
|
||
expect(b).toEqual(['m1'])
|
||
expect(fetchMock).toHaveBeenCalledTimes(2)
|
||
})
|
||
|
||
it('负缓存:失败后短期内不再重试', async () => {
|
||
const fetchMock = vi.fn()
|
||
global.fetch = fetchMock as any
|
||
const a = await getCopilotModels('')
|
||
const b = await getCopilotModels('')
|
||
expect(a).toEqual(COPILOT_FALLBACK_MODELS.map(m => m.id))
|
||
expect(b).toEqual(COPILOT_FALLBACK_MODELS.map(m => m.id))
|
||
// 无 token 时根本不会调 fetch
|
||
expect(fetchMock).not.toHaveBeenCalled()
|
||
})
|
||
|
||
it('并发请求合并:同时调用 N 次只发一组请求', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
const fetchMock = vi.fn()
|
||
.mockResolvedValueOnce({ ok: true, json: async () => ({ token: 'tok' }) })
|
||
.mockResolvedValueOnce({
|
||
ok: true,
|
||
json: async () => ({ data: [{ id: 'x', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] }] }),
|
||
})
|
||
global.fetch = fetchMock as any
|
||
const [a, b, c] = await Promise.all([
|
||
getCopilotModels(''),
|
||
getCopilotModels(''),
|
||
getCopilotModels(''),
|
||
])
|
||
expect(a).toEqual(['x'])
|
||
expect(b).toEqual(['x'])
|
||
expect(c).toEqual(['x'])
|
||
expect(fetchMock).toHaveBeenCalledTimes(2)
|
||
})
|
||
})
|
||
|
||
describe('getCopilotModels noise filter & detailed meta', () => {
|
||
function mockFetchSequence(responses: Array<Partial<Response> | Error>) {
|
||
let i = 0
|
||
global.fetch = vi.fn(async () => {
|
||
const r = responses[i++]
|
||
if (r instanceof Error) throw r
|
||
return r as Response
|
||
}) as any
|
||
}
|
||
|
||
it('过滤掉噪音 ID(accounts/、text-embedding、rerank 前缀)', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
mockFetchSequence([
|
||
{ ok: true, json: async () => ({ token: 'tok' }) } as any,
|
||
{
|
||
ok: true,
|
||
json: async () => ({
|
||
data: [
|
||
{ id: 'gpt-5.4', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
{ id: 'accounts/msft/routers/abc', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
{ id: 'text-embedding-3-small', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
{ id: 'rerank-v1', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
],
|
||
}),
|
||
} as any,
|
||
])
|
||
const ids = await getCopilotModels('')
|
||
expect(ids).toEqual(['gpt-5.4'])
|
||
})
|
||
|
||
it('detailed 返回 preview 字段', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
mockFetchSequence([
|
||
{ ok: true, json: async () => ({ token: 'tok' }) } as any,
|
||
{
|
||
ok: true,
|
||
json: async () => ({
|
||
data: [
|
||
{ id: 'gemini-3-pro-preview', preview: true, capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
{ id: 'gpt-4o', preview: false, capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
],
|
||
}),
|
||
} as any,
|
||
])
|
||
const detailed = await getCopilotModelsDetailed('')
|
||
expect(detailed).toEqual([
|
||
{ id: 'gemini-3-pro-preview', preview: true, disabled: false },
|
||
{ id: 'gpt-4o', preview: false, disabled: false },
|
||
])
|
||
})
|
||
|
||
it('detailed 返回 disabled 字段(policy.state === "disabled")', async () => {
|
||
process.env.GH_TOKEN = 'gho_token'
|
||
mockFetchSequence([
|
||
{ ok: true, json: async () => ({ token: 'tok' }) } as any,
|
||
{
|
||
ok: true,
|
||
json: async () => ({
|
||
data: [
|
||
{ id: 'gpt-3.5-turbo', policy: { state: 'disabled' }, capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
{ id: 'gpt-4o', policy: { state: 'enabled' }, capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
{ id: 'claude-sonnet-4', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] },
|
||
],
|
||
}),
|
||
} as any,
|
||
])
|
||
const detailed = await getCopilotModelsDetailed('')
|
||
const map = new Map(detailed.map((m) => [m.id, m]))
|
||
expect(map.get('gpt-3.5-turbo')?.disabled).toBe(true)
|
||
expect(map.get('gpt-4o')?.disabled).toBe(false)
|
||
expect(map.get('claude-sonnet-4')?.disabled).toBe(false)
|
||
})
|
||
|
||
it('缓存按 oauth token 隔离:切换账号会重新拉取', async () => {
|
||
const fetchMock = vi.fn()
|
||
// 账号 A:token exchange + models
|
||
.mockResolvedValueOnce({ ok: true, json: async () => ({ token: 'tokA' }) })
|
||
.mockResolvedValueOnce({
|
||
ok: true,
|
||
json: async () => ({ data: [{ id: 'model-a', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] }] }),
|
||
})
|
||
// 账号 B:另一组 token exchange + models
|
||
.mockResolvedValueOnce({ ok: true, json: async () => ({ token: 'tokB' }) })
|
||
.mockResolvedValueOnce({
|
||
ok: true,
|
||
json: async () => ({ data: [{ id: 'model-b', capabilities: { type: 'chat' }, supported_endpoints: ['/chat/completions'] }] }),
|
||
})
|
||
global.fetch = fetchMock as any
|
||
|
||
process.env.GH_TOKEN = 'gho_account_A'
|
||
const a = await getCopilotModels('')
|
||
expect(a).toEqual(['model-a'])
|
||
|
||
// 切换到账号 B,不 reset cache
|
||
process.env.GH_TOKEN = 'gho_account_B'
|
||
const b = await getCopilotModels('')
|
||
expect(b).toEqual(['model-b'])
|
||
|
||
// 再切回 A:应该命中 A 的缓存(不再发请求)
|
||
process.env.GH_TOKEN = 'gho_account_A'
|
||
const a2 = await getCopilotModels('')
|
||
expect(a2).toEqual(['model-a'])
|
||
|
||
// 总共 4 次请求(A.exchange、A.models、B.exchange、B.models),切回 A 时命中缓存
|
||
expect(fetchMock).toHaveBeenCalledTimes(4)
|
||
})
|
||
})
|