feat: enhance ThemedTable2 with widget tests and performance improvements; remove debug prints

Author: ManuelRomeroA

CHANGELOG.md

@@ -7,6 +7,8 @@
 - Fixed `ThemedTable2` unsendable closure crash: sort keys are now precomputed on the main thread before `compute()`, ensuring `valueBuilder` closures that capture `BuildContext` or `i18n` objects never cross the isolate boundary.
 - Improved `ThemedTable2` multiselect performance: introduced an internal `Set<T>` mirroring `_selectedItems` for O(1) `contains()` lookups instead of O(n) on the backing `List<T>`.
 - Improved `ThemedTable2` `didUpdateWidget` performance: replaced `DeepCollectionEquality` (O(n)) with a fast O(1) heuristic using `identical` + length check.
+- Removed all `debugPrint` statements from `ThemedTable2._filterAndSort` (startup, precompute, sort, finish, queue logs).
+- Added widget tests for `ThemedTable2` covering: renders item values after compute completes, shows loading indicator while computing, renders without errors on empty dataset, column key collision regression (two columns with identical `headerText` show distinct data), search filters by matching query, search matches values from all columns, search restores all items when cleared, `controller.sort` ascending (A→Z), `controller.sort` descending (Z→A), numeric sort as numbers not strings, `controller.refresh` preserves sort order, `didUpdateWidget` reflects new items on rebuild, `didUpdateWidget` handles growing list without losing existing rows.
 - Add `ThemedTable2` skill.
 
 ## 7.5.21

example/pubspec.lock

@@ -390,7 +390,7 @@ packages:
       path: ".."
       relative: true
     source: path
-    version: "7.5.21"
+    version: "7.5.22"
   leak_tracker:
     dependency: transitive
     description:

lib/src/table2/src/table.dart

@@ -236,12 +236,10 @@ class _ThemedTable2State<T> extends State<ThemedTable2<T>> {
   }
 
   Future<void> _filterAndSort(String source) async {
-    debugPrint("layrz_theme/ThemedTable2: Starting _filterAndSortAsync from $source...");
     if (_isLoading.value) {
       // Queue the update instead of silently dropping it.
       // _filterAndSort will re-run once the current operation finishes.
       _pendingUpdate = true;
-      debugPrint('layrz_theme/ThemedTable2: Queuing _filterAndSortAsync from $source (already loading)');
       return;
     }
 
@@ -250,12 +248,10 @@ class _ThemedTable2State<T> extends State<ThemedTable2<T>> {
     try {
       List<T> items = .from(widget.items, growable: true);
       if (items.isEmpty) {
-        debugPrint("layrz_theme/ThemedTable2: No items to filter and sort from $source.");
         _filteredData.value = items;
         return;
       }
 
-      debugPrint("layrz_theme/ThemedTable2: Precomputing data from $source...");
       // Key: item.hashCode (stable for value-based domain objects, consistent across isolate boundaries).
       // Value: List<String> indexed by column position — avoids col.hashCode collisions
       //        (e.g. two columns with the same headerText).
@@ -267,7 +263,6 @@ class _ThemedTable2State<T> extends State<ThemedTable2<T>> {
       }
 
       if (_searchController.text.isNotEmpty) {
-        debugPrint("layrz_theme/ThemedTable2: Filtering data from $source...");
         final searchLower = _searchController.text.toLowerCase();
         items = items.where((row) {
           final cols = _itemsStrings[row.hashCode];
@@ -279,13 +274,11 @@ class _ThemedTable2State<T> extends State<ThemedTable2<T>> {
         }).toList();
       }
 
-      debugPrint("layrz_theme/ThemedTable2: Sorting data...");
       // Precompute sort keys on the main thread so that valueBuilder closures
       // (which may capture BuildContext or i18n objects) never cross the isolate boundary.
       final columnIndex = widget.columns.indexOf(_colSelected);
       final sortKeys = [
-        for (final item in items)
-          _itemsStrings[item.hashCode]?[columnIndex] ?? _colSelected.valueBuilder(item),
+        for (final item in items) _itemsStrings[item.hashCode]?[columnIndex] ?? _colSelected.valueBuilder(item),
       ];
       _filteredData.value = await compute(
         _sort,
@@ -297,7 +290,6 @@ class _ThemedTable2State<T> extends State<ThemedTable2<T>> {
         ),
       );
     } finally {
-      debugPrint("layrz_theme/ThemedTable2: Finished filtering and sorting from $source.");
       _isLoading.value = false;
       if (mounted) WidgetsBinding.instance.addPostFrameCallback((_) => setState(() {}));
       // Process a queued update that arrived while we were loading.
@@ -342,7 +334,6 @@ class _ThemedTable2State<T> extends State<ThemedTable2<T>> {
       final ascending = event.ascending;
 
       if (columnIndex < 0 || columnIndex >= widget.columns.length) {
-        debugPrint('layrz_theme/ThemedTable2: Invalid column index $columnIndex for sorting');
         return;
       }
 
@@ -357,8 +348,6 @@ class _ThemedTable2State<T> extends State<ThemedTable2<T>> {
       _filterAndSort('CONTROLLER_REFRESH');
       return;
     }
-
-    debugPrint('layrz_theme/ThemedTable2: Unknown controller event type: ${event.runtimeType}');
   }
 
   @override
@@ -650,8 +639,9 @@ class _ThemedTable2State<T> extends State<ThemedTable2<T>> {
                                                   }
                                                 } else {
                                                   if (_selectedSet.contains(item)) {
-                                                    _selectedItems.value =
-                                                        selectedList.where((i) => i != item).toList();
+                                                    _selectedItems.value = selectedList
+                                                        .where((i) => i != item)
+                                                        .toList();
                                                   }
                                                 }
                                               },
@@ -701,8 +691,7 @@ class _ThemedTable2State<T> extends State<ThemedTable2<T>> {
                                               // Use column index as key (avoids col.hashCode collisions
                                               // when two columns share the same headerText).
                                               String text =
-                                                  _itemsStrings[data.hashCode]?[colIndex] ??
-                                                  header.valueBuilder(data);
+                                                  _itemsStrings[data.hashCode]?[colIndex] ?? header.valueBuilder(data);
 
                                               Widget child;
                                               if (header.richTextBuilder != null) {