ponder-lab / Hybridize-Functions-Refactoring / 27392210002

79%

push

github

Return a string from `addInputSignature`; compose one decorator edit (#606)

* Return a string from `addInputSignature`; compose one decorator edit

`addInputSignature` now returns `Optional<String>` (the parenthesized
`(input_signature=...)` text) instead of an `Optional<TextEdit>`. `convertToHybrid`
composes the whole decorator into a single `InsertEdit` rather than three
same-offset zero-length `InsertEdit`s, so correctness no longer depends on
Eclipse sequencing same-offset edits by add-order. The `reconfigure()`
argless-decorator branch wraps the returned string in an `InsertEdit` at its
AST-derived offset.

Pure refactor; behavior unchanged.

Closes #575

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* Wrap the composed decorator edit in a `MultiTextEdit`

Keep every element of `convertToHybrid`'s returned list a container. The
processor builds the per-file `TextChange` by making the first edit the root and
adding the rest as children, which requires the root to accept children; a bare
`InsertEdit` root would fail when a file has multiple such edits. Restores the
container shape while keeping the single composed edit from #575.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

4 of 4 new or added lines in 1 file covered. (100.0%)

1442 of 1826 relevant lines covered (78.97%)

0.79 hits per line