Skip to content

Revise AI Integration section in SKILL.md#869

Open
kzu wants to merge 2 commits intogithub:mainfrom
kzu:patch-1
Open

Revise AI Integration section in SKILL.md#869
kzu wants to merge 2 commits intogithub:mainfrom
kzu:patch-1

Conversation

@kzu
Copy link

@kzu kzu commented Mar 3, 2026

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

  • Update to existing instruction, prompt, agent, plugin, skill, or workflow.

github-actions bot and others added 2 commits March 3, 2026 04:36
Updated AI integration section to reflect new libraries and practices.
@kzu kzu requested a review from aaronpowell as a code owner March 3, 2026 22:26
Copilot AI review requested due to automatic review settings March 3, 2026 22:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment on lines +60 to +61
- Use Microsoft.Extensions.AI and Microsoft.Agents.AI for AI operations
- Implement proper configuration and service registration
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
- 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

Copilot uses AI. Check for mistakes.
Comment on lines +58 to +60
## 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
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot uses AI. Check for mistakes.
@github-actions github-actions bot force-pushed the main branch 2 times, most recently from 63ca70f to fd5fc76 Compare March 3, 2026 23:05
@aaronpowell
Copy link
Contributor

PR's should target the staged branch and not contain file addons to the plugin folder.

@github-actions github-actions bot force-pushed the main branch 4 times, most recently from 1dfa971 to dafb59a Compare March 4, 2026 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants