Uh oh!
There was an error while loading. Please reload this page.
docs(qa,pm): qa-run joins the sweep exclusion list; run records owe defect extraction at close-out - #9366
Merged
Merged
Conversation
…efect extraction at close-out (#9363) Per the maintainer's 2026-08-17 ruling, qa-run records are protocol carriers like the pm:seat post, not work: the triage sweep excludes them, and the extraction obligation (each product defect extracted as its own standalone card at report close-out; checklist-accuracy and fixture gaps to the wave anchor; environment blockers recorded only) becomes part of the qa-run protocol itself in RUNNER.md, with the checklist-test skill and the two checklist READMEs aligned — run issues carry qa-run only, work labels ride the extracted cards. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017TNzEetykdh7ceZGwuAPLq
This was referenced Aug 17, 2026
os-zhuang
marked this pull request as ready for review
August 17, 2026 21:54
Uh oh!
There was an error while loading. Please reload this page.
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#9363
Implements the maintainer's 2026-08-17 ruling, quoted verbatim and untranslated: 「把 qa-run 加进 sweep 排除清单 —— run 记录和 pm:seat 贴同类:协议载体,不是工作」— plus the closing half the card mandates: once run records are excluded from the sweep, nobody fishes them for dispatchable work, so the extraction obligation becomes a clause of the qa-run protocol itself (precedent pair: run record #9351 → extracted card #9362).
Surface 1 —
.claude/skills/pm-dispatch/SKILL.md(backlog sweep exclusion)qa-runrecords added to the triage sweep's exclusion list, parallel with thepm:seatposts, with the one-phrase rationale the card prescribes (协议载体非工作). Net line change +0 — the edit lands inside the existing exclusion sentence, no new lines, no re-wrap; the file stays at exactly its 682-line ceiling (check:pm-skill-ratchetgreen below).Surface 2 —
docs/qa/platform-checklist/RUNNER.md(the qa-run producer's protocol)New clause "Extraction obligation — the run record is a protocol carrier, not work", carrying the card's four points:
bug, priority, …); work labels ride the extracted cards;qa-runand enter triage first-touch normally;The ruling is quoted in the clause verbatim, untranslated. RUNNER.md is not on the line ratchet; the clause is written tight (net +23 lines in that file).
Consistency sweep — the old "plus
bug" convention removed everywhereThe prior convention (run issue gains
bug/regressionlabels so it triages straight from the run) contradicts the new clause and was stated in four places. All are reconciled in this PR — same surface, same defect class, each named here:docs/qa/platform-checklist/RUNNER.md— accuracy rule 2 ("which is labeledbug") and the Run-records header ("plusbugwhen any clause failed") → run issue isqa-run-only;.claude/skills/checklist-test/SKILL.md§4 — the label bullet rewritten (qa-runonly, work labels ride the extracted cards) and a close-out extraction bullet added; §3's "defects go to FOLLOW-UPS.md (or an issue)" now routes through the §4 extraction, keeping the expected-fail-probe half and the "never tick green on a real defect" rule intact;docs/qa/platform-checklist/runs/README.md— thebug-label sentence → extraction at close-out;docs/qa/platform-checklist/README.md(release-sweep section) — same.Tier derivation (
node scripts/pm/dispatch-gates.mjs --tieron the changed paths, output verbatim)Verification — full gate union re-run at the final commit,
git rev-parse --short HEAD=9c797322cPath-derived gates (
dispatch-gates.mjsnamed 5) plus implicated families, all green at9c797322c:check:pm-skill-ratchet(+--self-test, 11 cases) — SKILL.md 682/682, headroom 0check:pm-skill-id-lint— 9 files cleancheck:skill-frame-sync— 4 copies isomorphic, 3 axescheck:doc-authoring(+ self-test) — 376 files cleancheck:doc-formula-expressions(+ self-test, 24 cases) — 22 formula examples / 395 files clean (built the@objectstack/lintdependency closure first in the fresh worktree)check:nul-bytes— 6087 text files cleancheck:platform-checklist(+checklist-select --self-test, 17 cases) — 15 areas, 190 items OKNo changeset: docs/
.claude-only, releases nothing (skip-changesetapplied).Per the card: draft PR, never ready/queued/auto-merged — parked for the maintainer.
Generated by Claude Code