קוד רוויו זה לא עונש
אחרי שישבתם עם הסוכן וכתבתם איפיון מקיף לפיצ'ר הגיע הרגע הקשה בו צריך לעבור על הקוד שהסוכן כתב, מה שקוראים Code Review. פסיכולוגית, זה השלב שהכי קשה לנו ממספר סיבות:
לא תמיד אנחנו מבינים כל שורה בקוד. גם אנחנו בני אדם ואולי העברנו את רוב החיים בלי לדעת איך עובדים ביטויים רגולאריים.
גם כשאנחנו מבינים כל שורה, לא תמיד אנחנו מבינים את ההגיון בבחירות מסוימות. לפעמים בחירה נראית טפשית והיא באמת טפשית. שינוי החלטות טפשיות של הסוכן, גם עם סוכן AI נותן מעט ערך לאיכות הכללית של הקוד ומבזבז המון זמן.
אם נגלה שהקוד לא עומד בסטנדרטים או לא מתאים לאיפיון שרצינו נצטרך לפרק את הכל ולהתחיל מהתחלה. שוב לחכות שהסוכן יכתוב את כל הקוד, שוב להבין את המבנה שהוא יצר, שוב למצוא את הטעויות הטפשיות והגדולות.
איך אספר לכל האנשים סביבי שרק מדברים על איך AI כותב קוד מעולה שדווקא אצלי זה לא עבד? שדווקא אני לא מצליח לקבל ממנו קוד שפותר את הבעיה בצורה נכונה.
אז אנחנו דוחים, או מתלוננים, או מספרים ש Code Review זה צוואר הבקבוק החדש, או מתמקדים בטעויות הטפשיות. בקיצור מנצנצים.
יותר כדאי להסתכל על העבודה עם ה AI בתור תהליך. אנחנו עדיין מנסים להבין איך זה עובד ומה מצליח. המעבר על הקוד בסוף הוא החלק הכי חשוב ומעניין בתהליך - שם אנחנו לומדים מה עבד, מה לא עבד ומה אפשר לשפר. במקום לפחד מ Code Review אני מציע לנסות את הדברים הבאים:
להאט. כן אנחנו עדיין בלימוד, באימון. עבודה איטית ונכונה מבטיחה שכשנהיה מספיק טובים נוכל להאיץ מהר יותר מהאחרים שסביבנו.
בשביל לקבל קוד AI מפתחים רציניים מעדיפים לתת לו לייצר מימוש כמה פעמים. זה ממש בסדר לזרוק הכל ולהריץ את הפרומפט שוב. כמו להפעיל מדיח שוב אם הכלים לא הכי נקיים.
אם AI השתמש במנגנון שאני לא מבין עליי לבחור - או שאני לומד את זה (חלון נפרד, AI תלמד אותי ביטויים רגולאריים, תכין לי תרגילים מסודרים, מקדיש שעה-שעתיים רק לזה), או להוסיף לפרומפט "אין להשתמש בביטויים רגולאריים" וכך בפעם הבאה ש AI מייצר את הקוד הזה אוכל לקבל רק מנגנונים שאני מכיר.
חפשו אי הבנות ופערים בין המימוש לאיפיון. אני נתקל בהמון כאלה ולכן אני בטוח שגם אתם. כל פער זה רמז שעלינו לחזור לפרומפט, לחדד את האיפיון ולהדגיש את הכוונה שלנו. ממילא תכף נבקש מהסוכן לייצר את הקוד מחדש.
חפשו החלטות נקודתיות לא נכונות - פונקציה שמחזירה מחרוזת ריקה במקום לזרוק Exception כשמשהו לא מסתדר, פונקציית תקשורת בלי timeout, פונקציה שמפעילה תוכנית משורת הפקודה שאולי לא מותקנת בלי לוודא קודם. מכל פער כזה נסו לגזור מסקנה כללית ולהוסיף אותה לפרומפט.
חפשו מבנה פרויקט שלא מתאים לקוד שרציתם. קובץ אחד איפה שהיה צריך להפריד לכמה קבצים, כל הפונקציות במקום אחד במקום פונקציה בכל קובץ, עיצוב ב CSS במקום בטיילווינד (או בלוק style במקום קובץ CSS). כל פער כזה זו הזדמנות - נוסיף אותו לפרומפט או לקובץ פרומפט קבוע כמו AGENTS.md כדי לשפר את כל הקוד העתידי שניצור.
מעבר על הקוד יכול להיות החלק הכי מהנה בעבודה עם ה AI. השינוי הוא בגישה - במקום להסתכל עליו בתור צעד חוסם או צוואר בקבוק שעוצר אותי מלהעלות את הפיצ'ר עלינו להסתכל עליו בתור מדרגה. אנחנו עולים עוד מדרגה בדרך לפרויקט ידידותי ל AI.