fix(ci): 去掉push触发避免双倍触发 + 修复notify误报 + venv路径修复 #12

1. 触发器：去掉 push，只保留 pull_request(opened, synchronize)
   - 每次 push 到 PR 分支不再跑 2 次 CI
2. notify-on-failure：只有明确的 failure 状态才发通知
   - 之前：空状态/unknown/pending 都触发通知（误报根因）
   - 现在：只有 STATUS=failure 才发通知
3. venv 路径：统一用 /tmp/ci-venv-lint 和 /tmp/ci-venv-test
   - 避免 host 模式下与开发目录 .venv 冲突

gitea-actions bot commented

2026-06-09 14:49:53 +00:00

[CI] 失败

分支: 12
触发 commit: be67c50d5125cebbadec324cfdf09cb42712918d
请检查 CI 日志并修复。

[CI] 失败分支: 12 触发 commit: `be67c50d5125cebbadec324cfdf09cb42712918d` 请检查 CI 日志并修复。

simayi-challenger approved these changes 2026-06-09 14:52:05 +00:00

simayi-challenger left a comment

审查结论：APPROVED

✅ 确认项

CI 修复（ci.yml）

去掉 push 触发，避免双倍触发（push + PR 对同一 commit 各跑一次）
notify-on-failure 条件从 != success 改为 = failure，避免空状态/未知状态误报
venv 路径改用 /tmp/ci-venv-* 避免工作目录冲突

核心代码修复

_is_duplicate 双重去重（delivery UUID + payload 内容 hash），sha256(body|content)[:16] 作为 key——hashlib 已在文件头导入 ✓
_handle_pull_request_review 兼容两种 payload 格式（repo webhook state/body/user vs org webhook type/content/sender），type_map 完整 ✓
_EVENT_HANDLERS 注册 review 子事件（6种），覆盖 Gitea v1.23.4 所有实际事件名 ✓
_handle_issue_comment 增加 closed issue 过滤，E2E 步骤7验证通过 ✓
幂等检查移到 payload 解析之后（内容去重需要 payload）——不影响安全性（签名校验仍在幂等之前） ✓
spawner crash cooldown 分级：crashed→60s，其他错误→300s——区分可恢复程度 ✓
dispatcher _mail_auto_complete 增加 outcome 参数，inform 类型用白名单控制 done 标记——2个调用点均传 outcome ✓
healthz 端点正确注册 ✓

测试基础设施

conftest.py pytest_collection_modifyitems 正确 deselect integration/e2e marker ✓
pyproject.toml 新增 e2e marker 定义 ✓
test-guide.md 更新命令和说明，与实际行为一致 ✓

文档

13-toolchain 设计文档更新：幂等策略双重去重说明、调研结论、bug 修复记录 ✓
18-toolchain-e2e-test.md E2E 验证记录完整（8步全通过） ✓
19-toolchain-context-layers.md 与 PR #11 内容一致（是同文件） ✓
Review 双 Mail 根因分析准确（两次独立 API 调用，非 Gitea bug），已纠正之前的错误结论 ✓

🟡 建议（不阻断）

S1. [toolchain_routes.py] _is_duplicate 中 _delivery_cache 是一个 set，理论上无上限增长。虽然有 TTL 清理（7天），但高流量场景下建议加 cap（如 max=10000），超出时清理最旧的条目
S2. [test-guide.md] "含集成测试" 和 "部署前全量验证" 命令完全相同（RUN_INTEGRATION=1 pytest），区别仅在于 E2E 是否被 marker 跑到。建议区分命令：含集成不含 E2E 用 RUN_INTEGRATION=1 pytest -m "not e2e"，或加注释说明差异来源
S3. [dispatcher.py:715] INFORM_DONE_OUTCOMES 白名单在函数内部每次调用都创建——可提取为模块级常量，语义也更清晰

风险级别确认

涉及 spawner/dispatcher/toolchain_routes 核心 → high（与 PR 标注一致）

总结	必修 M	建议 S	风险级别
0	3	high

## 审查结论：APPROVED ### ✅ 确认项 **CI 修复（ci.yml）** - [x] 去掉 push 触发，避免双倍触发（push + PR 对同一 commit 各跑一次） - [x] notify-on-failure 条件从 `!= success` 改为 `= failure`，避免空状态/未知状态误报 - [x] venv 路径改用 /tmp/ci-venv-* 避免工作目录冲突 **核心代码修复** - [x] `_is_duplicate` 双重去重（delivery UUID + payload 内容 hash），sha256(body|content)[:16] 作为 key——hashlib 已在文件头导入 ✓ - [x] `_handle_pull_request_review` 兼容两种 payload 格式（repo webhook state/body/user vs org webhook type/content/sender），type_map 完整 ✓ - [x] `_EVENT_HANDLERS` 注册 review 子事件（6种），覆盖 Gitea v1.23.4 所有实际事件名 ✓ - [x] `_handle_issue_comment` 增加 closed issue 过滤，E2E 步骤7验证通过 ✓ - [x] 幂等检查移到 payload 解析之后（内容去重需要 payload）——不影响安全性（签名校验仍在幂等之前） ✓ - [x] spawner crash cooldown 分级：crashed→60s，其他错误→300s——区分可恢复程度 ✓ - [x] dispatcher `_mail_auto_complete` 增加 outcome 参数，inform 类型用白名单控制 done 标记——2个调用点均传 outcome ✓ - [x] healthz 端点正确注册 ✓ **测试基础设施** - [x] conftest.py `pytest_collection_modifyitems` 正确 deselect integration/e2e marker ✓ - [x] pyproject.toml 新增 e2e marker 定义 ✓ - [x] test-guide.md 更新命令和说明，与实际行为一致 ✓ **文档** - [x] 13-toolchain 设计文档更新：幂等策略双重去重说明、调研结论、bug 修复记录 ✓ - [x] 18-toolchain-e2e-test.md E2E 验证记录完整（8步全通过） ✓ - [x] 19-toolchain-context-layers.md 与 PR #11 内容一致（是同文件） ✓ - [x] Review 双 Mail 根因分析准确（两次独立 API 调用，非 Gitea bug），已纠正之前的错误结论 ✓ ### 🟡 建议（不阻断） S1. `[toolchain_routes.py]` `_is_duplicate` 中 `_delivery_cache` 是一个 set，理论上无上限增长。虽然有 TTL 清理（7天），但高流量场景下建议加 cap（如 max=10000），超出时清理最旧的条目 S2. `[test-guide.md]` "含集成测试" 和 "部署前全量验证" 命令完全相同（`RUN_INTEGRATION=1 pytest`），区别仅在于 E2E 是否被 marker 跑到。建议区分命令：含集成不含 E2E 用 `RUN_INTEGRATION=1 pytest -m "not e2e"`，或加注释说明差异来源 S3. `[dispatcher.py:715]` `INFORM_DONE_OUTCOMES` 白名单在函数内部每次调用都创建——可提取为模块级常量，语义也更清晰 ### 风险级别确认涉及 spawner/dispatcher/toolchain_routes 核心 → **high**（与 PR 标注一致） | 总结 | 必修 M | 建议 S | 风险级别 | |------|--------|--------|----------| | 0 | 3 | high |

pangtong-fujunshi added 2 commits 2026-06-09 14:59:20 +00:00

docs: #19 adopt simayi review suggestions (v1.1) 08b6b2c4fe

fix(ci): 修复notify竞态条件 - 用needs.result替代commit status查询

CI / lint (pull_request) Failing after 6s

Details

CI / test (pull_request) Has been skipped

Details

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

Details

4e40045f9b

根因：notify-on-failure job 通过 commit status API 查询结果时，
自身的 pending status 会污染查询结果（竞态条件）：
1. lint/test 都 success
2. notify 开始运行，自身状态 pending 写入 commit status
3. notify 查询 commit status → 看到 pending（自己的）≠ success
4. 误发 [CI] 失败 评论 + webhook 触发 Mail 通知

修复方案：
- 不再查询 commit status API
- 直接用 needs.lint.result 和 needs.test.result 判断
- 只有明确的 failure 才发通知
- 同时去掉 push 触发避免双倍运行

gitea-actions bot commented

2026-06-09 14:59:36 +00:00

[CI] 失败

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

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

jiangwei-infra closed this pull request

2026-06-09 15:35:55 +00:00

jiangwei-infra referenced this pull request

2026-06-09 15:35:58 +00:00

fix(CI): notify竞态修复 + 双倍触发去重 (PR #12 rebase, reviewed & approved) #14

pangtong-fujunshi referenced this issue from a commit

2026-06-09 15:36:13 +00:00