Alan-Jowett / CoPilot-For-Consensus / 20117775796

79%

push

github

Add ConfigProvider abstraction and reorganize SDK into self-contained modules (#45)

* Initial plan

* Add ConfigProvider abstraction layer with tests

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

* Update services to use ConfigProvider abstraction

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

* Add ConfigProvider documentation to SDK README

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

* Reorganize SDK into self-contained modules

- Split SDK into copilot_events and copilot_config modules
- Each module now has its own code, tests, README, and setup.py
- Updated all services to import from copilot_config instead of copilot_events
- Config functionality moved to copilot_config module
- Created top-level SDK README explaining the new structure
- All tests passing (36 config tests + 47 events tests + 9 ingestion tests)

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

* Editing

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Fix tests

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Editing

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Cleanup

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Fix test failure

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Remove duplicate test file and unused pytest import

- Remove test_config.py from sdk/copilot_events/tests/ (belongs only in copilot_config)
- Remove unused pytest import from sdk/copilot_config/tests/test_config.py
- All 36 copilot_config tests still passing

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

---------

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>
Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>

5 of 6 new or added lines in 1 file covered. (83.33%)

222 of 393 relevant lines covered (56.49%)

1.13 hits per line