‏تیم Claude Code چطوری کد رو ریویو می‌کنه

تیم Claude Code می‌گه حدود ۹۰ درصد کد خود Claude Code رو Claude می‌نویسه، پس ریویوی دستی خط به خط دیگه جواب نمی‌ده. هدفشون اینه که بیشتر PRها با حداکثر یه آدم تو چرخه merge بشن (که می‌تونه خود نویسنده باشه)، بدون اینکه revert و حادثه بیشتر بشه. به گفته‌ی نویسنده‌ها، از ژوئن تا سپتامبر PRهای merge‌شده بیش از دو برابر شد و سهم revertها بیشتر از ۴۰ درصد پایین اومد.

ریویو حالا قبل از ساخته شدن PR شروع می‌شه، چیزی شبیه spellcheck موقع نوشتن. به Claude یه چیز قابل اجرا مثل تست یا build می‌دن تا خروجی رو نشون بده، بعد یه subagent با context تازه diff رو بررسی می‌کنه. دو فایل هم کمک می‌کنن: CLAUDE.md که پروژه و قراردادهاش رو توضیح می‌ده، و REVIEW.md که صاحبای هر بخش از کد می‌نویسن تا بگن ریویوکننده باید دنبال چی باشه.

روی هر PR چند ایجنت هم‌زمان دنبال باگ می‌گردن و یه گروه دیگه روی هر یافته رأی می‌ده. فقط اونایی که بیشتر قبولشون دارن پست می‌شن. با این ابزار، سهم PRهایی که کامنت جدی می‌گیرن از ۱۶ به ۵۴ درصد رسید. هر یافته باید یا درست بشه یا با دلیل بسته بشه. یه ریویوی امنیتی خودکار هم هست و تا یافته‌ای باز باشه، PR merge نمی‌شه.

تعداد تأیید انسانی رو ریسک خود تغییر تعیین می‌کنه، نه نویسنده. تغییرهای کم‌ریسک مثل مستندات بدون تأیید آدم رد می‌شن، تغییرهای معمولی یه نفر و حساس‌ترین بخش‌ها دو نفر لازم دارن. یه classifier ریسک رو تشخیص می‌ده. تأیید یه ایجنت هیچ‌وقت جای آدم رو نمی‌گیره و هر کسی که ایجنت رو هدایت کرده، مسئول اون تغییره. بعدش merge queue، feature flag، انتشار اول برای کارمندای Anthropic و rollback سریع میاد.

یه مشکل جالب این بود که ایجنت نویسنده و ایجنت ریویوکننده می‌تونن تا ابد هم رو مشغول کنن. یه PR بیشتر از پنجاه دور ریویو و اصلاح خورد. حدود ۵۹ درصد چیزایی که ریویوکننده مدام گیر می‌داد، کد قدیمی بود. برای همین سقف سه دور گذاشتن، بعد از چند دور فقط diff جدید بررسی می‌شه و یافته‌ی تکراری هم دوباره پست نمی‌شه. حادثه‌های قبلی هم به یه مجموعه‌ی eval تبدیل شدن که ریویوکننده هر روز روش اجرا می‌شه.

به گفته‌ی تیم، کار آدم‌ها حالا تصمیم‌هاییه که ماشین توشون ضعیفه: اینکه اصلاً یه چیزی ساخته بشه یا نه، کدهای پرریسک مثل billing و auto-updater، و معماری. برای اینکه اعضای تیم کد رو بشناسن، از Claude می‌خوان برای همون بخشی که لازم دارن یه توضیح بنویسه. قانونشون هم اینه که هر کی تغییری رو land کرد، اگه خراب شد خودش درستش می‌کنه، ولی این به معنی مقصر دونستن نیست.

نکات کلیدی:

• از ژوئن تا سپتامبر PRهای merge‌شده بیش از ۲ برابر و نرخ revert بیش از ۴۰ درصد کمتر شد
• سهم PRهای دارای کامنت جدی با ریویوی AI از ۱۶ به ۵۴ درصد رسید
• سقف سه دور ریویو، بعدش فقط diff جدید بررسی می‌شه
• تأیید ایجنت هیچ‌وقت جای تأیید انسانی رو نمی‌گیره
• Code Review for Claude Code روی پلن‌های Team و Enterprise در دسترسه و /code-review نسخه‌ی سبک‌ترش رو لوکال اجرا می‌کنه