Skip to content

fix(cli): bootstrap optional CLI dependencies#36

Merged
smekcio merged 3 commits intomainfrom
fix/optional-cli-bootstrap
Mar 26, 2026
Merged

fix(cli): bootstrap optional CLI dependencies#36
smekcio merged 3 commits intomainfrom
fix/optional-cli-bootstrap

Conversation

@smekcio
Copy link
Copy Markdown
Owner

@smekcio smekcio commented Mar 26, 2026

Summary

  • keep ksef-client as the base SDK install and move CLI-only dependencies to the cli extra path
  • replace the eager ksef entrypoint import with a lightweight bootstrap that prints a friendly install hint when ksef-client[cli] is missing
  • rename the internal Typer app module to avoid collisions with the public from ksef_client.cli import app export and keep CLI imports lazy
  • update CLI installation docs and add wheel-packaging smoke coverage so both ksef-client and ksef-client[cli] are verified in CI

Testing

  • python -m pytest --cov=ksef_client --cov-report=term-missing --cov-fail-under=100
  • python tools/check_cli_packaging.py

Closes #34

@smekcio smekcio merged commit 93f04f4 into main Mar 26, 2026
16 checks passed
@smekcio smekcio mentioned this pull request Mar 26, 2026
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.

No module named typer

1 participant