סקירת קוד לפני שדוחפים: פקודה אחת שרצה ברקע ומחפשת באגים בדיף
מאת ירון דויטשר עודכן לאחרונה:
“סקירת קוד בקלוד קוד” היא שני דברים שונים לגמרי, ומי שקרא על הנושא בעברית פגש בדרך כלל רק את אחד מהם.
הראשון הוא מוצר ארגוני: אפליקציית GitHub שנצמדת למאגר, עוברת על כל Pull Request שנפתח ומשאירה הערות ישירות על השורות שבהן מצאה בעיה. היא בתצוגת מחקר, פתוחה למנויי Team ו-Enterprise בלבד, ומחויבת בנפרד מהמכסה של התוכנית.
השני הוא פקודה, והיא זו שאפשר להריץ בחמש השניות הבאות. /code-review עובדת בתוך כל סשן קלוד קוד — בלי אפליקציית GitHub, בלי הרשאות ארגון ובלי קשר לתוכנית. התיעוד אומר את זה במפורש: בשאר התוכניות עדיין אפשר לסקור דיף מקומית עם הפקודה.
מה היא סוקרת?
מריצים בלי ארגומנטים, והיא לוקחת את מה שיש: הקומיטים שהענף מקדים בהם את ה-upstream, ובנוסף השינויים שטרם נשמרו בעץ העבודה. אם אין אף אחד משניהם, אין לה מה להחזיר.
רוצים משהו אחר? מעבירים מטרה — נתיב לקובץ, מספר PR, שם ענף, או טווח כמו main...my-feature. ויש שני דגלים:
- —fix: מחילה את הממצאים על עץ העבודה במקום רק לדווח עליהם
- —comment: מפרסמת אותם כהערות שורה על ה-PR
הפרט שמשנה את שגרת העבודה: היא רצה ברקע
מגרסה 2.1.218 הסקירה רצה כסוכן משנה ברקע, עם חלון הקשר משלה. שתי תוצאות מעשיות: השיחה שלכם לא מתמלאת במאות שורות דיף, ואפשר להמשיך לעבוד בזמן שהיא עובדת. הממצאים נוחתים בשיחה כשהיא מסיימת.
בשלושה מקרים היא חוזרת לחזית — כשמריצים אותה שוב בזמן שסקירה קודמת עדיין רצה, בהרצה לא אינטראקטיבית (-p או דרך ה-SDK), וכשמשימות רקע מכובות לגמרי.
רמת המאמץ עובדת הפוך ממה שנדמה
הפקודה מקבלת רמת מאמץ, והכיוון שלה אינו אינטואיטיבי. ברמות הנמוכות הסקירה מדווחת רק על מה שהיא בטוחה בו — כלומר פחות התראות שווא. ברמות הגבוהות היא פורשת רשת רחבה יותר, וחלק מהממצאים יהיו כאלה שהיא פחות בטוחה בהם.
מכאן כלל פשוט: לפני דחיפה שגרתית, רמה נמוכה מספיקה ולא תבזבז את הזמן על רעש. לפני מיזוג של שינוי רגיש — מעלים, ומקבלים גם את הרעש.
בלי ארגומנט של מאמץ, הפקודה השתמשה עד היום ברמת המאמץ של הסשן. בגרסה 2.1.223, שיצאה היום (6 באוגוסט 2026), זה השתנה: היא חוזרת לרמה שהקלדתם בפעם הקודמת. באותה גרסה /review הפכה לכינוי של אותה פקודה.
המלכודת: התיקונים בורחים מכפתור הביטול
זה החלק ששווה לקרוא פעמיים. עריכות שסוכן משנה מבצע נוחתות מחוץ לנקודות השמירה של הסשן. כלומר: הרצתם עם —fix, הסקירה נגעה בשבעה קבצים, והתוצאה לא מוצאת חן בעיניכם — /rewind לא יחזיר אתכם. התיעוד מפנה ל-git, וזו באמת הדרך היחידה.
ההבחנה מדויקת: זה נכון לסקירה שרצה ברקע, שהיא ברירת המחדל. סקירה שרצה בחזית עורכת את עץ העבודה בתוך התור שלכם, ולכן שם השחזור עובד כרגיל.
ההרגל שנובע מזה הוא שורה אחת: commit לפני שמריצים עם —fix.
ומה שחשוב לומר ביושר
- היא לא קוראת
REVIEW.md: קובץ הכוונון הזה שייך למוצר הארגוני בלבד. הפקודה המקומית קוראת אתCLAUDE.mdכמו כל סשן אחר, וזה גם המקום להשפיע על מה שיסומן - אי אפשר לתזמן אותה: הפקודה מסומנת כך שהמודל אינו רשאי להפעיל אותה בעצמו. מי שישים אותה כפרומפט של משימה מתוזמנת יגלה שקלוד קורא אותה כטקסט רגיל, ולא מריץ סקירה
- הסקירה העמוקה בענן אינה זמינה לכולם: הווריאנט שמריץ סקירה מעמיקה בענן דורש התחברות עם חשבון claude.ai, ואינו זמין דרך Bedrock, דרך הפלטפורמה של גוגל או לארגונים ללא שמירת נתונים. כשהוא לא זמין, הפקודה פשוט מריצה סקירה מקומית במקומו
- השם התחלף בדרך: עד גרסה 2.1.147 הפקודה נקראה /simplify. היום /simplify היא סקירת ניקיון בלבד, שמסדרת קוד ואינה מחפשת באגים. מי שכתב סקריפט על השם הישן מקבל היום משהו אחר לגמרי
שורה תחתונה
לתת לקלוד בדיקה שהוא יכול להריץ סוגר את הלולאה של מי שכתב את הקוד. סקירה היא הדבר המשלים: מודל שני, בהקשר נקי, שמסתכל על הדיף בלי להיות מאוהב בו.
השילוב הזה הוא שגרה ולא פרויקט — בדיקה שרצה בסוף המשימה, וסקירה שרצה לפני הדחיפה. מי שרוצה להבין לעומק את המנגנון שמתחת לפקודה, סוכני משנה הוא העמוד הנכון; ומי שמסתמך על כפתור הביטול כרשת ביטחון, כדאי שיכיר בדיוק איפה נקודות השמירה נגמרות.