Skip to content

Cache container dependencies and improve health checks#30

Merged
GauteR merged 1 commit into
mainfrom
feature/container-optimization
Jun 15, 2026
Merged

Cache container dependencies and improve health checks#30
GauteR merged 1 commit into
mainfrom
feature/container-optimization

Conversation

@GauteR

@GauteR GauteR commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • Cacher lexical store, noise filter og use cases via cached_property
  • Advarsel i health når FAISS er valgt men ikke lastet
  • Cacher Chroma reachability-sjekk med 30s TTL

Test plan

  • uv run pytest tests/test_container.py tests/test_benchmark.py -v
  • Verifiser at use cases gjenbrukes på tvers av requests
  • Health viser FAISS-advarsel uten faiss-extra

Made with Cursor

Reuse lexical store, noise filter, and use cases via cached_property,
warn when FAISS is selected but unavailable, and cache Chroma health.

Co-authored-by: Cursor <cursoragent@cursor.com>
@GauteR GauteR force-pushed the feature/container-optimization branch from ecbdfd6 to 3267155 Compare June 15, 2026 12:39
@GauteR GauteR merged commit e0edabc into main Jun 15, 2026
1 check failed
@GauteR GauteR deleted the feature/container-optimization branch June 15, 2026 12:40
GauteR added a commit that referenced this pull request Jun 15, 2026
## Summary
- Ny `section_store_registry` med `json` og `memory` providere
- `SECTION_STORE_PROVIDER` env-variabel
- `ENABLE_PDF_INDEXING` skilt fra `ENABLE_LLAMAPARSE` — pypdf-fallback
fungerer uten LlamaParse

**Avhenger av:** #30 — merge container-optimization først.

## Test plan
- [ ] `uv run pytest tests/test_vector_store_registry.py
tests/test_api.py -k pdf -v`
- [ ] PDF-indeksering med `ENABLE_LLAMAPARSE=false` og
`ENABLE_PDF_INDEXING=true` → 200
- [ ] `SECTION_STORE_PROVIDER=memory` i testmiljø

Co-authored-by: Cursor <cursoragent@cursor.com>
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