3 Commits

Include renames

Author	SHA1	Message	Date
John O'Keefe	3ab294bf81	feat(scanner): store EPUB reading direction at scan time ... Fixed-layout EPUBs lean on the reading_direction column (the web reader forces book.dir = rtl from it when the file didn't set direction itself), but the scanner never populated it for EPUBs - only ComicInfo fed it. Meanwhile real Japanese EPUBs declare page-progression-direction on the OPF spine, which foliate reads client-side but nothing stored. Read the spine attribute in the structured OPF parser and map it into ReadingDirection in parseOPFContent (EPUB2/3, case-insensitive, plus 'right-to-left'/'left-to-right' spellings); undeclared stays empty rather than forcing ltr, preserving the editor's Auto default. Sidecar OPFs are metadata-only documents without spines, so the Calibre path is a no-op. The merge gap-fill copies an embedded-only direction into a blank sidecar field, and hand-set values keep winning through the existing OverrideReadingDirection protection. Tests: declared rtl/RTL/ltr, undeclared and unknown values staying empty, plus sidecar-wins vs embedded-fills merge cases. Existing manga EPUBs declaring rtl (verified live in-library) pick the value up on their next scan, feeding the API and reader config mobile clients consume.	2026-09-13 13:13:25 -04:00
John O'Keefe	1fa8ee3a59	feat(scanner): Calibre-aligned OPF metadata extraction ... Adopt Calibre's reading conventions for the Dublin Core metadata that parseOPFContent now pulls from the structured OPF parse: - Titles: EPUB3 title-type selection (prefer 'main', join a distinct subtitle with ': ' exactly as Calibre stores it). There is no separate subtitle column by design - Calibre-sidecar books arrive pre-joined, so a column would stay empty for most libraries and force every client to reimplement concatenation. - Genre: first dc:subject, mirroring the existing processGenresAndTags behavior of the Calibre-sidecar path; the embedded path never populated Genre before. Subjects stay one-element-one-tag - Library of Congress headings legitimately contain commas ("Holmes, Sherlock (Fictitious character) -- Fiction") and must not be split. - Identifiers: urn:isbn:/urn:asin: prefixed values parse in addition to opf:scheme attributes, and the scheme-less fallback now requires an ISBN-shaped value (10/13 digits, optional separators/trailing X) so URIs like the Gutenberg identifiers cannot masquerade as ISBNs - observed live on 'A Study in Scarlet'. - Series: EPUB3 belongs-to-collection with collection-type=series and group-position refines, ahead of the classic calibre:series metas. - Audiobookshelf metadata.json sidecars join their subtitle field into the title the same way. Tests cover title-type main+subtitle joining, belongs-to-collection series with fractional group-position, urn:isbn extraction, genre/tag parity, and comma preservation inside subject headings.	2026-09-12 23:45:18 -04:00
John O'Keefe	72d167005f	fix(scanner): resolve EPUB covers via structured OPF parsing, Calibre chain ... The cover lookup scraped the OPF with attribute-order-sensitive regexes. Real books serialize attributes in any order - Grand Central's '3 Days to Live' puts href before id on manifest items and content before name on the cover meta - so all three regex paths missed and the book fell through to filename guessing, extracting no cover at all. Attribute order is meaningless in XML; the regexes were never safe. Replace them with a structured parse (encoding/xml, namespace and attribute-order agnostic; see the new media_scanner_opf.go) and follow Calibre's read_raster_cover resolution order: 1. manifest item with properties=cover-image (non-(X)HTML media only) 2. <meta name=cover> resolved through the manifest, same media guard 3. first spine item that is itself a raster image (store manga) 4. NEW cover-page fallback: books declaring no raster cover at all - the classic EPUB2/Adobe cover.xhtml wrapper - are mined for <img src> / SVG <image xlink:href> references (Calibre renders the page with Qt; extracting the referenced image covers the practical cases without a rendering engine) 5. existing zip filename guessing stays as the last resort, and the old regex chain survives as findCoverInOPFLegacy for OPFs too malformed for a real XML parse. Hrefs are now URL-decoded and posix-normalized against the OPF's own path (path.Join semantics), so '../art/cover.jpg' from a nested cover page and %20-encoded names resolve correctly. Tests: attribute-order chaos modeled on the failing Patterson book, SVG-wrapped cover pages via guide references, image-first spines, and path resolution edge cases. Verified live against the real '3 Days to Live' EPUB, which previously produced no cover.	2026-09-12 23:45:01 -04:00