pico-ioc 2.2.0__tar.gz → 2.2.1__tar.gz

{pico_ioc-2.2.0 → pico_ioc-2.2.1}/CHANGELOG.md

@@ -7,6 +7,20 @@ and this project adheres to Semantic Versioning (https://semver.org/spec/v2.0.ht
 
 ---
 
+## [2.2.1] - 2026-02-03
+
+### Fixed 🧩
+
+- **DOT Graph Export**: Fixed incorrect variable reference in `export_graph()` where dependency edges used `{child}` instead of `{cid}`, causing malformed DOT output when visualizing the dependency graph.
+- **Race Condition in Lazy Proxy**: Fixed a race condition in `UnifiedComponentProxy._async_init_if_needed()` where concurrent async access could trigger duplicate object creation. The fix implements proper double-check locking and moves `@configure` execution outside the critical section to prevent deadlocks.
+- **Silent Cleanup Failures**: Replaced silent `except Exception: pass` blocks in `ScopedCaches` cleanup methods with proper logging. Cleanup method failures are now logged at WARNING level, aiding debugging of resource leaks without crashing the application.
+
+### Internal 🔧
+
+- Added `logging` import and `_logger` instance to `scope.py` for structured error reporting during component cleanup.
+
+---
+
 ## [2.2.0] - 2025-11-29
 
 ### Added ✨

{pico_ioc-2.2.0 → pico_ioc-2.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pico-ioc
-Version: 2.2.0
+Version: 2.2.1
 Summary: A minimalist, zero-dependency Inversion of Control (IoC) container for Python.
 Author-email: David Perez Cabrera <dperezcabrera@gmail.com>
 License: MIT License
@@ -347,7 +347,7 @@ tox
 
 ## 🧾 Changelog
 
-See [CHANGELOG.md](https://www.google.com/search?q=./CHANGELOG.md) — Significant redesigns and features in v2.0+.
+See [CHANGELOG.md](./CHANGELOG.md) — Significant redesigns and features in v2.0+.
 
 -----
 

{pico_ioc-2.2.0 → pico_ioc-2.2.1}/README.md

@@ -297,7 +297,7 @@ tox
 
 ## 🧾 Changelog
 
-See [CHANGELOG.md](https://www.google.com/search?q=./CHANGELOG.md) — Significant redesigns and features in v2.0+.
+See [CHANGELOG.md](./CHANGELOG.md) — Significant redesigns and features in v2.0+.
 
 -----
 

pico_ioc-2.2.1/docs/README.md

@@ -0,0 +1,142 @@
+# Pico-IoC Documentation
+
+`pico-ioc` is a lightweight, async-native Inversion of Control (IoC) container for Python 3.10+. It brings enterprise-grade dependency injection, configuration binding, and AOP to the Python ecosystem.
+
+---
+
+## Quick Install
+
+```bash
+pip install pico-ioc
+
+# Optional: YAML configuration support
+pip install pico-ioc[yaml]
+```
+
+---
+
+## 30-Second Example
+
+```python
+from pico_ioc import component, init
+
+@component
+class Database:
+    def query(self) -> str:
+        return "data from DB"
+
+@component
+class UserService:
+    def __init__(self, db: Database):  # Auto-injected
+        self.db = db
+
+    def get_users(self) -> str:
+        return self.db.query()
+
+# Initialize and use
+container = init(modules=[__name__])
+service = container.get(UserService)
+print(service.get_users())  # "data from DB"
+```
+
+---
+
+## Key Features
+
+| Feature | Description |
+|---------|-------------|
+| **Async-Native** | Full `async`/`await` support: `aget()`, `__ainit__`, async `@cleanup` |
+| **Unified Configuration** | `@configured` maps ENV vars or YAML/JSON to dataclasses |
+| **Fail-Fast Validation** | All wiring errors detected at `init()`, not runtime |
+| **AOP Interceptors** | `@intercepted_by` for logging, caching, security |
+| **Scoped Lifecycles** | singleton, prototype, request, session, transaction |
+| **Observable** | Built-in stats, health checks, dependency graph export |
+
+---
+
+## Documentation Structure
+
+| # | Section | Description | Link |
+|---|---------|-------------|------|
+| 1 | **Getting Started** | 5-minute tutorial | [getting-started.md](./getting-started.md) |
+| 2 | **User Guide** | Core concepts, configuration, scopes, testing | [user-guide/](./user-guide/README.md) |
+| 3 | **Advanced Features** | Async, AOP, Event Bus, conditional binding | [advanced-features/](./advanced-features/README.md) |
+| 4 | **Observability** | Metrics, tracing, graph export | [observability/](./observability/README.md) |
+| 5 | **Cookbook** | Real-world patterns (multi-tenant, CQRS, etc.) | [cookbook/](./cookbook/README.md) |
+| 6 | **Architecture** | Design principles, internals | [architecture/](./architecture/README.md) |
+| 7 | **API Reference** | Decorators, exceptions, protocols | [api-reference/](./api-reference/README.md) |
+| 8 | **FAQ** | Common questions and solutions | [faq.md](./faq.md) |
+| 9 | **Examples** | Complete runnable applications | [examples/](./examples/README.md) |
+
+---
+
+## Core APIs at a Glance
+
+### Registration
+
+```python
+from pico_ioc import component, factory, provides
+
+@component                    # Register a class
+class MyService: ...
+
+@factory                      # Group related providers
+class ClientFactory:
+    @provides(RedisClient)    # Provide third-party types
+    def build_redis(self, config: RedisConfig) -> RedisClient:
+        return RedisClient(config.url)
+```
+
+### Configuration
+
+```python
+from dataclasses import dataclass
+from pico_ioc import configured, configuration, init
+
+@configured(prefix="DB_")
+@dataclass
+class DBConfig:
+    host: str = "localhost"
+    port: int = 5432
+
+container = init(
+    modules=[__name__],
+    config=configuration()  # Reads from ENV: DB_HOST, DB_PORT
+)
+```
+
+### Async Support
+
+```python
+@component
+class AsyncService:
+    async def __ainit__(self):
+        self.conn = await open_connection()
+
+    @cleanup
+    async def close(self):
+        await self.conn.close()
+
+# Resolve async components
+service = await container.aget(AsyncService)
+
+# Cleanup
+await container.ashutdown()
+```
+
+### Testing with Overrides
+
+```python
+container = init(
+    modules=[__name__],
+    overrides={Database: FakeDatabase()}  # Replace for tests
+)
+```
+
+---
+
+## Next Steps
+
+1. **New to pico-ioc?** Start with the [Getting Started](./getting-started.md) tutorial
+2. **Coming from Spring/Guice?** Check the [Architecture Comparison](./architecture/comparison.md)
+3. **Building a real app?** See the [Cookbook](./cookbook/README.md) patterns

pico_ioc-2.2.1/docs/overview.md

@@ -0,0 +1,5 @@
+# Overview
+
+This page has been merged into the main documentation.
+
+**Please see: [Documentation Home](./README.md)**

{pico_ioc-2.2.0 → pico_ioc-2.2.1}/mkdocs.yml

@@ -146,10 +146,6 @@ nav:
     - Custom Scanners: advanced-features/custom-scanners.md
     - Self Injection: advanced-features/self-injection.md
 
-  - Integrations:
-    - Overview: integrations.md
-    - Celery: integrations-celery.md
-
   - Observability:
     - observability/README.md
     - Container Context: observability/container-context.md

pico_ioc-2.2.1/src/pico_ioc/_version.py

@@ -0,0 +1 @@
+__version__ = '2.2.1'

{pico_ioc-2.2.0 → pico_ioc-2.2.1}/src/pico_ioc/aop.py

@@ -145,23 +145,25 @@ class UnifiedComponentProxy:
             return
 
         lock = object.__getattribute__(self, "_lock")
-        tgt = object.__getattribute__(self, "_target")
-        if tgt is not None:
-            return
 
-        creator = object.__getattribute__(self, "_creator")
-        container = object.__getattribute__(self, "_container")
-        
-        tgt = creator() 
-        
+        with lock:
+            # Double-check inside lock to prevent race condition
+            tgt = object.__getattribute__(self, "_target")
+            if tgt is not None:
+                return
+
+            creator = object.__getattribute__(self, "_creator")
+            container = object.__getattribute__(self, "_container")
+
+            tgt = creator()
+            object.__setattr__(self, "_target", tgt)
+
+        # Run configure methods outside the lock to avoid blocking
         if container and hasattr(container, "_run_configure_methods"):
             res = container._run_configure_methods(tgt)
             if inspect.isawaitable(res):
                 await res
         
-        with lock:
-            object.__setattr__(self, "_target", tgt)
-        
     def _scope_signature(self) -> Tuple[Any, ...]:
         container = object.__getattribute__(self, "_container")
         key = object.__getattribute__(self, "_component_key")

{pico_ioc-2.2.0 → pico_ioc-2.2.1}/src/pico_ioc/container.py

@@ -438,7 +438,7 @@ class PicoContainer:
             pid = _node_id(parent)
             for child in deps:
                 cid = _node_id(child)
-                lines.append(f"  {pid} -> {child};")
+                lines.append(f"  {pid} -> {cid};")
 
         lines.append("}")
 

{pico_ioc-2.2.0 → pico_ioc-2.2.1}/src/pico_ioc/scope.py

@@ -1,9 +1,12 @@
 import contextvars
 import inspect
+import logging
 from typing import Any, Dict, Optional, Tuple
 from collections import OrderedDict
 from .exceptions import ScopeError
 
+_logger = logging.getLogger(__name__)
+
 class ScopeProtocol:
     def get_id(self) -> Any | None: ...
 
@@ -108,10 +111,15 @@ class ScopedCaches:
                 if meta.get("cleanup", False):
                     try:
                         m()
-                    except Exception:
-                        pass
-        except Exception:
-            pass
+                    except Exception as e:
+                        _logger.warning(
+                            "Cleanup method %s.%s failed: %s",
+                            type(obj).__name__,
+                            getattr(m, "__name__", "<unknown>"),
+                            e
+                        )
+        except Exception as e:
+            _logger.debug("Failed to inspect object for cleanup: %s", e)
             
     def cleanup_scope(self, scope_name: str, scope_id: Any) -> None:
         bucket = self._by_scope.get(scope_name)
@@ -123,8 +131,8 @@ class ScopedCaches:
         try:
             for _, obj in container.items():
                 self._cleanup_object(obj)
-        except Exception:
-            pass
+        except Exception as e:
+            _logger.debug("Failed to cleanup container: %s", e)
 
     def for_scope(self, scopes: ScopeManager, scope: str) -> ComponentContainer:
         if scope == "singleton":

{pico_ioc-2.2.0 → pico_ioc-2.2.1}/src/pico_ioc.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pico-ioc
-Version: 2.2.0
+Version: 2.2.1
 Summary: A minimalist, zero-dependency Inversion of Control (IoC) container for Python.
 Author-email: David Perez Cabrera <dperezcabrera@gmail.com>
 License: MIT License
@@ -347,7 +347,7 @@ tox
 
 ## 🧾 Changelog
 
-See [CHANGELOG.md](https://www.google.com/search?q=./CHANGELOG.md) — Significant redesigns and features in v2.0+.
+See [CHANGELOG.md](./CHANGELOG.md) — Significant redesigns and features in v2.0+.
 
 -----
 

{pico_ioc-2.2.0 → pico_ioc-2.2.1}/src/pico_ioc.egg-info/SOURCES.txt

@@ -102,14 +102,27 @@ src/pico_ioc.egg-info/dependency_links.txt
 src/pico_ioc.egg-info/requires.txt
 src/pico_ioc.egg-info/top_level.txt
 tests/test_aop.py
+tests/test_aop_coverage.py
+tests/test_aop_extended.py
+tests/test_bugfixes_v221.py
 tests/test_collection_injection.py
+tests/test_config_builder.py
+tests/test_config_registrar_coverage.py
+tests/test_config_runtime_coverage.py
 tests/test_config_value.py
 tests/test_configured.py
 tests/test_container_context.py
+tests/test_container_coverage.py
+tests/test_container_coverage_boost.py
+tests/test_container_extended.py
+tests/test_container_integrated_coverage.py
 tests/test_container_runtime.py
 tests/test_container_self_injection.py
 tests/test_custom_scanner.py
 tests/test_event_bus.py
+tests/test_event_bus_coverage_boost.py
+tests/test_event_bus_extended.py
+tests/test_locator.py
 tests/test_pico_extends.py
 tests/test_pico_integration.py
 tests/test_protocol_resolution_and_graph.py
@@ -117,4 +130,5 @@ tests/test_provides_module_functions.py
 tests/test_provides_static_methods.py
 tests/test_proxy_unit.py
 tests/test_resolution_graph.py
-tests/test_scope.py
+tests/test_scope.py
+tests/test_scope_coverage.py

pico_ioc-2.2.1/tests/test_aop_coverage.py

@@ -0,0 +1,72 @@
+import pytest
+import pickle
+import inspect
+import threading
+from pico_ioc.aop import (
+    UnifiedComponentProxy, 
+    intercepted_by, 
+    MethodInterceptor, 
+    MethodCtx,
+    _gather_interceptors_for_method
+)
+from pico_ioc.exceptions import SerializationError, AsyncResolutionError
+
+def test_intercepted_by_validation():
+    with pytest.raises(TypeError, match="requires at least one"):
+        intercepted_by()
+        
+    with pytest.raises(TypeError, match="expects interceptor classes"):
+        intercepted_by("not_a_class")
+        
+    decorator = intercepted_by(MethodInterceptor)
+    with pytest.raises(TypeError, match="only decorate callables"):
+        decorator("not_callable")
+
+def test_proxy_init_validation():
+    with pytest.raises(ValueError, match="non-null container"):
+        UnifiedComponentProxy(container=None)
+        
+    with pytest.raises(ValueError, match="requires either a target"):
+        UnifiedComponentProxy(container="stub")
+
+def test_proxy_creator_errors():
+    proxy = UnifiedComponentProxy(container="stub", object_creator="not_callable")
+    with pytest.raises(TypeError, match="must be callable"):
+        getattr(proxy, "any_attr")
+
+    proxy = UnifiedComponentProxy(container="stub", object_creator=lambda: None)
+    with pytest.raises(RuntimeError, match="returned None"):
+        getattr(proxy, "any_attr")
+
+def test_proxy_async_error_on_sync_access():
+    async def async_configure(obj):
+        pass
+        
+    class ContainerMock:
+        def _run_configure_methods(self, obj):
+            return async_configure(obj)
+            
+    proxy = UnifiedComponentProxy(
+        container=ContainerMock(), 
+        object_creator=lambda: "target"
+    )
+    
+    with pytest.raises(AsyncResolutionError):
+        str(proxy)
+
+def test_proxy_serialization_error():
+    class Unpicklable:
+        def __getstate__(self):
+            raise ValueError("No pickle")
+            
+    proxy = UnifiedComponentProxy(container="stub", target=Unpicklable())
+    
+    with pytest.raises(SerializationError):
+        pickle.dumps(proxy)
+        
+    with pytest.raises(SerializationError):
+        proxy.__setstate__({"data": b"bad_data"})
+
+def test_gather_interceptors_edge_cases():
+    assert _gather_interceptors_for_method(str, "non_existent") == ()
+    assert _gather_interceptors_for_method(list, "append") == ()