fix: coderabbitai comments

Author: bradjin8

services/workspace_listing.py

@@ -134,7 +134,11 @@ def _safe_fetchall(query: str, params: tuple = ()) -> list:
                             e,
                         )
             except Exception as e:
-                _logger.error("Failed to load composer rows from global storage: %s", e)
+                _logger.error(
+                    "Failed to load composer rows from global storage: %s",
+                    e,
+                    exc_info=True,
+                )
 
     # Group workspace entries by normalized folder path
     folder_to_entries: dict[str, list] = {}

services/workspace_tabs.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import hashlib
 import json
 import logging
 import os
@@ -46,6 +47,17 @@ def _try_loads_kv_value(raw: str | None) -> Any | None:
         return None
 
 
+def _kv_payload_log_meta(value: object | None) -> tuple[int, str | None]:
+    """Byte length and short SHA-256 prefix for logs without emitting raw KV payloads."""
+    if value is None:
+        return 0, None
+    if isinstance(value, bytes):
+        payload = value
+    else:
+        payload = str(value).encode("utf-8", errors="replace")
+    return len(payload), hashlib.sha256(payload).hexdigest()[:12]
+
+
 def assemble_workspace_tabs(
     workspace_id: str,
     workspace_path: str,
@@ -123,11 +135,13 @@ def _safe_fetchall(query: str, params: tuple = ()) -> list:
                 try:
                     parsed = json.loads(row["value"])
                 except json.JSONDecodeError as e:
+                    payload_len, payload_fp = _kv_payload_log_meta(row["value"])
                     _logger.warning(
-                        "Failed to decode Bubble from %s: %s (value: %r)",
+                        "Failed to decode Bubble from %s: %s (payload_len=%d, payload_sha256=%s)",
                         row["key"],
                         e,
-                        row["value"],
+                        payload_len,
+                        payload_fp,
                     )
                     continue
                 try:
@@ -207,11 +221,14 @@ def _safe_fetchall(query: str, params: tuple = ()) -> list:
             try:
                 parsed = json.loads(row["value"])
             except json.JSONDecodeError as e:
+                payload_len, payload_fp = _kv_payload_log_meta(row["value"])
                 _logger.warning(
-                    "Failed to decode Composer from composerData:%s: %s (value: %r)",
+                    "Failed to decode Composer from composerData:%s: %s (key=%s, payload_len=%d, payload_sha256=%s)",
                     composer_id,
                     e,
-                    row["value"],
+                    row["key"],
+                    payload_len,
+                    payload_fp,
                 )
                 continue
             try: