Prhub

#25947 fix(model-gateway): rustfmt nightly in conversations/handlers.rs

原始 PR 作者 whybeyoung 合并时间 2026-05-21 14:29 文件变更 1 提交数 1 评论 2 代码增减 +1 / -1

执行摘要

修复 Rust 格式化检查失败

PR body 中说明:json!({"error": error_obj}) 不满足 #19524 引入的 nightly rustfmt 钩子要求,应用格式化工具建议的空格 json!({ "error": error_obj }) 以使 cargo +nightly fmt --check 通过。这是一个纯格式修复,旨在保持 CI 通过。

该 PR 属于基础设施维护,可快速合入。建议后续在 CI 中统一强制格式化检查,避免类似问题反复出现。

讨论亮点

无 review 讨论。

实现拆解

  1. 修改 sgl-model-gateway/src/routers/conversations/handlers.rs 第 86 行:在 json! 宏的大括号内添加空格,即将 json!({"error": error_obj}) 改为 json!({ "error": error_obj })
  2. 该修改仅影响 bad_request_structured 函数的实现,不改变逻辑行为。
  3. 无测试、配置或部署配套改动。
文件 模块 状态 重要度
sgl-model-gateway/src/routers/conversations/handlers.rs 请求路由 modified 4.08

关键符号

bad_request_structured

关键源码片段

sgl-model-gateway/src/routers/conversations/handlers.rs style

唯一变更文件,修复了 `bad_request_structured` 函数中的 `json!` 宏格式,确保通过 nightly rustfmt 检查。

// bad_request_structured: 返回结构化的错误响应
// 仅将 json! 宏中的空格调整为 nightly rustfmt 要求
fn bad_request_structured(error_obj: Value) -> Response {
    // 原代码 : (StatusCode::BAD_REQUEST, Json(json!({"error": error_obj}))).into_response()
    // 修复后:添加空格以通过 cargo +nightly fmt --check
    (StatusCode::BAD_REQUEST, Json(json!({ "error": error_obj }))).into_response()
}

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

该 PR 仅为格式调整,不涉及逻辑变更,风险极低。但若其他函数中也有类似未格式化的 json! 宏,未来可能继续触发 CI 失败,建议统一审查。

正面影响:修复 CI 格式化检查,使 cargo +nightly fmt --check 通过,避免因格式问题阻塞流水线。负面影响:无。

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论