fix: optimize case listing and polish test run pages

Agent-Logs-Url: https://github.com/bg-playground/BGSTM/sessions/294d8d86-ccd0-403f-8efe-8eea70b4325f

Co-authored-by: bg-playground <259109604+bg-playground@users.noreply.github.com>

Author: Copilot

backend/app/crud/external_case_results.py

@@ -1,6 +1,7 @@
 """CRUD operations for External Case Results (BGSTM#303)."""
 
 import uuid
+from collections import defaultdict
 from typing import Any
 from uuid import UUID
 
@@ -367,6 +368,16 @@ async def list_case_results_for_session(
     count_stmt = select(func.count()).select_from(ExternalCaseResult).where(ExternalCaseResult.session_id == session_id)
     total = (await db.execute(count_stmt)).scalar_one()
     rows = (await db.execute(stmt)).scalars().all()
+    requirement_ids_by_test_case_id: dict[UUID, list[UUID]] = defaultdict(list)
+    linked_test_case_ids = [row.test_case_id for row in rows if row.test_case_id is not None]
+    if linked_test_case_ids:
+        requirement_rows = await db.execute(
+            select(RequirementTestCaseLink.test_case_id, RequirementTestCaseLink.requirement_id)
+            .where(RequirementTestCaseLink.test_case_id.in_(linked_test_case_ids))
+            .order_by(RequirementTestCaseLink.created_at.asc())
+        )
+        for test_case_id, requirement_id in requirement_rows.all():
+            requirement_ids_by_test_case_id[test_case_id].append(requirement_id)
     for row in rows:
-        row.requirement_ids = await _get_requirement_ids_for_test_case(db, test_case_id=row.test_case_id)
+        row.requirement_ids = requirement_ids_by_test_case_id.get(row.test_case_id, []) if row.test_case_id else []
     return list(rows), total

backend/app/schemas/external_results.py

@@ -95,7 +95,7 @@ class SessionResponse(BaseModel):
     git_branch: str | None = None
     ci_url: HttpUrl | None = None
     metadata: dict[str, Any] = Field(default_factory=dict)
-    summary: dict[str, Any] | None = None
+    summary: dict[str, Any] = Field(default_factory=dict)
 
 
 _TERMINAL_STATUSES = {RunStatus.passed, RunStatus.failed, RunStatus.aborted}

frontend/src/pages/TestRunDetailPage.tsx

@@ -52,12 +52,20 @@ const TestRunDetailPage: React.FC = () => {
     }
     try {
       setLoading(true);
-      const [sessionRes, casesRes] = await Promise.all([
-        externalResultsApi.getSession(sessionId),
-        externalResultsApi.listSessionCases(sessionId, { skip: 0, limit: 500 }),
-      ]);
+      const sessionPromise = externalResultsApi.getSession(sessionId);
+      const allCases: CaseResult[] = [];
+      let skip = 0;
+      const limit = 500;
+      let total = 0;
+      do {
+        const caseRes = await externalResultsApi.listSessionCases(sessionId, { skip, limit });
+        allCases.push(...caseRes.data.cases);
+        total = caseRes.data.total;
+        skip += caseRes.data.cases.length;
+      } while (skip < total);
+      const sessionRes = await sessionPromise;
       setSession(sessionRes.data);
-      setCases(casesRes.data.cases);
+      setCases(allCases);
     } catch (error) {
       console.error('Failed to load test run details:', error);
       showToast('Failed to load test run details', 'error');

frontend/src/pages/TestRunsPage.tsx

@@ -79,16 +79,18 @@ export const TestRunsPage: React.FC = () => {
     await loadSessions();
   }, [loadSessions]);
 
+  const handleStatusFilterChange = useCallback((event: React.ChangeEvent<HTMLSelectElement>) => {
+    setStatusFilter(event.target.value as StatusFilter);
+    setPage(1);
+  }, []);
+
   return (
     <div className="container mx-auto px-4 py-8">
       <div className="flex items-center justify-between mb-6">
         <h1 className="text-3xl font-bold text-gray-900">Test Runs</h1>
         <select
           value={statusFilter}
-          onChange={(e) => {
-            setStatusFilter(e.target.value as StatusFilter);
-            setPage(1);
-          }}
+          onChange={handleStatusFilterChange}
           className="px-3 py-2 border border-gray-300 rounded-md text-sm focus:ring-primary-500 focus:border-primary-500"
         >
           <option value="all">All</option>