review-ops

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 oku
• ccplugin-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

• Beklenen cikti uretildi
• Yan etki yok (dosya/ayar)
• Gerekli log/rapor paylasildi