3 Commits (71ff5cb5c3e7d59d33573757b49d7ec679f56fd3)

Author	SHA1	Message	Date
Broque Thomas	71ff5cb5c3	Retire artists.js and inline Artists page, ship unification at 2.40 ... Part D + E of the deferred cleanup + the final version bump that publishes the whole Search/Artists unification project. Deletions: - webui/static/artists.js (1903 lines) — removed entirely. The 2 remaining externally-referenced helpers (lazyLoadArtistImages + showCompletionError) moved into shared-helpers.js first. - webui/index.html — 140-line #artists-page HTML block and the <script src="artists.js"> tag both removed. init.js wiring: - 'case artists:' removed from loadPageData switch (no page to init). - navigateToPage top-level alias extended: 'artists' → 'search' (same pattern as the existing 'downloads' → 'search' alias). Legacy /artists bookmarks land on the unified Search page, the natural place to find an artist now. - _getPageFromPath now maps artist-detail → library as its parent (was artists). Matches the existing library-nav-highlight at init.js:2161. Version bump: - _SOULSYNC_BASE_VERSION 2.39 → 2.40. - WHATS_NEW entries lose the 'unreleased' scaffolding and gain a new top entry summarizing the unified artist-detail page + the final artists.js retirement. - version-info modal gets a 'Search & Artists Unification' section at the top. - The _getLatestWhatsNewVersion filter added during the unreleased- tracking phase is rolled back — entries now display as soon as they land in WHATS_NEW, matching the pre-unification behaviour. Test suite: - tests/test_script_split_integrity.py SPLIT_MODULES updated: 'artists.js' dropped, 'shared-helpers.js' added. escapeHtml's cross-file dupe list entry updated to reference shared-helpers. - 354/354 tests pass. User-visible result after this commit: - Sidebar: Search, Downloads, Discover, Library, Wishlist, etc. — no more Artists entry. - Click any artist anywhere: lands on the same /artist-detail page. - Search page has a source dropdown; Soulseek is just another option. - Legacy /downloads and /artists URLs alias to /search. - Version button shows v2.3 (Docker major); "What's New" panel opens to the unification summary. Closes the project Cin requested in Discord. Future work: source-aware /api/artist-detail could be extended to fall back through the whole source priority chain when a specific source is given but returns no discography. Not needed for the current flows.	1 month ago
Broque Thomas	a5d97261e4	Extract shared helpers from artists.js to shared-helpers.js ... Part C of the deferred unification cleanup. The Artists page is no longer in the sidebar, but its JS file can't be deleted yet because it houses ~20 general-purpose helpers that other modules depend on (escapeHtml used in 229 places, service-status polling, image-colour extraction, download-bubble infrastructure, discography completion checking, enrichment card rendering). Moved all non-page-specific code from artists.js into the new webui/static/shared-helpers.js — pure copy/paste, zero logic change. Two contiguous blocks extracted: Block A (lines 1097..1398 of original artists.js): discography completion suite — checkDiscographyCompletion, handleStreaming- CompletionUpdate, cacheCompletionData, updateAlbumCompletion- Overlay, getCompletionStatusText, setAlbumDownloadedStatus, setAlbumDownloadingStatus. Block B (lines 2206..EOF of original artists.js): download-bubble infrastructure (artist + search + Beatport clusters with their snapshot/hydrate/modal/monitor helpers), openDownloadMissingModal- ForArtistAlbum, image-colour extractor and dynamic-glow helper, escapeHtml, service-status polling, renderEnrichmentCards. Function declarations in a plain <script> tag are auto-global, so all existing callers continue to resolve without any import/export changes. Load order in index.html: shared-helpers.js loads right after core.js (which defines the artistDownloadBubbles / search- DownloadBubbles / beatportDownloadBubbles globals these helpers use). Stats: artists.js: 4638 → 1903 lines (-2735) shared-helpers.js: new, 2762 lines No function duplicated between the two files All 357 tests pass (3 new from split-integrity parametrization) What's left in artists.js is purely the Artists page — search UI, detail view, state switching, watchlist button, discography loading. All of that is reachable only by typing /artists in the URL bar since the sidebar entry was retired in Phase 4b. Parts D + E will delete that remainder and the file itself.	1 month ago
JohnBaumb	77b069acf4	Add split integrity tests (61 tests)	1 month ago