Repository navigation
Migrate Python Agent Framework samples to uv - #1068
Merged
Merged
Conversation
luyajun0205
marked this pull request as ready for review
October 13, 2026 04:56
luyajun0205
requested review from
a team,
Tao Chen (TaoChenOSU),
Ben Thomas (alliscode) and
Linda Li (lindazqli)
as code owners
October 13, 2026 04:56
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The shared README still advertises requirements-based setup even though the migrated tree contains no requirements files.
1 open finding
What changed in this PR
Migrates remaining Python Agent Framework samples to uv-based dependency management and aligns container and setup documentation.
Changes:
- Replaces requirements files with pinned
pyproject.tomlprojects. - Updates Docker builds and provisioning hooks to use locked uv environments.
- Standardizes Python 3.13 and uv setup documentation.
Paths are relative to samples/python/hosted-agents/agent-framework/.
| File | Description |
|---|---|
responses/22-foundry-toolbox-mcp-skills/src/agent-framework-agent-foundry-toolbox-mcp-skills-responses/uv.toml |
Enables system certificates for uv. |
responses/22-foundry-toolbox-mcp-skills/src/agent-framework-agent-foundry-toolbox-mcp-skills-responses/requirements.txt |
Removes legacy requirements. |
responses/22-foundry-toolbox-mcp-skills/src/agent-framework-agent-foundry-toolbox-mcp-skills-responses/pyproject.toml |
Defines pinned project dependencies. |
responses/22-foundry-toolbox-mcp-skills/src/agent-framework-agent-foundry-toolbox-mcp-skills-responses/Dockerfile |
Builds and runs with uv. |
responses/22-foundry-toolbox-mcp-skills/README.md |
Documents uv setup and execution. |
responses/20-harness-data-processing/README.md |
Simplifies uv environment setup. |
responses/19-harness-research/README.md |
Simplifies uv environment setup. |
responses/18-egress-control/src/agent-framework-egress-control-responses/uv.toml |
Enables system certificates for uv. |
responses/18-egress-control/src/agent-framework-egress-control-responses/requirements.txt |
Removes legacy requirements. |
responses/18-egress-control/src/agent-framework-egress-control-responses/pyproject.toml |
Defines pinned project dependencies. |
responses/18-egress-control/src/agent-framework-egress-control-responses/Dockerfile |
Builds and runs with uv. |
responses/18-egress-control/README.md |
Adds uv-based local setup. |
responses/16-content-safety-guardrail/README.md |
Simplifies uv environment setup. |
responses/15-optimization-travel-approver/README.md |
Simplifies uv environment setup. |
responses/14-browser-automation-agent/src/bat-python-maf/uv.toml |
Enables system certificates for uv. |
responses/14-browser-automation-agent/src/bat-python-maf/requirements.txt |
Removes legacy requirements. |
responses/14-browser-automation-agent/src/bat-python-maf/pyproject.toml |
Defines pinned browser-agent dependencies. |
responses/14-browser-automation-agent/src/bat-python-maf/Dockerfile |
Builds and runs with uv. |
responses/14-browser-automation-agent/README.md |
Documents locked uv setup. |
responses/13-foundry-memory/src/agent-framework-agent-foundry-memory-responses/uv.toml |
Enables system certificates for uv. |
responses/13-foundry-memory/src/agent-framework-agent-foundry-memory-responses/requirements.txt |
Removes compiled requirements. |
responses/13-foundry-memory/src/agent-framework-agent-foundry-memory-responses/requirements.in |
Removes pip-compile inputs. |
responses/13-foundry-memory/src/agent-framework-agent-foundry-memory-responses/pyproject.toml |
Defines pinned project dependencies. |
responses/13-foundry-memory/src/agent-framework-agent-foundry-memory-responses/hooks/postprovision.sh |
Runs provisioning through uv. |
responses/13-foundry-memory/src/agent-framework-agent-foundry-memory-responses/hooks/postprovision.ps1 |
Runs Windows provisioning through uv. |
responses/13-foundry-memory/src/agent-framework-agent-foundry-memory-responses/Dockerfile |
Builds and runs with uv. |
responses/13-foundry-memory/README.md |
Documents uv provisioning and execution. |
responses/12-foundry-skills/src/agent-framework-agent-foundry-skills-responses/uv.toml |
Enables system certificates for uv. |
responses/12-foundry-skills/src/agent-framework-agent-foundry-skills-responses/requirements.txt |
Removes compiled requirements. |
responses/12-foundry-skills/src/agent-framework-agent-foundry-skills-responses/requirements.in |
Removes pip-compile inputs. |
responses/12-foundry-skills/src/agent-framework-agent-foundry-skills-responses/pyproject.toml |
Defines pinned project dependencies. |
responses/12-foundry-skills/src/agent-framework-agent-foundry-skills-responses/Dockerfile |
Builds and runs with uv. |
responses/12-foundry-skills/README.md |
Documents uv provisioning and execution. |
responses/11-azure-search-rag/README.md |
Simplifies uv environment setup. |
responses/10-downstream-azure/README.md |
Simplifies uv environment setup. |
responses/09-declarative-customer-support/README.md |
Adds uv and Python prerequisites. |
responses/08-observability/README.md |
Simplifies uv environment setup. |
responses/07-skills/src/agent-framework-agent-with-skills-responses/uv.toml |
Enables system certificates for uv. |
responses/07-skills/src/agent-framework-agent-with-skills-responses/requirements.txt |
Removes compiled requirements. |
responses/07-skills/src/agent-framework-agent-with-skills-responses/requirements.in |
Removes pip-compile inputs. |
responses/07-skills/src/agent-framework-agent-with-skills-responses/pyproject.toml |
Defines pinned project dependencies. |
responses/07-skills/src/agent-framework-agent-with-skills-responses/Dockerfile |
Builds and runs with uv. |
responses/07-skills/README.md |
Documents uv setup and execution. |
responses/06-files/README.md |
Adds uv and Python prerequisites. |
responses/05-workflows/README.md |
Simplifies uv environment setup. |
responses/04-foundry-toolbox/README.md |
Simplifies uv environment setup. |
responses/02-tools/README.md |
Simplifies uv environment setup. |
responses/01-basic/README.md |
Simplifies uv environment setup. |
README.md |
Updates shared uv installation guidance. |
invocations/02-content-safety-guardrail/README.md |
Simplifies uv environment setup. |
invocations/01-basic/README.md |
Simplifies uv environment setup. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Shared documentation incorrectly makes Python 3.13 and uv universal and retains obsolete requirements-based Agent Framework paths.
3 open findings
Previously missed (1)
In code that hasn't changed since last review
samples/python/hosted-agents/agent-framework/README.md:212
The manual-run workflow still presents uv as optional and retains the requirements.txt setup below, although the migrated Agent Framework tree has no such manifests and every service pyproject requires Python 3.13. Remove the Python 3.10/requirements branch here as well so users are not directed to a nonexistent file.
🧠 Review effort: Balanced
This reverts commit 9a21273.
Resolve browser automation conflicts by keeping uv-based installation and adopting the GA Toolbox runtime and documentation from main. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Wei Meng (m5i-work)
previously approved these changes
Oct 19, 2026
Wei Meng (m5i-work)
approved these changes
Oct 19, 2026
Linda Li (lindazqli)
approved these changes
Oct 22, 2026
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.