fix: 同步 4 个 post-PR#26 commit 到 Gitea #31

2026-06-11T00:08:32Z

pangtong-fujunshi commented

2026-06-11 00:08:32 +00:00

将 PR #26 之后的 4 个 fix commit cherry-pick 到 gitea main：

36cee16 fix: review 修复
603af2a fix: import 路径
83694ad fix: SKILL_BASE_PATH
846fcbd docs: E2E 验证报告

纯同步，无新代码。

将 PR #26 之后的 4 个 fix commit cherry-pick 到 gitea main： 1. 36cee16 fix: review 修复 2. 603af2a fix: import 路径 3. 83694ad fix: SKILL_BASE_PATH 4. 846fcbd docs: E2E 验证报告纯同步，无新代码。

pangtong-fujunshi added 4 commits 2026-06-11 00:08:33 +00:00

fix: v3.0→HEAD review 修复 — handler 注册 + review verdict + skill 全文注入 25863634c2

基于庞统+司马懿背靠背 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

fix: dispatcher _legacy_on_complete 闭包内删除错误的重复 import 6bb0596e6d

get_connection 已在文件顶部 L22 import（from src.blackboard.db），
闭包内 L258 错误地 from src.daemon.db import（模块不存在），
会触发 ImportError 被 try/except 吞掉导致 review verdict 逻辑不生效。

司马懿 review 发现。

fix: task_handler SKILL_BASE_PATH 硬编码改为环境变量 3c30a9b135

与 bootstrap.py 保持一致，支持 MOZI_SKILL_PATH 环境变量覆盖。
默认值不变。

docs: §21 handler 注册后 E2E 验证报告

CI / lint (pull_request) Failing after 8s

Details

CI / test (pull_request) Has been skipped

Details

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

Details

d98fae007f

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 路径待明天验证。

pangtong-fujunshi merged commit 4bf3d97a04 into main

2026-06-11 00:08:44 +00:00

pangtong-fujunshi referenced this issue from a commit

2026-06-11 00:08:47 +00:00

Merge pull request 'fix: 同步 4 个 post-PR#26 commit 到 Gitea' (#31) from fix/sync-to-gitea into main

gitea-actions bot commented

2026-06-11 00:09:03 +00:00

[CI] 失败

分支: 31
触发 commit: d98fae007f583d500ad65872fe4d273287feab4a
失败 Job: lint
请检查 CI 日志并修复。

[CI] 失败分支: 31 触发 commit: `d98fae007f583d500ad65872fe4d273287feab4a` 失败 Job: lint 请检查 CI 日志并修复。

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#31