From 49fc97e6e45a2e45d2698fe03e20d48497a470d1 Mon Sep 17 00:00:00 2001 From: Molecule AI Core Platform Lead Date: Fri, 24 Apr 2026 18:30:36 +0000 Subject: [PATCH] refactor(canvas): remove unused EmbeddedTeam component from WorkspaceNode MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit EmbeddedTeam was defined in WorkspaceNode.tsx but had no call site — TeamMemberChip (which is called directly) covers the same rendering responsibility. The function was stranded after a prior refactor and was flagged by github-code-quality on PR #1989 (merged 2026-04-24T14:09Z without this cleanup because the token died before push). Removes 25 lines of dead code. MAX_NESTING_DEPTH is kept — it is used by TeamMemberChip at line 498. Co-Authored-By: Claude Sonnet 4.6 --- canvas/src/components/WorkspaceNode.tsx | 25 ------------------------- 1 file changed, 25 deletions(-) diff --git a/canvas/src/components/WorkspaceNode.tsx b/canvas/src/components/WorkspaceNode.tsx index 49c093e6..a2a8962f 100644 --- a/canvas/src/components/WorkspaceNode.tsx +++ b/canvas/src/components/WorkspaceNode.tsx @@ -322,31 +322,6 @@ function countDescendants(nodeId: string, allNodes: Node[], v * infinite recursion on circular parentId references and keeps the UI readable. */ const MAX_NESTING_DEPTH = 3; -/** Subscribes to allNodes only when children exist — isolates re-renders from parent */ -function EmbeddedTeam({ members, depth, onSelect, onExtract }: { - members: Node[]; - depth: number; - onSelect: (id: string) => void; - onExtract: (id: string) => void; -}) { - const allNodes = useCanvasStore((s) => s.nodes); - // Use grid layout at depth 0 when there are multiple members (departments side-by-side) - const useGrid = depth === 0 && members.length >= 2; - return ( -
-
Team Members
-
- {members.map((child) => ( - - ))} -
-
- ); -} - /** Recursive mini-card — mirrors parent card layout at smaller scale */ function TeamMemberChip({ node,