Skip to content

加入仓库合并权限约定与分支保护说明 - #6

Merged
Slacker-LLC merged 1 commit into
mainfrom
codex/repo-guardrails
Oct 13, 2026
Merged

Slacker-LLC merged 1 commit into
mainfrom
codex/repo-guardrails

Conversation

@Slacker-LLC

Copy link
Copy Markdown
Owner

目的

把「合并进 main 必须由仓库所有者完成」写成仓库规则与 GitHub 侧的保护,而不是依赖口头约定。

本 PR 内容

  • 新增 AGENTS.md:自动化在仓库内只能创建分支、提交、推送、开 PR、等 CI;不得合并 pull request、不得直接推送 main、不得关闭或放宽分支保护;并写明语言与验证要求(离线测试 + 对应实机回归,按事实记录失败)。
  • CONTRIBUTING.md 新增「分支保护与合并归属」:说明 main 需要 PR 与四个 offline 必需状态检查、禁止强推与删除、管理员同样受约束;并说明单账号下审批数暂为 0 的原因——GitHub 不允许作者批准自己的 PR,现在设成 1 会让所有合并死锁;加入第二个账号(或让自动化使用独立账号)后改成 1,这一条才会从约定升级为技术闸门。

已在 GitHub 侧生效的保护(本 PR 之前已通过 API 启用)

  • enforce_admins: true
  • 必需状态检查:offline (3.10)、offline (3.11)、offline (3.12)、offline (3.13)
  • 要求 PR,过期审批作废,必须解决全部会话
  • 禁止强推、禁止删除 main

说明

本 PR 由自动化创建,按新规则保持打开,等仓库所有者合并。

- 新增 AGENTS.md:面向用户的输出使用简体中文;自动化只能建分支、推送、开 PR、
  等 CI,不得合并 pull request、不得直接推送 main、不得关闭或放宽分支保护;
  并写明离线测试与实机回归的验证要求
- CONTRIBUTING.md 补充「分支保护与合并归属」一节:main 需要 PR 与四个 offline
  状态检查,禁止强推与删除,管理员同样受约束;说明单账号下审批数暂为 0 的原因,
  以及加入第二个账号后改成 1 才能让「只由所有者合并」成为技术闸门
- 对应的 GitHub 分支保护已通过 API 启用(enforce admins、必需状态检查、
  禁止强推/删除、必须解决会话)
@Slacker-LLC
Slacker-LLC merged commit 1e53e90 into main Oct 13, 2026
4 checks passed
RIPBillyMays pushed a commit to RIPBillyMays/solidworks-mcp that referenced this pull request Oct 13, 2026
…, active document in results

- Sketch draw tools create entities with AddToDB on (no inference snapping),
  read geometry back and return ok:false with requested/actual on mismatch.
  New draw_polyline tool. draw_polygon honours inscribed=false on 2017.
- Tool results carry active_document; new activate_document tool.
- Live probes of field findings (Slacker-LLC#1 reproduced; Slacker-LLC#3/Slacker-LLC#4/Slacker-LLC#6/Slacker-LLC#8 not), offline
  and live verification reports, plan update (Slacker-LLC#4/Slacker-LLC#6/Slacker-LLC#8 parked in Phase 7).

Co-Authored-By: Claude Opus 5.5 <[email protected]>
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