Skip to content

fix: MCPToolApprovalRequest definition#600

Merged
mrh997 merged 1 commit intorelease/agentic_modelfrom
feat/mrh/agentic_model
Dec 2, 2025
Merged

fix: MCPToolApprovalRequest definition#600
mrh997 merged 1 commit intorelease/agentic_modelfrom
feat/mrh/agentic_model

Conversation

@mrh997
Copy link
Copy Markdown
Contributor

@mrh997 mrh997 commented Dec 2, 2025

What type of PR is this?

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

@mrh997 mrh997 merged commit 32c386b into release/agentic_model Dec 2, 2025
8 of 9 checks passed
@mrh997 mrh997 deleted the feat/mrh/agentic_model branch December 2, 2025 11:09
ContentBlockTypeMCPToolResult ContentBlockType = "mcp_tool_result"
ContentBlockTypeMCPListTools ContentBlockType = "mcp_list_tools"
ContentBlockTypeMCPListToolsResult ContentBlockType = "mcp_list_tools_result"
ContentBlockTypeMCPToolApprovalRequest ContentBlockType = "mcp_tool_approval_request"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚨 Breaking API Changes Detected

Package: github.com/cloudwego/eino/schema

Incompatible changes:

  • ContentBlockTypeMCPListTools: removed
  • MCPToolApprovalRequest.CallID: removed
Review Guidelines

Please ensure that:

  • The changes are absolutely necessary
  • They are properly documented
  • Migration guides are provided if needed

⚠️ Please resolve this thread after reviewing the breaking changes.

mrh997 added a commit that referenced this pull request Dec 22, 2025
mrh997 added a commit that referenced this pull request Jan 6, 2026
mrh997 added a commit that referenced this pull request Jan 6, 2026
mrh997 added a commit that referenced this pull request Jan 15, 2026
mrh997 added a commit that referenced this pull request Jan 15, 2026
mrh997 added a commit that referenced this pull request Jan 22, 2026
mrh997 added a commit that referenced this pull request Jan 22, 2026
mrh997 added a commit that referenced this pull request Feb 24, 2026
mrh997 added a commit that referenced this pull request Mar 12, 2026
mrh997 added a commit that referenced this pull request Apr 10, 2026
N3kox pushed a commit that referenced this pull request Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant