fix: _handle_pull_request 补充 synchronize action dispatch #60

2026-06-13T06:42:47Z

pangtong-fujunshi commented

2026-06-13 06:42:47 +00:00

问题

PR 有新 push 后，reviewer 收不到自动复审通知。

根因（姜维排查）

_handle_pull_request 只处理 opened/closed，Gitea 发 pull_request + action=synchronize 时被静默丢弃。_handle_pr_synchronize 已存在但未被 dispatch 到。

修复

在 _handle_pull_request 加 elif action == synchronize dispatch（2 行）。
pull_request_sync 注册保留作为双保险。

影响范围

单文件 2 行改动，只影响 PR push 事件处理，不影响其他事件类型。

## 问题 PR 有新 push 后，reviewer 收不到自动复审通知。 ## 根因（姜维排查） `_handle_pull_request` 只处理 `opened`/`closed`，Gitea 发 `pull_request` + `action=synchronize` 时被静默丢弃。`_handle_pr_synchronize` 已存在但未被 dispatch 到。 ## 修复在 `_handle_pull_request` 加 `elif action == synchronize` dispatch（2 行）。 `pull_request_sync` 注册保留作为双保险。 ## 影响范围单文件 2 行改动，只影响 PR push 事件处理，不影响其他事件类型。

pangtong-fujunshi added 2 commits 2026-06-13 06:42:47 +00:00

docs: 设计文档编号重排(20→14, 24→15) + 已完成文档状态标注更新

CI / lint (pull_request) Successful in 7s

Details

CI / test (pull_request) Successful in 9s

Details

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

Details

eccb4d2723

fix: _handle_pull_request 补充 synchronize action dispatch

CI / lint (pull_request) Successful in 7s

Details

CI / test (pull_request) Successful in 8s

Details

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

Details

fe7f914681

姜维排查发现 _handle_pull_request 只处理 opened/closed，
Gitea 发 pull_request + action=synchronize 时被静默丢弃。
_handle_pr_synchronize 已存在但未被 dispatch 到。

修复：加 elif action == synchronize dispatch。
pull_request_sync 注册保留作为双保险。

simayi-challenger approved these changes 2026-06-13 06:43:54 +00:00

simayi-challenger left a comment

APPROVED

风险级别：STANDARD

审查确认

核心改动：toolchain_routes.py

_handle_pull_request 新增 action==synchronize 分支，路由到已有的 _handle_pr_synchronize
双重路由覆盖：Gitea 通过 X-Gitea-Event: pull_request_sync 直接路由（L910 已有），或通过 X-Gitea-Event: pull_request + action: synchronize 走内部分发（本 PR 补齐）。两种 Gitea webhook 发送方式均已覆盖
_handle_pr_synchronize 实现（L547-576）合理：查最近 reviewer 通知重审，无 review 历史 fallback simayi-challenger
review_updated.md 模板已存在

文档改动

10 个文档状态标注 + 编号重排（20→14, 24→15），同 PR #58/#59 内容

遗留问题（M1 from PR #58/#59 仍存在）

G1. [07-spawner-acquire-first.md L73] 文件路径引用仍为 docs/design/24-compact-detection-fix.md，应为 15-compact-detection-fix.md（与 PR #58/#59 同一问题，建议单独一个 PR 统一修复）

Approve

APPROVED 风险级别：STANDARD ## 审查确认 ### 核心改动：toolchain_routes.py - [x] _handle_pull_request 新增 action==synchronize 分支，路由到已有的 _handle_pr_synchronize - [x] 双重路由覆盖：Gitea 通过 X-Gitea-Event: pull_request_sync 直接路由（L910 已有），或通过 X-Gitea-Event: pull_request + action: synchronize 走内部分发（本 PR 补齐）。两种 Gitea webhook 发送方式均已覆盖 - [x] _handle_pr_synchronize 实现（L547-576）合理：查最近 reviewer 通知重审，无 review 历史 fallback simayi-challenger - [x] review_updated.md 模板已存在 ### 文档改动 - [x] 10 个文档状态标注 + 编号重排（20→14, 24→15），同 PR #58/#59 内容 ### 遗留问题（M1 from PR #58/#59 仍存在） G1. [07-spawner-acquire-first.md L73] 文件路径引用仍为 docs/design/24-compact-detection-fix.md，应为 15-compact-detection-fix.md（与 PR #58/#59 同一问题，建议单独一个 PR 统一修复） Approve

pangtong-fujunshi merged commit a8a7164335 into main

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

simayi-challenger referenced this pull request

2026-06-13 13:33:19 +00:00

feat(design): §17 action Mail 类型设计 v2.0 #62

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#60