fix: use record.name instead of record.id for grid row lookups

ObjectGrid applies $select using only column fields, which strips the
`id` property from records returned by ValueDataSource.find(). This
caused handleRowClick, handleEdit, and handleDelete to fail silently
since objects.find(o => o.id === record.id) always returned undefined.

Fixed by matching on `name` (which IS in the grid columns) instead of
`id` in both ObjectManager and FieldDesigner. Also strengthened the
ObjectManagerPage click-to-navigate test to assert the detail view
actually renders after clicking a primary-field-link.

Agent-Logs-Url: https://github.com/objectstack-ai/objectui/sessions/2766f2c6-b5ea-42f9-8327-dd8201c1afb7

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

Author: Copilot

apps/console/src/__tests__/ObjectManagerPage.test.tsx

@@ -155,18 +155,21 @@ describe('ObjectManagerPage', () => {
     it('should navigate to detail when primary field link is clicked', async () => {
       renderPage();
 
-      // ObjectGrid renders data asynchronously. Primary field links are auto-generated.
+      // ObjectGrid renders data asynchronously via ValueDataSource.
+      // Wait for primary-field-link buttons to appear.
       await waitFor(() => {
         const links = screen.queryAllByTestId('primary-field-link');
-        if (links.length > 0) {
-          fireEvent.click(links[0]);
-        }
-      });
+        expect(links.length).toBeGreaterThan(0);
+      }, { timeout: 5000 });
+
+      // Click the first primary field link (should be 'account')
+      const links = screen.getAllByTestId('primary-field-link');
+      fireEvent.click(links[0]);
 
-      // Either detail view shows (if link was found) or list remains
-      const detailView = screen.queryByTestId('object-detail-view');
-      const objectManager = screen.queryByTestId('object-manager');
-      expect(detailView || objectManager).toBeDefined();
+      // Should navigate to the object detail view
+      await waitFor(() => {
+        expect(screen.getByTestId('object-detail-view')).toBeDefined();
+      });
     });
   });
 });

apps/console/src/__tests__/click_test.test.tsx

@@ -565,13 +565,15 @@ export function FieldDesigner({
   }), [gridColumns]);
 
   // Handlers
+  // Note: ObjectGrid applies $select using column fields, which strips `id`.
+  // Use `name` (always in columns) as the lookup key instead of `id`.
   const handleEdit = useCallback((record: Record<string, unknown>) => {
-    const field = fields.find((f) => f.id === record.id);
+    const field = fields.find((f) => f.name === record.name);
     if (field) setEditingField(field);
   }, [fields]);
 
   const handleDelete = useCallback(async (record: Record<string, unknown>) => {
-    const field = fields.find((f) => f.id === record.id);
+    const field = fields.find((f) => f.name === record.name);
     if (!field || field.isSystem) return;
     const confirmed = await confirmDialog.confirm(
       t('appDesigner.fieldDesigner.deleteConfirmTitle'),

packages/plugin-designer/src/FieldDesigner.tsx

@@ -120,21 +120,23 @@ export function ObjectManager({
   }), [gridColumns]);
 
   // Handlers
+  // Note: ObjectGrid applies $select using column fields, which strips `id`.
+  // Use `name` (always in columns) as the lookup key instead of `id`.
   const handleRowClick = useCallback((record: Record<string, unknown>) => {
-    const obj = objects.find((o) => o.id === record.id);
+    const obj = objects.find((o) => o.name === record.name);
     if (obj) onSelectObject?.(obj);
   }, [objects, onSelectObject]);
 
   const handleEdit = useCallback((record: Record<string, unknown>) => {
-    const obj = objects.find((o) => o.id === record.id);
+    const obj = objects.find((o) => o.name === record.name);
     if (obj) {
       setEditingObject(obj);
       setFormOpen(true);
     }
   }, [objects]);
 
   const handleDelete = useCallback(async (record: Record<string, unknown>) => {
-    const obj = objects.find((o) => o.id === record.id);
+    const obj = objects.find((o) => o.name === record.name);
     if (!obj || obj.isSystem) return;
     const confirmed = await confirmDialog.confirm(
       t('appDesigner.objectManager.deleteConfirmTitle'),