savepoint 1.0.3 → 1.0.4

package/.savepoint/releases/v1.1/epics/E15-hardening/tasks/T008-ci-and-release-automation.md

@@ -0,0 +1,46 @@
+---
+id: E15-hardening/T008-ci-and-release-automation
+status: done
+objective: Add repo-local CI coverage for build/test checks and keep the buildtool-driven release flow reproducible
+depends_on: []
+---
+
+# T008: Add CI and Release Automation Guardrails
+
+## Context Files
+
+- `Makefile` - local entrypoint for build, test, and dist commands
+- `internal/buildtool/main.go` - build orchestration and packaging logic
+- `.savepoint/releases/v1.1/epics/E15-hardening/E15-Detail.md` - epic scope and component map
+
+## Acceptance Criteria
+
+- [x] `.github/workflows/ci.yml` exists and runs on push and pull_request events
+- [x] The CI workflow runs `go test ./...` and `make build` at minimum
+- [x] The CI workflow uses the repo-local Makefile targets rather than duplicating buildtool logic inline
+- [x] `Makefile` exposes a `ci` target that runs the local verification steps used by CI
+- [x] Generated binaries and archives remain excluded from version control via `.gitignore`
+
+## Implementation Plan
+
+- [x] Add a GitHub Actions workflow for CI in `.github/workflows/ci.yml`
+- [x] Add a `ci` Makefile target that wraps the local verification commands
+- [x] Reuse existing `build`, `test`, and `smoke-test` targets instead of duplicating orchestration
+- [x] Confirm artifact ignore rules still cover local binaries and archives
+- [x] Run `make build && make test`
+
+## Context Log
+
+Files read:
+- Makefile
+- internal/buildtool/main.go
+- .gitignore
+- .savepoint/releases/v1.1/epics/E15-hardening/E15-Detail.md
+- main.go
+- go.mod
+
+Estimated input tokens:
+- 
+
+Notes:
+- 

package/.savepoint/router.md

@@ -12,11 +12,11 @@ Read `.savepoint/PRD.md` only for vision changes. Read `.savepoint/Design.md` on
 ## Current state
 
 ```yaml
-state: task-building
+state: epic-design
 release: v1.1
-epic: E14
-task: E15-hardening/T001-benchmarks
-next_action: Build E15-hardening/T001-benchmarks.
+epic: ""
+task: ""
+next_action: Plan next epic.
 ```
 
 ## State → action

package/AGENTS.md

@@ -85,8 +85,8 @@ make build && make test
 | `main.go` | CLI entrypoint, --version |
 | `cmd/` | CLI command arg parsing and dispatch for init, board, and doctor |
 | `internal/init/` | Target validation, scaffold writing from templates |
-| `internal/board/` | TUI board, overlays, epic sidebar, Next Activity line, router priority key, detail checklist rendering, status glyphs, forced color profile, async update I/O commands, shared board utilities |
-| `internal/buildtool/` | Makefile helper, cross-compile, archives |
+| `internal/board/` | TUI board, overlays, epic sidebar, Next Activity line, router priority key, detail checklist rendering, status glyphs, forced color profile, debug logging hooks, async update I/O commands, shared board utilities |
+| `internal/buildtool/` | Makefile helper, cross-compile including Windows targets, archives, distribution checksums |
 | `internal/doctor/` | Read-only project diagnostics, integrity checks, timed quality gate execution, report formatting, typed repair suggestions |
 | `internal/data/` | Task/router models, frontmatter parsing/splitting, lifecycle validation/defaulting, discovery including root-dir traversal, unified task status constants, canonical write helpers |
 | `internal/testutil/` | Shared Go test fixtures and filesystem helpers for internal package tests |

package/Makefile

@@ -1,4 +1,4 @@
-.PHONY: build test run clean build-linux build-darwin build-all dist smoke-test
+.PHONY: build test run clean build-linux build-darwin build-all dist smoke-test ci
 
 VERSION ?=
 
@@ -27,3 +27,5 @@ dist:
 
 smoke-test:
 	go run ./internal/buildtool -version "$(VERSION)" smoke-test
+
+ci: test build

package/agent_skills_test.go

@@ -1,4 +1,4 @@
-package main
+package main_test
 
 import (
 	"os"

package/internal/board/board.go

@@ -36,10 +36,12 @@ func newProjectModel(start, releaseFilter, epicFilter string) (Model, error) {
 func newProjectModelWithDependencies(start, releaseFilter, epicFilter string, deps ModelDependencies) (Model, error) {
 	deps = modelDependencies([]ModelDependencies{deps})
 
+	debugf("board init: finding savepoint root from %q", start)
 	root, err := deps.Discoverer.FindSavepointRoot(start)
 	if err != nil {
 		return Model{}, err
 	}
+	debugf("board init: root = %q", root)
 
 	routerState, err := readRouterState(root, deps.RouterReader)
 	if err != nil {
@@ -50,6 +52,7 @@ func newProjectModelWithDependencies(start, releaseFilter, epicFilter string, de
 	if err != nil {
 		return Model{}, err
 	}
+	debugf("board init: loaded %d tasks across %d releases", len(tasks), len(releaseIDs))
 
 	preferredRelease := routerState.Release
 	if releaseFilter != "" {
@@ -78,6 +81,7 @@ func newProjectModelWithDependencies(start, releaseFilter, epicFilter string, de
 		return Model{}, err
 	}
 	model.Watcher = watcher
+	debugf("board init: file watcher started at %q", root)
 
 	return model, nil
 }

package/internal/board/card_test.go

@@ -252,3 +252,36 @@ func TestRenderCard_doneShowsDoneText(t *testing.T) {
 		t.Error("RenderCard missing DONE phase label")
 	}
 }
+
+func BenchmarkRenderCard_narrow(b *testing.B) {
+	task := data.Task{ID: "E06-atari-noir-layout/T004-component-refinement", Title: "Refine card layout", Stage: data.StageBuild}
+	b.ReportAllocs()
+	for b.Loop() {
+		RenderCard(task, 24, false, nil)
+	}
+}
+
+func BenchmarkRenderCard_standard(b *testing.B) {
+	task := data.Task{ID: "E06-atari-noir-layout/T004-component-refinement", Title: "Refine card layout for the board view", Stage: data.StageTest}
+	b.ReportAllocs()
+	for b.Loop() {
+		RenderCard(task, 40, false, nil)
+	}
+}
+
+func BenchmarkRenderCard_wide(b *testing.B) {
+	task := data.Task{ID: "E06-atari-noir-layout/T004-component-refinement", Title: "Refine card layout for the board view with extra details", Stage: data.StageAudit}
+	b.ReportAllocs()
+	for b.Loop() {
+		RenderCard(task, 60, false, nil)
+	}
+}
+
+func BenchmarkRenderCard_focused(b *testing.B) {
+	task := data.Task{ID: "E06-atari-noir-layout/T004-component-refinement", Title: "Refine card layout", Stage: data.StageBuild}
+	router := &data.RouterState{Release: "v1", Epic: "E06", Task: "T004"}
+	b.ReportAllocs()
+	for b.Loop() {
+		RenderCard(task, 40, true, router)
+	}
+}

package/internal/board/column.go

@@ -33,11 +33,6 @@ func RenderColumn(tasks []data.Task, col data.ColumnType, width, maxHeight, offs
 			contentBudget = 1
 		}
 
-		reserveAbove := 0
-		if offset > 0 {
-			reserveAbove = 1
-		}
-
 		type cardEntry struct {
 			card  string
 			lines int
@@ -48,12 +43,17 @@ func RenderColumn(tasks []data.Task, col data.ColumnType, width, maxHeight, offs
 			cardEntries = append(cardEntries, cardEntry{card: c, lines: strings.Count(c, "\n") + 1})
 		}
 
+		// Standard window: fit as many cards as possible from the start of cardEntries.
+		reserveAbove := 0
+		if offset > 0 {
+			reserveAbove = 1
+		}
 		usedLines := reserveAbove
 		endIdx := 0
 		for endIdx < len(cardEntries) {
-			needsMore := endIdx < len(cardEntries)-1
+			hasMore := (offset + endIdx + 1) < len(tasks)
 			bottomReserve := 0
-			if needsMore {
+			if hasMore {
 				bottomReserve = 1
 			}
 			if usedLines+cardEntries[endIdx].lines+bottomReserve > contentBudget {
@@ -62,19 +62,49 @@ func RenderColumn(tasks []data.Task, col data.ColumnType, width, maxHeight, offs
 			usedLines += cardEntries[endIdx].lines
 			endIdx++
 		}
-
 		if endIdx == 0 && len(cardEntries) > 0 {
 			endIdx = 1
 		}
 
-		if offset > 0 {
-			lines = append(lines, renderScrollIndicator("↑", offset, "above"))
+		// Determine what portion of cardEntries to render.
+		renderStart := 0
+		renderEnd := endIdx
+
+		focusedRelIdx := focusedTask - offset
+		if focused && focusedRelIdx >= 0 && focusedRelIdx < len(cardEntries) && focusedRelIdx >= endIdx {
+			// Focused task is beyond the standard window.
+			// Anchor viewport at focused task: fill backward to use remaining budget.
+			bottomCost := 0
+			if focusedTask+1 < len(tasks) {
+				bottomCost = 1
+			}
+			cardsLines := cardEntries[focusedRelIdx].lines
+			newStart := focusedRelIdx
+			for newStart > 0 {
+				prev := cardEntries[newStart-1]
+				topCost := 1
+				if offset+newStart-1 == 0 {
+					topCost = 0
+				}
+				if cardsLines+prev.lines+topCost+bottomCost > contentBudget {
+					break
+				}
+				cardsLines += prev.lines
+				newStart--
+			}
+			renderStart = newStart
+			renderEnd = focusedRelIdx + 1
+		}
+
+		effectiveOffset := offset + renderStart
+		if effectiveOffset > 0 {
+			lines = append(lines, renderScrollIndicator("↑", effectiveOffset, "above"))
 		}
-		for i := 0; i < endIdx; i++ {
+		for i := renderStart; i < renderEnd; i++ {
 			lines = append(lines, cardEntries[i].card)
 		}
-		if endIdx < len(cardEntries) {
-			remaining := len(tasks) - (offset + endIdx)
+		if offset+renderEnd < len(tasks) {
+			remaining := len(tasks) - (offset + renderEnd)
 			lines = append(lines, renderScrollIndicator("↓", remaining, "more"))
 		}
 	}
@@ -124,4 +154,3 @@ func columnTitle(col data.ColumnType) string {
 		return strings.ToUpper(string(col))
 	}
 }
-

package/internal/board/column_test.go

@@ -1,6 +1,7 @@
 package board
 
 import (
+	"fmt"
 	"strings"
 	"testing"
 
@@ -137,3 +138,73 @@ func TestVisibleColumnTaskLimitDefaultsToFourAtStandardHeight(t *testing.T) {
 		t.Errorf("visibleColumnTaskLimit(standard height) = %d, want 4", got)
 	}
 }
+
+func BenchmarkRenderColumn_empty(b *testing.B) {
+	b.ReportAllocs()
+	for b.Loop() {
+		RenderColumn(nil, data.ColumnPlanned, 30, 20, 0, 0, false, nil)
+	}
+}
+
+func BenchmarkRenderColumn_fewTasks(b *testing.B) {
+	tasks := []data.Task{
+		{ID: "E06/T001", Title: "First task", Column: data.ColumnPlanned, Stage: data.StageBuild},
+		{ID: "E06/T002", Title: "Second task", Column: data.ColumnPlanned, Stage: data.StageTest},
+		{ID: "E06/T003", Title: "Third task", Column: data.ColumnPlanned, Stage: data.StageAudit},
+	}
+	b.ReportAllocs()
+	for b.Loop() {
+		RenderColumn(tasks, data.ColumnPlanned, 30, 20, 0, 0, false, nil)
+	}
+}
+
+func BenchmarkRenderColumn_manyTasks(b *testing.B) {
+	tasks := make([]data.Task, 20)
+	stages := []data.ProgressStage{data.StageBuild, data.StageTest, data.StageAudit}
+	for i := range tasks {
+		tasks[i] = data.Task{
+			ID:      fmt.Sprintf("E06/T%03d", i+1),
+			Title:   fmt.Sprintf("Task number %d with a reasonable length title", i+1),
+			Column:  data.ColumnPlanned,
+			Stage:   stages[i%3],
+			Release: "v1.1",
+		}
+	}
+	b.ReportAllocs()
+	for b.Loop() {
+		RenderColumn(tasks, data.ColumnPlanned, 40, 24, 0, 0, false, nil)
+	}
+}
+
+func BenchmarkRenderColumn_focused(b *testing.B) {
+	tasks := []data.Task{
+		{ID: "E06/T001", Title: "First task", Column: data.ColumnInProgress, Stage: data.StageBuild},
+		{ID: "E06/T002", Title: "Second task", Column: data.ColumnInProgress, Stage: data.StageTest},
+		{ID: "E06/T003", Title: "Third task", Column: data.ColumnInProgress, Stage: data.StageAudit},
+	}
+	router := &data.RouterState{Release: "v1", Epic: "E06", Task: "T001"}
+	b.ReportAllocs()
+	for b.Loop() {
+		RenderColumn(tasks, data.ColumnInProgress, 40, 20, 0, 0, true, router)
+	}
+}
+
+func TestRenderColumn_focusedLastTaskVisibleWhenScrolled(t *testing.T) {
+	// When scrolled (offset > 0) the top scroll indicator steals 1 line.
+	// The focused last-in-page task must still appear, not be cut off.
+	tasks := []data.Task{
+		{ID: "T1", Title: "Task one", Column: data.ColumnPlanned},
+		{ID: "T2", Title: "Task two", Column: data.ColumnPlanned},
+		{ID: "T3", Title: "Task three", Column: data.ColumnPlanned},
+		{ID: "T4", Title: "Task four", Column: data.ColumnPlanned},
+		{ID: "T5", Title: "Task five", Column: data.ColumnPlanned},
+	}
+	// offset=2, focusedTask=4 (last task), maxHeight=14 (24-line terminal)
+	got := RenderColumn(tasks, data.ColumnPlanned, 30, 14, 2, 4, true, nil)
+	if !strings.Contains(got, "Task five") {
+		t.Errorf("focused last task must be visible when scrolled, got:\n%s", got)
+	}
+	if !strings.Contains(got, "↑") {
+		t.Error("scroll indicator above must appear when offset > 0")
+	}
+}

package/internal/board/debug.go

@@ -0,0 +1,26 @@
+package board
+
+import (
+	"fmt"
+	"os"
+	"sync/atomic"
+)
+
+var debugEnabled atomic.Bool
+
+// SetDebug enables or disables debug logging for the board package.
+func SetDebug(enabled bool) {
+	debugEnabled.Store(enabled)
+}
+
+// DebugEnabled reports whether debug logging is active.
+func DebugEnabled() bool {
+	return debugEnabled.Load()
+}
+
+func debugf(format string, args ...any) {
+	if !debugEnabled.Load() {
+		return
+	}
+	fmt.Fprintf(os.Stderr, "[savepoint debug] "+format+"\n", args...)
+}

package/internal/board/debug_test.go

@@ -0,0 +1,108 @@
+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

@@ -85,6 +85,36 @@ func renderChecklistSentences(text, glyph string, width int, style lipgloss.Styl
 	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 {
@@ -98,6 +128,9 @@ func splitChecklistSentences(text string) []string {
 		if r != '.' && r != '!' && r != '?' {
 			continue
 		}
+		if r == '.' && isKnownAbbreviation(normalized, i) {
+			continue
+		}
 		end := i + len(string(r))
 		if end < len(normalized) && normalized[end] != ' ' {
 			continue

package/internal/board/detail_test.go

@@ -321,6 +321,54 @@ func TestUpdate_detailOverlayScrollsWithJK(t *testing.T) {
 	}
 }
 
+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)

package/internal/board/epic_panel.go

@@ -113,6 +113,8 @@ func RenderEpicAuditTab(epicSlug, content string, overlayW, maxHeight, offset in
 	return styles.EpicDetailOverlay.Width(overlayW).Render(strings.Join(lines, "\n"))
 }
 
+// epicAuditHiddenSectionHeadings lists markdown section headings suppressed in the audit tab overlay.
+// Sections that are implementation details or planning artifacts clutter the summary view.
 var epicAuditHiddenSectionHeadings = map[string]struct{}{
 	"12. Distribution & build": {},
 	"Acceptance Criteria":      {},

package/internal/board/update.go

@@ -18,10 +18,12 @@ var columnOrder = []data.ColumnType{
 func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 	switch msg := msg.(type) {
 	case fileChangeMsg:
+		debugf("dispatch: fileChangeMsg")
 		if m.Root != "" {
 			return m, reloadTasks(m.Root, m.Dependencies)
 		}
 	case reloadMsg:
+		debugf("dispatch: reloadMsg tasks=%d releases=%d", len(msg.tasks), len(msg.releases))
 		m.AllTasks = msg.tasks
 		m.Releases = append([]string(nil), msg.releases...)
 		m.ReleaseEpics = copyReleaseEpics(msg.releaseEpics)
@@ -521,6 +523,23 @@ func (m Model) columnPageSize() int {
 	return visibleColumnTaskLimit(CalculateLayout(m.Width, h).ContentHeight)
 }
 
+// conservativeColumnPageSize reserves 2 lines for scroll indicators so that
+// ensureFocusedTaskVisible never sets an offset where the focused card is hidden
+// by a top or bottom indicator that wasn't accounted for in the page budget.
+func (m Model) conservativeColumnPageSize() int {
+	h := m.Height
+	if h == 0 {
+		h = defaultTermH
+	}
+	contentHeight := CalculateLayout(m.Width, h).ContentHeight
+	// contentHeight - 2 = card budget; subtract 2 more for both indicators.
+	limit := (contentHeight - 4) / 3
+	if limit < 1 {
+		return 1
+	}
+	return limit
+}
+
 func (m Model) detailPageSize() int {
 	return max(detailMaxHeight(m.Height)-3, 1)
 }

package/internal/board/update_test.go

@@ -554,6 +554,33 @@ func TestReloadMsgUpdatesRouterState(t *testing.T) {
 	}
 }
 
+func TestEnsureFocusedTaskVisible_lastTaskAlwaysVisible(t *testing.T) {
+	// 5 tasks, pageSize=4 at standard height. Pressing down past the page
+	// boundary must keep the focused task in the rendered window.
+	tasks := make([]data.Task, 5)
+	for i := range tasks {
+		tasks[i] = data.Task{ID: "T00" + string(rune('1'+i)), Column: data.ColumnPlanned}
+	}
+	m := NewModel(tasks, "v1", "E01")
+	m.Width = 100
+	m.Height = 24
+	m.FocusedColumn = data.ColumnPlanned
+	m.FocusedTask = 4
+
+	m.ensureFocusedTaskVisible()
+
+	offset := m.ColumnOffsets[data.ColumnPlanned]
+	pageSize := m.columnPageSize() // 4
+	if offset+pageSize-1 < 4 {
+		t.Errorf("offset=%d pageSize=%d: focused task 4 not in [offset, offset+pageSize-1]", offset, pageSize)
+	}
+	// Render must actually show the focused task (not cut off by scroll indicator).
+	got := RenderColumn(tasks, data.ColumnPlanned, 30, CalculateLayout(100, 24).ContentHeight, offset, 4, true, nil)
+	if !strings.Contains(got, tasks[4].ID) {
+		t.Errorf("focused last task %q not visible in rendered column (offset=%d):\n%s", tasks[4].ID, offset, got)
+	}
+}
+
 func writeRouterFixture(t *testing.T) string {
 	t.Helper()
 	root := t.TempDir()