Entity.removeAllChildren() does not remove all children

[eonarheim]

Steps to Reproduce

Reported by @chrisk-7777 #2419 (comment)

https://codepen.io/chrisk7777/pen/gOeKOOy?editors=0010

Note how every second child is removed? Not all of them. This had me scratching my head, but after some digging, and I could be wrong, but I believe its because we're altering the array inside a loop:

https://github.com/excaliburjs/Excalibur/blob/main/src/engine/EntityComponentSystem/Entity.ts#L258

Expected Result

All children should be removed

Actual Result

Every other child is removed

Environment

• browsers and versions: NA
• operating system: NA
• Excalibur versions: v0.27.0

Current Workaround

Manually iterate over the children and call removeChild individually

// Avoid modifying the array issue by walking backwards
for (let i = entity.children.length - 1; i >= 0; i--) {
  entity.removeChild(this.children[i]);
}