Skip to content

fix(mcp,text): MCP activation hints and fix error caused by enable_thinking - #126

Merged
XXPermanentXX merged 9 commits into
mainfrom
fix/web-search-and-thinking
Jul 28, 2026
Merged

fix(mcp,text): MCP activation hints and fix error caused by enable_thinking#126
XXPermanentXX merged 9 commits into
mainfrom
fix/web-search-and-thinking

Conversation

@hopelynd

@hopelyndhopelynd commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add actionable activation hints when Bailian MCP servers return 404 not-activated / invalid errors, with deep links to the MCP marketplace detail page (mcpMarketplaceDetailPage).
  • Generalize WebSearch-specific hint logic into shared activate-hint helpers and apply them to search web, mcp tools, and mcp call (skipped when --url overrides the endpoint).
  • Simplify text chat / API key validation: only send enable_thinking when explicitly requested; omit the field by default and pass model constraint errors through to the user.
  • Switch the token-plan default text model to qwen3.7-plus and align related e2e expectations.

…s false
Non-streaming chat no longer forces enable_thinking=false, which breaks
thinking-only models. Retry once with false only when the server demands it.
Pass through model constraint errors instead of auto-retrying, switch token-plan default text model to qwen3.7-plus, and align related e2e expectations.
@hopelyndhopelynd changed the title fix(mcp,text): MCP activation hints and enable_thinking retryfix(mcp,text): MCP activation hints and simpler enable_thinking defaultsJul 28, 2026
@hopelyndhopelynd changed the title fix(mcp,text): MCP activation hints and simpler enable_thinking defaultsfix(mcp,text): MCP activation hints and fix error caused by enable_thinkingJul 28, 2026
@XXPermanentXX
XXPermanentXX merged commit 3b7993e into mainJul 28, 2026
2 checks passed
@XXPermanentXX
XXPermanentXX deleted the fix/web-search-and-thinking branch July 28, 2026 11:32
Sign up for freeto 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.

2 participants

@hopelynd@XXPermanentXX