📘 תיעוד 34

פתרון בעיות בסקירת קוד

תקנו סקירת קוד שלא התחילה, נכשלה, או שדולגה בגלל תקציב, מצאו ממצאים חסרים, ופתרו בעיות הגדרה ובעיות ב-REVIEW.md.

כשסקירת קוד לא עושה מה שציפיתם בבקשת משיכה, התחילו בהרצת הבדיקה Claude Code Review. הכותרת שלה אומרת עד איפה הסקירה הגיעה.

#בדקו עד איפה הסקירה הגיעה

מצאו את הרצת הבדיקה Claude Code Review בבקשת המשיכה, ואז התאימו את הכותרת שלה לאחת מהכותרות הנפוצות האלה:

#סקירה לא התחילה

בדקו את המקרים האלה כשלא מופיעה הרצת בדיקה Claude Code Review, או כש-Claude מפרסם תגובה שאומרת שהסקירה דולגה.

#פקודת תגובה לא התחילה סקירה

אחרי שהגבתם @claude review, חפשו בבקשת המשיכה הרצת בדיקה Claude Code Review שכותרתה Code review queued... או Code review in progress.... תגובת 👀 על התגובה שלכם לא מאשרת שסקירה בדרך. אם לא מופיעה הרצת בדיקה, בדקו כל אחת מהסיבות האלה:

• סקירת קוד לא מופעלת במאגר הזה: Claude לא מפרסם תגובה ולא הרצת בדיקה בתגובה לפקודה. בקשו מ-Owner להוסיף את המאגר.
• התגובה לא עומדת בתנאי: זו הייתה הערה בשורה או גוף סקירה ולא תגובה חדשה ברמה העליונה, או שהפקודה לא הייתה בתחילת התגובה. גם עריכה של תגובה קודמת במקום פרסום תגובה חדשה לא נספרת, ובקשת המשיכה חייבת להיות פתוחה. ראו את התנאים לפקודות תגובה ופרסמו תגובה חדשה שעומדת בהם.
• אין לכם הרשאת כתיבה: אתם צריכים הרשאת write, maintain או admin במאגר. בבקשת משיכה מ-fork, ההרשאה שקובעת היא במאגר הבסיס. אלא אם בקשת המשיכה שלכם, Claude יכול להשיב עם טיפ: "No review was started: manual reviews can only be requested by someone with write access to this repository." בקשו ממישהו עם הרשאת כתיבה להגיב במקומכם.
• התקציב נגמר: Claude מפרסם תגובה שמתחילה ב-"⚠️ Code review skipped". ראו הסקירה לא רצה וה-PR מציג הודעת תקציב.

#הסקירה לא רצה וה-PR מציג הודעת תקציב

כשהתקציב של הארגון לא יכול לכסות סקירה, סקירת קוד מדלגת על הסקירה ו-Claude מפרסם בבקשת המשיכה תגובה שמתחילה ב-"⚠️ Code review skipped". בחלק מהמקרים האלה מופיעה גם הרצת בדיקה שכותרתה Code review declined.

התגובה מקשרת אל claude.ai/admin-settings/usage, שם מנהל משחזר את התקציב. הניסוח שלה אומר איזו מגבלה הושגה:

• "has reached its monthly code review spending cap": הושגה מגבלת ההוצאה החודשית לסקירת קוד. הסקירות מתחדשות בתחילת תקופת החיוב הבאה. כדי לקבל אותן מוקדם יותר, בקשו ממנהל להעלות את המגבלה.
• "has no extra usage available to pay for this review": יתרת זיכויי השימוש של הארגון ריקה, או שהושגה מגבלת ההוצאה החודשית הכוללת שלו. בקשו ממנהל להוסיף זיכויי שימוש או להעלות את המגבלה הזאת.
• "is too low to start another review": מה שנשאר מהתקציב או מהיתרה נמוך מהעלות הצפויה של סקירה אחת. בקשו ממנהל להעלות את המגבלה או להוסיף זיכויי שימוש.

אחרי שהתקציב שוחזר, הגיבו @claude review בבקשת המשיכה כדי לקבל את הסקירה שדולגה. התנאים לפקודות תגובה חלים, ולכן מי שמגיב צריך הרשאת כתיבה.

#הפעלה מחדש של סקירה שנכשלה או שנגמר לה הזמן

אם הסיכום של הרצת הבדיקה אומר "The review session was interrupted before it could finish. A new review of this commit has been queued automatically.", אין צורך לעשות דבר. הסקירה הזאת מנוסה שוב פעם אחת באופן אוטומטי.

אחרת, כשכותרת הרצת הבדיקה היא Code review failed או Code review timed out, הריצו את הסקירה שוב באמצעות תגובה או הרצה מחדש:

• הגיבו @claude review: פרסמו את זה כתגובה ברמה העליונה בבקשת המשיכה. זה מתחיל סקירה חדשה בלי לרשום את בקשת המשיכה ל-push עתידיים.
• לחצו על Re-run: אם בקשת המשיכה אינה מ-fork, לחצו על Re-run בבדיקה Claude Code Review בלשונית Checks ב-GitHub. ההרצה מחדש מכסה את ה-commit הנוכחי בראש בקשת המשיכה, וגם היא לא רושמת את בקשת המשיכה.

#מצאו בעיות שלא מופיעות כהערות בשורה

אם כותרת הרצת הבדיקה אומרת שנמצאו בעיות אבל אינכם רואים הערות סקירה בשורה על ה-diff, חפשו במקומות האחרים האלה שבהם ממצאים מוצגים:

• Details של הרצת הבדיקה: לחצו על Details ליד הבדיקה Claude Code Review בלשונית Checks. טבלת החומרה מפרטת ממצאים עם הקובץ, השורה והסיכום שלהם, בין אם ההערה בשורה התקבלה ובין אם לא.
• הערות ב-Files changed: פתחו את הלשונית Files changed ב-PR. ממצא שיש לו שורה אבל לא פורסם כהערה בשורה מופיע שם כהערה על השורות הנוגעות בדבר.
• גוף הסקירה: ממצאים שאין להם מיקום בשורה, או שנמצאים מחוץ ל-diff הנוכחי, מופיעים תחת כותרת Additional findings בגוף הסקירה.

#הרצת הבדיקה מזכירה את REVIEW.md

אלה ארבע מההודעות שהרצת בדיקה יכולה לשאת על REVIEW.md:

• "This review applied the repository's review instructions (REVIEW.md) from its default branch, which is not protected against direct pushes.": הסקירה קראה רק את קובץ השורש. ראו מאיזה ענף נקרא REVIEW.md כדי להבין מה משתנה כשמגנים על הענף.
• "were not applied to this review: branch protection could not be confirmed": הסקירה לא קראה שום REVIEW.md.
• "they link to a file this pull request modifies, or carry more links than the review checks": הסקירה דילגה על REVIEW.md בשורש, או על מספר קבצי REVIEW.md בתת תיקיות שההודעה מציינת. בדקו את הקובץ שדולג לכל סיבה. עבור REVIEW.md בשורש, בדקו גם את הקבצים שהוא כולל:
  • יותר מדי קישורים: יש בו יותר מ-64 קישורי Markdown. צמצמו קישורים ל-64 או פחות.
  • קישור שהסקירה לא יכולה לפענח: יש בו קישור Markdown לא תקין. תקנו או הסירו את הקישור.
  • קישור לקובץ שהשתנה: הוא מקשר לקובץ או לתיקייה שבקשת המשיכה הזאת משנה, והסקירה לא יכלה להשאיר בחוץ רק את השורות עם הקישור הזה. כתבו את הכלל בתוך REVIEW.md עצמו.
• "include line(s) in the root REVIEW.md were skipped on this review": REVIEW.md בשורש שלכם הוחל, אבל הקבצים שהשורות האלה מציינות לא הוחלו. בדקו את הרשימות תחת כלול קבצים אחרים ב-REVIEW.md, ועבור skill, תחת כלול קובץ skill.

#בעיות הגדרה

המקרים האלה עולים אצל Owner בזמן הגדרת סקירת קוד.

#מאגר לא מופיע ב-Add a repository

חפשו קודם את המאגר לפי שם, כי תיבת הדו שיח לא טוענת רשימה ארוכה במלואה. תיבת הדו שיח מציגה מאגר רק אם אפליקציית Claude GitHub App יכולה לגשת אליו. לחצו על Install GitHub App ותנו לאפליקציה גישה למאגר. כשאין מאגרים ברשימה, Install GitHub App הוא כפתור בתיבת הדו שיח. אחרת, הוא בתחתית תפריט הסינון של ארגון GitHub בתיבת הדו שיח.

#לא מופיעה הרצת בדיקה בבקשת המשיכה לבדיקה

אשרו ש-Allow code review מופעל, שהמאגר מופיע תחת Repositories במקטע Code review, ושל-Claude GitHub App יש גישה אליו. סקירות אוטומטיות מדלגות על בקשות משיכה בטיוטה, ומאגר שמוגדר ל-Manual (@claude review) צריך תגובת @claude review כדי להתחיל את הסקירה הראשונה.

#משאבים קשורים

• קריאה ובקשה של סקירות בבקשת משיכה: מה אומרים הממצאים והרצת הבדיקה, ופקודות התגובה
• הגדרת סקירת קוד: ל-Owners, הפעלת סקירת קוד וניהול מאגרים
• עלויות ושימוש של סקירת קוד: מגבלות ההוצאה שמאחורי הודעת תקציב
• התאמה אישית של סקירת קוד: איזה עותק של REVIEW.md סקירה קוראת