@object-ui/plugin-list 3.1.2 → 3.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@object-ui/plugin-list",
-  "version": "3.1.2",
+  "version": "3.1.4",
   "type": "module",
   "license": "MIT",
   "description": "ListView plugin for Object UI - unified view component with view type switching",
@@ -24,13 +24,13 @@
     }
   },
   "dependencies": {
-    "lucide-react": "^0.576.0",
-    "@object-ui/components": "3.1.2",
-    "@object-ui/core": "3.1.2",
-    "@object-ui/i18n": "3.1.2",
-    "@object-ui/mobile": "3.1.2",
-    "@object-ui/react": "3.1.2",
-    "@object-ui/types": "3.1.2"
+    "lucide-react": "^0.577.0",
+    "@object-ui/components": "3.1.4",
+    "@object-ui/core": "3.1.4",
+    "@object-ui/i18n": "3.1.4",
+    "@object-ui/mobile": "3.1.4",
+    "@object-ui/react": "3.1.4",
+    "@object-ui/types": "3.1.4"
   },
   "peerDependencies": {
     "react": "^18.0.0 || ^19.0.0",
@@ -39,11 +39,11 @@
   "devDependencies": {
     "@types/react": "19.2.14",
     "@types/react-dom": "19.2.3",
-    "@vitejs/plugin-react": "^5.1.4",
+    "@vitejs/plugin-react": "^6.0.1",
     "typescript": "^5.9.3",
-    "vite": "^7.3.1",
+    "vite": "^8.0.1",
     "vite-plugin-dts": "^4.5.4",
-    "vitest": "^4.0.18"
+    "vitest": "^4.1.0"
   },
   "scripts": {
     "build": "vite build",

package/src/ListView.tsx

@@ -661,6 +661,8 @@ export const ListView: React.FC<ListViewProps> = ({
               items = (results as any).data; 
            } else if (Array.isArray((results as any).records)) {
               items = (results as any).records;
+           } else if (Array.isArray((results as any).value)) {
+              items = (results as any).value;
            }
         }
         

package/src/ObjectGallery.tsx

@@ -193,7 +193,7 @@ export const ObjectGallery: React.FC<ObjectGalleryProps> = (props) => {
     if (!items.length) return <div className="p-4 text-sm text-muted-foreground">No items to display</div>;
 
     const renderCard = (item: Record<string, unknown>, i: number) => {
-        const id = (item._id ?? item.id ?? i) as string | number;
+        const id = (item.id ?? item._id ?? i) as string | number;
         const title = String(item[titleField] ?? 'Untitled');
         const imageUrl = item[coverField] as string | undefined;
 

package/src/__tests__/DataFetch.test.tsx

@@ -40,7 +40,7 @@ describe('ListView Data Fetch', () => {
     it('shows data limit warning when items reach the page size', async () => {
       // Generate exactly 100 items (default page size)
       const items = Array.from({ length: 100 }, (_, i) => ({
-        _id: String(i),
+        id: String(i),
         name: `Item ${i}`,
       }));
       mockDataSource.find.mockResolvedValue(items);
@@ -62,8 +62,8 @@ describe('ListView Data Fetch', () => {
 
     it('does not show data limit warning when items are below page size', async () => {
       const items = [
-        { _id: '1', name: 'Alice' },
-        { _id: '2', name: 'Bob' },
+        { id: '1', name: 'Alice' },
+        { id: '2', name: 'Bob' },
       ];
       mockDataSource.find.mockResolvedValue(items);
 
@@ -84,7 +84,7 @@ describe('ListView Data Fetch', () => {
 
     it('uses custom page size from schema.pagination', async () => {
       const items = Array.from({ length: 50 }, (_, i) => ({
-        _id: String(i),
+        id: String(i),
         name: `Item ${i}`,
       }));
       mockDataSource.find.mockResolvedValue(items);
@@ -113,6 +113,35 @@ describe('ListView Data Fetch', () => {
     });
   });
 
+  // =========================================================================
+  // OData { value: [] } response format handling
+  // =========================================================================
+  describe('OData value response format', () => {
+    it('should extract records from { value: [] } OData response', async () => {
+      const items = [
+        { id: '1', name: 'Alice' },
+        { id: '2', name: 'Bob' },
+      ];
+      mockDataSource.find.mockResolvedValue({ value: items, '@odata.count': 2 });
+
+      const schema: ListViewSchema = {
+        type: 'list-view',
+        objectName: 'contacts',
+        viewType: 'grid',
+        fields: ['name'],
+      };
+
+      renderWithProvider(<ListView schema={schema} dataSource={mockDataSource} />);
+
+      await vi.waitFor(() => {
+        expect(screen.getByTestId('record-count-bar')).toBeInTheDocument();
+      });
+
+      // Records should be extracted and rendered (not empty)
+      expect(screen.queryByTestId('empty-state')).not.toBeInTheDocument();
+    });
+  });
+
   // =========================================================================
   // $expand race condition fix (Issue #939 Bug 1)
   // =========================================================================
@@ -126,7 +155,7 @@ describe('ListView Data Fetch', () => {
         },
       });
       mockDataSource.find.mockResolvedValue([
-        { _id: '1', name: 'Order 1', customer: { name: 'Alice' }, account: { name: 'Acme' } },
+        { id: '1', name: 'Order 1', customer: { name: 'Alice' }, account: { name: 'Acme' } },
       ]);
 
       const schema: ListViewSchema = {
@@ -152,7 +181,7 @@ describe('ListView Data Fetch', () => {
       let resolveSchema: (value: any) => void;
       const schemaPromise = new Promise(resolve => { resolveSchema = resolve; });
       mockDataSource.getObjectSchema = vi.fn().mockReturnValue(schemaPromise);
-      mockDataSource.find.mockResolvedValue([{ _id: '1', name: 'Item 1' }]);
+      mockDataSource.find.mockResolvedValue([{ id: '1', name: 'Item 1' }]);
 
       const schema: ListViewSchema = {
         type: 'list-view',
@@ -203,14 +232,14 @@ describe('ListView Data Fetch', () => {
       );
 
       // Resolve second (newer) request first
-      resolveSecond!([{ _id: '2', name: 'Second' }]);
+      resolveSecond!([{ id: '2', name: 'Second' }]);
       
       await vi.waitFor(() => {
         expect(mockDataSource.find).toHaveBeenCalled();
       });
 
       // Resolve first (stale) request later
-      resolveFirst!([{ _id: '1', name: 'First (stale)' }]);
+      resolveFirst!([{ id: '1', name: 'First (stale)' }]);
 
       // Wait for state to settle — second request data should win
       await vi.waitFor(() => {

package/src/__tests__/Export.test.tsx

@@ -76,8 +76,8 @@ describe('ListView Export', () => {
 
   it('should handle export with complex object fields in CSV safely', async () => {
     const mockItems = [
-      { _id: '1', name: 'Alice', tags: ['admin', 'user'], metadata: { role: 'lead' } },
-      { _id: '2', name: 'Bob', tags: ['user'], metadata: null },
+      { id: '1', name: 'Alice', tags: ['admin', 'user'], metadata: { role: 'lead' } },
+      { id: '2', name: 'Bob', tags: ['user'], metadata: null },
     ];
     mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -134,7 +134,7 @@ describe('ListView Export', () => {
 
   it('should handle export with JSON format', async () => {
     const mockItems = [
-      { _id: '1', name: 'Alice', email: 'alice@test.com' },
+      { id: '1', name: 'Alice', email: 'alice@test.com' },
     ];
     mockDataSource.find.mockResolvedValue(mockItems);
 

package/src/__tests__/ListView.test.tsx

@@ -430,9 +430,9 @@ describe('ListView', () => {
 
   it('should show record count bar when data is loaded', async () => {
     const mockItems = [
-      { _id: '1', name: 'Alice', email: 'alice@test.com' },
-      { _id: '2', name: 'Bob', email: 'bob@test.com' },
-      { _id: '3', name: 'Charlie', email: 'charlie@test.com' },
+      { id: '1', name: 'Alice', email: 'alice@test.com' },
+      { id: '2', name: 'Bob', email: 'bob@test.com' },
+      { id: '3', name: 'Charlie', email: 'charlie@test.com' },
     ];
     mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -930,8 +930,8 @@ describe('ListView', () => {
   describe('showRecordCount flag', () => {
     it('should hide record count bar when showRecordCount is false', async () => {
       const mockItems = [
-        { _id: '1', name: 'Alice', email: 'alice@test.com' },
-        { _id: '2', name: 'Bob', email: 'bob@test.com' },
+        { id: '1', name: 'Alice', email: 'alice@test.com' },
+        { id: '2', name: 'Bob', email: 'bob@test.com' },
       ];
       mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -957,7 +957,7 @@ describe('ListView', () => {
 
     it('should show record count bar by default (showRecordCount undefined)', async () => {
       const mockItems = [
-        { _id: '1', name: 'Alice', email: 'alice@test.com' },
+        { id: '1', name: 'Alice', email: 'alice@test.com' },
       ];
       mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -1453,7 +1453,7 @@ describe('ListView', () => {
   describe('pageSizeOptions', () => {
     it('should render page size selector when pageSizeOptions is provided', async () => {
       const mockItems = [
-        { _id: '1', name: 'Alice', email: 'alice@test.com' },
+        { id: '1', name: 'Alice', email: 'alice@test.com' },
       ];
       mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -1474,7 +1474,7 @@ describe('ListView', () => {
 
     it('should not render page size selector when pageSizeOptions is not provided', async () => {
       const mockItems = [
-        { _id: '1', name: 'Alice', email: 'alice@test.com' },
+        { id: '1', name: 'Alice', email: 'alice@test.com' },
       ];
       mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -1539,8 +1539,8 @@ describe('ListView', () => {
         data: {
           provider: 'value',
           items: [
-            { _id: '1', name: 'Alice', email: 'alice@test.com' },
-            { _id: '2', name: 'Bob', email: 'bob@test.com' },
+            { id: '1', name: 'Alice', email: 'alice@test.com' },
+            { id: '2', name: 'Bob', email: 'bob@test.com' },
           ],
         } as any,
       };
@@ -1562,8 +1562,8 @@ describe('ListView', () => {
         viewType: 'grid',
         fields: ['name', 'email'],
         data: [
-          { _id: '1', name: 'Alice', email: 'alice@test.com' },
-          { _id: '2', name: 'Bob', email: 'bob@test.com' },
+          { id: '1', name: 'Alice', email: 'alice@test.com' },
+          { id: '2', name: 'Bob', email: 'bob@test.com' },
         ] as any,
       };
 
@@ -1584,9 +1584,9 @@ describe('ListView', () => {
         viewType: 'grid',
         fields: ['name', 'email'],
         data: [
-          { _id: '1', name: 'Alice', email: 'alice@test.com' },
-          { _id: '2', name: 'Bob', email: 'bob@test.com' },
-          { _id: '3', name: 'Charlie', email: 'charlie@test.com' },
+          { id: '1', name: 'Alice', email: 'alice@test.com' },
+          { id: '2', name: 'Bob', email: 'bob@test.com' },
+          { id: '3', name: 'Charlie', email: 'charlie@test.com' },
         ] as any,
       };
 
@@ -1616,8 +1616,8 @@ describe('ListView', () => {
         data: {
           provider: 'value',
           items: [
-            { _id: '1', name: 'Alice', email: 'alice@test.com' },
-            { _id: '2', name: 'Bob', email: 'bob@test.com' },
+            { id: '1', name: 'Alice', email: 'alice@test.com' },
+            { id: '2', name: 'Bob', email: 'bob@test.com' },
           ],
         } as any,
       };
@@ -1641,7 +1641,7 @@ describe('ListView', () => {
 
     it('should fall back to dataSource.find when schema.data is not set', async () => {
       const mockItems = [
-        { _id: '1', name: 'Alice', email: 'alice@test.com' },
+        { id: '1', name: 'Alice', email: 'alice@test.com' },
       ];
       mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -1947,8 +1947,8 @@ describe('ListView', () => {
   describe('pageSizeOptions dynamic integration', () => {
     it('should render page size selector as controlled component', async () => {
       const mockItems = [
-        { _id: '1', name: 'Alice', email: 'alice@test.com' },
-        { _id: '2', name: 'Bob', email: 'bob@test.com' },
+        { id: '1', name: 'Alice', email: 'alice@test.com' },
+        { id: '2', name: 'Bob', email: 'bob@test.com' },
       ];
       mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -1972,8 +1972,8 @@ describe('ListView', () => {
 
     it('should re-fetch data when page size changes', async () => {
       const mockItems = [
-        { _id: '1', name: 'Alice', email: 'alice@test.com' },
-        { _id: '2', name: 'Bob', email: 'bob@test.com' },
+        { id: '1', name: 'Alice', email: 'alice@test.com' },
+        { id: '2', name: 'Bob', email: 'bob@test.com' },
       ];
       mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -2008,7 +2008,7 @@ describe('ListView', () => {
 
     it('should render all page size options in the selector', async () => {
       const mockItems = [
-        { _id: '1', name: 'Alice', email: 'alice@test.com' },
+        { id: '1', name: 'Alice', email: 'alice@test.com' },
       ];
       mockDataSource.find.mockResolvedValue(mockItems);
 
@@ -2036,7 +2036,7 @@ describe('ListView', () => {
 
     it('should not render page size selector when pageSizeOptions is not configured', async () => {
       const mockItems = [
-        { _id: '1', name: 'Alice', email: 'alice@test.com' },
+        { id: '1', name: 'Alice', email: 'alice@test.com' },
       ];
       mockDataSource.find.mockResolvedValue(mockItems);