Skip to content

Conversation

AdamVyborny
Copy link
Contributor

Summary

  • Add required query_name parameter to query_data SQL tool
  • Return structured output with both query name and CSV data
  • Update all related tests to handle new parameter and output format

Why this change is needed

MCP clients need to work with named query results for better tracking and identification. Having the query name come directly from the MCP server response makes client-side handling much simpler and more reliable than managing query names separately on the client side.

Test plan

  • Unit tests pass with new parameter and structured output
  • Integration tests updated and passing
  • Code style checks pass

🤖 Generated with Claude Code

Add required query_name parameter to query_data SQL tool and return structured output containing both query name and CSV data. This enables MCP clients to work with named query results for better tracking and identification.

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

Co-Authored-By: Claude <[email protected]>
@AdamVyborny AdamVyborny requested a review from a team July 17, 2025 11:52
@radcliffkey
Copy link
Contributor

please update the version in pyproject.toml

@radcliffkey radcliffkey merged commit bb343f4 into main Jul 18, 2025
19 checks passed
@radcliffkey radcliffkey deleted the adamvyborny-add-query-name branch July 18, 2025 11:47
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.

2 participants