You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This commit was created on GitHub.com and signed with GitHub’s verified signature.
What's Changed
Breaking Changes
analyze_file / analyze_symbol: remove force, verbose, and ast_recursion_limit parameters (#1130, closes #1129): Usage telemetry across 1,798 analyze_file and 103 analyze_symbol calls showed 98.9% and 99% of callers injecting ast_recursion_limit: 0 mechanically in response to documented examples, not intentional tuning. force and verbose likewise saw no meaningful use. Removing the parameters eliminates the noise and lets library defaults apply unconditionally. Callers still passing these fields will have them silently ignored; serde does not use deny_unknown_fields on these structs.
Bug Fixes
analyze_symbol: add missing no_cache_meta to pagination error responses (#1131): err_to_tool_result_from_pagination returned CallToolResult::error() without attaching no_cache_meta(). The two sibling helpers err_to_tool_result and tool_error both attached it correctly. MCP clients respecting cache hints could cache the pagination error and return stale failures on retry. A regression test (test_no_cache_meta_on_pagination_error) is added to the integration suite.