feat(twitch-bot): add random raid system

[barbequeorbarbecue]

No description provided.

[tw4]

📊 PR Review - Random Raid System

Merhaba! PR'ı inceledim. Genel olarak iyi yapılandırılmış bir feature ama merge öncesi düzeltilmesi gereken kritik bir bug ve birkaç iyileştirme noktası var.

---

✅ Güçlü Yönler

1. İyi Tasarlanmış Sistem:

   • Community yönetimi (topluluk oluşturma/üye ekleme)
   • Subscription mekanizması
   • Random raid özelliği
   • Veritabanı modelleri düzgün yapılandırılmış

2. Kullanışlı Komutlar: !addcm, !subc, !rr, !unrr, etc.

---

❌ KRİTİK BUG

apps/twitch-bot/internal/service/service.go:418

func (s *service) GetRandomLiveStreamer(ctx context.Context, communityId uint) (*string, error) {
    // ... kod ...
    
    // Return random live streamer
    rand.New(rand.NewSource(time.Now().UnixNano()))
    streamerId := liveStreamers[rand.Intn(len(liveStreamers))]
    log.Println("[service.GetRandomLiveStreamer] Streamer ID:", streamerId)

    return nil, nil  // ❌ BUG: Her zaman nil dönüyor!
}

Düzeltme:

return &streamerId, nil  // ✅ Bulunan streamer ID'sini döndürmeli

Bu bug yüzünden !rr komutu hiçbir zaman çalışmayacak.

---

🧹 Temizlik Gereken Kodlar

1. Debug kodları silinmeli (db/postgresql/postgresql.go:39-41):

//db.Where("1 = 1").Delete(&model.TwitchCommunityMember{})
//db.Where("1 = 1").Delete(&model.TwitchCommunity{})
//db.Where("1 = 1").Delete(&model.TwitchCommunitySubscription{})

2. Yorumlanmış kodlar (apps/twitch-bot/internal/command/command.go:78-87):

//addcm: c.AddCommunityCommand,
//dcm:   c.DeleteCommunityCommand,
// ... 

Bu kodlar kullanılacak mı yoksa silinmeli mi?

3. TODO yorumu (apps/twitch-bot/internal/command/command.go:68):

// REFACTOR THESE COMMANDS' NAMES

Komut isimleri belirlenmeli (addcm, subc, rr vs. daha açıklayıcı olabilir)

---

⚠️ İyileştirme Önerileri

1. Error Handling İyileştirilebilir:

apps/twitch-bot/internal/command/community_members.go:48-50:

userInfo, err := c.twitchService.GetUserInfoById(member.MemberChannelID)
if err != nil {
    continue  // ⚠️ Error sessizce ignore ediliyor
}

2. Transaction Eksikliği:

apps/twitch-bot/internal/service/service.go:339-345 - Community oluşturma + subscription işlemleri atomik değil. Birisi başarısız olursa inconsistent state kalabilir.

3. Yorumlanmış Locale Kodu:

apps/twitch-bot/internal/command/helpers/helpers.go:104-111 - Bu kod kullanılacak mı?

---

📋 Merge Öncesi Checklist

• KRİTİK: GetRandomLiveStreamer bug'ını düzelt
• Debug kodlarını temizle
• Yorumlanmış kodları sil veya kullan
• Komut isimlerini netleştir (veya TODO'yu kaldır)
• Error handling iyileştir
• (Opsiyonel) Transaction ekle
• (Önerilen) Unit testler ekle

---

Bu düzeltmeler yapıldıktan sonra PR merge'e hazır olur! 🚀

[tw4]

🐛 Specific Line Comments

Critical Bug - service.go:418
GetRandomLiveStreamer fonksiyonu canlı yayıncı bulsa bile nil dönüyor. Bu yüzden !rr komutu asla çalışmayacak.

Debug Code - postgresql.go:39-41
Commented-out delete statements should be removed before merge.

Incomplete Code - command.go:68
"REFACTOR THESE COMMANDS' NAMES" - Komut isimleri belirlenmeli veya bu TODO silinmeli.

Error Handling - community_members.go:48-50
GetUserInfoById error'u ignore ediliyor. En azından loglanmalı.

Bu noktalar düzeltilirse PR approve edilebilir! 👍