Uh oh!
There was an error while loading. Please reload this page.
docs(qa-checklist): correct two knownGaps that still deny the provisioning.use resolve - #10883
Merged
Merged
Conversation
…oning.use resolve The area-scoped `provisioning.use` resolve landed with the gap-1 check, so two knownGaps clauses stating the validator "does not resolve it either way" are now false. The inversion matters more than a stale footnote: the clause reads as PERMISSION for a cross-area `use`, and that spelling now fails check:platform-checklist naming the item — the prose invited exactly the edit the gate rejects. Both clauses now state what is enforced today (same-area resolves, cross-area fails). The cross-area SPELLING is still undecided and is deliberately not predicted here, so this correction cannot become the next stale permission. records-forms.crud-roundtrip rev 5 -> 6 (its own fixtures.knownGaps). The sibling clause is on the AREA-LEVEL recipe qa-contributor-bound-member, which carries no revision/history of its own, so search.rls-both-personas rev 4 -> 5 is the only pin a run record can hold that recipe text against; its own fields are unchanged. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DdCnBGcHeufjrq7drTD3wt
os-zhuang
marked this pull request as ready for review
August 21, 2026 16:01
Uh oh!
There was an error while loading. Please reload this page.
Contributor
⛔ merge queue 构建失败 — 先分诊,再决定要不要重排队列构建 32500800720 红了。队列跑的是全量套件(PR 侧 CI 只跑 affected 子集), 失败的 job(日志抽取,best effort):
跨 PR 相同签名(24h,按失败测试文件聚合):
历史信号:
分诊清单:
Generated by Claude Code · merge-queue-triage workflow (#4859) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes#10809
Two
knownGapsclauses stated that the validator does not resolveprovisioning.usein either direction. The area-scoped resolve landed with the gap-1 work on #10593, so that
is false today: a same-area
useresolves, and a cross-area one fails. The clausesalso still cited the old deferral (#7716 / #7720) as if it were live.
The inversion matters more than a stale footnote. The clause reads as permission: an
author following it would spell a cross-area
useand get a red they were told to expectnot to get. A stale permission outranks a stale fact, which is why this is corrected now
rather than folded into the pending convention work.
What changed
areas/records-forms.jsonrecords-forms.crud-roundtrip→ its ownfixtures.knownGaps[0]areas/search.jsonqa-contributor-bound-member→knownGaps[0]areas/search.jsonsearch.rls-both-personas— revision pin only, own fields unchangedBoth clauses keep their first half (the mechanism is AREA-SCOPED — still true and now
load-bearing) and state what is enforced today. The cross-area spelling is deliberately
not predicted: that convention is the open half of #10593, still undecided, and writing a
guess into a checklist is how the next stale permission gets created.
One correction to the card
The card places the sibling clause on item
search.rls-both-personas. It is actually on thearea-level recipe
qa-contributor-bound-member, which carries norevision/history—the lifecycle check in
scripts/check-platform-checklist.mjsis item-scoped, and none of thefour recipes in this checklist has either field. So the "both items owe a revision bump"
obligation was re-derived rather than applied literally:
search.rls-both-personasis bumpedbecause it
uses that recipe, making its revision the only pin a run record can hold therecipe text against. Its own steps, clauses, fixtures and provisioning are unchanged, and the
history entry says so explicitly, so no run verdict is invalidated.
Verification — this gate is not CI-wired
check:platform-checklistis not wired into CI, so a greenLint & Repo Gateson this PRsays nothing about it. Both halves were run by hand at
af2e9ee9b4, before and after:Invariants are identical to the pre-edit baseline (205 items, 4 recipes, 6 references resolved).
Ablation — to show the revision bump is enforced rather than assumed. Predicted before
running: exactly one reddened case naming the item. Reverting
crud-roundtripto revision 5while its history's last entry stays 6 (mutation proven on disk: target anchor 1 → 0,
replacement +1, and the parsed field read back as
revision=5, last history=6):The restore leg was run too: byte-identical to the commit, all three halves green again.
Derived gates (
scripts/pm/dispatch-gates.mjs, 2 paths) and the hand-runscripts/**pair:No changeset
skip-changeset: nothing here is published.content/docs/is the docs-site source; thesefiles live in
docs/qa/, which noapps/docsorcontent/path references and nopackage.jsonships.Generated by Claude Code