3 people 5e5cac4883 fix: address copilot and coderabbit review feedback ...

- Use named returns for IsAlreadyConverted and WriteChapterToCBZ so
  errs.Capture defers propagate close errors to the caller.
- Add context.Context to IsAlreadyConverted and ExtractChapter for
  cancellation/timeout support; thread it from Optimize.
- Filter non-image files (__MACOSX, Thumbs.db, .DS_Store, etc.) and
  only extract supported image extensions in ExtractChapter.
- Validate TempDir is non-empty in ConvertChapter before creating output dir.
- Separate fatal conversion errors from PageIgnoredError in the WebP
  converter result aggregation so real failures aren't masked.
- Fix error message "failed to load chapter" → "failed to extract chapter".
- Use %w instead of %v in fmt.Errorf for proper error wrapping.
- Fix LoadChapter doc comment to match its actual behavior.
- Extract shared parseConvertedComment/parseConvertedCommentTime helpers
  to avoid duplicated zip-comment parsing logic.
- Convert tests to use testify assertions (assert/require) per guidelines.

Co-authored-by: Belphemur <197810+Belphemur@users.noreply.github.com>

2026-07-04 22:39:19 +00:00

..

cbz_creator_test.go

refactor!: disk-first zero-copy architecture with file-to-file cwebp conversion

2026-07-04 22:18:22 +00:00

cbz_creator.go

fix: address copilot and coderabbit review feedback

2026-07-04 22:39:19 +00:00

cbz_loader_test.go

fix: address copilot and coderabbit review feedback

2026-07-04 22:39:19 +00:00

cbz_loader.go

fix: address copilot and coderabbit review feedback

2026-07-04 22:39:19 +00:00