Başlangıç
Başlangıç denetimi ve puan
Kod tabanının P0-P3 sırasıyla taranması ve kritik akışlar için puan.
- Ödeme, giriş, veri yazma yolları
- Testi olmayan kritik yollar
Kod desteği
Ekibin yazdığı kodu okuyup önem sırasıyla bulgu çıkarıyorum; canlıdaki hatayı yeniden üretip kökünü buluyoruz; düzeltmeyi, geri alındığında kırılan bir testle birlikte teslim ediyoruz.
Başkasından devralınan kodda işe, en çok para ve veriye dokunan yoldan başlıyorum.
- P0hemen
- Güvenlik, para, veri kaybı. Yayında kalmamalı; ilk iş bu.
- P1yayından önce
- Yanlış sonuç. Kullanıcıya yanlış rakam ya da yanlış durum gösteriyor.
- P2planlı
- Kırılgan yer. Bugün çalışıyor, ilk beklenmedik durumda düşer.
- P3iyileştirme
- Okunurluk ve tekrar. Kod çalışıyor, ama bir sonraki değişikliği zorlaştırıyor.
06.1İşin üç parçası
Yeşil test tek başına kanıt sayılmaz. Düzeltme geri alındığında kırılmayan test, hatayı yakalamıyor demektir.
aNe yapıyoruz
- Kod incelemeDeğişiklikleri okuyup bulguları P0-P3 sırasıyla, satır numarasıyla yazıyorum.
- Hata ayıklamaCanlıdaki hatayı önce yeniden üretiyoruz, sonra kökünü buluyoruz.
- TestKritik akışlar için hatayı gerçekten yakalayan testler.
- DevralmaBelgesi olmayan kodun haritası: ne nerede, hangi parça neye dokunuyor.
bNe teslim ediyoruz
- Bulgu listesiHer bulgu: yer, sorun, önerilen düzeltme, test.
- Düzeltme dalıAyrı dalda, küçük ve açıklamalı kayıtlarla; birleştirme kararı sizde.
- TestlerDüzeltme geri alınınca kırıldığı gösterilmiş.
- Kod haritasıDevralınan projede tek sayfalık yapı notu.
cNasıl ölçüyoruz
- Kaçan hataCanlıya çıkıp sonradan bulunan hata sayısı, ay ay.
- Düzeltme süresiHatanın bildirilmesinden düzeltmenin yayına çıkmasına kadar geçen süre.
- TekrarAynı türden hatanın yeniden görülme sayısı.
- KapsamKritik akışlarda (ödeme, giriş, veri yazma) testi olan yol oranı.
06.2Bir inceleme nasıl görünür
Kurgusal bir sipariş fonksiyonunda dört bulgu. Satırdaki numaraya ya da sağdaki bulguya tıklayın.
Kod inceleme
19 satırlık kurgusal örnek
siparis.js4 bulgu
export async function siparisToplami(sepet, kupon) {let toplam = 0;for (const urun of sepet.urunler) {toplam += urun.fiyat * urun.adet;}if (kupon) {toplam = toplam - toplam * kupon.oran;}const kargo = await kargoUcreti(sepet.adres);return Math.round(toplam + kargo);}app.post('/api/siparis', async (req, res) => {const { id, kupon } = req.body;const sepet = await db.query(`SELECT * FROM sepet WHERE id = ${id}`);const toplam = await siparisToplami(sepet, kupon);res.json({ toplam });});
-
Sorguya kullanıcı girdisi ekleniyor
P0 satır 15-16
- Sorun
- Sepet kimliği doğrudan SQL metnine yazılıyor; dışarıdan gelen bir değer bütün tabloyu okuyabilir ya da silebilir.
- Düzeltme
- Parametreli sorgu; kimlik sayıya çevrilip doğrulanır.
- Test
- Kimlik yerine SQL parçası gönderen istek 400 dönmeli; düzeltme geri alınınca test kırılmalı.
-
İndirim oranı istemciden geliyor
P0 satır 7, 14
- Sorun
- Kupon nesnesi istekten olduğu gibi alınıyor; oran 1 gönderilirse sipariş bedava olur.
- Düzeltme
- İstemciden yalnız kupon kodu alınır; oran, süre ve kullanım hakkı sunucuda kontrol edilir.
- Test
- Süresi geçmiş ve sahte kuponla sipariş toplamı değişmemeli.
-
Para ondalıklı sayıyla hesaplanıyor
P1 satır 4, 10
- Sorun
- Fiyatlar ondalıklı sayıyla toplanıyor, sonra liraya yuvarlanıyor; kuruşlar kayboluyor.
- Düzeltme
- Bütün tutarlar kuruş cinsinden tam sayı tutulur; yuvarlama tek yerde ve kuralı yazılı.
- Test
- 0,10 + 0,20 gibi tutarlarla toplamın kuruşu kuruşuna doğru olduğu.
-
Kargo servisi hata verirse sipariş düşüyor
P2 satır 9
- Sorun
- Kargo ücreti servisi yanıt vermezse istek 500 ile bitiyor, kullanıcı nedenini görmüyor.
- Düzeltme
- Zaman aşımı ve anlaşılır hata mesajı; olay kaydı.
- Test
- Servis taklidi hata döndüğünde kullanıcıya doğru mesaj gidiyor mu.
06.3İlgili paketler
Kod desteğiyle en çok birlikte yürüyen üç paket.
Güvenlik
Uyum ve risk gözden geçirmesi
Sırların, kişisel verinin ve üçüncü taraf bağımlılıkların gözden geçirilmesi.
- Kodda ve kayıtlarda sır taraması
- Eski ve açığı bilinen bağımlılıklar
Hız
Projeye özel AI agent'lar
Kod agent'ları: kendi dalında yazan, ayrı bir agent'ın incelediği, kararı ekibe bırakan düzen.
Kodun nerede durduğunu ve neyin bozuk olduğunu yaz.
Depo erişimi sonra konuşulur; önce hatanın kısa tarifi ve ne zamandır olduğu yeterli.