docs(AGENTS): §7 names the third class of PR that never lands on green alone — over 5,000 changed lines

[os-elon-musk]

Fixes #19037
Clause-②: no

What changed

AGENTS.md Multi-agent discipline §7 said 「Two classes of PR never enter this path on green alone」 — (a) a governed-surface diff, (b) the Version Packages / release-performing PR. The maintainer's ruling adds a third, and the rules layer contradicted the landing rule until it said so. The paragraph now names three classes:

(c) a PR whose changed lines exceed 5,000 (additions + deletions, generated files included) — it lands only by a human merge, which is its review record.

and the closing instruction reads the size in the same breath as the file list and the author: 「Read the PR's file list (get_files), its author and its size before you arm anything.」

Ruling (verbatim, untranslated), recorded on #16045 (comment 5729462393) and carried by #19012 / PR #19033:

「还有应该完善skills，修改代码量超过某个行数（比如5000）就应该人工审核。」

Read as the landing rule already spells it: additions + deletions on the PR, strictly greater than 5,000, generated files INCLUDED, no exemption; the terminal is the human's merge. The sentence states the ruling, not a line number of PR #19033, so it reads true whether or not that PR has landed. No new rule beyond the ruling: no exemption, no lower number, no review that is not the human's merge.

Why the sentence carries no card number or date. AGENTS.md's header: a rule 「carries no incident narrative, no ruling date or quotation, and no issue-number citation (pnpm check:pm-skill-id-lint) — a rule's provenance lives in the PR that landed it」; that gate's pattern is /#[0-9]{3,}/ over AGENTS.md, so #16045 / #19012 in the sentence would go red. The dispatch's mechanism assumption 1 (name the ruling by card) is falsified by the gate and by the file's own header — the provenance is this PR body, and the sentence is self-contained.

Line budget — net 0 at the ceiling (1099 / 1099)

The paragraph was wrapped at ~88 bytes with a 27-byte last line; re-flowed at the file's own 120-byte per-line budget (the Prime Directives block runs to 120 bytes; §8 beside it to 116) it absorbs the clause with no new line. Every bold and code span stays whole on its line, as before.

line	before (bytes)	after (bytes)
495	90	113
496	88	117
497	88	109
498	81	109
499	86	111
500	27	75
paragraph	466	640
file	1099 lines	1099 lines

Before:

   ⛔ **Two classes of PR never enter this path on green alone:** (a) a diff touching any
   **governed surface** (**Prime Directive #14**, which names them and holds the current
   list — **this file and `CLAUDE.md` are on it**, so re-read it rather than recalling
   it); (b) the **Version Packages** PR, or any PR whose merge performs a release
   (**Prime Directive #15**). Read the PR's file list (`get_files`) **and its author**
   before you arm anything.

After:

   ⛔ **Three classes of PR never enter this path on green alone:** (a) a diff touching any **governed surface**
   (**Prime Directive #14**, which names them and holds the current list — **this file and `CLAUDE.md` are on it**,
   so re-read it rather than recalling it); (b) the **Version Packages** PR, or any PR whose merge performs a
   release (**Prime Directive #15**); (c) a PR whose **changed lines exceed 5,000** (`additions + deletions`,
   generated files included) — it lands only by a human merge, which is its review record. Read the PR's file
   list (`get_files`), **its author and its size** before you arm anything.

Reader test

A seat about to arm a 6,000-line PR reads §7 and stops: 「Three classes … (c) a PR whose changed lines exceed 5,000 … lands only by a human merge」. Before this PR the same seat read two classes, found a 6,000-line diff in neither, and armed.

Other mentions in AGENTS.md (mechanism assumption 3)

grep -n -i -E 'green alone|two classes|Version Packages' on origin/main dbd474431:

• :291, :299, :307, :309 — Prime Directive Add missing Field.phone() helper and factory methods for Action/Dashboard/Report #15's own text (no seat merges the Version Packages PR; that merge is the release trigger). Left: it is class (b)'s anchor, not an enumeration of the never-on-green-alone set.
• :495, :498 — the §7 sentence. Changed (this PR).
• No other line enumerates the set. Two neighbours name class (a) alone by design and were left: the Skills section's 「Both roots are governed surfaces — human-merge only, or Prime Directive feat: Comprehensive CRM example demonstrating all ObjectStack protocol features #14's pinned-approval path」 (about governed surfaces only), and Post-Task Checklist step 2's 「⛔ Except a diff touching a governed surface」, which defers to §7 by reference for the arming rule — see Acceptance notes.

Gates (local, at 845470659)

node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack derived 14 families from the changeset (1 path vs merge base dbd474431); all 14 ran, exit codes captured redirect-then-$?, plus the two the dispatch named:

node scripts/check-closing-keyword-parity.mjs :: exit 0
node scripts/check-closing-keyword-parity.mjs --self-test :: exit 0
node scripts/check-comment-mask-corpus.mjs :: exit 0
pnpm check:agent-test-spelling :: exit 0
pnpm check:docs-audit-scope :: exit 0
pnpm check:driver-memory-census :: exit 0
pnpm check:nul-bytes :: exit 0
pnpm check:pm-governed-merges :: exit 0
pnpm check:pm-governed-prose :: exit 0
pnpm check:pm-skill-id-lint :: exit 0
pnpm check:pm-skill-ratchet :: exit 0
pnpm check:refd-timer-probe :: exit 0
pnpm check:required-contexts :: exit 0
pnpm check:watch-hint-literal :: exit 0
pnpm check:skill-frame-sync :: exit 0
pnpm check:doc-authoring :: exit 0

Reconciliation: ✓ dispatch-gates --ran: 14 derived famil(ies) accounted for — 14 run, 0 NOT-MEASURED. Ratchet: ✓ check-skill-line-ratchet: AGENTS.md is 1099 lines (ceiling 1099; headroom 0). and widest table row is 768 bytes (pin 768; headroom 0). Id-lint: 0 citations in AGENTS.md. Control-byte scan of the file: no match.

Not measured locally: check:doc-formula-expressions (a CEL gate over fenced formula examples in the docs/skills corpus; this diff adds no fenced block and the family is outside the derivation; CI runs it). check:pm-dispatch-gates ran detached — its verdict is in the report comment on #19037.

Changeset

skip-changeset: AGENTS.md is a repo-root instruction file; the root package is private and no package files[] ships it — nothing published moves.

Acceptance notes

• noted, not filed: Post-Task Checklist step 2 names only the governed class in its 「⛔ Except」 clause; the size class reaches a dev's own PR too. It defers to §7 by reference, and the file header's one-statement-per-rule principle argues against restating; a fold there is one line of re-flow if the seat wants it. 承接者: skills seat.
• noted, not filed: the size class has no Prime Directive of its own (it is a ruling, not a directive); a numbered home would be a new directive beside feat: Comprehensive CRM example demonstrating all ObjectStack protocol features #14 / Add missing Field.phone() helper and factory methods for Action/Dashboard/Report #15 — a separate decision, not taken here.

维护者速读(草稿)

• 改了什么:AGENTS.md 多 agent 纪律 §7 那句「两类 PR 绿了也不能自动进合并队列」改成三类:新增 (c) 改动行数(additions + deletions,含生成文件)超过 5,000 行的 PR,只能由人工合并。文件行数不变(1099 / 1099),只是把那一段按文件自身的行宽重排。
• 为什么改:您 09-18 的裁决「修改代码量超过某个行数(比如5000)就应该人工审核」已经落进了 PM 的落地规则与门禁脚本,但规则层 AGENTS.md 还写着两类,座席读到的规则与落地规则互相矛盾。
• 风险与代价(含回滚):纯文本改动,不动任何脚本或门禁;本地 16 项门禁全绿。回滚 = revert 这一个 commit。
• 席位意见:(留空,由席位定稿)
• 你要做的:确认这句话与您的裁决一致(尤其「含生成文件、无豁免」与「人工合并即审核记录」两点),然后由您合并。

---

Generated by Claude Code

[os-elon-musk]

Contract review

Served-tier: CONTRACT_REVIEW_TIER
Head-sha: 8454706591

① Derived judgments

• Surface: AGENTS.md only (+6 / −6, one commit, merge base dbd474431), one hunk: the §7 paragraph old :495–:500 re-flowed in place — 6 lines → 6 lines, 1099 / 1099 (ratchet ✓ on the head, seat-run; row pin 768 ✓), check-skill-id-lint ✓ 27 files (0 citations in AGENTS.md), every changed line ≤ 117 B against the file's 120-byte budget (113 / 117 / 109 / 109 / 111 / 75), check-governed-merges.mjs --pr 19042 → GOVERNED. No other file; no open PR touches AGENTS.md.
• The rule. 「⛔ Three classes of PR never enter this path on green alone: (a) … governed surface (Prime Directive feat: Comprehensive CRM example demonstrating all ObjectStack protocol features #14 …); (b) the Version Packages PR, or any PR whose merge performs a release (Prime Directive Add missing Field.phone() helper and factory methods for Action/Dashboard/Report #15); (c) a PR whose changed lines exceed 5,000 (additions + deletions, generated files included) — it lands only by a human merge, which is its review record. Read the PR's file list (get_files), its author and its size before you arm anything.」 — the maintainer's ruling in the section's own register: strictly greater (「exceed」), additions + deletions, generated files included, no exemption, the human merge as the review record — the same constant and reading PR feat(pm): a PR over 5,000 changed lines lands only by a human merge — size predicate in check-governed-merges --test, the same reading in dispatch-gates, one rule line in SKILL.md and landing-operations #19033 wrote into check-governed-merges.mjs (HUMAN_MERGE_LINE_THRESHOLD = 5000), SKILL.md :608 and landing-operations.md :26. Classes (a) and (b) keep their Prime Directive anchors; no directive was added (a separate decision, correctly not taken). The closing instruction gains the third reading (「its size」) beside the file list and the author — all three come from the same GET /pulls/{n}.
• Self-contained, by the file's own rule. The dispatch said 「cite the ruling ([finding] @objectstack/spec pins 5309 exports by name-and-kind but only 27 by signature — 99.5% of the ratified surface passes green through a shape change #16045 / [ruling] a PR whose changed line count exceeds 5,000 lands only by a human merge — rule line in SKILL.md + landing-operations.md, and a size predicate in the landing gate #19012)」; the dev measured that AGENTS.md's header forbids ruling dates, quotations and issue-number citations and that check:pm-skill-id-lint enforces it over this file (/#[0-9]{3,}/) — a citation would have gone red. The sentence carries the rule and no card; the provenance lives in the PR body. The seat's error, the dev's reading correct (os-dev.md: the file's binding rule wins over the dispatch word), recorded.
• The re-flow. The paragraph is re-wrapped from ~88 to ≤ 117 bytes per line, absorbing the clause at net 0; every bold / code span stays whole on its line. The seat's one reading: this paragraph is now wider than its neighbours (the Prime Directives block already runs to 120 at :468 and §8 to 116 at :535, so the width is within the file's own practice) — a style seam, not a defect; no line is owed.
• Other mentions, measured on the head (green alone / two classes / Version Packages): Prime Directive Add missing Field.phone() helper and factory methods for Action/Dashboard/Report #15's own text (:291 · :299 · :307 · :309) is class (b)'s anchor, not an enumeration — left, agreed; the Skills section's 「Both roots are governed surfaces — human-merge only …」 and Post-Task Checklist step 2's 「⛔ Except a diff touching a governed surface」 name class (a) alone by design and defer to §7 by reference — left, agreed; the checklist line is a one-line re-flow fold if ever wanted, carried by the seat to the next AGENTS.md touch ([finding] the os-dev dispatch contract and the harness attribution reminder contradict each other on the commit trailer — and landed history follows the reminder #18698's ⑤ once ruled), ⛔ not a card.
• Seat re-measure on a scratch worktree at this head: 1099 lines, ratchet ✓ (headroom 0), id-lint ✓, --pair 19042 exit 0, closingKeywordTargets = {19037 → Fixes}, the newest (only) Claim: (5730566948) names this branch; the body carries ## 维护者速读(草稿) and the session-URL footer once.

② Semver level

• skip-changeset is correct (AGENTS.md, nothing published moves; applied by label-write.mjs, read back twice). Clause-②: no on the claim and in the body; --pair 19042 at 2026-09-18T13:45Z: exit 0, no widening tell.

③ Boundary flags

• Dev flags: no open questions. Four declared deviations, all reporting: the citation rule (①, the seat's error); the model-free trailer pair (the pre-push hook's rule, per AGENTS.md); the session-URL footer form (AGENTS.md's, stored once); no classifier denial. Two noted: the checklist step-2 fold (①, carried) and the size class's lack of a Prime Directive of its own (a decision, not taken — agreed).
• Gates: 14 derived / 14 run / 0 NOT-MEASURED at 845470659 (every exit 0; --ran reconciliation a derived zero) + check:skill-frame-sync 0, check:doc-authoring 0, check:pm-dispatch-gates 1849 cases (detached, verdict line read, exit not captured — declared); check:doc-formula-expressions not owed (AGENTS.md outside its roots — verified by the dev at packages/lint/scripts/check-doc-formula-expressions.mjs :179); check-governed-queue-guard.mjs --pair exit 1 = PREREQUISITE (no GITHUB_EVENT_PATH locally) — NOT MEASURED here, CI's own job reads it. 0 MCP calls; 5 REST writes / pushes as budgeted.
• CI on this head at 2026-09-18T13:45Z: 12 success · 11 skipped · 2 in progress (latest run per check) — nothing red; the Check Changeset failure at 13:31Z was the pre-label run, superseded by the labeled run at 13:40Z (skipped, label honoured).
• Landing: governed rule layer (AGENTS.md) ⇒ draft at the four-piece terminal (ACCEPT on [finding] AGENTS.md §7 still names TWO classes of PR that never land on green alone — the maintainer's 5,000-changed-lines class (#19012) is the third, and the rules layer contradicts the landing rule until it says so #19037, needs-user-decision on this PR, the 维护者速读 below, review requested from os-zhuang and hotlong). On an authorized APPROVED: ruling C by the seat, channel permitting — or the approver's own hand. PR feat(pm): a PR over 5,000 changed lines lands only by a human merge — size predicate in check-governed-merges --test, the same reading in dispatch-gates, one rule line in SKILL.md and landing-operations #19033 (the rule's mechanism) and this PR can land in either order: the sentence states the ruling, not that PR's lines.

Implemented-by: claude/issue-19037-agents-third-class-human-merge
Reviewed-by: session_01BTeBejoPUvRHN8WdAJC6oF

VERDICT: PASS

---

Generated by Claude Code

[os-elon-musk]

维护者速读(终稿)· skills 席 · 2026-09-18T13:47Z

这个 PR 做什么:把您的 5000 行规则写进 AGENTS.md(规则层)。§7 原来说「两类 PR 不能凭绿灯自动进合并队列」—— (a) 受管面、(b) 发版 PR;现在是三类,第三类:改动超过 5,000 行(增+删,含生成物)的 PR 只能由人合并,人合就是它的复核记录;席位入队前除了看文件清单和作者,还要看 size。段落原地重排,行数不变(1099 / 1099)。句子里不带卡号 —— AGENTS.md 自己的规则禁止引卡号且有 lint 把守(我派发时写错了,dev 按文件规则改对了)。它和 PR #19033(这条规则的脚本与 SKILL.md 部分)谁先谁后都行:这里写的是规则本身,不引用那个 PR 的行。

验收:复核记录 5730909983(PASS);ACCEPT 5730916820;派生门禁 14 / 14 绿;行数棘轮 1099 / 1099、每行 ≤ 117 字节;CI 12 成功 · 11 跳过 · 2 在跑、0 红(2026-09-18T13:45Z);Fixes #19037、Clause-②: no。

请您做的一件事:Approve(受管文本),或直接 ready + squash 合并。没有要问的字。

---

Generated by Claude Code