Don't create errors for ignored failed commands

[brianreavis]

Objective

Reduce overhead from error handling for ECS commands that intentionally ignore errors, such as try_despawn. These commands currently allocate error objects and pass them to a no-op handler (ignore), which can impact performance when many operations fail.

Fix a hang when removing ChildOf components during entity despawning. Excessive logging of these failures can cause significant hangs (I'm noticing around 100ms).

Solution

• Made the error handler on handle_error_with optional to avoid creating error objects when the error is just going to be swallowed.
• Replaced remove::<ChildOf> with try_remove::<ChildOf> to suppress expected (?) errors and reduce log noise.

Future Work

• Take a similar approach for observers

Testing

• Not super versed with ECS internals! Happy to make changes or go with a completely other approach. The Option approach on handle_error_with is not elegant or super clear, but didn't have any better ideas at the time.
• I ran these changes on a local project.