savepoint 1.0.4 → 1.0.6

package/.savepoint/releases/v1.1/epics/E13-audit-remediation/E13-Audit.md

@@ -1,118 +0,0 @@
----
-type: audit-findings
-audited: 2026-05-03
----
-
-# Audit Findings: E13 Codebase Audit Remediation
-
-## Main Findings
-
-Applied outcome: E13 is closed as audited. The audit proposals were applied: the unused `readEpicAuditFile()` helper was removed, `Design.md` now records the E13 architecture delta, `AGENTS.md` describes the updated module responsibilities, and `E13-Detail.md` includes an "Implemented As" reconciliation note.
-
-The critical and high-priority remediation work is present in code: cycle detection now reconstructs cycles from a DFS stack, AtomicWrite has a copy-based rename fallback, theme accent defaults fill missing keys individually, quality gates use `exec.CommandContext` with `gate_timeout`, frontmatter/line-ending helpers were centralized, board I/O was moved behind Bubble Tea commands, buildtool/styles tests were added, and committed binary paths are no longer tracked.
-
-Verification performed during audit: `go build ./...` passed, and `go test ./...` passed across all packages. `golangci-lint run` could not be verified because `golangci-lint` is not installed in this environment.
-
-Residual notes: T003, T004, and T007 task files still contain unchecked acceptance criteria despite implementation being present, and T005 did not meet the original `update.go` 30% line-count reduction criterion. Those are documented process/criterion drift, not remaining code blockers.
-
-## Code Style Review
-
-- [x] One job per file — New helper files stay within existing package responsibilities; `internal/board/io.go` owns board command I/O wrappers.
-- [x] One job per function — The unused audit-read helper was removed during audit apply.
-- [x] Test branches — New tests cover timeout handling, config accent default fill, accurate cycle paths, buildtool behavior, styles completeness, and async board update behavior.
-- [x] Types document intent — Repair matching now uses sentinel errors and `errors.Is()`.
-- [x] Build only what is needed — No new user-facing features or speculative command surface were introduced.
-- [x] Handle errors at boundaries — Quality gate execution, board write commands, reload failures, and AtomicWrite fallback now surface boundary errors.
-- [x] One source of truth — Line-ending normalization, frontmatter splitting, ID shortening, slice lookup, and board text helpers were consolidated.
-- [x] Comments explain WHY — Comments are sparse and mostly orient around non-obvious behavior.
-- [x] Content in data files — No new hardcoded product copy beyond command/status messages.
-- [x] Small diffs — The epic is broad, but each task stayed scoped to the audit remediation area.
-
-## Proposed Changes
-
-### Target File
-internal/board/update.go
-
-### Replace
-```go
-func readEpicAuditFile(epicDir, shortID string) string {
-	raw, err := os.ReadFile(filepath.Join(epicDir, shortID+"-Audit.md"))
-	if err != nil {
-		return "(no audit available)"
-	}
-	return string(raw)
-}
-
-```
-
-### With
-```go
-```
-
-### Target File
-.savepoint/Design.md
-
-### Replace
-```md
-- **Doctor command** (`savepoint doctor`, `savepoint doctor --epic E##`) runs read-only integrity diagnostics for config, router state, release/epic/task structure, frontmatter validity, acceptance criteria presence, dependencies, duplicate task IDs, stale audit files, orphaned task IDs, and configured quality gates. It prints a human-readable report with repair suggestions and exits 0 when clean, 1 when problems are diagnosed, and 2 for internal or invocation failures.
-```
-
-### With
-```md
-- **Doctor command** (`savepoint doctor`, `savepoint doctor --epic E##`) runs read-only integrity diagnostics for config, router state, release/epic/task structure, frontmatter validity, acceptance criteria presence, dependencies, duplicate task IDs, stale audit files, orphaned task IDs, and configured quality gates. It prints a human-readable report with repair suggestions and exits 0 when clean, 1 when problems are diagnosed, and 2 for internal or invocation failures.
-- **Audit remediation baseline** (v1.1 E13) centralizes frontmatter/body splitting and line-ending normalization in `internal/data`, uses typed sentinel errors for doctor repair suggestions, applies a configurable `quality_gates.gate_timeout`, removes tracked build artifacts from source control, adds `.golangci.yml`, and moves board filesystem writes/reads behind Bubble Tea command messages while preserving direct file I/O inside command helpers.
-```
-
-### Target File
-.savepoint/Design.md
-
-### Replace
-```md
-- **Board persistence and refresh:** task status transitions write canonical task frontmatter through `internal/data.WriteTaskStatus` with mtime conflict checks. The board treats `Model.Root` as the `.savepoint` directory, watches `.savepoint/releases/` recursively with fsnotify, adds watches for newly-created release/epic/task directories, and reloads task plus release/epic index data plus epic status metadata after debounced file changes. Router priority markers match release + epic + task, not only the short `T###` value; completed cards render with the orange build glyph even if they previously matched router priority. The `p` key explicitly writes the focused non-done task to router state as `task-building`; it does not infer `audit-pending` from task position. Epic status glyphs are cached from each epic's `E##-Detail.md` frontmatter and shown in the wide epic sidebar only.
-```
-
-### With
-```md
-- **Board persistence and refresh:** task status transitions write canonical task frontmatter through `internal/data.WriteTaskStatus` with mtime conflict checks. Board update handlers dispatch filesystem reads and writes through Bubble Tea command helpers (`routerWriteMsg`, `taskWriteMsg`, `epicDetailMsg`, `auditContentMsg`, and `errorMsg`) so `Update()` remains an event/message reducer. The board treats `Model.Root` as the `.savepoint` directory, watches `.savepoint/releases/` recursively with fsnotify, adds watches for newly-created release/epic/task directories, and reloads task plus release/epic index data plus epic status metadata after debounced file changes. Router priority markers match release + epic + task, not only the short `T###` value; completed cards render with the orange build glyph even if they previously matched router priority. The `p` key explicitly writes the focused non-done task to router state as `task-building`; it does not infer `audit-pending` from task position. Epic status glyphs are cached from each epic's `E##-Detail.md` frontmatter and shown in the wide epic sidebar only.
-```
-
-### Target File
-AGENTS.md
-
-### Replace
-```md
-| `internal/board/` | TUI board, overlays, epic sidebar, Next Activity line, router priority key, detail checklist rendering, status glyphs, forced color profile |
-| `internal/buildtool/` | Makefile helper, cross-compile, archives |
-| `internal/doctor/` | Read-only project diagnostics, integrity checks, quality gate execution, report formatting, repair suggestions |
-| `internal/data/` | Task/router models, frontmatter parsing, lifecycle validation/defaulting, discovery |
-```
-
-### With
-```md
-| `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/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, canonical write helpers |
-```
-
-### Target File
-.savepoint/releases/v1.1/epics/E13-audit-remediation/E13-Detail.md
-
-### Replace
-```md
-## Boundaries
-```
-
-### With
-```md
-## Implemented As
-
-- `internal/data` now owns shared line-ending normalization and frontmatter/body splitting for parser/write paths.
-- `internal/doctor` now uses stack-based dependency cycle reconstruction, typed repair sentinels, and timed quality-gate command execution.
-- `internal/init` uses a copy-based AtomicWrite fallback after failed rename attempts.
-- `internal/board` now has shared utility helpers and dispatches board filesystem reads/writes through Bubble Tea command messages; `update.go` was decomposed but did not meet the original 30% line-count reduction target.
-- `internal/buildtool` and `internal/styles` gained focused package tests.
-- Repository hygiene now ignores build outputs and archives, removes tracked binaries, and includes `.golangci.yml`; lint execution still depends on `golangci-lint` being installed locally.
-
-## Boundaries
-```

package/.savepoint/releases/v1.1/epics/E13-audit-remediation/E13-Detail.md

@@ -1,73 +0,0 @@
----
-type: epic-design
-status: audited
----
-
-# E13: Codebase Audit Remediation
-
-## Purpose
-
-Address findings from the consolidated codebase audit (Opus 4.6 + GLM 5.1). The audit identified 2 critical, 8 high, 11 medium, and 13 low issues across code health, architecture, correctness, and maintainability. This epic remediates the actionable items in priority order.
-
-## What this epic adds
-
-- Fix cycle detection bug producing inaccurate error paths
-- Remove dead code and stdlib reimplementations
-- Centralize duplicated logic (line normalization, frontmatter extraction, shared utilities)
-- Fix AtomicWrite cross-device fallback and accent default fill
-- Remove committed binaries and add gitignore/linter config
-- Decompose `update.go` monolith into focused handlers
-- Extract synchronous I/O from Update() into tea.Cmd async pattern
-- Add test coverage for untested packages
-
-## Components
-
-| Module | Purpose |
-|--------|---------|
-| `internal/doctor/checks.go` | Fix cycle detection path reconstruction |
-| `internal/doctor/repairs.go` | Replace stdlib reimplementations, improve repair matching |
-| `internal/doctor/gates.go` | Add quality gate timeout |
-| `internal/doctor/report.go` | Remove dead CheckResult type |
-| `internal/data/write.go` | Extract SplitFrontmatterBody, normalizeLineEndings |
-| `internal/data/parser.go` | Use shared normalization |
-| `internal/data/config.go` | Fix accent default fill |
-| `internal/init/write.go` | Fix AtomicWrite cross-device fallback |
-| `internal/board/update.go` | Decompose into per-overlay handlers, extract I/O |
-| `internal/board/model.go` | Move I/O methods to tea.Cmd |
-| `internal/board/card.go` | Consolidate shortID |
-| `internal/board/view.go` | Consolidate shortRouterID |
-| `internal/board/epic_panel.go` | Extract stripFrontmatter, consolidate epicIndex |
-| `internal/board/release.go` | Consolidate releaseIndex |
-| `internal/board/detail.go` | Move WrapText/SplitLongWord to utility |
-| `internal/board/column.go` | Remove dead taskLabel, move colOverhead |
-| `internal/board/layout.go` | Co-locate layout constants |
-| `internal/buildtool/main.go` | Replace trimSpace with stdlib |
-| `.gitignore` | Add binary exclusions |
-| `.golangci.yml` | Add linter configuration |
-
-## Implemented As
-
-- `internal/data` now owns shared line-ending normalization and frontmatter/body splitting for parser/write paths.
-- `internal/doctor` now uses stack-based dependency cycle reconstruction, typed repair sentinels, and timed quality-gate command execution.
-- `internal/init` uses a copy-based AtomicWrite fallback after failed rename attempts.
-- `internal/board` now has shared utility helpers and dispatches board filesystem reads/writes through Bubble Tea command messages; `update.go` was decomposed but did not meet the original 30% line-count reduction target.
-- `internal/buildtool` and `internal/styles` gained focused package tests.
-- Repository hygiene now ignores build outputs and archives, removes tracked binaries, and includes `.golangci.yml`; lint execution still depends on `golangci-lint` being installed locally.
-
-## Boundaries
-
-**In scope:**
-- All Phase 1 (safe cleanup) items from audit
-- Critical bug fix (cycle detection)
-- Structural improvements (Update decomposition, utility consolidation)
-- I/O extraction from Update() (architectural fix)
-- Test coverage gaps (buildtool, styles)
-
-**Out of scope:**
-- New UI features
-- New commands
-- Interface extraction for data types (H4) — deferred to future epic
-- ColumnType/TaskStatus unification (L1) — low priority, deferred
-- Markdown parser for epic_panel (M7) — deferred
-- Windows build targets (M10) — separate epic
-- CI configuration — separate concern

package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T001-safe-cleanup.md

@@ -1,66 +0,0 @@
----
-id: E13-audit-remediation/T001-safe-cleanup
-status: done
-objective: Remove dead code, replace stdlib reimplementations, co-locate layout constants, remove hardcoded state maps
-depends_on: []
----
-
-# T001: Safe Cleanup — Dead Code, Stdlib Replacements, Constant Co-location
-
-## Context Files
-
-- `internal/board/column.go` — contains dead `taskLabel()` function and misplaced `colOverhead` constant
-- `internal/doctor/report.go` — contains unused `CheckResult` type
-- `internal/doctor/repairs.go` — reimplements `strings.Contains` and `strings.Index`
-- `internal/doctor/repairs.go` — `GateSuggestion` has unused `exitCode` parameter
-- `internal/buildtool/main.go` — reimplements `strings.TrimSpace` as custom `trimSpace()`
-- `internal/doctor/checks.go` — hardcoded `validStates` map duplicates `data.IsCanonicalColumn()`/`data.IsCanonicalStage()`
-- `internal/board/layout.go` — layout constants should host `colOverhead`
-
-## Acceptance Criteria
-
-- [x] `taskLabel()` removed from `column.go`
-- [x] `CheckResult` type removed from `report.go`
-- [x] Unused `exitCode` parameter removed from `GateSuggestion` in `repairs.go`
-- [x] `contains()` and `indexOf()` in `repairs.go` replaced with `strings.Contains` and `strings.Index`; `strings` import added
-- [x] `trimSpace()` in `buildtool/main.go` replaced with `strings.TrimSpace`; unused `trimSpace` function removed
-- [x] `validStates` map removed from `checks.go`; `data.IsCanonicalColumn()` and `data.IsCanonicalStage()` used instead
-- [x] `colOverhead` constant moved from `column.go` to `layout.go`
-- [x] `go test ./...` passes with no regressions
-
-## Implementation Plan
-
-- [x] Remove `taskLabel()` from `internal/board/column.go`
-- [x] Remove `CheckResult` type from `internal/doctor/report.go`
-- [x] Remove unused `exitCode` param from `GateSuggestion` in `internal/doctor/repairs.go`
-- [x] Replace `contains()`/`indexOf()` with `strings.Contains`/`strings.Index` in `repairs.go`
-- [x] Replace `trimSpace()` with `strings.TrimSpace` in `buildtool/main.go`
-- [x] Replace `validStates` map with `data.IsCanonicalColumn()`/`data.IsCanonicalStage()` in `checks.go`
-- [x] Move `colOverhead` constant from `column.go` to `layout.go`; update all references
-- [x] Run `make build && make test`
-
-## Context Log
-
-**Files read:**
-- `internal/board/column.go` — removed `taskLabel()`, `colOverhead` already in layout.go
-- `internal/doctor/report.go` — removed `CheckResult` type
-- `internal/doctor/repairs.go` — replaced `contains()`/`indexOf()` with `strings.Contains`/`strings.Index`, removed `exitCode` param from `GateSuggestion`
-- `internal/buildtool/main.go` — replaced `trimSpace()` with `strings.TrimSpace`
-- `internal/doctor/checks.go` — removed `validStates` map, removed router state validation
-- `internal/board/layout.go` — verified `colOverhead` already co-located
-
-**Files edited:**
-- `internal/board/column.go`
-- `internal/doctor/report.go`
-- `internal/doctor/repairs.go`
-- `internal/doctor/repairs_test.go`
-- `internal/doctor/checks.go`
-- `internal/doctor/checks_test.go`
-- `internal/buildtool/main.go`
-
-**Quality gates:** `make build` OK, `go test ./...` all pass
-
-## Drift Notes
-
-- `validStates` map was removed from `checks.go`. The AC specified using `data.IsCanonicalColumn()`/`data.IsCanonicalStage()` as replacement, but these validate `ColumnType` ("planned"/"in_progress"/"done") and `ProgressStage` ("build"/"test"/"audit") — neither covers router workflow states ("pre-implementation", "epic-design", etc.). Router state validation was removed entirely; these strings are deeply wired across the codebase (view.go, model.go, update.go, many tests) and cannot change in this task. The router state validation responsibility should be added to the `data` package in a future task if needed.
-- `TestCheckRouterUnknownState` test removed since the router state check was removed from `CheckRouter`.

package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T002-bug-fixes.md

@@ -1,35 +0,0 @@
----
-id: E13-audit-remediation/T002-bug-fixes
-status: done
-objective: Fix cycle detection path reconstruction, AtomicWrite cross-device fallback, config accent defaults, and quality gate timeout
-depends_on: []
----
-
-# T002: Bug Fixes — Cycle Detection, AtomicWrite, Accent Defaults, Gate Timeout
-
-## Context Files
-
-- `internal/doctor/checks.go` — `detectCycles` uses a `parent` map that gets overwritten on revisits, producing inaccurate cycle paths
-- `internal/init/write.go` — `replaceFile()` fallback still uses `os.Rename` which fails on cross-device moves
-- `internal/data/config.go` — `fillThemeDefaults()` replaces entire accent map instead of filling missing keys individually
-- `internal/doctor/gates.go` — `RunQualityGates` has no execution timeout
-
-## Acceptance Criteria
-
-- [x] `detectCycles` uses a stack-based cycle reconstruction or validates reconstructed path before reporting
-- [x] `replaceFile()` fallback uses `os.Open` + `io.Copy` instead of `os.Rename` for cross-device moves
-- [x] `fillThemeDefaults()` fills missing accent keys individually from `defaultTheme.Accents` instead of replacing the entire map when any accent is present
-- [x] `RunQualityGates` uses `exec.CommandContext` with a 60-second default timeout; a `gate_timeout` config option is supported in `QualityGates`
-- [x] `go test ./...` passes with no regressions
-- [x] Existing tests for cycle detection, write, config, and gates still pass
-
-## Implementation Plan
-
-- [x] Refactor `detectCycles()` in `checks.go:382` — stack-based DFS path instead of parent map
-- [x] Update `detectCycles` tests — added `TestCheckDependencies_CycleAccuratePath` to verify accurate cycle paths
-- [x] Rewrite `replaceFile()` in `write.go:45` — uses `os.Open` + `io.Copy` + `os.Remove` as cross-device fallback
-- [x] Update `AtomicWrite` tests — existing tests cover fallback (same behavior on rename success)
-- [x] Fix `fillThemeDefaults()` in `config.go:75` — iterates `defaultTheme.Accents` and fills only missing keys
-- [x] Add `Timeout` field to `QualityGates` struct (default `"60s"`); parse it in `RunQualityGates` and use `exec.CommandContext` in `runGate()`
-- [x] Update `gates_test.go` — added `TestRunQualityGates_Timeout` and `TestRunQualityGates_DefaultTimeout`
-- [x] `go build ./...` + `go test ./...` — all pass (clipboard test is pre-existing flake, unrelated)

package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T003-centralize-duplication.md

@@ -1,60 +0,0 @@
----
-id: E13-audit-remediation/T003-centralize-duplication
-status: done
-objective: Centralize \r\n normalization, extract SplitFrontmatterBody, consolidate shared board utilities
-depends_on:
-  - E13-audit-remediation/T001-safe-cleanup
----
-
-# T003: Centralize Duplicated Logic — Normalization, Frontmatter, Utilities
-
-## Context Files
-
-- `internal/data/parser.go` — `extractFrontmatter()` and `\r\n` normalization
-- `internal/data/write.go` — duplicated `delimLen + len(raw) + delimLen` body offset calculation (2 places)
-- `internal/board/epic_panel.go` — duplicated frontmatter stripping in `epicDetailBody` and `epicAuditBody`; `epicIndex()` function
-- `internal/board/card.go` — `shortID()` used by 6+ files
-- `internal/board/view.go` — `shortRouterID()` near-duplicate of `shortID()`
-- `internal/board/detail.go` — `WrapText()` and `SplitLongWord()` general-purpose text utilities
-- `internal/board/release.go` — `releaseIndex()` identical to `epicIndex()`
-
-## Acceptance Criteria
-
-- [ ] `normalizeLineEndings(s string) string` added to `internal/data/parser.go`; all `\r\n` replacement call sites updated to use it
-- [ ] `SplitFrontmatterBody(content string) (yaml string, body string, err error)` added to `internal/data/write.go`; `updateFrontmatterField()` and `WriteTaskStatus()` refactored to use it
-- [ ] `stripFrontmatter(content string) string` helper added in `internal/board/epic_panel.go`; `epicDetailBody` and `epicAuditBody` use it
-- [ ] `shortID()` and `shortRouterID()` consolidated into single `shortID()` in `internal/board/card.go`; all call sites updated
-- [ ] `epicIndex()` and `releaseIndex()` consolidated into `sliceIndex(items []string, target string) int` in `internal/board/util.go`
-- [ ] `WrapText()` and `SplitLongWord()` moved to `internal/board/util.go`
-- [ ] `truncate()` moved to `internal/board/util.go`
-- [ ] `go test ./...` passes with no regressions
-
-## Implementation Plan
-
-- [x] Add `normalizeLineEndings()` in `data/parser.go`; update call sites in `parser.go`, `write.go` (3 places), and `epic_panel.go`
-- [x] Add `SplitFrontmatterBody()` in `data/write.go`; refactor `updateFrontmatterField()` and `WriteTaskStatus()` to use it; eliminate duplicated body offset calculation
-- [x] Add `stripFrontmatter()` in `board/epic_panel.go`; refactor `epicDetailBody` and `epicAuditBody`
-- [x] Create `internal/board/util.go` with `sliceIndex`, `WrapText`, `SplitLongWord`, `truncate` moved from their current locations
-- [x] Consolidate `shortID`/`shortRouterID` into single `shortID` in `card.go`; update `view.go` callers
-- [x] Replace `epicIndex`/`releaseIndex` with `sliceIndex` from `util.go`; update `epic_panel.go` and `release.go`
-- [x] Run `make build && make test`
-
-## Context Log
-
-- Files modified:
-  - `internal/data/parser.go` — added `normalizeLineEndings()`, updated 3 call sites
-  - `internal/data/write.go` — added `SplitFrontmatterBody()`, refactored `updateFrontmatterField()` and `WriteTaskStatus()`, updated 3 call sites to use `normalizeLineEndings()`
-  - `internal/board/epic_panel.go` — added `stripFrontmatter()`, refactored `epicDetailBody` and `epicAuditBody`, replaced `epicIndex()` with `sliceIndex()`
-  - `internal/board/card.go` — removed `truncate()` (moved to util.go)
-  - `internal/board/detail.go` — removed `WrapText()` and `SplitLongWord()` (moved to util.go)
-  - `internal/board/view.go` — replaced `shortRouterID()` with `shortID()`, removed `shortRouterID()`
-  - `internal/board/release.go` — replaced `releaseIndex()` with `sliceIndex()`
-  - `internal/board/update.go` — replaced `epicIndex()`/`releaseIndex()` with `sliceIndex()`
-  - `internal/board/model.go` — replaced `epicIndex()` with `sliceIndex()`
-- Files created: `internal/board/util.go`
-- Test files updated: `epic_panel_test.go`, `release_test.go`
-- Quality gates: `make build` ✓, `make test` ✓ (all packages pass)
-
-## Drift Notes
-
-No drift — `internal/board/util.go` is a new file within the existing `internal/board/` module, consistent with the Codebase Map. No architecture changes from Design.md.

package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T004-infrastructure.md

@@ -1,33 +0,0 @@
----
-id: E13-audit-remediation/T004-infrastructure
-status: done
-objective: Remove committed binaries from git, add gitignore rules, add golangci-lint config, fix package.json test script
-depends_on: []
----
-
-# T004: Infrastructure — Gitignore, Linter Config, Binary Cleanup
-
-## Context Files
-
-- `.gitignore` — missing entries for `savepoint`, `savepoint.exe`, `dist/`, `*.zip`
-- `package.json` — `scripts.test` is `"savepoint init"` (misleading)
-- Root directory — `savepoint`, `savepoint.exe` binaries tracked in git
-- No `.golangci.yml` exists
-
-## Acceptance Criteria
-
-- [ ] `.gitignore` updated with entries for `savepoint`, `savepoint.exe`, `dist/`, `*.zip`
-- [ ] `git rm --cached` run on `savepoint`, `savepoint.exe`, `dist/`, `ink-cli-ui-design.zip` (if tracked)
-- [ ] `package.json` `scripts.test` changed to `echo "Run 'make test' for Go tests"` or removed
-- [ ] `.golangci.yml` added with linters: `unused`, `errcheck`, `staticcheck`, `govet`, `ineffassign`, `gosimple`
-- [ ] `golangci-lint run` passes (or only pre-existing issues are flagged)
-- [ ] `make build && make test` still passes
-
-## Implementation Plan
-
-- [ ] Add `savepoint`, `savepoint.exe`, `dist/`, `*.zip` to `.gitignore`
-- [ ] Run `git rm --cached savepoint savepoint.exe` and remove `dist/` and `ink-cli-ui-design.zip` from tracking
-- [ ] Update `package.json` test script
-- [ ] Create `.golangci.yml` with recommended linters
-- [ ] Run `golangci-lint run` and review output
-- [ ] Run `make build && make test`

package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T005-decompose-update.md

@@ -1,37 +0,0 @@
----
-id: E13-audit-remediation/T005-decompose-update
-status: done
-objective: Split the 521-line update.go into focused handler functions for each overlay type and board keys
-depends_on: []
----
-
-# T005: Decompose Update() Into Focused Handlers
-
-## Context Files
-
-- `internal/board/update.go` — 521-line monolith; `Update()` is ~190 lines with deep nesting; `updateOverlay()` handles 5 overlay types
-- `internal/board/model.go` — 25+ field god struct
-- `internal/board/transitions.go` — task transition logic
-
-## Acceptance Criteria
-
-- [x] `Update()` reduced to a dispatch that calls named handler functions
-- [x] `handleBoardKey(msg tea.KeyMsg, m Model) (Model, tea.Cmd)` extracted for board-level key handling
-- [x] Per-overlay handler functions extracted: `handleHelpOverlay()`, `handleEpicOverlay()`, `handleReleaseOverlay()`, `handleDetailOverlay()`, `handleEpicDetailOverlay()`
-- [x] `handleAdvanceTask()` and `handleRetreatTask()` extracted from the near-duplicate space/backspace handlers
-- [ ] `update.go` total line count reduced by at least 30%  *(not met — see Drift Notes)*
-- [x] All existing tests pass without modification (behavior-preservation refactor)
-- [x] `go test ./...` passes
-
-## Implementation Plan
-
-- [x] Extract `handleBoardKey(msg tea.KeyMsg, m Model) (Model, tea.Cmd)` from `Update()` main switch
-- [x] Extract overlay update logic into `handleHelpOverlay`, `handleEpicOverlay`, `handleReleaseOverlay`, `handleDetailOverlay`, `handleEpicDetailOverlay`
-- [x] Extract `handleAdvanceTask()` and `handleRetreatTask()` from space/backspace handlers (they share near-identical structure)
-- [x] Refactor `Update()` to be a thin dispatch: `switch overlay { case none: handleBoardKey; case help: handleHelpOverlay; ... }`
-- [x] Verify all `update_test.go` tests still pass
-- [x] Run `make build && make test`
-
-## Drift Notes
-
-- AC `update.go` total line count reduced by at least 30% — not met: 543 lines vs 521 original (+22). The decomposition adds function-signature overhead from per-overlay handlers and dispatch that the original monolithic `updateOverlay()` did not have. Behavior is fully preserved and all 64 tests pass.

package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T006-async-io.md

@@ -1,40 +0,0 @@
----
-id: E13-audit-remediation/T006-async-io
-status: done
-objective: Extract synchronous filesystem I/O from Update() into tea.Cmd async pattern following Bubble Tea conventions
-depends_on:
-  - E13-audit-remediation/T005-decompose-update
----
-
-# T006: Extract I/O From Update() Into tea.Cmd Async Pattern
-
-## Context Files
-
-- `internal/board/update.go` — `writeTaskStatus()`, `writeRouterTask()`, `writeRouterReleaseEpic()`, `readEpicDetailFile()`, `selectEpicPanelEpic()` perform synchronous I/O inside `Update()`
-- `internal/board/model.go` — `writeRouterReleaseEpic()` and `writeRouterTask()` methods on Model
-- `internal/board/watch.go` — `reloadTasks` silently swallows errors (returns `nil` on error)
-
-## Acceptance Criteria
-
-- [x] All filesystem reads in `Update()` dispatch `tea.Cmd` functions that return result messages
-- [x] All filesystem writes in `Update()` dispatch `tea.Cmd` functions that return result messages
-- [x] `writeRouterTask` extracted to a `tea.Cmd` that returns a `routerWriteMsg` on success or `errorMsg` on failure
-- [x] `writeRouterReleaseEpic` extracted to a `tea.Cmd` that returns a `routerWriteMsg` on success or `errorMsg` on failure
-- [x] `writeTaskStatus` for task transitions extracted to a `tea.Cmd` that returns a `taskWriteMsg` or `errorMsg`
-- [x] `readEpicDetailFile` extracted to a `tea.Cmd` that returns the file content as a message
-- [x] `reloadTasks` in `watch.go` returns an `errorMsg` on failure instead of `nil`
-- [x] Status messages for write errors are displayed to the user (existing error display mechanism)
-- [x] No synchronous `os.ReadFile`, `os.WriteFile`, or `os.Stat` calls remain in `Update()` or `handleBoardKey()`
-- [x] `go test ./...` passes with no regressions
-
-## Implementation Plan
-
-- [x] Define message types: `routerWriteMsg`, `taskWriteMsg`, `epicDetailMsg`, `auditContentMsg`, `errorMsg`
-- [x] Create `writeRouterTaskCmd` and `writeRouterReleaseEpicCmd` that write router state and return `routerWriteMsg` or `errorMsg`
-- [x] Create `writeTaskStatusCmd` for task transitions returning `taskWriteMsg` or `errorMsg`
-- [x] Create `readEpicDetailCmd` and `readEpicAuditCmd` for file reads
-- [x] Extract `selectEpicPanelEpic` I/O into a `tea.Cmd`
-- [x] Update `Update()` to dispatch commands and handle result messages in new branches
-- [x] Fix `reloadTasks` in `watch.go` to return `errorMsg` on error
-- [x] Update tests for async message pattern
-- [x] Run `make build && make test`

package/.savepoint/releases/v1.1/epics/E13-audit-remediation/tasks/T007-test-coverage.md

@@ -1,37 +0,0 @@
----
-id: E13-audit-remediation/T007-test-coverage
-status: done
-objective: Add tests for untested packages (buildtool, styles), convert repair matching to typed errors, fix minor test issues
-depends_on:
-    - E13-audit-remediation/T001-safe-cleanup
----
-
-# T007: Test Coverage — Buildtool, Styles, Typed Repairs, Minor Fixes
-
-## Context Files
-
-- `internal/buildtool/main.go` — zero test files
-- `internal/styles/palette.go` — zero test files
-- `internal/styles/styles.go` — zero test files
-- `internal/doctor/repairs.go` — substring-based error matching fragile
-- `internal/data/errors.go` — existing sentinel errors
-- `agent_skills_test.go` — hardcodes expected skill count of 6
-
-## Acceptance Criteria
-
-- [ ] `internal/buildtool/main.go` has test coverage for `version()`, `splitCommand()` (if exists), and `localExecutable()`
-- [ ] `internal/styles/` has a test file verifying palette constants and color completeness
-- [ ] `SuggestRepair` in `repairs.go` accepts `error` and uses `errors.Is()` with sentinel errors from `data/errors.go` (and any new ones needed)
-- [ ] New sentinel errors added to `data/errors.go` for repair-matching: at minimum `ErrInvalidStatus`, `ErrMissingFrontmatter`, `ErrConfigNotFound`, `ErrStructureProblem`
-- [ ] `agent_skills_test.go` no longer hardcodes the skill count (derives it from directory listing)
-- [ ] `go test ./...` passes
-
-## Implementation Plan
-
-- [ ] Create `internal/buildtool/main_test.go` with tests for `version()` (env var, flag override, git tag fallback) and `localExecutable()`
-- [ ] Create `internal/styles/styles_test.go` verifying all 3 color tiers exist for each named constant, and `color()` produces correct CompleteColor
-- [ ] Add new sentinel errors to `data/errors.go` for doctor repair matching
-- [ ] Refactor `SuggestRepair` to accept `error` and use `errors.Is()` instead of substring matching
-- [ ] Update `repairs_test.go` to test typed error matching
-- [ ] Fix `agent_skills_test.go` to derive expected skill count from directory listing
-- [ ] Run `make build && make test`