Alan-Jowett / CoPilot-For-Consensus / 21217139491

77%

Build Type

push

github

Committed by web-flow

Commit Message

Orchestrator-owned context selection for summarization (#981)

* Initial plan

* Add context selector abstraction and integrate into orchestrator

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

* Update summarization service to use pre-selected chunks from orchestrator

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

* Add unit tests for context selector implementations

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

* Fix code review issues: remove duplicate code, add constants, improve validation

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

* Remove duplicate docstring lines in summarization service

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

* Fix final code review issues: remove unused import, fix dataclass mutation, restore schema validation

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

* Address nitpick code review comments: add named constants, TODO comments, and deprecation notes

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

* Polish documentation: improve TODO comments and clarify future enhancements

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

* Address PR review comments: improve docs, add warnings, fix bugs, implement metrics

- Move unimplemented selector types to "Future Plans" section in docstrings
- Update config to mark unimplemented strategies as "planned"
- Add error handling for vector store creation
- Add schema documentation explaining optional fields and migration path
- Add warnings when selected chunks not found in document store (data inconsistency)
- Improve CANDIDATE_POOL_MULTIPLIER comment with rationale
- Fix rank assignment bug: use separate counter to ensure 0-indexed ranks
- Log TopKCohesiveSelector warning only once to avoid log noise
- Rename NEUTRAL_SIMILARITY_SCORE t... (continued)

Run Details

257 of 292 new or added lines in 7 files covered. (88.01%)

1 existing line in 1 file now uncovered.

8518 of 11015 relevant lines covered (77.33%)

0.87 hits per line