fix(radarr): trigger search for existing monitored movies without files

[Kenshin9977]

Description

When a movie already exists in Radarr and is monitored but has no file, Seerr currently skips it entirely without triggering a search, even when preventSearch is set to false.

This PR adds a check in addMovie() so that when searchNow is true and the movie has no file yet, a search is triggered in Radarr via searchMovie(). This is consistent with the existing !monitored code path (line 140-184) which already updates and searches in the same way.

Use case: A user requests a movie via Seerr that was previously added to Radarr (monitored, no file). Previously, nothing happened. Now, Radarr will automatically search for it.

• Fixes Requested quality not applied in Radarr for existing movies #1822 (partially — search triggering only; quality profile updates are out of scope)

How Has This Been Tested?

Tested locally with Radarr running in Docker and Seerr in dev mode (pnpm dev):

1. Added a movie to Radarr as monitored with no file
2. Called addMovie() with searchNow: true via a test script
3. Verified Seerr logs show the new code path being triggered
4. Verified Radarr received and completed the MoviesSearch command for the movie

[info][Radarr]: Movie is already monitored in Radarr. {"movieId":1,"movieTitle":"Title"}
[info][Radarr]: Triggering search for existing monitored movie without file {"movieId":1,"movieTitle":"Title"}
[info][Radarr API]: Executing movie search command {"movieId":1}

Radarr command confirmation:

{"name": "MoviesSearch", "body": {"movieIds": [1]}, "status": "completed"}

• pnpm typecheck:server passes
• pnpm build passes

Checklist:

• I have read and followed the contribution guidelines.
• Disclosed any use of AI (see our policy)
• I have updated the documentation accordingly.
• All new and existing tests passed.
• Successful build pnpm build
• Translation keys pnpm i18n:extract
• Database migration (if required)

AI Disclosure

This PR was written with assistance from Claude Code. The fix logic and testing were reviewed and validated by a human.

Summary by CodeRabbit

• Improvements
  • Enhanced handling when adding existing movies to your library with improved logging for troubleshooting and automatic file search when unavailable.

[coderabbitai]

📝 Walkthrough

Walkthrough

Enhanced logging and conditional search triggering in Radarr's addMovie method when an existing monitored movie is requested with searchNow enabled and has no file.

Changes

Cohort / File(s)	Summary
Radarr API Enhancement server/api/servarr/radarr.ts	Improved logging context (movieId, movieTitle, hasFile) when handling existing monitored movies. Added search trigger logic: if options.searchNow is true and the movie lacks a file, initiates a search via searchMovie() before returning early.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A rabbit hops through code so neat,
When movies match, the search's sweet,
No file? No worries, search it now!
Radarr updates with grace and bow,
Quality flows where it should go! 🎬

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	❓ Inconclusive	The PR implements search triggering for existing monitored movies, partially addressing issue #1822, but quality profile updates remain out of scope and unimplemented.	This PR only partially addresses #1822 by adding search triggering; quality profile update functionality (the primary issue) is explicitly deferred and not included.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Out of Scope Changes check	✅ Passed	The changes are focused on adding search logic for monitored movies without files, which is directly related to the issue and PR objectives.
Title check	✅ Passed	The title clearly and specifically describes the main change: triggering searches for existing monitored movies without files when searchNow is true.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[gauthier-th]

Edit: my bad, I got confused.

[Copilot]

Pull request overview

This PR fixes an issue where Seerr would not trigger a search in Radarr when a user requests a movie that already exists in Radarr, is monitored, but has no file. The fix adds logic to trigger a search when searchNow is true for such movies, making the behavior consistent with the existing code path for unmonitored movies.

Changes:

• Enhanced logging for existing monitored movies to include movieId, movieTitle, and hasFile status
• Added conditional logic to trigger search for existing monitored movies without files when searchNow is requested

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.