Prefer preserving WithItem parentheses

[MichaReiser]

Summary

This PR fixes a deviation to Black when formatting WithItems.

if True:
    with anyio.CancelScope(shield=True) if get_running_loop() else contextlib.nullcontext() as b:
        pass
    
    with (anyio.CancelScope(shield=True) if get_running_loop() else contextlib.nullcontext()):
        pass

Black formats this as:

if True:
    with anyio.CancelScope(
        shield=True
    ) if get_running_loop() else contextlib.nullcontext() as b:
        pass

    with (
        anyio.CancelScope(shield=True)
        if get_running_loop()
        else contextlib.nullcontext()
    ):
        pass

Notice that the two examples are identical except that the second example has parentheses around the with context expression.

Ruff formats both examples as if the context expression is not parenthesized:

if True:
    with anyio.CancelScope(
        shield=True
    ) if get_running_loop() else contextlib.nullcontext() as b:
        pass

This PR uses the IfBreaks layout that prefers parenthesizing the context manager when it is already parenthesized in source, aligning the behavior with black.

Closes #7441

Test Plan

Added tests. I reviewed the changes and they now match black's formatting.

The similarity index remains unchanged.

[MichaReiser]

Current dependencies on/for this PR:

• main
  • PR Prefer preserving WithItem parentheses #7694 👈

This comment was auto-generated by Graphite.