Fix textual-fastdatatable 0.19 rendering compatibility

pyproject.toml

@@ -27,7 +27,7 @@ classifiers = [
 ]
 dependencies = [
     "textual[syntax]>=6.10.0",
-    "textual-fastdatatable>=0.14.0",
+    "textual-fastdatatable==0.19.0",
     "pyperclip>=1.8.2",
     "keyring>=24.0.0",
     "docker>=7.0.0",

sqlit/shared/ui/widgets_tables.py

@@ -11,9 +11,8 @@
 from rich.markup import escape
 from rich.protocol import is_renderable
 from rich.text import Text
-from textual.coordinate import Coordinate
-
 from textual.containers import Container
+from textual.coordinate import Coordinate
 from textual.events import Key
 from textual.strip import Strip
 from textual_fastdatatable import DataTable as FastDataTable
@@ -59,8 +58,18 @@ def render_line(self, y: int) -> Strip:
 
         return self._render_line(y, scroll_x, scroll_x + width, self.rich_style)
 
-    def _get_cell_renderable(self, row_index: int, column_index: int) -> Any:
-        """Format cells with plain text for NULL/bool/date values."""
+    def _get_cell_renderable(
+        self,
+        row_index: int,
+        column_index: int,
+        max_width: int | None = None,
+    ) -> Any:
+        """Format cells with plain text for NULL/bool/date values.
+
+        ``textual-fastdatatable`` 0.19 passes the available cell width to this
+        hook. Keep accepting it even though sqlit's formatter currently relies
+        on Rich to crop the returned renderable.
+        """
         if row_index == -1:
             return self.ordered_columns[column_index].label
 

tests/ui/test_fastdatatable_compatibility.py

@@ -0,0 +1,33 @@
+"""Compatibility coverage for textual-fastdatatable's rendering hooks."""
+
+import pytest
+from textual.app import App, ComposeResult
+
+from sqlit.shared.ui.widgets_tables import SqlitDataTable
+
+
+class _TableApp(App[None]):
+    def __init__(self, table: SqlitDataTable) -> None:
+        super().__init__()
+        self.table = table
+
+    def compose(self) -> ComposeResult:
+        yield self.table
+
+
+def test_cell_renderable_accepts_width_from_fastdatatable_019() -> None:
+    """0.19 passes the available width to subclass cell renderers."""
+    table = SqlitDataTable(data={"id": [1]})
+
+    renderable = table._get_cell_renderable(-1, 0, max_width=10)
+
+    assert getattr(renderable, "plain", renderable) == "id"
+
+
+@pytest.mark.asyncio
+async def test_table_renders_with_fastdatatable_019_width_argument() -> None:
+    """Rendering a result table must not crash when 0.19 supplies max_width."""
+    table = SqlitDataTable(data={"id": list(range(100))})
+
+    async with _TableApp(table).run_test(size=(40, 10)):
+        assert "id" in table.render_line(0).text