savepoint 1.0.4 → 1.0.7

package/internal/board/release_test.go

@@ -1,177 +0,0 @@
-package board
-
-import (
-	"strings"
-	"testing"
-
-	tea "github.com/charmbracelet/bubbletea"
-	"github.com/opencode/savepoint/internal/data"
-)
-
-func TestRenderReleaseDropdown_showsReleases(t *testing.T) {
-	releases := []string{"v1", "v2", "v3"}
-	out := RenderReleaseDropdown(releases, 0, 40)
-	for _, r := range releases {
-		if !strings.Contains(out, r) {
-			t.Errorf("output missing release %q", r)
-		}
-	}
-}
-
-func TestRenderReleaseDropdown_marksCurrentCursor(t *testing.T) {
-	releases := []string{"v1", "v2"}
-	out := RenderReleaseDropdown(releases, 1, 40)
-	if !strings.Contains(out, releaseActiveMarker) {
-		t.Error("output missing active marker")
-	}
-}
-
-func TestRenderReleaseDropdown_emptyList(t *testing.T) {
-	out := RenderReleaseDropdown(nil, 0, 40)
-	if !strings.Contains(out, "(none)") {
-		t.Error("expected '(none)' for empty releases")
-	}
-}
-
-func TestRenderReleaseDropdown_hintsPresent(t *testing.T) {
-	out := RenderReleaseDropdown([]string{"v1"}, 0, 40)
-	if !strings.Contains(out, "esc:cancel") {
-		t.Error("expected hint text in dropdown")
-	}
-}
-
-func TestSliceIndexRelease_found(t *testing.T) {
-	releases := []string{"v1", "v2", "v3"}
-	if got := sliceIndex(releases, "v2"); got != 1 {
-		t.Errorf("sliceIndex = %d, want 1", got)
-	}
-}
-
-func TestSliceIndexRelease_notFound(t *testing.T) {
-	releases := []string{"v1", "v2"}
-	if got := sliceIndex(releases, "v9"); got != 0 {
-		t.Errorf("sliceIndex = %d, want 0", got)
-	}
-}
-
-func TestUpdate_rOpensReleaseOverlay(t *testing.T) {
-	m := NewModel(nil, "v1", "E03")
-	m.Releases = []string{"v1", "v2"}
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune("r")})
-	updated := requireModel(t, got)
-	if updated.Overlay != OverlayRelease {
-		t.Errorf("Overlay = %q, want %q", updated.Overlay, OverlayRelease)
-	}
-}
-
-func TestUpdate_rSetsCursorToCurrentRelease(t *testing.T) {
-	m := NewModel(nil, "v2", "E03")
-	m.Releases = []string{"v1", "v2", "v3"}
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune("r")})
-	updated := requireModel(t, got)
-	if updated.ReleaseCursor != 1 {
-		t.Errorf("ReleaseCursor = %d, want 1", updated.ReleaseCursor)
-	}
-}
-
-func TestUpdate_escClosesReleaseOverlay(t *testing.T) {
-	m := NewModel(nil, "v1", "E03")
-	m.Overlay = OverlayRelease
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyEsc})
-	updated := requireModel(t, got)
-	if updated.Overlay != OverlayNone {
-		t.Errorf("Overlay = %q, want empty", updated.Overlay)
-	}
-}
-
-func TestUpdate_releaseNavDown(t *testing.T) {
-	m := NewModel(nil, "v1", "E03")
-	m.Releases = []string{"v1", "v2"}
-	m.Overlay = OverlayRelease
-	m.ReleaseCursor = 0
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune("j")})
-	updated := requireModel(t, got)
-	if updated.ReleaseCursor != 1 {
-		t.Errorf("ReleaseCursor = %d, want 1", updated.ReleaseCursor)
-	}
-}
-
-func TestUpdate_releaseNavUp(t *testing.T) {
-	m := NewModel(nil, "v1", "E03")
-	m.Releases = []string{"v1", "v2"}
-	m.Overlay = OverlayRelease
-	m.ReleaseCursor = 1
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune("k")})
-	updated := requireModel(t, got)
-	if updated.ReleaseCursor != 0 {
-		t.Errorf("ReleaseCursor = %d, want 0", updated.ReleaseCursor)
-	}
-}
-
-func TestUpdate_releaseEnterSelects(t *testing.T) {
-	tasks := []data.Task{
-		{ID: "T1", Epic: "E03", Release: "v1", Column: data.ColumnPlanned},
-		{ID: "T2", Epic: "E03", Release: "v2", Column: data.ColumnPlanned},
-	}
-	m := NewModel(tasks, "v1", "E03")
-	m.Releases = []string{"v1", "v2"}
-	m.Overlay = OverlayRelease
-	m.ReleaseCursor = 1
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyEnter})
-	updated := requireModel(t, got)
-	if updated.SelectedRelease != "v2" {
-		t.Errorf("SelectedRelease = %q, want %q", updated.SelectedRelease, "v2")
-	}
-	if updated.Overlay != OverlayNone {
-		t.Errorf("Overlay = %q, want empty after selection", updated.Overlay)
-	}
-	if got := len(updated.Tasks[data.ColumnPlanned]); got != 1 {
-		t.Errorf("planned task count = %d, want 1 after release selection", got)
-	}
-	if updated.Tasks[data.ColumnPlanned][0].ID != "T2" {
-		t.Errorf("visible task = %q, want T2", updated.Tasks[data.ColumnPlanned][0].ID)
-	}
-}
-
-func TestUpdate_releaseEnterRefreshesEpics(t *testing.T) {
-	tasks := []data.Task{
-		{ID: "T1", Epic: "E01", Release: "v1", Column: data.ColumnPlanned},
-		{ID: "T2", Epic: "E03", Release: "v2", Column: data.ColumnPlanned},
-	}
-	m := NewModel(tasks, "v1", "E01")
-	m.Releases = []string{"v1", "v2"}
-	m.ReleaseEpics = map[string][]string{
-		"v1": []string{"E01"},
-		"v2": []string{"E03"},
-	}
-	m.Overlay = OverlayRelease
-	m.ReleaseCursor = 1
-
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyEnter})
-	updated := requireModel(t, got)
-
-	if updated.SelectedEpic != "E03" {
-		t.Errorf("SelectedEpic = %q, want E03", updated.SelectedEpic)
-	}
-	if len(updated.Epics) != 1 || updated.Epics[0] != "E03" {
-		t.Errorf("Epics = %v, want [E03]", updated.Epics)
-	}
-	if got := len(updated.Tasks[data.ColumnPlanned]); got != 1 {
-		t.Fatalf("planned task count = %d, want 1", got)
-	}
-	if updated.Tasks[data.ColumnPlanned][0].ID != "T2" {
-		t.Errorf("visible task = %q, want T2", updated.Tasks[data.ColumnPlanned][0].ID)
-	}
-}
-
-func TestView_releaseDropdownKeepsBoardBehind(t *testing.T) {
-	m := NewModel(nil, "v1", "E03")
-	m.Width = 100
-	m.Height = 24
-	m.Overlay = OverlayRelease
-	m.Releases = []string{"v1", "v2"}
-	got := m.View()
-	if !strings.Contains(got, "S A V E P O I N T") {
-		t.Error("View() with OverlayRelease should keep board visible behind overlay")
-	}
-}

package/internal/board/render_policy_test.go

@@ -1,77 +0,0 @@
-package board
-
-import (
-	"strings"
-	"testing"
-
-	"github.com/charmbracelet/lipgloss"
-	"github.com/muesli/termenv"
-	"github.com/opencode/savepoint/internal/data"
-)
-
-func TestRenderPolicy_noBackgroundEscapes(t *testing.T) {
-	lipgloss.SetColorProfile(termenv.ANSI256)
-
-	task := data.Task{
-		ID:      "E05-tasking-permissions/T004-implement-m-hotkey",
-		Title:   "Implement router priority",
-		Release: "v1.1",
-		Epic:    "E05-tasking-permissions",
-		Column:  data.ColumnInProgress,
-		Stage:   data.StageBuild,
-	}
-	m := NewModel([]data.Task{
-		{
-			ID:      task.ID,
-			Title:   task.Title,
-			Release: task.Release,
-			Epic:    task.Epic,
-			Column:  task.Column,
-			Stage:   task.Stage,
-		},
-	}, "v1.1", "E05-tasking-permissions")
-	m.Width = 120
-	m.Height = 30
-	m.FocusedColumn = data.ColumnInProgress
-	m.RouterState = &data.RouterState{
-		State:      "task-building",
-		Release:    "v1.1",
-		Epic:       "E05-tasking-permissions",
-		Task:       "E05-tasking-permissions/T004-implement-m-hotkey",
-		NextAction: "Build E05-tasking-permissions/T004-implement-m-hotkey.",
-	}
-
-	cases := map[string]string{
-		"board":            m.View(),
-		"card":             RenderCard(task, 30, true, m.RouterState),
-		"detail":           RenderDetail(task, 60, m.RouterState, 0, 0),
-		"epic dropdown":    RenderEpicDropdown([]string{"E05-tasking-permissions"}, 0, 40),
-		"release dropdown": RenderReleaseDropdown([]string{"v1.1"}, 0, 40),
-		"help":             RenderHelp(60),
-	}
-	for name, got := range cases {
-		assertNoBackgroundEscapes(t, name, got)
-	}
-}
-
-func assertNoBackgroundEscapes(t *testing.T, name, got string) {
-	t.Helper()
-	for _, escape := range []string{"\x1b[48;", "\x1b[40m"} {
-		if strings.Contains(got, escape) {
-			t.Fatalf("%s emitted background escape prefix %q in %q", name, escape, got)
-		}
-	}
-}
-
-func TestRenderPolicy_usesSingleLineBorders(t *testing.T) {
-	m := NewModel([]data.Task{{ID: "T001", Title: "Task", Column: data.ColumnPlanned}}, "v1", "E01")
-	m.Width = 120
-	got := m.View()
-
-	if strings.Contains(got, "╭") || strings.Contains(got, "╮") || strings.Contains(got, "╰") || strings.Contains(got, "╯") {
-		t.Fatalf("View should use single-line borders, got rounded border glyphs in %q", got)
-	}
-	if !strings.Contains(got, "┌") || !strings.Contains(got, "┐") || !strings.Contains(got, "└") || !strings.Contains(got, "┘") {
-		t.Fatalf("View missing expected single-line border glyphs in %q", got)
-	}
-}

package/internal/board/status.go

@@ -1,23 +0,0 @@
-package board
-
-import (
-	"github.com/opencode/savepoint/internal/data"
-	"github.com/opencode/savepoint/internal/styles"
-)
-
-const statusGlyphDefault = " "
-
-func statusGlyph(status string) string {
-	switch status {
-	case string(data.ColumnPlanned):
-		return styles.CardMeta.Render("○")
-	case string(data.ColumnInProgress):
-		return styles.GlyphBuild.Render("▶")
-	case string(data.ColumnDone):
-		return styles.TagDone.Render("◉")
-	case "audited":
-		return styles.TagDone.Render("✓")
-	default:
-		return statusGlyphDefault
-	}
-}

package/internal/board/theme.go

@@ -1,24 +0,0 @@
-package board
-
-import (
-	"os"
-
-	"github.com/charmbracelet/lipgloss"
-	"github.com/muesli/termenv"
-)
-
-// applyColorProfile detects terminal color support and configures lipgloss.
-// Priority: NO_COLOR → COLORTERM env hint → termenv auto-detect.
-func applyColorProfile() {
-	lipgloss.SetColorProfile(detectProfile())
-}
-
-func detectProfile() termenv.Profile {
-	if os.Getenv("NO_COLOR") != "" {
-		return termenv.Ascii
-	}
-	if c := os.Getenv("COLORTERM"); c == "truecolor" || c == "24bit" {
-		return termenv.TrueColor
-	}
-	return termenv.ColorProfile()
-}

package/internal/board/theme_test.go

@@ -1,31 +0,0 @@
-package board
-
-import (
-	"testing"
-
-	"github.com/muesli/termenv"
-)
-
-func TestDetectProfileNOCOLOR(t *testing.T) {
-	t.Setenv("NO_COLOR", "1")
-	t.Setenv("COLORTERM", "")
-	if got := detectProfile(); got != termenv.Ascii {
-		t.Fatalf("want Ascii, got %v", got)
-	}
-}
-
-func TestDetectProfileTruecolor(t *testing.T) {
-	t.Setenv("NO_COLOR", "")
-	t.Setenv("COLORTERM", "truecolor")
-	if got := detectProfile(); got != termenv.TrueColor {
-		t.Fatalf("want TrueColor, got %v", got)
-	}
-}
-
-func TestDetectProfile24bit(t *testing.T) {
-	t.Setenv("NO_COLOR", "")
-	t.Setenv("COLORTERM", "24bit")
-	if got := detectProfile(); got != termenv.TrueColor {
-		t.Fatalf("want TrueColor, got %v", got)
-	}
-}

package/internal/board/transitions.go

@@ -1,113 +0,0 @@
-package board
-
-import (
-	"fmt"
-
-	"github.com/opencode/savepoint/internal/data"
-)
-
-// Advance moves a task forward through the phase lifecycle.
-func Advance(t *data.Task) {
-	stage := t.Stage
-	if stage == "" {
-		stage = data.StageBuild
-	}
-	switch t.Column {
-	case data.ColumnPlanned:
-		t.Column = data.ColumnInProgress
-		t.Stage = data.StageBuild
-	case data.ColumnInProgress:
-		switch stage {
-		case data.StageBuild:
-			t.Stage = data.StageTest
-		case data.StageTest:
-			t.Stage = data.StageAudit
-		case data.StageAudit:
-			t.Column = data.ColumnDone
-			t.Stage = ""
-		}
-	}
-	t.Status = string(t.Column)
-}
-
-// Retreat moves a task backward through the phase lifecycle.
-func Retreat(t *data.Task) {
-	stage := t.Stage
-	if stage == "" {
-		stage = data.StageBuild
-	}
-	switch t.Column {
-	case data.ColumnDone:
-		t.Column = data.ColumnInProgress
-		t.Stage = data.StageAudit
-	case data.ColumnInProgress:
-		switch stage {
-		case data.StageAudit:
-			t.Stage = data.StageTest
-		case data.StageTest:
-			t.Stage = data.StageBuild
-		case data.StageBuild:
-			t.Column = data.ColumnPlanned
-			t.Stage = ""
-		}
-	}
-	t.Status = string(t.Column)
-}
-
-func taskTransitionMessage(prefix string, task data.Task) string {
-	if task.Column == data.ColumnInProgress {
-		return fmt.Sprintf("%s %s to %s", prefix, shortID(task.ID), task.Stage)
-	}
-	return fmt.Sprintf("%s %s to %s", prefix, shortID(task.ID), task.Column)
-}
-
-// CanAdvance checks whether a task is allowed to advance to its next phase.
-// It validates phase adjacency and dependency completion.
-// Returns (true, "") if allowed, or (false, reason) if blocked.
-func CanAdvance(t *data.Task, allTasks []data.Task) (bool, string) {
-	switch t.Column {
-	case data.ColumnPlanned:
-		return dependenciesDone(t, allTasks)
-	case data.ColumnInProgress:
-		stage := t.Stage
-		if stage == "" {
-			stage = data.StageBuild
-		}
-		switch stage {
-		case data.StageBuild:
-			return true, ""
-		case data.StageTest:
-			return true, ""
-		case data.StageAudit:
-			return dependenciesDone(t, allTasks)
-		default:
-			return false, fmt.Sprintf("unknown stage %q", stage)
-		}
-	case data.ColumnDone:
-		return false, "task is already done"
-	default:
-		return false, fmt.Sprintf("unknown column %q", t.Column)
-	}
-}
-
-func dependenciesDone(t *data.Task, allTasks []data.Task) (bool, string) {
-	for _, depID := range t.DependsOn {
-		dep := findTask(depID, allTasks)
-		if dep == nil {
-			return false, fmt.Sprintf("dependency %q not found", depID)
-		}
-		if dep.Column != data.ColumnDone {
-			return false, fmt.Sprintf("dependency %q is not done", depID)
-		}
-	}
-	return true, ""
-}
-
-func findTask(id string, tasks []data.Task) *data.Task {
-	for i := range tasks {
-		if tasks[i].ID == id {
-			return &tasks[i]
-		}
-	}
-	return nil
-}

package/internal/board/transitions_test.go

@@ -1,164 +0,0 @@
-package board
-
-import (
-	"testing"
-
-	"github.com/opencode/savepoint/internal/data"
-)
-
-func TestAdvance(t *testing.T) {
-	tests := []struct {
-		name       string
-		initialCol data.ColumnType
-		initialSt  data.ProgressStage
-		expectCol  data.ColumnType
-		expectSt   data.ProgressStage
-	}{
-		{"planned to in_progress/build", data.ColumnPlanned, "", data.ColumnInProgress, data.StageBuild},
-		{"in_progress/build to test", data.ColumnInProgress, data.StageBuild, data.ColumnInProgress, data.StageTest},
-		{"in_progress/test to audit", data.ColumnInProgress, data.StageTest, data.ColumnInProgress, data.StageAudit},
-		{"in_progress/audit to done", data.ColumnInProgress, data.StageAudit, data.ColumnDone, ""},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			task := data.Task{Column: tt.initialCol, Stage: tt.initialSt}
-			Advance(&task)
-			if task.Column != tt.expectCol || task.Stage != tt.expectSt {
-				t.Errorf("Advance() = %v/%v, want %v/%v", task.Column, task.Stage, tt.expectCol, tt.expectSt)
-			}
-			if task.Status != string(tt.expectCol) {
-				t.Errorf("Advance() status = %q, want %q", task.Status, tt.expectCol)
-			}
-		})
-	}
-}
-
-func TestRetreat(t *testing.T) {
-	tests := []struct {
-		name       string
-		initialCol data.ColumnType
-		initialSt  data.ProgressStage
-		expectCol  data.ColumnType
-		expectSt   data.ProgressStage
-	}{
-		{"done to in_progress/audit", data.ColumnDone, "", data.ColumnInProgress, data.StageAudit},
-		{"in_progress/audit to test", data.ColumnInProgress, data.StageAudit, data.ColumnInProgress, data.StageTest},
-		{"in_progress/test to build", data.ColumnInProgress, data.StageTest, data.ColumnInProgress, data.StageBuild},
-		{"in_progress/build to planned", data.ColumnInProgress, data.StageBuild, data.ColumnPlanned, ""},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			task := data.Task{Status: string(tt.initialCol), Column: tt.initialCol, Stage: tt.initialSt}
-			Retreat(&task)
-			if task.Column != tt.expectCol || task.Stage != tt.expectSt {
-				t.Errorf("Retreat() = %v/%v, want %v/%v", task.Column, task.Stage, tt.expectCol, tt.expectSt)
-			}
-			if task.Status != string(tt.expectCol) {
-				t.Errorf("Retreat() status = %q, want %q", task.Status, tt.expectCol)
-			}
-		})
-	}
-}
-
-func TestCanAdvance_plannedAllowedWhenDependenciesDone(t *testing.T) {
-	allTasks := []data.Task{
-		{ID: "T1", Column: data.ColumnPlanned, DependsOn: []string{"T2"}},
-		{ID: "T2", Column: data.ColumnDone},
-	}
-	ok, reason := CanAdvance(&allTasks[0], allTasks)
-	if !ok {
-		t.Errorf("CanAdvance(planned with done dep) = false %q, want true", reason)
-	}
-}
-
-func TestCanAdvance_plannedBlockedByDependency(t *testing.T) {
-	allTasks := []data.Task{
-		{ID: "T1", Column: data.ColumnPlanned, DependsOn: []string{"T2"}},
-		{ID: "T2", Column: data.ColumnInProgress},
-	}
-	ok, reason := CanAdvance(&allTasks[0], allTasks)
-	if ok {
-		t.Fatal("CanAdvance(planned with unfinished dep) = true, want false")
-	}
-	if reason != "dependency \"T2\" is not done" {
-		t.Errorf("reason = %q, want dependency warning", reason)
-	}
-}
-
-func TestCanAdvance_buildAlwaysAllowed(t *testing.T) {
-	task := data.Task{ID: "T1", Column: data.ColumnInProgress, Stage: data.StageBuild}
-	ok, reason := CanAdvance(&task, nil)
-	if !ok {
-		t.Errorf("CanAdvance(build) = false %q, want true", reason)
-	}
-}
-
-func TestCanAdvance_testAlwaysAllowed(t *testing.T) {
-	task := data.Task{ID: "T1", Column: data.ColumnInProgress, Stage: data.StageTest}
-	ok, reason := CanAdvance(&task, nil)
-	if !ok {
-		t.Errorf("CanAdvance(test) = false %q, want true", reason)
-	}
-}
-
-func TestCanAdvance_auditDoneBlockedByDependency(t *testing.T) {
-	allTasks := []data.Task{
-		{ID: "T1", Column: data.ColumnInProgress, Stage: data.StageAudit, DependsOn: []string{"T2"}},
-		{ID: "T2", Column: data.ColumnInProgress, Stage: data.StageBuild},
-	}
-	ok, reason := CanAdvance(&allTasks[0], allTasks)
-	if ok {
-		t.Fatal("CanAdvance(audit with undep) = true, want false")
-	}
-	if reason == "" {
-		t.Fatal("expected non-empty reason string")
-	}
-}
-
-func TestCanAdvance_auditDoneAllowedWhenDepsDone(t *testing.T) {
-	allTasks := []data.Task{
-		{ID: "T1", Column: data.ColumnInProgress, Stage: data.StageAudit, DependsOn: []string{"T2"}},
-		{ID: "T2", Column: data.ColumnDone},
-	}
-	ok, reason := CanAdvance(&allTasks[0], allTasks)
-	if !ok {
-		t.Errorf("CanAdvance(audit with dep done) = false %q, want true", reason)
-	}
-}
-
-func TestCanAdvance_doneBlocked(t *testing.T) {
-	task := data.Task{ID: "T1", Column: data.ColumnDone}
-	ok, reason := CanAdvance(&task, nil)
-	if ok {
-		t.Fatal("CanAdvance(done) = true, want false")
-	}
-	if reason == "" {
-		t.Fatal("expected non-empty reason string")
-	}
-}
-
-func TestCanAdvance_unknownStageBlocked(t *testing.T) {
-	task := data.Task{ID: "T1", Column: data.ColumnInProgress, Stage: "invalid"}
-	ok, reason := CanAdvance(&task, nil)
-	if ok {
-		t.Fatal("CanAdvance(unknown stage) = true, want false")
-	}
-	if reason == "" {
-		t.Fatal("expected non-empty reason string")
-	}
-}
-
-func TestCanAdvance_auditDepsNotFoundBlocked(t *testing.T) {
-	allTasks := []data.Task{
-		{ID: "T1", Column: data.ColumnInProgress, Stage: data.StageAudit, DependsOn: []string{"T2"}},
-	}
-	ok, reason := CanAdvance(&allTasks[0], allTasks)
-	if ok {
-		t.Fatal("CanAdvance(audit missing dep) = true, want false")
-	}
-	if reason == "" {
-		t.Fatal("expected non-empty reason string")
-	}
-}

package/internal/board/tui.go

@@ -1,32 +0,0 @@
-package board
-
-import (
-	"fmt"
-	"os"
-	"path/filepath"
-
-	tea "github.com/charmbracelet/bubbletea"
-)
-
-// RunTUI launches the Bubble Tea board with optional release/epic filters.
-func RunTUI(release, epic string) error {
-	model, err := newProjectModel(".", release, epic)
-	if err != nil {
-		return err
-	}
-
-	cfg, err := model.Dependencies.ConfigReader.Read(filepath.Join(model.Root, "config.yml"))
-	if err != nil {
-		return err
-	}
-	model.Theme = cfg.Theme
-
-	applyColorProfile()
-
-	p := tea.NewProgram(model, tea.WithAltScreen())
-	if _, err := p.Run(); err != nil {
-		fmt.Fprintf(os.Stderr, "Error: %v\n", err)
-		return err
-	}
-	return nil
-}