Alan-Jowett / CoPilot-For-Consensus / 21148278415

77%

Build Type

push

github

Committed by web-flow

Commit Message

Implement complete event-driven cascade cleanup for source deletion across all services (#930)

* Initial plan

* Add cascade cleanup event schemas and update ingestion service to publish events

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

* Fix ruff linting issues in updated files

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

* Add test for cascade cleanup event publishing in ingestion service

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

* Add cascade cleanup documentation

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

* Address code review feedback: fix correlation_id handling and cleanup formatting

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

* Add SPDX header to cascade cleanup docs

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Implement cascade cleanup handlers in all services (parsing, chunking, embedding, reporting)

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

* Update cascade cleanup documentation to reflect completed implementation

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

* Fix embedding service VectorStore.delete() call to match interface signature

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

* Address code review comments: fix documentation ambiguity, idempotency tracking, and embedding race condition

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

* Add test coverage for cascade cleanup handlers in all services

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

* Fix embedding cascade cleanup typing

Ensure chunk IDs are validated as non-empty strings before calling vector_store.delete(), satisfying the pyright typed config safety gate.

* chunking: update tests for new subscriptions

Signed-off-by: Alan Jowett... (continued)

Run Details

172 of 274 new or added lines in 6 files covered. (62.77%)

1 existing line in 1 file now uncovered.

8022 of 10390 relevant lines covered (77.21%)

0.87 hits per line