Merge feature/azure-ai-projects/2.3.0 in Main#47500
Open
dargilco wants to merge 45 commits into
Open
Conversation
* sample update routines * Add test for routines samples and update sample skipping logic
* sample update routines * Add test for routines samples and update sample skipping logic
…Azure/azure-sdk-for-python into feature/azure-ai-projects/2.3.0
* sample update routines * Add test for routines samples and update sample skipping logic
…Azure/azure-sdk-for-python into feature/azure-ai-projects/2.3.0
dargilco
requested review from
ChenxiJiang333,
bobogogo1990,
catalinaperalta,
glharper,
howieleung,
iscai-msft,
kingernupur,
msyyc,
nick863,
trangevi and
trrwilson
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
This PR appears to merge the feature/azure-ai-projects/2.3.0 work into main, updating the azure-ai-projects package to the 2.3.0 (Unreleased) state with new TypeSpec source locations, updated generated SDK surface (notably Agent Optimization moving to V2 preview), and refreshed samples/tests/docs.
Changes:
- Update TypeSpec input locations (
tsp-location.yaml) and refresh generated SDK models/operations for the 2.3.0 API surface (including new tool/model types and Agent Optimization V2 preview wiring). - Refresh samples + sample test routing for routines and hosted agents; add a new routines dispatch sample.
- Update changelog/docs and adjust emitter/post-processing scripts and repo-level emitter dependency pins.
Reviewed changes
Copilot reviewed 24 out of 26 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/ai/azure-ai-projects/tsp-location.yaml.saved | Removes the previously saved TypeSpec location metadata file. |
| sdk/ai/azure-ai-projects/tsp-location.yaml | Adds the new TypeSpec directory/commit and a large set of additionalDirectories. |
| sdk/ai/azure-ai-projects/tests/samples/test_samples.py | Restructures sample test parametrization; adds/adjusts skips for routines samples. |
| sdk/ai/azure-ai-projects/tests/redteams/test_redteams.py | Removes a snippet marker line from the red teams test. |
| sdk/ai/azure-ai-projects/tests/redteams/test_redteams_async.py | Removes a snippet marker line from the async red teams test. |
| sdk/ai/azure-ai-projects/tests/foundry_features_header/foundry_features_header_test_base.py | Updates expected Foundry feature header for AgentsOptimization from V1Preview to V2Preview. |
| sdk/ai/azure-ai-projects/samples/routines/sample_routines_with_timer_trigger.py | Updates routine sample client construction (removes allow_preview=True). |
| sdk/ai/azure-ai-projects/samples/routines/sample_routines_with_schedule_trigger.py | Updates routine sample client construction (removes allow_preview=True). |
| sdk/ai/azure-ai-projects/samples/routines/sample_routines_with_dispatch.py | Adds a new sample demonstrating manual routine dispatch and polling run history. |
| sdk/ai/azure-ai-projects/samples/routines/sample_routines_crud.py | Updates routine CRUD sample client construction (removes allow_preview=True). |
| sdk/ai/azure-ai-projects/post-emitter-fixes.cmd | Adjusts post-emitter fix script; removes a large PowerShell rewrite block and makes black invocation strict. |
| sdk/ai/azure-ai-projects/docs/subclients.md | Updates method counts and annotates hand-written methods with *. |
| sdk/ai/azure-ai-projects/CHANGELOG.md | Adds 2.3.0 breaking change notes (beta optimization rewrite) and new sample bullets. |
| sdk/ai/azure-ai-projects/azure/ai/projects/models/_patch.py | Switches default opt-in key from AgentsOptimization V1 preview to V2 preview. |
| sdk/ai/azure-ai-projects/azure/ai/projects/models/_models.py | Regenerates models: adds/removes optimization-related types, adds new tool params/types, and updates various model fields/signatures. |
| sdk/ai/azure-ai-projects/azure/ai/projects/models/_enums.py | Adds new enums (OptimizationDatasetInputType, SearchContentType, ToolSearchExecutionType, etc.) and updates existing ones. |
| sdk/ai/azure-ai-projects/azure/ai/projects/models/init.py | Updates public exports to match regenerated model surface (adds/removes many types). |
| sdk/ai/azure-ai-projects/azure/ai/projects/_utils/utils.py | Updates multipart normalization to enforce filename/content-type handling and adjusts multipart inclusion behavior. |
| sdk/ai/azure-ai-projects/azure/ai/projects/_types.py | Adds a RoutineRunStatus alias used by regenerated models. |
| sdk/ai/azure-ai-projects/assets.json | Updates the assets tag pointer. |
| sdk/ai/azure-ai-projects/apiview-properties.json | Updates ApiView mapping metadata (but currently contains unresolved merge conflict markers). |
| sdk/ai/azure-ai-projects/.github/skills/README.md | Updates skill usage docs and adds a Skills section describing package skills. |
| sdk/ai/azure-ai-projects/.github/skills/azure-ai-projects-update-changelog/SKILL.md | Adds a new skill definition for updating the changelog via API comparisons. |
| sdk/ai/azure-ai-projects/.github/skills/azure-ai-projects-emit-from-typespec/SKILL.md | Removes the embedded changelog update step and renumbers subsequent steps. |
| sdk/ai/azure-ai-projects/azure/ai/projects/operations/_operations.py | Generated sync operations file appears to contain unresolved merge conflict markers. |
| sdk/ai/azure-ai-projects/azure/ai/projects/aio/operations/_operations.py | Generated async operations file appears to contain unresolved merge conflict markers. |
| eng/emitter-package.json | Updates emitter dependency pins (but currently contains unresolved merge conflict markers). |
| eng/emitter-package-lock.json | Updates emitter lockfile (but currently contains unresolved merge conflict markers). |
Comments suppressed due to low confidence (1)
sdk/ai/azure-ai-projects/azure/ai/projects/_utils/utils.py:52
_normalize_multipart_file_entryreturns tuple entries unchanged, which means callers that already pass a 2-tuple(filename, content)will still go through azure-core’s multipart formatting path without an explicit content-type. Since this helper is explicitly trying to preserve file-part semantics (and avoid server-side rejection), it should also normalize 2-tuples into 3-tuples with a binary content-type.
if isinstance(entry, tuple):
return entry
Comment on lines
+60
to
+66
| # Return a 3-tuple with an explicit "application/octet-stream" content type. | ||
| # A 2-tuple (filename, content) would leave the part's Content-Type unset, and | ||
| # the sdk core library only defaults to "application/octet-stream" for bare | ||
| # (non-tuple) values - a tuple bypasses that default and falls back to the | ||
| # HTTP "text/plain" default instead. Setting it explicitly preserves the | ||
| # pre-existing behavior for bare bytes/IO across all transports. | ||
| return (filename, entry, "application/octet-stream") |
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.