wry 0.3.2.dev2__tar.gz → 0.4.1__tar.gz

{wry-0.3.2.dev2 → wry-0.4.1}/AI_KNOWLEDGE_BASE.md

@@ -1234,22 +1234,209 @@ class Config(AutoWryModel):
 
 See `examples/autowrymodel_comprehensive.py` and `examples/wrymodel_comprehensive.py` for complete examples.
 
+### Pattern 8: Model Inheritance (v0.3.3+)
+
+**New in v0.3.3**: Full support for inheriting from `WryModel` and `AutoWryModel` classes!
+
+#### Basic Inheritance
+
+```python
+from wry import AutoWryModel
+from pydantic import Field
+
+class BaseConfig(AutoWryModel):
+    """Common configuration shared across all environments."""
+    env_prefix = "APP_"
+    debug: bool = Field(default=False, description="Enable debug mode")
+    log_level: str = Field(default="INFO", description="Logging level")
+
+class ProductionConfig(BaseConfig):
+    """Production-specific configuration."""
+    workers: int = Field(default=4, ge=1, le=32, description="Number of workers")
+    timeout: int = Field(default=30, ge=1, description="Request timeout")
+
+# ProductionConfig has ALL fields: debug, log_level, workers, timeout
+@click.command()
+@ProductionConfig.generate_click_parameters()
+@click.pass_context
+def serve(ctx: click.Context, **kwargs: Any):
+    config = ProductionConfig.from_click_context(ctx, **kwargs)
+    print(f"Workers: {config.workers}, Debug: {config.debug}")
+    print(f"Log level: {config.log_level}")
+```
+
+**CLI Output:**
+
+```bash
+$ python app.py --help
+Options:
+  --debug / --no-debug  Enable debug mode  [default: False]
+  --log-level TEXT      Logging level  [default: INFO]
+  --workers INTEGER     Number of workers  [default: 4]
+  --timeout INTEGER     Request timeout  [default: 30]
+```
+
+#### Multiple Inheritance Levels
+
+```python
+class Level1(AutoWryModel):
+    field1: str = Field(default="v1", description="Level 1")
+
+class Level2(Level1):
+    field2: str = Field(default="v2", description="Level 2")
+
+class Level3(Level2):
+    field3: str = Field(default="v3", description="Level 3")
+
+# Level3 has: field1, field2, field3
+config = Level3()
+assert config.field1 == "v1"
+assert config.field2 == "v2"
+assert config.field3 == "v3"
+```
+
+#### Inheritance with Multi-Model
+
+```python
+from wry import multi_model, create_models
+
+class BaseServer(AutoWryModel):
+    host: str = Field(default="localhost", description="Host")
+
+class ProductionServer(BaseServer):
+    port: int = Field(default=8080, description="Port")
+    ssl_enabled: bool = Field(default=True, description="Enable SSL")
+
+class BaseDatabase(AutoWryModel):
+    db_url: str = Field(default="sqlite:///app.db", description="Database URL")
+
+class ProductionDatabase(BaseDatabase):
+    pool_size: int = Field(default=10, description="Connection pool size")
+    timeout: int = Field(default=30, description="Query timeout")
+
+@click.command()
+@multi_model(ProductionServer, ProductionDatabase)
+@click.pass_context
+def deploy(ctx: click.Context, **kwargs: Any):
+    configs = create_models(ctx, kwargs, ProductionServer, ProductionDatabase)
+    server = configs[ProductionServer]
+    db = configs[ProductionDatabase]
+
+    # All inherited fields available
+    print(f"Server: {server.host}:{server.port} (SSL: {server.ssl_enabled})")
+    print(f"Database: {db.db_url} (pool={db.pool_size}, timeout={db.timeout})")
+```
+
+#### WryModel Inheritance
+
+```python
+from wry import WryModel, AutoOption
+from typing import Annotated
+
+class BaseConfig(WryModel):
+    base_field: Annotated[str, AutoOption] = Field(default="base")
+
+class ChildConfig(BaseConfig):
+    child_field: Annotated[str, AutoOption] = Field(default="child")
+
+# Both fields available with explicit annotations
+```
+
+#### Mixing WryModel and AutoWryModel
+
+```python
+class BaseWry(WryModel):
+    # Explicit annotation required
+    explicit_field: Annotated[str, AutoOption] = Field(default="explicit")
+
+class ChildAuto(BaseWry, AutoWryModel):
+    # This automatically becomes an option (AutoWryModel processing)
+    auto_field: str = Field(default="auto")
+
+# Both fields work in CLI
+```
+
+#### Use Cases for Inheritance
+
+1. **Environment-Specific Configs:**
+
+   ```python
+   class BaseConfig(AutoWryModel):
+       app_name: str
+       debug: bool = False
+
+   class DevConfig(BaseConfig):
+       hot_reload: bool = True
+
+   class ProdConfig(BaseConfig):
+       workers: int = 4
+       ssl_cert: str
+   ```
+
+2. **Feature Flags:**
+
+   ```python
+   class CoreFeatures(AutoWryModel):
+       feature_a: bool = True
+       feature_b: bool = True
+
+   class BetaFeatures(CoreFeatures):
+       experimental_feature: bool = False
+   ```
+
+3. **Shared Authentication:**
+
+   ```python
+   class BaseAuthConfig(AutoWryModel):
+       api_key: str
+       timeout: int = 30
+
+   class ServiceAConfig(BaseAuthConfig):
+       service_a_endpoint: str
+
+   class ServiceBConfig(BaseAuthConfig):
+       service_b_endpoint: str
+   ```
+
+**Key Features:**
+
+- ✅ All inherited fields automatically become CLI options
+- ✅ Source tracking works correctly for inherited fields
+- ✅ `env_prefix` can be inherited or overridden in child classes
+- ✅ Field constraints and validation are inherited
+- ✅ Works with all wry features: arguments, options, exclusions, aliases
+- ✅ Multiple inheritance supported
+- ✅ Deep inheritance chains work correctly
+
+**Technical Details:**
+
+The inheritance bug was fixed in v0.3.3. Previously, `__init_subclass__` used `hasattr()` which checked the entire inheritance chain, causing child class fields to be skipped. Now it uses `"_autowrymodel_processed" in cls.__dict__` to check only the current class.
+
+**Tests:** See `tests/features/test_inheritance.py` for 24 comprehensive tests covering all inheritance scenarios.
+
 ---
 
 ## Test Coverage Details
 
 ### Test Statistics
 
-- **Total Tests**: 407 (all passing)
-- **Total Coverage**: 92%
+- **Total Tests**: 431+ (all passing, includes 24 inheritance tests added in v0.3.3)
+- **Total Coverage**: 92%+
 - **Core Modules**: 100% coverage (sources, accessors, env_utils, multi_model)
-- **Test Files**: 50+ test files
-- **Lines of Test Code**: ~3000 lines
+- **Test Files**: 51+ test files (including test_inheritance.py)
+- **Lines of Test Code**: ~3600+ lines
 
 ### Critical Test Files
 
 **`tests/features/test_source_precedence.py`** ⭐ MOST IMPORTANT
 
+**`tests/features/test_inheritance.py`** ⭐ NEW IN v0.3.3
+
+- 24 comprehensive inheritance tests
+- Covers AutoWryModel, WryModel, and multi-model inheritance
+- Tests multiple inheritance levels, source tracking, mixed model types
+- Validates the inheritance bug fix
+
 ```python
 class TestSourcePrecedence:
     def test_complete_precedence_chain(self):

{wry-0.3.2.dev2 → wry-0.4.1}/CHANGELOG.md

@@ -7,6 +7,49 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.4.1] - 2025-10-07
+
+### Fixed
+
+- **AutoWryModel inheritance bug** 🐛
+  - Fixed critical bug where child classes inheriting from `AutoWryModel` were not processing their own fields
+  - Previously used `hasattr()` which checked the entire inheritance chain, causing child class fields to be skipped
+  - Now uses `"_autowrymodel_processed" in cls.__dict__` to check only the current class
+  - All inherited fields now correctly become CLI options in child classes
+  - Multiple levels of inheritance now work correctly
+  - Note: `WryModel` was not affected by this bug and has always supported inheritance correctly
+
+### Added
+
+- **Comprehensive inheritance test suite** (24 new tests in `tests/features/test_inheritance.py`)
+  - Tests for `AutoWryModel` inheritance (10 tests - verify the bug fix)
+  - Tests for `WryModel` inheritance (6 tests - verify continued functionality)
+  - Tests for multi-model inheritance (5 tests - mix of both types)
+  - Edge case tests (3 tests - various scenarios)
+  - Coverage includes: basic inheritance, multiple levels, CLI generation, source tracking, constraints, env_prefix, mixed model types, and more
+
+### Documentation
+
+- **New inheritance section in README.md**
+  - Basic inheritance examples
+  - Multiple levels of inheritance
+  - Inheritance with multi-model
+  - Mixing `WryModel` and `AutoWryModel`
+  - Use cases and best practices
+
+- **New Pattern 8 in AI_KNOWLEDGE_BASE.md**
+  - Comprehensive inheritance patterns and examples
+  - Technical details about the bug fix
+  - Use cases: environment-specific configs, feature flags, shared authentication
+  - Updated test statistics (407 → 431+ tests)
+
+### Technical Details
+
+- Bug was in `AutoWryModel.__init_subclass__()` at line 46 of `wry/auto_model.py`
+- Changed from checking inherited attributes to checking class-specific attributes
+- This enables proper field processing for each class in an inheritance hierarchy
+- All existing tests continue to pass (no breaking changes)
+
 ## [0.4.0] - 2025-10-04
 
 ### Added

{wry-0.3.2.dev2 → wry-0.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: wry
-Version: 0.3.2.dev2
+Version: 0.4.1
 Summary: Why Repeat Yourself? - Define your CLI once with Pydantic models
 Author-email: Tyler House <26489166+tahouse@users.noreply.github.com>
 License: MIT
@@ -101,6 +101,7 @@ if __name__ == "__main__":
 ```
 
 **See comprehensive examples:**
+
 - `examples/autowrymodel_comprehensive.py` - All AutoWryModel features including aliases
 - `examples/wrymodel_comprehensive.py` - WryModel with source tracking
 - `examples/multimodel_comprehensive.py` - Multi-model usage
@@ -462,9 +463,119 @@ class Config(AutoWryModel):
 See `examples/autowrymodel_comprehensive.py` for examples of explicit Click decorators.
 
 **See also:**
+
 - `examples/autowrymodel_comprehensive.py` - Complete AutoWryModel example with aliases
 - `examples/wrymodel_comprehensive.py` - WryModel with aliases and source tracking
 
+### Model Inheritance
+
+**New in v0.3.3+**: Both `WryModel` and `AutoWryModel` fully support inheritance! Create base configuration classes and extend them with additional fields.
+
+#### Basic Inheritance
+
+```python
+from wry import AutoWryModel
+from pydantic import Field
+
+class BaseConfig(AutoWryModel):
+    """Common configuration shared across environments."""
+    env_prefix = "APP_"
+    debug: bool = Field(default=False, description="Enable debug mode")
+    log_level: str = Field(default="INFO", description="Logging level")
+
+class ProductionConfig(BaseConfig):
+    """Production-specific configuration."""
+    workers: int = Field(default=4, ge=1, le=32, description="Number of workers")
+    timeout: int = Field(default=30, ge=1, description="Request timeout")
+
+# ProductionConfig has all fields: debug, log_level, workers, timeout
+@click.command()
+@ProductionConfig.generate_click_parameters()
+@click.pass_context
+def serve(ctx: click.Context, **kwargs: Any):
+    config = ProductionConfig.from_click_context(ctx, **kwargs)
+    click.echo(f"Starting with {config.workers} workers, debug={config.debug}")
+```
+
+#### Multiple Levels of Inheritance
+
+```python
+class Level1Config(AutoWryModel):
+    field1: str = Field(default="v1", description="Level 1 field")
+
+class Level2Config(Level1Config):
+    field2: str = Field(default="v2", description="Level 2 field")
+
+class Level3Config(Level2Config):
+    field3: str = Field(default="v3", description="Level 3 field")
+
+# Level3Config has all fields: field1, field2, field3
+```
+
+#### Inheritance with Multi-Model
+
+```python
+from wry import multi_model, create_models
+
+class BaseServer(AutoWryModel):
+    host: str = Field(default="localhost", description="Server host")
+
+class ExtendedServer(BaseServer):
+    port: int = Field(default=8080, description="Server port")
+
+class BaseDatabase(AutoWryModel):
+    db_url: str = Field(default="sqlite:///app.db", description="Database URL")
+
+class ExtendedDatabase(BaseDatabase):
+    pool_size: int = Field(default=5, description="Connection pool size")
+
+@click.command()
+@multi_model(ExtendedServer, ExtendedDatabase)
+@click.pass_context
+def main(ctx: click.Context, **kwargs: Any):
+    configs = create_models(ctx, kwargs, ExtendedServer, ExtendedDatabase)
+    server = configs[ExtendedServer]
+    db = configs[ExtendedDatabase]
+
+    # Both base and extended fields are available
+    click.echo(f"Server: {server.host}:{server.port}")
+    click.echo(f"Database: {db.db_url} (pool={db.pool_size})")
+```
+
+#### Mixing WryModel and AutoWryModel
+
+```python
+from typing import Annotated
+from wry import WryModel, AutoWryModel, AutoOption
+
+# Start with explicit WryModel
+class BaseConfig(WryModel):
+    base_field: Annotated[str, AutoOption] = Field(default="base")
+
+# Extend with AutoWryModel for automatic options
+class ChildConfig(BaseConfig, AutoWryModel):
+    # This automatically becomes an option
+    child_field: str = Field(default="child")
+```
+
+#### Benefits of Inheritance
+
+1. **DRY Principle**: Define common fields once
+2. **Environment-Specific Configs**: Base config + dev/staging/prod extensions
+3. **Feature Flags**: Base config + feature-specific configurations
+4. **Reusable Components**: Create libraries of configuration classes
+5. **Full Source Tracking**: Inherited fields track their sources correctly
+
+**Important Notes:**
+
+- All inherited fields automatically become CLI options (for `AutoWryModel`)
+- Source tracking works correctly for inherited fields
+- `env_prefix` can be inherited or overridden
+- Field constraints and validation are inherited
+- Works with all wry features: arguments, options, exclusions, aliases
+
+See `tests/features/test_inheritance.py` for comprehensive examples.
+
 ## Development
 
 ### Prerequisites

{wry-0.3.2.dev2 → wry-0.4.1}/README.md

@@ -57,6 +57,7 @@ if __name__ == "__main__":
 ```
 
 **See comprehensive examples:**
+
 - `examples/autowrymodel_comprehensive.py` - All AutoWryModel features including aliases
 - `examples/wrymodel_comprehensive.py` - WryModel with source tracking
 - `examples/multimodel_comprehensive.py` - Multi-model usage
@@ -418,9 +419,119 @@ class Config(AutoWryModel):
 See `examples/autowrymodel_comprehensive.py` for examples of explicit Click decorators.
 
 **See also:**
+
 - `examples/autowrymodel_comprehensive.py` - Complete AutoWryModel example with aliases
 - `examples/wrymodel_comprehensive.py` - WryModel with aliases and source tracking
 
+### Model Inheritance
+
+**New in v0.3.3+**: Both `WryModel` and `AutoWryModel` fully support inheritance! Create base configuration classes and extend them with additional fields.
+
+#### Basic Inheritance
+
+```python
+from wry import AutoWryModel
+from pydantic import Field
+
+class BaseConfig(AutoWryModel):
+    """Common configuration shared across environments."""
+    env_prefix = "APP_"
+    debug: bool = Field(default=False, description="Enable debug mode")
+    log_level: str = Field(default="INFO", description="Logging level")
+
+class ProductionConfig(BaseConfig):
+    """Production-specific configuration."""
+    workers: int = Field(default=4, ge=1, le=32, description="Number of workers")
+    timeout: int = Field(default=30, ge=1, description="Request timeout")
+
+# ProductionConfig has all fields: debug, log_level, workers, timeout
+@click.command()
+@ProductionConfig.generate_click_parameters()
+@click.pass_context
+def serve(ctx: click.Context, **kwargs: Any):
+    config = ProductionConfig.from_click_context(ctx, **kwargs)
+    click.echo(f"Starting with {config.workers} workers, debug={config.debug}")
+```
+
+#### Multiple Levels of Inheritance
+
+```python
+class Level1Config(AutoWryModel):
+    field1: str = Field(default="v1", description="Level 1 field")
+
+class Level2Config(Level1Config):
+    field2: str = Field(default="v2", description="Level 2 field")
+
+class Level3Config(Level2Config):
+    field3: str = Field(default="v3", description="Level 3 field")
+
+# Level3Config has all fields: field1, field2, field3
+```
+
+#### Inheritance with Multi-Model
+
+```python
+from wry import multi_model, create_models
+
+class BaseServer(AutoWryModel):
+    host: str = Field(default="localhost", description="Server host")
+
+class ExtendedServer(BaseServer):
+    port: int = Field(default=8080, description="Server port")
+
+class BaseDatabase(AutoWryModel):
+    db_url: str = Field(default="sqlite:///app.db", description="Database URL")
+
+class ExtendedDatabase(BaseDatabase):
+    pool_size: int = Field(default=5, description="Connection pool size")
+
+@click.command()
+@multi_model(ExtendedServer, ExtendedDatabase)
+@click.pass_context
+def main(ctx: click.Context, **kwargs: Any):
+    configs = create_models(ctx, kwargs, ExtendedServer, ExtendedDatabase)
+    server = configs[ExtendedServer]
+    db = configs[ExtendedDatabase]
+
+    # Both base and extended fields are available
+    click.echo(f"Server: {server.host}:{server.port}")
+    click.echo(f"Database: {db.db_url} (pool={db.pool_size})")
+```
+
+#### Mixing WryModel and AutoWryModel
+
+```python
+from typing import Annotated
+from wry import WryModel, AutoWryModel, AutoOption
+
+# Start with explicit WryModel
+class BaseConfig(WryModel):
+    base_field: Annotated[str, AutoOption] = Field(default="base")
+
+# Extend with AutoWryModel for automatic options
+class ChildConfig(BaseConfig, AutoWryModel):
+    # This automatically becomes an option
+    child_field: str = Field(default="child")
+```
+
+#### Benefits of Inheritance
+
+1. **DRY Principle**: Define common fields once
+2. **Environment-Specific Configs**: Base config + dev/staging/prod extensions
+3. **Feature Flags**: Base config + feature-specific configurations
+4. **Reusable Components**: Create libraries of configuration classes
+5. **Full Source Tracking**: Inherited fields track their sources correctly
+
+**Important Notes:**
+
+- All inherited fields automatically become CLI options (for `AutoWryModel`)
+- Source tracking works correctly for inherited fields
+- `env_prefix` can be inherited or overridden
+- Field constraints and validation are inherited
+- Works with all wry features: arguments, options, exclusions, aliases
+
+See `tests/features/test_inheritance.py` for comprehensive examples.
+
 ## Development
 
 ### Prerequisites