Skip to content

feat: Streaming interrupts and errors for stateless runs #107

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.

Sign up for GitHub

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

Jump to bottom
Merged
gicont merged 12 commits into from
May 28, 2025
Merged

feat: Streaming interrupts and errors for stateless runs #107

gicont merged 12 commits into from
May 28, 2025
+7,385 −8,345

Conversation

@jnapper7
Copy link
Contributor

@jnapper7 jnapper7 commented May 26, 2025 •
edited
Loading

  • Adds request timeout support for ACP clients and generator templates.
  • Updates echo-agent to have configurable interrupts and logs. Also moves echo-agent to uv as package manager.
  • Move test-client to uv package manager. Add tests for streaming interrupts.
  • Add interrupt and error unit tests to streaming tests.
  • Update workflow-server-manager submodule.
  • Bump package version to include request support.

jnapper7 added 11 commits May 23, 2025 14:09
@jnapper7
chore: add streaming interrupt and error to unit tests.
a07279d 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
fix: add missing arg
09f2bc8 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
chore: update echo-agent to add interrupt to manifest and configs
936cacb 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
fix: format and remove unnecessary file
51f1510 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
chore: update test-client and echo-agent to use uv
6a3d288 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
chore: add more asserts to stream tests.
6348024 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
fix: updated stateless streaming to new sdk release.
ef2523a 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
fix: removed extraneous print from SDK. Update echo-agent to support …
1784f1c 
…resumable interrupts and multiple interrupts.

Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
chore: formatting updates
85fe08a 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
feat: fixes for interrupt handling in test-client and echo agent.
2562dd7 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7
fix: update to match new echo-agent
5108063 
Signed-off-by: Jeff Napper <[email protected]>
@jnapper7 jnapper7 requested a review from a team as a code owner May 26, 2025 11:01
@jnapper7 jnapper7 requested a review from gicont May 26, 2025 11:01
@jnapper7
fix: formatting
df6ab0e 
Signed-off-by: Jeff Napper <[email protected]>
gicont
gicont approved these changes May 28, 2025
View reviewed changes
Copy link
Contributor

@gicont gicont left a comment

Choose a reason for hiding this comment

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

LGTM

@gicont gicont merged commit 80bee3a into agntcy:main May 28, 2025
5 checks passed
@jnapper7 jnapper7 deleted the streaming-interrupts branch May 28, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gicont gicont gicont approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jnapper7 @gicont