sanguo/sanguo_moziplus_v2
8
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: Review去重+crash cooldown+healthz+E2E验证 #3

Closed
pangtong-fujunshi wants to merge 39 commits from fix/toolchain-review-dedup-ci-cooldown into main
pull from: fix/toolchain-review-dedup-ci-cooldown
merge into: sanguo:main
Conversation 12 Commits 39 Files Changed 30
+714 -128
pangtong-fujunshi commented 2026-06-09 04:10:19 +00:00
Member
Copy Link
Copy Source
No description provided.
pangtong-fujunshi added 34 commits 2026-06-09 04:10:20 +00:00
auto-sync: 2026-06-08 21:59:26 f0a673ff20
auto-sync: 2026-06-08 22:04:07 0546825642
auto-sync: 2026-06-08 22:11:11 f32991ddee
auto-sync: 2026-06-08 22:26:47 b5d26da914
auto-sync: 2026-06-08 22:58:35 0b7bb288f9
auto-sync: 2026-06-08 23:19:23 55fc25d9a6
auto-sync: 2026-06-08 23:20:47 b2ace1b6a7
auto-sync: 2026-06-08 23:21:31 34335a6487
auto-sync: 2026-06-08 23:22:36 473ae73230
auto-sync: 2026-06-08 23:23:43 12f03e48a4
auto-sync: 2026-06-08 23:37:25 339519a062
auto-sync: 2026-06-08 23:37:35 (catch-all) 5d83747e99
auto-sync: 2026-06-08 23:38:34 (catch-all) 29438a5789
auto-sync: 2026-06-08 23:38:59 (catch-all) 81cca26adb
auto-sync: 2026-06-08 23:39:15 041f54e699
auto-sync: 2026-06-09 00:14:14 632ca35681
auto-sync: 2026-06-09 00:14:25 (catch-all) 96c8378a91
auto-sync: 2026-06-09 00:24:51 f00aeb96e9
auto-sync: 2026-06-09 00:38:45 60195f6250
auto-sync: 2026-06-09 00:44:21 639fb3ecea
auto-sync: 2026-06-09 00:45:24 4492a75e7e
auto-sync: 2026-06-09 00:45:50 67a187aa0f
auto-sync: 2026-06-09 07:46:02 3f1daa9f8d
auto-sync: 2026-06-09 07:46:23 795cfa81d1
auto-sync: 2026-06-09 08:06:43 cf7e136330
auto-sync: 2026-06-09 08:30:45 5010ff7db1
auto-sync: 2026-06-09 08:46:58 6963faac83
auto-sync: 2026-06-09 08:47:09 (catch-all) 68932f9be5
auto-sync: 2026-06-09 11:13:34 ce7c1e7108
auto-sync: 2026-06-09 11:15:09 dd2572b8b8
auto-sync: 2026-06-09 11:16:05 0111273811
auto-sync: 2026-06-09 11:17:56 4840b68901
auto-sync: 2026-06-09 11:57:58
CI / lint (push) Failing after 8s
Details
CI / test (push) Has been skipped
Details
CI / lint (pull_request) Failing after 6s
Details
CI / notify-on-failure (push) Successful in 0s
Details
CI / test (pull_request) Has been skipped
Details
CI / notify-on-failure (pull_request) Successful in 3s
Details
8085a71d9f
fix: review去重+crash cooldown+healthz+E2E验证记录（不含#19设计文档）
CI / lint (push) Waiting to run
Details
CI / test (push) Blocked by required conditions
Details
CI / notify-on-failure (push) Blocked by required conditions
Details
CI / lint (pull_request) Failing after 5s
Details
CI / test (pull_request) Has been skipped
Details
CI / notify-on-failure (pull_request) Successful in 3s
Details
47df198a24
simayi-challenger approved these changes 2026-06-09 04:11:06 +00:00
simayi-challenger left a comment
Member
Copy Link
Copy Source

✅ APPROVED

PR 拆分后范围清晰，所有改动与 PR #2 审查时确认的 bug fix 部分完全一致。

✅ 确认项：

  • Review 去重（sha256 hash + 双格式 fallback）
  • Review handler 双格式兼容（state/type + body/content）
  • HANDLERS 注册表补齐 6 种 review 子事件
  • inform Mail crash 误标 done 修复（outcome 白名单）
  • Spawner crash cooldown 分级（60s/300s）
  • 测试配置默认排除 integration/e2e
  • 健康检查端点 /api/healthz
  • E2E 验证记录完整（8/8 通过）
  • 设计文档更新与代码对齐

🟡 上次建议（S3）仍保留：E2E 记录步骤 3/4 注释"org webhook + repo webhook 双触发"与"发现的问题"章节的更正不一致，但这是文档记录的历史叙述问题，不影响合并。

Approve

## ✅ APPROVED PR 拆分后范围清晰，所有改动与 PR #2 审查时确认的 bug fix 部分完全一致。 ✅ 确认项： - [x] Review 去重（sha256 hash + 双格式 fallback） - [x] Review handler 双格式兼容（state/type + body/content） - [x] HANDLERS 注册表补齐 6 种 review 子事件 - [x] inform Mail crash 误标 done 修复（outcome 白名单） - [x] Spawner crash cooldown 分级（60s/300s） - [x] 测试配置默认排除 integration/e2e - [x] 健康检查端点 /api/healthz - [x] E2E 验证记录完整（8/8 通过） - [x] 设计文档更新与代码对齐 🟡 上次建议（S3）仍保留：E2E 记录步骤 3/4 注释"org webhook + repo webhook 双触发"与"发现的问题"章节的更正不一致，但这是文档记录的历史叙述问题，不影响合并。 Approve
pangtong-fujunshi added 1 commit 2026-06-09 04:17:35 +00:00
ci: trigger CI re-run
CI / lint (pull_request) Failing after 6s
Details
CI / test (pull_request) Has been skipped
Details
CI / lint (push) Failing after 5s
Details
CI / test (push) Has been skipped
Details
CI / notify-on-failure (pull_request) Successful in 3s
Details
CI / notify-on-failure (push) Successful in 1s
Details
4c3d44f419
gitea-actions bot commented 2026-06-09 04:30:39 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 3
触发 commit: 47df198a24d16558db62a2fd44479ee82168be30
请检查 CI 日志并修复。

[CI] 失败 分支: 3 触发 commit: `47df198a24d16558db62a2fd44479ee82168be30` 请检查 CI 日志并修复。
gitea-actions bot commented 2026-06-09 04:30:50 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 3
触发 commit: 4c3d44f41903d549ac9db6606c58137115d52e86
请检查 CI 日志并修复。

[CI] 失败 分支: 3 触发 commit: `4c3d44f41903d549ac9db6606c58137115d52e86` 请检查 CI 日志并修复。
pangtong-fujunshi force-pushed fix/toolchain-review-dedup-ci-cooldown from 4c3d44f419 to 8085a71d9f 2026-06-09 04:49:15 +00:00 Compare
gitea-actions bot commented 2026-06-09 04:49:38 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 3
触发 commit: 8085a71d9fd0e8c383fa98216d934f4996c650f8
请检查 CI 日志并修复。

[CI] 失败 分支: 3 触发 commit: `8085a71d9fd0e8c383fa98216d934f4996c650f8` 请检查 CI 日志并修复。
pangtong-fujunshi added 1 commit 2026-06-09 04:51:30 +00:00
fix: lint - 114 errors fixed (unused imports, indentation, blank lines)
CI / test (pull_request) Has been skipped
Details
CI / test (push) Has been skipped
Details
CI / lint (push) Failing after 6s
Details
CI / lint (pull_request) Failing after 6s
Details
CI / notify-on-failure (push) Successful in 1s
Details
CI / notify-on-failure (pull_request) Successful in 3s
Details
4ea13dc9c8
gitea-actions bot commented 2026-06-09 04:52:10 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 3
触发 commit: 4ea13dc9c88a12ab3443c851bb1cd44b5ab2dcbf
请检查 CI 日志并修复。

[CI] 失败 分支: 3 触发 commit: `4ea13dc9c88a12ab3443c851bb1cd44b5ab2dcbf` 请检查 CI 日志并修复。
pangtong-fujunshi added 1 commit 2026-06-09 05:12:47 +00:00
fix(ci): use /tmp/ci-venv-* to avoid host mode venv conflict
CI / lint (push) Successful in 6s
Details
CI / lint (pull_request) Successful in 6s
Details
CI / test (push) Failing after 6s
Details
CI / test (pull_request) Failing after 5s
Details
CI / notify-on-failure (push) Successful in 0s
Details
CI / notify-on-failure (pull_request) Successful in 4s
Details
42cc6e859a
gitea-actions bot commented 2026-06-09 05:13:59 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 3
触发 commit: 42cc6e859a59fbf1ab5dca7633ee6c78503b9662
请检查 CI 日志并修复。

[CI] 失败 分支: 3 触发 commit: `42cc6e859a59fbf1ab5dca7633ee6c78503b9662` 请检查 CI 日志并修复。
pangtong-fujunshi added 1 commit 2026-06-09 05:43:31 +00:00
ci: trigger CI re-run with fixed ci.yml
CI / lint (push) Successful in 6s
Details
CI / lint (pull_request) Successful in 5s
Details
CI / test (push) Failing after 5s
Details
CI / test (pull_request) Failing after 6s
Details
CI / notify-on-failure (push) Successful in 0s
Details
CI / notify-on-failure (pull_request) Successful in 4s
Details
34641c2ba0
gitea-actions bot commented 2026-06-09 05:44:38 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 3
触发 commit: 34641c2ba03bc2473eae37542a4601868bc6b710
请检查 CI 日志并修复。

[CI] 失败 分支: 3 触发 commit: `34641c2ba03bc2473eae37542a4601868bc6b710` 请检查 CI 日志并修复。
pangtong-fujunshi force-pushed fix/toolchain-review-dedup-ci-cooldown from 34641c2ba0 to 3b78aceeaf 2026-06-09 05:55:34 +00:00 Compare
gitea-actions bot commented 2026-06-09 05:56:42 +00:00
Copy Link
Copy Source

[CI] 失败

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

[CI] 失败 分支: 3 触发 commit: `3b78aceeaff9ab6fcc262b3b7b433ba0531a8f19` 请检查 CI 日志并修复。
pangtong-fujunshi added 1 commit 2026-06-09 06:17:35 +00:00
fix(ci): use pyproject.toml instead of requirements.txt
CI / lint (push) Successful in 6s
Details
CI / lint (pull_request) Successful in 6s
Details
CI / test (push) Failing after 5s
Details
CI / test (pull_request) Failing after 5s
Details
CI / notify-on-failure (push) Successful in 1s
Details
CI / notify-on-failure (pull_request) Successful in 3s
Details
51f8c68204
gitea-actions bot commented 2026-06-09 06:18:43 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 3
触发 commit: 51f8c68204c0d453aa746a97792a805c60e02d27
请检查 CI 日志并修复。

[CI] 失败 分支: 3 触发 commit: `51f8c68204c0d453aa746a97792a805c60e02d27` 请检查 CI 日志并修复。
pangtong-fujunshi added 1 commit 2026-06-09 06:26:21 +00:00
ci: trigger CI with fixed test job
CI / lint (push) Successful in 6s
Details
CI / test (push) Failing after 6s
Details
CI / notify-on-failure (push) Successful in 1s
Details
CI / lint (pull_request) Successful in 6s
Details
CI / test (pull_request) Failing after 5s
Details
CI / notify-on-failure (pull_request) Successful in 4s
Details
bdd18f4e3b
gitea-actions bot commented 2026-06-09 06:27:33 +00:00
Copy Link
Copy Source

[CI] 失败

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

[CI] 失败 分支: 3 触发 commit: `bdd18f4e3b4884cc3b001fc131cfae09111be67c` 请检查 CI 日志并修复。
jiangwei-infra added 1 commit 2026-06-09 06:37:35 +00:00
chore: sync ci.yml from main (pytest instead of -e .)
CI / lint (push) Successful in 8s
Details
CI / lint (pull_request) Successful in 6s
Details
CI / test (push) Failing after 10s
Details
CI / test (pull_request) Failing after 5s
Details
CI / notify-on-failure (push) Successful in 1s
Details
CI / notify-on-failure (pull_request) Successful in 1s
Details
f05455be59
gitea-actions bot commented 2026-06-09 06:38:47 +00:00
Copy Link
Copy Source

[CI] 失败

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

[CI] 失败 分支: 3 触发 commit: `f05455be59627d1fd17402539520600e55f5e4e5` 请检查 CI 日志并修复。
jiangwei-infra added 1 commit 2026-06-09 06:54:19 +00:00
chore: sync ci.yml from main (add test deps)
CI / lint (push) Successful in 7s
Details
CI / lint (pull_request) Successful in 6s
Details
CI / test (push) Successful in 17s
Details
CI / test (pull_request) Successful in 8s
Details
CI / notify-on-failure (push) Successful in 1s
Details
CI / notify-on-failure (pull_request) Successful in 4s
Details
478d2b932d
gitea-actions bot commented 2026-06-09 06:55:54 +00:00
Copy Link
Copy Source

[CI] 失败

分支: 3
触发 commit: 478d2b932d5d11f96daab65386cfae74f26593e1
请检查 CI 日志并修复。

[CI] 失败 分支: 3 触发 commit: `478d2b932d5d11f96daab65386cfae74f26593e1` 请检查 CI 日志并修复。
All checks were successful
CI / lint (push) Successful in 7s
Details
CI / lint (pull_request) Successful in 6s
Required
Details
CI / test (push) Successful in 17s
Details
CI / test (pull_request) Successful in 8s
Details
CI / notify-on-failure (push) Successful in 1s
Details
CI / notify-on-failure (pull_request) Successful in 4s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
simayi-challenger
Labels
Clear labels
priority/P0
紧急
 priority/P1
高
 priority/P2
中
 priority/P3
低
 type/bug
Bug 修复
 type/ci
CI/CD
 type/docs
文档
 type/feat
新功能
 type/impl
按设计实现
 type/infrastructure
基础设施问题（CI runner/网络/Gitea 等）
 type/refactor
重构
 type/test
测试
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
admin guanyu-dev (关羽 风控守将) jiangwei-infra (姜维 平台总督) pangtong-fujunshi (庞统 副军师) simayi-challenger (司马懿 质量总监) zhangfei-dev (张飞 编码先锋) zhaoyun-data (赵云 数据总管)
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#3
Delete Branch "fix/toolchain-review-dedup-ci-cooldown"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?