fix(ui): handle missing block log metrics gracefully (#168)

## Summary
- Make `timing`, `throughput`, `state_reads`, `state_writes`, and
`cache` optional on `BlockLogEntry` type
- **BlockLogDetails**: conditionally render timing breakdown, overhead
pie, cache chart, and state ops sections only when their data exists.
Metric cards show "N/A" instead of misleading zeros
- **BlockLogsComparison / TestComparisonTable / useProcessedData**: use
optional chaining (`?.`) with `?? 0` fallbacks for safe metric
extraction

## Test plan
- [x] View a run with complete block log data — all sections render as
before
- [x] View a run with partial block log data (e.g. missing
`state_read_ms`) — missing metrics show "N/A", sections without data are
hidden
- [x] Compare page still works with partial block log data
- [x] Block logs dashboard table/charts handle partial data without
crashes

Author: skylenet

ui/src/api/types.ts

@@ -491,11 +491,11 @@ export interface BlockLogCache {
 
 export interface BlockLogEntry {
   block: BlockLogBlock
-  timing: BlockLogTiming
-  throughput: BlockLogThroughput
-  state_reads: BlockLogStateReads
-  state_writes: BlockLogStateWrites
-  cache: BlockLogCache
+  timing?: BlockLogTiming
+  throughput?: BlockLogThroughput
+  state_reads?: BlockLogStateReads
+  state_writes?: BlockLogStateWrites
+  cache?: BlockLogCache
 }
 
 export type BlockLogs = Record<string, BlockLogEntry>

ui/src/components/compare/BlockLogsComparison.tsx

@@ -69,12 +69,12 @@ function buildBlockLogDataPoints(
     points.push({
       testIndex: index + 1,
       testName,
-      throughput: entry.throughput.mgas_per_sec,
-      executionMs: entry.timing.execution_ms,
-      overheadMs: entry.timing.state_read_ms + entry.timing.state_hash_ms + entry.timing.commit_ms,
-      accountCacheHitRate: entry.cache.account.hit_rate,
-      storageCacheHitRate: entry.cache.storage.hit_rate,
-      codeCacheHitRate: entry.cache.code.hit_rate,
+      throughput: entry.throughput?.mgas_per_sec ?? 0,
+      executionMs: entry.timing?.execution_ms ?? 0,
+      overheadMs: (entry.timing?.state_read_ms ?? 0) + (entry.timing?.state_hash_ms ?? 0) + (entry.timing?.commit_ms ?? 0),
+      accountCacheHitRate: entry.cache?.account?.hit_rate ?? 0,
+      storageCacheHitRate: entry.cache?.storage?.hit_rate ?? 0,
+      codeCacheHitRate: entry.cache?.code?.hit_rate ?? 0,
     })
   })
   return points

ui/src/components/compare/TestComparisonTable.tsx

@@ -50,12 +50,12 @@ const BLOCK_LOG_METRICS: MetricTab[] = [
 
 function extractBlockLogMetric(entry: BlockLogEntry, metricId: string): number {
   switch (metricId) {
-    case 'bl-throughput': return entry.throughput.mgas_per_sec
-    case 'bl-execution': return entry.timing.execution_ms
-    case 'bl-overhead': return entry.timing.state_read_ms + entry.timing.state_hash_ms + entry.timing.commit_ms
-    case 'bl-account-cache': return entry.cache.account.hit_rate
-    case 'bl-storage-cache': return entry.cache.storage.hit_rate
-    case 'bl-code-cache': return entry.cache.code.hit_rate
+    case 'bl-throughput': return entry.throughput?.mgas_per_sec ?? 0
+    case 'bl-execution': return entry.timing?.execution_ms ?? 0
+    case 'bl-overhead': return (entry.timing?.state_read_ms ?? 0) + (entry.timing?.state_hash_ms ?? 0) + (entry.timing?.commit_ms ?? 0)
+    case 'bl-account-cache': return entry.cache?.account?.hit_rate ?? 0
+    case 'bl-storage-cache': return entry.cache?.storage?.hit_rate ?? 0
+    case 'bl-code-cache': return entry.cache?.code?.hit_rate ?? 0
     default: return 0
   }
 }