feat(sonarr): monitor new seasons when latest is requested - #3157
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (4)
Included review availability: Your plan includes up to 10 reviews per rolling hour; 9 remain after this review. 📝 WalkthroughWalkthroughThis PR adds a ChangesSonarr 'latest' season monitoring feature
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This change adds a localized Sonarr monitoring option with targeted tests and validation reported by the author; no actionable merge-blocking risk remains beyond normal checks and review. Sequence Diagram(s)sequenceDiagram
participant User
participant SonarrModal
participant MediaRequestSubscriber
participant resolveMonitorNewItems
participant Sonarr
User->>SonarrModal: Select 'Latest Season Requested' and request seasons
SonarrModal->>MediaRequestSubscriber: Submit media request with seasons
MediaRequestSubscriber->>resolveMonitorNewItems: Pass setting, requestedSeasons, and availableSeasons
alt Latest regular season is requested
resolveMonitorNewItems-->>MediaRequestSubscriber: Return 'all'
else Latest regular season is not requested
resolveMonitorNewItems-->>MediaRequestSubscriber: Return 'none'
end
MediaRequestSubscriber->>Sonarr: Create series with resolved monitorNewItems
Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
I don't think we want tests for Sonarr yet.
@fallenbagel wdyt? Maybe we should do add all *arr tests at once in another PR?
If so, the whole PR could be simplified without that resolveMonitorNewItems helper function.
|
This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged. |
Description
Adds a third option to the Sonarr Monitor New Seasons setting: Latest Season Requested.
Future seasons are monitored only when the request includes the latest regular season. Older-only requests use No New Seasons, while the existing All and None behavior remains unchanged.
How Has This Been Tested?
git diff --checkAll,None, latest-season selection, older-season selection, and specials4.0.17.2952No New SeasonsAll SeasonsScreenshots / Logs (if applicable)
The Simpsons with only older seasons requested:

S.W.A.T. with the latest season included:

Checklist:
pnpm buildpnpm i18n:extractSummary by CodeRabbit
New Features
Documentation
Tests