Alan-Jowett / CoPilot-For-Consensus / 21101385375 / 19
77%
main: 77%

DEFAULT BRANCH: main
Ran
Files 31
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: 64.142%. Remained the same
21101385375.19

push

github

web-flow 
Add option to cascade delete associated data when source is removed (#913)

* Initial plan

* Implement cascade delete functionality for source deletion

- Add delete_source_cascade() method in IngestionService
  - Deletes archives from document_store and archive_store
  - Deletes threads, messages, chunks, and summaries
  - Logs warnings for embeddings (handled by embedding service)
  - Comprehensive audit logging for all deletions
  - Emits metrics for observability
- Update delete_source() to support cascade parameter
- Update DELETE /api/sources/{name} endpoint to accept cascade query param
- Add comprehensive tests for cascade delete functionality
- Maintain backward compatibility (cascade=false by default)

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

* Add cascade delete documentation to ingestion README

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

* Address code review feedback: refactor cascade delete

- Add helper function _extract_doc_ids() to reduce code duplication
- Improve terminology: "embeddings associated with deleted chunks"
- Use helper function consistently throughout cascade delete method
- Remove redundant str() conversions when IDs are already strings

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

* Fix indentation and message_ids initialization

- Fix incorrect indentation in logger.warning call
- Initialize message_ids before try block to preserve IDs even if deletion fails
- Ensures chunks can still be queried by message_id in partial failure scenarios

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

* Fix None handling and add comprehensive tests

- Add `or []` to all query_documents calls to handle None returns
- Document cascade delete behavior when it fails (source not deleted)
- Add test for partial failure scenario with non-blocking error handling
- Add test to verify archive_store.delete_archive... (continued)

1245 of 1941 relevant lines covered (64.14%)

0.64 hits per line

Source Files on job copilot_config - 21101385375.19