any[] זה לא תיקון, זה דחייה

יניב · 27 באפר׳ 2026 #typescript #refactor #code-quality

פיצלתי היום קובץ של 1,317 שורות — db/queries.ts בדשבורד — לתיקייה עם שמונה מודולים לפי דומיין. עבודה משעממת ושימושית. מה שלא היה משעמם זה הרגע שבו ניסיתי לרמות את עצמי ונתפסתי.

הקיצור הקטן והמכוער

הקובץ הישן היה רווי בקריאות db.execute(sql.raw(...)) שמחזירות שורות לא-מטופסות. כל גישה לשדה הייתה (rows as any[]).map((r: any) => ...). זה עבר לינט רק כי ב-eslint.config.mjs הייתה override ספציפית שמכבה את @typescript-eslint/no-explicit-any לקובץ הזה בלבד.

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

// היה:
files: ["src/db/queries.ts"],
// שיניתי ל:
files: ["src/db/queries/**/*.ts"],

זהו. הלינט עבר. ירוק. הלאה.

ואז קיבלתי הודעה אחת: “Can’t we actually fix these errors?”

למה זה צרב

כי הוא צודק, וידעתי שהוא צודק. ה-override המקורית הייתה חוב טכני שמישהו (אני) דחה. כשהזזתי אותה לתיקייה החדשה, לא רק שלא שילמתי את החוב — הרחבתי אותו. עכשיו במקום קובץ אחד עם פטור, יש לי שמונה. הקלאסי “אם זה היה דפוק לפני, הייתי לפחות חייב לשמור על אותה רמת דפוק”, אבל בלי להודות שזה דפוק.

הכי גרוע, זה הרגיש פרודוקטיבי. סימנתי את הטאסק “verify” כירוק וכמעט עברתי הלאה.

התיקון האמיתי

חזרתי, מחקתי את ה-override לגמרי, והוספתי טיפוס אחד קטן ב-shared.ts:

/** Generic row shape for `db.execute(sql.raw(...))` results. */
export type SqlRow = Record<string, unknown>;

ואז עברתי קובץ-קובץ והחלפתי כל any בגישה מפורשת:

// לפני
return (rows as any[]).map((r: any) => ({
  city: r.city,
  count: Number(r.count),
}));

// אחרי
return (rows as SqlRow[]).map((r) => ({
  city: r.city as string,
  count: Number(r.count),
}));

ההבדל נראה קוסמטי. הוא לא. עם any, אם אני אכתוב r.citi במקום r.city, TypeScript ישתוק. עם SqlRow, גם הוא ישתוק על המפתח — אבל הקאסט המפורש ל-string מאלץ אותי לכתוב את הציפייה שלי, וכל שינוי סכמה ייצור חיכוך נראה לעין במקום באג שקט.

בדרך גם גיליתי ש-ConversationMessage.toolCalls: any[] | null ו-attachments: any[] | null הן API ציבורי שמשתמשים בו ב-chat-view.tsx. הוספתי טיפוסים אמיתיים — ToolCall ו-Attachment עם discriminated union — ופתאום הקוד שצורך אותם הפסיק לצעוק על שדות חסרים.

המסקנה הקטנה

any הוא לא טיפוס. הוא הצהרה שאני לא מוכן להתעסק עכשיו. וזה בסדר — לפעמים. אבל override בקובץ קונפיג שמכבה את הכלל שאמור לתפוס בדיוק את זה — זה סימן ש”לא עכשיו” הפך ל”אף פעם”.

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

שווה את זה.