-
Notifications
You must be signed in to change notification settings - Fork 75
adding Retrieval Bot sample #166
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a new Retrieval Bot sample for the Build Conference 2025, including all necessary solution, project, configuration, plugin, and controller files to launch a .NET web service bot that integrates with Microsoft Graph and Semantic Kernel. Key changes include:
- Addition of multiple solution and project files along with configuration files (appsettings.json, .gitignore, and manifest.json)
- Implementation of various plugins (e.g., AdaptiveCard, SendMail, BuildRetrieval) and an agent class that leverages them
- Setup of authentication and bot controllers for handling messages
Reviewed Changes
Copilot reviewed 124 out of 124 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
Microsoft.Agents.M365Copilot.sln | Adds new solution to support the Retrieval Bot sample |
.gitignore | Includes standard Visual Studio exclusions for the new project |
appsettings.json | Configures authentication, token validation, and AI service parameters for the bot |
RetrievalBot.sln & RetrievalBot.csproj | Define the solution and project structure; note duplicate package references |
RetrievalBot.cs | Implements the core bot functionality using the Retrieval Agent |
Various Plugins & Controllers | Introduce functionality for adaptive cards, email sending, contact retrieval, etc. |
AspNetExtentions.cs & manifest.json | Provide authentication setup and app manifest for Teams integration |
Comments suppressed due to low confidence (1)
samples/complex/RetrievalBot/dotnet/RetrievalBot/RetrievalBot.cs:30
- [nitpick] The variable name 'weatherAgent' is misleading since the agent handles retrieval tasks; consider renaming it to 'retrievalAgent' for clarity.
RetrievalAgent weatherAgent = new RetrievalAgent(_kernel, this);
"Scopes": [ | ||
"https://api.botframework.com/.default" | ||
], | ||
"TenantId": "" // This is the Teannt ID used for the Connection. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment contains a spelling error ('Teannt'); consider updating it to 'Tenant ID used for the Connection.'
"TenantId": "" // This is the Teannt ID used for the Connection. | |
"TenantId": "" // This is the Tenant ID used for the Connection. |
Copilot uses AI. Check for mistakes.
<PackageReference Include="Microsoft.IdentityModel.Validators" Version="8.7.0" /> | ||
</ItemGroup> | ||
<ItemGroup> | ||
<PackageReference Include="Azure.Identity" Version="1.13.2" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The 'Azure.Identity' package is referenced more than once across different ItemGroup sections; consider removing the duplicate to simplify project maintenance.
<PackageReference Include="Azure.Identity" Version="1.13.2" /> |
Copilot uses AI. Check for mistakes.
Added the Retrieval Bot sample that will work for Build.