Merge pull request #382 from objectstack-ai/copilot/update-action-step-instructions

Author: hotlong

packages/plugin-chatbot/src/ChatbotEnhanced.tsx

@@ -338,6 +338,7 @@ const ChatbotEnhanced = React.forwardRef<HTMLDivElement, ChatbotEnhancedProps>(
                 size="icon"
                 onClick={() => fileInputRef.current?.click()}
                 disabled={disabled}
+                aria-label="Attach file"
               >
                 <Paperclip className="h-4 w-4" />
               </Button>
@@ -358,6 +359,7 @@ const ChatbotEnhanced = React.forwardRef<HTMLDivElement, ChatbotEnhancedProps>(
             onClick={handleSend}
             disabled={disabled || (!inputValue.trim() && selectedFiles.length === 0)}
             size="icon"
+            aria-label="Send message"
           >
             <Send className="h-4 w-4" />
           </Button>

packages/plugin-chatbot/src/__tests__/ChatbotEnhanced.test.tsx

@@ -6,6 +6,7 @@
  * LICENSE file in the root directory of this source tree.
  */
 
+import '@testing-library/jest-dom';
 import { describe, it, expect, vi } from 'vitest';
 import { render, screen, fireEvent, waitFor } from '@testing-library/react';
 import { ChatbotEnhanced, type ChatMessage } from '../ChatbotEnhanced';
@@ -56,7 +57,7 @@ describe('ChatbotEnhanced', () => {
     fireEvent.click(sendButton);
 
     await waitFor(() => {
-      expect(onSendMessage).toHaveBeenCalledWith('Test message', undefined);
+      expect(onSendMessage).toHaveBeenCalledWith('Test message', []);
     });
   });
 
@@ -70,7 +71,7 @@ describe('ChatbotEnhanced', () => {
     fireEvent.keyDown(input, { key: 'Enter', code: 'Enter' });
 
     await waitFor(() => {
-      expect(onSendMessage).toHaveBeenCalledWith('Test message', undefined);
+      expect(onSendMessage).toHaveBeenCalledWith('Test message', []);
     });
   });
 

packages/plugin-chatbot/vite.config.ts

@@ -47,6 +47,8 @@ export default defineConfig({
     },
   },
   test: {
+    globals: true,
+    environment: 'happy-dom',
     passWithNoTests: true,
   },
 });

packages/plugin-dashboard/src/DashboardRenderer.tsx

@@ -9,7 +9,7 @@
 import type { DashboardSchema, DashboardWidgetSchema } from '@object-ui/types';
 import { SchemaRenderer } from '@object-ui/react';
 import { cn, Card, CardHeader, CardTitle, CardContent } from '@object-ui/components';
-import { forwardRef, useMemo } from 'react';
+import { forwardRef } from 'react';
 
 // Color palette for charts
 const CHART_COLORS = [
@@ -39,7 +39,7 @@ export const DashboardRenderer = forwardRef<HTMLDivElement, { schema: DashboardS
             // Logic to determine what to render
             // Supports both Component Schema (widget.component) and Shorthand (widget.type)
 
-            const componentSchema = useMemo(() => {
+            const getComponentSchema = () => {
                 if (widget.component) return widget.component;
 
                 // Handle Shorthand Registry Mappings
@@ -77,7 +77,9 @@ export const DashboardRenderer = forwardRef<HTMLDivElement, { schema: DashboardS
                 }
 
                 return widget; // Fallback to widget itself as schema
-            }, [widget]);
+            };
+            
+            const componentSchema = getComponentSchema();
 
             // Check if the widget is self-contained (like a Metric Card) to avoid double borders
             const isSelfContained = (widget as any).type === 'metric';

packages/plugin-kanban/src/KanbanEnhanced.tsx

@@ -236,7 +236,7 @@ function KanbanColumnEnhanced({
   )
 }
 
-export default function KanbanEnhanced({
+export function KanbanEnhanced({
   columns,
   onCardMove,
   onColumnToggle,
@@ -390,3 +390,5 @@ export default function KanbanEnhanced({
     </DndContext>
   )
 }
+
+export default KanbanEnhanced;

packages/plugin-kanban/src/__tests__/KanbanEnhanced.test.tsx

@@ -116,9 +116,10 @@ describe('KanbanEnhanced', () => {
   it('should display card count for each column', () => {
     render(<KanbanEnhanced columns={mockColumns} />);
 
-    // Columns should show card counts
-    expect(screen.getByText(/2/)).toBeInTheDocument(); // To Do has 2 cards
-    expect(screen.getByText(/1/)).toBeInTheDocument(); // In Progress has 1 card
+    // Columns should show card counts in the format "count" or "count / limit"
+    expect(screen.getByText('To Do')).toBeInTheDocument();
+    expect(screen.getByText('In Progress')).toBeInTheDocument();
+    expect(screen.getByText('1 / 3')).toBeInTheDocument(); // In Progress has 1 card with limit 3
   });
 
   it('should display column limit warning when at 80% capacity', () => {
@@ -134,11 +135,10 @@ describe('KanbanEnhanced', () => {
       },
     ];
 
-    render(<KanbanEnhanced columns={columnsNearLimit} />);
-    
-    // Should show warning indicator (80% of 5 = 4)
     const { container } = render(<KanbanEnhanced columns={columnsNearLimit} />);
-    expect(container.querySelector('[class*="warning"]')).toBeTruthy();
+    
+    // Should show warning indicator (80% of 5 = 4) - AlertTriangle icon with yellow color
+    expect(container.querySelector('.text-yellow-500')).toBeTruthy();
   });
 
   it('should collapse/expand columns when toggle button is clicked', () => {