Only return render_coll columns that differ from default, refs #2619

Author: simonw

datasette/views/row.py

@@ -140,9 +140,6 @@ async def template_data():
                             break
                     if plugin_display_value:
                         rendered_row[column] = str(plugin_display_value)
-                    else:
-                        # Default: convert value to string
-                        rendered_row[column] = "" if value is None else str(value)
                 rendered_rows.append(rendered_row)
             data["render_cell"] = rendered_rows
 

datasette/views/table.py

@@ -1516,9 +1516,6 @@ async def extra_render_cell():
                         break
                 if plugin_display_value:
                     rendered_row[column] = str(plugin_display_value)
-                else:
-                    # Default: convert value to string
-                    rendered_row[column] = "" if value is None else str(value)
             rendered_rows.append(rendered_row)
         return rendered_rows
 

tests/test_api.py

@@ -792,14 +792,15 @@ def render_cell(self, value, column, table, database):
         assert "rows" in data
 
         # render_cell should be a list with one row (since this is a row page)
+        # Only columns modified by plugins are included (sparse output)
         render_cell = data["render_cell"]
         assert len(render_cell) == 1
 
         # The row: id=1, name='Alice'
         # The 'name' column should be rendered by our plugin as <strong>Alice</strong>
         assert render_cell[0]["name"] == "<strong>Alice</strong>"
-        # The 'id' column should use default rendering (just the value as string)
-        assert render_cell[0]["id"] == "1"
+        # The 'id' column is not included since no plugin modified it
+        assert "id" not in render_cell[0]
 
         # The regular rows should still contain raw values
         assert data["rows"] == [{"id": 1, "name": "Alice"}]

tests/test_table_api.py

@@ -1426,18 +1426,19 @@ def render_cell(self, value, column, table, database):
         assert "rows" in data
 
         # render_cell should be a list of rows, each row being a dict of column -> rendered HTML
+        # Only columns modified by plugins are included (sparse output)
         render_cell = data["render_cell"]
         assert len(render_cell) == 2
 
         # First row: id=1, name='Alice'
         # The 'name' column should be rendered by our plugin as <strong>Alice</strong>
         assert render_cell[0]["name"] == "<strong>Alice</strong>"
-        # The 'id' column should use default rendering (just the value as string)
-        assert render_cell[0]["id"] == "1"
+        # The 'id' column is not included since no plugin modified it
+        assert "id" not in render_cell[0]
 
         # Second row: id=2, name='Bob'
         assert render_cell[1]["name"] == "<strong>Bob</strong>"
-        assert render_cell[1]["id"] == "2"
+        assert "id" not in render_cell[1]
 
         # The regular rows should still contain raw values
         assert data["rows"] == [