co-maintainer,
seni taklit eden bir araç

Altı ay arayla üç pull request, üç ayrı katkıcı. Her seferinde aynı yorum, her seferinde biraz farklı cümlelerle. Kural gerçek ve herkes onu biliyor. Sadece hiçbir yerde bir makinenin okuyabileceği biçimde yazılı değil.

Bir inceleyici yamayı okurken bu kod tabanının dört yılını da kafasında taşır. Bunun ne kadar değerli olduğu şurada görünüyor.

Tipler derlenir, çünkü markalı bir number hâlâ bir number’dır. Ücret kesirli çıkar, ledger.post() tam sayı bekler ve iade, düzgün bölünmeyen ilk tutarda hata verir.

Kaçan bir bulgu çoğunlukla eksik bir girdidir, zayıf bir model değil.

Her incelemede modele bütün repoyu vermek mümkün ama saçma. Repo onun yerine bir kez, incelemenin dışında okunur. Çıkan kılavuzlar da her yamaya götürecek kadar küçüktür.

Bunun bir kez yapılmasının sebebi:

Modeller kiralıktır. Bu çeyrek en iyisi hangisiyse gelecek çeyrek en iyisi o olmayacak. Sahip olduğunuz kısım bağlamdır.

Yüksek sesle okuyun, iki yıldır bu repoya merge eden kişi gibi konuşurlar. Başlık buradan geliyor. Ekibinizden daha iyi bir inceleyici olmaya çalışmıyor, zaten sahip olduğunuz inceleyicinin ucuz bir taklidi olmaya çalışıyor. Gece 2’de bile uyanık, sıkıcı pull request’lerde de.

Bir pull request açılır. Kılavuzlar zaten diskte, bu yüzden inceleyici hiç görmediği bir yamadan değil, okumuş olduğu bir repodan başlar.

Charge providers: percentage refund fee #312 northwind:feat/refund-fee Open

Review / co-maintainer reading guides and diff…2 findings, 1 blocking 24sDetails

co-maintainerbot reviewed just now

Reviewed against SKILL.md, CODEBASE.md and PR_REVIEW_GUIDE.md for this repository, with codegraph enabled.

1 blocking 1 non blocking 9,312 tokens 24.7s $0.0041

src/billing/refund.ts+3−0

3939export function refundFee(amount: Money) {

40 const fee = amount * 0.029;

41 return { fee, net: amount - fee };

4042}

P1 · blocking

refundFee() returns a fractional amount. ledger.post() asserts an integer count of minor units, so every refund whose fee does not divide evenly throws in production instead of rounding. CODEBASE.md puts rate math through mulRate for exactly this reason.

src/billing/refund.ts:41 · 3 call sites reach this

Suggested change

41 const fee = amount * 0.029;

41 const fee = mulRate(amount, 29, 1000);

Commit suggestion Add to batch

Kılavuzlar olmadan burada iki dürüst seçenek var. Hiçbir şey söylememek, çünkü yüzdelik ücret yazılması normal bir şey. Ya da kayan noktayla ilgili belirsiz bir şey söylemek, ki herkes bunu kaydırıp geçmeyi öğrendi.

• Bulgular taşınır. Düzeltmeyi gönderin, yeniden çalıştırın, bulgu iki kez açılmak yerine kapanır.
• Engelleyici bir bulgu sıfırdan farklı kodla çıkar. CI’ın ihtiyacı olan tek şey bu.
• stdout’ta --json, log’lar stderr’de. Bulgular, sayılar, token’lar, süre, codegraph durumu, tek bir belgede.
• Codegraph isteğe bağlı. Varsayılan olarak açık, ikili dosya sizin haberiniz olmadan değil, istek üzerine kurulur.

İnsanların gerçekten istediği yalnızca üç cevap var: çalışma ağacımdaki dağınıklık, şuradaki pull request ve birincisi, ama her dizüstünü init için ödeme yapmaya zorlamadan.

Biri pazartesi günü ekibe katılır ve repoyu klonlar. Yerel inceleme kılavuz ister, kılavuz init ister, init ise dakikalar ve para ister. Bunu dizüstü başına bir kez ödemek, bir aracın üçüncü haftasında ölme biçimidir.

Sıkıcı güvenlik kısımları. Token’lar diskte hash’lenir ve birini devre dışı bırakmak, onunla süren her incelemeyi iptal eder. CLI, bir host’a ilk kez diff yüklediğinde tek seferlik bir bildirim yazdırır, böylece kimse bunu altı ay sonra kaynağı okuyarak öğrenmez.

co-maintainer serve tek bir süreç ve tek bir app.db. GitHub App webhook’unu, bir job worker’ı, panoyu ve son bölümdeki uzak inceleme API’sini barındırır.

localhost:5000

Bir repo ekleyin, init izleyebileceğiniz bir iş olarak çalışır. Otomatik incelemeyi açın, bir pull request’in açılması bir inceleme başlatmaya yeter. Yukarıdaki repo 8 gündür bağlı. init, remake ve tüm incelemeler dahil toplam maliyeti $1.20 oldu.

Kendi repolarımdan birinde otuz pull request. On dördü bilerek eklenmiş, elle doğrulanmış bir hata taşıyor. On altısı temiz kontrol, ki çoğu inceleme benchmark’ı bu kısmı atlar ve precision’ı ölçülebilir kılan da bu kısımdır.

Diff dışında bir şeye ihtiyaç duyan her hata yakalandı. İki kaçırma, gereken her şeyin zaten yamada durduğu eksenlerde, ki bu komik ve biraz da alçaltıcı: bağlam mekanizması çalışıyor, hâlâ kaçan şey sıradan yakın mesafe okuma.

Otuz pull request, tekrar koşu yok, yani varyans ölçülmedi. Temiz pull request başına yaklaşık bir gereksiz bulgu üretiyor ve bir ekibin inceleme botunu tutup tutmayacağını ya da susturup susturmayacağını belirleyen sayı bu.

Soru hiçbir zaman hangi modelin en iyi incelediği değildi. Modeller birkaç ayda bir değişir ve cevap onlarla birlikte eskir. Soru, modelin başlarken elinde ne tuttuğudur. Daha fazla context vermek daha doğru tahmin etmesini sağlamaz.

1. 01daha az context
2. 02daha az halüsinasyon
3. 03daha hızlı inceleme
4. 04daha doğru tahmin