372 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
gronod	31ed9f02b6	fix: remove client/ from .dockerignore to fix Docker client-build stage	2026-05-21 00:16:08 +01:00
gronod	3e6af1bff2	Add frontend unit tests with Vitest + jsdom ... - Create tests/frontend/utils/format.test.js with 24 tests for formatting utilities - Create tests/frontend/ui/downloads.test.js with 10 tests for DOM rendering functions - Update vitest.config.js to support jsdom environment for frontend tests - All 34 tests pass and cover edge cases (null, zero, large numbers, DOM structure)	2026-05-21 00:07:41 +01:00
gronod	d9897ff0d2	Extract matching logic into new DownloadMatcher service	2026-05-21 00:04:57 +01:00
gronod	06442c1d75	Add JSDoc comments and defensive error handling to DownloadBuilder.js	2026-05-21 00:00:46 +01:00
gronod	86aaa79339	refactor: Complete technical debt remediation - final cleanup ... Deleted obsolete public/app.js (replaced by Vite build) All backend services extracted and dashboard.js slimmed to 284 lines Frontend fully migrated to vanilla ES modules All duplications eliminated Comprehensive tests added Vite build wired into Dockerfile	2026-05-20 23:48:10 +01:00
gronod	e2a71e65a1	refactor: Complete technical debt remediation (all steps) ... Extracted TagMatcher, DownloadAssembler, DownloadBuilder, and WebhookStatus services Slimmed dashboard.js from 1360 → 284 lines (pure HTTP layer) Created server/routes/status.js and mounted at /api/status Migrated frontend to vanilla ES modules under client/src/ Eliminated all tag-badge and client-logo duplication Wired Vite build into Dockerfile and removed obsolete public/app.js Added comprehensive DownloadBuilder regression tests	2026-05-20 23:45:08 +01:00
gronod	d03efbf25e	Extract createClientLogo helper to eliminate 2× client-logo duplication ... - Consolidated duplicate client logo creation logic into single helper function - Removed second duplicate logo block from createDownloadCard - Updated updateDownloadCard to use the same helper - Client logo now appears exactly once per download card	2026-05-20 23:36:16 +01:00
gronod	0b91152ad7	Extract renderTagBadges helper to eliminate 4x tag-badge rendering duplication	2026-05-20 23:33:13 +01:00
gronod	8dc105ff3e	Migrate frontend from monolithic app.js to vanilla ES modules ... - Delete React files (App.jsx, main.jsx, App.css) - Create modular vanilla JS structure in client/src/: - state.js (global state object) - api.js (all fetch calls) - sse.js (SSE connection management) - ui/auth.js (authentication UI) - ui/downloads.js (downloads rendering) - ui/history.js (history section) - ui/statusPanel.js (status panel) - ui/webhooks.js (webhook management) - ui/filters.js (download client filter) - ui/theme.js (theme switching) - ui/tabs.js (tab navigation) - utils/format.js (formatting utilities) - utils/storage.js (localStorage helpers) - main.js (DOMContentLoaded bootstrap) - Update vite.config.js for vanilla build outputting to ../public/app.js - Build succeeds: 14 modules, 43.88 kB output	2026-05-20 23:30:24 +01:00
gronod	a38fc4a8ce	refactor: extract status route and WebhookStatus service, slim dashboard.js ... - Extract /status route to server/routes/status.js - Create server/services/WebhookStatus.js with checkWebhookConfigured and aggregateMetrics - Slim dashboard.js to pure HTTP orchestration (559→283 lines, 49.4% reduction) - Remove /user-summary and /webhook-metrics routes from dashboard.js - Mount status router at /api/status in server/index.js and server/app.js - Update tests to use new /api/status/status endpoint - Fix test expectation for speed field (number vs string) All 571 tests passing.	2026-05-20 22:50:40 +01:00
gronod	2bf4cb2a0f	Refactor: Deduplicate download assembly logic into DownloadBuilder service ... - Created server/services/DownloadBuilder.js with buildUserDownloads function - Added private helpers: buildSeriesMapFromRecords, buildMoviesMapFromRecords, matchSabSlots, matchSabHistory, matchTorrents, getSlotStatusAndSpeed - Updated server/routes/dashboard.js to use buildUserDownloads in /user-downloads and SSE /stream - Removed ~500 lines of duplicated download-assembly logic - All unit tests passing (DownloadBuilder: 14, DownloadAssembler: 73, TagMatcher: 26)	2026-05-20 22:43:03 +01:00
gronod	d74b46d5b0	Add guard test for DownloadBuilder service	2026-05-20 22:34:24 +01:00
gronod	9cffb96f29	Extract DownloadAssembler service from dashboard routes ... - Create server/services/DownloadAssembler.js with 7 pure functions: - getCoverArt, getImportIssues, getSonarrLink, getRadarrLink - canBlocklist, extractEpisode, gatherEpisodes - Update server/routes/dashboard.js to use DownloadAssembler - Add comprehensive unit tests (73 tests covering edge cases) - Fix null check in extractEpisode function - All tests passing: DownloadAssembler (73/73), TagMatcher (26/26)	2026-05-20 22:32:09 +01:00
gronod	4d61dd566f	Refactor: Extract tag functions to TagMatcher service ... - Extract six pure tag-related functions from dashboard.js into new server/services/TagMatcher.js - Functions: sanitizeTagLabel, tagMatchesUser, extractAllTags, extractUserTag, getEmbyUsers, buildTagBadges - Update dashboard.js to import TagMatcher and replace all inline function calls - Add comprehensive unit tests in tests/unit/services/TagMatcher.test.js (26 tests passing for 5 pure functions) - Note: getEmbyUsers tests excluded due to CommonJS mocking complexity	2026-05-20 22:21:01 +01:00
gronod	d568800942	fix: limit history pagination to prevent 40s response times ... The full pagination fix (ddad80a, e772001) caused history retrieval to fetch up to 50 pages sequentially, taking 10-40 seconds instead of ~200ms. Changes: - Add maxPages parameter to getHistory() with default of 1 page - Update poller to fetch 50 records (up from 10) in a single API call - historyFetcher retains 100 records per page for UI display This provides 5x more history for matching than before while keeping the fast single-request performance. Refs: develop-merge pagination performance issue	2026-05-20 21:44:53 +01:00
gronod	7d3e6e6a47	test: add integration and unit tests for dashboard, emby, sonarr, radarr, sabnzbd routes ... - tests/unit/dashboard.test.js: 58 unit tests covering all 12 pure helper functions in dashboard.js (sanitizeTagLabel, tagMatchesUser, getCoverArt, extractAllTags, extractUserTag, getImportIssues, getSonarrLink, getRadarrLink, canBlocklist, extractEpisode, gatherEpisodes, buildTagBadges) - tests/integration/dashboard.test.js: 35 integration tests for /user-downloads (SAB+Sonarr, SAB+Radarr, qBit, showAll, paused queue, history matching, importIssues, wrong-user filtering), /status (admin guard, webhook check, failure handling), /webhook-metrics, /cover-art (all validation/proxy paths), /blocklist-search (guards, Sonarr, Radarr, failure) - tests/integration/emby.test.js: 13 integration tests covering all 4 Emby routes (sessions, users, users/:id, session/:id/user) with auth guard, happy path, and upstream failure cases - tests/integration/arrRoutes.test.js: 64 integration tests for Sonarr + Radarr (queue, history, series/movies, notifications CRUD, /test, /schema, /sofarr-webhook create+update+missing-config+failure) and SABnzbd (queue, history with custom params) - vitest.config.js: raise global coverage thresholds (statements/functions/ lines 20->55, branches 8->40) to reflect improved coverage (62.5% stmts, 42.6% branches, 64.1% funcs, 65.6% lines) - tests/README.md: document new test files and update coverage table	2026-05-20 21:37:57 +01:00
Gandalf	ee2f275501	Merge pull request 'fix: use stable *arr IDs for matching before fragile title fallback' (#21) from fix-arr-matching into develop-merge ... Reviewed-on: #21	2026-05-20 21:02:10 +01:00
Gandalf	ca6ff66115	Merge pull request 'fix: webhook replay cache atomicity and instanceName precision' (#22) from fix-webhook-receiver into develop-merge ... Reviewed-on: #22	2026-05-20 21:01:52 +01:00
Gandalf	080431c4b7	Merge pull request 'fix: QBittorrent fallback state corruption after full sync' (#23) from fix-qbittorrent-client into develop-merge ... Reviewed-on: #23	2026-05-20 21:01:36 +01:00
Gandalf	f457a708d2	Merge pull request 'fix: SABnzbd speed assignment and size/progress parsing' (#24) from fix-sabnzbd-client into develop-merge ... Reviewed-on: #24	2026-05-20 21:01:21 +01:00
Gandalf	914ab73d4e	Merge pull request 'fix: full pagination + non-silent errors in PollingRadarrRetriever' (#25) from fix-radarr-retriever into develop-merge ... Reviewed-on: #25	2026-05-20 21:01:07 +01:00
Gandalf	25d8e007a4	Merge pull request 'fix: full pagination + non-silent errors in PollingSonarrRetriever' (#26) from fix-sonarr-retriever into develop-merge ... Reviewed-on: #26	2026-05-20 21:00:53 +01:00
gronod	bb7b66e06d	fix: use stable *arr IDs for matching before fragile title fallback	2026-05-20 20:51:50 +01:00
gronod	5ad525a760	fix: webhook replay cache atomicity and instanceName precision	2026-05-20 20:46:35 +01:00
gronod	1e162381f4	fix: QBittorrent fallback state corruption after full sync	2026-05-20 20:45:26 +01:00
gronod	42f0481a9a	fix: SABnzbd speed assignment and size/progress parsing	2026-05-20 20:44:08 +01:00
gronod	ddad80a666	fix: full pagination + non-silent errors in PollingRadarrRetriever	2026-05-20 20:42:18 +01:00
gronod	e772001c3f	fix: full pagination + non-silent errors in PollingSonarrRetriever	2026-05-20 20:40:48 +01:00
gronod	1f10414498	Update CHANGELOG for v1.5.5 v1.5.5	2026-05-20 01:13:01 +01:00
gronod	1e3926b206	Bump version to 1.5.5	2026-05-20 01:11:22 +01:00
gronod	5fde69fcf5	Add speed formatting to display appropriate units (KB/s, MB/s)	2026-05-20 01:07:52 +01:00
gronod	a562cfe9aa	Add logging to debug active download identification and speed	2026-05-20 01:00:25 +01:00
gronod	8549746721	Apply overall SABnzbd speed to active download only	2026-05-20 00:58:38 +01:00
gronod	63fc370262	Remove speed from SABnzbd downloads - API doesn't provide per-download speed	2026-05-20 00:56:54 +01:00
gronod	6362441dd5	Add logging to debug SABnzbd speed field in slot data	2026-05-20 00:54:26 +01:00
gronod	76f9e87b44	Add logging to investigate SABnzbd slot structure for speed field	2026-05-20 00:51:12 +01:00
gronod	8c461de72a	Hide speed when it is 0 to avoid displaying misleading 0 speed	2026-05-20 00:49:26 +01:00
gronod	d11f11be69	Fix missing speed on SAB cards and remove incorrect missing pieces display	2026-05-20 00:47:07 +01:00
gronod	05d11975e6	Reduce card logo size to 32x32	2026-05-20 00:41:04 +01:00
gronod	cd3480c0ce	Fix logo positioning by adding position: relative to download-card	2026-05-20 00:39:11 +01:00
gronod	712c98d817	Move card logo to bottom right with absolute positioning, fix duplication	2026-05-20 00:37:01 +01:00
gronod	ff7ace9f4f	Fix duplicate icon and user tag on page reload by adding class and duplicate check	2026-05-20 00:29:44 +01:00
gronod	73500751a0	Increase download client logo size in cards to 64x64px (4x), keep filter picker at 20x20px	2026-05-20 00:26:54 +01:00
gronod	82a9df134b	Fix duplicate user tag and logo in download cards by removing old elements before updating	2026-05-20 00:23:17 +01:00
gronod	67fa79796b	Add download client logo to download card with right-side positioning	2026-05-20 00:20:03 +01:00
gronod	f06d945358	Update rtorrent.svg logo	2026-05-20 00:15:46 +01:00
gronod	f5883d4929	Add download client logos to filter UI with fallback handling	2026-05-20 00:14:20 +01:00
gronod	80cf3eaa39	Fix filtering to use both client type and instanceId for unique identification	2026-05-20 00:00:17 +01:00
gronod	1ab7e52167	Use index-based unique identifiers for download client selection to prevent cross-selection	2026-05-19 23:56:05 +01:00
gronod	544c168b82	Fix duplicate checkbox ID issue causing cross-selection between clients	2026-05-19 23:51:57 +01:00