wez
fix: skip FreshLine by semantic integration in ble.sh
A problem was originally reported in https://github.com/akinomyoga/ble.sh/issues/413. ble.sh (a line editor for Bash) keeps track of the cursor position in the terminal display (including the horizontal position relative to the bottom of the terminal) based on the terminal control sequences that it sends.
However, with WezTerm's semantic integration, WezTerm may insert a newline in precmd (FreshLine), which breaks the cursor position tracking of ble.sh. The needed cursor position adjustment at the end of the command execution is already performed by ble.sh, so WezTerm does not need to adjust it. In this patch, we skip the request for FreshLine in ble.sh.
But I'm not sure if this is the correct fix. The original sequence OSC A seems to perform FreshLine and StartPrompt. To only perform StartPrompt, one seems to be able to use the sequence OSC P ... BEL. Although OSC P seems to receive a parameter k=?, it doesn't seem to be used in the codebase, so I just omitted the parameter k. Could you take a look at this?
Note: As an alternative option, we can specify the ble.sh option bleopt prompt_command_changes_layout=1 to notify that precmd/PROMPT_COMMAND can change the cursor position (in particular horizontal position), but this adds unneeded flickering of the rendered contents. Because we can just suppress FreshLine this time, I didn't use this approach.
@wez I'm sorry to bother you, but would it be possible to take a look at this PR? This is a small adjustment for the integration with ble.sh to fix https://github.com/akinomyoga/ble.sh/issues/413. The current Bash integration by wezterm already includes the integration code with ble.sh, but it causes a conflict with a certain setup of ble.sh.
Sorry, I've been fairly swamped this past year(!)
I think this looks fine; it would be nice if we had a way to capture the aid (pid) for the ble case, but wezterm doesn't do anything with that information anyway, so it's a moot point.
