umputun / revdiff / 24287986461

90%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(ui): extract sidepane sub-package with FileTree and TOC (#93)

* add plan: sidepane-extraction

* feat(sidepane): bootstrap sidepane package with enums, interfaces, and shared helper

Add app/ui/sidepane/ sub-package skeleton with Motion/Direction enums
(via go-pkgz/enum), consumer-side Resolver/Renderer interfaces,
FileTreeRender/TOCRender option structs, and shared ensureVisible helper.
Includes table-driven tests for ensureVisible and enum exhaustiveness.

* feat(sidepane): implement FileTree type with all methods

Port filetree.go logic into app/ui/sidepane/filetree.go with exported
API: NewFileTree, Move(Motion), StepFile(Direction), HasFile(Direction),
Rebuild, SelectedFile, TotalFiles, FileStatus, FilterActive, ReviewedCount,
ToggleFilter, RefreshFilter, ToggleReviewed, SelectByPath, EnsureVisible,
and Render(FileTreeRender).

* feat(sidepane): port FileTree tests to sidepane package

* feat(sidepane): implement TOC type with all methods

* feat(sidepane): port TOC tests to sidepane package

* feat(sidepane): add consumer-side interfaces and factory injection to Model

Add FileTreeComponent and TOCComponent exported interfaces to ui/model.go,
change Model.tree and Model.mdTOC field types from concrete to interface,
add NewFileTree/ParseTOC factory fields to ModelConfig and Model, validate
factories in NewModel, and initialize m.tree with empty tree for nil-safety.

* feat(sidepane): migrate loaders.go call sites to new API

* feat(sidepane): migrate view.go call sites to new API

* feat(sidepane): migrate diffnav.go call sites to new API

* feat(sidepane): migrate handlers, annotate, annotlist, model call sites to new API

* feat(sidepane): wire factory closures in main.go

* feat(sidepane): delete old filetree.go and mdtoc.go from ui package

Remove the original filetree.go, filetree_test.go, mdtoc.go, and
mdtoc_test.go files that were ported to app/ui/sidepane/ in earlier
tasks. Relocate isMarkdownFile and isFullContext helper method... (continued)

Coverage Stats

443 of 517 new or added lines in 13 files covered. (85.69%)

2 existing lines in 1 file now uncovered.

6367 of 7257 relevant lines covered (87.74%)

359.75 hits per line