Skip to content

fix: avoid local task module collisions - #123

Merged
D3f0 merged 3 commits into
mainfrom
fix/multiple-local-tasks-discovery
Aug 12, 2026
Merged

fix: avoid local task module collisions#123
D3f0 merged 3 commits into
mainfrom
fix/multiple-local-tasks-discovery

Conversation

@D3f0

@D3f0 D3f0 commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • load each local_tasks.py under a path-derived private module name
  • preserve independent local task collections when multiple directories are loaded
  • add regression coverage for distinct local task modules

Verification

  • uv run pytest tests/test_collection.py tests/test_program_with_collection.py -q
  • uv run ruff check src/invoke_toolkit/collections.py tests/test_collection.py
  • uv run pytest -q (670 passed, 1 existing failure in tests/extensions/test_package_template_entrypoint.py caused by Rich line wrapping)
  • commit hooks passed except pre-existing pylint warning W0237 in ToolkitCollection.configure

Fixes #122

@D3f0
D3f0 merged commit b425e43 into main Aug 12, 2026
4 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.

Multiple local_tasks.py files can crash task discovery in a Git repository

1 participant