Skip to content

Conversation

@trrwilson
Copy link
Member

Problem

After the original integration, AzureAISearchAgentTool added a resource property -- this wasn't customized for pattern or naming consistency and the AgentTool.CreateAzureAISearchTool() factory method wasn't updated to match, instead using a now-internal, deserialization-only constructor that didn't set a value for this required resource property.

Fixes

  • Renamed the new "resource" to AzureAISearchToolOptions for naming/pattern consistency
    • Renamed consumed type for index to AzureAISearchIndex
  • Made both parameterless and parameterized constructors for the tool work as intended (empty options for parameterless)
  • Updated tool factory method to accept the now-renamed options type
  • Updated smoke test coverage to exercise this specific tool/data

@trrwilson trrwilson merged commit 60edffb into feature/ai-foundry/agents-v2 Nov 4, 2025
13 of 21 checks passed
@trrwilson trrwilson deleted the user/travisw/ai-foundry/agents-v2/fix-ai-search-tool branch November 4, 2025 19:36
nick863 pushed a commit that referenced this pull request Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants