Skip to content

chore: Use thread.get API to fetch most recent thread messages in memory component - #416

Merged
paul-paliychuk merged 1 commit into
mainfrom
update-autogen-integration
Aug 9, 2025
Merged

paul-paliychuk merged 1 commit into
mainfrom
update-autogen-integration

Conversation

@paul-paliychuk

@paul-paliychuk paul-paliychuk commented Aug 9, 2025

Copy link
Copy Markdown
Collaborator

Important

Use thread.get API for recent messages in memory.py and update autogen_basic.py and pyproject.toml.

  • Behavior:
    • In memory.py, update_context() now uses thread.get API to fetch the last 10 messages, improving message retrieval.
    • In autogen_basic.py, add_message() now includes a name parameter for user messages.
  • Dependencies:
    • Updated zep-cloud dependency to >=3.2.0 in pyproject.toml.
    • Bumped version to 1.0.0 in pyproject.toml.

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

@ellipsis-dev ellipsis-dev Bot left a comment

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.

Important

Looks good to me! 👍

Reviewed everything up to 65eb07b in 57 seconds. Click for details.
  • Reviewed 74 lines of code in 3 files
  • Skipped 0 files when reviewing.
  • Skipped posting 4 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. integrations/python/zep_autogen/examples/autogen_basic.py:66
  • Draft comment:
    Explicitly passing the user name ('Alice') improves clarity in message provenance. Ensure consistency in naming across all message additions if intended.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
2. integrations/python/zep_autogen/pyproject.toml:3
  • Draft comment:
    Version bumped to 1.0.0 and 'zep-cloud' dependency upgraded to >=3.2.0. Ensure the changelog and documentation reflect any breaking changes introduced.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
3. integrations/python/zep_autogen/src/zep_autogen/memory.py:243
  • Draft comment:
    Using the new thread.get API with the 'lastn=10' parameter efficiently fetches recent messages. Verify that the returned messages meet your ordering and content expectations.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
4. integrations/python/zep_autogen/src/zep_autogen/memory.py:262
  • Draft comment:
    Consider renaming the variable 'thread' to avoid potential conflicts with built-in names or ambiguity, even though it functions correctly as is.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_r6bpl8QhOQaKTBlT

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 a8a83fc into main Aug 9, 2025
9 checks passed
@paul-paliychuk
paul-paliychuk deleted the update-autogen-integration branch August 9, 2025 00:30
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