feat: delete knowledge-graph layer (phase 3 — pyoxigraph/spaCy removed) - #89
Merged
Conversation
…kerfile/tests Fork left the app non-importable and the image unbuildable; this completes it: - admin/stats.py rewritten (counts+content-items only; SPARQL endpoints removed) - api/content.py ingest path no longer references deleted stores.entities/extraction - run_ingestion signature trimmed to (job_id, content_id, chunk_records, stores, embedding_client) - clients/rag.py prompt builder is chunk-only (no triples/contradictions) - health/admin entity routes stubbed; eval/runner chunk-only - Dockerfile: drop spaCy model download (spaCy removed from deps) - tests: delete graph-only suites, fix surviving ingest/ask/health/stats/config tests Gates: 305 passed, ruff clean, app imports, docker image builds.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Phase 3 of the no-graph redesign: delete the now-dead graph layer entirely.
Removed: pyoxigraph triple store, spaCy NLP, ontology bootstrap/registry/namespaces, reasoning/inference, triple-extraction (ExtractionClient), coreference, the triple-processing ingest pipeline, outbox drainer, maintenance normalizer, and the graph API endpoints (knowledge/claims/contradictions/changes).
pyoxigraph,spacy,python-igraphdropped from deps; spaCy model download dropped from the Dockerfile.Kept (chunk-only RAG): content/chunk store (pgvector+hybrid), chunking, parsing, EmbeddingClient, search, chunk-only ask synthesis, eval harness,
ontology/uri.py(pure URI helper).Reviewer note: the spawning agent skipped its own gates and left the app non-importable (broken admin/stats.py), the ingest path crashing on
stores.entities, the RAG prompt builder referencingknowledge_triples, and the Dockerfile still downloading spaCy. All fixed here. Verified locally: 305 tests pass,ruff check+formatclean, app imports, docker image builds (~11s, much slimmer).🤖 Generated with Claude Code