docs: KG construction research findings#475
Conversation
- Validate all 11 code snippets from the use-case doc against main (9 pass, 2 fail) - Run full 8-entity-type / 26-relationship pipeline test (31/31 checks pass) - Document 6 friction points (FP-1 through FP-6) with runnable problem code and workarounds - Competitive analysis against Neo4j, KùzuDB, LangChain integration - 5 prioritised recommendations (R-1 through R-5) with concrete method signatures - Investigate ontology-constrained KG construction in depth: three approaches (Pydantic-only, GF ontology graph, hybrid) all validated against current main - Issues filed: #468 (shortestPath SyntaxError), #469 (schema introspection), #470 (to_json/from_json), #471 (merge_node), #472 (add_graph_documents), #473 (CREATE vs MERGE docs), #474 (EXISTS {} planner bug) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #475 +/- ##
=======================================
Coverage 88.01% 88.01%
=======================================
Files 40 40
Lines 14449 14449
Branches 3430 3430
=======================================
Hits 12718 12718
Misses 1141 1141
Partials 590 590
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
|
Closes #449
Summary
docs/use-cases/knowledge-graph-construction.mdagainst currentmain(9 pass, 2 fail)Issues filed
shortestPath()causes SyntaxError — parser does not recognise the functionlabels(),relationship_types(),node_count(),relationship_count()to_json()/from_json()merge_node()— safe label injectionadd_graph_documents()— LangChain-compatible batch ingestionEXISTS {}subquery fails in planner with outer variable bindingTest plan
🤖 Generated with Claude Code
Need help on this PR? Tag
@codesmithwith what you need.