Skip to content

Fix Vector Search MCP tool names; add end-to-end RAG walkthrough#213

Merged
calreynolds merged 1 commit intodatabricks-solutions:mainfrom
CheeYuTan:feat/vector-search-fix-tools-add-rag
Mar 5, 2026
Merged

Fix Vector Search MCP tool names; add end-to-end RAG walkthrough#213
calreynolds merged 1 commit intodatabricks-solutions:mainfrom
CheeYuTan:feat/vector-search-fix-tools-add-rag

Conversation

@CheeYuTan
Copy link
Contributor

Summary

Addresses #106.

Bug fix — incorrect MCP tool names in SKILL.md:

  • Replaced non-existent consolidated tools (create_or_update_vs_endpoint, manage_vs_data) with actual individual tools (create_vs_endpoint, list_vs_endpoints, sync_vs_index, upsert_vs_data, etc.)
  • This was a real bug — agents would fail trying to call tools that don't exist

New file — end-to-end-rag.md:

  • Complete walkthrough: source table → endpoint → index → query → agent
  • Covers columns_to_sync guidance, filter syntax differences (Standard vs Storage-Optimized), hybrid search
  • Shows VectorSearchRetrieverTool integration for ChatAgent

All MCP tool names verified against live workspace.

Test plan

  • MCP tool names verified against tool JSON schemas and live workspace
  • Filter syntax differences (Standard vs Storage-Optimized) documented correctly
  • install_skills.sh updated with new extra file

Addresses databricks-solutions#106.

Fix incorrect MCP tool names in SKILL.md:
- Replace non-existent consolidated tools (create_or_update_vs_endpoint,
  manage_vs_data) with actual individual tools (create_vs_endpoint,
  list_vs_endpoints, sync_vs_index, upsert_vs_data, etc.)
- This was a real bug — agents would fail trying to call tools that
  don't exist

Add end-to-end-rag.md:
- Complete walkthrough: source table -> endpoint -> index -> query -> agent
- Covers columns_to_sync guidance, filter syntax differences
  (Standard vs Storage-Optimized), hybrid search
- Shows VectorSearchRetrieverTool integration for ChatAgent

All MCP tool names verified against live workspace.
Copy link
Collaborator

@calreynolds calreynolds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!!! 👍

@calreynolds calreynolds linked an issue Mar 5, 2026 that may be closed by this pull request
@calreynolds
Copy link
Collaborator

Doesn't go into issue 106 since we'd want to do some testing with our .test framework, but these are great skill modifications 👍

@calreynolds calreynolds merged commit e9d3980 into databricks-solutions:main Mar 5, 2026
2 checks passed
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