Skip to content

DI Refactor: Azure.Mcp.Tools.AzureIsv#1902

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/refactor-azureisv-dependency-injection
Draft

DI Refactor: Azure.Mcp.Tools.AzureIsv#1902
Copilot wants to merge 3 commits intomainfrom
copilot/refactor-azureisv-dependency-injection

Conversation

Copy link
Contributor

Copilot AI commented Mar 2, 2026

  • Inject IDatadogService via constructor in MonitoredResourcesListCommand and remove context.GetService<IDatadogService>() call in ExecuteAsync
  • Update MonitoredResourcesListCommandTests to pass service directly to command constructor (no service provider needed)
  • Add changelog entry with PR number and timestamp-based filename
Original prompt

This section details on the original issue you should resolve

<issue_title>DI Refactor: Azure.Mcp.Tools.AzureIsv</issue_title>
<issue_description>## Summary

Refactor Azure.Mcp.Tools.AzureIsv to use constructor dependency injection instead of resolving services via context.GetService<T>() in ExecuteAsync.

This is a sub-issue of #158.

Instructions

Follow the algorithm described in tools/di-refactor-plan.md with {ToolArea} = AzureIsv.

An example PR is: #1815</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…rcesListCommand

Co-authored-by: conniey <10136526+conniey@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor Azure.Mcp.Tools.AzureIsv for constructor DI DI Refactor: Azure.Mcp.Tools.AzureIsv Mar 2, 2026
@@ -0,0 +1,3 @@
changes:
Copy link
Member

Choose a reason for hiding this comment

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

@copilot Doublecheck step 5. PR number is missing. It does not look like New-ChangelogEntry script was used.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 933f58a — replaced the hand-crafted filename with a timestamp-based one (1772482319643.yaml) matching what New-ChangelogEntry.ps1 would generate, and added pr: 1902.

Co-authored-by: conniey <10136526+conniey@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.

DI Refactor: Azure.Mcp.Tools.AzureIsv

2 participants