Open
Conversation
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR syncs code from the container_agent repository, bumping three agentserver packages from version 1.0.0b1 to 1.0.0b2. The changes include bug fixes, code formatting improvements, environment variable standardization, and the addition of a conversation field to response objects.
- Version bumps for azure-ai-agentserver-langgraph, azure-ai-agentserver-core, and azure-ai-agentserver-agentframework from 1.0.0b1 to 1.0.0b2
- Code formatting fixes (removing extra spaces before type/mypy ignore comments)
- Refactoring of error message formatting with new
_format_errorhelper function - Environment variable name standardization for Application Insights
- ID generation format fix in FoundryIdGenerator
- Addition of conversation field to agent framework response output
Reviewed Changes
Copilot reviewed 14 out of 17 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/agentserver/azure-ai-agentserver-langgraph/azure/ai/agentserver/langgraph/models/response_event_generators/response_output_text_event_generator.py | Fixed code formatting by removing extra spaces before mypy ignore comments |
| sdk/agentserver/azure-ai-agentserver-langgraph/azure/ai/agentserver/langgraph/models/response_event_generators/response_event_generator.py | Fixed code formatting and added trailing comma for better style consistency |
| sdk/agentserver/azure-ai-agentserver-langgraph/azure/ai/agentserver/langgraph/models/response_event_generators/response_content_part_event_generator.py | Fixed code formatting by removing extra spaces before mypy ignore comments |
| sdk/agentserver/azure-ai-agentserver-langgraph/_version.py | Version bump from 1.0.0b1 to 1.0.0b2 |
| sdk/agentserver/azure-ai-agentserver-langgraph/CHANGELOG.md | Added changelog entry for 1.0.0b2 release |
| sdk/agentserver/azure-ai-agentserver-core/azure/ai/agentserver/core/server/common/id_generator/foundry_id_generator.py | Changed ID generation format from {prefix}{entropy}{infix}{pkey} to {prefix}{infix}{pkey}{entropy} |
| sdk/agentserver/azure-ai-agentserver-core/azure/ai/agentserver/core/server/common/agent_run_context.py | Fixed code formatting by removing extra spaces before type ignore comments |
| sdk/agentserver/azure-ai-agentserver-core/azure/ai/agentserver/core/server/base.py | Added import for APPINSIGHT_CONNSTR_ENV_NAME, improved log messages, refactored error formatting with new _format_error helper function |
| sdk/agentserver/azure-ai-agentserver-core/azure/ai/agentserver/core/logger.py | Introduced APPINSIGHT_CONNSTR_ENV_NAME constant and updated references to use it |
| sdk/agentserver/azure-ai-agentserver-core/azure/ai/agentserver/core/constants.py | Removed APPLICATION_INSIGHTS_CONNECTION_STRING constant and changed ENABLE_APPLICATION_INSIGHTS_LOGGER value |
| sdk/agentserver/azure-ai-agentserver-core/_version.py | Version bump from 1.0.0b1 to 1.0.0b2 |
| sdk/agentserver/azure-ai-agentserver-core/CHANGELOG.md | Added changelog entry for 1.0.0b2 release |
| sdk/agentserver/azure-ai-agentserver-agentframework/azure/ai/agentserver/agentframework/models/agent_framework_output_streaming_converter.py | Added conversation field to response output |
| sdk/agentserver/azure-ai-agentserver-agentframework/azure/ai/agentserver/agentframework/models/agent_framework_output_non_streaming_converter.py | Fixed code formatting by removing extra spaces before type ignore comment |
| sdk/agentserver/azure-ai-agentserver-agentframework/azure/ai/agentserver/agentframework/agent_framework.py | Updated import and reference to use APPINSIGHT_CONNSTR_ENV_NAME |
| sdk/agentserver/azure-ai-agentserver-agentframework/_version.py | Version bump from 1.0.0b1 to 1.0.0b2 |
| sdk/agentserver/azure-ai-agentserver-agentframework/CHANGELOG.md | Added changelog entry for 1.0.0b2 release |
...gentserver-core/azure/ai/agentserver/core/server/common/id_generator/foundry_id_generator.py
Show resolved
Hide resolved
sdk/agentserver/azure-ai-agentserver-agentframework/CHANGELOG.md
Outdated
Show resolved
Hide resolved
sdk/agentserver/azure-ai-agentserver-core/azure/ai/agentserver/core/logger.py
Show resolved
Hide resolved
* Tool Client V1 Version * Langraph integration * Updates fixing langgraph adapter * fix build * fix cspel * fix cspell * Add ToolClient integration with Agent Framework and examples for dynamic agent creation * Made changes to return tools instead of toolclient * Address comments --------- Co-authored-by: Lu Sun <lusu@microsoft.com>
…e-sdk-for-python into lusu/agentserver-1110
* try skip apistub for langgraph * added beta classifier * fix pylint * fix pylint
* add unittests for tools * add more tests * remove unnecessary test case * move to upper level
…e-sdk-for-python into lusu/agentserver-1110
* make project endpoint optional * add UT
* Bump version to 1.0.0b10 * Revert "remove "from __future__ import annotations" as we support py>=3.10" This reverts commit 39a4b21. * fix pylint
* make conversation id optional * Moved the LangGraph checkpointer guard back to agent_run and switched it to a dedicated exception so the core base server handles it (now surfaces as server_error), with tests updated to assert the new behavior.
* init AGENTS.md * init AGENTS.md, PLANNING.md and TASK.md * Task.md template * Task.md template * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * move task.md tempalte instructions to agents.md --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ader (#44929) * use git worktree * [agentserver] Attach package metadata to OpenAIResponse.metadata + header * [agentserver] Attach package metadata to OpenAIResponse.metadata + header
…ork (#45004) * [Hosted Agents] Implement managed checkpoints feature for AgentFramework * misc: use project_endpoint instead of project_id and foundry_endpoint * misc: use get_project_endpoint * misc: refine checkpoint init logic * misc: Add a mutual-exclusion check * misc: move the check into workflow_agent_adapter
Some MCP tool servers (e.g. HuggingFace) return tool manifests where certain parameters have an empty string for their JSON Schema 'type' field. This causes a Pydantic ValidationError when the SDK tries to deserialise the ListFoundryConnectedToolsResponse, because SchemaType only accepts the six standard JSON Schema types. Changes: - SchemaProperty.type is now Optional[SchemaType] with a model_validator that coerces empty type strings to None. - FoundryToolClient._create_ai_function skips parameters whose type is None with a warning log, instead of crashing with an AttributeError.
…45041) * [Hosted Agents] Implement managed checkpoints feature for Langgraph * misc: preserves all other compile parameters * chore: refactor AgentFramework to remove managed_checkpoints flag * chore: refactor langgraph to remove managed_checkpoints flag
…d langgraph (#45064) * [Hosted Agents] Add foundry checkpoints samples for agentframework and langgraph * docs: Add README files for Foundry checkpoint samples
Co-authored-by: Declan <lixiaoli@microsoft.com>
Applies the same guard as the agentframework fix (PR #45051) to the LangGraph tool resolver. When an MCP tool manifest contains a property with an empty or unrecognised schema type, the resolver now skips that field with a warning instead of crashing with AttributeError.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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:
General Guidelines and Best Practices
Testing Guidelines