Skip to content

Conversation

@dargilco
Copy link
Member

No description provided.

@dargilco dargilco self-assigned this Oct 30, 2025
Copilot AI review requested due to automatic review settings October 30, 2025 15:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces significant changes to the Azure AI Projects SDK test infrastructure and samples, including:

  • Removal of agent and evaluation-specific tests from the main test suite
  • Addition of new test infrastructure for responses, conversations, files, and agents operations
  • Migration from get_openai_client to get_openai_client_legacy for backward compatibility
  • Addition of helper methods in test base class for client creation
  • Update of sample installation instructions to use version constraint "azure-ai-projects>=2.0.0"
  • Commenting out several SDK packages in CI configuration
  • Addition of comprehensive new sample files demonstrating responses, conversations, file operations, and various agent tools

Reviewed Changes

Copilot reviewed 148 out of 155 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
sdk/ai/ci.yml Comments out azure-ai-inference, azure-ai-agents, and azure-ai-voicelive from CI artifacts
sdk/ai/azure-ai-projects/tsp-location.yaml Updates commit hash and switches repo to azure-rest-api-specs-pr
sdk/ai/azure-ai-projects/tests/test_base.py Adds helper methods for client creation, validation functions, and new test parameters
sdk/ai/azure-ai-projects/tests/test_agents*.py Removes legacy agent tests
sdk/ai/azure-ai-projects/tests//test_.py Migrates to use helper methods and get_openai_client_legacy
sdk/ai/azure-ai-projects/samples/**/*.py Updates installation instructions and adds numerous new sample files
sdk/ai/azure-ai-projects/tests/test_data/files/test_file.jsonl Adds test data file for file operations
Files not reviewed (1)
  • eng/emitter-package-lock.json: Language not supported

@github-actions
Copy link

github-actions bot commented Oct 30, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

azure-ai-projects

dargilco and others added 12 commits October 31, 2025 06:17
* Re-emit, and update post-emitter-fixes.cmd to restore deleted files

* Remove constants related to old Evaluations
* adding responses instrumentor

* couple of pylint fixes

* pylint and cspell fixes

* adding pylint disable
* minor change to get a ci build

* pylint fix
* workflow samples

* Resolved comment

* Resolve misconfigured `dpkg` (#43777)

* image has a new issue from update. some player is leaving apt-get unconfigured, so we need to add a dpkg configure before apt-get installing anything

---------

Co-authored-by: Scott Beddall <[email protected]>
dargilco and others added 13 commits November 4, 2025 14:14
* Samples update

* Resolved comments

* Resolved comments

* rename files

* resolved comment

* Remove TODO comment regarding 'input' parameter in response creation

* update

* resolved build

* fix install

* Resolve misconfigured `dpkg` (#43777)

* image has a new issue from update. some player is leaving apt-get unconfigured, so we need to add a dpkg configure before apt-get installing anything

* Remove aiofiles dependency and update installation instructions in sample agent code

---------

Co-authored-by: Scott Beddall <[email protected]>
* Migrate samples from agentsv2-preview repo

* run black

* update

* fix

* comment out some samples
* adding tool call instrumentation

* fixing code checker issues

* moving import into test functions
* adding tracing for more conversation item types

* code format checker changes

* code checker fixes
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.

5 participants