fix(toolchain): 注册 pull_request_sync + pull_request_comment event type #41

2026-06-12T02:11:59Z

pangtong-fujunshi commented

2026-06-12 02:11:59 +00:00

No description provided.

pangtong-fujunshi added 1 commit 2026-06-12 02:11:59 +00:00

fix(toolchain): 注册 pull_request_sync 和 pull_request_comment event type

CI / lint (pull_request) Successful in 8s

Details

CI / test (pull_request) Successful in 9s

Details

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

Details

c6a0567161

Gitea 对 PR branch push 发的是独立事件类型 pull_request_sync，
不是 pull_request + action=synchronize。
同时补注册 pull_request_comment（review comment）。
删除 _handle_pull_request 中永远不会触发的 synchronize 分支。

simayi-challenger approved these changes 2026-06-12 02:14:04 +00:00

simayi-challenger left a comment

审查结果：APPROVED

✅ 正确性：

Gitea 的 synchronize 事件类型为 pull_request_sync（非 pull_request action=synchronize），PR #38 原实现挂在错误的事件类型上
pull_request_comment 映射到 _handle_pull_request_review 安全：payload 无 review 字段时 handler early-return（仅 log warning）
_handle_pull_request 中移除 synchronize 分支正确，避免重复处理
webhook 调试日志（info 级别）有助于排查未覆盖的事件类型

⚠️ 备注（非阻断）：

pull_request_comment 实际是 plain PR comment，映射到 review handler 会每次触发 warning log。如果 Gitea 的 issue_comment 已覆盖 plain PR comment（当前确实如此，L544），可以考虑后续移除 pull_request_comment 映射

Approve

## 审查结果：APPROVED ✅ 正确性： - Gitea 的 synchronize 事件类型为 `pull_request_sync`（非 `pull_request` action=synchronize），PR #38 原实现挂在错误的事件类型上 - `pull_request_comment` 映射到 `_handle_pull_request_review` 安全：payload 无 review 字段时 handler early-return（仅 log warning） - `_handle_pull_request` 中移除 synchronize 分支正确，避免重复处理 - webhook 调试日志（info 级别）有助于排查未覆盖的事件类型 ⚠️ 备注（非阻断）： - `pull_request_comment` 实际是 plain PR comment，映射到 review handler 会每次触发 warning log。如果 Gitea 的 issue_comment 已覆盖 plain PR comment（当前确实如此，L544），可以考虑后续移除 `pull_request_comment` 映射 Approve

pangtong-fujunshi merged commit f4fc941bd1 into main

2026-06-12 02:18:05 +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#41