Skip to content

upload code files and along with code interpreter for LLM analysis#45404

Merged
howieleung merged 3 commits intomainfrom
howie/recording-6
Feb 26, 2026
Merged

upload code files and along with code interpreter for LLM analysis#45404
howieleung merged 3 commits intomainfrom
howie/recording-6

Conversation

@howieleung
Copy link
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.

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 updates the azure-ai-projects sample test harness to optionally upload the sample’s Python source files so LLM validation can use the Code Interpreter tool for deeper analysis, alongside a few related test/sample sanitization tweaks.

Changes:

  • Add logic in the sample test executor to discover local imported Python files and (in live mode) upload them for Code Interpreter–enabled responses.create validation; also delete uploaded files after validation.
  • Update sanitization/config values used by tests (sanitized model deployment name, assets tag).
  • Minor sample prompt text adjustment.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
sdk/ai/azure-ai-projects/tests/test_base.py Updates sanitized model deployment name used by test env var loader.
sdk/ai/azure-ai-projects/tests/samples/sample_executor.py Adds local-module file discovery + optional Code Interpreter file upload/delete around LLM validation requests.
sdk/ai/azure-ai-projects/samples/agents/tools/sample_agent_openapi.py Tweaks the example prompt text used for OpenAPI tool sample.
sdk/ai/azure-ai-projects/assets.json Bumps assets tag to point at updated recorded assets.

@Azure Azure deleted a comment from Copilot AI Feb 26, 2026
@Azure Azure deleted a comment from Copilot AI Feb 26, 2026
@howieleung howieleung merged commit 332fa94 into main Feb 26, 2026
20 checks passed
@howieleung howieleung deleted the howie/recording-6 branch February 26, 2026 16:24
howieleung added a commit that referenced this pull request Feb 26, 2026
…45404)

* upload code files and along with code interpreter for LLM analysis

* fix recording

* fix fine tuning recording sanitization
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.

3 participants