Fix code review feedback: simplify startEdit and fix test selector

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/components/src/renderers/complex/__tests__/data-table-batch-editing.test.tsx

@@ -71,11 +71,10 @@ describe('Data Table - Batch Editing', () => {
       expect(modifiedIndicator).toBeInTheDocument();
     });
 
-    // Edit second cell in same row - now the name shows as 'John Smith'
-    const emailCell = container.querySelector('td:has-text("john@example.com")') || 
-                      Array.from(container.querySelectorAll('td')).find(el => 
-                        el.textContent?.includes('john@example.com')
-                      );
+    // Edit second cell in same row - find by searching through all cells
+    const emailCell = Array.from(container.querySelectorAll('td')).find(el => 
+      el.textContent?.includes('john@example.com')
+    );
 
     expect(emailCell).toBeInTheDocument();
     if (emailCell) {

packages/components/src/renderers/complex/data-table.tsx

@@ -350,16 +350,17 @@ const DataTableRenderer = ({ schema }: { schema: DataTableSchema }) => {
   };
 
   // Cell editing handlers
-  const startEdit = (rowIndex: number, columnKey: string, currentValue: any) => {
+  const startEdit = (rowIndex: number, columnKey: string) => {
     if (!editable) return;
 
     const column = columns.find(col => col.accessorKey === columnKey);
     if (column?.editable === false) return;
 
     setEditingCell({ rowIndex, columnKey });
 
-    // Check if there's a pending change for this cell
+    // Check if there's a pending change for this cell, otherwise use current data value
     const rowChanges = pendingChanges.get(rowIndex);
+    const currentValue = paginatedData[rowIndex][columnKey];
     const valueToEdit = rowChanges?.[columnKey] ?? currentValue ?? '';
     setEditValue(valueToEdit);
   };
@@ -461,8 +462,7 @@ const DataTableRenderer = ({ schema }: { schema: DataTableSchema }) => {
 
     if (e.key === 'Enter' && !editingCell) {
       e.preventDefault();
-      const value = paginatedData[rowIndex][columnKey];
-      startEdit(rowIndex, columnKey, value);
+      startEdit(rowIndex, columnKey);
     }
   };
 
@@ -707,7 +707,7 @@ const DataTableRenderer = ({ schema }: { schema: DataTableSchema }) => {
                               minWidth: columnWidth,
                               maxWidth: columnWidth
                             }}
-                            onDoubleClick={() => isEditable && startEdit(rowIndex, col.accessorKey, originalValue)}
+                            onDoubleClick={() => isEditable && startEdit(rowIndex, col.accessorKey)}
                             onKeyDown={(e) => handleCellKeyDown(e, rowIndex, col.accessorKey)}
                             tabIndex={0}
                           >