Skip to content

完成 MR 通知与 Hook/Channel 模块重构 - #30

Merged
coolbeevip merged 1 commit into
mainfrom
2026-08-02-debt-hook-channel-module-separation
Aug 2, 2026
Merged

完成 MR 通知与 Hook/Channel 模块重构#30
coolbeevip merged 1 commit into
mainfrom
2026-08-02-debt-hook-channel-module-separation

Conversation

@coolbeevip

Copy link
Copy Markdown
Owner

变更目的

为 MR 合并、审批通知建立独立的 notifications、channels、hooks、delivery 模块边界,并完成 Issue、Merge Request、Note Webhook Handler 的目录归一化。移除所有旧兼容 facade,使职责模块成为唯一代码入口。

Task 与 Commit

  • T001 建立通知模型、Channel 契约与包骨架 — 1745ca2ee8009599f07f87919b88283d11cf9a00
  • T002 拆分审批与合并通知 Hook 模块 — fa558ae93a5fbc7d6016792db1188aed36f61a28
  • T003 拆分 SQLite 投递、幂等包装器与投递协调器 — 4d130f3a85108e56245b707610a7157978d04052
  • T004 建立模块边界守护与完整回归验证 — 9a89a09a25078336650965a606950433d5918af4
  • T005 统一 Issue、Merge Request、Note Webhook Hook 模块 — b1143db8cc68c01966d6fde927f1be1c2d59e293
  • T006 删除旧 Hook 与通知兼容 facade — 943ef78d77bcf174394837d75508a2f5175cf1ea

另包含已完成的 MR 合并通知功能基线提交:

  • d8653d0f36268e56188b1e8040f834469ec4267b
  • 16dacabd89553787cb7a6922f22ab20cf2fd6abd

验证

  • 75 passed, 1 deselected
  • ruff 检查通过
  • 导入 smoke、模块边界和旧入口扫描通过
  • git diff --check 通过

兼容性与影响

  • 不改变 Webhook 业务规则、通知文案、SQLite schema、幂等键或恢复语义。
  • 不保留旧 facade;仓库代码统一从职责模块导入。
  • 目标分支为 main

远端为无中间提交的最终树提交 a97d0d369dc263b52516fec69d21d792badeadf1,对应本地已验证的完整变更。

@coolbeevip
coolbeevip merged commit 0a55ccc into main Aug 2, 2026
3 checks passed
@coolbeevip
coolbeevip deleted the 2026-08-02-debt-hook-channel-module-separation branch August 2, 2026 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant