Skip to content

Conversation

Copy link

Copilot AI commented Nov 4, 2025

Finalized documentation and implementation for the avendehut CLI tool based on pending review feedback.

Documentation

  • Installation: Added pip/pipx installation instructions for users, separated from development setup
  • Usage: Added comprehensive examples for both local and OneDrive workflows
  • CI/CD: Created .github/workflows/publish.yml for PyPI package publishing with trusted publishing support

Implementation

  • Path handling: Updated build and watch commands to detect and handle both local paths and OneDrive URIs (onedrive:/path)
  • Metadata extraction: Extended PDF metadata extraction to include subject, keywords, creation_date, modification_date, creator, and producer
  • Type safety: Added mypy overrides for modules with third-party library type challenges to maintain strict type checking on core code

Configuration

  • Created .flake8 config (flake8 doesn't read pyproject.toml natively)
  • Added mypy overrides for dynamic typing in OneDrive/search modules

Example usage:

# Local folder
avendehut build --src ./books --out ./dist

# OneDrive path
export ONEDRIVE_CLIENT_ID="..." ONEDRIVE_CLIENT_SECRET="..." ONEDRIVE_TENANT_ID="consumers"
avendehut build --src "onedrive:/Documents/Books" --out ./dist

Future work

Handling multiple formats of the same book (EPUB, PDF, MOBI) would require architectural changes to group books by base filename into a unified "book" model - added TODO comment.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Add initial documentation for avendehut CLI tool Complete avendehut CLI documentation and implementation Nov 4, 2025
Copilot AI requested a review from khnumdev November 4, 2025 02:24
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.

2 participants