[vavallee]

Closes #2055
Closes #2375
Closes #1985
Closes #3014
Closes #2423

Not included: #2793 is already fixed on main by #2824 (64095b0): SetFilePath no longer has the fallback write at all, it returns the load error instead. It can be closed against that.

What

One commit per issue.

#2375 refreshBookStatus rewrote the whole row. It read the book, derived status and paths, then wrote every column back through Update from that snapshot, so a user edit landing in between (unmonitor, rename, media type) was reverted by an import recording its file. It now writes only the five columns it derives plus updated_at, guarded on the media_type and status it derived them from, and derives again (up to five attempts) when either changed. The second call site from #2758 (HydrateHardcoverEditions) was already moved onto UpdateHydratedMetadata in #2794.

#3014 scan requested during a scan. StartScan now sets a pending flag instead of refusing and returns ErrScanQueued; the endpoint answers 202 {"message":"library scan queued","queued":true}. The running scan runs exactly one follow-up when it finishes, however many requests arrived. Claim, release and the flag change together under a mutex so a request racing the end of a scan is never lost; a cancelled context drops the queued request. The cron path still skips rather than queues, but runs a manual request queued during it. Settings says the scan was queued instead of "Scan failed". Reported by foobarbigtime.

#1985 flat layout audiobooks. An audio file with no book folder of its own (directly in a root, or in Author/) no longer marks its parent as tracked. That folder instead remembers the track stems of its tracked files (name with part, track, chapter, disc numbers stripped), and only a sibling with the same stem or a bare number (02.mp3) is absorbed. Book folder layouts are unchanged. Known limit, documented: titles differing only by a number (Saga 1.mp3, Saga 2.mp3) still read as tracks of one book, which is the old behaviour for them.

#2423 book page live refresh. GET /book/{id} carries importInFlight, true while a download row for the book is grabbed, downloading, completed, importPending or importing. The parked states (importExternal, importHeld) are left out so a page never polls for hours. The page polls the book every 5s while it is set (via usePolling, so it pauses on hidden tabs) and refreshes history on the reload that clears it.

#2055 Fix match, correct match only. #2224 shipped the warning and destination preview; this is the deferred half. POST /queue/manual-import/reassign takes relocate. false moves the book_files row to the target through MoveBookFile, keeps the stored path spelling, writes a bookFileMoved history row and answers 200: no import, nothing on disk changes. true or absent is the old import, so API callers are unchanged. The modal asks what should happen to the file and defaults to Correct the match only; Also move and rename the file keeps the destination preview and the no undo warning.

Fail before evidence

Each test below was run against the main code and failed, then passed with the fix:

• Unconfirmed: refreshBookStatus rewrites the whole books row from a stale snapshot, so a concurrent edit can be lost during a fanned out BatchImport #2375 TestRefreshBookStatus_KeepsConcurrentEdit, TestRefreshBookStatus_RederivesWhenInputsChange (with only the test seam applied): monitored and title overwritten, media_type clobbered to ebook and status imported.
• Queue a library scan requested while one is running, instead of rejecting it with 409 #3014 TestStartScan_QueuesARequestMadeDuringAScan (seam only): "request 1 during a running scan was refused instead of queued"; web says a scan requested during a running one is queued fails on main's GeneralTab.
• Library scan: an untracked audiobook beside a tracked one is never reconciled in a flat Author/Title.mp3 layout #1985 TestScanLibrary_FlatLayoutUntrackedAudiobookReconciles: Beta Business.mp3 not reconciled. TestScanLibrary_FlatLayoutSiblingTracksStayTracked passes on both, as a guard for Scan Library: parent-dir tracked-path suppression silently skips sibling ebooks in flat author folders #1436.
• Live refresh on the book page has never armed: it keyed off a book status nothing writes (#1161) #2423 TestBookGet_ImportInFlight: importInFlight absent; web reloads the book every 5s while importInFlight is set fails on main's page.
• Fix match moves and renames the file on disk with no warning, when it should only correct the metadata link #2055 TestFixMatch_CorrectMatchOnlyLeavesFileInPlace: main answers 202 and runs an import; web corrects the match only by default fails on main's modal.

Checks

gofmt -l: clean
go vet ./...: clean
golangci-lint (v2.11.4) on internal/api, db, importer, models: 0 issues
go test internal/api internal/db internal/importer internal/models: ok
web: npm ci, lint (0 errors, existing warnings only), typecheck, vitest 150 files / 1845 tests passed, build ok

No migration.

Docs

docs/API.md (scan queued answer, importInFlight, relocate), docs/User-Guide-Wiki.md (Fix match modes, flat layout scan), docs/Migrating-From-Readarr-Wiki.md (Fix match now matches Readarr by default). One changelog fragment per issue.

Review follow ups

• Library scan: an untracked audiobook beside a tracked one is never reconciled in a flat Author/Title.mp3 layout #1985: the stem is now the text before the first track number (digits, or a spelled number after a track label), with track labels removed; a name starting with its number is a bare track. "01 - Chapter One", "Title - 01 - The Beginning", "Part One / Part Two", "(1 of 12)" and "CD1" stay tracked again; each shape is in TestScanLibrary_FlatLayoutTrackShapesStayTracked and failed on the previous head.
• Live refresh on the book page has never armed: it keyed off a book status nothing writes (#1161) #2423: HasImportInFlight takes the queue's QueryScope owner filter with ListScopeUserID; TestBookGet_ImportInFlightIsOwnerScoped fails unscoped.
• Queue a library scan requested while one is running, instead of rejecting it with 409 #3014: each scan gets an id (boot id plus sequence). The 202 returns scanId, the stored result carries scan_id, and the status adds live running/queued. Settings waits for its own scan_id and gives up after two minutes only when the server reports nothing walking or queued. Added the scheduled scan + queued manual + skipped second tick test.
• Unconfirmed: refreshBookStatus rewrites the whole books row from a stale snapshot, so a concurrent edit can be lost during a fanned out BatchImport #2375: the guard also compares the raw ebook_file_path/audiobook_file_path columns (with IS); TestRefreshBookStatus_KeepsConcurrentPathMove fails without it.
• Fix match moves and renames the file on disk with no warning, when it should only correct the metadata link #2055: relinking a file nothing tracked writes a history row too.

Merged main (no conflicts). Rechecked: vet, golangci-lint 0 issues, go test api/db/importer/models ok, web lint/typecheck, vitest 150 files / 1852 tests, build.

🤖 Generated with Claude Code

https://claude.ai/code/session_016fJcCVbNnKsmj2MAAMwWj9

[codecov]

Codecov Report

❌ Patch coverage is 64.15584% with 138 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
internal/importer/path_owned.go	0.00%	60 Missing ⚠️
internal/db/books.go	68.18%	17 Missing and 4 partials ⚠️
internal/db/book_files.go	0.00%	12 Missing ⚠️
internal/db/downloads.go	0.00%	10 Missing ⚠️
internal/api/library.go	61.90%	7 Missing and 1 partial ⚠️
internal/api/manual_import.go	65.21%	6 Missing and 2 partials ⚠️
internal/importer/scanner.go	95.03%	5 Missing and 2 partials ⚠️
internal/api/books.go	63.63%	3 Missing and 1 partial ⚠️
web/src/pages/settings/GeneralTab.tsx	80.95%	2 Missing and 2 partials ⚠️
web/src/components/FixMatchModal.tsx	84.61%	1 Missing and 1 partial ⚠️
... and 2 more

📢 Thoughts on this report? Let us know!