Skip to content

Clone graph#263

Merged
paul-paliychuk merged 37 commits intomainfrom
v2
Jun 20, 2025
Merged

Clone graph#263
paul-paliychuk merged 37 commits intomainfrom
v2

Conversation

@paul-paliychuk
Copy link
Copy Markdown
Contributor

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

Important

Add graph cloning functionality with clone() method in graph/client.py and update version to 2.16.0.

  • Graph Cloning:
    • Added clone() method in graph/client.py to clone user or group graphs.
    • Supports optional parameters source_group_id, source_user_id, target_group_id, target_user_id.
    • Returns CloneGraphResponse with new graph identifiers.
  • Types:
    • Added CloneGraphResponse in types/clone_graph_response.py.
  • Version Update:
    • Updated version to 2.16.0 in pyproject.toml and core/client_wrapper.py.
  • Deprecations:
    • Marked several methods in memory/client.py as deprecated, including classify_session, extract_data, end_session, and search_sessions.

This description was created by Ellipsis for 38806fd. 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.

Caution

Changes requested ❌

Reviewed everything up to 38806fd in 2 minutes and 3 seconds. Click for details.
  • Reviewed 460 lines of code in 9 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 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 auto-generated CloneGraphResponse model looks clear and correctly overrides json() and dict() to merge defaults. Verify that deep_union_pydantic_dicts handles all edge cases, but overall this code meets our standards.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_NnhQMoxOBynhmtH9

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

Comment thread reference.md
<dl>
<dd>

**source_group_id:** `typing.Optional[str]` — group_id of the group whose graph is being cloned. Required if user_id is not provided
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Typo: The description for source_group_id mentions "Required if user_id is not provided". For consistency, it should probably read "Required if source_user_id is not provided".

Suggested change
**source_group_id:** `typing.Optional[str]` — group_id of the group whose graph is being cloned. Required if user_id is not provided
**source_group_id:** `typing.Optional[str]` — group_id of the group whose graph is being cloned. Required if source_user_id is not provided

This comment was generated because it violated a code review rule: mrule_akx655bcesz2pGU1.

Comment thread reference.md
<dl>
<dd>

**source_user_id:** `typing.Optional[str]` — user_id of the user whose graph is being cloned. Required if group_id is not provided
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Typo: The description for source_user_id mentions "Required if group_id is not provided". For consistency, it should probably read "Required if source_group_id is not provided".

Suggested change
**source_user_id:** `typing.Optional[str]` — user_id of the user whose graph is being cloned. Required if group_id is not provided
**source_user_id:** `typing.Optional[str]` — user_id of the user whose graph is being cloned. Required if source_group_id is not provided

This comment was generated because it violated a code review rule: mrule_akx655bcesz2pGU1.

@paul-paliychuk paul-paliychuk merged commit 438fb25 into main Jun 20, 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