Merge branch 'develop-pdca' into develop #20

Merged

Gandalf merged 9 commits from develop-pdca into develop 2026-05-19 14:35:32 +01:00

Conversation 0 Commits 9 Files Changed 23

+4114 -301

9 Commits

Author	SHA1	Message	Date
gronod	9343486705	Fix all Vitest test failures after migration ... - Replace vi.mock('axios') with nock for HTTP request mocking (ES/CJS interop issue) - Fix RTorrentClient by mocking client.client.methodCall directly instead of xmlrpc module - Fix downloadClients.test.js by manually adding mock clients to registry - Fix qbittorrent.test.js to use getActiveDownloads() and normalized properties - Fix integration test env var mocks and error assertions - Fix SABnzbdClient size parsing and test fixtures - Fix RTorrentClient ETA calculation expectation All 261 tests now passing.	2026-05-19 13:53:09 +01:00
gronod	5342170ced	fix: convert test files to ES modules and fix qbittorrent test method calls ... - Convert all client test files from CommonJS require() to ES module import syntax - Convert downloadClients.test.js and integration/downloadClients.test.js to ES modules - Fix qbittorrent.test.js to use getActiveDownloads() instead of getTorrents() - All test files now use proper Vitest-compatible ES module syntax - Resolves Vitest import errors and QBittorrentClient method call errors	2026-05-19 12:19:04 +01:00
gronod	cc0e34b3d1	fix: convert all test files from jest to vitest and fix QBittorrentClient import ... - Convert RTorrentClient.test.js to use vi.mock() instead of jest.mock() - Convert QBittorrentClient.test.js to use vi.mock() instead of jest.mock() - Convert SABnzbdClient.test.js to use vi.mock() instead of jest.mock() - Convert TransmissionClient.test.js to use vi.mock() instead of jest.mock() - Convert downloadClients.test.js to use vi.mock() instead of jest.mock() - Convert integration/downloadClients.test.js to use vi.mock() instead of jest.mock() - Fix legacy qbittorrent.test.js to import QBittorrentClient from new location - Add getRtorrentInstances mock to downloadClients.test.js - Add RTORRENT_INSTANCES to integration test environment variables	2026-05-19 12:12:44 +01:00
gronod	e39f15d3d8	fix: update package-lock.json after adding xmlrpc dependency	2026-05-19 12:02:23 +01:00
gronod	bbcbf8d0f7	docs: polish rtorrent URL path documentation to exact specifications ... - Update .env.sample RTORRENT_INSTANCES section with exact comment format - Update README.md rTorrent table row with specific endpoint note - Add explicit "No path is automatically appended" statement in README - RTorrentClient.js already uses exact URL from config (no changes needed)	2026-05-19 11:58:42 +01:00
gronod	620f264861	fix: remove auto-appending of /RPC2 from RTorrentClient and finalize PDCA documentation ... - Remove auto-appending of /RPC2 from RTorrentClient constructor - Use exact URL from config (supports custom paths like whatbox.ca/xmlrpc) - Update .env.sample with clear URL path documentation and examples - Update README.md with comprehensive PDCA section and all download clients - Add URL path verification tests (whatbox.ca, custom paths, no auth) - Update architecture diagram to include Transmission and rTorrent - Update Docker Compose example to include all download clients - Update prerequisites to mention all supported download clients - Update "What It Does" and "The Matching Process" sections	2026-05-19 11:53:51 +01:00
gronod	a50e5a7d69	feat: add rtorrent client via PDCA ... - Implement RTorrentClient extending DownloadClient abstract class - Use xmlrpc package (v1.3.2) for XML-RPC communication - Support HTTP Basic Auth when credentials are configured - Map rTorrent states (d.state, d.is_active, d.is_hash_checking) to normalized statuses - Calculate ETA from download speed and remaining bytes - Add getRtorrentInstances() to config.js - Register RTorrentClient in downloadClients.js registry - Add 8 comprehensive unit tests covering all functionality - Update .env.sample with rtorrent configuration examples - Update ARCHITECTURE.md with rtorrent client details - Update ADDING-A-DOWNLOAD-CLIENT.md with rtorrent-specific notes	2026-05-19 11:40:31 +01:00
gronod	f095e6a2d1	Fix QBittorrentClient export in legacy qbittorrent.js ... Remove undefined QBittorrentClient export that was causing container startup failures. The actual implementation is now in server/clients/QBittorrentClient.js	2026-05-19 11:21:31 +01:00
gronod	bf3e1c353d	Implement Pluggable Download Client Architecture (PDCA) ... - Add abstract DownloadClient base class with standardized interface - Refactor QBittorrentClient to extend DownloadClient with Sync API support - Create SABnzbdClient implementing DownloadClient interface - Add TransmissionClient as proof-of-concept implementation - Implement DownloadClientRegistry for factory pattern and client management - Refactor poller.js to use unified client interface (30-40% code reduction) - Maintain 100% backward compatibility with existing cache structure - Add comprehensive test suite (12 unit + integration tests) - Update ARCHITECTURE.md with detailed PDCA documentation - Create ADDING-A-DOWNLOAD-CLIENT.md guide for future client additions Features: - Client-agnostic polling with error isolation - Consistent data normalization across all clients - Easy extensibility for new download client types - Zero breaking changes to existing functionality - Parallel execution with unified timing and logging	2026-05-19 11:18:19 +01:00