Breadway/breadarr
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Check eligibility before queuing a low-confidence match for review

Browse source 
process_item queued anything landing in the matcher's "needs review"
confidence band immediately, without ever checking whether the show/
season/episode/movie actually needed anything — that check only ran on
the auto-match path. A low-confidence match against an already-complete
show gained nothing from a human's yes/no, it was just noise that
reappeared every cycle the source kept re-listing the same old release
(verified live: fully-complete shows' season-pack re-releases piling up
in the review queue indefinitely). Reordered so the eligibility check
(movie/season-pack/episode) runs first regardless of confidence, and only
a genuinely-needed release ever reaches the queue-for-review decision.
This commit is contained in:
Breadway 2026-07-21 21:38:00 +08:00
parent d1909f3083
commit 2f0d8300ce
1 changed files with 24 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

30
breadarrd/src/scheduler.rs
View file

@ -474,12 +474,9 @@ async fn process_item(
let parsed = parser::parse(&item.title); let parsed = parser::parse(&item.title);
let candidate = match matcher.match_title(conn, &parsed.title_normalized)? { let (candidate, needs_review) = match matcher.match_title(conn, &parsed.title_normalized)? {
MatchOutcome::Auto(c) => c, MatchOutcome::Auto(c) => (c, false),
MatchOutcome::NeedsReview(c) => { MatchOutcome::NeedsReview(c) => (c, true),
matcher::queue_for_review(conn, &item.title, &c, Some(&item.link), Some(source_id))?;
return Ok(ProcessOutcome::QueuedForReview);
}
MatchOutcome::NoMatch => return Ok(ProcessOutcome::NoMatch), MatchOutcome::NoMatch => return Ok(ProcessOutcome::NoMatch),
}; };
@ -488,6 +485,16 @@ async fn process_item(
let mut episode_id: Option<i64> = None; let mut episode_id: Option<i64> = None;
let mut season_pack_number: Option<u32> = None; let mut season_pack_number: Option<u32> = None;
// Whether this match needs a human's confirmation (queued for review)
// or was confident enough to act on automatically, the show/season/
// episode/movie still has to actually need *something* first — a
// low-confidence title match against an already-complete show gains
// nothing from a human's yes/no, it's just noise that reappears every
// cycle the source keeps re-listing the same old release (verified
// live: fully-complete shows' season-pack re-releases piling up in the
// review queue indefinitely because this check only ever ran on the
// auto-match path). So this eligibility check runs before the
// queue-for-review decision below, not only on the auto-grab path.
if media_item.kind == "movie" { if media_item.kind == "movie" {
// A release carrying a season/episode/absolute-episode marker // A release carrying a season/episode/absolute-episode marker
// title-matched a movie by name alone — it's actually an episode // title-matched a movie by name alone — it's actually an episode
@ -527,6 +534,17 @@ async fn process_item(
episode_id = Some(eid); episode_id = Some(eid);
} }
if needs_review {
matcher::queue_for_review(
conn,
&item.title,
&candidate,
Some(&item.link),
Some(source_id),
)?;
return Ok(ProcessOutcome::QueuedForReview);
}
let anime = match media_item.tvdb_id { let anime = match media_item.tvdb_id {
Some(id) => is_anime(conn, id)?, Some(id) => is_anime(conn, id)?,
None => false, None => false,