Updated how nested function calls are overflowed

[ytmimi]

Fixes #5356

I feel like there's got to be a simpler way to handle this, so any recommendations on how we can update the implementation are greatly appreciated. I noticed that in the original issue the callee was being split overly multiple lines and tried to use that to conditionally determine whether we rewrite the expression as we did before by just calling expr.rewrite or go down a path where we do some extra work to figure out how we'll rewrite the overflowing function call.

I found it a bit strange that in some cases we need to indent the shape, while in others we need to unindent it, but I imagine that has something to do with what's happening when last_item_shape is called.

[ytmimi]

Also, I wonder if we should hold off on this until after #5337 is merged, to see if these changes play well with the proposed fn_call_layout option.

[ytmimi]

wondering if these changes need to be version gated 🤔