נחיל רב-סוכני לסקירת קוד אוטונומית: Planner, Implementer, Tester ו-Critic

הבלוג למפתחים TormentNexus פרסם מדריך מעשי למערכת רב-סוכנית שמבצעת סקירת קוד אוטונומית. המאמר פורסם מחדש ב-Dev.to ב-26 בספטמבר. לפי המחבר, לעוזר AI יחיד חסרים האיזונים והבלמים של צוות הנדסה בוגר: הצעות אינן נבדקות, תיקונים אינם נבחנים לתופעות לוואי, והשלכות העיצוב כמעט ואינן נדונות.

למה סוכן אחד אינו מספיק

סוכן בודד עלול להחזיר קוד תקין תחבירית אך חלש מבחינה ארכיטקטונית. החלופה המוצעת היא לדמות צוות חזק: סוכנים מתמחים שמתקשרים בתוך שיחה מתואמת אחת ומספקים לא רק תשובה, אלא פתרון מאומת ומשופר.

ארבעה תפקידים בנחיל

כל סוכן הוא קריאת LLM נפרדת עם system prompt משלו. ה-Planner משמש כמנהל טכני: הוא מנתח את הקוד וקובע סדר יום לסקירה סביב אבטחה, ביצועים וקריאות. ה-Implementer כותב את התיקון או הריפקטורינג בפועל. ה-Tester מאמת את השינויים בבדיקות יחידה, בדיקות אינטגרציה וסריקות אבטחה. ה-Critic מטיל ספק בהנחות, מערער על ההחלטות הארכיטקטוניות ובודק את יכולת התחזוקה.

דוגמה: חיזוק טוען קונפיגורציה

הדוגמה עוקבת אחר פונקציית Python קצרה שקוראת קובץ YAML ומחזירה את מקטע מסד הנתונים. ה-Planner קובע סדר יום: לנתח סיכוני אבטחה כמו path traversal ואמון בנתונים, לשפר את טיפול השגיאות במפתחות חסרים וב-YAML שגוי, ואז להעריך את העיצוב. ה-Implementer מוסיף בלוקי try/except שמעלים הודעות ValueError ברורות, וה-Tester כותב חבילת pytest שמכסה את נתיבי השגיאה החדשים.

ה-Critic חולק יותר משמעותית משהוא מקבל. הוא מכנה את החזרת config['database'] בלבד אנטי-תבנית שמסתירה את ההקשר המלא של הקונפיגורציה, ומצביע על סיכון TOCTOU: ניתן לשנות את נתיב הקובץ בין הבדיקה לשימוש. המלצתו: להחזיר את כל ה-config ולפתור קישורים סימבוליים עם os.path.realpath לפני פתיחת הקובץ.

קונצנזוס ותזמור

כשה-Implementer וה-Critic חלוקים, ה-Planner משמש כבורר. בדוגמה הוא מורה ל-Implementer להחזיר את ה-config המלא, ול-Tester לעדכן את הבדיקות ולהוסיף בדיקת אבטחה לפתרון קישורים סימבוליים. כל המחזור פועל בצ'אט עם מצב: רשימת הודעות משותפת שכל הסוכנים קוראים ממנה ומוסיפים לה, בעוד מתזמר מרכזי מנהל את סדר התורות לפי הנחיות ה-Planner.

המחבר מפרט את התועלת: ניתוח קוד עמוק יותר, פחות פיקוח אנושי, תיעוד אוטומטי של החלטות עיצוב ביומן הצ'אט, ודרך פשוטה לשלבב ניתוח סטטי וסריקות אבטחה בזרימה. המודל ניתן להרחבה: אפשר להוסיף סוכן DevOps להמלצות על קונטיינרים או סוכן תיעוד שכותב docs מוטמעים.