Update param wrapping rule for functions with no params

[Copilot]

Prevents the wrap and wrapArguments rules from wrapping inside empty () of trivial function declarations — functions with no parameters, no return type, and no effects (just () {).

For example, this prevents wrapping:

func longFunctionNameBeyondWrappingLimit() {
}

to either of these undesirable forms:

// Previously possible — now prevented
func longFunctionNameBeyondWrappingLimit(
) {
}

// Also prevented for trivial () { functions
func longFunctionNameBeyondWrappingLimit()
{
}

The fix also actively unwraps already-wrapped empty parameter lists for trivial functions back to () {.

Functions with return types, generics, effects, etc. are not affected — wrapping is still allowed for those cases:

// This can still wrap as before
func foo() -> Int {}

Changes

• Sources/FormattingHelpers.swift: Added a guard in wrapCollectionsAndArguments that skips wrapping for empty parameter lists when the function is trivial (no return type, effects, generics — just () {). Also unwraps any already-wrapped empty parens in that case.
• Tests/Rules/WrapTests.swift: Added 3 tests covering no-wrap for multi-line and single-line functions, and unwrapping of already-wrapped empty parens.
• Tests/Rules/WrapArgumentsTests.swift: Added 3 tests covering wrapParameters: .beforeFirst for func, init, and unwrapping of already-wrapped empty parens.

All 5471 existing tests pass with the changes.

---

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

[calda]

@copilot, also ensure we don't wrap it to:

func `long function name beyond wrapping limit()
{
}

It should be preserved as-is.

If it has a return type, generics, params, etc, it's fine to wrap. Just don't wrap a trivial () { function like that.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.28%. Comparing base (4bab950) to head (3af77a9).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2467      +/-   ##
===========================================
- Coverage    95.28%   95.28%   -0.01%     
===========================================
  Files          165      165              
  Lines        25230    25243      +13     
===========================================
+ Hits         24041    24052      +11     
- Misses        1189     1191       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

@copilot, also ensure we don't wrap it to:

...

Implemented in 726f5d0. The fix in wrapCollectionsAndArguments skips wrapping for empty parameter lists where the function has no return type, effects, or other complexity — just a trivial () {. It also actively unwraps any already-wrapped empty parens in that case. Functions with return types (func foo() -> Int {}), effects, generics, etc. are still allowed to wrap as before.