fix: Log NULL KV rows instead of silently skipping them.

Author: bradjin8

services/workspace_tabs.py

@@ -115,6 +115,10 @@ def _safe_fetchall(query: str, params: tuple = ()) -> list:
             if len(parts) >= 3:
                 bid = parts[2]
                 if row["value"] is None:
+                    _logger.warning(
+                        "Skipping Bubble cursorDiskKV row with NULL value: key=%r",
+                        row["key"],
+                    )
                     continue
                 try:
                     parsed = json.loads(row["value"])
@@ -195,6 +199,10 @@ def _safe_fetchall(query: str, params: tuple = ()) -> list:
         for row in composer_rows:
             composer_id = row["key"].split(":")[1]
             if row["value"] is None:
+                _logger.warning(
+                    "Skipping Composer cursorDiskKV row with NULL value: key=%r",
+                    row["key"],
+                )
                 continue
             try:
                 parsed = json.loads(row["value"])

tests/test_workspace_tabs_null_bubble.py

@@ -72,15 +72,21 @@ def tearDown(self):
     def test_null_bubble_row_is_skipped_without_exception(self):
         """assemble_workspace_tabs must not raise when a bubble row has NULL value."""
         try:
-            _payload, status = assemble_workspace_tabs(
-                workspace_id="global",
-                workspace_path=self.workspace_path,
-                rules=[],
-            )
+            with self.assertLogs("services.workspace_tabs", level="WARNING") as cm:
+                _payload, status = assemble_workspace_tabs(
+                    workspace_id="global",
+                    workspace_path=self.workspace_path,
+                    rules=[],
+                )
         except TypeError as exc:
             self.fail(f"NULL bubble row raised TypeError: {exc}")
 
         self.assertEqual(status, 200, "NULL bubble row must not turn tabs load into an error response")
+        messages = [r.getMessage() for r in cm.records]
+        self.assertTrue(
+            any("NULL value" in m and "bubble-null" in m for m in messages),
+            f"expected NULL-value warning for bubble-null row, got: {messages}",
+        )
 
     def test_healthy_bubbles_still_load_when_null_row_present(self):
         """The healthy bubble surfaces in a tab even when a NULL row is present."""