wry 0.5.1.dev4__tar.gz → 0.5.2__tar.gz

{wry-0.5.1.dev4 → wry-0.5.2}/.cursorrules

@@ -29,7 +29,12 @@ Before making any commit, AI assistants MUST:
    - Fix linter errors (ruff, mypy)
    - Format code properly (ruff format)
 
-5. **Follow Conventional Commits**:
+5. **Update TODO.md** (if applicable):
+   - Mark completed tasks as done
+   - Remove completed items or move to archive section
+   - Update progress on ongoing work
+
+6. **Follow Conventional Commits**:
    - Use format: `type: description`
    - Types: feat, fix, docs, refactor, test, chore, build
 

{wry-0.5.1.dev4 → wry-0.5.2}/CHANGELOG.md

@@ -7,6 +7,42 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.5.2] - 2025-10-14
+
+### Fixed
+
+- **Optional list fields with comma-separated parsing** 🐛
+  **THE BIG ONE:** Fixed critical bug where `list[str] | None` and `Optional[list[str]]` fields didn't work with comma-separated parsing
+  - **Root cause**: Nested `Annotated` types created by `AutoWryModel` (e.g., `Annotated[Optional[Annotated[list[str], CommaSeparated]], AutoOption]`) weren't being properly unwrapped
+  - **Fix 1**: Extract metadata from inner `Annotated` types within `Optional`/`Union` wrappers
+  - **Fix 2**: Unwrap `Annotated` types after unwrapping `Optional` to get to the base list type
+  - Works with both model-wide `comma_separated_lists` ClassVar and per-field `CommaSeparated` annotation
+  - Works with `list[str] | None`, `Optional[list[T]]`, and all numeric types (int, float)
+
+### Added
+
+- **Comprehensive tests for optional comma-separated lists** ✅
+  Added 13 test cases covering all use cases and edge cases:
+  - MVP bug: `list[str] | None` with model-wide `comma_separated_lists`
+  - Per-field `Annotated[list[T], CommaSeparated] | None`
+  - Old-style `Optional[list[T]]` syntax
+  - Multiple types: str, int, float
+  - Required vs optional lists
+  - Source tracking verification
+  - Default values: `None`, `[]`, `default_factory`
+  - Space handling in comma-separated input
+  - Double-nested `Annotated` types (the actual bug pattern)
+  - **Total test count: 507 tests (all passing)** ✨
+
+## [0.5.1] - 2025-10-14
+
+### Fixed
+
+- **Comma-separated lists with `default_factory`** 🐛
+  - Fixed bug where fields with `default_factory=list` would receive `None` instead of empty list when not provided
+  - Added proper handling in `click_integration.py` to call `default_factory()` for default values
+  - All comma-separated list tests now pass without validation errors
+
 ### Added
 
 - **Development guidelines** 📚
@@ -17,6 +53,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
+- **Test quality improvements** ✨
+  - Fixed Pydantic shadow warnings by renaming `source` field to `source_path`/`source_file` in test cases
+  - Added `@pytest.mark.filterwarnings` to suppress intentional warnings in tests
+  - Tests that validate warning behavior no longer pollute test output
+  - All 494 tests pass cleanly with zero warnings
+
+- **Test quality improvements** ✨
+  - Fixed Pydantic shadow warnings by renaming `source` field to `source_path`/`source_file` in test cases
+  - Added `@pytest.mark.filterwarnings` to suppress intentional warnings in tests
+  - Tests that validate warning behavior no longer pollute test output
+  - All 494 tests pass cleanly with zero warnings
+
 - **Documentation cross-references** 🔗
   - README.md: Added Contributing section with links to CONTRIBUTING.md, .cursorrules, AI_KNOWLEDGE_BASE.md
   - AI_KNOWLEDGE_BASE.md: Updated header with related documentation links
@@ -32,6 +80,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - RELEASE_PROCESS.md: Documented that commits add to [Unreleased] during development
   - RELEASE_PROCESS.md: Explained that release commit converts [Unreleased] to [X.Y.Z]
 
+- **TODO.md integration** 📋
+  - .cursorrules: Added TODO.md update as step 5 in pre-commit requirements
+  - CONTRIBUTING.md: Added TODO.md checklist item (mark completed, update progress)
+  - RELEASE_PROCESS.md: Check TODO.md before release, update after release
+  - README.md: Added comprehensive "Documentation Index" section
+  - README.md: Listed TODO.md in "For Contributors" section
+  - README.md: Added "Quick Start Navigation" for users/contributors/AI assistants
+
 ### Tests
 
 - **Additional comma-separated test** (494 total tests)

{wry-0.5.1.dev4 → wry-0.5.2}/CONTRIBUTING.md

@@ -485,7 +485,15 @@ mypy wry/
 
 **Note**: Git commits automatically run pre-commit hooks (ruff, mypy, pytest). All must pass.
 
-### 5. wry-Specific Checks
+### 5. Update TODO.md (if applicable)
+
+- [ ] **TODO.md**: Update if working on tracked tasks
+  - Mark completed tasks as done (✅)
+  - Remove completed items or move to archive
+  - Update progress notes on ongoing work
+  - Add new tasks if discovered during development
+
+### 6. wry-Specific Checks
 
 - [ ] **Source tracking works**: If touching config/model code, verify source tracking
 - [ ] **Precedence correct**: CLI > JSON > ENV > DEFAULT (verify if changing)
@@ -648,5 +656,6 @@ For information about creating releases, see **`RELEASE_PROCESS.md`**.
 - Check `AI_KNOWLEDGE_BASE.md` for complete technical reference
 - Check `README.md` for user documentation
 - Check `RELEASE_PROCESS.md` for release workflow
+- Check `TODO.md` for current tasks and planned features
 - Check `tests/features/test_source_precedence.py` for source tracking examples
 - Open an issue for questions or clarifications

{wry-0.5.1.dev4/wry.egg-info → wry-0.5.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: wry
-Version: 0.5.1.dev4
+Version: 0.5.2
 Summary: Why Repeat Yourself? - Define your CLI once with Pydantic models
 Author-email: Tyler House <26489166+tahouse@users.noreply.github.com>
 License: MIT
@@ -1066,6 +1066,56 @@ mypy wry
 - Start a discussion for general questions
 - Check existing issues/PRs before creating new ones
 
+## Documentation Index
+
+wry has comprehensive documentation for different audiences:
+
+### For Users
+
+- 📘 **`README.md`** (this file) - Getting started, features, usage examples
+- 📁 **`examples/`** - Working code examples
+  - `examples/autowrymodel_comprehensive.py` - Complete AutoWryModel features
+  - `examples/wrymodel_comprehensive.py` - WryModel with source tracking
+  - `examples/multimodel_comprehensive.py` - Multi-model usage
+
+### For Contributors
+
+- 📖 **`CONTRIBUTING.md`** - Complete contributor guide with code patterns and checklists
+- 🤖 **`.cursorrules`** - AI assistant quick reference (references CONTRIBUTING.md)
+- 🚀 **`RELEASE_PROCESS.md`** - How to create releases and manage versions
+- 📋 **`TODO.md`** - Current tasks, planned features, and work in progress
+
+### Technical Reference
+
+- 📚 **`AI_KNOWLEDGE_BASE.md`** - Complete technical reference for AI/LLMs (also useful for humans)
+- 📝 **`CHANGELOG.md`** - Version history and all changes
+- 🧪 **`tests/README.md`** - Test organization and structure
+- 🔧 **`scripts/README.md`** - Development scripts and tools
+
+### Quick Start Navigation
+
+**I'm a user, I want to...**
+
+- Get started → README.md "Quick Start" section
+- See examples → `examples/` directory
+- Understand features → README.md "Features" section
+- Track config sources → README.md "Value Source Tracking" section
+
+**I'm a contributor, I want to...**
+
+- Set up development → CONTRIBUTING.md "Development Setup" section
+- Add a feature → CONTRIBUTING.md "Adding New Features" section
+- Run tests → CONTRIBUTING.md "Testing" section
+- Create a release → RELEASE_PROCESS.md
+- Check current tasks → TODO.md
+
+**I'm an AI assistant, I want to...**
+
+- Quick reference → `.cursorrules`
+- Technical details → `AI_KNOWLEDGE_BASE.md`
+- Code patterns → `CONTRIBUTING.md`
+- Test examples → `tests/features/test_source_precedence.py`
+
 ## License
 
 This project is licensed under the MIT License - see the [LICENSE](LICENSE) file for details.

{wry-0.5.1.dev4 → wry-0.5.2}/README.md

@@ -1022,6 +1022,56 @@ mypy wry
 - Start a discussion for general questions
 - Check existing issues/PRs before creating new ones
 
+## Documentation Index
+
+wry has comprehensive documentation for different audiences:
+
+### For Users
+
+- 📘 **`README.md`** (this file) - Getting started, features, usage examples
+- 📁 **`examples/`** - Working code examples
+  - `examples/autowrymodel_comprehensive.py` - Complete AutoWryModel features
+  - `examples/wrymodel_comprehensive.py` - WryModel with source tracking
+  - `examples/multimodel_comprehensive.py` - Multi-model usage
+
+### For Contributors
+
+- 📖 **`CONTRIBUTING.md`** - Complete contributor guide with code patterns and checklists
+- 🤖 **`.cursorrules`** - AI assistant quick reference (references CONTRIBUTING.md)
+- 🚀 **`RELEASE_PROCESS.md`** - How to create releases and manage versions
+- 📋 **`TODO.md`** - Current tasks, planned features, and work in progress
+
+### Technical Reference
+
+- 📚 **`AI_KNOWLEDGE_BASE.md`** - Complete technical reference for AI/LLMs (also useful for humans)
+- 📝 **`CHANGELOG.md`** - Version history and all changes
+- 🧪 **`tests/README.md`** - Test organization and structure
+- 🔧 **`scripts/README.md`** - Development scripts and tools
+
+### Quick Start Navigation
+
+**I'm a user, I want to...**
+
+- Get started → README.md "Quick Start" section
+- See examples → `examples/` directory
+- Understand features → README.md "Features" section
+- Track config sources → README.md "Value Source Tracking" section
+
+**I'm a contributor, I want to...**
+
+- Set up development → CONTRIBUTING.md "Development Setup" section
+- Add a feature → CONTRIBUTING.md "Adding New Features" section
+- Run tests → CONTRIBUTING.md "Testing" section
+- Create a release → RELEASE_PROCESS.md
+- Check current tasks → TODO.md
+
+**I'm an AI assistant, I want to...**
+
+- Quick reference → `.cursorrules`
+- Technical details → `AI_KNOWLEDGE_BASE.md`
+- Code patterns → `CONTRIBUTING.md`
+- Test examples → `tests/features/test_source_precedence.py`
+
 ## License
 
 This project is licensed under the MIT License - see the [LICENSE](LICENSE) file for details.

{wry-0.5.1.dev4 → wry-0.5.2}/RELEASE_PROCESS.md

@@ -26,6 +26,7 @@ Follow these steps IN ORDER:
 
 1. **Verify all changes are in [Unreleased]**
    - Check that CHANGELOG.md `[Unreleased]` section has all changes since last release
+   - Check TODO.md - all planned release items should be completed
    - Ensure all tests pass (`pytest`)
    - Fix any linting issues
 
@@ -68,7 +69,13 @@ Follow these steps IN ORDER:
 
 ### After the Release
 
-Continue development by adding changes to the (now empty) `[Unreleased]` section.
+Continue development by:
+1. Adding new changes to the (now empty) `[Unreleased]` section in CHANGELOG.md
+2. Updating TODO.md:
+   - Archive or remove completed items from the released version
+   - Add new tasks or features planned for next release
+   - Update roadmap and priorities
+3. Update version numbers in AI_KNOWLEDGE_BASE.md if not already done
 
 ## Example
 

{wry-0.5.1.dev4 → wry-0.5.2}/tests/features/test_inheritance.py

@@ -483,6 +483,8 @@ class TestInheritanceEdgeCases:
         assert len(ChildConfig.model_fields) == 2
 
 
+@pytest.mark.filterwarnings("ignore:Function.*already decorated:UserWarning")
+@pytest.mark.filterwarnings("ignore:The parameter.*is used more than once:UserWarning")
 class TestMultiModelInheritance:
     """Test inheritance scenarios with multi-model commands."""
 

{wry-0.5.1.dev4 → wry-0.5.2}/tests/features/test_multi_model.py

@@ -44,6 +44,7 @@ class CacheArgs(WryModel):
 class TestSplitKwargs:
     """Test the split_kwargs_by_model function."""
 
+    @pytest.mark.filterwarnings("ignore:Unused kwargs:UserWarning")
     def test_split_kwargs_basic(self):
         """Test basic kwarg splitting between models."""
         kwargs = {"host": "example.com", "port": 3306, "debug": True, "workers": 8, "unknown": "ignored"}
@@ -133,6 +134,8 @@ class TestCreateModels:
             create_models(ctx, kwargs, DatabaseArgs)
 
 
+@pytest.mark.filterwarnings("ignore:Function.*already decorated:UserWarning")
+@pytest.mark.filterwarnings("ignore:The parameter.*is used more than once:UserWarning")
 class TestMultiModelDecorator:
     """Test the multi_model decorator."""
 
@@ -279,6 +282,8 @@ class TestSingletonOption:
         # They should be different objects
         assert option1 is not option2
 
+    @pytest.mark.filterwarnings("ignore:Function.*already decorated:UserWarning")
+    @pytest.mark.filterwarnings("ignore:The parameter.*is used more than once:UserWarning")
     def test_singleton_option_in_model(self):
         """Test using singleton option in a model."""
         from wry.multi_model import _SINGLETON_OPTIONS

{wry-0.5.1.dev4 → wry-0.5.2}/tests/integration/test_click_integration_extended.py

@@ -457,6 +457,7 @@ class TestAddConfigOption:
 class TestExplicitClickDecorators:
     """Test handling of explicit Click decorators in annotations."""
 
+    @pytest.mark.filterwarnings("ignore:The parameter.*is used more than once:UserWarning")
     def test_explicit_option_decorator(self):
         """Test field with explicit click.option in annotation."""
 

{wry-0.5.1.dev4 → wry-0.5.2}/tests/integration/test_context_handling.py

@@ -3,6 +3,7 @@
 from typing import Annotated, Any
 
 import click
+import pytest
 from click.testing import CliRunner
 from pydantic import Field
 
@@ -78,6 +79,8 @@ class TestContextHandling:
         assert result.exit_code == 0
         assert "Name: direct" in result.output
 
+    @pytest.mark.filterwarnings("ignore:Function.*already decorated:UserWarning")
+    @pytest.mark.filterwarnings("ignore:The parameter.*is used more than once:UserWarning")
     def test_multiple_decorators_requires_care(self):
         """Test that multiple decorators need explicit pass_context control."""