[maxdswain]

Related Issues

• fixes Add natural-language-to-Cypher retrieval for FalkorDB #4156

Proposed Changes:

• Add FalkorDBTextToCypherRetriever to turn natural-language requests into Cypher queries using a Haystack chat generator.
• Discover and cache a sampled FalkorDB graph schema to guide query generation.
• Execute generated queries in read-only mode and return Haystack documents, with synchronous and asynchronous support.

How did you test it?

Added integration and unit tests.

Notes for the reviewer

Checklist

• I have read the contributors guidelines and the code of conduct
• I have updated the related issue with new insights and changes
• I added unit tests and updated the docstrings
• I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.

[github-actions]

Hi @maxdswain, thanks for your interest in contributing to Haystack! 🙏

⚠️ You currently have 3 open pull requests in this repository (#4147, #4014 and this one). Our review capacity is limited, so please hold off opening more PRs until we've had a chance to review your first 2 open PRs. This helps us give each contribution the attention it deserves. Thank you!

This is an automated message to help us keep the review queue healthy.

[github-actions]

Coverage report (falkordb)

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
integrations/falkordb/src/haystack_integrations/components/retrievers/falkordb
text_to_cypher_retriever.py
integrations/falkordb/src/haystack_integrations/document_stores/falkordb
document_store.py					1554-1561
Project Total

This report was generated by python-coverage-comment-action

[davidsbatista]

Hi @maxdswain, thanks for this, it's a well-built component! We'd like to get it merged, so I pushed a few commits directly to your branch instead of going back and forth in review

• Non-node query results (e.g. RETURN d.content) now raise instead of becoming empty Documents. This also affects FalkorDBCypherRetriever.
• The schema is sampled per label and per relationship type, without duplicates.
• The nested chat generator is now warmed up.
• New top_k and timeout params; results are deduplicated and limited.
• document_store type is validated in init.

[davidsbatista]

I will do one last review tomorrow before merging and doing a new release

[davidsbatista]

thanks for this @maxdswain I will merge and make a new release

[maxdswain]

Thanks for implement those changes and merging @davidsbatista!