savepoint 1.0.4 → 1.0.6

package/internal/board/debug_test.go

@@ -1,108 +0,0 @@
-package board
-
-import (
-	"bytes"
-	"fmt"
-	"io"
-	"os"
-	"strings"
-	"testing"
-)
-
-func TestSetDebugToggle(t *testing.T) {
-	t.Cleanup(func() { SetDebug(false) })
-
-	if DebugEnabled() {
-		t.Fatal("debug should be off by default")
-	}
-
-	SetDebug(true)
-	if !DebugEnabled() {
-		t.Fatal("debug should be on after SetDebug(true)")
-	}
-
-	SetDebug(false)
-	if DebugEnabled() {
-		t.Fatal("debug should be off after SetDebug(false)")
-	}
-}
-
-func TestDebugfWritesToStderr(t *testing.T) {
-	t.Cleanup(func() { SetDebug(false) })
-
-	r, w, err := os.Pipe()
-	if err != nil {
-		t.Fatal(err)
-	}
-	orig := os.Stderr
-	os.Stderr = w
-
-	SetDebug(true)
-	debugf("test message %d", 42)
-
-	os.Stderr = orig
-	w.Close()
-
-	var buf bytes.Buffer
-	io.Copy(&buf, r)
-	r.Close()
-
-	out := buf.String()
-	if !strings.Contains(out, "test message 42") {
-		t.Fatalf("expected debug output, got: %q", out)
-	}
-	if !strings.Contains(out, "[savepoint debug]") {
-		t.Fatalf("expected debug prefix, got: %q", out)
-	}
-}
-
-func TestDebugfSilentWhenDisabled(t *testing.T) {
-	t.Cleanup(func() { SetDebug(false) })
-
-	r, w, err := os.Pipe()
-	if err != nil {
-		t.Fatal(err)
-	}
-	orig := os.Stderr
-	os.Stderr = w
-
-	SetDebug(false)
-	debugf("should not appear")
-
-	os.Stderr = orig
-	w.Close()
-
-	var buf bytes.Buffer
-	io.Copy(&buf, r)
-	r.Close()
-
-	if buf.Len() != 0 {
-		t.Fatalf("expected no debug output when disabled, got: %q", buf.String())
-	}
-}
-
-func TestDebugfFormat(t *testing.T) {
-	t.Cleanup(func() { SetDebug(false) })
-
-	r, w, err := os.Pipe()
-	if err != nil {
-		t.Fatal(err)
-	}
-	orig := os.Stderr
-	os.Stderr = w
-
-	SetDebug(true)
-	debugf("key=%q value=%d", "hello", 7)
-
-	os.Stderr = orig
-	w.Close()
-
-	var buf bytes.Buffer
-	io.Copy(&buf, r)
-	r.Close()
-
-	want := fmt.Sprintf("[savepoint debug] key=%q value=%d\n", "hello", 7)
-	if buf.String() != want {
-		t.Fatalf("expected %q, got %q", want, buf.String())
-	}
-}

package/internal/board/detail.go

@@ -1,218 +0,0 @@
-package board
-
-import (
-	"strings"
-
-	"github.com/charmbracelet/lipgloss"
-	"github.com/opencode/savepoint/internal/data"
-	"github.com/opencode/savepoint/internal/styles"
-)
-
-const detailBorderPad = 4        // rounded border (2) + padding (2×1)
-const detailVerticalOverhead = 4 // overlay border (2) + fixed title/header rows (2)
-
-// RenderDetail renders a task detail overlay panel at the given display width.
-// When router state matches t's release/epic/task, a "(router priority)" label is shown.
-func RenderDetail(t data.Task, overlayW int, routerState *data.RouterState, maxHeight, offset int) string {
-	inner := overlayW - detailBorderPad
-	if inner < 4 {
-		inner = 4
-	}
-
-	lines := []string{
-		styles.ColumnTitleFocused.Render("TASK DETAIL"),
-		strings.Repeat("─", inner),
-	}
-	body := []string{
-		detailRow("ID", t.ID, inner),
-		detailRow("Title", t.Title, inner),
-		detailRow("Epic", t.Epic, inner),
-		detailRow("Release", t.Release, inner),
-		detailRow("Status", string(t.Column), inner),
-		detailRow("Phase", phaseLabel(t.Stage), inner),
-	}
-
-	if t.Description != "" {
-		body = append(body,
-			"",
-			styles.ColumnTitle.Render("Description:"),
-		)
-		for _, line := range WrapText(t.Description, inner) {
-			body = append(body, styles.CardMeta.Render(line))
-		}
-	}
-
-	if len(t.Checklist) > 0 {
-		body = append(body, "", styles.ColumnTitle.Render("Implementation Plan:"), "")
-		for _, item := range t.Checklist {
-			glyph := "[ ] "
-			style := styles.CardMeta
-			if item.Done {
-				glyph = "[x] "
-				style = styles.TagDone
-			}
-			body = append(body, renderChecklistSentences(item.Text, glyph, inner, style)...)
-		}
-	}
-
-	if t.Column != data.ColumnDone && isRouterPriority(t, routerState) {
-		body = append(body, "", styles.TagDone.Render("(router priority)"))
-	}
-	body = append(body, "", styles.CardMeta.Render("esc:close"))
-	lines = append(lines, visibleDetailLines(body, maxHeight-detailVerticalOverhead, offset)...)
-
-	return styles.DetailOverlay.Width(overlayW).Render(strings.Join(lines, "\n"))
-}
-
-func renderChecklistSentences(text, glyph string, width int, style lipgloss.Style) []string {
-	textWidth := width - len(glyph)
-	if textWidth < 4 {
-		textWidth = 4
-	}
-
-	lines := []string{}
-	continuationIndent := strings.Repeat(" ", len(glyph))
-	for _, sentence := range splitChecklistSentences(text) {
-		wrapped := WrapText(sentence, textWidth)
-		for i, line := range wrapped {
-			if i == 0 {
-				lines = append(lines, style.Render(glyph+line))
-				continue
-			}
-			lines = append(lines, style.Render(continuationIndent+line))
-		}
-	}
-	return lines
-}
-
-// knownAbbreviations is the set of dot-terminated tokens that must not trigger
-// sentence splits. Add entries (lowercase, trailing dot) to extend the list.
-var knownAbbreviations = map[string]bool{
-	"e.g.":    true,
-	"i.e.":    true,
-	"vs.":     true,
-	"etc.":    true,
-	"fig.":    true,
-	"no.":     true,
-	"mr.":     true,
-	"mrs.":    true,
-	"dr.":     true,
-	"st.":     true,
-	"jr.":     true,
-	"sr.":     true,
-	"prof.":   true,
-	"approx.": true,
-	"est.":    true,
-}
-
-// isKnownAbbreviation reports whether the period at dotPos in s is the trailing
-// dot of a known abbreviation (e.g. "e.g.", "Dr.").
-func isKnownAbbreviation(s string, dotPos int) bool {
-	start := dotPos
-	for start > 0 && s[start-1] != ' ' {
-		start--
-	}
-	return knownAbbreviations[strings.ToLower(s[start:dotPos+1])]
-}
-
-func splitChecklistSentences(text string) []string {
-	fields := strings.Fields(text)
-	if len(fields) == 0 {
-		return nil
-	}
-	normalized := strings.Join(fields, " ")
-
-	sentences := []string{}
-	start := 0
-	for i, r := range normalized {
-		if r != '.' && r != '!' && r != '?' {
-			continue
-		}
-		if r == '.' && isKnownAbbreviation(normalized, i) {
-			continue
-		}
-		end := i + len(string(r))
-		if end < len(normalized) && normalized[end] != ' ' {
-			continue
-		}
-		sentence := strings.TrimSpace(normalized[start:end])
-		if sentence != "" {
-			sentences = append(sentences, sentence)
-		}
-		start = end
-	}
-	if tail := strings.TrimSpace(normalized[start:]); tail != "" {
-		sentences = append(sentences, tail)
-	}
-	return sentences
-}
-
-func visibleDetailLines(lines []string, maxBodyHeight, offset int) []string {
-	total := len(lines)
-	if maxBodyHeight <= 0 || total <= maxBodyHeight {
-		return lines
-	}
-	offset = clampDetailOffset(offset, total)
-	available := maxBodyHeight
-	if offset > 0 {
-		available--
-	}
-	if available < 1 {
-		available = 1
-	}
-	end := min(offset+available, total)
-	if end < total && available > 1 {
-		available--
-		end = min(offset+available, total)
-	}
-
-	visible := make([]string, 0, available+2)
-	if offset > 0 {
-		visible = append(visible, renderScrollIndicator("↑", offset, "above"))
-	}
-	visible = append(visible, lines[offset:end]...)
-	if end < total {
-		visible = append(visible, renderScrollIndicator("↓", total-end, "more"))
-	}
-	return visible
-}
-
-func clampDetailOffset(offset, total int) int {
-	if offset < 0 || total <= 0 {
-		return 0
-	}
-	if offset >= total {
-		return total - 1
-	}
-	return offset
-}
-
-func detailRow(label, value string, width int) string {
-	prefix := label + ": "
-	wrapped := WrapText(value, width-len(prefix))
-	if len(wrapped) == 0 {
-		wrapped = []string{""}
-	}
-	lines := make([]string, 0, len(wrapped))
-	for i, line := range wrapped {
-		if i == 0 {
-			lines = append(lines, styles.ColumnTitle.Render(prefix)+styles.CardMeta.Render(line))
-			continue
-		}
-		lines = append(lines, strings.Repeat(" ", len(prefix))+styles.CardMeta.Render(line))
-	}
-	return strings.Join(lines, "\n")
-}
-
-func phaseLabel(s data.ProgressStage) string {
-	switch s {
-	case data.StageTest:
-		return "test"
-	case data.StageAudit:
-		return "audit"
-	default:
-		return "build"
-	}
-}
-
-

package/internal/board/detail_test.go

@@ -1,388 +0,0 @@
-package board
-
-import (
-	"regexp"
-	"strings"
-	"testing"
-
-	tea "github.com/charmbracelet/bubbletea"
-	"github.com/opencode/savepoint/internal/data"
-)
-
-var ansiPattern = regexp.MustCompile(`\x1b\[[0-9;?]*[ -/]*[@-~]`)
-
-func plainTerminal(s string) string {
-	return ansiPattern.ReplaceAllString(s, "")
-}
-
-func sampleTask() data.Task {
-	return data.Task{
-		ID:      "E04/T001",
-		Title:   "My Task",
-		Epic:    "E04-board-components",
-		Release: "v1",
-		Column:  data.ColumnInProgress,
-		Stage:   data.StageBuild,
-	}
-}
-
-func TestRenderDetail_containsID(t *testing.T) {
-	got := RenderDetail(sampleTask(), 60, nil, 0, 0)
-	if !strings.Contains(got, "E04/T001") {
-		t.Error("RenderDetail missing task ID")
-	}
-}
-
-func TestRenderDetail_containsTitle(t *testing.T) {
-	got := RenderDetail(sampleTask(), 60, nil, 0, 0)
-	if !strings.Contains(got, "My Task") {
-		t.Error("RenderDetail missing task title")
-	}
-}
-
-func TestRenderDetail_containsEpic(t *testing.T) {
-	got := RenderDetail(sampleTask(), 60, nil, 0, 0)
-	if !strings.Contains(got, "E04-board-components") {
-		t.Error("RenderDetail missing epic")
-	}
-}
-
-func TestRenderDetail_containsRelease(t *testing.T) {
-	got := RenderDetail(sampleTask(), 60, nil, 0, 0)
-	if !strings.Contains(got, "v1") {
-		t.Error("RenderDetail missing release")
-	}
-}
-
-func TestRenderDetail_containsStatus(t *testing.T) {
-	got := RenderDetail(sampleTask(), 60, nil, 0, 0)
-	if !strings.Contains(got, "in_progress") {
-		t.Error("RenderDetail missing status")
-	}
-}
-
-func TestRenderDetail_containsPhase(t *testing.T) {
-	got := RenderDetail(sampleTask(), 60, nil, 0, 0)
-	if !strings.Contains(got, "build") {
-		t.Error("RenderDetail missing phase")
-	}
-}
-
-func TestRenderDetail_containsEscHint(t *testing.T) {
-	got := RenderDetail(sampleTask(), 60, nil, 0, 0)
-	if !strings.Contains(got, "esc") {
-		t.Error("RenderDetail missing esc:close hint")
-	}
-}
-
-func TestRenderDetail_containsDescription(t *testing.T) {
-	tk := sampleTask()
-	tk.Description = "some description text"
-	got := RenderDetail(tk, 60, nil, 0, 0)
-	if !strings.Contains(got, "some description text") {
-		t.Error("RenderDetail missing description text")
-	}
-}
-
-func TestRenderDetail_noDescriptionSectionWhenEmpty(t *testing.T) {
-	got := RenderDetail(sampleTask(), 60, nil, 0, 0)
-	if strings.Contains(got, "Description:") {
-		t.Error("RenderDetail should not show Description section when empty")
-	}
-}
-
-func TestRenderDetail_containsChecklist(t *testing.T) {
-	tk := sampleTask()
-	tk.Checklist = []data.CheckItem{{Text: "first implementation item"}, {Text: "second implementation item", Done: true}}
-	got := RenderDetail(tk, 60, nil, 0, 0)
-	if !strings.Contains(got, "Implementation Plan:") {
-		t.Error("RenderDetail missing implementation plan heading")
-	}
-	if !strings.Contains(got, "first implementation item") {
-		t.Error("RenderDetail missing first checklist item")
-	}
-	if !strings.Contains(got, "second implementation item") {
-		t.Error("RenderDetail missing second checklist item")
-	}
-}
-
-func TestRenderDetail_checklistSingleSentenceGetsOneCheckbox(t *testing.T) {
-	tk := sampleTask()
-	tk.Checklist = []data.CheckItem{{Text: "single sentence task"}}
-
-	got := plainTerminal(RenderDetail(tk, 60, nil, 0, 0))
-
-	if count := strings.Count(got, "[ ]"); count != 1 {
-		t.Fatalf("RenderDetail checkbox count = %d, want 1\n%s", count, got)
-	}
-	if strings.Contains(got, "[x]") {
-		t.Fatal("RenderDetail should not render checked marker for unchecked item")
-	}
-}
-
-func TestRenderDetail_checklistMultiSentenceGetsOneCheckboxPerSentence(t *testing.T) {
-	tk := sampleTask()
-	tk.Checklist = []data.CheckItem{{Text: "First sentence. Second sentence! Third sentence?"}}
-
-	got := plainTerminal(RenderDetail(tk, 60, nil, 0, 0))
-
-	if count := strings.Count(got, "[ ]"); count != 3 {
-		t.Fatalf("RenderDetail checkbox count = %d, want 3\n%s", count, got)
-	}
-	for _, want := range []string{"[ ] First sentence.", "[ ] Second sentence!", "[ ] Third sentence?"} {
-		if !strings.Contains(got, want) {
-			t.Fatalf("RenderDetail missing sentence checkbox line %q\n%s", want, got)
-		}
-	}
-}
-
-func TestRenderDetail_checklistHardWrappedSentenceDoesNotDuplicateCheckbox(t *testing.T) {
-	tk := sampleTask()
-	tk.Checklist = []data.CheckItem{{
-		Text: "This sentence is intentionally long enough to wrap inside a narrow detail overlay while remaining one semantic sentence.",
-	}}
-
-	got := plainTerminal(RenderDetail(tk, 34, nil, 0, 0))
-
-	if count := strings.Count(got, "[ ]"); count != 1 {
-		t.Fatalf("RenderDetail checkbox count = %d, want 1\n%s", count, got)
-	}
-	if !strings.Contains(got, "    intentionally long enough") {
-		t.Fatalf("RenderDetail continuation line should align under checkbox text\n%s", got)
-	}
-}
-
-func TestRenderDetail_checklistCheckedSentenceUsesCheckedMarker(t *testing.T) {
-	tk := sampleTask()
-	tk.Checklist = []data.CheckItem{{Text: "already done. still done.", Done: true}}
-
-	got := plainTerminal(RenderDetail(tk, 60, nil, 0, 0))
-
-	if count := strings.Count(got, "[x]"); count != 2 {
-		t.Fatalf("RenderDetail checked checkbox count = %d, want 2\n%s", count, got)
-	}
-	if strings.Contains(got, "[ ]") {
-		t.Fatal("RenderDetail should not render unchecked marker for checked item")
-	}
-}
-
-func TestRenderDetail_wrapsLongDescription(t *testing.T) {
-	tk := sampleTask()
-	tk.Description = "alpha beta gamma delta epsilon zeta eta theta iota kappa lambda"
-	got := RenderDetail(tk, 30, nil, 0, 0)
-	if strings.Contains(got, tk.Description) {
-		t.Error("RenderDetail should wrap long description text")
-	}
-	if !strings.Contains(got, "alpha beta") || !strings.Contains(got, "lambda") {
-		t.Error("RenderDetail should preserve wrapped description words")
-	}
-}
-
-func TestRenderDetail_noAcceptanceSectionWhenEmpty(t *testing.T) {
-	got := RenderDetail(sampleTask(), 60, nil, 0, 0)
-	if strings.Contains(got, "Acceptance Criteria:") {
-		t.Error("RenderDetail should not show Acceptance section when empty")
-	}
-}
-
-func TestPhaseLabel_build(t *testing.T) {
-	if got := phaseLabel(data.StageBuild); got != "build" {
-		t.Errorf("phaseLabel(StageBuild) = %q, want %q", got, "build")
-	}
-}
-
-func TestPhaseLabel_test(t *testing.T) {
-	if got := phaseLabel(data.StageTest); got != "test" {
-		t.Errorf("phaseLabel(StageTest) = %q, want %q", got, "test")
-	}
-}
-
-func TestPhaseLabel_audit(t *testing.T) {
-	if got := phaseLabel(data.StageAudit); got != "audit" {
-		t.Errorf("phaseLabel(StageAudit) = %q, want %q", got, "audit")
-	}
-}
-
-func TestPhaseLabel_default(t *testing.T) {
-	if got := phaseLabel(""); got != "build" {
-		t.Errorf("phaseLabel(%q) = %q, want %q", "", got, "build")
-	}
-}
-
-func TestUpdate_enterOpensDetailOverlay(t *testing.T) {
-	tasks := []data.Task{sampleTask()}
-	m := NewModel(tasks, "v1", "E04-board-components")
-	m.FocusedColumn = data.ColumnInProgress
-	m.FocusedTask = 0
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyEnter})
-	updated := requireModel(t, got)
-	if updated.Overlay != OverlayDetail {
-		t.Errorf("Overlay = %q, want %q", updated.Overlay, OverlayDetail)
-	}
-}
-
-func TestUpdate_enterNoOpWhenNoTasks(t *testing.T) {
-	m := NewModel(nil, "v1", "E04-board-components")
-	m.FocusedColumn = data.ColumnPlanned
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyEnter})
-	updated := requireModel(t, got)
-	if updated.Overlay != OverlayNone {
-		t.Errorf("Overlay = %q, want none when column has no tasks", updated.Overlay)
-	}
-}
-
-func TestUpdate_detailOverlayEscCloses(t *testing.T) {
-	m := NewModel(nil, "v1", "E04-board-components")
-	m.Overlay = OverlayDetail
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyEsc})
-	updated := requireModel(t, got)
-	if updated.Overlay != OverlayNone {
-		t.Errorf("Overlay = %q after esc, want none", updated.Overlay)
-	}
-}
-
-func TestUpdate_detailOverlayBlocksColumnNav(t *testing.T) {
-	m := NewModel(nil, "v1", "E04-board-components")
-	m.Overlay = OverlayDetail
-	m.FocusedColumn = data.ColumnPlanned
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune("l")})
-	updated := requireModel(t, got)
-	if updated.FocusedColumn != data.ColumnPlanned {
-		t.Error("column nav should be blocked when detail overlay is open")
-	}
-}
-
-func TestView_detailOverlayRendered(t *testing.T) {
-	tasks := []data.Task{sampleTask()}
-	m := NewModel(tasks, "v1", "E04-board-components")
-	m.Width = 100
-	m.Height = 30
-	m.FocusedColumn = data.ColumnInProgress
-	m.FocusedTask = 0
-	m.Overlay = OverlayDetail
-	got := m.View()
-	if !strings.Contains(got, "TASK DETAIL") {
-		t.Error("View() with OverlayDetail missing TASK DETAIL header")
-	}
-	if !strings.Contains(got, "E04/T001") {
-		t.Error("View() with OverlayDetail missing task ID")
-	}
-}
-
-func TestRenderDetail_routerPriorityLabel(t *testing.T) {
-	task := sampleTask()
-	router := &data.RouterState{Release: task.Release, Epic: task.Epic, Task: task.ID}
-	got := RenderDetail(task, 60, router, 0, 0)
-	if !strings.Contains(got, "(router priority)") {
-		t.Error("RenderDetail missing router priority label for matching task")
-	}
-}
-
-func TestRenderDetail_noRouterPriorityLabelWhenNoMatch(t *testing.T) {
-	task := sampleTask()
-	router := &data.RouterState{Release: task.Release, Epic: task.Epic, Task: "other-id"}
-	got := RenderDetail(task, 60, router, 0, 0)
-	if strings.Contains(got, "(router priority)") {
-		t.Error("RenderDetail should not show router priority label for non-matching task")
-	}
-}
-
-func TestRenderDetail_viewportShowsScrollIndicators(t *testing.T) {
-	task := sampleTask()
-	task.Description = "alpha beta gamma delta epsilon zeta eta theta iota kappa lambda mu nu xi omicron"
-
-	got := RenderDetail(task, 32, nil, 8, 2)
-
-	if !strings.Contains(got, "↑ 2 above") {
-		t.Error("RenderDetail missing above indicator")
-	}
-	if !strings.Contains(got, "↓") || !strings.Contains(got, "more") {
-		t.Error("RenderDetail missing more indicator")
-	}
-	if strings.Contains(got, "ID:") {
-		t.Error("RenderDetail should not render body lines above viewport")
-	}
-}
-
-func TestUpdate_detailOverlayScrollsWithJK(t *testing.T) {
-	m := NewModel([]data.Task{sampleTask()}, "v1", "E04-board-components")
-	m.Overlay = OverlayDetail
-
-	got, _ := m.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune("j")})
-	updated := requireModel(t, got)
-	if updated.DetailOffset != 1 {
-		t.Errorf("DetailOffset after j = %d, want 1", updated.DetailOffset)
-	}
-
-	got, _ = updated.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune("k")})
-	updated = requireModel(t, got)
-	if updated.DetailOffset != 0 {
-		t.Errorf("DetailOffset after k = %d, want 0", updated.DetailOffset)
-	}
-}
-
-func TestSplitChecklistSentences_abbreviationEg(t *testing.T) {
-	got := splitChecklistSentences("Use e.g. a widget. Done.")
-	if len(got) != 2 {
-		t.Fatalf("splitChecklistSentences with e.g. = %d sentences, want 2: %v", len(got), got)
-	}
-	if got[0] != "Use e.g. a widget." {
-		t.Errorf("sentence[0] = %q, want %q", got[0], "Use e.g. a widget.")
-	}
-}
-
-func TestSplitChecklistSentences_abbreviationIe(t *testing.T) {
-	got := splitChecklistSentences("Call i.e. the function. Done.")
-	if len(got) != 2 {
-		t.Fatalf("splitChecklistSentences with i.e. = %d sentences, want 2: %v", len(got), got)
-	}
-}
-
-func TestSplitChecklistSentences_abbreviationDr(t *testing.T) {
-	got := splitChecklistSentences("Dr. Smith approved it. Done.")
-	if len(got) != 2 {
-		t.Fatalf("splitChecklistSentences with Dr. = %d sentences, want 2: %v", len(got), got)
-	}
-	if got[0] != "Dr. Smith approved it." {
-		t.Errorf("sentence[0] = %q, want %q", got[0], "Dr. Smith approved it.")
-	}
-}
-
-func TestSplitChecklistSentences_abbreviationEtc(t *testing.T) {
-	got := splitChecklistSentences("Add widgets, buttons, etc. to the panel. Done.")
-	if len(got) != 2 {
-		t.Fatalf("splitChecklistSentences with etc. = %d sentences, want 2: %v", len(got), got)
-	}
-}
-
-func TestSplitChecklistSentences_abbreviationCaseInsensitive(t *testing.T) {
-	got := splitChecklistSentences("See Fig. 3 for details. Done.")
-	if len(got) != 2 {
-		t.Fatalf("splitChecklistSentences with Fig. = %d sentences, want 2: %v", len(got), got)
-	}
-}
-
-func TestSplitChecklistSentences_normalSplitUnaffected(t *testing.T) {
-	got := splitChecklistSentences("First sentence. Second sentence.")
-	if len(got) != 2 {
-		t.Fatalf("splitChecklistSentences normal split = %d sentences, want 2: %v", len(got), got)
-	}
-}
-
-func TestOverlayWidth_clampMax(t *testing.T) {
-	if got := overlayWidth(120); got != 80 {
-		t.Errorf("overlayWidth(120) = %d, want 80", got)
-	}
-}
-
-func TestOverlayWidth_termMinus4(t *testing.T) {
-	if got := overlayWidth(60); got != 56 {
-		t.Errorf("overlayWidth(60) = %d, want 56", got)
-	}
-}
-
-func TestOverlayWidth_clampMin(t *testing.T) {
-	if got := overlayWidth(10); got != 20 {
-		t.Errorf("overlayWidth(10) = %d, want 20", got)
-	}
-}