Merge pull request #383 from objectstack-ai/copilot/fix-all-build-and-test

Author: hotlong

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

@@ -491,18 +491,18 @@ const DataTableRenderer = ({ schema }: { schema: DataTableSchema }) => {
                       key={rowId} 
                       data-state={isSelected ? 'selected' : undefined}
                       className={cn(
-                        // @ts-ignore
+                        // @ts-expect-error - onRowClick might not be in schema type definition
                         schema.onRowClick && "cursor-pointer"
                       )}
                       onClick={(e) => {
-                        // @ts-ignore
+                        // @ts-expect-error - onRowClick might not be in schema type definition
                         if (schema.onRowClick && !e.defaultPrevented) {
                            // Simple heuristic to avoid triggering on interactive elements if they didn't stop propagation
                            const target = e.target as HTMLElement;
                            if (target.closest('button') || target.closest('[role="checkbox"]') || target.closest('a')) {
                              return;
                            }
-                           // @ts-ignore
+                           // @ts-expect-error - onRowClick might not be in schema type definition
                            schema.onRowClick(row);
                         }
                       }}

packages/plugin-kanban/src/ObjectKanban.tsx

@@ -101,7 +101,7 @@ export const ObjectKanban: React.FC<ObjectKanbanProps> = ({
     if (!Array.isArray(rawData)) return [];
 
     // Support cardTitle property from schema (passed by ObjectView)
-    // @ts-ignore - cardTitle might not be in KanbanSchema type definition yet
+    // Fallback to legacy titleField for backwards compatibility
     let titleField = schema.cardTitle || (schema as any).titleField;
 
     // Fallback: Try to infer from object definition

packages/plugin-kanban/vitest.config.ts

@@ -1,7 +1,6 @@
 /// <reference types="vitest" />
 import { defineConfig } from 'vite';
 import react from '@vitejs/plugin-react';
-import path from 'path';
 
 export default defineConfig({
   plugins: [react()],

packages/plugin-timeline/src/ObjectTimeline.test.tsx

@@ -62,6 +62,8 @@ describe('ObjectTimeline', () => {
              expect(mockDataSource.find).toHaveBeenCalled();
         });
 
-        expect(screen.getByText('Event 1')).toBeDefined();
+        await waitFor(() => {
+            expect(screen.getByText('Event 1')).toBeDefined();
+        });
     });
 });