chore(fmt): prefer_compact + indent bin expr w/ parenthesis

[0xrusowsky]

Motivation

• Turn call_compact_args into an enum prefer_compact to:
  • closes feat(fmt): multiline_func_header equivalent for events #4024
  • closes feat(fmt): add config to force multi-param structs to not break in a compact manner #12253
• Document document the new param prefer_compact (we forgot to doc call_compact_args when added after the migration, so i guess nobody should be using it yet)
• Ensure proper indentation in binary exprs with parentheses

PR Checklist

• Added Tests
• Added Documentation
• Breaking changes

[grandizzy]

make sense! should we apply prefer compact config to emits and reverts with named args too? e.g. if we don't have preferred compact events then break also emit as

        emit WithdrawFromLockupStream({
            streamId: ids.defaultStream,
            to: users.recipient,
            token: dai,
            amount: withdrawAmount
        });

and if we don't prefer compact reverts then break also revert as

        revert InsufficientBalance({
            balance: bal,
            withdrawAmount: _withdrawAmount,
            address1: _address,
            token: dai,
            to: users.recipient
        });

[0xrusowsky]

good call, will do it!

[grandizzy]

lgtm!

[grandizzy]

I think there's one more case that is not covered but we can fix in a follow-up if so - with prefer_compact = "none"

         // Make the withdrawal.
         lockup.withdraw{ value: LOCKUP_MIN_FEE_WEI }({
-            streamId: differentSenderRecipientStreamId,
-            to: address(recipientGood),
-            amount: withdrawAmount
+            streamId: differentSenderRecipientStreamId, to: address(recipientGood), amount: withdrawAmount
         });
     }
}