Alan-Jowett / CoPilot-For-Consensus / 20285446958

78%

push

github

Add filtering and search capabilities to reporting-ui (#315)

* Initial plan

* Add filtering and search capabilities to reporting API and UI

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

* Add comprehensive tests for filtering and search capabilities, fix route ordering

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

* Address code review feedback: add constant for metadata filter buffer size

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

* Fix optional imports for vectorstore and embedding provider

Make copilot_vectorstore and copilot_embedding imports optional using
TYPE_CHECKING to avoid import errors when these adapters aren't installed.
These dependencies are only needed for the optional topic search feature.

This fixes CI test failures where adapters aren't available during test
collection phase.

Changes:
- Use TYPE_CHECKING for conditional imports
- Update type hints to use string literals for forward references
- Tests pass without requiring vectorstore/embedding adapters

Signed-off-by: GitHub Copilot <copilot@github.com>

* Address PR review feedback for filtering and search

Performance improvements:
- Implement batch fetching to eliminate N+1 query problem (fetch all threads
  and archives in 2 queries instead of O(n*2))
- Cache archive_id and participant_count to avoid redundant calculations
- Only fetch archive once when needed for source filtering

Scalability improvements:
- Increase archive limit from 1000 to 10000 with TODO for distinct query
- Add comment explaining rationale for high limit

UI/UX fixes:
- Fix thread_id truncation to only show ellipsis when actually truncated
- Remove topic parameter from pagination URLs (topic search doesn't paginate)
- Pass event parameter explicitly to copyToClipboard for strict mode compatibility

These changes improve query efficiency from O(n*2) to O(3) database calls,
fix visual inconsistencies, a... (continued)

109 of 137 new or added lines in 1 file covered. (79.56%)

4204 of 5022 relevant lines covered (83.71%)

0.95 hits per line