Skip to content
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

Updates Neo4jChatMessageHistory to use neo4j-graphrag queries #48

Conversation

alexthomas93
Copy link
Collaborator

Description

  • Refactors Neo4jChatMessageHistory to use Cypher queries from the neo4j-graphrag package.
  • Renames the type property to role on Message nodes saved by the Neo4jChatMessageHistory class.
  • Adds a delete_session_node parameter to Neo4jChatMessageHistory's clear method allowing developers to optionally select if they want to delete the Session node along with all Message nodes in a chat message chain.

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Project configuration change

Complexity

Low

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Manual tests

Checklist

  • Unit tests updated
  • Integration tests updated
  • CHANGELOG.md updated

@alexthomas93 alexthomas93 force-pushed the refactor/neo4j-chat-message-history branch from 16443bd to 0ca4779 Compare February 25, 2025 18:30
@alexthomas93 alexthomas93 marked this pull request as ready for review February 26, 2025 09:51
@alexthomas93 alexthomas93 force-pushed the refactor/neo4j-chat-message-history branch from 0ca4779 to 4743719 Compare February 26, 2025 09:53
@alexthomas93 alexthomas93 merged commit b6f58b2 into langchain-ai:main Feb 26, 2025
8 checks passed
@alexthomas93 alexthomas93 deleted the refactor/neo4j-chat-message-history branch February 26, 2025 10:21
@alexthomas93
Copy link
Collaborator Author

@CodiumAI-Agent /update_changelog

@CodiumAI-Agent
Copy link

Changelog updates: 🔄

2025-02-26

Changed

  • Refactored Neo4jChatMessageHistory to use Cypher queries from the neo4j-graphrag package.
  • Renamed the type property to role on Message nodes in Neo4jChatMessageHistory.

Added

  • Introduced a delete_session_node parameter to the clear method in Neo4jChatMessageHistory for optional deletion of the Session node.

to commit the new content to the CHANGELOG.md file, please type:
'/update_changelog --pr_update_changelog.push_changelog_changes=true'

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.

3 participants