Skip to content

Declare click as a direct dependency#2

Merged
Anmolnoor merged 1 commit into
mainfrom
fix/declare-click-dependency
May 27, 2026
Merged

Declare click as a direct dependency#2
Anmolnoor merged 1 commit into
mainfrom
fix/declare-click-dependency

Conversation

@Anmolnoor
Copy link
Copy Markdown
Owner

Summary

src/foundation/cli.py imports click directly, but click was never declared in pyproject.toml — it came in transitively through typer. Newer typer (>=0.26) dropped click as a dependency, so a fresh pip install -e ".[dev]" now resolves without click and the test suite errors at collection:

ModuleNotFoundError: No module named 'click'

This adds click>=8.0,<9.0 to dependencies, since cli.py imports it directly.

Verification

  • Recreated the venv from scratch (Python 3.12, no manual click install) → click installs from the declared dependency alone.
  • Full suite: 380 passed.
  • ruff check clean.

🤖 Generated with Claude Code

cli.py imports click directly, but it was only available transitively
via typer. typer >=0.26 dropped click as a dependency, so a fresh
install resolved without click and failed at test collection with
ModuleNotFoundError. Declare click>=8.0,<9.0 explicitly since it is a
direct import.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Anmolnoor Anmolnoor merged commit 3698b1f into main May 27, 2026
2 checks passed
@Anmolnoor Anmolnoor deleted the fix/declare-click-dependency branch May 30, 2026 22:29
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