fix(cli): fix Prompt.text clear when input wraps terminal lines

[m9tdev]

Summary

When the user's input in Prompt.text exceeds the terminal width and wraps to a new line, renderClearScreen miscalculates lines to erase because it only considers options.message, not the full rendered line including the input value.

This causes duplicate prompt lines to be printed on each keystroke after the text wraps:

? What would you like to say? › what do you propose to add to play around with? it can be
? What would you like to say? › what do you propose to add to play around with? it can be
? What would you like to say? › what do you propose to add to play around with? it can be
...

Root Cause

In renderClearScreen, line erasure is calculated based only on options.message:

const clearOutput = InternalAnsiUtils.eraseText(options.message, columns)

But the actual rendered line includes: "? " + message + " › " + inputValue

Fix

Calculate the full rendered line length:

const inputValue = state.value.length > 0 ? state.value : options.default
const fullLine = `? ${options.message} › ${inputValue}`
const clearOutput = InternalAnsiUtils.eraseText(fullLine, columns)

Related

This is the same root cause as #5978 (for Prompt.select), and applies the equivalent fix from #5979 to Prompt.text.

Closes #5978

[changeset-bot]

🦋 Changeset detected

Latest commit: 2eeb3fa

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@effect/cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[IMax153]

Thanks @m9tdev ! Would you be willing to test the Prompt impl in Effect v4 to see if a similar issue exists and open an issue there if so? If not, I can take a look.

[IMax153]

Thanks @m9tdev ! Would you be willing to test the Prompt impl in Effect v4 to see if a similar issue exists and open an issue there if so? If not, I can take a look.

Actually, nevermind. I think Tim's lalph bot is picking up backports we need to do. Effect-TS/effect-smol#1159