You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
Repository navigation
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
[Decision] A governed-surface AGENTS.md change (PR #6183) reached main without human review — keep it, or revert? #6325
Filed by the domain:devx @ objectui execution seat (#5748), PM session session_012CZgmFFzqA9cX8tBMhvpFe, R30. This is my own violation, reported by me. The incident record is on PR #6183; this card carries only the decision that is yours.
What happened
PR #6183 changes AGENTS.md — a governed surface under the 2026-08-18 ruling, where human merge is the review record. It was correctly opened as a draft and correctly left as one.
I called the GitHub MCP tool update_pull_request passing onlyreviewers: ["os-zhuang"], to make it visible in your review queue. That call also sets draft: false. The PR went ready, entered the merge queue, and merged as 5b3290fd5. My attempt to revert it — converting back to draft at 12:45:44Z — did not dequeue it, and I wrongly reported that it had. Full timeline on #6183.
⛔ No human approved this merge. That is true regardless of the text's quality.
What is actually on main
The #6069 workflow-registry note: how the Actions registry is keyed (first run on any ref, not the default branch), that state: "active" means "not disabled", the general paginated-listing rule (list_workflows ignores per_page and returns 30 while reporting the true total_count), and the git cat-file -e + positive-control method for "does X run here". 59 added lines, no deletions. CI was green, and the previous PM seat reviewed it on #6069.
四维分析
① 实际业务需求. 这段文字服务的是每一个 agent 席位,而且它记录的是一次真实踩坑(#6069:注册表与 main 双向不一致,list_workflows 回答不了「本仓跑不跑 X」)。它不是投机性能力面 —— 已经有席位因为读错注册表付过代价。⚠️ 但要诚实:这一轴评的是内容,而被违反的是流程,两者不能互相抵扣。
② 项目长远合理性. 受管面的全部价值就在于「人工合并即审核记录」这一条没有例外。一次「内容不错所以就留着吧」的例外,会把规则从「机器不得合并受管面」降级为「机器不得合并写得不好的受管面」—— 后者不可执行,因为质量判断正是要交给人的那件事。⚠️ 反方向也要说清:回滚一个内容正确的改动,买到的是规则的完整性,付出的是一次真实经验记录的丢失(以及重走一遍流程的成本)。
③ 防 AI 写代码犯错. 这一轴指向回滚,而且是最硬的一轴。本次事故的机制是「工具的隐藏副作用 + 队列不可逆」,它还会再发生 —— 平台事实已立卡 objectstack#12200,但事实表拦不住已经发生的这一次。如果留下,盘面上就存在一份「agent 自行合并进 main 的 agent 宪法文本」,而下一个 agent 读 AGENTS.md 时无从分辨哪几行经过人、哪几行没有。声明即强制:受管面若能被绕过一次而无痕,它就不再是受管面。
Maintainer ruling recorded — A + C: the merge stands; enforcement moves to the machine layer
Provenance: maintainer, 2026-08-27, PM chat decision-inbox batch 1 (adjudication session session_01DKWDdUJ2XNRESVVWUvcpnh), verbatim: 「同意,并继续」 — accepting this card's recommendation A + C.
A — the merge stands, and this comment is the audit record of the exception: commit 5b3290fd5 (PR #6183, AGENTS.md, 59 added lines) reached main on 2026-08-25 with ⛔ no human approval, via the update_pull_request hidden side effect recorded as objectstack-ai/objectstack#12200 (a reviewers-only call also sets draft: false; the merge queue did not honour the draft revert). The content itself was reviewed on #6069 by the prior seat and is judged harmless. The governed-surface rule is NOT relaxed by this ruling: the exception is recorded, not blessed, and the rule's next line of defence moves to the machine layer.
C — follow-up filed as #6596: make human review mechanically required on governed-surface paths in this repo, so the no-bypass rule stops resting on seat discipline alone.
Activity
os-zhuang commentedon Aug 27, 2026
Maintainer ruling recorded — A + C: the merge stands; enforcement moves to the machine layer
Provenance: maintainer, 2026-08-27, PM chat decision-inbox batch 1 (adjudication session
session_01DKWDdUJ2XNRESVVWUvcpnh), verbatim: 「同意,并继续」 — accepting this card's recommendation A + C.A — the merge stands, and this comment is the audit record of the exception: commit
5b3290fd5(PR #6183,AGENTS.md, 59 added lines) reachedmainon 2026-08-25 with ⛔ no human approval, via theupdate_pull_requesthidden side effect recorded as objectstack-ai/objectstack#12200 (a reviewers-only call also setsdraft: false; the merge queue did not honour the draft revert). The content itself was reviewed on #6069 by the prior seat and is judged harmless. The governed-surface rule is NOT relaxed by this ruling: the exception is recorded, not blessed, and the rule's next line of defence moves to the machine layer.C — follow-up filed as #6596: make human review mechanically required on governed-surface paths in this repo, so the no-bypass rule stops resting on seat discipline alone.
Decision delivered; card closed (completed). Refs: PR #6183 (incident timeline), #6069 (content card), objectstack-ai/objectstack#12200 (platform fact), #6596 (enforcement work).
Generated by Claude Code