feat(aidd-orchestrator): add independent acceptance QA before draft pull requests
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 35/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Active
- Domain
- devtools, release, testing-qa
Research direction
Start with the SDLC flow in plugins/aidd-orchestrator/skills/01-sdlc/, then read the Acceptance QA hardening from #918, the developer journey in plugins/aidd-dev/skills/06-test/, and pull-request evidence in plugins/aidd-vcs/skills/02-pull-request/. Trace the existing Review, Challenge, and draft pull-request flow before defining the provider, routing, artifact ownership, and installation checks. Done means the listed passing, failing, blocked, skipped, artifact, and end-to-end fixture criteria are verified without treating developer-side evidence as acceptance evidence.
Written by the indexing model from the issue text.
Description
Problem
aidd-orchestrator:01-sdlc currently hands a committed candidate from Deliver to an independent Review, then Challenge, then a draft pull request. It never invokes aidd-qa:01-acceptance-qa, so a browser-facing change can reach the draft pull request without the independent qa.md and video evidence introduced in #917.
The existing aidd-dev:06-test browser journey is intentionally developer-side. It provides fast implementation feedback before the candidate is committed and must not become the owner of final acceptance evidence.
Two integration constraints also need an explicit contract:
aidd-qais still outside the recommended installation path;- the independent checker cannot currently invoke
aidd-qaor write its report artifacts.
Outcome
As a developer running the autonomous SDLC, I want browser-observable acceptance criteria independently exercised before a draft pull request is opened, so the pull request carries trustworthy acceptance evidence and failures return to the correct delivery stage.
Dependency
- Blocked by #918. Complete the Acceptance QA hardening before enabling it as an SDLC gate.
Ownership decision
- Keep
aidd-dev:06-testtest-journeyin Deliver for developer-side feedback during implementation. - Make
aidd-qa:01-acceptance-qathe sole owner of durable acceptance evidence:qa.mdand finalqa/*.webmfiles. - A developer-side journey report or screenshot never satisfies the Acceptance QA gate.
SDLC flow
Extend Check to run this sequence:
- Review the committed candidate with the existing fresh checker.
- When Review has no actionable finding, let that checker invoke
aidd-qa:01-acceptance-qaagainst the same contract and candidate. - Permit the checker to write only the report and media paths owned by Acceptance QA. It must never edit the product, contract, plan, or validator.
- Route the Acceptance QA result:
pass: continue to Challenge withqa.mdand the final videos as evidence;skipped: continue to Challenge with the skip report when no browser-observable criterion exists;fail: return the product mismatch to Deliver for repair;blockedby ambiguous or incomplete criteria: return to Frame;blockedby environment, permissions, dependencies, or tooling: stop with the exact blocker and required user action instead of dispatching a code repair.
- After Acceptance QA and Challenge both clear, verify that every new worktree change is an Acceptance QA artifact. Any product or validator change re-enters Review.
- Commit the accepted QA artifacts, then open the draft pull request.
Failed or blocked evidence remains available during the repair cycle but is not committed as final pull-request evidence. A subsequent Acceptance QA run replaces it with the result for the repaired candidate.
Provider availability
- After #918 is complete, move
aidd-qainto the recommended installation path and update the documented stable-plugin count and installation guidance. - Keep the SDLC's provider check. If
aidd-qa:01-acceptance-qais unavailable, stop with the shortest installation instruction; never skip the gate silently.
Pull request evidence
- Pass
qa.mdand every finalqa/*.webmpath toaidd-vcs:02-pull-request. - Extend the pull-request draft contract to link
qa.mdas well as its existing video links under the testing or verification section. - The pull-request summary states
passorskipped; it never presents a failed or blocked run as cleared.
Acceptance criteria
-
aidd-dev:06-testtest-journeyremains developer-side and produces no artifact that can satisfy the independent Acceptance QA gate. - After #918,
aidd-qais included in the recommended installation path and the affected installation documentation is consistent. - An SDLC run verifies that
aidd-qa:01-acceptance-qais available before depending on it and reports one actionable installation blocker when it is absent. - A committed candidate with browser-observable acceptance criteria runs Acceptance QA after Review clears and before Challenge.
- The independent checker may invoke Acceptance QA and write only its owned
qa.mdand final video artifacts. -
passandskippedsupply their report to Challenge;fail, criteria-blocked, and operationally blocked results follow the routing defined above. - No failed or blocked run reaches draft pull-request creation as a cleared gate.
- After Challenge clears, only Acceptance QA artifacts may be committed without repeating Review.
- The draft pull request links
qa.mdand every changed finalqa/*.webmfile. - A non-browser change can produce
skippedwithout running browser prerequisites or recording video. - An end-to-end SDLC fixture proves both the passing browser-change route and one failure route back to Deliver.
Out of scope
- API or CLI QA interfaces.
- The Acceptance QA behavior gaps owned by #918.
- Removing developer-side browser journey testing.
- Automatically installing a missing plugin during an SDLC run.
References
- Acceptance QA plugin: #917
- Acceptance QA hardening: #918
- SDLC flow:
plugins/aidd-orchestrator/skills/01-sdlc/ - Developer-side journey:
plugins/aidd-dev/skills/06-test/ - Pull-request evidence:
plugins/aidd-vcs/skills/02-pull-request/
- Dominant language
- TypeScript
- Stars
- 481
- Forks
- 45
- Avg merge
- 18h 48m
- Merged PRs (30d)
- 111
Getting set up
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ai-driven-dev/framework
-
refactor(aidd-orchestrator): the check zone says when to stop, and reviews its axes in one roundOpen
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ai-driven-dev/framework#887 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
ai-driven-dev/framework#873 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
ai-driven-dev/framework#625 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ai-driven-dev/framework#467 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
ai-driven-dev/framework#921 ·
Maintainers usually reply within 1 day
All issues in ai-driven-dev/framework
Similar issues
-
ADD openalgoOpentemplate
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day
-
factory-active factory-automatic task-bug-reproduction-success task-identify-harness-labels-done task-identify-issue-type-done
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
vercel/ai#21528 · 3 comments ·
Maintainers usually reply within 1 day
-
bug Needs: Triage :mag:
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
microsoft/fluentui-contrib#671 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
sveltejs/acorn-typescript#150 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100