[kevinheneveld]

Summary

The automatic-search sweep ran on a hardcoded 6-hour cadence that also served as the "not searched since" cutoff, and it searched every eligible book back-to-back. On a large wanted list that hammers the indexers (and a shared Prowlarr). This makes the timing configurable and, per review, keeps the sweep cadence and the per-book eligibility separate: the worker can check hourly while each book is only searched once per cooldown.

Scope is timing only. The title-only fallback from the earlier revision is gone; that lane belongs to #1072 and the identity-matching follow-ups.

Changes

Added

• ApplicationSettings.AutomaticSearchIntervalHours (default 1, clamped [1, 168]) — how often the worker sweeps for eligible books. Only the cadence of the check.
• ApplicationSettings.AutomaticSearchCooldownHours (default 24, clamped [1, 720]) — how long a book stays ineligible after its last search. One shared cooldown for every book.
• ApplicationSettings.AutomaticSearchBookDelaySeconds (default 5, clamped [0, 300]) — pause between consecutive books inside one sweep. 0 disables it. Manual "Search now" is unaffected.
• AutomaticSearchTiming (HostedServices/Search/) — resolves and clamps the three from settings so AutomaticSearchService and AutomaticSearchProcessor share one definition; EligibilityCutoff(now) is now − cooldown.
• Migration 20261021212418_AddAutomaticSearchTiming (+ Designer, snapshot). Column defaults match the property initializers so the existing singleton settings row behaves exactly like a fresh install.

Changed

• AutomaticSearchService: the periodic runner's intervalProvider reads the sweep interval from settings each cycle (falls back to the default on a read error), so a change takes effect without a restart.
• AutomaticSearchProcessor: the eligibility cutoff comes from the cooldown instead of the sweep interval; the per-book delay is applied between books (not after the last one).

Removed

• The title-only fallback (AutomaticSearchTitleOnlyFallback, BuildTitleOnlySearchQuery, the retry in ProcessAudiobookAsync).

Testing

• AutomaticSearchTimingTests (new, 17 cases): defaults, each clamp at both bounds, and that the eligibility cutoff follows the cooldown rather than the sweep interval.
• MigrationMetadataTests.AddAutomaticSearchTiming_IsDiscoverableAndAddsSettingsColumnsWithDefaults: migration id, the three AddColumn operations with their defaults, the three drops, and the target model.
• SqliteMigrationSchemaTests (migrated-schema column parity, migration history, released-schema upgrades) and HostedServicesRegistrationTests pass with the new migration; ActiveProductionSourceFiles_RemainFocused still passes.
• Release build 0 errors; dotnet format --verify-no-changes clean on the touched files.

Notes

• Defaults follow the review suggestion (hourly sweep, 24h cooldown). Upstream's previous behaviour was 6h/6h; existing installs will sweep more often but search each book less often than before.
• No UI in this PR — the three settings are plain ApplicationSettings fields reachable through the settings API like the other untuned knobs. Happy to add a small Settings block in a follow-up if you'd like them surfaced.
• Rebased onto canary v1.3.17 (thanks for the rebase); the earlier July-dated migration is replaced by a fresh one so it sorts after 20261002210137_AddOperationEvidencePublications.

🤖 Generated with Claude Code

[therobbiedavis]

[Removed, meant to post with changes requested]

[therobbiedavis]

@kevinheneveld Could we keep this PR focused on configurable search timing and leave the fallback to #1072 and the identity-matching follow-ups already underway?

I’d separate the sweep interval from the per-book cooldown: the worker could check hourly while each book remains eligible only once every 24 hours. The delay between books would remain a separate throttle to protect indexers. Currently, AutomaticSearchIntervalHours controls both the sweep cadence and eligibility cutoff; could we give the cutoff its own setting? One shared cooldown for all books should be enough for now.

[kevinheneveld]

@therobbiedavis reworked as suggested, thanks for the rebase:

• Title-only fallback removed — left to Retry automatic search without the author when the full query returns nothing (Fixes #1068) #1072.
• Sweep cadence and eligibility are now separate settings: AutomaticSearchIntervalHours (default 1 — how often the worker checks) and AutomaticSearchCooldownHours (default 24 — one shared per-book cooldown that drives the LastSearchTime cutoff). AutomaticSearchBookDelaySeconds stays as the between-books throttle.
• A small AutomaticSearchTiming resolver holds the clamps so the service and processor read the same definition; 17 unit cases cover defaults/clamps and that the cutoff follows the cooldown, not the interval.
• Fresh migration (20261021212418_AddAutomaticSearchTiming) replacing the July-dated one so it sorts after AddOperationEvidencePublications; the migration-history and schema-parity tests are updated and green.

Description updated to match. No UI in this PR — say the word if you'd like a Settings block for the three and I'll add it as a follow-up.

[therobbiedavis]

@kevinheneveld reviewed and tested the updated head. The fallback is removed and the sweep interval, per-book cooldown, and delay are separate now. 57 tests passed with 2 platform skips, along with 19 live container checks. No remaining blockers from me.

[therobbiedavis]

@kevinheneveld Thanks, this matches what we discussed. API-only is fine for this PR. Yes, let’s do a follow-up to add the Settings block.

[therobbiedavis]

@kevinheneveld Please fix the failing tests. Thank you!

[kevinheneveld]

Fixed in 950bb94 (amended, same single commit on your rebase): AutomaticSearchTimingTests now inherits BaseTests (the TestClasses_FollowRepositoryConventions violation on Windows) and the two usings that format:backend:check flagged as IDE0005 are gone (they were already in GlobalUsings.cs). Locally: BackendArchitectureTests 45/45, the timing tests 17/17, dotnet format listenarr.slnx --verify-no-changes clean. Nothing else changed.