From 41db7bc56c35da8514197ef81eea5644d2acf4dd Mon Sep 17 00:00:00 2001 From: tradecatlabs <288998340+tradecatlabs@users.noreply.github.com> Date: Fri, 16 Oct 2026 22:15:33 +0800 Subject: [PATCH] docs: github - scope review exemption to repository owner (#143) --- AGENTS.md | 13 +++++++++---- README.md | 7 +++++-- 2 files changed, 14 insertions(+), 6 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 511d3b0e..31be7c0b 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -267,8 +267,12 @@ git push origin HEAD - `main` 是默认主线，开发更新和贡献 PR 统一以 `main` 为目标；远端长期保留 `main`、`minimal`。 - `minimal` 保留现有内容，不作为贡献集成分支；`main` 保留签名和必需 CI 保护。 -- 按用户明确要求，Ruleset 与经典分支保护均不要求审批：数量为 0，Code Owner、最近推送及额外审批均关闭。 - `CODEOWNERS` 仅提供自愿审查入口，不作为合并前置条件；不得擅自恢复必需审批。 +- 审批由独立 Ruleset `contributor-approval-owner-exemption`（ID `25244224`）管理：默认至少 1 个有效批准， + 仅账号 `tradecatlabs`（User ID `288998340`）拥有 `pull_request` 模式豁免；只对其自己发起的 PR 使用免审。 +- GitHub 原生豁免按合并操作账号判断，不按 PR 作者自动判断；合并他人 PR 时必须核对有效批准，不能借此豁免绕过。 +- 主 Ruleset `17185792` 不配置豁免；CI、签名、线性历史、禁止强推/删除等保护仍对所有人适用。 + 主 Ruleset 与经典分支保护的审批数保持 0，避免重复阻断账号豁免；不代表普通贡献者免审。 +- Code Owner、最近推送及额外审批要求保持关闭；`CODEOWNERS` 仅作为路径级审查入口，不强制指定审批人。 - 按用户授权收敛远端分支前，先迁移开放 PR 的目标、保存全部精确 SHA 与已验证恢复 bundle，并用 ref lease 防止误删并行更新。 - 远端清理不删除本地分支或 worktree；尚未整合的历史只备份，不自动合入主线。临时任务分支在交付后按同一规则清理。 - 不将 squash 后的非祖先 HEAD 自动判为未整合；须核对已合并 PR 的精确 HEAD 与主线中的 merge commit。 @@ -277,8 +281,9 @@ git push origin HEAD - 用户已明确要求：本仓修改完成并完成必要验证后，自动提交、推送并合并，不再停在“本地未推送”，不重复询问发布许可；用户后续暂停、仅本地或先看方案等明确要求优先。 - 只选择性提交本任务范围，不夹带并行或无关改动；不自动切分支、改写历史、强推或删除交付引用。 -- 受保护主线走精确 PR/HEAD 流程；核对必需 CI 和其他保留的保护，不额外要求审批或伪造批准、成功 checks、独立审核。 - 流程例外须按用户授权留痕；临时调整的保护按约定恢复，但不得恢复用户已明确移除的必需审批。 +- 受保护主线走精确 PR/HEAD 流程；合并前核对 PR 作者、操作账号、必需 CI 和其他保留的保护。 + `tradecatlabs` 合入自己发起的 PR 无须审批；他人发起的 PR 仍须有效批准，不伪造批准、成功 checks 或独立审核。 + 流程例外须按用户授权留痕；临时调整的保护按约定恢复，不擅自扩大账号豁免或叠加审批门槛。 凭据、权限和数据边界不因自动交付扩大。 ### Commit 规范 diff --git a/README.md b/README.md index df36d0d7..1d31c52b 100644 --- a/README.md +++ b/README.md @@ -290,8 +290,11 @@ pip install -r .github/tools/prompts-library/scripts/requirements.txt - 默认分支：`main`，开发更新和贡献 PR 统一以 `main` 为目标。 - 远端长期保留 `main`、`minimal`；`minimal` 保留现有内容，不作为贡献集成分支。主线保留签名与必需 CI 保护。 -- 不要求审批：Ruleset 与经典分支保护的审批数量均为 0，Code Owner、最近推送及额外审批均关闭。 -- 自愿审查入口：`.github/CODEOWNERS`（当前维护者：`@tukuaiai`、`@tradecatlabs`），不作为合并前置条件。 +- 审批规则单独管理：普通贡献者合入 PR 需至少 1 个有效批准；仅 `tradecatlabs` 可在合并 PR 时豁免审批。 +- 豁免只用于 `tradecatlabs` 自己发起的 PR；合并他人的 PR 仍须先获得批准。GitHub 原生豁免按合并账号判断， + 不会自动按 PR 作者区分，因此该账号也具备绕过他人 PR 审批的能力，不得将其当作他人免审的日常流程。 +- 审批豁免不覆盖 CI、签名或其他保护；Code Owner、最近推送及额外审批要求保持关闭，不叠加审批门槛。 +- 路径级审查入口：`.github/CODEOWNERS`（当前维护者：`@tukuaiai`、`@tradecatlabs`），不强制指定审批人。 - Markdown lint 配置：`.github/lint_config.json` - Markdown lint 版本：`Makefile` 中固定为 `markdownlint-cli@0.48.0` - 外部链接检查配置：`.lychee.toml`，统一管理外链检查的超时、重试、并发上限和排除项