Skip to content

Implement message-id rfd#76

Merged
nerzhulart merged 4 commits intoagentclientprotocol:masterfrom
fabmax:fabmax/rfd-message-id
Mar 4, 2026
Merged

Implement message-id rfd#76
nerzhulart merged 4 commits intoagentclientprotocol:masterfrom
fabmax:fabmax/rfd-message-id

Conversation

@fabmax
Copy link
Copy Markdown
Contributor

@fabmax fabmax commented Mar 4, 2026

This adds the messageId and userMessageId fields from the message-id RFD

@fabmax
Copy link
Copy Markdown
Contributor Author

fabmax commented Mar 4, 2026

Maybe messageId should also be added to the ClientSession.prompt() interface but I'm not sure if that's ok since messageId is an unstable feature.

@nerzhulart
Copy link
Copy Markdown
Contributor

nerzhulart commented Mar 4, 2026 •
edited
Loading

Please make the Id strongly typed as it's done for AuthMethodId, ToolCallId, SessionId, etc

@fabmax
Copy link
Copy Markdown
Contributor Author

fabmax commented Mar 4, 2026

Changed the message ID to a value type

@nerzhulart nerzhulart self-requested a review March 4, 2026 19:13
@nerzhulart nerzhulart merged commit ca7df84 into agentclientprotocol:master Mar 4, 2026
1 check passed
@fabmax fabmax deleted the fabmax/rfd-message-id branch March 4, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nerzhulart nerzhulart nerzhulart 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

@fabmax @nerzhulart