Skip to content

Comments

refactor: extract QdrantOps shared helper#304

Merged
bug-ops merged 1 commit intomainfrom
refactor/qdrant-ops-helper
Feb 15, 2026
Merged

refactor: extract QdrantOps shared helper#304
bug-ops merged 1 commit intomainfrom
refactor/qdrant-ops-helper

Conversation

@bug-ops
Copy link
Owner

@bug-ops bug-ops commented Feb 15, 2026

Summary

  • Add QdrantOps helper in zeph-memory consolidating common Qdrant operations (ensure_collection, upsert, search, delete, scroll)
  • Refactor 4 stores (QdrantStore, CodeStore, QdrantSkillMatcher, McpToolRegistry) to delegate low-level Qdrant calls via QdrantOps
  • Net reduction: ~180 lines of duplicated boilerplate removed
  • 5 new unit tests for QdrantOps

Closes #298

@github-actions github-actions bot added refactor skills SKILL.md system memory Persistence and memory rust dependencies size/XL and removed refactor labels Feb 15, 2026
@bug-ops bug-ops force-pushed the refactor/qdrant-ops-helper branch from f601415 to 50eadc2 Compare February 15, 2026 11:54
@codecov-commenter
Copy link

codecov-commenter commented Feb 15, 2026

Codecov Report

❌ Patch coverage is 27.23404% with 171 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
crates/zeph-memory/src/qdrant_ops.rs 40.76% 77 Missing ⚠️
crates/zeph-index/src/store.rs 5.55% 34 Missing ⚠️
crates/zeph-memory/src/qdrant.rs 5.71% 33 Missing ⚠️
crates/zeph-skills/src/qdrant_matcher.rs 11.76% 15 Missing ⚠️
crates/zeph-mcp/src/registry.rs 29.41% 12 Missing ⚠️

❌ Your patch status has failed because the patch coverage (27.23%) is below the target coverage (60.00%). You can increase the patch coverage or adjust the target coverage.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #304      +/-   ##
==========================================
+ Coverage   80.01%   80.29%   +0.27%     
==========================================
  Files          98       99       +1     
  Lines       23926    23900      -26     
==========================================
+ Hits        19145    19191      +46     
+ Misses       4781     4709      -72     
Files with missing lines Coverage Δ
crates/zeph-mcp/src/registry.rs 75.06% <29.41%> (+8.18%) ⬆️
crates/zeph-skills/src/qdrant_matcher.rs 51.02% <11.76%> (+9.83%) ⬆️
crates/zeph-memory/src/qdrant.rs 34.86% <5.71%> (+2.10%) ⬆️
crates/zeph-index/src/store.rs 44.80% <5.55%> (+4.22%) ⬆️
crates/zeph-memory/src/qdrant_ops.rs 40.76% <40.76%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…late

Consolidate common Qdrant operations (ensure_collection, upsert, search,
delete, scroll) into QdrantOps in zeph-memory. Refactor QdrantStore,
CodeStore, QdrantSkillMatcher, and McpToolRegistry to delegate via QdrantOps.

Closes #298
@bug-ops bug-ops force-pushed the refactor/qdrant-ops-helper branch from 50eadc2 to a9cf6bb Compare February 15, 2026 12:18
@bug-ops bug-ops enabled auto-merge (squash) February 15, 2026 12:18
@bug-ops bug-ops merged commit 546ac9a into main Feb 15, 2026
18 checks passed
@bug-ops bug-ops deleted the refactor/qdrant-ops-helper branch February 15, 2026 12:29
@bug-ops bug-ops mentioned this pull request Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: extract QdrantOps shared helper

2 participants