Repository navigation
docs(agents): a force-push is governed by the landing repo's AGENTS.md (objectui#9666) - #10958
Conversation
Implements ruling C on objectui#9666 (comment 5724939229): neither force-push text changes; objectui's multi-agent section gains one sentence, placed at the end of the absolute force-push ban, saying the rule that governs a force-push is the AGENTS.md of the repository the branch lives in -- objectui and cloud forbid it absolutely, objectstack allows --force-with-lease under its five criteria -- and that a seat working across repositories reads the landing repo's rule before instructing a dev. The three-repo reading is marked as unre-derived per commandment #9. Claude-Session: https://claude.ai/code/session_01EBx9rvB7dufCz4at53x35U Co-authored-by: Claude <noreply@anthropic.com>
|
changeset-claim-re-read
|
Contract reviewServed-tier: Reviewed against: card objectui#9666 (body and all five comments; ruling C is comment 5724939229), PR #10958's body and file list, the net diff of ① Derived judgmentsAccept set / public surface: none changed — right. The diff is one file, The existing ban is left byte-identical — right. Byte comparison of The sentence against the ruling — substance exact, two in-voice annotations, judged right. Ruling C's sentence has four clauses and the appended sentence carries each one:
No clause of the ruling is missing, and no rule content beyond it is added: no hotcrm, no framework, no gate, no strength — the card said those were not asked and the ruling added none. What IS added beyond the ruling's words: (a) the provenance parenthetical 「(objectui#9666,维护者裁决 C,评论 5724939229)」, the same shape the neighbouring trailer bullet uses for its own ruling (「objectui#9441,维护者裁决 A,评论 5724938982」); (b) the epistemic clause 「这三仓读数取自裁决当日,本仓没有任何东西复核它(诫条 #9)」. Neither changes what the rule permits or forbids; both are what this file's commandments #9 and #11 require of any prose in it, and the ruling did not suspend those. It is still grammatically one sentence (one full stop). The dev declared (b) as a deviation; I read it as the file's own convention correctly applied, not a departure from the ruling. Factual claims in the sentence, checked on the cited trees:
Conflicts with other text in the same file — none found. The only other Placement — right, with the disclosure the approver needs already in the PR. The ruling says the "cross-repo section" gains the sentence; the head file's 28 headings include no cross-repo section (the only sibling-naming heading, 「服务纪律(本仓库与 Premise (H1) re-measured — right. Check-runs on ② Semver levelnone — right. The diff publishes nothing: ③ Boundary flagsDev report (card comment 5868493385):
Governed surface: Observation outside the diff, not a flag on it: the card's claim comment (5868312809) carries a Implemented-by: VERDICT: PASS Generated by Claude Code |
维护者速读(终稿)席位 改了什么:只改 objectui 根 为什么改:执行 9 月 18 日本卡的裁决 C( 风险与代价(含回滚):纯文字,不碰代码、构建与发布,无 changeset。句中对三个仓的转述是裁决当日的读数;兄弟仓日后改规则,这句转述会过时,句子本身已写明「本仓没有任何东西复核它,以落地仓原文为准」。回滚:revert 本 PR。 席位意见:建议批准。有两点请知悉:
你要做的:读一下新增的这一句,同意就在本 PR 上点 Approve( Generated by Claude Code |
Fixes #9666
Clause-②: no —
AGENTS.mdis an agent-facing convention, not a published contract, accept set or public surfaceWhat this does
Implements ruling C on objectui#9666 (maintainer, 2026-09-18, Ruling-ref comment
5724939229): neither force-push text changes, and objectui's rootAGENTS.mdgains one sentence saying that a force-push is governed by theAGENTS.mdof the repository the branch lives in. The sentence is appended to the existing absolute ban bullet (the one opening with 「绝不git push --force/--force-with-lease,绝不推main」) in 「多 agent 协作纪律」, in that section's Chinese voice:One file, one line changed (
AGENTS.md, +1/-1 as git counts it: the sentence is appended to an existing single-line bullet). No other edit.Placement — stated because the ruling names a section this file does not have
The ruling places the sentence in objectui
AGENTS.md's 「cross-repo section」. No heading in the file is a cross-repo section (the headings were enumerated on the branch base); the only heading naming a sibling is 「服务纪律(本仓库与../objectstack多 agent 并行开发)」, which is about dev servers. The sentence therefore sits beside the absolute ban it qualifies, at the end of that bullet, inside the multi-agent section that already carries the other cross-repo rules (the verify lock living in../objectstack, the model-free trailer pair shared with../objectstack).How the sentence obeys this file's own commandments
Never force-push a *shared* branch, not by a line address;pnpm check:new-line-citationsreports 0 added citations.landing repois kept in parentheses so the premise query that found zero before this change (git grep -n -iE "landing repo|the branch lives in" -- AGENTS.md) now finds the sentence.Premise, re-measured on the branch base
06a96e9480git grep -n -iE "landing repo|the branch lives in" 06a96e9480 -- AGENTS.md⇒ 0 hits (exit 1); controlgit grep -n -e "--force-with-lease"on the same tree ⇒ 1 hit (the ban bullet). After this commit the first query answers 1.origin/main(7dc45eb9): the Multi-agent working discipline rule opening 「Never force-push a shared branch」 still states the five criteria ①claude/issue-*name ② this worktree created it ③ nobody else pushed ④ no reviewer/approval on its PR ⑤ explicit--force-with-lease=BRANCH:SHA— matches the sentence.main:.claude/agents/os-dev.mdcarries 「force-push 按落地仓 AGENTS.md:objectui/cloud 绝对禁;objectstack §3 五条全立才--force-with-lease」.af2221d4). This PR is the remaining half, and with it the card is complete.Verification — all on HEAD
94e5b21280scripts/markdown-test-inputs.mjsrecords asAGENTS.mdreaders (check-doc-links,check-installed-spec-pin-claims,check-required-check-set,check-shell-escape-residue,dollar-dialect-alias-census),pnpm exec vitest run --maxWorkers=2 …under the shared verify lockTest Files 5 passed (5),Tests 260 passed (260),VERDICT command-exit 0pnpm check:control-bytesOK (scanned 9234 tracked text file(s); skipped 85 binary)pnpm check:new-line-citationsVERDICT new-cross-file-line-citations: 0 new citation(s)(merge base06a96e9480, 1 file compared)node scripts/check-doc-links.mjsLinks are valid across 17 scan roots.pnpm check:shell-escape-residueAGENTS.md: 1 file(s), 15 fence(s)among the rootspnpm check:installed-pin-claimspnpm check:required-check-set(self-test)node scripts/check-changeset-presence.mjsnode scripts/check-governed-queue-guard.mjs --test AGENTS.mdGOVERNED — 1 of 1 path(s)(expected)Not run locally, CI-owned:
pnpm lintand the fullpnpm testfarm.Acceptance notes
objectstack-ai/cloudis not reachable from this session (REST answers 403, repository access not enabled). The sentence's 「cloud 一律绝对禁止」 carries the ruling's own reading of cloud'sAGENTS.md, and the sentence marks the whole three-repo reading as un-re-derived.AGENTS.md, Tier H): this PR stays DRAFT. It is not flipped ready, not enqueued and not armed for auto-merge by the implementing agent; landing waits for an APPROVED review by an authorized approver and is then done by the claiming seat.维护者速读(草稿)
AGENTS.md「多 agent 协作纪律」里那条「绝不 force-push」的末尾加了一句话:force-push 归分支所在仓的AGENTS.md管 —— objectui 与 cloud 绝对禁止,objectstack 在五项条件全部成立时允许--force-with-lease;跨仓工作的席位给 dev 下指令前,先读落地仓自己的那条。只改这一处。Implemented in session
https://claude.ai/code/session_01EBx9rvB7dufCz4at53x35U.Generated by Claude Code