What problems was I solving

The ContainerUnit abstract class in flexi-builder shipped with a protected cloneWithChildren() method that had no callers anywhere in the codebase. Its presence was confusing because the existing clone() method already clones child units, and invoking cloneWithChildren() would have cloned children a second time into the already-populated map.

What user-facing changes did I ship

There are no user-facing changes; this is an internal code-cleanup change in an abstract builder utility class.

How I implemented it

Removed the unused cloneWithChildren() method from architect/flexi-builder/main/platform/core/units/abstract/container-unit.ts. Verified via repo-wide search that no code, tests, or subclasses reference the deleted method, and confirmed that clone() and cloneGraph() continue to handle child cloning as before.

Description for the changelog

Removed the dead ContainerUnit.cloneWithChildren() protected method from flexi-builder. The method had no callers and duplicated the child-cloning logic already performed by clone(), so deleting it reduces surface area and prevents future misuse.