Repository navigation
docs(qa): a verification re-run files no qa-run record; sweeps still file one per run - #24302
Merged
objectstack-fleet[bot] merged 1 commit intoOct 18, 2026
Conversation
…file one per run RUNNER.md, README.md and runs/README.md still said every completed run files a qa-run issue. Maintainer ruling C (record 6230593290) exempts a verification re-run, a run whose trigger is the same PR's ledger edit of that item: no record, its per-item table in the PR body, the item's history row naming the PR, a partial/blocked gap also in fixtures.knownGaps. Each of the three record-filing sentences now scopes the one-record rule to sweep runs (release candidate, area or all run, maintainer-named target) and states the exception, in agreement with the checklist-test skill's verification re-run paragraph (section 5). Title grammar, label discipline and extraction obligation unchanged. Claude-Session: https://claude.ai/code/session_01VwoC2RxFfgzY2eXuC7Xo6g Co-authored-by: Claude <noreply@anthropic.com>
Contributor
Author
席位核对 · ACCEPT: PR #24302 at
|
This was referenced Oct 18, 2026
objectstack-fleet
Bot
deleted the
claude/issue-24173-verification-rerun-docs
branch
October 18, 2026 10:19
This was referenced Oct 18, 2026
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 free
to 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 #24173
Clause-②: no
Docs-only follow-up to maintainer ruling C on #24127 (ruling record
6230593290, the maintainer's choice verbatim: 「C 有问题才开卡 (Recommended)」). The checklist-test skill half merged asb0fe9fad47(#24172). Threedomain:devxdocuments that the skill tells its runner to read first still said that every completed run files aqa-runissue. A runner that obeyed them would file the record the ruling says not to file.What changes (three record-filing sentences, nothing else in those files)
docs/qa/platform-checklist/RUNNER.md(Run records, first paragraph)allrun, or a maintainer-named target) files oneqa-runissue. A verification re-run files none: a run whose trigger is the same PR's ledger edit of that item. Its per-item verdict table goes in the PR body, and the item'shistoryrow carries the PR as itsref. Apartial/blockedgap goes in the PR body and infixtures.knownGaps. A product-defectfailis still extracted as its own card, pointing at the PR. A stale clause is fixed in the same diff. The sentence points at.claude/skills/checklist-test/SKILL.md§5, 「验证重测」.docs/qa/platform-checklist/runs/README.mdL11–12qa-runGitHub issue」historyrow, and apartial/blockedgap also goes infixtures.knownGaps. It defers to RUNNER.md, Run records, for the full exception. The list lead-in becomes "A sweep's record:".docs/qa/platform-checklist/README.md(How a release sweep works)qa-runGitHub issue」historyrow (RUNNER.md, Run records).Unchanged, per the card's boundary: RUNNER.md's title grammar (「The record title」), its label discipline (
qa-runand nothing else) and its extraction obligation section. No gate is added.Agreement with the merged skill text (acceptance 1)
Each clause of the skill's paragraph at
origin/maind537a318b3(.claude/skills/checklist-test/SKILL.mdL223–225, §5), set against RUNNER.md's new sentence:qa-run记录history行的ref指向该 PRhistoryrow carries the PR as itsref"partial/blocked的缺口进 PR 正文与fixtures.knownGapspartialorblockedgap goes in the PR body and the item'sfixtures.knownGaps"fail照常抽卡、溯源指向该 PRfailis still extracted as its own card, pointing at the PR"all、维护者点名目标)恰好立一张allrun, or a maintainer-named target) … files one GitHub issue"Premise re-check (on
origin/maind537a318b3, with controls)RUNNER.md「pass or fail alike」 → 1 hit (L460). Control 「The record title」 → 1 hit.runs/README.md「every completed run files one」 → 1 hit (L11). Controlqa-run→ 2 hits.README.md「every run files」 → 1 hit (L466). Controlhistory→ 7 hits.c40506aac(docs(qa): RUNNER.md reads the connector self-URL from /meta/connector and states the scheduled-work switch #24156). Nothing is newer: history was deepened 60 commits past the shallow boundary to read this.git grepforRUNNER.md:/#Lforms). Pins quoting the three old sentences outside these files: none found (git grepexit 1; controlqa-runhits inscripts/qa/qa-rollup.mjs).Gates (local, all run at head
27ab96524, exit captured before any pipe)The list is derived by
node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstackfrom the actual diff: 13 families, plus the watchdog that the card's acceptance names.node scripts/check-ci-filter-parity.mjs: exit 0node scripts/check-closing-keyword-parity.mjs: exit 0.--self-test: exit 0node scripts/check-comment-mask-corpus.mjs: exit 0 (9472 files, 0 disagree)pnpm --filter @objectstack/lint run check:doc-formula-expressions: exit 0turbo run build --filter=@objectstack/formula --filter=@objectstack/lint, 4/4 cached), then re-ran.pnpm check:cross-package-test-inputs: exit 0pnpm check:doc-authoring: exit 0pnpm check:driver-memory-census: exit 0pnpm check:gitlink-declared: exit 0pnpm check:nul-bytes: exit 0. A separate control-byte scan of the three files found none.pnpm check:platform-checklist: exit 0 (16 areas, 327 items; traps 19 documented, 19 in use)pnpm check:refd-timer-probe: exit 0pnpm check:watch-hint-literal: exit 0node scripts/check-platform-checklist-watchdog.mjs: exit 0dispatch-gates --ranreconciliation: "13 derived, 13 run, 0 NOT-MEASURED, 0 UNRUN". All 13 carry an exit code, so the zero is derived from them. The watchdog is recorded as outside the derivation.No tests or ablation: this is a docs-only diff with no tested behaviour. Families that CI alone runs are read from this PR's check-runs.
Changeset
skip-changeset, measured.docs/qa/**sits at the repo root. The rootpackage.jsonisprivate: true, and no package manifest referencesplatform-checklist. Control:git grep '"files"'overpackages/*/package.jsonhits 74 manifests. Nothing in this diff ships in any published tarball.Acceptance notes
qa-runrecord when every clause passes" and then coverspartial/blockedand the two kinds offailone by one. The merged skill states the no-record rule for every verification re-run, with the product-defectfailgoing to a card that points at the PR. These docs follow the merged skill text, as acceptance 1 requires.Generated by Claude Code