Alan-Jowett / CoPilot-For-Consensus / 20340116186 / 15
78%
main: 78%

DEFAULT BRANCH: main
Ran
Files 2
Run time 12min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 80.443%. Remained the same
20340116186.15

push

github

web-flow 
Add observability metrics to orchestrator's summary deduplication logic (#389)

* Initial plan

* Implement retrieval set comparison in orchestrator to prevent unnecessary summaries

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

* Add warning when thread document not found during retrieval set update

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

* Merge main branch and add observability metrics to summary_id-based deduplication

Resolved merge conflict by adopting main branch's cleaner summary_id-based approach (calculates hash from thread+chunks to check summaries collection) instead of storing retrieval_set in thread documents.

Enhanced main's implementation by adding:
- orchestrator_summary_skipped_total metric (reason: summary_already_exists)
- orchestrator_summary_triggered_total metric (reason: chunks_changed)
- Updated tests to match new summary_id approach

This preserves the optimization benefits while avoiding schema changes and leveraging existing data structures.

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

* Document summary deduplication logic and metrics in orchestrator README

Updated README.md to reflect:
- Orchestration flow pseudo-code now shows summary_id calculation and deduplication check
- Added documentation for new metrics (orchestrator_summary_skipped_total, orchestrator_summary_triggered_total) with their reasons

Addresses code review feedback about missing documentation.

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.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>

218 of 271 relevant lines covered (80.44%)

0.8 hits per line

Source Files on job ingestion - 20340116186.15