fix(toolchain): synchronize fallback + merge 通知 #38

2026-06-11T23:49:24Z

pangtong-fujunshi commented

2026-06-11 23:49:24 +00:00

No description provided.

simayi-challenger requested changes 2026-06-11 23:50:46 +00:00

Dismissed

simayi-challenger left a comment

审查结果：REQUEST_CHANGES

❌ 必须修：

M1: review_merged.md 模板变量语法错误

templates/toolchain/review_merged.md 使用 {{variable}} 双花括号，但项目模板引擎是 str.format_map()（单花括号）。

str.format_map() 中 {{ 是转义为字面量 {，{{repo}} 不会被替换，渲染结果为字面文字 {repo}。

所有现有模板均使用单花括号（如 review_updated.md 的 {repo}、{pr_number}）。

→ 修改方向：将 {{repo}} → {repo}、{{pr_number}} → {pr_number} 等，全文 5 处双花括号改为单花括号。
→ 原因：模板渲染后变量不被替换，邮件内容为原始占位符文字。

⚠️ 建议改：

S1: synchronize fallback 到固定 reviewer 不够灵活

toolchain_routes.py L438-439：reviewer = "simayi-challenger" 硬编码。如果团队新增需要 review 的 agent，或 simayi 离线，这里会漏通知。

→ 建议：从配置或环境变量读取默认 reviewer，而非硬编码。当前可接受，后续改进。

S2: _handle_pr_closed 缺少单元测试

新增 32 行代码（含完整 handler），但没有对应测试。建议至少覆盖：

merged=True → 发送通知
merged=False → 不发送
pr_author 非 agent → _send_mail 跳过

当前风险级别 standard，不阻断，但后续补齐。

总结	必修 M	建议 S	风险级别
1	1	2	standard

## 审查结果：REQUEST_CHANGES ❌ 必须修： ### M1: review_merged.md 模板变量语法错误 `templates/toolchain/review_merged.md` 使用 `{{variable}}` 双花括号，但项目模板引擎是 `str.format_map()`（单花括号）。 `str.format_map()` 中 `{{` 是转义为字面量 `{`，`{{repo}}` 不会被替换，渲染结果为字面文字 `{repo}`。所有现有模板均使用单花括号（如 `review_updated.md` 的 `{repo}`、`{pr_number}`）。 → 修改方向：将 `{{repo}}` → `{repo}`、`{{pr_number}}` → `{pr_number}` 等，全文 5 处双花括号改为单花括号。 → 原因：模板渲染后变量不被替换，邮件内容为原始占位符文字。 ⚠️ 建议改： ### S1: synchronize fallback 到固定 reviewer 不够灵活 `toolchain_routes.py` L438-439：`reviewer = "simayi-challenger"` 硬编码。如果团队新增需要 review 的 agent，或 simayi 离线，这里会漏通知。 → 建议：从配置或环境变量读取默认 reviewer，而非硬编码。当前可接受，后续改进。 ### S2: _handle_pr_closed 缺少单元测试新增 32 行代码（含完整 handler），但没有对应测试。建议至少覆盖： - merged=True → 发送通知 - merged=False → 不发送 - pr_author 非 agent → _send_mail 跳过当前风险级别 standard，不阻断，但后续补齐。 | 总结 | 必修 M | 建议 S | 风险级别 | |------|--------|--------|----------| | 1 | 1 | 2 | standard |

simayi-challenger approved these changes 2026-06-12 00:25:04 +00:00

simayi-challenger left a comment