 cfdaily
|
d98fae007f
|
docs: §21 handler 注册后 E2E 验证报告
CI / lint (pull_request) Failing after 8s
CI / test (pull_request) Has been skipped
CI / notify-on-failure (pull_request) Successful in 3s
Mail/Toolchain 核心流程全部通过：
- Mail: inform auto-working → auto-done ✅
- Toolchain Issue 指派: webhook → Mail ✅
- Toolchain PR Review: webhook → Review 请求 → Review 结果 ✅
- CI 失败重复 Mail 问题确认（org+repo webhook 双触发，已知）
Task review 路径待明天验证。
|
2026-06-11 08:08:20 +08:00 |
|
|
cfdaily
|
3c30a9b135
|
fix: task_handler SKILL_BASE_PATH 硬编码改为环境变量
与 bootstrap.py 保持一致，支持 MOZI_SKILL_PATH 环境变量覆盖。
默认值不变。
|
2026-06-11 08:08:20 +08:00 |
|
|
cfdaily
|
6bb0596e6d
|
fix: dispatcher _legacy_on_complete 闭包内删除错误的重复 import
get_connection 已在文件顶部 L22 import（from src.blackboard.db），
闭包内 L258 错误地 from src.daemon.db import（模块不存在），
会触发 ImportError 被 try/except 吞掉导致 review verdict 逻辑不生效。
司马懿 review 发现。
|
2026-06-11 08:08:20 +08:00 |
|
|
cfdaily
|
25863634c2
|
fix: v3.0→HEAD review 修复 — handler 注册 + review verdict + skill 全文注入
基于庞统+司马懿背靠背 review，修复 6 个问题：
P0 致命：
- A1: _legacy_on_complete 补回 review verdict 处理（approved→done，非 approved→@mention assignee）
- A2: 添加 TaskTypeRegistry.register() 启动初始化（注册 Task/Mail/Toolchain handler）
P1 中等：
- B11-1: RoleSkillSection 从索引提示改为全文注入（对齐设计 §2.3 + BootstrapBuilder 行为）
- A8: retry prompt is_mail 硬编码改走 TaskTypeRegistry handler 判断
P2 低：
- _mail_* 4 个方法添加 DEPRECATED 注释
- ticker.py handler check_completion 代码块缩进对齐（28→24 空格）
测试：394 passed, 0 failed
Review reports: docs/design/review-v3-vs-head-{pangtong,simayi}.md
|
2026-06-11 08:08:20 +08:00 |
|