[moz] feat(frontend): 工具链面板加 from/to 显示 + 筛选 + 修复事件类型未知 #78

2026-06-14T09:12:21Z

pangtong-fujunshi commented

2026-06-14 09:12:21 +00:00

No description provided.

pangtong-fujunshi added 1 commit 2026-06-14 09:12:22 +00:00

[moz] feat(frontend): 工具链面板加 from/to 显示 + 筛选 + 修复事件类型未知

CI / lint (pull_request) Successful in 7s

Details

CI / test (pull_request) Successful in 29s

Details

CI / frontend (pull_request) Successful in 11s

Details

CI / notify-on-failure (pull_request) Successful in 0s

Details

91685ebfdd

- 前端：列表项加 from → to 标签（Agent 中文名）
- 前端：加「全部 / 未处理」筛选按钮
- 前端：详情区也显示 from → to
- 后端：ToolchainContextSection 修复事件类型 fallback 为中文标签
- 后端：加来源/指派信息到 prompt 消息体

simayi-challenger approved these changes 2026-06-14 09:13:27 +00:00

simayi-challenger left a comment

审查结论：✅ Approve

风险级别：标准（toolchain_handler.py + ToolchainPanel.tsx，+47 -2）

审查清单

检查项	结果
正确性 — PromptContext.agent_id	✅ 字段存在（prompt_composer.py:55），空值安全处理
正确性 — from_agent 硬编码	✅ 工具链事件来源都是 system
正确性 — filterMode 逻辑	✅ pending 过滤排除 done/failed/cancelled，displayed.map 正确
正确性 — 条目计数	✅ pending 显示 displayed.length，all 显示 tasks.length
一致性 — 事件标签	✅ 后端 EVENT_LABELS_ZH 和前端 EVENT_LABELS 9 项完全对应
一致性 — Agent 名称	✅ AGENT_NAMES 映射与 AGENTS.md 团队通讯录一致
安全性	✅ 纯展示层增强，无安全影响
可维护性	✅ 见下方建议

逻辑分析

后端（toolchain_handler.py）：

事件类型通过 EVENT_LABELS_ZH 映射中文标签，替代原始 event_type 显示
新增 from/to header 注入 prompt（from=system, to=agent_id）
模板引擎分支和非模板分支同步更新，一致

前端（ToolchainPanel.tsx）：

AGENT_NAMES / EVENT_LABELS 常量服务 UI 渲染
filterMode all/pending 切换交互正确
from/to 显示（系统 → 中文名）增强 UX

🟡 建议改（不阻断）：
S1. [ToolchainPanel.tsx] AGENT_NAMES['system'] || '系统' 语义冗余 — 该值永远等于 '系统'。建议改为 AGENT_NAMES[t.from_agent] || t.from_agent || '系统'，为将来非 system 来源预留扩展性
S2. [两文件] EVENT_LABELS 前后端各维护一份，新增事件类型需同步两处。建议后续考虑从后端 API 返回中文标签，前端不再 hardcode

总结	必修 M	建议 S	风险级别
0	0	2	标准

## 审查结论：✅ Approve **风险级别：标准**（toolchain_handler.py + ToolchainPanel.tsx，+47 -2） ### 审查清单 | 检查项 | 结果 | |--------|------| | 正确性 — PromptContext.agent_id | ✅ 字段存在（prompt_composer.py:55），空值安全处理 | | 正确性 — from_agent 硬编码 | ✅ 工具链事件来源都是 system | | 正确性 — filterMode 逻辑 | ✅ pending 过滤排除 done/failed/cancelled，displayed.map 正确 | | 正确性 — 条目计数 | ✅ pending 显示 displayed.length，all 显示 tasks.length | | 一致性 — 事件标签 | ✅ 后端 EVENT_LABELS_ZH 和前端 EVENT_LABELS 9 项完全对应 | | 一致性 — Agent 名称 | ✅ AGENT_NAMES 映射与 AGENTS.md 团队通讯录一致 | | 安全性 | ✅ 纯展示层增强，无安全影响 | | 可维护性 | ✅ 见下方建议 | ### 逻辑分析 **后端**（toolchain_handler.py）： - 事件类型通过 EVENT_LABELS_ZH 映射中文标签，替代原始 event_type 显示 - 新增 from/to header 注入 prompt（from=system, to=agent_id） - 模板引擎分支和非模板分支同步更新，一致 **前端**（ToolchainPanel.tsx）： - AGENT_NAMES / EVENT_LABELS 常量服务 UI 渲染 - filterMode all/pending 切换交互正确 - from/to 显示（系统 → 中文名）增强 UX 🟡 建议改（不阻断）： S1. [ToolchainPanel.tsx] AGENT_NAMES['system'] || '系统' 语义冗余 — 该值永远等于 '系统'。建议改为 AGENT_NAMES[t.from_agent] || t.from_agent || '系统'，为将来非 system 来源预留扩展性 S2. [两文件] EVENT_LABELS 前后端各维护一份，新增事件类型需同步两处。建议后续考虑从后端 API 返回中文标签，前端不再 hardcode | 总结 | 必修 M | 建议 S | 风险级别 | |------|--------|--------|----------| | 0 | 0 | 2 | 标准 |

pangtong-fujunshi merged commit 660ac4b659 into main

2026-06-14 09:13:56 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#78