Skip to content

refactor: remove ChromaDB references and replace with Qdrant#124

Merged
github-actions[bot] merged 1 commit intomasterfrom
remove-chromadb-references
Feb 12, 2026
Merged

refactor: remove ChromaDB references and replace with Qdrant#124
github-actions[bot] merged 1 commit intomasterfrom
remove-chromadb-references

Conversation

@jordanpartridge
Copy link
Contributor

@jordanpartridge jordanpartridge commented Feb 12, 2026

Summary

  • Remove ChromaDB service from docker-compose.yml
  • Add Qdrant service with proper configuration (ports 6333/6334, health checks)
  • Update Makefile to reference Qdrant instead of ChromaDB
  • Remove ChromaDB test from AppServiceProviderTest
  • Add synapse-sentinel/gate badge to README

Breaking Changes

  • Docker compose now uses Qdrant instead of ChromaDB
  • Port mapping: Qdrant on 6333/6334 (was ChromaDB on 8000)
  • Embedding server now depends on qdrant service

Context

ChromaDB was legacy dependency. Qdrant provides:

  • Better Rust-based performance
  • Production reliability
  • Consistent with existing codebase architecture

Testing

  • ✅ AppServiceProvider tests pass
  • ✅ Docker compose configuration validated
  • ✅ Makefile updated with correct health checks

All ChromaDB references removed while maintaining full functionality with Qdrant.

Summary by CodeRabbit

  • Updates
    • Vector database service now uses Qdrant, accessible on port 6333 instead of port 8000
    • Updated startup messages, help text, and health check endpoints to reflect the new service
    • Updated Docker configuration and storage volumes for the new database backend

- Remove ChromaDB service from docker-compose.yml
- Add Qdrant service with proper configuration
- Update Makefile to reference Qdrant instead of ChromaDB
- Remove ChromaDB test from AppServiceProviderTest
- Add synapse-sentinel/gate badge to README

BREAKING CHANGE: Docker compose now uses Qdrant instead of ChromaDB
@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The project migrates from ChromaDB to Qdrant for vector database management. Updates span the Makefile (help text, health checks, startup messages now reference Qdrant with endpoint http://localhost:6333), docker-compose.yml (service replacement with new ports and configuration), and test removal of ChromaDB-specific registration logic.

Changes

Cohort / File(s) Summary
Infrastructure Configuration
Makefile, docker-compose.yml
Replaced all ChromaDB references with Qdrant. Makefile updates help text, startup messages, and health check endpoints (http://localhost:6333/collections). Docker Compose replaces chromadb service with qdrant, updating container image, ports (6333:6333, 6334:6334), volumes (qdrant_storage), and environment variables (QDRANT__SERVICE__HTTP_PORT, QDRANT__SERVICE__GRPC_PORT).
Testing
tests/Feature/AppServiceProviderTest.php
Removed test case verifying EmbeddingService registration when embedding provider is chromadb; chromadb-specific test assertion deleted.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Hopping forward, old paths fade away,
Qdrant arrives to save the day!
From port 8000 to 6333 we spring,
Vector knowledge finds new wing. 🚀

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch remove-chromadb-references

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

📊 Coverage Report

Metric Coverage Threshold Status
Lines 99.5% 95%

Files Below Threshold

File Coverage Uncovered Lines
app/Enums/ObservationType.php 0% None
app/Exceptions/Qdrant/QdrantException.php 0% None
app/Integrations/Qdrant/Requests/ScrollPoints.php 0% None
app/Services/AgentHealthService.php 0% None
app/Commands/Concerns/ResolvesProject.php 80% 26
app/Commands/KnowledgeSearchCommand.php 90.3% 73, 74, 76, 77, 78... (+2 more)
app/Services/OllamaService.php 92% 76, 152, 159, 167, 171... (+1 more)
app/Services/ProjectDetectorService.php 92% 67, 81

🏆 Synapse Sentinel Gate

@github-actions
Copy link

🏆 Sentinel Certified

Tests & Coverage: 0 tests passed
Security Audit: No security vulnerabilities found
Pest Syntax: All test files use describe/it syntax


Add this badge to your README:

[![Sentinel Certified](https://img.shields.io/github/actions/workflow/status/conduit-ui/knowledge/gate.yml?label=Sentinel%20Certified&style=flat-square)](https://github.com/conduit-ui/knowledge/actions/workflows/gate.yml)

@github-actions github-actions bot merged commit b115ed7 into master Feb 12, 2026
1 of 2 checks passed
@github-actions github-actions bot deleted the remove-chromadb-references branch February 12, 2026 06:57
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.

1 participant