Skip to content

Conversation

@ashwin-ant
Copy link
Collaborator

Hide hooks, tool permission callbacks, and SDK MCP server APIs from public interface while keeping implementation code intact. These features are not yet stable and should not be documented or exposed to users.

Changes:

  • Remove hook-related exports (HookCallback, HookContext, HookMatcher) from all
  • Remove tool permission exports (CanUseTool, ToolPermissionContext) from all
  • Remove SDK MCP exports (tool, create_sdk_mcp_server, SdkMcpTool) from all
  • Delete examples using unstable APIs (tool_permission_callback.py, mcp_calculator.py)
  • Remove SDK MCP server documentation from README

🤖 Generated with Claude Code

ashwin-ant and others added 3 commits September 5, 2025 11:35
Hide hooks, tool permission callbacks, and SDK MCP server APIs from public interface while keeping implementation code intact. These features are not yet stable and should not be documented or exposed to users.

Changes:
- Remove hook-related exports (HookCallback, HookContext, HookMatcher) from __all__
- Remove tool permission exports (CanUseTool, ToolPermissionContext) from __all__
- Remove SDK MCP exports (tool, create_sdk_mcp_server, SdkMcpTool) from __all__
- Delete examples using unstable APIs (tool_permission_callback.py, mcp_calculator.py)
- Remove SDK MCP server documentation from README

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Update test_tool_callbacks.py to import hidden types directly from types module instead of from main package exports.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Remove test_tool_callbacks.py since it tests hooks and tool permission callbacks which are no longer part of the public API.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ashwin-ant ashwin-ant merged commit e4feaf2 into main Sep 5, 2025
6 checks passed
@jasonzbao
Copy link

just curious - what's unstable about these features?

@ashwin-ant ashwin-ant deleted the ashwin/rmstuff branch September 7, 2025 03:44
@ashwin-ant
Copy link
Collaborator Author

@jasonzbao we discovered some bugs that we're working through, but wanted to release a version bump of the SDK to get some other fixes released. Stay tuned on this, we're gonna bring these back!

ashwin-ant added a commit that referenced this pull request Sep 8, 2025
@nishikantparmariam1
Copy link

Really interested in the support for in-process tools, @ashwin-ant is there is tentative timeline when it will be brought back bug-free.

Thanks in advance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants