[REFACTOR] - Refactor audio player service architecture by atlet99 · Pull Request #47 · Gosayram/jabook

atlet99 · 2025-12-06T20:39:49Z

Extract player logic into separate manager classes for better maintainability and separation of concerns.

Extract playback control logic to PlaybackController
Extract position management to PositionManager
Extract player state handling to PlayerStateHelper
Extract metadata management to MetadataManager
Extract notification handling to NotificationHelper
Extract playlist management to PlaylistManager
Extract sleep timer logic to SleepTimerManager
Extract unload logic to UnloadManager
Add FileDurationDatabaseService for caching file durations
Add chapterNumber support from native player
Improve error handling by using Object instead of Exception
Update version to 1.2.7+2;

…d mirror health checks, distinguishing expected failures from errors;

… with detailed information and empty result checks;

…g and search operations;

… indexing operations;

…mplement notification permission request, and display indexing progress UI;

…foreground service notification, and show clearing index status;

…ueries and add debug commands/logging for indexing;

…cator and consistent comment sorting;

…t, and library improvements across the application;

…te parser and repository tests accordingly;

…ry files;

…est detection, and enhance author extraction from titles;

…il UI with new metadata rows, and update string resources;

…data extraction and enhancing fallback logic for topic details and search results;

…picViewModel tests;

… a new helper function, accompanied by a new test;

…ance error handling and readability by standardizing the logging format. This includes updating logger calls to use structured logging practices in various services, ensuring consistent error reporting and improved clarity in log messages.

…t` type for error handling;

…ds, simplify torrent creation date assignment, and remove audiobook-specific forum indexing validation;

…unction signatures, and simplifying type imports;

…x when play is invoked after book completion;

… checks to use coroutines for debouncing and improved cancellation;

… Coil3 and re-enable the NotificationManager;

… with new UI components for lyrics and settings, and update the navigation drawer;

…d playback capabilities;

…reparation logic;

…sion, removing the custom NotificationManager;

…aSession handles it automatically;

…tate when playback is manually initiated;

…utility across the audio module;

… logging, and add player action logs;

… and handle audio becoming noisy;

…yback speed from user preferences, and ensure explicit foreground notification cancellation on service termination;

…icit release calls and listener removal for audio components;

…e audio player controller initialization with connection state tracking and retries;

…new playback-related dependencies into constructors;

atlet99 added this to the 1.2.7-beta milestone Dec 6, 2025

atlet99 self-assigned this Dec 6, 2025

atlet99 added the enhancement New feature or request label Dec 6, 2025

Abdurakhman R. added 27 commits January 1, 2026 01:47

[FIX] - remove parent category IDs from AUDIOBOOKS_FORUM_IDS constant;

4b1d250

[UPD] - bump patch version to 1.2.7+68;

d735a4f

[DOCS] - add rutracker infrastructure data flow documentation;

b712cb1

[REFACTOR] - adjust logging levels and messages for debug features an…

10098d7

…d mirror health checks, distinguishing expected failures from errors;

[UPD] - add diagnostic DAO methods and enhance indexed search logging…

f2c6333

… with detailed information and empty result checks;

[FEATURE] - add mirror domain logging and stability checks to indexin…

2b3905c

…g and search operations;

[UPD] - bump CHANGELOG;

d50f022

[UPD] - bump patch version to 1.2.7+69;

26acc65

[FEATURE] - enhance service logging and error handling for foreground…

be61d4b

… indexing operations;

[FEATURE] - add indexing start time and clearing progress tracking, i…

b35f8eb

…mplement notification permission request, and display indexing progress UI;

[FEATURE] - display elapsed time for indexing progress in the UI and …

d6ae6ef

…foreground service notification, and show clearing index status;

[FEATURE] - Implement fuzzy search for indexed topics using raw SQL q…

71b741d

…ueries and add debug commands/logging for indexing;

[UPD] - bump patch version to 1.2.7+70;

798e7d9

[FEATURE] - add topic details refresh functionality with loading indi…

7f4198f

…cator and consistent comment sorting;

[FEATURE] - Implement comprehensive audio playback, torrent managemen…

7f9090a

…t, and library improvements across the application;

[FIX] - map offline search results using toDomainFromIndex and upda…

bf25346

…te parser and repository tests accordingly;

[REFACTOR] - improve index verification logging in ForumIndexer;

aab6f41

[UPD] - bump patch version to 1.2.7+71;

89f57e8

[DOCS] - add .gitignore to exclude Quarto build artifacts and tempora…

6a368f7

…ry files;

[FEATURE] - adjust Rutracker parsing column indices, improve bad requ…

4fb2ae3

…est detection, and enhance author extraction from titles;

[UPD] - bump patch version to 1.2.7+72;

90e7a8e

[UPD] - bump patch version to 1.2.7+73;

bea0627

[FEATURE] - Introduce uploader and bitrate fields, enhance topic deta…

f552765

…il UI with new metadata rows, and update string resources;

[FEATURE] - Improve Rutracker parsing by prioritizing structural meta…

58de8a9

…data extraction and enhancing fallback logic for topic details and search results;

[TEST] - add test for parsing comments and avatars in topic details;

f318e07

[FEATURE] - Implement avatar preloading for topic comments and add To…

0849811

…picViewModel tests;

[FEATURE] - add quote parsing and refactor comment HTML cleaning into…

d94d7af

… a new helper function, accompanied by a new test;

Abdurakhman R. added 30 commits January 12, 2026 20:29

[REFACTOR] - logging infrastructure and introduce a structured `Resul…

561c24f

…t` type for error handling;

[REFACTOR] - Introduce NONE log level, explicitly type logger metho…

d8e5a02

…ds, simplify torrent creation date assignment, and remove audiobook-specific forum indexing validation;

[REFACTOR] - Improve code readability by reformatting logger calls, f…

dc12793

…unction signatures, and simplifying type imports;

[UPD] - bump patch version to 1.2.7+89;

dc0f530

[FEATURE] - reset book completion state and last completed track inde…

10b7d2e

…x when play is invoked after book completion;

[REFACTOR] - migrate PlayerListener notification updates and position…

f4ce4c8

… checks to use coroutines for debouncing and improved cancellation;

[FEATURE] - implement asynchronous notification artwork loading using…

eae4f94

… Coil3 and re-enable the NotificationManager;

[UPD] - bump patch version to 1.2.7+90;

797d3d6

[FEATURE] - Implement global crash handling, enhance the audio player…

cf0104c

… with new UI components for lyrics and settings, and update the navigation drawer;

[FEATURE] - Implement initial Android TV application with browsing an…

868ddc6

…d playback capabilities;

[FEATURE] - add smart rewind functionality and improve audio player p…

581267a

…reparation logic;

[UPD] - bump patch version to 1.2.7+91;

fd0f051

[REFACTOR] - migrate audio notification management to Media3 MediaSes…

49c1dd4

…sion, removing the custom NotificationManager;

[REFACTOR] - remove redundant manual notification update call as Medi…

28d6b47

…aSession handles it automatically;

[UPD] - bump CHANGELOG;

91da12a

[UPD] - bump patch version to 1.2.7+92;

8cf40a8

[FEATURE] - allow replaying completed books by resetting completion s…

655ad76

…tate when playback is manually initiated;

[UPD] - bump patch version to 1.2.7+93;

bbc4685

[REFACTOR] - replace direct Android Log calls with a custom LogUtils …

fd17363

…utility across the audio module;

[UPD] - bump patch version to 1.2.7+94;

1b12c51

[REFACTOR] - streamline player permission requests, standardize debug…

196f48a

… logging, and add player action logs;

[FEATURE] - configure ExoPlayer to keep the CPU awake during playback…

1ed8a45

… and handle audio becoming noisy;

[FEATURE] - Implement error skipping for playback errors, restore pla…

ea1a54e

…yback speed from user preferences, and ensure explicit foreground notification cancellation on service termination;

[UPD] - decrease default audio rewind duration from 15 to 10 seconds;

ecff8a5

[REFACTOR] - improve audio service resource management by adding expl…

d7edbe7

…icit release calls and listener removal for audio components;

[UPD] - bump patch version to 1.2.7+95;

88d0bc9

[FEATURE] - extend debug logging to non-production flavors and enhanc…

8ee701b

…e audio player controller initialization with connection state tracking and retries;

[FEATURE] - update various test setups to inject LoggerFactory and …

d9f026b

…new playback-related dependencies into constructors;

[UPD] - bump patch version to 1.2.7+96;

8977716

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[REFACTOR] - Refactor audio player service architecture#47

[REFACTOR] - Refactor audio player service architecture#47
atlet99 wants to merge 794 commits intomainfrom
feature/beta_1.2.7

atlet99 commented Dec 6, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

atlet99 commented Dec 6, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant