Merge pull request #801 from objectstack-ai/copilot/fix-ci-build-test-errors-another-one

Author: hotlong

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

@@ -132,9 +132,9 @@ vi.mock('sonner', () => ({
 
 // Mock Radix Dialog portal to render inline for testing
 vi.mock('@radix-ui/react-dialog', async () => {
-  const actual = await vi.importActual<typeof import('@radix-ui/react-dialog')>('@radix-ui/react-dialog');
+  const actual = await vi.importActual('@radix-ui/react-dialog');
   return {
-    ...actual,
+    ...(actual as Record<string, unknown>),
     Portal: ({ children }: { children: React.ReactNode }) => <>{children}</>,
   };
 });

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

@@ -82,9 +82,9 @@ vi.mock('sonner', () => ({
 
 // Mock Radix Dialog portal to render inline for testing
 vi.mock('@radix-ui/react-dialog', async () => {
-  const actual = await vi.importActual<typeof import('@radix-ui/react-dialog')>('@radix-ui/react-dialog');
+  const actual = await vi.importActual('@radix-ui/react-dialog');
   return {
-    ...actual,
+    ...(actual as Record<string, unknown>),
     Portal: ({ children }: { children: React.ReactNode }) => <>{children}</>,
   };
 });
@@ -189,7 +189,7 @@ describe('DesignDrawer — Right-Side Editor Panel', () => {
   });
 
   it('should not navigate away when edit button is clicked', () => {
-    const { container } = renderPageView();
+    renderPageView();
 
     fireEvent.click(screen.getByTestId('page-edit-button'));
 

apps/console/src/components/DesignDrawer.tsx

@@ -6,7 +6,7 @@
  * main workspace remains visible while the user edits.
  */
 
-import { useState, useCallback, useEffect, useRef } from 'react';
+import { useCallback, useEffect, useRef } from 'react';
 import {
   Sheet,
   SheetContent,

packages/plugin-aggrid/src/AgGridImpl.tsx

@@ -8,7 +8,7 @@
 
 import React, { useMemo, useRef, useCallback } from 'react';
 import { AgGridReact } from 'ag-grid-react';
-import type { ColDef, GridOptions, GridReadyEvent, CellClickedEvent, RowClickedEvent, SelectionChangedEvent, CellValueChangedEvent, StatusPanelDef, GetContextMenuItemsParams, MenuItemDef } from 'ag-grid-community';
+import type { ColDef, GridOptions, GridReadyEvent, CellClickedEvent, RowClickedEvent, SelectionChangedEvent, CellValueChangedEvent, StatusPanelDef, GetContextMenuItemsParams, MenuItemDef, DefaultMenuItem } from 'ag-grid-community';
 import type { AgGridCallbacks, ExportConfig, StatusBarConfig, ColumnConfig, ContextMenuConfig, TreeDataConfig, RowGroupingConfig, ExcelExportConfig } from './types';
 
 export interface AgGridImplProps {
@@ -145,10 +145,10 @@ export default function AgGridImpl({
   }, [excelExport, exportConfig, callbacks, rowData]);
 
   // Context Menu handler
-  const getContextMenuItems = useCallback((params: GetContextMenuItemsParams): (string | MenuItemDef)[] => {
+  const getContextMenuItems = useCallback((params: GetContextMenuItemsParams): (DefaultMenuItem | MenuItemDef)[] => {
     if (!contextMenu?.enabled) return [];
 
-    const items: (string | MenuItemDef)[] = [];
+    const items: (DefaultMenuItem | MenuItemDef)[] = [];
     const defaultItems = contextMenu.items || ['copy', 'copyWithHeaders', 'separator', 'export'];
 
     defaultItems.forEach(item => {
@@ -183,7 +183,7 @@ export default function AgGridImpl({
           },
         });
       } else {
-        items.push(item);
+        items.push(item as DefaultMenuItem);
       }
     });
 

packages/plugin-list/src/__tests__/ListViewGroupingPropagation.test.tsx

@@ -51,11 +51,15 @@ vi.mock('@object-ui/mobile', () => ({
   usePullToRefresh: () => ({ pullRef: { current: null } }),
 }));
 
-vi.mock('@object-ui/core', () => ({
-  ExpressionEvaluator: {
-    evaluate: vi.fn((expr: string) => expr),
-  },
-}));
+vi.mock('@object-ui/core', async (importOriginal) => {
+  const actual = await importOriginal<Record<string, unknown>>();
+  return {
+    ...actual,
+    ExpressionEvaluator: {
+      evaluate: vi.fn((expr: string) => expr),
+    },
+  };
+});
 
 vi.mock('@object-ui/i18n', () => ({
   useObjectTranslation: () => ({