fix: address code review - add error logging, improve test assertions, simplify condition

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

Author: Copilot

packages/plugin-detail/src/RelatedList.tsx

@@ -86,8 +86,10 @@ export const RelatedList: React.FC<RelatedListProps> = ({
 
   // Auto-fetch object schema when api/dataSource available but columns missing
   React.useEffect(() => {
-    if (api && dataSource?.getObjectSchema && (!columns || columns.length === 0)) {
-      dataSource.getObjectSchema(api).then(setObjectSchema).catch(() => {});
+    if (api && dataSource?.getObjectSchema && !columns?.length) {
+      dataSource.getObjectSchema(api).then(setObjectSchema).catch((err: unknown) => {
+        console.warn(`[RelatedList] Failed to fetch schema for ${api}:`, err);
+      });
     }
   }, [api, dataSource, columns]);
 

packages/plugin-detail/src/__tests__/RelatedList.test.tsx

@@ -91,6 +91,14 @@ describe('RelatedList', () => {
     await waitFor(() => {
       expect(mockDataSource.getObjectSchema).toHaveBeenCalledWith('order_item');
     });
+
+    // Verify columns are generated from schema (excluding _id)
+    await waitFor(() => {
+      expect(screen.getByText('Product')).toBeInTheDocument();
+      expect(screen.getByText('Quantity')).toBeInTheDocument();
+    });
+    // _id should be filtered out
+    expect(screen.queryByText('ID')).not.toBeInTheDocument();
   });
 
   it('should not fetch object schema when explicit columns are provided', () => {