本次提交一次性落盘 Phase 2e-2 的 Task 0(PoC Spike)、Task 1(骨架 + Fastify 5 升级) 和 Task 2(9 个内部 REST API + code-reviewer 审查修复),覆盖 media-server 子项目 从零到可用的全部工作。 【Task 0 - PoC Spike】 - media-server/poc/:Node + mediasoup + fastify-websocket + 前端 mediasoup-client - Playwright 双 tab 自动化验证 2 人会议:4 transports / 4 producers / 4 consumers / RSS 61MB - media-server/docs/poc-notes.md 归档 7 项关键坑 + 启动步骤 + Task 1/2/9 复用映射 - 锁定技术栈:mediasoup + fastify + mediasoup-client,不改用 livekit-server 【Task 1 - 骨架 + Fastify 5 升级】 - 依赖版本:mediasoup@3.19.0 + fastify@5.8.5 + fastify-plugin@5.1.0 + @fastify/sensible@6.0.4 + @fastify/websocket@11.2.0 + pino@9.3.2 + zod@3.23.8 - src 五件套:app.ts / config.ts / utils/logger.ts / mediasoup/worker.ts / middlewares/internal-auth.ts - /healthz + /readyz + /internal/info 三端点实测通过 - X-Internal-Token 鉴权用 timingSafeEqual 防侧信道 - mediasoup Worker died 指数退避自愈通过 kill -9 验证 - 多阶段 Dockerfile:非 root + curl HEALTHCHECK + 暴露 40000-40199 UDP/TCP - Fastify 4 → 5 升级:loggerInstance: logger 消灭两个 pino 实例 【Task 2 - 9 个内部 REST API + 审查修复】 接口全部挂 /internal/v1/* 前缀,覆盖 Router/Transport/Producer/Consumer 完整生命周期: - POST /routers, DELETE /routers/:id - POST /transports, POST /transports/:id/connect - POST /producers, DELETE /producers/:id - POST /consumers, POST /consumers/:id/resume, DELETE /consumers/:id 工程特性: - zod 手动 parse + 全局 errorHandler(不引入 fastify-type-provider-zod 避免 zod v4 依赖冲突) - AppError 统一错误码(NOT_FOUND/CONFLICT/CAN_NOT_CONSUME/ROUTER_LIMIT_EXCEEDED /MEDIASOUP_ERROR/VALIDATION_ERROR/UNAUTHORIZED/INTERNAL_ERROR) - 所有资源用 Map + observer.once('close') 自清理,Consumer 监听 producerclose 级联关闭 - Consumer 强制 paused:true 创建,/resume 独立接口 - Transport direction 强约束:recv 拒 produce、send 拒 consume code-reviewer 子代理审查"有条件通过",同步修复: - M1 _clearXxxMap 新增 src/utils/test-guard.ts#assertTestOnly 守卫(生产误调用抛错) - M2 新增 src/schemas/rtp.ts 对 rtpParameters / rtpCapabilities 做 codecs 浅层校验 (mimeType/clockRate/payloadType 必填、codecs ≥1),消除 as unknown as 双跳断言 - m1 connectTransport 改乐观锁:先置位再 await,失败回退 - m2 改读 consumer.producerPaused(更符合 mediasoup 语义) - m3 producerclose 改为 once(风格一致) - m5 internal-auth 改为反向白名单 PRIVATE_PATH_PREFIXES = ['/internal/'] 验证: - typecheck / lint 0 错误 - vitest:65 passed / 8 spec 文件 - 覆盖率 stmts 82.87% / branches 75.83% / funcs 91.3% / lines 82.87% - 9 接口 happy path + 6 类错误路径 curl 手测全部按预期返回 【文档同步】 - CURRENT_STATUS.md +232 行:新增 Task 0/1/2 完整记录 + 代码审查修复章节 + 延后清单 - project-context.mdc:Task 0/1/2 状态同步 - phase2e-2-design.md +20 行:Fastify 5 升级相关决策记录 - phase2e-2-implementation.plan.md +117 行:Task 0/1/2 实际产出 + 修复记录 余下 Minor/Nits(m4/m6~m10/n1~n10)登记至 Task 16 收尾清单一次性清扫。 Made-with: Cursor
82 lines
2.4 KiB
TypeScript
82 lines
2.4 KiB
TypeScript
import { afterAll, afterEach, beforeAll, describe, expect, it } from 'vitest';
|
|
|
|
import { config } from '../../src/config.js';
|
|
import { closeWorker, startWorker } from '../../src/mediasoup/worker.js';
|
|
import {
|
|
_clearRouterMap,
|
|
closeRouter,
|
|
createRouter,
|
|
getRouter,
|
|
getRouterStats,
|
|
tryGetRouter,
|
|
} from '../../src/services/router.service.js';
|
|
import { AppError } from '../../src/utils/errors.js';
|
|
|
|
beforeAll(async () => {
|
|
await startWorker();
|
|
});
|
|
|
|
afterAll(async () => {
|
|
_clearRouterMap();
|
|
await closeWorker();
|
|
});
|
|
|
|
afterEach(() => {
|
|
_clearRouterMap();
|
|
});
|
|
|
|
describe('router.service', () => {
|
|
it('creates a router and returns rtpCapabilities', async () => {
|
|
const { routerId, rtpCapabilities } = await createRouter('ROOM-TEST-1');
|
|
expect(routerId).toBeTruthy();
|
|
expect(Array.isArray(rtpCapabilities.codecs)).toBe(true);
|
|
expect(rtpCapabilities.codecs?.length).toBeGreaterThan(0);
|
|
|
|
const stats = getRouterStats();
|
|
expect(stats.total).toBe(1);
|
|
expect(stats.rooms[0]?.roomCode).toBe('ROOM-TEST-1');
|
|
});
|
|
|
|
it('getRouter throws NOT_FOUND for unknown id', () => {
|
|
try {
|
|
getRouter('non-existent-id');
|
|
throw new Error('expected throw');
|
|
} catch (err) {
|
|
expect(err).toBeInstanceOf(AppError);
|
|
expect((err as AppError).code).toBe('NOT_FOUND');
|
|
}
|
|
});
|
|
|
|
it('tryGetRouter returns undefined instead of throwing', () => {
|
|
expect(tryGetRouter('non-existent-id')).toBeUndefined();
|
|
});
|
|
|
|
it('closeRouter removes entry via observer close event', async () => {
|
|
const { routerId } = await createRouter('ROOM-TEST-2');
|
|
expect(getRouterStats().total).toBe(1);
|
|
|
|
await closeRouter(routerId);
|
|
// observer.once('close') executes synchronously when router.close() is called
|
|
expect(getRouterStats().total).toBe(0);
|
|
});
|
|
|
|
it('closeRouter throws NOT_FOUND for unknown router', async () => {
|
|
await expect(closeRouter('non-existent-id')).rejects.toBeInstanceOf(AppError);
|
|
});
|
|
|
|
it('enforces router limit', async () => {
|
|
const originalLimit = config.mediasoup.maxRouters;
|
|
(config.mediasoup as { maxRouters: number }).maxRouters = 2;
|
|
|
|
try {
|
|
await createRouter('ROOM-LIMIT-1');
|
|
await createRouter('ROOM-LIMIT-2');
|
|
await expect(createRouter('ROOM-LIMIT-3')).rejects.toMatchObject({
|
|
code: 'ROUTER_LIMIT_EXCEEDED',
|
|
});
|
|
} finally {
|
|
(config.mediasoup as { maxRouters: number }).maxRouters = originalLimit;
|
|
}
|
|
});
|
|
});
|