Alan-Jowett / CoPilot-For-Consensus / 20285499364 / 5
78%
main: 78%

DEFAULT BRANCH: main
Ran
Files 9
Run time 0s
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: 68.273%. Remained the same
20285499364.5

push

github

web-flow 
Remove idempotency check from summarization service (#320)

* Remove idempotency check from summarization service

The orchestrator service is responsible for deciding when summaries need
to be generated or regenerated. The summarization service should execute
all requests without checking for existing summaries, allowing the
orchestrator to control regeneration policy.

This enables:
- Regenerating summaries when new chunks/messages arrive for a thread
- Implementing flexible regeneration policies (time-based, content-based)
- Keeping reports current with evolving discussions

Changes:
- Removed idempotency check from _process_thread()
- Updated docstring to clarify orchestrator controls policy
- Updated test to verify service honors regeneration requests

Fixes #319

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

* Address PR 320 review feedback: Remove orchestrator idempotency check and update docs

Fixes both review comments on PR 320:

1. Remove idempotency check from orchestrator service
   - Removed existing summary check in orchestrator/app/service.py
   - Allows orchestrator to control summary regeneration policy
   - Enables summaries to be regenerated when new content arrives
   - Updated test_idempotent_orchestration to verify regeneration is allowed

2. Update FORWARD_PROGRESS.md documentation
   - Removed outdated reference to summarization/app/service.py idempotency
   - Removed orchestrator/app/service.py read-before-write pattern reference
   - Documentation now reflects current architecture

All tests passing:
- summarization: 32/32 tests pass
- orchestrator: 18/18 tests pass

This completes the changes needed to fully enable summary regeneration
as described in PR 320.

---------

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

170 of 249 relevant lines covered (68.27%)

0.68 hits per line

Source Files on job copilot_archive_fetcher - 20285499364.5