-
Notifications
You must be signed in to change notification settings - Fork 2.1k
docs(spec): Added normative content to specification.md that lived in topics #1050
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.
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
base: main
Are you sure you want to change the base?
Conversation
|
Looks good Darrel! |
|
|
||
| ### 14.1. Ordering | ||
|
|
||
| Servers **MUST** emit: initial `Task` → zero/more updates → single `final` status update → close. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't the case if the client sends a message using message/stream and the agent decides to respond with a message.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, cause if the server sends a message than the stream should be considered terminated, so no resubscribe is possible (there also does not exist a task id to resubscribe to)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But yes, the spec should make that clear
Servers MUST EITHER
- Emit a single message and close the stream
- OR emit an initial
Task-> zero or more updates. If there are any updates the last one MUST be afinalstatus update
Description
This PR attempts to add content to the specification.md file that currently is only mentioned in the topics section.
The text in this PR is primarily LLM generated and this PR is draft because it is only here for the purpose of getting TSC alignment on moving forward with this approach.
CONTRIBUTINGGuide.nox -s formatfrom the repository root to format)