Skip to content

Use Click for console output and fix CLI entrypoint - #10

Merged
smurftyy merged 2 commits into
smurftyy:mainfrom
Koded0214h:whip-ux
Jun 30, 2026
Merged

Use Click for console output and fix CLI entrypoint#10
smurftyy merged 2 commits into
smurftyy:mainfrom
Koded0214h:whip-ux

Conversation

@Koded0214h

@Koded0214h Koded0214h commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Switch terminal output to click.echo for better Unicode/Windows behavior, and fix the module
entrypoint so python -m devklean calls the actual main() function.

Changes

  • Replaced print() calls in the console layer with click.echo()
  • Updated the SIGINT abort path to use Click output
  • Added click as a runtime dependency
  • Fixed src/devklean/__main__.py to import main from devklean.cli.main
  • Made package __init__.py imports lazy to avoid pulling in send2trash during unrelated
    imports
  • Updated the Windows guard test to work in subprocesses with PYTHONPATH=src

Verification

  • pytest tests/test_console.py tests/test_text_renderer.py tests/test_cli_parser.py tests/ test_windows_guard.py -q
  • Result: 28 passed

Notes

This is a UX-focused change for terminal reliability. It does not alter deletion behavior.

@Koded0214h Koded0214h changed the title Whip ux Use Click for console output and fix CLI entrypoint Jun 30, 2026
@Koded0214h

Copy link
Copy Markdown
Contributor Author

solution to #11

@smurftyy

Copy link
Copy Markdown
Owner

Good work, all checks pass. Merging this one.

@smurftyy
smurftyy merged commit 25ff24e into smurftyy:main Jun 30, 2026
17 checks passed
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