Merged
Conversation
Merged
heyitsaamir
force-pushed
the
aamirj/addCardsTest
branch
from
b145223 to
c2c004d
Compare
heyitsaamir
force-pushed
the
aamirj/aitests
branch
3 times, most recently
from
cb5246e to
89d0540
Compare
heyitsaamir
added a commit
that referenced
this pull request
Sep 12, 2025
- Tests the cards package out. Found some bugs in serialization - Adds a cards test - Made sure these things work from the snippets that are in the docs #### PR Dependency Tree * **PR #150** 👈 * **PR #151** This tree was auto-generated by [Charcoal](https://github.com/danerwilliams/charcoal)
heyitsaamir
force-pushed
the
aamirj/aitests
branch
from
89d0540 to
8e95605
Compare
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds comprehensive AI testing capabilities to the Microsoft Teams AI Library for Python, replacing a simple echo bot with a full-featured AI test application. It also extends the core AI functionality by allowing instructions to be passed as simple strings instead of requiring SystemMessage objects.
- Comprehensive AI test application with various AI scenarios including function calling, streaming, citations, and feedback
- Enhanced Agent and ChatPrompt APIs to accept string instructions alongside SystemMessage objects
- Modular handler architecture with separate modules for different AI functionality types
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/ai-test/src/main.py | Complete rewrite from simple echo to comprehensive AI test with pattern-based command handlers |
| tests/ai-test/src/handlers/*.py | New modular handlers for function calling, memory management, citations, plugins, and feedback |
| tests/ai-test/README.md | New comprehensive documentation for AI test scenarios and usage |
| packages/ai/src/microsoft/teams/ai/agent.py | Enhanced to accept string instructions parameter |
| packages/ai/src/microsoft/teams/ai/chat_prompt.py | Enhanced to convert string instructions to SystemMessage objects |
| packages/openai/src/microsoft/teams/openai/completions_model.py | Added error logging for function execution failures |
| packages/apps/src/microsoft/teams/apps/http_plugin.py | Fixed return value handling in HTTP plugin |
Comments suppressed due to low confidence (1)
tests/ai-test/src/handlers/feedback_management.py:1
- This long string formatting expression should be broken into multiple lines for better readability instead of using a noqa comment to suppress the line length warning.
"""
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
1 participant
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.
Also add the ability for instructions to be simple strings
PR Dependency Tree
This tree was auto-generated by Charcoal