Alan-Jowett / CoPilot-For-Consensus / 20285499364
78%

DEFAULT BRANCH: main
Ran
Jobs 28
Files 108
Run time 1min
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: 83.75% (+0.04%) from 83.712%
20285499364

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>

4190 of 5003 relevant lines covered (83.75%)

0.95 hits per line

Jobs
ID Job ID Ran Files Coverage
1 summarization - 20285499364.1 2
93.79
 GitHub Action Run
2 embedding - 20285499364.2 2
91.67
 GitHub Action Run
3 copilot_vectorstore - 20285499364.3 7
69.89
 GitHub Action Run
4 copilot_schema_validation_integration - 20285499364.4 6
49.55
 GitHub Action Run
5 copilot_archive_fetcher - 20285499364.5 9
68.27
 GitHub Action Run
6 copilot_logging - 20285499364.6 5
93.94
 GitHub Action Run
7 copilot_vectorstore_integration - 20285499364.7 7
23.15
 GitHub Action Run
8 copilot_archive_fetcher_integration - 20285499364.8 9
46.99
 GitHub Action Run
9 chunking - 20285499364.9 2
85.33
 GitHub Action Run
10 copilot_storage_integration - 20285499364.10 5
57.18
 GitHub Action Run
11 copilot_auth - 20285499364.11 7
98.98
 GitHub Action Run
12 parsing - 20285499364.12 6
84.25
 GitHub Action Run
13 orchestrator - 20285499364.13 2
85.91
 GitHub Action Run
14 copilot_summarization - 20285499364.14 7
94.09
 GitHub Action Run
15 copilot_events_integration - 20285499364.15 9
46.2
 GitHub Action Run
16 reporting-ui - 20285499364.16 1
0.0
 GitHub Action Run
17 copilot_draft_diff - 20285499364.17 6
98.8
 GitHub Action Run
18 copilot_chunking - 20285499364.18 2
93.67
 GitHub Action Run
19 copilot_events - 20285499364.19 9
63.77
 GitHub Action Run
20 copilot_reporting - 20285499364.20 5
73.17
 GitHub Action Run
21 copilot_schema_validation - 20285499364.21 6
90.0
 GitHub Action Run
22 copilot_embedding - 20285499364.22 3
92.16
 GitHub Action Run
23 copilot_storage - 20285499364.23 5
64.81
 GitHub Action Run
24 copilot_consensus - 20285499364.24 3
98.64
 GitHub Action Run
25 copilot_metrics - 20285499364.25 5
56.83
 GitHub Action Run
26 copilot_config - 20285499364.26 10
77.57
 GitHub Action Run
27 ingestion - 20285499364.27 2
81.34
 GitHub Action Run
28 reporting - 20285499364.28 2
88.51
 GitHub Action Run
Source Files on build 20285499364