bg query workaround causes unwanted output

[rsteube]

Describe the bug

The bg query added as workaround in #1107 causes unwanted output when no tea application is started and output is redirected to /dev/stdout.

Setup

• OS [e.g. Ubuntu, macOS]
  linux
• Shell [e.g. zsh, fish]
  bash
• Terminal Emulator [e.g. kitty, iterm]
  foot
• Terminal Multiplexer [e.g. tmux]
  none

To Reproduce

go run . >/dev/stdout

Source Code

package main

import (
	_ "github.com/charmbracelet/bubbletea"
)

func main() {}

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots

bsh ❯ go run . >/dev/stdout
Documents/development/moep via 🐹 v1.23.0 
bsh ❯ 11;rgb:0000/0000/0000R

Additional context

[meowgorithm]

Oof, thanks for the report, @rsteube. We’ll prioritize this one.

Also—one of the primary reasons for a v2 is to eliminate this situation entirely.

[caarlos0]

@aymanbagabas does v2 fixes this ?

[rsteube]

Ah, yes. I think I still had to patch it for alpha.1 (tab-dance@7502ab8).
But I'm now on alpha.2 without that.

Still got this nasty bit in the code though:

	case tea.KeyMsg:
		switch msg.String() {
		case "ctrl+c", "esc":
			return m, tea.Quit
		case "alt+]", "alt+\\", "10;rgb:ffff/ffff/ffff", "11;rgb:0000/0000/0000": // TODO
			return m, nil
		}

Don't think it's necessary anymore, but gotta check (didn't have the time to work on this for a while).

[rsteube]

Yeah, i think it's fine to close this now. Seems to work.

[meowgorithm]

Sounds good! Yeah, we've completely solved this in v2 and you shouldn't need that anymore. Sorry for the inconvenience, Ralf!