bombsimon / wsl / 24284976958

95%

Build Type

push

github

Committed by web-flow

Commit Message

feat: add cuddle-max-statements config (#245)

* feat: add cuddle-max-statements config

Allow configuring how many statements can be cuddled above block
statements (if, for, range, switch, select, send, go, defer), provided
each cuddled statement has at least one identifier used in the block.
Default is 1 (backward compatible), 0 means unlimited.

This has been requested multiple times over the years. The new
`cuddle-max-statements` config replaces the hardcoded limit of 1
while preserving all existing behavior at the default setting.

Also refactors: extract isAssignDeclOrIncDec, add Cursor.NthPrevious,
replace identIntersection with short-circuiting identsIntersect, build
target ident set once via cuddleTargetIdents, and clean up the
enforceLimit parameter to use bool instead of int sentinel.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Update docs for flag and README

* Fix panic, drop `groupedDecls`, add more tests

With the old `groupedDecls` we got overlapping reports resulting in
panic in some code bases (e.g. the `assignVarAssign` test case).

Never create overlapping diagnostics.

* Add more tests

* Add better error message

* Add non-block tests

* Add tests for first-in-block

* Use early return pattern

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

104 of 107 new or added lines in 4 files covered. (97.2%)

2 existing lines in 1 file now uncovered.

1378 of 1463 relevant lines covered (94.19%)

321.2 hits per line