Skip to content

Python: Foundry Agent Completeness - #954

Merged
Evan Mattson (moonbox3) merged 15 commits into
microsoft:mainfrom
giles17:azureai_agent_completeness
Oct 1, 2025
Merged

Python: Foundry Agent Completeness#954
Evan Mattson (moonbox3) merged 15 commits into
microsoft:mainfrom
giles17:azureai_agent_completeness

Conversation

@giles17

@giles17 Giles Odigwe (giles17) commented Sep 26, 2025

Copy link
Copy Markdown
Contributor

Motivation and Context

Description

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

Copilot AI review requested due to automatic review settings September 26, 2025 21:59
@markwallace-microsoft Mark Wallace (markwallace-microsoft) added documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs python Usage: [Issues, PRs], Target: Python labels Sep 26, 2025
@github-actions github-actions Bot changed the title Foundry Agent Completeness Python: Foundry Agent Completeness Sep 26, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for Azure AI File Search functionality to the Agent Framework, enabling agents to search through uploaded documents using vector stores.

  • Adds HostedFileSearchTool support with file upload and vector store creation
  • Introduces tool_resources configuration for proper Azure AI file search integration
  • Includes comprehensive test coverage and sample code demonstrating file search capabilities

Reviewed Changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 2 comments.

File Description
azure_ai_with_file_search.py New sample demonstrating file search capabilities with document upload and querying
README.md Adds documentation for the new file search example
test_azure_ai_agent_client.py Adds integration test for file search functionality and updates test helper function
_chat_client.py Implements file search tool support with tool_resources configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread python/packages/azure-ai/tests/test_azure_ai_agent_client.py Outdated
Comment thread python/packages/azure-ai/tests/test_azure_ai_agent_client.py Outdated
@markwallace-microsoft

Mark Wallace (markwallace-microsoft) commented Sep 26, 2025

Copy link
Copy Markdown
Contributor

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
packages/azure-ai/agent_framework_azure_ai
   _chat_client.py3686682%193, 198, 220–223, 227–228, 230, 304, 370, 439–440, 442–443, 451, 463–465, 469–471, 479–482, 492, 500–505, 512, 520–521, 529–530, 535–536, 540–547, 552–553, 561, 567, 575–577, 580, 661–663, 666–667, 682, 687, 726–727, 906
TOTAL10138168783% 

Python Unit Test Overview

Tests Skipped Failures Errors Time
1213 98 💤 0 ❌ 0 🔥 31.575s ⏱️

Comment thread python/samples/getting_started/agents/azure_ai/azure_ai_with_bing_grounding.py Outdated
Comment thread python/samples/getting_started/agents/azure_ai/azure_ai_with_file_search.py Outdated
@giles17
Giles Odigwe (giles17) added this pull request to the merge queue Sep 30, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 30, 2025
@moonbox3
Evan Mattson (moonbox3) added this pull request to the merge queue Oct 1, 2025
Merged via the queue into microsoft:main with commit b0971fd Oct 1, 2025
17 checks passed
Reuben Bond (ReubenBond) pushed a commit to ReubenBond/agent-framework that referenced this pull request Oct 28, 2025
* foundry completeness

* tests + openapi sample

* bing grounding sample

* options integration tests

* merge conflict fix

* fix failing test

* add mcp approval handling
AQ-1970 (Arturo-Quiroga-MSFT) pushed a commit to Arturo-Quiroga-MSFT/agent-framework-public that referenced this pull request Nov 23, 2025
* foundry completeness

* tests + openapi sample

* bing grounding sample

* options integration tests

* merge conflict fix

* fix failing test

* add mcp approval handling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs python Usage: [Issues, PRs], Target: Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants