Chat demo #848
Merged
Chat demo #848
Conversation
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
There was a problem hiding this comment.
Pull Request Overview
This pull request introduces a complete AI chat demo application featuring real-time multi-room chat with AI assistance. The system supports both local development and Azure deployment with minimal configuration changes.
Key changes:
- Complete React + TypeScript frontend with real-time messaging and markdown support
- Python Flask backend with Azure Web PubSub integration and configurable transport/storage modes
- Full Azure deployment infrastructure using Bicep templates with managed identity and RBAC
- Comprehensive test coverage for both frontend and backend components
Reviewed Changes
Copilot reviewed 83 out of 89 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
samples/ai/chat-demo/python_server/ |
Complete Python backend with Flask, WebSocket support, AI integration, and Azure services |
samples/ai/chat-demo/client/ |
React TypeScript frontend with real-time messaging, room management, and markdown rendering |
samples/ai/chat-demo/infra/ |
Azure infrastructure as code using Bicep for App Service, Web PubSub, and Storage |
samples/ai/chat-demo/docs/ADVANCED.md |
Comprehensive architecture documentation and deployment guide |
samples/ai/chat-demo/azure.yaml |
Azure Developer CLI configuration for automated deployment |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
JialinXin
approved these changes
Sep 28, 2025
Fixing AI generated logic Lots of bugs... Clean not used files Fix server, adding logger Kind of working, remaining work: logger, cacellation, user/conn Add connect hook, user logic User to be set through url too, and simply client logic Fix issue that user should receive the message sent to themselves simply client, a dd a message reducer update wording add rooms sidebar Async handler UPdate readme and remove user related logic Update readme and script temp temp2 Fix the client bug Minor updates Update to include history when sending to ai, and add azure related skeleton WebPubSub works when running locally with tunnel tool Naming refactor Simplify the client side code Cleanup server a bit and it is working Move files Refactor temp Webpage works but negotiate 500 Now works, one last thing is the blob -> table Changing to azure table works Fix some bugs and add ut Fix webpubsub bug when send to group, not need to stringify A clean room management approach using rest api Add more tests Fix package failure and add more tests
azd and local run works
refactor room store, transport, and rest api Improve code and test
xingsy97
approved these changes
Oct 11, 2025
vwxyzh
approved these changes
Oct 13, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
No description provided.