Skip to content

Revert "feat!: relax tool result structuredContent type (SEP-2106)"#932

Merged
DaleSeo merged 1 commit into
mainfrom
revert-919-fix/tool-result-content-structured-value
Jun 26, 2026
Merged

Revert "feat!: relax tool result structuredContent type (SEP-2106)"#932
DaleSeo merged 1 commit into
mainfrom
revert-919-fix/tool-result-content-structured-value

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Jun 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Reverts #919. This is a breaking change for 2026-07-28 so should wait for v2 to come out first.

@github-actions github-actions Bot added T-test Testing related changes T-config Configuration file changes T-core Core library changes T-model Model/data structure changes labels Jun 26, 2026
@DaleSeo DaleSeo marked this pull request as ready for review June 26, 2026 19:11
@DaleSeo DaleSeo requested a review from a team as a code owner June 26, 2026 19:11
@DaleSeo DaleSeo merged commit 4b9bea7 into main Jun 26, 2026
18 of 19 checks passed
@DaleSeo DaleSeo deleted the revert-919-fix/tool-result-content-structured-value branch June 26, 2026 20:00
@DaleSeo DaleSeo mentioned this pull request Jun 26, 2026
Draft
9 tasks
@github-actions github-actions Bot mentioned this pull request Jun 26, 2026
Open
DaleSeo added a commit that referenced this pull request Jun 26, 2026
@DaleSeo
feat!: relax tool result structuredContent type (SEP-2106)
5104d3d 
Re-applies #919 (reverted by #932): ToolResultContent.structured_content
becomes Option<Value> so non-object structured content is accepted,
matching CallToolResult and SEP-2106.

BREAKING CHANGE: ToolResultContent.structured_content changes from
Option<JsonObject> to Option<Value>.
DaleSeo added a commit that referenced this pull request Jun 26, 2026
@DaleSeo
feat!: relax tool result structuredContent type (SEP-2106)
6755d12 
Re-applies #919 (reverted by #932): ToolResultContent.structured_content
becomes Option<Value> so non-object structured content is accepted,
matching CallToolResult and SEP-2106.

BREAKING CHANGE: ToolResultContent.structured_content changes from
Option<JsonObject> to Option<Value>.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexhancock alexhancock alexhancock approved these changes

Assignees

No one assigned

Labels

T-config Configuration file changes T-core Core library changes T-model Model/data structure changes T-test Testing related changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DaleSeo @alexhancock