Alan-Jowett / CoPilot-For-Consensus / 20417129570

78%

push

github

Implement centralized schema registry for versioned schema resolution (#501)

* Initial plan

* Add schema registry module with CLI validation script

- Created schema_registry.py with centralized schema mapping
- Implemented get_schema_path, load_schema, list_schemas utilities
- Added validate_registry for checking schema integrity
- Created comprehensive test suite (25 tests, all passing)
- Added CLI script for validation and documentation generation
- Generated SCHEMAS.md documentation

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add README and example for schema registry usage

- Created comprehensive SCHEMA_REGISTRY.md documentation
- Added schema_registry_usage.py example script with 6 usage examples
- Verified all tests pass (76 passed, 1 skipped)
- All schema registry functionality working correctly

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix docstring example to use registered schema type

- Changed example from "Draft" to "Archive" (which is registered)
- Addressed code review feedback

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix remaining Draft references in docstrings to use Archive

* Address code review feedback - improve robustness

- Made schema directory discovery more robust by walking up the tree
- Fixed test to use pytest monkeypatch instead of modifying global state
- Optimized categorization from O(n²) to O(n) in CLI script
- All 76 tests still passing

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix error message to not reference undefined variable

- Fixed error message in _get_schema_base_dir() to avoid referencing undefined candidate_bases
- All tests still passing

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add clarifying comment about raw JSON vs Pydantic model

- Added comment to example explaining that it uses raw JSON validation
- Clarifie... (continued)

75 of 93 new or added lines in 2 files covered. (80.65%)

5284 of 6590 relevant lines covered (80.18%)

0.9 hits per line