Skip to content

[Store] Add Neo4j #183

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 29, 2025
Merged

[Store] Add Neo4j #183

merged 1 commit into from
Jul 29, 2025

Conversation

Guikingone
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
Docs? yes
Issues None
License MIT

Hi 👋🏻

This PR aim to introduce the Neo4J store as a vector store, the full implementation is based around the HTTP client and, the PR is not fully ready, unit tests must be completed.

@Guikingone Guikingone changed the title feat(store): neo4j started [Store] Add Neo4j Jul 22, 2025
@OskarStark OskarStark added the Store Issues & PRs about the AI Store component label Jul 23, 2025
@OskarStark OskarStark changed the title [Store] Add Neo4j Add Neo4j Jul 23, 2025
@OskarStark OskarStark added Store Issues & PRs about the AI Store component and removed Store Issues & PRs about the AI Store component labels Jul 23, 2025
@OskarStark OskarStark changed the title Add Neo4j [Store] Add Neo4j Jul 23, 2025
Copy link
Member

@chr-hertel chr-hertel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From a functional point of view this looks good - only two tiny things on code level 👍

@Guikingone
Copy link
Contributor Author

@chr-hertel Totally, I'm almost done with the tests, one still tricky, need a few hours on it and this PR should be good, sorry for the delay.

@Guikingone
Copy link
Contributor Author

Just to add context, the tests are not built around "failure / success" as the neo4j API handle things differently, if an index / document already exist, the operation is not performed but 202 is returned, that prevent failures at runtime and the need to test them.

@Guikingone Guikingone requested a review from chr-hertel July 29, 2025 09:09
Copy link
Member

@chr-hertel chr-hertel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code & functional test look good 👍

@chr-hertel
Copy link
Member

Thank you @Guikingone.

@chr-hertel chr-hertel merged commit 99dc222 into symfony:main Jul 29, 2025
13 of 14 checks passed
OskarStark added a commit that referenced this pull request Aug 1, 2025
…e (chr-hertel)

This PR was merged into the main branch.

Discussion
----------

[Examples] Fix OpenAI and GPT namespaces in Neo4j RAG example

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| Issues        |
| License       | MIT

Follow up after #217 and #183

Commits
-------

7770885 Fix OpenAI and GPT namespaces in Neo4j RAG example
@Guikingone Guikingone deleted the store/neo4j branch August 1, 2025 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Work Store Issues & PRs about the AI Store component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants