Apply code review feedback: optimize performance and remove debug code

Co-authored-by: huangyiirene <7665279+huangyiirene@users.noreply.github.com>

Author: Copilot

packages/components/src/renderers/complex/kanban.tsx

@@ -4,34 +4,10 @@ import React from 'react';
 
 ComponentRegistry.register('kanban', 
   ({ schema, className, ...props }) => {
-    const [columns, setColumns] = React.useState<KanbanColumn[]>(schema.columns || []);
-
-    React.useEffect(() => {
-      if (schema.columns) {
-        setColumns(schema.columns);
-      }
-    }, [schema.columns]);
-
-    const handleCardMove = (
-      cardId: string,
-      fromColumnId: string,
-      toColumnId: string,
-      newIndex: number
-    ) => {
-      // This is where you would handle the card move event
-      // For example, you could call an API or trigger an action
-      console.log('Card moved:', { cardId, fromColumnId, toColumnId, newIndex });
-      
-      // If there's an onCardMove callback in schema, call it
-      if (schema.onCardMove) {
-        schema.onCardMove({ cardId, fromColumnId, toColumnId, newIndex });
-      }
-    };
-
     return (
       <KanbanBoard 
-        columns={columns}
-        onCardMove={schema.onCardMove ? handleCardMove : undefined}
+        columns={schema.columns || []}
+        onCardMove={schema.onCardMove}
         className={className}
         {...props}
       />

packages/components/src/ui/kanban.tsx

@@ -220,21 +220,30 @@ export function KanbanBoard({ columns, onCardMove, className }: KanbanBoardProps
     }
   }
 
-  const findCard = (cardId: string): KanbanCard | null => {
-    for (const column of boardColumns) {
-      const card = column.cards.find((c) => c.id === cardId)
-      if (card) return card
-    }
-    return null
-  }
+  const findCard = React.useCallback(
+    (cardId: string): KanbanCard | null => {
+      for (const column of boardColumns) {
+        const card = column.cards.find((c) => c.id === cardId)
+        if (card) return card
+      }
+      return null
+    },
+    [boardColumns]
+  )
 
-  const findColumnByCardId = (cardId: string): KanbanColumn | null => {
-    return boardColumns.find((col) => col.cards.some((c) => c.id === cardId)) || null
-  }
+  const findColumnByCardId = React.useCallback(
+    (cardId: string): KanbanColumn | null => {
+      return boardColumns.find((col) => col.cards.some((c) => c.id === cardId)) || null
+    },
+    [boardColumns]
+  )
 
-  const findColumnById = (columnId: string): KanbanColumn | null => {
-    return boardColumns.find((col) => col.id === columnId) || null
-  }
+  const findColumnById = React.useCallback(
+    (columnId: string): KanbanColumn | null => {
+      return boardColumns.find((col) => col.id === columnId) || null
+    },
+    [boardColumns]
+  )
 
   return (
     <DndContext