Skip to content

Feature/azure ai projects/vnext - #49574

Merged
Howie Leung (howieleung) merged 106 commits into
mainfrom
feature/azure-ai-projects/vnext
Oct 16, 2026
Merged

Howie Leung (howieleung) merged 106 commits into
mainfrom
feature/azure-ai-projects/vnext

Conversation

@howieleung

Copy link
Copy Markdown
Member

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:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

Darren Cohen (dargilco) and others added 30 commits July 1, 2026 15:27
…est, add new hosted agent samples for Teams message trigger and reminder preview (#48234)

* Remove sample_agent_toolbox_skill.py per bakcned folks and Linda request, add new hosted agent samples for Teams message trigger and reminder preview

* change log
- Introduced Microsoft Copilot managed harness and toolset support for prompt agents.
- Updated CHANGELOG to reflect new features and breaking changes.
- Enhanced README and API documentation with new Copilot classes and configurations.
- Added new YAML configuration for TSP location.
- Updated tests and logging samples for compatibility with new features.
Copilot AI balanced review requested due to automatic review settings October 16, 2026 18:31

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.

🟡 Changes recommended

The Microsoft Copilot preview header is missing, and concurrent commit acknowledgments can cause telemetry to lose the latest audio transcript.

4 open findings

🧠 Review effort: Balanced

Comment thread sdk/ai/azure-ai-projects/azure/ai/projects/models/_enums.py
Copilot AI balanced review requested due to automatic review settings October 16, 2026 18:52

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.

🔵 Needs a closer look

Outstanding preview opt-in wiring and realtime telemetry correctness concerns prevent approval.

4 open findings

🧠 Review effort: Balanced

Copilot AI balanced review requested due to automatic review settings October 16, 2026 19:02

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.

🔵 Needs a closer look

The previously reported Voice Agent telemetry accuracy issues remain unresolved in the supplied code.

4 open findings

🧠 Review effort: Balanced

@github-actions

This comment has been minimized.

Copilot AI balanced review requested due to automatic review settings October 16, 2026 19:58

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.

🟡 Changes recommended

A rejected audio commit leaves stale tracking state that causes subsequent valid transcripts to be omitted from telemetry.

5 open findings

🧠 Review effort: Balanced

Copilot AI balanced review requested due to automatic review settings October 16, 2026 20:08

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.

🔵 Needs a closer look

Process-wide tracing wrappers and privacy-sensitive concurrent lifecycle handling warrant human review and runtime validation.

5 open findings

🧠 Review effort: Balanced

@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor
[Pilot] PR Pipeline Failure Analysis

What failed

Build 6969013, check python - pullrequest / Build Analyze job — pylint failed for package azure-ai-projects (exit code 20). Two sibling packages (azure-ai-agents, azure-ai-finetuningsessions) passed cleanly.

Root cause: azure/ai/projects/_patch.py has one method (around line 507–514) with:

  • C4741 (docstring-missing-return) at line 507 — docstring missing a :return: description.
  • C4742 (docstring-missing-rtype) at line 507 — docstring missing a :rtype: declaration.
  • W0107 (unnecessary-pass) at line 514 — redundant pass statement.

All three findings share one root cause (incomplete docstring plus leftover pass) in the same method.

Relevant pipeline output
************* Module azure.ai.projects._patch
azure/ai/projects/_patch.py:507: [C4741(docstring-missing-return), patch_sdk] A return doc is missing in the docstring.
azure/ai/projects/_patch.py:507: [C4742(docstring-missing-rtype), patch_sdk] A return type is missing in the docstring.
azure/ai/projects/_patch.py:514: [W0107(unnecessary-pass), patch_sdk] Unnecessary pass statement
Your code has been rated at 9.99/10 (previous run: 10.00/10, -0.01)
pylint check completed with exit code 20

Recommended next steps

  • In sdk/ai/azure-ai-projects/azure/ai/projects/_patch.py, add :return: and :rtype: entries to the docstring of the method at line 507.
  • Remove the unnecessary pass statement at line 514.
  • Re-run pylint locally (e.g. azpysdk pylint sdk/ai/azure-ai-projects) to confirm a clean score.
  • See https://aka.ms/azsdk/python/pylint-guide and https://aka.ms/ci-fix

Automated fix: Fix found, view and apply fix

Generated by Pipeline Analysis Next Steps · copilot · sonnet50 · 62.9 AIC · ⌖ 5.2 AIC · ⊞ 8.9K · ◷

Copilot AI balanced review requested due to automatic review settings October 16, 2026 21:50

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.

@howieleung
Howie Leung (howieleung) merged commit 46c0c84 into main Oct 16, 2026
20 checks passed
@howieleung
Howie Leung (howieleung) deleted the feature/azure-ai-projects/vnext branch October 16, 2026 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants