Skip to content

feat(core): support sync messaging in uagents-core - #755

Merged
jrriehl merged 6 commits into
mainfrom
feat/support-sync-messaging-in-uagents-core
Aug 7, 2025
Merged

feat(core): support sync messaging in uagents-core#755
jrriehl merged 6 commits into
mainfrom
feat/support-sync-messaging-in-uagents-core

Conversation

@jrriehl

@jrriehl jrriehl commented Aug 4, 2025

Copy link
Copy Markdown
Contributor

Proposed Changes

Add support for sync messaging in uagents_core

Types of changes

What type of change does this pull request make (put an x in the boxes that apply)?

  • Bug fix (non-breaking change that fixes an issue).
  • New feature added (non-breaking change that adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to stop working as expected).
  • Documentation update.
  • Something else (e.g., tests, scripts, example, deployment, infrastructure).

Checklist

Put an x in the boxes that apply:

  • I have read the CONTRIBUTING guide
  • Checks and tests pass locally

If applicable

  • I have added tests that prove my fix is effective or that my feature works
  • I have added/updated the documentation (executed the script in python/scripts/generate_api_docs.py)

@jrriehl
jrriehl marked this pull request as ready for review August 4, 2025 11:01
@qati
qati requested a review from Copilot August 4, 2025 11:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for synchronous messaging in the uagents-core library, allowing agents to send messages and wait for responses in a blocking manner.

Key changes:

  • Added sync parameter to message sending functions to enable synchronous communication
  • Implemented response parsing and validation for sync messages with expected response types
  • Added appropriate headers for sync message identification

Comment thread python/uagents-core/uagents_core/utils/messages.py Outdated
Comment thread python/uagents-core/uagents_core/utils/messages.py
Comment thread python/uagents-core/uagents_core/utils/messages.py
Comment thread python/uagents-core/uagents_core/utils/messages.py Outdated
@jrriehl

jrriehl commented Aug 4, 2025

Copy link
Copy Markdown
Contributor Author

Update

  • fixed some redundant logic with the with the number of endpoints resolved
  • once a message is sent successfully, do not continue to send to other endpoints
  • refactored parse envelope response into new function

@jrriehl
jrriehl merged commit fe1ac87 into main Aug 7, 2025
10 checks passed
@jrriehl
jrriehl deleted the feat/support-sync-messaging-in-uagents-core branch August 7, 2025 10:25
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.

2 participants