fix some stray terminal query output

Author: zeroshade

cmd/dbc/auth.go

@@ -88,6 +88,8 @@ type authSuccessMsg struct {
 	cred auth.Credential
 }
 
+func (loginModel) NeedsRenderer() {}
+
 type loginModel struct {
 	baseModel
 

cmd/dbc/install.go

@@ -148,6 +148,8 @@ func (s installState) String() string {
 	}
 }
 
+func (progressiveInstallModel) NeedsRenderer() {}
+
 type progressiveInstallModel struct {
 	baseModel
 

cmd/dbc/main.go

@@ -59,6 +59,13 @@ type HasStatus interface {
 	Err() error
 }
 
+// NeedsRenderer is implemented by models that render a live TUI (spinners,
+// progress bars, interactive lists). Models that only use tea.Println /
+// tea.Printf and return an empty View do not need the renderer.
+type NeedsRenderer interface {
+	NeedsRenderer()
+}
+
 // use this so we can override this in tests
 var getDriverRegistry = dbc.GetDriverList
 
@@ -218,7 +225,8 @@ func main() {
 	// }
 	// defer f.Close()
 
-	if !isatty.IsTerminal(os.Stdout.Fd()) {
+	_, needsRenderer := m.(NeedsRenderer)
+	if !isatty.IsTerminal(os.Stdout.Fd()) || !needsRenderer {
 		prog = tea.NewProgram(m, tea.WithoutRenderer(), tea.WithInput(nil))
 	} else if args.Quiet {
 		// Quiet still prints stderr as GNU standard is to suppress "usual" output

cmd/dbc/sync.go

@@ -64,6 +64,8 @@ func (c SyncCmd) GetModel() tea.Model {
 	}
 }
 
+func (syncModel) NeedsRenderer() {}
+
 type syncModel struct {
 	baseModel
 

cmd/dbc/tui_driver_list.go

@@ -42,6 +42,8 @@ func (i item) Title() string       { return i.d.Title }
 func (i item) Description() string { return i.d.Desc }
 func (i item) FilterValue() string { return i.d.Title }
 
+func (model) NeedsRenderer() {}
+
 type model struct {
 	Prev tea.Model
 

cmd/dbc/tui_menu.go

@@ -46,6 +46,8 @@ func getTuiModel() tea.Model {
 	return m
 }
 
+func (menu) NeedsRenderer() {}
+
 type menu struct {
 	options list.Model
 }