| name | review-ops |
| description | Orchestrator/A1 büyük iş bitince tetikle — batch task review, PR audit, kalite skoru, Jira güncel mi, eksik varsa yeniden task aç. Triggers: review-ops, ops review, batch bitti, orchestrator review, büyük iş bitti, toplu review, ne yaptık. |
| argument-hint | [jira-task-ids | repo-list | son-batch] |
/review-ops — Orchestrator / Batch İş Sonrası Review
Tamamlanan bir iş grubunu (Jira task seti, repo batch, sprint) incele.
Puan ver, eksik bul, gerekirse yeni task aç veya yeniden yaptır.
WARNING — DO NOT invoke via forked subagent. Fork would inherit the context of the code/work being audited and bias the reviewer toward "I wrote this, it's fine." Always spawn review-ops as a normal (blank) subagent, regardless of how the parent session arrived here.
Ne zaman tetiklenir
- A1 veya büyük bir agent batch'i tamamlandıktan sonra
- Sprint sonu
- Kullanıcı "bitti mi, iyi mi" tarzında bir şey sorduğunda
/review-sprint CC-10,CC-13,CC-14 gibi explicit çağrı
Akış
1. Kapsam belirle (max 3 tool call)
Argüman gelirse parse et:
CC-10,CC-13 → o task'ları Jira'dan okuccplugin-sprint-planner,ccplugin-code-quality → o repoları incele- Argümansız → son 24 saatteki Done task'ları Jira'dan çek
2. Her task/repo için review (C1/C3 sub-agent dispatch)
Her öğe için paralel C3 (Local AI Reviewer) başlat:
Jira task review:
- Jira'dan task detayını oku: açıklama, acceptance criteria var mı?
- GitHub'dan ilgili commit'leri bul (task key ile ara)
- PR açılmış mı? (dal kuralına uyulmuş mu?)
- Commit mesajı convention'a uygun mu? (feat/fix/chore...)
Kod review:
- İlgili dosyalara bak (değişen dosyalar)
- Doğruluk: amaçla örtüşüyor mu?
- Güvenlik: gizli bilgi, injection riski?
- Basitlik: gereksiz karmaşıklık?
3. Puanla
Her task/repo için 0-10 arası skor:
| Kriter | Ağırlık |
|---|
| Acceptance criteria karşılandı mı? | 30% |
| Kod kalitesi (doğruluk + güvenlik) | 30% |
| Git workflow uyumu (PR, branch, commit) | 20% |
| Jira güncelliği (Done, yorum, link) | 20% |
4. Raporla
## Sprint Review Raporu
### ✅ CC-10 — /sprint-plan sync (8.5/10)
- Acceptance: ✓
- Kod: Sync workflow doğru implement edilmiş
- Git: direkt push — kural uygun (tek commit, 1 dosya) ✓
- Jira: Done ✓
Yorum: İyi iş. Edge case: sprint kapalıysa ne olur?
### ⚠️ CC-13 — /memory-prune (6/10)
- Acceptance: ✓
- Kod: Temel implement var ama edge case eksik
- Git: PR açılmamış — 3 dosya değişti, PR gerekirdi ✗
- Jira: Done ✓
Aksiyon: CC-XX açıldı — PR retroaktif aç veya edge case'leri tamamla
Genel Skor: 7.2/10
5. Aksiyon al
| Skor | Aksiyon |
|---|
| ≥ 8 | Onayla, kapat |
| 6-8 | Uyarı + Jira yorumu, kullanıcıya sor |
| < 6 | Yeni task aç (CC-XX: "Revize: [orijinal task]"), A1'e ver |
PR eksikse:
- 4+ dosya değişmiş ama PR yoksa →
mcp__github__create_pull_request ile retroaktif PR aç
- Commit convention uyumsuzsa → Jira'ya yorum bırak
Jira güncel değilse:
- Transition uygula, yorum ekle
6. Özet
Toplam: N task incelendi
✅ Onaylandı: X
⚠️ Uyarı: Y
🔁 Revize task açıldı: Z
📋 PR retroaktif açıldı: W
Kurallar
- C3 sub-agent'larını paralel çalıştır
- Max 40 tool call (büyük batch'lerde)
- Retroaktif PR açmadan önce kullanıcıya sor — sadece açıkça kötüyse direkt aç
- Revize task'ı orijinal task'a "blocks" ile bağla Jira'da
- Skor 10 üzerinden, yarım puan hassasiyetle
When NOT to Use
- Tek satirlik basit soru/cevap ise
- Skill'in scope'u disindaysa
- Riskli/destructive is ise (ayri onay gerekir)
Red Flags
- Belirsiz hedef/kabul kriteri
- Gerekli dosya/izin/secret eksik
- Ayni adim 2+ kez tekrarlandi
Error Handling
- Gerekli kaynak yoksa → dur, blocker'i raporla
- Komut/akıs hatasi → en yakin guvenli noktadan devam et
- 3 basarisiz deneme → daha uygun skill/agent'a yonlendir
Verification
