refactor: cleanup, ux/ui improvements

fix: improve bubble tea TUI visual appearance

- Remove unnecessary status line showing 'Snapshot accepted/rejected/skipped'
This cluttered the UI without providing additional value
- Fix footer styling to properly extend to both screen edges using statusBarStyle
- Restore helpStyle application to footer text for proper color/formatting
- Ensure footer is properly positioned in the layout hierarchy

The TUI now has a cleaner appearance with the status bar properly styled
and the action confirmation removed from the viewport.

feat: auto-close TUI when all snapshots have been reviewed

- TUI now automatically exits after the last snapshot is reviewed (accept/reject/skip)
- Track individual action counts (acceptedAll, rejectedAll, skippedAll) for each snapshot
- Check m.done status after loadCurrentSnapshot() in handlers and exit if review is complete
- Improve exit message to show all non-zero action counts (e.g., '✓ Accepted 3 • ⊘ Rejected 1 • ⊘ Skipped 2')
- This provides a better user experience by not leaving the TUI hanging after the last action

The TUI now properly closes after reviewing each snapshot one-by-one, matching the behavior
of the 'Accept All', 'Reject All', and 'Skip All' commands.

docs: update README with TUI usage documentation and add missing max() helper function

feat: implement semantic colors for TUI footer actions (green/red/yellow) and vertical layout

Improve TUI footer layout: horizontal actions with scroll percentage positioned right

- Changed footer layout from vertical to horizontal action buttons
- Reduced footer height from 4 to 2 lines for better space utilization
- Positioned scroll percentage (%) at the bottom-right corner
- Added spacing between action buttons for better visual clarity
- Minimized whitespace between diff content and action options

.gitignore

···
 go.work.sum
 .env
 cmd/tui/freeze-tui
-cmd/tui/tui
-cmd/freeze/freeze
-freeze-tui
-tui
-freeze

README.md

···
 
 ![New snapshot screen](./assets/screenshot-new.png "New snapshot view")
 
-![Snapshot review CLI](./assets/screenshots-diff-cli "Snapshot diff view (CLI)")
+![Snapshot review CLI](./assets/screenshots-diff-cli.png "Snapshot diff view (CLI)")
 
 ## Installation
 
···
 
 <!-- TODO: add example of `freeze.Review()` in go code -->
 
-Freeze also includes (in a separate Go module) with a [Bubbletea](https://github.com/charmbracelet/bubbletea) TUI in [cmd/tui/main.go](./cmd/tui/main.go). (The TUI is shipped in a separate module to make the added dependencies optional)
+Freeze also includes (in a separate Go module) a [Bubbletea](https://github.com/charmbracelet/bubbletea) TUI in [cmd/tui/main.go](./cmd/tui/main.go). (The TUI is shipped in a separate module to make the added dependencies optional)
+
+### TUI Usage
 
 ```sh
-# TODO: tui usage
+go run github.com/ptdewey/freeze/cmd/tui review
+```
+
+#### Interactive Controls
+
+- `a` - Accept current snapshot
+- `r` - Reject current snapshot
+- `s` - Skip current snapshot
+- `A` - Accept all remaining snapshots
+- `R` - Reject all remaining snapshots
+- `S` - Skip all remaining snapshots
+- `q` - Quit
+
+#### Alternative Commands
+
+```sh
+# Accept all new snapshots without review
+go run github.com/ptdewey/freeze/cmd/tui accept-all
+
+# Reject all new snapshots without review
+go run github.com/ptdewey/freeze/cmd/tui reject-all
 ```
 
 ## Disclaimer

__snapshots__/test_accept.snap

···
 test_name: TestAccept
 file_path: 
 func_name: 
+version: 
 ---
 new content to accept

__snapshots__/test_map.snap

···
 ---
 title: Map Test
 test_name: TestMap
-file_path: /home/patrick/projects/freeze/freeze.go
+file_path: 
 func_name: 
+version: 0.1.0
 ---
 map[string]interface{}{
  "foo": "bar",

__snapshots__/test_snap_custom_type.snap

···
 ---
 title: Custom Type Test
 test_name: TestSnapCustomType
-file_path: /home/patrick/projects/freeze/freeze.go
+file_path: 
 func_name: 
+version: 0.1.0
 ---
 freeze_test.CustomStruct{
  Name: "Alice",

__snapshots__/test_snap_func.snap

···
 ---
 title: TestSnapFunc
 test_name: TestSnapFunc
-file_path: /home/patrick/projects/freeze/freeze_test.go
+file_path: 
 func_name: 
+version: 0.1.0
 ---
 "helper result"

__snapshots__/test_snap_func_another_helper.snap

···
 ---
 title: TestSnapFuncAnotherHelper
 test_name: TestSnapFuncAnotherHelper
-file_path: /home/patrick/projects/freeze/freeze_test.go
+file_path: 
 func_name: 
+version: 0.1.0
 ---
 10

__snapshots__/test_snap_multiple.snap

···
 ---
 title: Multiple Values Test
 test_name: TestSnapMultiple
-file_path: /home/patrick/projects/freeze/freeze.go
+file_path: 
 func_name: 
+version: 0.1.0
 ---
 "value1"
 "value2"

__snapshots__/test_snap_string.snap

···
 ---
 title: Simple String Test
 test_name: TestSnapString
-file_path: /home/patrick/projects/freeze/freeze.go
+file_path: 
 func_name: 
+version: 0.1.0
 ---
 hello world

cmd/tui/main.go

···
 
 	contentStyle = lipgloss.NewStyle().
 			Padding(1, 2)
+
+	// Action styles with semantic colors
+	acceptStyle = lipgloss.NewStyle().
+			Foreground(lipgloss.Color("10")). // Green
+			Bold(true)
+
+	rejectStyle = lipgloss.NewStyle().
+			Foreground(lipgloss.Color("9")). // Red
+			Bold(true)
+
+	skipStyle = lipgloss.NewStyle().
+			Foreground(lipgloss.Color("11")). // Yellow
+			Bold(true)
+
+	keyStyle = lipgloss.NewStyle().
+			Foreground(lipgloss.Color("241"))
+
+	helpTextStyle = lipgloss.NewStyle().
+			Foreground(lipgloss.Color("240"))
 )
 
 type model struct {
···
 		m.height = msg.Height
 
 		headerHeight := 3
-		footerHeight := 2
+		footerHeight := 1
 		verticalMarginHeight := headerHeight + footerHeight
 
 		if !m.ready {
···
 			if err := files.AcceptSnapshot(testName); err != nil {
 				m.err = err
 			} else {
-				m.actionResult = "Snapshot accepted"
+				m.acceptedAll++
 				m.current++
 				if err := m.loadCurrentSnapshot(); err != nil {
 					m.err = err
+				}
+				if m.done {
+					return m, tea.Quit
 				}
 				m.updateViewportContent()
 			}
···
 			if err := files.RejectSnapshot(testName); err != nil {
 				m.err = err
 			} else {
-				m.actionResult = "Snapshot rejected"
+				m.rejectedAll++
 				m.current++
 				if err := m.loadCurrentSnapshot(); err != nil {
 					m.err = err
 				}
+				if m.done {
+					return m, tea.Quit
+				}
 				m.updateViewportContent()
 			}
 
 		case "s":
 			// Skip current snapshot
-			m.actionResult = "Snapshot skipped"
+			m.skippedAll++
 			m.current++
 			if err := m.loadCurrentSnapshot(); err != nil {
 				m.err = err
+			}
+			if m.done {
+				return m, tea.Quit
 			}
 			m.updateViewportContent()
 
···
 		}
 	}
 
-	if m.actionResult != "" {
-		b.WriteString("\n\n")
-		b.WriteString(pretty.Success("✓ " + m.actionResult))
-	}
+	// Add action options below the snapshot/diff box
+	b.WriteString("\n")
+	acceptLine := lipgloss.JoinHorizontal(lipgloss.Left,
+		keyStyle.Render("[a]"),
+		helpTextStyle.Render(" "),
+		acceptStyle.Render("accept"),
+	)
+	b.WriteString(acceptLine)
+	b.WriteString("\n")
+
+	rejectLine := lipgloss.JoinHorizontal(lipgloss.Left,
+		keyStyle.Render("[r]"),
+		helpTextStyle.Render(" "),
+		rejectStyle.Render("reject"),
+	)
+	b.WriteString(rejectLine)
+	b.WriteString("\n")
+
+	skipLine := lipgloss.JoinHorizontal(lipgloss.Left,
+		keyStyle.Render("[s]"),
+		helpTextStyle.Render(" "),
+		skipStyle.Render("skip"),
+	)
+	b.WriteString(skipLine)
 
 	m.viewport.SetContent(contentStyle.Render(b.String()))
 	m.viewport.GotoTop()
···
 			return pretty.Success("✓ No new snapshots to review\n")
 		}
 
+		// Build summary from counts
+		var summary []string
 		if m.acceptedAll > 0 {
-			return pretty.Success(fmt.Sprintf("✓ Accepted %d snapshot(s)\n", m.acceptedAll))
+			summary = append(summary, fmt.Sprintf("✓ Accepted %d", m.acceptedAll))
 		}
 		if m.rejectedAll > 0 {
-			return pretty.Warning(fmt.Sprintf("⊘ Rejected %d snapshot(s)\n", m.rejectedAll))
+			summary = append(summary, fmt.Sprintf("⊘ Rejected %d", m.rejectedAll))
 		}
 		if m.skippedAll > 0 {
-			return pretty.Warning(fmt.Sprintf("⊘ Skipped %d snapshot(s)\n", m.skippedAll))
+			summary = append(summary, fmt.Sprintf("⊘ Skipped %d", m.skippedAll))
+		}
+
+		if len(summary) > 0 {
+			return pretty.Success(strings.Join(summary, " • ") + "\n")
 		}
 		return pretty.Success("\n✓ Review complete\n")
 	}
···
 	// Header
 	header := lipgloss.JoinHorizontal(
 		lipgloss.Left,
-		titleStyle.Render("Review Snapshots"),
+		titleStyle.Render("Review Snapshots "),
 		counterStyle.Render(fmt.Sprintf("[%d/%d] %s", m.current+1, len(m.snapshots), m.snapshots[m.current])),
 	)
+	headerStyled := statusBarStyle.Width(m.width).Render(header)
 
-	// Footer with help
+	// Footer with scroll info only
 	scrollInfo := fmt.Sprintf("%3.f%%", m.viewport.ScrollPercent()*100)
-	helpText := "↑/↓/scroll: navigate • [a]ccept • [r]eject • [s]kip • [A]ll Accept • [R]ll Reject • [q]uit"
+	scrollStyled := helpStyle.Render(scrollInfo)
 
-	footerLeft := helpStyle.Render(helpText)
-	footerRight := helpStyle.Render(scrollInfo)
+	// Create footer with just scroll info on the right
+	footer := lipgloss.JoinHorizontal(lipgloss.Left,
+		strings.Repeat(" ", max(m.width-lipgloss.Width(scrollStyled)-1, 0)),
+		scrollStyled,
+	)
+	footerStyled := statusBarStyle.Width(m.width).Render(footer)
 
-	gap := max(m.width-lipgloss.Width(footerLeft)-lipgloss.Width(footerRight), 0)
+	// Viewport content
+	viewportContent := m.viewport.View()
 
-	footer := lipgloss.JoinHorizontal(
-		lipgloss.Left,
-		footerLeft,
-		strings.Repeat(" ", gap),
-		footerRight,
-	)
+	// Calculate how much vertical space we have
+	// Total height = terminal height
+	// Used by header = ~1 line (the rendered header)
+	// Used by footer = ~1 line
+	// Middle = viewport (takes remaining space)
 
-	// Main content with viewport
 	return lipgloss.JoinVertical(
 		lipgloss.Left,
-		statusBarStyle.Width(m.width).Render(header),
-		m.viewport.View(),
-		statusBarStyle.Width(m.width).Render(footer),
+		headerStyled,
+		viewportContent,
+		footerStyled,
 	)
+}
+
+func max(a, b int) int {
+	if a > b {
+		return a
+	}
+	return b
 }
 
 func main() {

freeze.go

···
 	"github.com/ptdewey/freeze/internal/review"
 )
 
+const version = "0.1.0"
+
 // TODO: probably make this (and other things) configurable
 func init() {
 	utter.Config.ElideType = true
···
 func snapWithTitle(t testingT, title string, testName string, content string) {
 	t.Helper()
 
-	_, filePath, _, _ := runtime.Caller(2)
-
 	snapshot := &files.Snapshot{
-		Title:    title,
-		Name:     testName,
-		FilePath: filePath,
-		Content:  content,
+		Title:   title,
+		Name:    testName,
+		Content: content,
+		Version: version,
 	}
 
 	accepted, err := files.ReadAccepted(testName)

freeze_test.go

···
 	}
 
 	serialized := snap.Serialize()
-	expected := "---\ntitle: My Test Title\ntest_name: TestExample\nfile_path: \nfunc_name: \n---\ntest content\nmultiline"
+	expected := "---\ntitle: My Test Title\ntest_name: TestExample\nfile_path: \nfunc_name: \nversion: \n---\ntest content\nmultiline"
 	if serialized != expected {
 		t.Errorf("expected:\n%s\ngot:\n%s", expected, serialized)
 	}

internal/files/files.go

···
 	Name     string
 	FilePath string
 	FuncName string
+	Version  string
 	Content  string
 }
 
 func (s *Snapshot) Serialize() string {
-	header := fmt.Sprintf("---\ntitle: %s\ntest_name: %s\nfile_path: %s\nfunc_name: %s\n---\n", s.Title, s.Name, s.FilePath, s.FuncName)
+	header := fmt.Sprintf("---\ntitle: %s\ntest_name: %s\nfile_path: %s\nfunc_name: %s\nversion: %s\n---\n", s.Title, s.Name, s.FilePath, s.FuncName, s.Version)
 	return header + s.Content
 }
 
···
 			snap.FilePath = value
 		case "func_name":
 			snap.FuncName = value
+		case "version":
+			snap.Version = value
 		}
 	}
 

internal/files/files_test.go

···
 		Title:    "Example Title",
 		Name:     "TestExample",
 		FilePath: "/path/to/test.go",
+		Version:  "1.0.0",
 		Content:  "test content\nmultiline",
 	}
 
 	serialized := snap.Serialize()
-	expected := "---\ntitle: Example Title\ntest_name: TestExample\nfile_path: /path/to/test.go\nfunc_name: \n---\ntest content\nmultiline"
+	expected := "---\ntitle: Example Title\ntest_name: TestExample\nfile_path: /path/to/test.go\nfunc_name: \nversion: 1.0.0\n---\ntest content\nmultiline"
 	if serialized != expected {
 		t.Errorf("Serialize():\nexpected:\n%s\n\ngot:\n%s", expected, serialized)
 	}
···
 	}
 	if deserialized.FilePath != snap.FilePath {
 		t.Errorf("FilePath mismatch: %s != %s", deserialized.FilePath, snap.FilePath)
+	}
+	if deserialized.Version != snap.Version {
+		t.Errorf("Version mismatch: %s != %s", deserialized.Version, snap.Version)
 	}
 	if deserialized.Content != snap.Content {
 		t.Errorf("Content mismatch: %s != %s", deserialized.Content, snap.Content)
···
 		input       string
 		wantTitle   string
 		wantTest    string
+		wantVersion string
 		wantContent string
 	}{
 		{
···
 			"---\ntitle: Simple Title\ntest_name: Test\nfile_path: /path\nfunc_name: \n---\ncontent",
 			"Simple Title",
 			"Test",
+			"",
+			"content",
+		},
+		{
+			"with version",
+			"---\ntitle: With Version\ntest_name: Test\nfile_path: /path\nfunc_name: \nversion: 1.0.0\n---\ncontent",
+			"With Version",
+			"Test",
+			"1.0.0",
 			"content",
 		},
 		{
···
 			"---\ntitle: Multi Title\ntest_name: MyTest\nfile_path: /path\nfunc_name: \n---\nline1\nline2\nline3",
 			"Multi Title",
 			"MyTest",
+			"",
 			"line1\nline2\nline3",
 		},
 		{
···
 			"---\ntitle: Extra Title\ntest_name: Test\nfile_path: /path\nfunc_name: \nextra: ignored\n---\ncontent",
 			"Extra Title",
 			"Test",
+			"",
 			"content",
 		},
 		{
···
 			"---\ntest_name: Test\nfile_path: /path\nfunc_name: \n---\ncontent",
 			"",
 			"Test",
+			"",
 			"content",
 		},
 	}
···
 			}
 			if snap.Name != tt.wantTest {
 				t.Errorf("Name = %s, want %s", snap.Name, tt.wantTest)
+			}
+			if snap.Version != tt.wantVersion {
+				t.Errorf("Version = %s, want %s", snap.Version, tt.wantVersion)
 			}
 			if snap.Content != tt.wantContent {
 				t.Errorf("Content = %s, want %s", snap.Content, tt.wantContent)

internal/pretty/boxes.go

···
 	if snap.Title != "" {
 		sb.WriteString(fmt.Sprintf("  title: %s\n", Blue("\""+snap.Title+"\"")))
 	}
-	if isFuncSnapshot && snap.FuncName != "" {
-		sb.WriteString(fmt.Sprintf("  func: %s\n", Blue("\""+snap.FuncName+"\"")))
-		sb.WriteString(fmt.Sprintf("  test: %s\n", Blue("\""+snap.Name+"\"")))
-	} else {
+	if snap.Name != "" {
 		sb.WriteString(fmt.Sprintf("  test: %s\n", Blue("\""+snap.Name+"\"")))
 	}
 	sb.WriteString("\n")

internal/review/review.go

···
 			diffLines := computeDiffLines(accepted, newSnap)
 			fmt.Println(pretty.DiffSnapshotBox(accepted, newSnap, diffLines))
 		} else {
-			if newSnap.FuncName != "" {
-				fmt.Println(pretty.NewSnapshotBoxFunc(newSnap))
-			} else {
-				fmt.Println(pretty.NewSnapshotBox(newSnap))
-			}
+			fmt.Println(pretty.NewSnapshotBox(newSnap))
 		}
 
 		for {