执行摘要

• 一句话：回滚OpenAI协议中为视频基准测试临时添加的video_fps参数。
• 推荐动作：该PR值得快速浏览以了解API清理决策，但技术内容简单。关注点：1) 回滚决策反映临时参数应避免加入核心协议；2) 视频处理参数传递的最佳实践（通过mm_processor_kwargs）。无需深入代码分析。

功能与动机

根据review评论，回滚原因未在PR body中明确说明，但推测是video_fps参数设计为临时性（原commit注释标注为"temporary parameter"），且视频处理应通过mm_processor_kwargs传递。fastdeploy-bot的评论建议补充回滚原因，例如"该参数设计不合理、与现有机制重复、或后续将通过mm_processor_kwargs传递等"。

实现拆解

该PR仅修改一个文件：fastdeploy/entrypoints/openai/protocol.py。具体改动是从CompletionRequest和ChatCompletionRequest两个Pydantic模型类中删除video_fps字段（Optional[float]类型）及其相关注释。删除操作通过revert commit实现，移除了6行代码。

关键文件：

• fastdeploy/entrypoints/openai/protocol.py（模块 APIServer）: 唯一修改文件，包含OpenAI协议请求类定义，删除video_fps参数直接影响API接口。

关键符号：CompletionRequest.to_dict_for_infer, ChatCompletionRequest.to_dict_for_infer

评论区精华

review讨论主要集中在PR规范而非技术内容：1) Copilot指出PR标题缺少标签，建议使用[Others]或[APIServer]；2) fastdeploy-bot详细列出规范问题：标题标签应为[APIServer]而非[Other]，描述缺少回滚原因和文档更新说明。讨论结论是回滚代码正确，但需完善标题和描述以符合规范。未解决的技术疑虑：回滚后视频基准测试如何传递fps参数未明确说明。

• PR标题标签规范 (style): 应调整标题以符合规范，但PR已合并未修改。
• PR描述完整性 (documentation): 描述未补充，但回滚操作被接受。

风险与影响

• 风险：技术风险较低：1) 回归风险：如果视频基准测试仍依赖video_fps参数，可能导致功能失效或错误，但原参数标记为临时性，且fastdeploy-bot评论指出视频处理可通过mm_processor_kwargs传递，因此影响可能有限；2) 兼容性风险：删除API字段可能影响客户端调用，但该参数为可选且临时添加，风险可控；3) 安全风险：无。
• 影响：影响范围：1) 用户影响：使用OpenAI协议且依赖video_fps参数的视频基准测试用户需调整参数传递方式，可能通过mm_processor_kwargs；2) 系统影响：简化协议定义，移除临时字段，提升API一致性；3) 团队影响：需更新相关文档或测试用例以反映变更。影响程度：小范围，仅涉及特定基准测试场景。
• 风险标记：API变更, 临时参数移除

关联脉络

• PR #7077 [Other] support video_fps args for video bench: 该PR回滚的正是#7077引入的video_fps参数，两者直接关联，反映了参数从添加到移除的完整生命周期。
• PR #7183 [Optimization] Enable text-only deployment for multimodal models: 涉及多模态模型处理，可能关联视频参数传递机制（如mm_processor_kwargs），提供上下文。
• PR #7109 [DataProcessor] Move image_processor to unified directory and add MultiModalProcessor: 涉及多模态处理器统一，可能与视频参数处理方式相关。