Skip to content

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 adds support for optional user input environment variables to three agent tool samples (Agent-to-Agent, Fabric, and Bing Custom Search), allowing automated testing without manual user prompts.

Changes:

  • Added three new optional user input environment variables for agent samples
  • Updated sample documentation to describe the new optional environment variables
  • Modified Fabric and Bing Custom Search samples to check environment variables before prompting

Reviewed changes

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

Show a summary per file
File Description
sdk/ai/azure-ai-projects/tests/test_base.py Added fabric_user_input, a2a_user_input, and bing_custom_user_input parameters to test configuration
sdk/ai/azure-ai-projects/tests/conftest.py Added blank line (formatting change)
sdk/ai/azure-ai-projects/samples/agents/tools/sample_agent_to_agent.py Added documentation for optional A2A_USER_INPUT environment variable
sdk/ai/azure-ai-projects/samples/agents/tools/sample_agent_fabric.py Added documentation and implementation to use FABRIC_SEARCH_USER_INPUT environment variable
sdk/ai/azure-ai-projects/samples/agents/tools/sample_agent_bing_custom_search.py Added documentation and implementation to use BING_CUSTOM_USER_INPUT environment variable
sdk/ai/azure-ai-projects/.env.template Added FABRIC_SEARCH_USER_INPUT, BING_CUSTOM_USER_INPUT, and A2A_USER_INPUT template entries

Copy link
Contributor

Copilot AI commented Jan 13, 2026

@howieleung I've opened a new pull request, #44642, to work on those changes. Once the pull request is ready, I'll request review from you.

@howieleung howieleung enabled auto-merge (squash) January 13, 2026 21:18
@scbedd
Copy link
Member

scbedd commented Jan 15, 2026

/azp run python - pullrequest

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@howieleung
Copy link
Member Author

/azp run python - pullrequest

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@howieleung
Copy link
Member Author

/azp run python - pullrequest

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@scbedd
Copy link
Member

scbedd commented Jan 16, 2026

/azp run python - pullrequest

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@howieleung howieleung merged commit edfb2bc into main Jan 16, 2026
20 checks passed
@howieleung howieleung deleted the howie/user-input branch January 16, 2026 01:17
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.

4 participants