qaa-agent 1.9.2 → 1.9.6

package/agents/qaa-codebase-mapper.md

@@ -924,6 +924,60 @@ Test the highest-risk gaps first:
 
 </critical_rules>
 
+
+<quality_gate>
+Before considering codebase mapping complete, verify ALL of the following:
+
+- [ ] Focus area received and acknowledged in the agent prompt (testability, risk, patterns, or existing-tests)
+- [ ] Documents produced for the specified focus area:
+  - testability → TESTABILITY.md and TEST_SURFACE.md
+  - risk → RISK_MAP.md and CRITICAL_PATHS.md
+  - patterns → CODE_PATTERNS.md and API_CONTRACTS.md
+  - existing-tests → TEST_ASSESSMENT.md and COVERAGE_GAPS.md
+- [ ] All file paths in produced docs are formatted with backticks (e.g., `src/services/payment.ts`)
+- [ ] Every observation has a concrete file:line reference, not vague descriptions
+- [ ] Mock boundaries documented (testability focus): "Mock X when testing Y"
+- [ ] Risk levels assigned per item (risk focus): HIGH | MEDIUM | LOW with justification
+- [ ] Code examples included with code patterns (patterns focus)
+- [ ] Existing tests catalogued with their quality assessment (existing-tests focus)
+- [ ] Output files written to ${output_dir}/codebase/
+
+If any check fails, fix the issue before returning. Do not deliver codebase docs that fail their own quality gate.
+</quality_gate>
+
+## MANDATORY verification — run ALL commands below, no exceptions, no skipping
+
+Before returning control, copy-paste and run this ENTIRE block. Do NOT decide which commands "apply" — run all of them every time. The output confirms what happened; you do not get to assume the answer.
+
+<bash_checklist>
+Run this checklist verbatim before returning control:
+
+```bash
+echo "=== CODEBASE-MAPPER CHECKLIST START ==="
+echo "1. Focus area received:"
+echo "${FOCUS_AREA:-MISSING}"
+echo "2. Output dir:"
+ls -d "${output_dir:-.qa-output}/codebase/" 2>/dev/null || echo "DIR_MISSING"
+echo "3. Documents produced for this focus:"
+ls "${output_dir:-.qa-output}"/codebase/*.md 2>/dev/null | wc -l
+echo "4. File paths backticked (per philosophy):"
+grep -cE '`[a-z][^`]+\.(ts|js|py|go|java)`' "${output_dir:-.qa-output}"/codebase/*.md 2>/dev/null
+echo "5. Mock boundaries (testability focus):"
+[ "$FOCUS_AREA" = "testability" ] && grep -ic "mock" "${output_dir:-.qa-output}/codebase/TESTABILITY.md" 2>/dev/null || echo "n/a"
+echo "6. Risk levels (risk focus):"
+[ "$FOCUS_AREA" = "risk" ] && grep -cE "HIGH|MEDIUM|LOW" "${output_dir:-.qa-output}/codebase/RISK_MAP.md" 2>/dev/null || echo "n/a"
+echo "7. Code examples (patterns focus):"
+[ "$FOCUS_AREA" = "patterns" ] && grep -c '```' "${output_dir:-.qa-output}/codebase/CODE_PATTERNS.md" 2>/dev/null || echo "n/a"
+echo "8. Existing tests catalogued (existing-tests focus):"
+[ "$FOCUS_AREA" = "existing-tests" ] && grep -ic "test" "${output_dir:-.qa-output}/codebase/TEST_ASSESSMENT.md" 2>/dev/null || echo "n/a"
+echo "=== CODEBASE-MAPPER CHECKLIST END ==="
+```
+
+**Rules:**
+- Run the block AS-IS. Do not modify it. Do not split it.
+- If any output shows a problem, fix it before returning.
+- Do NOT return control to the parent agent until the block has been executed.
+</bash_checklist>
 <success_criteria>
 - [ ] Focus area parsed correctly
 - [ ] Codebase explored thoroughly for focus area (3+ representative files read in depth)
@@ -935,4 +989,3 @@ Test the highest-risk gaps first:
 - [ ] No secrets or forbidden file contents leaked
 - [ ] Confirmation returned (not document contents)
 </success_criteria>
-