Skip to content

admin: Add graph collection lifecycle support and improve admin logging#192

Merged
m1rl0k merged 3 commits intoContext-Engine-AI:testfrom
voarsh2:admin-align
Jan 21, 2026
Merged

admin: Add graph collection lifecycle support and improve admin logging#192
m1rl0k merged 3 commits intoContext-Engine-AI:testfrom
voarsh2:admin-align

Conversation

@voarsh2
Copy link
Contributor

@voarsh2 voarsh2 commented Jan 20, 2026

admin: Add graph collection lifecycle support and improve admin logging

  • Add INDEX_GRAPH_EDGES to CONFIG_DRIFT_RULES and documentation
  • Extend delete_collection_everywhere to delete both main and graph collections
  • Add _delete_collection_and_graph helper for staging cleanup
  • Replace print statements with proper logging throughout admin scripts
  • Add flash message support to admin UI templates
  • Extend test coverage for collection deletion with graph collections
  • Add staging lifecycle and workspace state tests

…ng - Add INDEX_GRAPH_EDGES to CONFIG_DRIFT_RULES and documentation - Extend delete_collection_everywhere to delete both main and graph collections

- Add _delete_collection_and_graph helper for staging cleanup
 - Replace print statements with proper logging throughout admin scripts
- Add flash message support to admin UI templates
- Extend test coverage for collection deletion with graph collections                                                     - Add staging lifecycle and workspace state tests
@augmentcode
Copy link

augmentcode bot commented Jan 20, 2026

🤖 Augment PR Summary

Summary: This PR extends the admin/staging lifecycle to treat “graph edge” Qdrant collections as first-class companions of a main collection, and improves operational visibility in the admin UI.

Changes:

  • Documented INDEX_GRAPH_EDGES as a drift rule that requires collection recreation
  • Included index_graph_edges in the indexing config snapshot and drift classification
  • Extended admin deletion to delete both the main collection and its associated _graph collection (best-effort for graph)
  • Added staging support to clone/clean up graph collections alongside *_old clones
  • Replaced scattered print statements with structured logging across staging/admin paths
  • Added redirect-driven “flash” messages in the admin UI after deletion (including graph deletion status)
  • Added/updated unit tests for graph deletion, staging lifecycle behavior, and config snapshot drift coverage

Technical Notes: Graph collection operations are designed to be best-effort to avoid breaking main lifecycle flows when the graph collection is absent.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 2 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

@voarsh2 voarsh2 marked this pull request as draft January 20, 2026 16:57
Ensures graph collection deletion is more robust by handling potential errors and improves the fallback mechanism for determining the slug order when processing delta bundles.

It also introduces a new mechanism to delegate Qdrant collection deletion to an existing helper function `delete_collection_qdrant` if available, enhancing testability by allowing monkeypatching of this function.
@voarsh2 voarsh2 marked this pull request as ready for review January 20, 2026 17:36
Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@m1rl0k m1rl0k merged commit 469093c into Context-Engine-AI:test Jan 21, 2026
1 check passed
@voarsh2 voarsh2 deleted the admin-align branch January 21, 2026 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants