Final Quality Audit
Purpose
Provide the final go/no-go before release by confirming all gates are genuinely satisfied — correctness, tests, security, docs, scope — and aggregating specialist reviews into one verdict. Supports Gates 6–7.
When to Use
- Immediately before release/merge of a completed work item.
- As the closing check of the orchestration pipeline.
- Not as a replacement for the specialist reviews it aggregates.
Inputs
- The completed change set and its work item/acceptance criteria.
- Outputs of
code-review, security-review, performance-review, testing-strategy, ai-output-review.
Discovery Questions
- Do results meet the original acceptance criteria and success criteria?
- Are Gates 5–6 genuinely passed (not assumed)?
- Is anything out of scope or incomplete?
- Are unrun checks explicitly flagged?
Responsibilities
- Confirm the work meets acceptance/success criteria.
- Confirm tests exist and pass (or are flagged) — Gate 5.
- Confirm security review done, no open Critical/High — Gate 6.
- Confirm docs/changelog updated.
- Confirm scope wasn't expanded and nothing is half-done.
- Aggregate specialist findings into one verdict; block on any failure.
Required Workflow
- Re-read acceptance/success criteria.
- Collect specialist review outputs.
- Verify each gate's evidence (not claims).
- Check scope and completeness.
- Issue a go/no-go verdict with rationale; list blockers if no-go.
Decision Rules
- Any open Critical/High security issue → no-go.
- Missing required tests or failing tests → no-go.
- Unmet acceptance criteria or half-done work → no-go.
- Scope expansion beyond approval → flag and split, don't wave through.
- Evidence over claims: "done" requires verification.
Rules
- Aggregate, don't duplicate the specialists' work — rely on their reports.
- Be honest: distinguish verified from assumed.
- Block on failed gates; never override a gate to ship.
Anti-Patterns
- Rubber-stamping without checking gate evidence.
- Passing work with open Critical/High issues.
- Accepting "should work" as done.
- Letting scope creep through unremarked.
Validation Checklist
Definition of Done
A final go/no-go verdict backed by verified gate evidence and aggregated specialist findings; on no-go, a clear, prioritized blocker list.
Related Skills
code-review, security-review, performance-review, testing-strategy, ai-output-review, documentation, release-planning, project-orchestrator.
Related Knowledge
../../knowledge/, ../../projects/current/ (gate status).
Related References
../../checklists/ (gate checklists).
Context Loading Guidance
- Requires: acceptance criteria, gate status, specialist review outputs.
- Does not require: re-reading all source (rely on specialist reports), unrelated references.
- May load: any specialist report needed to confirm a gate.
- Stop when: the go/no-go verdict is issued.
Token Efficiency Guidance
Consume specialist summaries rather than re-deriving them. Verify the decisive evidence per gate; don't re-review the whole change from scratch.
1---2name: final-quality-audit3description: Use as the last gate before release to confirm the work is genuinely complete, correct, tested, secure, documented, and in scope. Aggregates the specialist reviews into one go/no-go verdict; blocks on any failed gate.4---56# Final Quality Audit78## Purpose910Provide the final go/no-go before release by confirming all gates are genuinely satisfied — correctness, tests, security, docs, scope — and aggregating specialist reviews into one verdict. Supports Gates 6–7.1112## When to Use1314- Immediately before release/merge of a completed work item.15- As the closing check of the orchestration pipeline.16- **Not** as a replacement for the specialist reviews it aggregates.1718## Inputs1920- The completed change set and its work item/acceptance criteria.21- Outputs of `code-review`, `security-review`, `performance-review`, `testing-strategy`, `ai-output-review`.2223## Discovery Questions2425- Do results meet the original acceptance criteria and success criteria?26- Are Gates 5–6 genuinely passed (not assumed)?27- Is anything out of scope or incomplete?28- Are unrun checks explicitly flagged?2930## Responsibilities3132- Confirm the work **meets acceptance/success criteria**.33- Confirm **tests** exist and pass (or are flagged) — Gate 5.34- Confirm **security review** done, no open Critical/High — Gate 6.35- Confirm **docs/changelog** updated.36- Confirm **scope** wasn't expanded and nothing is half-done.37- Aggregate specialist findings into **one verdict**; block on any failure.3839## Required Workflow40411. Re-read acceptance/success criteria.422. Collect specialist review outputs.433. Verify each gate's evidence (not claims).444. Check scope and completeness.455. Issue a go/no-go verdict with rationale; list blockers if no-go.4647## Decision Rules4849- Any open Critical/High security issue → no-go.50- Missing required tests or failing tests → no-go.51- Unmet acceptance criteria or half-done work → no-go.52- Scope expansion beyond approval → flag and split, don't wave through.53- Evidence over claims: "done" requires verification.5455## Rules5657- Aggregate, don't duplicate the specialists' work — rely on their reports.58- Be honest: distinguish verified from assumed.59- Block on failed gates; never override a gate to ship.6061## Anti-Patterns6263- Rubber-stamping without checking gate evidence.64- Passing work with open Critical/High issues.65- Accepting "should work" as done.66- Letting scope creep through unremarked.6768## Validation Checklist6970- [ ] Acceptance/success criteria met.71- [ ] Tests present/passing (or flagged) — Gate 5.72- [ ] Security review done; no open Critical/High — Gate 6.73- [ ] Docs/changelog updated.74- [ ] Scope respected; nothing half-done.75- [ ] Single go/no-go verdict with rationale.7677## Definition of Done7879A final go/no-go verdict backed by verified gate evidence and aggregated specialist findings; on no-go, a clear, prioritized blocker list.8081## Related Skills8283`code-review`, `security-review`, `performance-review`, `testing-strategy`, `ai-output-review`, `documentation`, `release-planning`, `project-orchestrator`.8485## Related Knowledge8687`../../knowledge/`, `../../projects/current/` (gate status).8889## Related References9091`../../checklists/` (gate checklists).9293## Context Loading Guidance9495- **Requires:** acceptance criteria, gate status, specialist review outputs.96- **Does not require:** re-reading all source (rely on specialist reports), unrelated references.97- **May load:** any specialist report needed to confirm a gate.98- **Stop when:** the go/no-go verdict is issued.99100## Token Efficiency Guidance101102Consume specialist summaries rather than re-deriving them. Verify the decisive evidence per gate; don't re-review the whole change from scratch.