执行摘要
本次PR修复了diffusion生成器中--prompt-path参数被解析但从未消费的bug，通过将prompt_path从sampling_params_kwargs传递到_resolve_prompts方法并设置优先级，确保用户能正确使用该参数指定提示文件。影响范围小，风险低，测试覆盖充分，是重要的功能修复。
功能与动机
问题背景：在sglang的diffusion生成器中，SamplingParams的CLI参数--prompt-path被解析但从未在运行时使用，导致用户无法通过此参数指定提示文件。仅--prompt-file-path（ServerArgs）在_resolve_prompts()中被检查，造成功能缺失。
解决目标：修复此bug，使prompt_path参数正确消费，优先级高于server_args.prompt_file_path，并保持向后兼容性。PR body中明确指出："--prompt-path (SamplingParams CLI arg) was parsed but never consumed by the runtime"。
实现拆解
-
核心代码修改：diffusion_generator.py中的generate方法新增prompt_path参数传递：
prompts = self._resolve_prompts(
sampling_params_kwargs.get("prompt"),
sampling_params_kwargs.get("prompt_path"),
)
同时修复null安全问题，初始化sampling_params_kwargs为空字典。
-
_resolve_prompts方法更新：新增prompt_path参数，优先级逻辑为path = prompt_path or self.server_args.prompt_file_path，并添加文件检查和错误处理：
- 文件不存在时抛出
FileNotFoundError。
- 从文件读取多行提示（一行一个）。
-
测试增强：新增test_resolve_prompts.py单元测试，覆盖场景包括：
- 内联提示（字符串、列表）。
prompt_path单行和多行文件。prompt_path优先于server_args.prompt_file_path。- 错误案例（文件缺失、空文件）。
在run_suite.py中集成测试到测试套件。
评论区精华
review讨论中，gemini-code-assist[bot]提出了两个关键改进：
"The generate method signature explicitly allows sampling_params_kwargs to be None, but the implementation immediately calls .get() on it, which will raise an AttributeError. Initializing it to an empty dictionary at the start of the method ensures robustness."
"The help text was changed to singular ('the prompt'), but the implementation in _resolve_prompts still supports multiple prompts (one per line)... The original help text was more accurate."
这些建议在提交中被采纳，避免了潜在崩溃并确保了文档准确性。
风险与影响
- 技术风险：
sampling_params_kwargs为None的风险已通过初始化修复。- 优先级引入可能影响旧版用户，但测试验证了
--prompt-file-path仍正常工作，兼容性良好。
- 文件路径处理中的错误已通过异常处理覆盖。
- 影响范围：仅限diffusion生成器的提示解析逻辑，用户现在能灵活使用
--prompt-path参数。对系统性能、安全无影响，局部修复提升了功能完整性。
关联脉络
与历史PR的关联显示diffusion模块的持续演进：
- PR #21763（CI改善）与本PR都聚焦diffusion测试，强调了CI稳定性和测试覆盖的重要性。
- PR #20379（添加日志前缀排除）展示了diffusion服务器参数的扩展，本PR则修复了参数消费的bug，共同体现了参数处理功能的完善。
这些PR表明团队在强化diffusion模块的可靠性和用户体验，本次修复是这一趋势的具体体现。
参与讨论