fixbug: function call rewrite request miss model

[muverystrong]

rewrite丢失model, 转发请求queue会报错。

[szl97]

@claude 检查提交的代码，使用中文回复

[szl97]

Claude finished @szl97's task —— View job

---

代码检查任务

• 读取变更文件了解具体修改内容
• 分析代码变更的正确性和潜在问题
• 提供详细的中文代码审查反馈

代码审查结果

✅ 修复内容正确性

这个修复是正确且必要的。在 SimulationHepler.java:99 的 rewrite 方法中，原本创建新的 CompletionRequest 时缺失了 .model(req.getModel()) 调用，导致重写后的请求丢失了模型参数。

📝 问题背景分析

• 原始问题: rewrite 方法用于重写带有 function call 的请求，将其转换为模拟执行的格式
• 缺失影响: 没有设置 model 参数会导致下游处理时无法确定使用哪个模型，从而引起队列转发错误
• 修复效果: 添加 .model(req.getModel()) 确保重写后的请求保持与原请求相同的模型配置

🔍 代码质量检查

✅ 优点:

• 修复位置准确，在 builder 模式的合适位置添加了缺失的参数
• 保持了与其他参数设置的一致性（如 user, temperature 等）
• 修复简洁，没有引入不必要的复杂性

⚠️ 注意事项:

• 建议添加单元测试验证 rewrite 方法确实保留了原请求的 model 参数
• 可以考虑在方法开始处添加 model 参数的空值检查，提高健壮性

📋 建议的后续改进

// 在 rewrite 方法开始处可以添加参数验证
if (req.getModel() == null || req.getModel().isEmpty()) {
    throw new IllegalArgumentException("Model parameter is required for request rewrite");
}

🎯 总结

这是一个高质量的 bug 修复，准确地解决了函数调用重写时丢失模型参数的问题。修改最小化且有效，符合代码规范。建议合并此 PR。

---