Skip to content
This repository was archived by the owner on Oct 17, 2026. It is now read-only.

fix(review): 在 PR 阶段审查并复用合并前覆盖 - #36

Merged
TimWongUp merged 3 commits into
mainfrom
fix/pr-review-lifecycle
Sep 5, 2026
Merged

TimWongUp merged 3 commits into
mainfrom
fix/pr-review-lifecycle

Conversation

@TimWongUp

@TimWongUp TimWongUp commented Sep 5, 2026 •

Copy link
Copy Markdown
Owner

创建或更新 PR、受托审查现有 PR 时即启动风险 Review,与相关 CI 并行;合并前复用仍覆盖最新候选的审查证据,仅补充受影响的检查。普通本地交付和仅分支交接保持正常验证,审查不授予修改贡献者分支或合并的权限。

同步 Skill、全局模板、Worktree/manager-only 契约、双语文档和验证器回归检查,并将批次成功条件统一引用 Integration Barrier 的条件审查规则。

验证:源码验证通过;unittest 71 项(2 项 Windows 专属在 macOS 跳过);Ruff lint/format、Pyright、diff 检查及 Skills 部署审计通过。最终修订 macOS/Windows × Python 3.9/3.13 CI 全部通过(run 33943885969)。

R2 Review:跨入口公开运行契约,correctness-reviewer 核对审查触发、证据复用及授权边界。发现的批次完成条件遗漏已修复并由原 Reviewer 定向复核关闭,无剩余阻塞项。最终 head 6e8df9442af276cdfb47b689d2b20b42e0bee109,target/base 509b1a653514797e2ac7a9cd686d6bafb9758a11。文案行为仍依赖宿主遵循指令,不构成操作系统权限隔离。

@TimWongUp
TimWongUp merged commit 5d15615 into main Sep 5, 2026
7 checks passed
@TimWongUp
TimWongUp deleted the fix/pr-review-lifecycle branch September 5, 2026 04:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant