[refactor] update DebugOverlayNetworkInterceptor (#75)

- make DebugOverlayNetworkInterceptor more robust
- captures headers and content for the detail screen which will be added

Author: Manabu-GT

debugoverlay-core/src/main/kotlin/com/ms/square/debugoverlay/internal/ui/NetworkTabContent.kt

@@ -384,8 +384,8 @@ private fun NetworkStats.augmentNetworkStatsWith(requests: List<NetworkRequest>)
 
   if (this == NetworkStats.UNSUPPORTED) {
     // since it might not have all the requests due to its size capping, this is just approximate.
-    val totalDownloaded = requests.filter { it.responseSize >= 0 }.sumOf { it.responseSize }
-    val totalUploaded = requests.filter { it.requestSize >= 0 }.sumOf { it.requestSize }
+    val totalDownloaded = requests.mapNotNull { it.responseSize }.filter { it >= 0 }.sumOf { it }
+    val totalUploaded = requests.mapNotNull { it.requestSize }.filter { it >= 0 }.sumOf { it }
     return NetworkStats(
       totalDownloaded = totalDownloaded,
       totalUploaded = totalUploaded,
@@ -409,8 +409,8 @@ private const val BYTES_PER_GB = 1024L * 1024L * 1024L
 /**
  * Format bytes to human-readable string.
  */
-private fun formatBytes(bytes: Long): String = when {
-  bytes < 0 -> "—"
+private fun formatBytes(bytes: Long?): String = when {
+  bytes == null || bytes < 0 -> "—"
   bytes < BYTES_PER_KB -> "$bytes B"
   bytes < BYTES_PER_MB -> {
     val kb = bytes / BYTES_PER_KB.toDouble()

debugoverlay-core/src/main/kotlin/com/ms/square/debugoverlay/model/NetworkRequest.kt

@@ -6,15 +6,23 @@ import java.util.UUID
  * Data class for network request.
  */
 public data class NetworkRequest(
+  // Base NetworkRequest fields
   val id: String = UUID.randomUUID().toString(),
+  val protocol: String?, // http/1.1, h2, quic..etc
   val method: HttpMethod, // GET, POST, etc.
   val fullUrl: String, // https://test.com/api/v1/feed
   val shortUrl: String, // /api/v1/feed
   val statusCode: Int?, // 200, 404, etc.
   val durationMs: Long, // 245
-  val responseSize: Long, // bytes
-  val requestSize: Long = 0, // bytes
+  val responseSize: Long?, // bytes
+  val requestSize: Long?, // bytes
   val timestamp: Long = System.currentTimeMillis(),
+  // Extended fields for detail screen
+  val requestHeaders: Map<String, String> = emptyMap(),
+  val responseHeaders: Map<String, String> = emptyMap(),
+  val requestBody: String? = null,
+  val responseBody: String? = null,
+  val error: NetworkError? = null,
 )
 
 public enum class HttpMethod {
@@ -28,3 +36,8 @@ public enum class HttpMethod {
   TRACE,
   UNKNOWN,
 }
+
+/**
+ * Error information for failed requests.
+ */
+public data class NetworkError(val title: String, val message: String, val stackTrace: String? = null)