[psteinroe]

Stacked on #938.

#898 put json_value/json_query/json_exists on the function-call layout and listed the other constructs the release review found past the line width. This PR handles those. Each had no break opportunity, or broke without an indent so that the continuation sat at the enclosing level.

Calls whose arguments the formatter writes itself

Two new helpers, open_call_arguments / close_call_arguments, give these the function-call layout: (, a break and one more indent when the call doesn't fit, ) back on the call's line. The keyword parts that join the arguments (in, from, placing, for, passing, as, the commas) become break opportunities:

• json_arrayagg / json_objectagg. The call has its own group when FILTER or OVER follows, so the line breaks before the clause first, as for other aggregates since fix(pretty_print): group function calls so count(*), arrays and JSON functions break sensibly #898. Their FILTER now uses the same filter (where ...) layout as other aggregates.
• xmlserialize, xmlroot, xmlelement, xmlparse, xmlpi, xmlexists
• position(... in ...), extract(... from ...), overlay(...), substring(...), trim(...), normalize(...)

-- before (lineWidth 60)
select
  position(some_long_substring_column_name in some_other_long_string_column_name)
from
  t;
-- after
select
  position(
    some_long_substring_column_name
    in some_other_long_string_column_name
  )
from
  t;

Statement parts

• CREATE SUBSCRIPTION: connection, publication and with (...) are indented continuation clauses, as in ALTER SUBSCRIPTION and GRANT. Before, a long statement printed the header on one line and then broke inside the publication list or the options.
• comment on ... is '...': breaks before is, one level in.
• Event trigger when tag in (...): the tags fill indented lines inside the parentheses.
• explain (...): the options are a parenthesized list, one per line when they don't fit.
• json_table columns: a column that doesn't fit puts path, the wrapper, quotes and the on empty / on error clauses on their own lines, one level in.

Snapshot changes

All reviewed; every change is one of the constructs above going from a too-long line, or a break without an indent, to the layout described:

• subscription, object_address, create_subscription_stmt_0: subscription clauses
• xml: XML functions
• sqljson: JSON aggregates and their FILTER
• sqljson_jsontable: json_table columns
• strings: overlay
• timetz, interval: extract
• plpgsql, create_table_like, comment_stmt_2: comment on ... is
• event_trigger: when tag in
• partition_prune: explain options

A few calls that were broken before now fit on one line.

New width_layout.rs tests format each construct at width 60 and check every line fits, with exact output for the main shapes. The option matrix passes.

Not covered

cast(x as type) still breaks before as without an indent (visible inside extract in timetz). Changing that touches most snapshots, so it is left for its own PR.