docs(agents): a pin-breaking removal ships with the sibling fix and pin bump - #11108
Conversation
…in bump Encode the C half of the maintainer's 2026-08-22 A+C ruling as Post-Task Checklist item 4: a removal or rename the pinned objectui checkout still imports must land together with the objectui-side fix and the .objectui-sha bump (sdui:manifest second half included), because the Console Pin Gate builds the pinned sibling against current main — merging the removal alone turns main red for every PR in the repo. The ruling that authorizes the removal implicitly authorizes the sibling fix + pin bump in the same landing. Line ratchet payment (AGENTS.md ceilinged at 961, zero headroom): the one added line is paid by deleting the seventeen-phantom-pins incident parenthetical in the Build & Test section — incident case law lives in git history per the ratchet header's own guidance. The optional one-line pre-merge check in the spec-property-retirement SKILL.md checklist is deferred: PR #11092 is in flight touching that file with auto-merge armed. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MsbKEG4LtERSLaDrbehM3e Fixes #10986
PM 验收 — ACCEPT(终态)复核:席位 session_01MsbKEG4LtERSLaDrbehM3e 全 diff 审毕(+6/−6 单文件)。裁定实现吻合:#10856 A+C 裁定的 C 半逐字落为 Post-Task Checklist 第 4 条,指令 + 单句因由(Console Pin Gate 以 pinned 治理面终态:AGENTS.md —— 本 PR 保持 draft,仅维护者合并;⛔ 不 ready、不入队、不布防。已指派维护者。合并后剥离 #10986 的 Generated by Claude Code |
⛔ merge queue 构建失败 — 先分诊,再决定要不要重排队列构建 32587834474 红了。队列跑的是全量套件(PR 侧 CI 只跑 affected 子集), 失败的 job(日志抽取,best effort):
跨 PR 相同签名(24h,按失败测试文件聚合):
历史信号:
分诊清单:
Generated by Claude Code · merge-queue-triage workflow (#4859) |
Fixes #10986
What
Encodes the C half of the maintainer's 2026-08-22 A+C ruling (accepted verbatim 「接受所有」) as a working agreement in
AGENTS.md, Post-Task Checklist item 4:A removal that breaks the pinned sibling checkout ships together with the sibling fix and the pin bump — or it does not ship.
Rationale encoded with the directive: the
Console Pin Gatejob builds objectui at the pinned.objectui-shaagainst currentmain, so a removal the pinned sibling still imports turnsmainred for every PR in the repo the moment it merges. A ruling that authorizes such a removal therefore implicitly authorizes the objectui-side fix and the pin bump as part of the same landing (including the bump'ssdui:manifestsecond half). The item ends with the one-line pre-merge check: does the pinned sibling import what you are removing?git grepit in../objectuiat the pinned SHA.Placement: new Post-Task Checklist item 4, directly after item 3's retirement paragraph (the prose that points a retirement author at the
spec-property-retirementskill) and directly before the spec-export-removal item — the PR-flow section a retirement author actually reads. Former items 4-6 renumbered to 5-7; the only externally referenced item number is item 3 (docs/audits/2026-07-unknown-key-strictness-ledger.md, ADR-0115), which is unchanged.Line ratchet payment
AGENTS.mdis ceilinged at 961/961 with zero headroom. The one added line is paid by deleting the seventeen-phantom-pins incident parenthetical in the Build and Test section (the@ts-expect-errorbullet) — incident case law lives in git history, per the ratchet header's own guidance. After the edit:check-skill-line-ratchet: AGENTS.md is 961 lines (ceiling 961; headroom 0)— green, verified at commite94c5759.Deferred (declared, not dropped)
The optional one-line pre-merge check in
.claude/skills/spec-property-retirement/SKILL.mdis deferred: PR #11092 is in flight touching that file with auto-merge armed, and creating a conflict with it is worse than a follow-up. The AGENTS.md item carries the same one-line check, so the rule is not silent in the meantime.Gates (all at commit
e94c5759, exit codes captured before any pipe)check:pm-skill-ratchet—AGENTS.md is 961 lines (ceiling 961; headroom 0)check:pm-skill-id-lint—17 file(s) clean (pattern of three-plus-digit issue refs); the directive is self-contained, no issue numberscheck:doc-authoring—389 files clean — no bare metadata literalscheck:nul-bytes—OK (scanned 6417 text file(s) ... no raw ASCII control bytes)check:docs-audit-scope,check:pm-governed-merges,check:pm-governed-prose,check:required-contexts(both spellings) — all greennode scripts/pm/dispatch-gates.mjswith no hand-fed paths; change set =AGENTS.mdonlyGoverned surface
AGENTS.mdis a governed face: this PR stays draft, human merge only — never flipped ready, never queued, never auto-merged.Generated by Claude Code
Generated by Claude Code