QA Fix Loop

Адаптировано из Auto-Claude QA Loop для SENTINEL DevKit

Философия

Вместо ручного ping-pong между reviewer и developer, создаём автономный цикл:

• Reviewer Agent находит проблемы

• Fixer Agent исправляет их

• Re-review до достижения качества

Архитектура

┌─────────────┐ issues ┌─────────────┐ │ Reviewer │ ─────────────→ │ Fixer │ │ Agent │ │ Agent │ └─────────────┘ └─────────────┘ ↑ │ │ fixed code │ └──────────────────────────────┘

Reviewer Agent

Обязанности

• Проверка spec compliance (Stage 1)

• Проверка code quality (Stage 2)

• Генерация структурированных issues

Output Format

{ "status": "NEEDS_FIX | APPROVED", "issues": [ { "id": "QA-001", "severity": "HIGH | MEDIUM | LOW", "file": "path/to/file.py", "line": 42, "type": "SPEC_VIOLATION | CODE_QUALITY | SECURITY", "description": "Что не так", "suggestion": "Как исправить" } ], "approved_aspects": ["tests", "documentation"] }

Fixer Agent

Обязанности

• Получает issues от Reviewer

• Исправляет каждый issue

• Документирует изменения

• Запрашивает re-review

Process

• Для каждого issue:

• Понять root cause

• Применить минимальное исправление

• Добавить тест если нужно

• Собрать все изменения

• Вернуть на review

Workflow (DOT)

digraph qa_fix_loop { rankdir=LR; node [shape=box, style=rounded];

code [label="New Code"];
review [label="Reviewer Agent"];
decision [shape=diamond, label="Issues?"];
fix [label="Fixer Agent"];
done [label="✅ Merged", style=filled, fillcolor=lightgreen];

code -> review;
review -> decision;
decision -> fix [label="Yes"];
decision -> done [label="No"];
fix -> review [label="re-review"];

}

Лимиты и Safety

Max Iterations

• Default: 3 итерации

• После 3 итераций: Эскалация к человеку

Severity Escalation

• HIGH severity не исправлена: Немедленная эскалация

• Security issue: Блокировка merge + alert

Infinite Loop Prevention

if iteration > MAX_ITERATIONS: escalate_to_human( reason="QA loop exceeded max iterations", issues=unresolved_issues )

Интеграция с SENTINEL

Для Engine QA

1. Engine code → Reviewer Agent
2. Reviewer checks:
   • All payloads detected (Strike integration)
   • Interface compliance
   • Performance bounds
3. Issues → Fixer Agent
4. Loop until approved

Для Shield QA

1. C code → Reviewer Agent (специализированный)
2. Reviewer checks:
   • Memory safety
   • Protocol compliance
   • Cross-platform compatibility
3. Issues → Fixer Agent
4. Loop until approved

Metrics

Трекинг эффективности цикла:

• Loop count — среднее число итераций

• Issue categories — какие типы проблем чаще

• Time to fix — время на исправление

• Escape rate — сколько issues пропущено на production

Anti-patterns

❌ Бесконечный loop — всегда ставить max iterations ❌ Fixer игнорирует issues — проверять что ВСЕ issues адресованы ❌ Reviewer слишком строгий — избегать nitpicking на каждой итерации ❌ Нет эскалации — человек должен быть в loop для сложных cases