Skip to content

Feat/mangadm refactor#7

Open
xMohnad wants to merge 12 commits into
mainfrom
feat/mangadm-refactor
Open

Feat/mangadm refactor#7
xMohnad wants to merge 12 commits into
mainfrom
feat/mangadm-refactor

Conversation

@xMohnad

@xMohnad xMohnad commented Oct 8, 2025

Copy link
Copy Markdown
Owner

BREAKING CHANGES:

  • Removed component-based architecture and base classes
  • Replaced Rich console with logging system
  • Rewrote core downloader logic
  • Moved MangaArchiver from components/ to top-level module
  • Consolidated types into schema module
  • Updated Python requirement from 3.12+ to 3.10+
  • Removed obsolete config files (.python-version, MANIFEST.in)
  • Removed old Click-based CLI and utility classes
  • Deleted cli_util, main, and CLI submodules
  • Updated main.py to use Typer app()
  • Removed obsolete CLI configuration handling
  • Renamed not_available_error.webp to placeholder_image.webp
  • Removed epub_style_css() and deleted style.css
  • Simplified build_chapter_content():
    • Accepts list of Path objects
    • Generates HTML content for EPUB chapters with images only

New Features:

  • Added schema validation with Manga, MangaDetails, and Chapter dataclasses
  • Added configurable retry logic and HTTP error handling
  • Integrated dacite for JSON deserialization
  • Centralized configuration via YAML at config path
  • Added show_config command to display current configuration
  • Added verbose logging option

ARCHIVER IMPROVEMENTS:

  • Replaced inheritance with standalone MangaArchiver
  • Simplified image path discovery using pathlib
  • Added dynamic archiver method lookup via getattr()
  • Unified archive creation interface with create_archive()
  • Enhanced EPUB creation: simplified spine & TOC, removed manual CSS/styling

For more details, see the updated files.

@xMohnad xMohnad force-pushed the feat/mangadm-refactor branch from 69753f5 to 67a714e Compare January 8, 2026 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant