Remove deprecated multi-agent workflow samples and update evaluation … - #48795
Open
Howie Leung (howieleung) wants to merge 1 commit into
Open
Remove deprecated multi-agent workflow samples and update evaluation …#48795Howie Leung (howieleung) wants to merge 1 commit into
Howie Leung (howieleung) wants to merge 1 commit into
Conversation
|
Azure Pipelines: Successfully started running 1 pipeline(s). 9 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
This comment has been minimized.
This comment has been minimized.
Member
Howie Leung (howieleung)
force-pushed
the
howie/reemit
branch
from
August 31, 2026 22:00
ab6b8d0 to
3dccafe
Compare
Member
Author
Yes |
This comment has been minimized.
This comment has been minimized.
Contributor
[Pilot] PR Pipeline Failure AnalysisWhat failedBuild: 6769789 (public) — failed Tests failing on all platforms (macOS 3.11, Ubuntu 24.04 3.10/3.10-coverage/3.13/3.14, Windows 2022 3.12 — both
Relevant pipeline outputRoot causeThis PR introduced a new Recommended next steps
Automated fix: Fix found, view and apply fix
|
Glenn Harper (glharper)
added a commit
to Azure/azure-sdk-for-js
that referenced
this pull request
Sep 1, 2026
### Packages impacted by this PR `@azure/ai-projects` ### Issues associated with this PR Follow-up to the Agent Insights API regeneration in #39797 and the cross-language customization in Azure/azure-sdk-for-python#48795. ### Describe the problem that is addressed by this PR Agent Insights `createRun` is emitted as an LRO that resolves to `AgentInsightRunResult`. That terminal result omits the initial `AgentInsightRun.id`, even though callers need the ID for the paired `getRun` and `cancelRun` operations. The feature branch already routes this operation through `JobPoller`, following the job LRO work in #39237 and #39455, but that exposes an Agent Insights run as `operationState.jobId`. This PR adds the run-specific public shape while keeping the shared implementation generic: - Adds `RunOperationState<TResult>` and `RunPoller<TResult>`, exposing `operationState.runId` after `submitted()` resolves. - Changes `project.beta.agentInsightMonitors.createRun` to return `RunPoller<AgentInsightRunResult>`. - Refactors `JobPoller` and `RunPoller` through one identity-preserving wrapper covering `operationState`, `poll()`, `onProgress()`, serialization, and resume. - Adds focused unit coverage for run IDs, serialized poller restoration, resource-location fallback, and existing `JobPoller` behavior. - Records the customization in the package's post-emitter workaround documentation and skill so regeneration does not remove it. The Agent Insights API is new in the unreleased `2.6.0` feature branch, so replacing its temporary `JobPoller` return type does not break a published API. ### Are there test cases added in this PR? _(If not, why?)_ Yes. `test/public/pollingHelpers.spec.ts` covers the new run poller and regression coverage for job pollers. ### Provide a list of related PRs _(if any)_ - Azure/azure-sdk-for-python#48795 - #39237 - #39455 - #39797 ### Validation - `npm run check-format` - ESLint on all changed TypeScript files - `tsc -p tsconfig.src.json --noEmit` - `tsc -p tsconfig.test.json --noEmit` - `TEST_MODE=playback npm run test:node` - `TEST_MODE=playback npm run test:browser` - `turbo build --filter=@azure/ai-projects... --token 1` - `check-package-version ai -packages azure-ai-projects` Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: da906d50-380f-43e0-bfe1-a5d82afd7ac6
- Introduced `BetaAgentInsightMonitorsOperations` for managing agent insight monitors. - Updated `BetaOperations` to include the new agent insight monitors operations. - Removed outdated sample scripts for multi-agent workflows and MCP approval. - Updated evaluation samples to reflect changes in Azure SDK and improve functionality. - Adjusted role assignment parameters in evaluation samples for better clarity and structure. - Updated `tsp-location.yaml` to include new directories for agent insights and voice agents.
Howie Leung (howieleung)
force-pushed
the
howie/reemit
branch
from
September 2, 2026 02:55
4ec6933 to
7eabb9c
Compare
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.
…scripts with new role assignment parameters. Update YAML configuration for additional directories and commit reference.
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new API spec, a link to the pull request containing these API spec changes should be included above.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines