Merge pull request #40 from hugoh/output-title

fix(tui): added command to output window

Author: hugoh

internal/tui/commands.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"strings"
 
 	tea "charm.land/bubbletea/v2"
 	"github.com/google/shlex"
@@ -127,6 +128,7 @@ func execCmd(m *model, selected []string, prefix, cmdStr string) tea.Cmd {
 	m.execTotal = len(selected)
 	m.execResults = nil
 	m.execOutputStr = ""
+	m.execLabel = strings.TrimSpace(prefix + " " + cmdStr)
 
 	concurrency := m.cfg.Settings.Concurrency
 

internal/tui/commands_test.go

@@ -40,6 +40,30 @@ func TestExecCmd(t *testing.T) {
 	require.NotNil(t, cmd)
 }
 
+func TestExecCmdSetsLabel(t *testing.T) {
+	tests := []struct {
+		name      string
+		prefix    string
+		cmdStr    string
+		wantLabel string
+	}{
+		{"vcs shortcut", "", "status", "status"},
+		{"git command", "git", "diff HEAD", "git diff HEAD"},
+		{"shell command", "sh", "ls -la", "sh ls -la"},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			m := &model{
+				cfg: config.Config{Settings: config.Settings{Concurrency: 1}},
+			}
+
+			execCmd(m, nil, tt.prefix, tt.cmdStr)
+			assert.Equal(t, tt.wantLabel, m.execLabel)
+		})
+	}
+}
+
 func TestShortcutCmdPrefixIndependent(t *testing.T) {
 	tests := []struct {
 		name   string

internal/tui/model.go

@@ -185,6 +185,7 @@ type model struct {
 	execCancel     context.CancelFunc
 	execResults    []execResult
 	execOutputStr  string
+	execLabel      string
 	resultsCh      <-chan runner.Result
 
 	statusCh <-chan runner.StatusResult

internal/tui/view.go

@@ -205,11 +205,16 @@ func (*model) renderFooter() string {
 }
 
 func (m *model) outputView() string {
+	label := m.execLabel
+	if label == "" {
+		label = "Output"
+	}
+
 	var header string
 	if m.executing {
-		header = styleHeader.Render(" Output " + m.spinner.View() + " ")
+		header = styleHeader.Render(" " + label + " " + m.spinner.View() + " ")
 	} else {
-		header = styleHeader.Render(" Output ")
+		header = styleHeader.Render(" " + label + " ")
 	}
 
 	sep := styleSeparator.Render(strings.Repeat(separatorChar, m.width))

internal/tui/view_test.go

@@ -104,6 +104,34 @@ func TestOutputView(t *testing.T) {
 	assert.Contains(t, view, "Output")
 }
 
+func TestOutputViewWithLabel(t *testing.T) {
+	tests := []struct {
+		name      string
+		execLabel string
+		want      string
+	}{
+		{"vcs shortcut", "status", "status"},
+		{"git command", "git diff HEAD", "git diff HEAD"},
+		{"shell command", "sh ls -la", "sh ls -la"},
+		{"no label falls back", "", "Output"},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			m := &model{
+				screen:    screenOutput,
+				width:     80,
+				height:    30,
+				execLabel: tt.execLabel,
+				output:    viewport.New(viewport.WithWidth(80), viewport.WithHeight(10)),
+			}
+			m.ready = true
+
+			assert.Contains(t, m.outputView(), tt.want)
+		})
+	}
+}
+
 func TestOutputViewExecuting(t *testing.T) {
 	m := &model{
 		screen:    screenOutput,