Conversation
Updated AI integration section to reflect new libraries and practices.
There was a problem hiding this comment.
Pull request overview
Updates the .NET best-practices skill’s AI guidance to reflect a move away from Semantic Kernel toward newer Microsoft AI libraries.
Changes:
- Renames the “Semantic Kernel & AI Integration” section to “AI Integration”.
- Replaces Semantic Kernel guidance with
Microsoft.Extensions.AI+Microsoft.Agents.AI. - Simplifies wording around configuration/service registration.
| - Use Microsoft.Extensions.AI and Microsoft.Agents.AI for AI operations | ||
| - Implement proper configuration and service registration |
There was a problem hiding this comment.
There is another copy of this skill at plugins/csharp-dotnet-development/skills/dotnet-best-practices/SKILL.md that still references Semantic Kernel. If the intent is to move away from Semantic Kernel, this change should be mirrored there as well (or the plugin copy should be removed/explicitly documented as intentionally different) to avoid conflicting guidance.
| - Use Microsoft.Extensions.AI and Microsoft.Agents.AI for AI operations | |
| - Implement proper configuration and service registration | |
| - Use Microsoft.Extensions.AI and Microsoft.Agents.AI for AI operations (this skill is intended for the non–Semantic Kernel stack) | |
| - Implement proper configuration and service registration for these AI services |
| ## AI Integration | ||
|
|
||
| - Use Microsoft.SemanticKernel for AI operations | ||
| - Implement proper kernel configuration and service registration | ||
| - Use Microsoft.Extensions.AI and Microsoft.Agents.AI for AI operations |
There was a problem hiding this comment.
The PR description suggests "drop SK" broadly, but this PR only updates this one SKILL.md and leaves other Semantic Kernel guidance in-repo (e.g., agents/semantic-kernel-*.agent.md and the plugin copy of this skill). Either narrow the PR description/scope, or include the additional updates needed to match the stated intent.
63ca70f to
fd5fc76
Compare
|
PR's should target the |
f657f97 to
0c4deb1
Compare
4ffdde1 to
9ef5453
Compare
Updated AI integration section to reflect new libraries and practices.
Description
Time to drop SK now that MS.A.AI is RC2
Type of Contribution