Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -60,11 +60,12 @@ dev-dependencies = [
"langchain-openai",
]

# Override semantic-kernel's azure-ai-projects constraint to allow 2.x
# semantic-kernel 1.39.3 constrains azure-ai-projects~=1.0.0b12 but this repo needs >=2.0.0b1
# The functionality is compatible; only the version constraint was tightened
# azure-ai-projects is capped at <2.0.0 because:
# 1. semantic-kernel>=1.39.3 requires azure-ai-projects~=1.0.0b12 (<2.0.0)
# 2. tooling-extensions-azureaifoundry uses the 1.x Agents API (project_client.agents.update_agent)
# which was removed in azure-ai-projects 2.x
override-dependencies = [
"azure-ai-projects >= 2.0.0b1",
"azure-ai-projects>=1.0.0b12,<2.0.0",
Copy link

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

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

The new requirement strings omit the spaces used by the other entries in this file (e.g., "azure-ai-agents >= ..." vs "azure-ai-projects>=..."). For consistency/readability, consider using the same spacing convention for azure-ai-projects in both override-dependencies and constraint-dependencies.

Suggested change
"azure-ai-projects>=1.0.0b12,<2.0.0",
"azure-ai-projects >= 1.0.0b12, <2.0.0",

Copilot uses AI. Check for mistakes.
]
Comment on lines +63 to 69
Copy link

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

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

The PR title/description says the pin is in libraries/microsoft-agents-a365-tooling-extensions-azureaifoundry/pyproject.toml, but the actual pin is being applied globally in the root pyproject.toml via [tool.uv].override-dependencies / constraint-dependencies. Please align the PR metadata with the implementation (or change the implementation to match the described approach). Note: adding version specifiers directly to package pyproject.toml deps would also conflict with the repo’s verify-constraints checks, so if the root-level pin is intentional, it should be called out explicitly in the PR description.

Copilot uses AI. Check for mistakes.

# Centralized version constraints for all external dependencies
Expand All @@ -80,7 +81,7 @@ constraint-dependencies = [

# --- Azure Services ---
"azure-ai-agents >= 1.0.0b251001",
"azure-ai-projects >= 2.0.0b1",
"azure-ai-projects>=1.0.0b12,<2.0.0",
"azure-identity >= 1.12.0",
Comment on lines 82 to 85
Copy link

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

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

uv.lock currently still records azure-ai-projects >= 2.0.0b1 in the manifest constraints/overrides. Since this PR changes the root constraint-dependencies to <2.0.0, consider regenerating and committing uv.lock so local uv sync --locked users (and anyone inspecting the lock) don’t get a stale 2.x resolution.

Copilot uses AI. Check for mistakes.
"azure-monitor-ingestion >= 1.0.0",
"azure-monitor-opentelemetry-exporter >= 1.0.0b39",
Expand Down
Loading