Fix return in filter closure eval

[sholderbach]

Description

Closes #12257

This was down to the use of eval_block instead of eval_block_with_early_return. We may want to reconsider how we differentiate between this behavior. We currently need to check all the remaining commands that can invoke a closure block, if they properly handle ShellError::Return as a passing of a Value

• Add test for return in filter closure
• Fix use of return in filter closure

User-Facing Changes

You can now return a value from a filter closure

Tests + Formatting

Regression test

[IanManske]

Yeah, I think in the future it would be nice to refactor how closures are evaluated (e.g., a ClosureEvaluator). Sometimes we reset the env vars, and sometimes we don't. Sometimes we use eval_block and sometimes we use eval_block_with_early_return.

But this looks good, thanks!