 徐涛
|
85b92ae9d4
|
fix(examples): 修复 Phase 8 端到端示例审查发现的 6 项问题
按 PM/SA/Code Reviewer 三方审计报告修复：
🔴 阻塞修复:
- CalcTool 除零 panic: `a / b` 改 `a.checked_div(b).ok_or_else(...)`,
b=0 时返回 ToolError::InvalidArguments 而非 panic
🟡 警告修复:
- end_to_end.rs 持久化验证注释与实际不符: 显式 drop(backend) 让注释
描述与 Arc 释放顺序一致
- quick_start EchoTool 参数验证: 用 args.get("text").and_then().ok_or_else()
替换 as_str().unwrap_or("") 静默降级, 缺失/类型错误时返回显式错误
💭 一致性修复:
- end_to_end.rs EchoTool 与 quick_start 一致化 (format!("收到: {text}"))
- quick_start mock 响应文本 "已通过 echo 回传" → "EchoTool 已收到并完成回传"
- quick_start 断言改为检查 "收到", 与方案字面要求一致
- quick_start 末尾追加 POSIX trailing newline
验收: cargo test 200/0 + clippy 0 警告 + doc 0 warning + 10 示例 exit 0
|
2026-07-05 21:00:36 +08:00 |
|