Rename azure-ai-agentserver-github → azure-ai-agentserver-githubcopilot - #46087
Merged
Paul Van Eck (pvaneck) merged 3 commits intoApr 3, 2026
Merged
Conversation
Dan Taylor approved, Travis Wilson confirmed, Johan Stenberg recommended removing the dash to avoid deeply nested module paths. - Rename package directory and Python module (github → githubcopilot) - Update all import paths, pyproject.toml, docs, tests, CI config - Add CODEOWNERS entry for new path - Delete old azure-ai-agentserver-github directory Validated: 14/14 unit tests pass, pip install + import OK, deployed to ADC as rename-test-01 and invoked successfully. Spec PR: https://github.com/coreai-microsoft/foundrysdk_specs/pull/48 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Jonathan DeKlotz (jodeklotzms)
requested review from
a team,
Jun'an Chen (JC-386),
Kashif Khan (kashifkhan),
Laurent Mazuel (lmazuel),
lusu-msft,
Yuchao Yan (msyyc),
Scott Beddall (scbedd) and
Xiang Yan (xiangyan99)
as code owners
April 2, 2026 17:33
Copilot started reviewing on behalf of
Jonathan DeKlotz (jodeklotzms)
April 2, 2026 17:34
View session
Contributor
There was a problem hiding this comment.
Pull request overview
Renames the Agent Server GitHub Copilot adapter package/module from azure-ai-agentserver-github / azure.ai.agentserver.github to azure-ai-agentserver-githubcopilot / azure.ai.agentserver.githubcopilot, and updates CI, docs, tests, and ownership to match.
Changes:
- Update package metadata, import paths, and CI artifact configuration for the new package name.
- Update documentation and integration test agent assets to reference the renamed package/module.
- Add CODEOWNERS entry for the renamed package directory and remove old package sample/test-agent requirement references.
Reviewed changes
Copilot reviewed 19 out of 38 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/agentserver/ci.yml | Switch CI artifact entry to azure-ai-agentserver-githubcopilot. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/unit_tests/test_copilot_request_converter.py | Update unit test imports to new module path. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/TESTING.md | Rename package reference in integration test docs. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/test_agent/skills/hello/SKILL.md | Update package name referenced by the test skill. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/test_agent/requirements.txt | New test-agent deps referencing renamed package. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/test_agent/main.py | Update adapter import to new module path. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/test_agent/Dockerfile | Container build/install for integration test agent. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/test_agent/AGENTS.md | Update package name referenced by the test agent prompt. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/logs.py | Update package-root comment to renamed directory. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/invoke.py | Add invocation helper script for integration test agent. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/deploy.py | Update package-root comment to renamed directory. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/_token_cache.py | Add cached az account get-access-token helper for integration scripts. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/samples/README.md | New samples README pointing to renamed package. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/README.md | Update install/import snippets to renamed package/module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/pyproject.toml | Rename distribution + update dynamic version import + ruff isort first-party module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/MANIFEST.in | Update py.typed include path to renamed module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/LICENSE | Add license file in renamed package. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/INSTRUCTIONS.md | Update “getting started” instructions and paths for rename. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/dev_requirements.txt | Add dev requirements for local validation. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/CHANGELOG.md | Add initial changelog for renamed package. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/_version.py | Add version constant for dynamic packaging. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/_tool_discovery.py | Tool auto-discovery helper under renamed module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/_tool_acl.py | Tool ACL implementation under renamed module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/_model_cache.py | Model selection cache helper under renamed module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/_foundry_model_discovery.py | Foundry model discovery helper under renamed module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/_copilot_response_converter.py | Response/streaming conversion under renamed module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/_copilot_request_converter.py | Request/prompt + attachment conversion under renamed module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/_copilot_adapter.py | Update trace namespace string to renamed module. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/init.py | Update usage example import to renamed module path. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/init.py | Namespace package init for azure. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/init.py | Namespace package init for azure.ai. |
| sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/init.py | Namespace package init for azure.ai.agentserver. |
| sdk/agentserver/azure-ai-agentserver-github/tests/integration/test_agent/requirements.txt | Remove old package reference from legacy integration test agent requirements. |
| sdk/agentserver/azure-ai-agentserver-github/samples/README.md | Remove old package sample README. |
| .github/CODEOWNERS | Add ownership entry for the renamed package directory. |
Paul Van Eck (pvaneck)
left a comment
Paul Van Eck (pvaneck)
left a comment
Member
There was a problem hiding this comment.
Jonathan DeKlotz (@jodeklotzms) Can you also update the entry you added in .vscode/cspell.json?
The cspell word override was still targeting the old directory azure-ai-agentserver-github/** instead of the renamed azure-ai-agentserver-githubcopilot/**. This caused all 44 spelling errors in the CI check. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Jonathan DeKlotz (jodeklotzms)
requested review from
Ben Broderick Phillips (benbp),
Laia Patiño (laiapat) and
Wes Haggard (weshaggard)
as code owners
April 3, 2026 00:17
Per Paul Van Eck's review — the PRLabel comment only applies to the path immediately following it. Our githubcopilot line needs its own. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Member
|
/check-enforcer override |
Paul Van Eck (pvaneck)
approved these changes
Apr 3, 2026
Paul Van Eck (pvaneck)
deleted the
feature/agentserver-githubcopilot-rename
branch
April 3, 2026 01:26
Nikolay Kim (fafhrd91)
pushed a commit
to fafhrd91/azure-sdk-for-python
that referenced
this pull request
Apr 28, 2026
…ot (Azure#46087) * Rename azure-ai-agentserver-github → azure-ai-agentserver-githubcopilot Dan Taylor approved, Travis Wilson confirmed, Johan Stenberg recommended removing the dash to avoid deeply nested module paths. - Rename package directory and Python module (github → githubcopilot) - Update all import paths, pyproject.toml, docs, tests, CI config - Add CODEOWNERS entry for new path - Delete old azure-ai-agentserver-github directory Validated: 14/14 unit tests pass, pip install + import OK, deployed to ADC as rename-test-01 and invoked successfully. Spec PR: https://github.com/coreai-microsoft/foundrysdk_specs/pull/48 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix: update cspell override glob for renamed package The cspell word override was still targeting the old directory azure-ai-agentserver-github/** instead of the renamed azure-ai-agentserver-githubcopilot/**. This caused all 44 spelling errors in the CI check. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix: add PRLabel comment for githubcopilot CODEOWNERS entry Per Paul Van Eck's review — the PRLabel comment only applies to the path immediately following it. Our githubcopilot line needs its own. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.
Summary
azure-ai-agentserver-githubtoazure-ai-agentserver-githubcopilotazure.ai.agentserver.github_copilot→azure.ai.agentserver.githubcopilot)@jodeklotzms @pradeepkintali @tendau @ganeshyb @code-vicarazure-ai-agentserver-githubdirectory (per Dan: "delete the old spec in the same PR")Changes
azure-ai-agentserver-github/→azure-ai-agentserver-githubcopilot/azure.ai.agentserver.github→azure.ai.agentserver.githubcopilotci.yml)Naming convention
langgraphazure-ai-agentserver-langgraphgithub-copilot-sdkazure-ai-agentserver-githubcopilotValidation
pip install -e .+from azure.ai.agentserver.githubcopilot import GitHubCopilotAdapter— OKrename-test-01, invoked successfullyRelated
🤖 Generated with Claude Code