Withdraw the PR-triggered coverage workflow (RFC 0004, second half) - #90
Merged
Merged
Conversation
…half)
Released too soon in 0.9.0: one demo repository is not enough evidence to run a
bot on other teams' pull requests. Removed from main: the reusable workflow
qa-buddy-pr.yml with its prompts, renderer and installer; bin/pr-coverage.js
(touched, heatmap, comment, merge, preflight, init, summary, issues) and its
test block; the /qa-setup PR-automation step and --pr (0.6.0, three fixtures);
the two CI guides; the README section and nav link. Callers pinned to v0.9.0 /
v0.9.1 keep working — tags are immutable.
Kept: headless mode (preamble, --headless, the claude -p runner) — the RFC 0005
eval bench runs on it; sources.json and exploratory/{date}.md in the KB spec,
reworded for their remaining readers; RFC 0004 as the record, status Withdrawn
for the workflow half. /qa-test-cases 0.6.3: self-check 8 no longer names the
heatmap. CHANGELOG (en, ko) under Removed. Suite 1715.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Open
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.
Why
Released too soon in 0.9.0. One demo repository is not enough evidence to put a bot on other teams' pull requests. This removes the feature from
main; RFC 0004 stays as the record with status Withdrawn for the workflow half.Removed
.github/workflows/qa-buddy-pr.ymland.github/qa-buddy-pr/(prompts, renderer, installer, MCP config, README)bin/pr-coverage.js— touched, heatmap, comment, merge, preflight, init, summary, issues — and itstest.jsblock (suite 1874 → 1715)/qa-setup0.6.0: the PR-automation step (Phase 5b),--pr, and the three fixtures that tested them, in both localesdocs/pr-coverage.md,docs/pr-coverage-en.md, the README section and nav link in both localesKept, on purpose
--headless, and theclaude -prunner. The RFC 0005 eval bench runs on it;.github/workflows/skill-eval.ymlis untouched.sources.jsonand §6.9exploratory/{date}.md— reworded for their remaining readers (the eval bench grades §6.9). §6.8 now says no shipped tool consumes it since 0.10.0 and the shape is kept for the next consumer./qa-test-cases0.6.3 — self-check 8 named "the PR heatmap"; it now names every reader. Theartifacts-parserubric statement matches.v0.9.0/v0.9.1tags. Callers pinned to them keep working; nothing onmainruns on a PR any more.Gate
eval.js scopesays RUN for test-cases (self-check 8's text changed, andartifacts-parsecites it) — the A/B is running and its table will be added here. It also says RUN for exploratory because KB spec §6.9 changed one sentence naming the consumer; I have not run that one: nothing the exploratory rubric cites changed, and the sentence renames a reader. If you want it before merging:node bin/eval.js ab exploratory --a origin/main --b remove/pr-coverage.Related
pr-coveragebug status, finding resolution, feature-scoped bugs, TC id collisions) is now mostly about deleted code. Recommend closing it; its writer-side contracts (bug**Status:**line, finding**Status:**re-check, per-feature TC numbering) are still sound hygiene and can come back as a small skills-only PR without the pipeline wording if you want them.qa:*labels are yours to remove; they will keep working against the pinned tag until then.CHANGELOGcarries this under Removed in both locales; the next release is a minor bump (0.10.0).🤖 Generated with Claude Code