Python: Update agentserver to x.1.0b1 - #7621
Conversation
There was a problem hiding this comment.
Pull request overview
Updates Python Foundry hosting for AgentServer 2.1, durable local storage, stable conversation identifiers, and custom Cosmos DB session storage.
Changes:
- Upgrades AgentServer dependencies and storage providers.
- Uses conversation-chain IDs for sessions and workflow checkpoints.
- Adds a custom Cosmos DB storage sample and updates tests/docs.
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
custom_storage/requirements.txt |
Adds sample dependencies. |
custom_storage/README.md |
Documents custom storage usage. |
custom_storage/main.py |
Implements Cosmos-backed sessions. |
custom_storage/Dockerfile |
Packages the sample. |
custom_storage/.env.example |
Lists required configuration. |
custom_storage/.dockerignore |
Excludes local artifacts. |
tests/test_state_store.py |
Updates provider expectations. |
tests/test_responses.py |
Replaces removed in-memory approval store. |
foundry_hosting/README.md |
Documents durable local persistence. |
foundry_hosting/pyproject.toml |
Updates AgentServer dependencies. |
_state_store.py |
Uses Foundry stores in all environments. |
_responses.py |
Revises session and checkpoint scoping. |
_request_context.py |
Removes obsolete path validation. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
MAF Automated Review — Iteration 2
Result: Findings reported
Scope: 16 net-new commit(s): 30996433ac86, 5e52c6a7183a, 27d82b1567a6, 56d13bce4eb2, d17a81e852cf, 171693f93da6, 3a5d00be5473, 35c6b880f737, 3aadac53c80c, 322101142718, 7cfa905486ac, e926ad2859fd, 6d25fb1e9c68, e9ed52e3916b, cf0cf904b3e2, 012d43d14717
Model: gpt-5.6-sol
Overview
The PR updates AgentServer dependencies and threads request-scoped platform context through Foundry-backed session, checkpoint, and approval stores, including explicit user and call identity. Those are useful isolation guardrails, and the new storage tests exercise call-ID forwarding. The remaining risk is that the public custom-store interface changes incompatibly and four state-store tests were not updated for the new user-ID argument.
Reviewed the supplied incremental change set across correctness, security/reliability, architecture, and failure behavior.
2 verified findings remained after source verification (2 medium) across 1 file. Details are attached to the affected lines below.
Affected areas: python/packages/foundry_hosting/agent_framework_foundry_hosting/_state_store.py
Python Test Coverage Report •
Python Unit Test Overview
|
||||||||||||||||||||||||||||||||||||||||||||||||||
Motivation & Context
Update FHA SDK dependencies to
x.1.0b1, which introduces the latest local state store feature.Description & Review Guide
azure-ai-agentserver-coreto 2.1.0b1,azure-ai-agentserver-responsesto 2.1.0b1, andazure-ai-agentserver-invocationsto 1.1.0b1.Contribution Checklist
breaking changelabel (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.