Skip to content

fix MCP list_databases output schema and params context handling#12378

Closed
lucarizzo03 wants to merge 1 commit into
mindsdb:mainfrom
lucarizzo03:fix/mcp-list-databases-output-schema
Closed

fix MCP list_databases output schema and params context handling#12378
lucarizzo03 wants to merge 1 commit into
mindsdb:mainfrom
lucarizzo03:fix/mcp-list-databases-output-schema

Conversation

@lucarizzo03
Copy link
Copy Markdown

Summary

  • Ensure list_databases returns only list output on success
  • Raise MCP tool errors instead of returning schema-incompatible error dicts
  • Add defensive handling for missing ctx.params in MCP execution paths

Validation

  • Reproduced original schema validation failure
  • Retested list_databases after fix; now returns database names without output schema error

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 14, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@lucarizzo03
Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request Apr 14, 2026
@StpMax
Copy link
Copy Markdown
Collaborator

StpMax commented Apr 17, 2026

@lucarizzo03 Thank you for your contribution. Unfortunately, I have to close your PR. The MCP server has already been significantly reworked in the upcoming update, which is located in this branch.

@StpMax StpMax closed this Apr 17, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants