fix(servarr): increase default API timeout from 5000ms to 10000ms

[fallenbagel]

Description

Increases the default timeout for Servarr API requests from 5000ms to 10000ms. Users were frequently hitting timeout errors on endpoints like /rootfolder that perform disk I/O, particularly when Radarr/Sonarr is under load or querying network-mounted storage. The previous 5s default was too aggressive for these scenarios. Since responses from these endpoints are cached, the longer timeout has negligible impact on user experience.

How Has This Been Tested?

Screenshots / Logs (if applicable)

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)

Summary by CodeRabbit

• Improvements
  • Enhanced API request timeout handling to improve system reliability and reduce potential connection interruptions.

[coderabbitai]

📝 Walkthrough

Walkthrough

The default timeout value for the ServarrBase constructor has been increased from 5000 milliseconds to 10000 milliseconds. This adjustment affects initialization behavior without modifying the constructor signature or other parameters.

Changes

Cohort / File(s)	Summary
Timeout Configuration server/api/servarr/base.ts	Default constructor timeout parameter increased from 5000ms to 10000ms.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A rabbit hops through timeless halls,
Where timeouts tick in ancient calls,
From five to ten—more time to play,
The API waits a longer day! ⏰

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Merge Conflict Detection	⚠️ Warning	❌ Merge conflicts detected (6 files): ⚔️ .github/renovate/helm.json5 (content) ⚔️ charts/seerr-chart/Chart.yaml (content) ⚔️ charts/seerr-chart/README.md (content) ⚔️ docs/getting-started/third-parties/unraid.mdx (content) ⚔️ docs/migration-guide.mdx (content) ⚔️ server/api/servarr/base.ts (content) These conflicts must be resolved before merging into develop.	Resolve conflicts locally and push changes to this branch.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: increasing the default API timeout from 5000ms to 10000ms in the servarr module, which matches the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fallenbagel/fix/increase-arr-timeout

⚔️ Resolve merge conflicts (beta)

• Auto-commit resolved conflicts to branch fallenbagel/fix/increase-arr-timeout
• Create stacked PR with resolved conflicts
• Post resolved changes as copyable diffs in a comment

---

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

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

[Copilot]

Pull request overview

This PR increases the default Axios request timeout used by the Servarr (Radarr/Sonarr) API client base class, aiming to reduce user-facing timeout failures on slower endpoints (e.g., disk I/O heavy /rootfolder) under load.

Changes:

• Increased the default Servarr API request timeout from 5000ms to 10000ms.

---

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

[gauthier-th]

LGTM

[geotibi]

it would be nice to have the timeout configurable in the interface

[gauthier-th]

it would be nice to have the timeout configurable in the interface

Yes, this is planned.

[fallenbagel]

it would be nice to have the timeout configurable in the interface

Thats already planned for next version