fix: add pr_title to CSV schema and fix modal JS crash

- Add pr_title column to CSV_FIELDNAMES and CSVBatchWriter.add_row()
- Pass result["title"] from GitHub API into all three CSV write sites in batch.py
- Use real pr_title in _build_team_dev_prs with fallback to URL-derived title
- Fix forEach crash on non-array chartData values (leaderboard is an object)
  which also blocked the modal X button from registering its event listener

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: OhadPerryBoomi

cli/batch.py

@@ -949,6 +949,7 @@ def _write_row_from_result(pr_url_result: str, result: Dict[str, Any]) -> None:
             lines_added=lines_added,
             lines_deleted=lines_deleted,
             approved_by=approved_by,
+            pr_title=result.get("title", ""),
         )
 
     try:
@@ -1378,6 +1379,7 @@ def process_single_pr(
                             lines_added=lines_added,
                             lines_deleted=lines_deleted,
                             source=result.get("source"),
+                            pr_title=result.get("title", ""),
                         )
 
                     if label_applied:
@@ -1443,6 +1445,7 @@ def process_single_pr(
                                     lines_added=lines_added,
                                     lines_deleted=lines_deleted,
                                     source=result.get("source"),
+                                    pr_title=result.get("title", ""),
                                 )
 
                             with completed_lock:

cli/csv_handler.py

@@ -22,6 +22,7 @@
     "explanation",
     "source",
     "approved_by",
+    "pr_title",
 ]
 
 
@@ -92,6 +93,7 @@ def add_row(
         lines_deleted: Optional[int] = None,
         source: Optional[str] = None,
         approved_by: Optional[str] = None,
+        pr_title: Optional[str] = None,
     ) -> None:
         """
         Add a row to the buffer, flush if batch size reached.
@@ -129,6 +131,7 @@ def add_row(
                 "explanation": explanation,
                 "source": source,
                 "approved_by": approved_by or "",
+                "pr_title": pr_title or "",
             }
             # Ensure all fieldnames present
             for fn in self._fieldnames:

reports/chart_data.py

@@ -87,8 +87,9 @@ def _build_team_dev_prs(df: pd.DataFrame) -> Dict[str, Any]:
         else:
             merged_at = ""
 
+        pr_title = str(row.get("pr_title", "") or "").strip()
         pr_dict = {
-            "title": _pr_title_from_url(pr_url),
+            "title": pr_title if pr_title else _pr_title_from_url(pr_url),
             "url": pr_url,
             "complexity": float(row.get("complexity", 0) or 0),
             "merged_at": merged_at,

reports/interactive_report.py

@@ -1805,7 +1805,9 @@ def build_interactive_report(
     const allChartEntries = [];
 
     tabOrder.forEach(key => {{
-      (chartData[key] || []).forEach((c, idx) => {{
+      const _tabVal = chartData[key];
+      if (!Array.isArray(_tabVal)) return;
+      _tabVal.forEach((c, idx) => {{
         allChartEntries.push({{ tab: key, idx, data: c, title: c.title || '', subtitle: c.subtitle || '' }});
       }});
     }});