What
Epic #1624 WebUI UX audit constraint: /preview/* routes must be removed once consolidated (preview becomes prod). Also naming hygiene.
Changes
- Delete
internal/webui/templates/preview/ — 6 files. Routes already removed from routes.go.
- Delete
internal/webui/preview.go — main preview package (not stub).
- Delete
internal/webui/preview_services.go
- Delete
internal/webui/preview_test.go
- Keep
internal/webui/preview_disabled.go — //go:build !webui_preview stub, no-op for non-preview builds.
- Rename
nav_consolidated.html → nav_main.html — proper noun not process descriptor.
Constraints
- API contract unchanged (no handler/data model changes)
- All work targets
alpha branch
- Commit signed via CLAUDE.md gitsign hook (wave pipeline, not worktree shortcut)
Acceptance
go build ./internal/webui/ passes (no webui_preview tag)
go build -tags=webui_preview ./internal/webui/ passes
- No
templates/preview/ directory remains
- No
preview.go, preview_services.go, preview_test.go remain
nav_main.html exists, nav_consolidated.html gone
What
Epic #1624 WebUI UX audit constraint:
/preview/*routes must be removed once consolidated (preview becomes prod). Also naming hygiene.Changes
internal/webui/templates/preview/— 6 files. Routes already removed from routes.go.internal/webui/preview.go— main preview package (not stub).internal/webui/preview_services.gointernal/webui/preview_test.gointernal/webui/preview_disabled.go—//go:build !webui_previewstub, no-op for non-preview builds.nav_consolidated.html→nav_main.html— proper noun not process descriptor.Constraints
alphabranchAcceptance
go build ./internal/webui/passes (no webui_preview tag)go build -tags=webui_preview ./internal/webui/passestemplates/preview/directory remainspreview.go,preview_services.go,preview_test.goremainnav_main.htmlexists,nav_consolidated.htmlgone