fix(oc): replace hex dump of ByteString fields with size summary in gRPC logging

bmc08gt · bmc08gt · commit e751e7271cc3 · 2026-05-04T16:35:42.000-04:00
Large ByteString values (e.g. image payloads) were being fully hex-encoded
and logged, causing excessive memory allocation and log output that degraded
image processing performance. Now logs just the byte count instead.

Signed-off-by: Brandon McAnsh &lt;git@bmcreations.dev&gt;
diff --git a/services/opencode/src/main/kotlin/com/getcode/opencode/utils/logging/LoggingClientCallListener.kt b/services/opencode/src/main/kotlin/com/getcode/opencode/utils/logging/LoggingClientCallListener.kt
@@ -135,7 +135,7 @@ private fun formatFieldValue(fieldName: String, value: Any): String? = when (val
         if (value.serializedSize == 0) null else formatProto(value)
     }
     is ByteString -> {
-        if (value.isEmpty) null else value.toByteArray().toHex().masked()
+        if (value.isEmpty) null else "<${value.size()} bytes>"
     }
     is List<*> -> {
         if (value.isEmpty()) null

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ private fun formatFieldValue(fieldName: String, value: Any): String? = when (val`
`135`	`135`	`if (value.serializedSize == 0) null else formatProto(value)`
`136`	`136`	`}`
`137`	`137`	`is ByteString -> {`
`138`		`- if (value.isEmpty) null else value.toByteArray().toHex().masked()`
	`138`	`+ if (value.isEmpty) null else "<${value.size()} bytes>"`
`139`	`139`	`}`
`140`	`140`	`is List<*> -> {`
`141`	`141`	`if (value.isEmpty()) null`