Skip to content

Remove unused SemanticKernel.Agents.Core dependency from Tooling Core #200

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.

Sign up for GitHub

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

Jump to bottom
Merged
flaviocdc merged 1 commit into from
Feb 9, 2026
Merged

Remove unused SemanticKernel.Agents.Core dependency from Tooling Core #200

flaviocdc merged 1 commit into from
Feb 9, 2026

Conversation

@flaviocdc
Copy link
Member

@flaviocdc flaviocdc commented Feb 9, 2026

Summary

  • Removes the Microsoft.SemanticKernel.Agents.Core package reference from Microsoft.Agents.A365.Tooling.csproj
  • The Tooling Core package does not directly use Semantic Kernel Agents — this dependency belongs in the framework-specific extension packages (e.g., Tooling.Extensions.SemanticKernel), not in Core
  • Reduces unnecessary transitive dependencies for consumers of the Core Tooling package

Test plan

  • Solution builds with zero warnings and zero errors
  • All 550 tests pass

🤖 Generated with Claude Code

@flaviocdc @claude
Remove unused Microsoft.SemanticKernel.Agents.Core dependency from To…
a33c1e4 
…oling Core

The Tooling Core package does not directly use Semantic Kernel Agents. This
dependency belongs in the framework-specific extension packages, not the core.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@flaviocdc flaviocdc requested a review from a team as a code owner February 9, 2026 16:33
Copilot AI reviewed Feb 9, 2026
View reviewed changes
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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@flaviocdc flaviocdc enabled auto-merge (squash) February 9, 2026 17:00
@flaviocdc flaviocdc merged commit 522f5a5 into main Feb 9, 2026
5 of 8 checks passed
@flaviocdc flaviocdc deleted the users/flcosta/remove-sk-agents-core-from-tooling branch February 9, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@pontemonti pontemonti pontemonti approved these changes

@rahuldevikar761 rahuldevikar761 rahuldevikar761 approved these changes

@mrunalhirve128 mrunalhirve128 Awaiting requested review from mrunalhirve128 mrunalhirve128 is a code owner automatically assigned from microsoft/agent365-tooling-approvers

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@flaviocdc @pontemonti @rahuldevikar761