Skip to content

chore: adopt modern-di 2.25 integration seams#23

Merged
lesnik512 merged 1 commit into
mainfrom
chore/adopt-2.25-seams
Jul 7, 2026
Merged

chore: adopt modern-di 2.25 integration seams#23
lesnik512 merged 1 commit into
mainfrom
chore/adopt-2.25-seams

Conversation

@lesnik512

Copy link
Copy Markdown
Member

Summary

  • Replace the isinstance-on-AbstractProvider dispatch in _resolve_di_params (modern_di_typer/main.py) with container.resolve_dependency(...), modern-di 2.25's blessed provider-or-type dispatch entry point for integrations. Marker unwrapping (_FromDI) is unchanged.
  • No providers_registry reach-in existed in this repo, so there was nothing to migrate to container.add_providers(...).
  • Bump the modern-di floor in pyproject.toml to >=2.25.0,<3 (constraint style preserved).
  • Update architecture/injection.md to describe the new single dispatch call instead of the two-branch description.

Ref: modern-python/modern-di#283

Test plan

  • just lint-ci — green
  • just test-ci — 11 passed, 100% coverage

Replace the isinstance-on-AbstractProvider dispatch in _resolve_di_params
with container.resolve_dependency(...), modern-di's blessed
provider-or-type dispatch entry point for integrations. Bump the
modern-di floor to >=2.25.0,<3 to require it.
@lesnik512 lesnik512 merged commit e1dbeef into main Jul 7, 2026
6 checks passed
@lesnik512 lesnik512 deleted the chore/adopt-2.25-seams branch July 7, 2026 19:18
@lesnik512 lesnik512 mentioned this pull request Jul 7, 2026
2 tasks
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