refactor: audit, split server.go, add unit tests, and fix latent bugs
- Remove dead code: browser cdp/content_scrape strategies, writer package,
printUsage, downloadAndStoreCoverCLI in main.go
- Fix bugs: defer-in-loop in pocketbase deleteWhere, listAll() pagination
hard cap removed, splitChapterTitle off-by-one in date extraction
- Split server.go (~1700 lines) into focused handler files:
handlers_audio, handlers_browse, handlers_progress, handlers_ranking,
handlers_scrape
- Export htmlutil.AttrVal/TextContent/ResolveURL; add storage/coverutil.go
to consolidate duplicate helpers
- Flatten deeply nested conditionals: voices() early-return guards,
ScrapeCatalogue next-link double attr scan, chapterNumberFromKey dead
strings.Cut line, splitChapterTitle double-nested unit/suffix loop
- Add unit tests: htmlutil (9 funcs), novelfire ScrapeMetadata (3 cases),
orchestrator Run (5 cases), storage chapterNumberFromKey/splitChapterTitle
(22 cases); all pass with go build/vet/test clean
fb6b364382