diff --git a/frontend/src/lib/http.test.ts b/frontend/src/lib/http.test.ts index b9fb5e9..ce057ae 100644 --- a/frontend/src/lib/http.test.ts +++ b/frontend/src/lib/http.test.ts @@ -21,8 +21,10 @@ globalThis.window = { } // 捕获每次 fetch 的入参供断言 +let lastUrl = '' let lastInit: RequestInit | undefined -globalThis.fetch = async (_url: string, init?: RequestInit) => { +globalThis.fetch = async (url: string | URL | Request, init?: RequestInit) => { + lastUrl = String(url) lastInit = init return new Response(JSON.stringify({ ok: true }), { status: 200, headers: { 'content-type': 'application/json' } }) } @@ -63,3 +65,19 @@ test('DELETE: method=DELETE, 无 body, 无 Content-Type', async () => { assert.equal(lastInit?.body, undefined) assert.equal((lastInit?.headers as Record)?.['Content-Type'], undefined) }) + +// ── 健康检查豁免: /health* 挂在根路径(后端 app.py 不在 /api/v1 下), +// 加前缀会 404 → 管理后台右上角永远「系统异常」 ── +test('build_url: /health 与 /health/ready 不加 /api/v1 前缀', async () => { + await http.get('/health') + assert.equal(lastUrl, '/health') + await http.get('/health/ready') + assert.equal(lastUrl, '/health/ready') +}) + +test('build_url: 常规 API 路径仍统一加 /api/v1', async () => { + await http.get('/admin/stats') + assert.equal(lastUrl, '/api/v1/admin/stats') + await http.get('/matches') + assert.equal(lastUrl, '/api/v1/matches') +}) diff --git a/frontend/src/lib/http.ts b/frontend/src/lib/http.ts index 251fdbc..c600f8d 100644 --- a/frontend/src/lib/http.ts +++ b/frontend/src/lib/http.ts @@ -17,22 +17,30 @@ export const UNAUTHORIZED_EVENT = 'profeto:unauthorized' const API_BASE = '/api/v1' const DEFAULT_TIMEOUT = 30_000 +/** 健康检查端点挂在根路径(app.py 不在 /api/v1 下,与 compose healthcheck 一致); + * 加前缀会 404,导致管理后台健康状态永远显示「系统异常」 */ +const ROOT_ONLY_PREFIXES = ['/health'] + /** 所有 API 路径统一走 /api/v1,避免浏览器直接请求 /matches 被 nginx 当 SPA 回退 */ function build_url(path: string): string { if (path.startsWith('http')) return path + if (ROOT_ONLY_PREFIXES.some(p => path === p || path.startsWith(`${p}/`))) return path if (path.startsWith(API_BASE)) return path if (path.startsWith('/')) return `${API_BASE}${path}` return `${API_BASE}/${path}` } export class ApiError extends Error { - constructor( - message: string, - public status: number, - public data?: unknown, - ) { + status: number + data?: unknown + + constructor(message: string, status: number, data?: unknown) { super(message) this.name = 'ApiError' + // 显式赋值而非构造函数参数属性(public x):strip-types 不支持后者, + // 会让 node --experimental-strip-types 跑 lib/http.test.ts 直接失败 + this.status = status + this.data = data } }