Skip to content

Add compose context string search utility#264

Merged
paul-paliychuk merged 39 commits intomainfrom
v2
Jun 26, 2025
Merged

Add compose context string search utility#264
paul-paliychuk merged 39 commits intomainfrom
v2

Conversation

@paul-paliychuk
Copy link
Copy Markdown
Contributor

@paul-paliychuk paul-paliychuk commented Jun 25, 2025

Important

Add compose_context_string utility and clone method to GraphClient, update version, and deprecate some methods.

  • New Functionality:
    • Add compose_context_string() in utils.py to format context from EntityEdge and EntityNode.
    • Add clone() method to GraphClient and AsyncGraphClient in client.py for cloning user or group graphs.
  • Version Update:
    • Update version to 2.17.0 in pyproject.toml and client_wrapper.py.
  • Deprecations:
    • Mark classify_session() and extract_data() as deprecated in memory.py and client.py.
  • Miscellaneous:
    • Add CloneGraphResponse to __init__.py and types/__init__.py.
    • Update reference.md to include new clone() method documentation.

This description was created by Ellipsis for d30bc63. You can customize this summary. It will automatically update as commits are pushed.

Copy link
Copy Markdown
Contributor

@ellipsis-dev ellipsis-dev Bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to d30bc63 in 48 seconds. Click for details.
  • Reviewed 548 lines of code in 11 files
  • Skipped 0 files when reviewing.
  • Skipped posting 3 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. src/zep_cloud/types/clone_graph_response.py:10
  • Draft comment:
    The CloneGraphResponse model looks well‐defined. The custom json() and dict() methods use deep_union_pydantic_dicts to merge the dict representations with exclusion of unset/none values. Verify that this merging behavior meets your API contract as it may mask some differences.
  • Reason this comment was not posted:
    Confidence changes required: 50% <= threshold 50% None
2. src/zep_cloud/graph/utils.py:8
  • Draft comment:
    The TEMPLATE_STRING contains inline documentation. Consider using a more structured format (e.g., markdown) to improve readability in client documentation. Also double-check the formatting of the resulting context string.
  • Reason this comment was not posted:
    Confidence changes required: 40% <= threshold 50% None
3. src/zep_cloud/memory/client.py:28
  • Draft comment:
    The extract() method's docstring clearly marks it as deprecated and provides usage examples. Ensure that the deprecation notice is communicated to users (for example, via warning logs) if intended.
  • Reason this comment was not posted:
    Confidence changes required: 30% <= threshold 50% None

Workflow ID: wflow_ZuBR5ioxD28mB3WS

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@paul-paliychuk paul-paliychuk merged commit 0f6f476 into main Jun 26, 2025
5 checks passed
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.

1 participant