Skip to content

feat: ✨ Introduce MappedScope#286

Merged
remimd merged 5 commits intodevfrom
mapped_scope
Nov 7, 2025
Merged

feat: ✨ Introduce MappedScope#286
remimd merged 5 commits intodevfrom
mapped_scope

Conversation

@remimd
Copy link
Copy Markdown
Member

@remimd remimd commented Nov 7, 2025

No description provided.

@remimd remimd requested a review from Copilot November 7, 2025 16:19
@remimd remimd self-assigned this Nov 7, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a new MappedScope feature to streamline scoped dependency injection, allowing users to bind dependencies to a specific scope using a descriptor-based API. The PR also updates the coverage dependency from version 7.11.0 to 7.11.1.

Key changes:

  • Adds MappedScope descriptor class and Scoped type alias for declarative scope management
  • Implements BoundMappedScope to handle context manager operations for scope definition
  • Updates coverage package and documentation to support the new feature

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
uv.lock Updates coverage from 7.11.0 to 7.11.1
injection/_core/descriptors.py Implements MappedScope and BoundMappedScope classes with type alias Scoped[T]
injection/init.pyi Adds type stubs for new MappedScope and Scoped API
injection/init.py Exports new MappedScope and Scoped identifiers
tests/core/test_descriptors.py Adds test coverage for MappedScope functionality
documentation/scoped-dependencies.md Documents the new MappedScope feature with usage examples

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread injection/_core/descriptors.py
Comment thread tests/core/test_descriptors.py
@remimd remimd force-pushed the mapped_scope branch 3 times, most recently from ec600dc to c646b3f Compare November 7, 2025 16:25
@remimd remimd merged commit 783323d into dev Nov 7, 2025
6 checks passed
@remimd remimd deleted the mapped_scope branch November 7, 2025 16:38
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