feat(Masonry): [tower] fixed iteration through nested and stacked children in a tower

Author: saumyashahi

modules/masonry/playground/pages/workspace/WorkspaceCanvas.tsx

@@ -259,12 +259,18 @@ export default function WorkspaceCanvas(): JSX.Element {
       tower.addNestedBrick(nestedCompound.uuid, nestedChild, { x: 0, y: 0 });
 
       const nestedChild2 = createSimpleBrick({ label: 'Nested Simple' });
-      tower.addNestedBrick(nestedCompound.uuid, nestedChild2, { x: 0, y: 0 });
+      tower.addBrick(nestedChild.uuid, nestedChild2, { x: 0, y: 0 });
+
+      const nestedChild3 = createSimpleBrick({ label: 'Nested Simple' });
+      tower.addBrick(nestedChild2.uuid, nestedChild3, { x: 0, y: 0 });
 
       // Add a simple nested brick directly to root
       const simpleNested = createSimpleBrick({ label: 'Simple Nested' });
       tower.addBrick(nestedCompound.uuid, simpleNested, { x: 0, y: 0 });
 
+      const simpleNested2 = createSimpleBrick({ label: 'Simple Nested' });
+      tower.addBrick(simpleNested.uuid, simpleNested2, { x: 0, y: 0 });
+
       // Add stacked bricks
       let lastUuid = root.uuid;
       for (let i = 1; i <= 2; i++) {

modules/masonry/src/tower/utils/towerUtils.ts

@@ -79,22 +79,11 @@ export function calculateNestedAreaDimensions(
   let totalNestedHeight = 0;
   let totalNestedWidth = 0;
 
-  // Recursively process all nested children and their descendants
-  const processNestedChain = (nodeId: string) => {
-    const node = allNodes.get(nodeId);
-    if (!node) return;
-
-    const { h, w } = calculateCompleteSubtreeDimensions(nodeId, allNodes, memoMap);
+  // Process only nested children and their full subtrees
+  children.nested.forEach(nestedChild => {
+    const { h, w } = calculateCompleteSubtreeDimensions(nestedChild.brick.uuid, allNodes, memoMap);
     totalNestedHeight += h;
     totalNestedWidth = Math.max(totalNestedWidth, w);
-
-    const childChildren = getNodeChildren(nodeId, allNodes);
-    childChildren.nested.forEach(child => processNestedChain(child.brick.uuid));
-    childChildren.stacked.forEach(child => processNestedChain(child.brick.uuid)); // Include stacked chains as part of nested area
-  };
-
-  children.nested.forEach(nestedChild => {
-    processNestedChain(nestedChild.brick.uuid);
   });
 
   const result = { w: totalNestedWidth, h: totalNestedHeight };
@@ -245,6 +234,17 @@ export function debugBoundingBoxCalculation(
       console.log(`         Original: ${childOriginal.w}×${childOriginal.h}`);
       console.log(`         Calculated: ${childBB?.w || 0}×${childBB?.h || 0}`);
     });
+
+    // Additional debug for stacked children affecting nested area
+    console.log(`    Stacked children affecting nested area:`);
+    children.stacked.forEach((child, index) => {
+      const childBB = bbMap.get(child.brick.uuid);
+      const childLabel = child.brick.name || child.brick.type || 'Unknown';
+      const childOriginal = child.brick.boundingBox;
+      console.log(`     ${index + 1}. "${childLabel}":`);
+      console.log(`         Original: ${childOriginal.w}×${childOriginal.h}`);
+      console.log(`         Calculated: ${childBB?.w || 0}×${childBB?.h || 0}`);
+    });
   });
 
   console.log('\n=== END DETAILED DEBUG ===\n');