Skip to content

Conversation

@lusu-msft
Copy link
Member

@lusu-msft lusu-msft commented Nov 7, 2025

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.

Copilot AI review requested due to automatic review settings November 7, 2025 00:37
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 a new Azure AI Agent Server Core package (azure-ai-agentserver-core) with comprehensive CI/CD setup, test infrastructure, sample applications, and core server implementation. The package provides a framework for building custom agent servers with support for streaming responses, OpenTelemetry tracing, and integration with Azure services.

Key Changes

  • New agent server framework with FoundryCBAgent base class for custom agent implementations
  • CI/CD pipeline configuration for automated testing and analysis
  • Test infrastructure with fixtures for multiple sample scenarios (mock agents, MCP integration, bilingual planner)
  • Sample applications demonstrating various agent patterns (mock agents, MCP tools, multi-agent workflows)

Reviewed Changes

Copilot reviewed 51 out of 53 changed files in this pull request and generated no comments.

Show a summary per file
File Description
sdk/agentserver/ci.yml CI/CD pipeline configuration with triggers and test matrix
sdk/agentserver/tests.yml Weekly analysis pipeline configuration
sdk/agentserver/platform-matrix.json Platform test matrix with PortAudio requirement
sdk/agentserver/azure-ai-agentserver-core/tests/test_custom.py Test suite for custom agent samples
sdk/agentserver/azure-ai-agentserver-core/tests/conftest.py Pytest configuration with test client fixtures
sdk/agentserver/azure-ai-agentserver-core/tests/env-template Environment variable documentation for tests
sdk/agentserver/azure-ai-agentserver-core/samples/*/ Sample implementations (mock, MCP, bilingual planner)
sdk/agentserver/azure-ai-agentserver-core/azure/ai/agentserver/core/server/base.py Core server implementation with streaming support
sdk/agentserver/azure-ai-agentserver-core/azure/ai/agentserver/core/server/common/ Agent context and ID generation utilities
sdk/agentserver/azure-ai-agentserver-core/pyproject.toml Package metadata and dependencies
Comments suppressed due to low confidence (2)

sdk/agentserver/ci.yml:1

  • The ServiceDirectory parameter is set to ai but should be agentserver to match the actual service directory structure. This mismatch could cause the CI pipeline to look in the wrong directory for tests and artifacts.
    sdk/agentserver/azure-ai-agentserver-core/samples/bilingual_weekend_planner/main.py:1
  • Corrected spelling of 'ANSII' to 'ANSI'. ANSI is the correct acronym for American National Standards Institute. The attack strategy should be ANSI_ATTACK.

@lusu-msft lusu-msft merged commit 9c8a278 into main Nov 7, 2025
30 checks passed
@lusu-msft lusu-msft deleted the lusu/agentserver-core-1106 branch November 7, 2025 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants