DI Refactor: Azure.Mcp.Tools.Monitor

[Copilot]

Removes context.GetService<T>() usage from all Monitor command ExecuteAsync implementations, replacing with constructor-injected dependencies to improve testability and make dependencies explicit.

Commands refactored

• ActivityLogListCommand, ResourceLogQueryCommand, WorkspaceLogQueryCommand, TableListCommand, TableTypeListCommand, WorkspaceListCommand — now inject IMonitorService
• MetricsDefinitionsCommand, MetricsQueryCommand — now inject IMonitorMetricsService
• WebTestsCreateOrUpdateCommand, WebTestsGetCommand — now inject IMonitorWebTestService
• EntityGetHealthCommand — now injects IMonitorHealthModelService

Before / After

// Before
public sealed class WorkspaceListCommand(ILogger<WorkspaceListCommand> logger) : ...
{
    public override async Task<CommandResponse> ExecuteAsync(...)
    {
        var monitorService = context.GetService<IMonitorService>(); // resolved at call time
        ...
    }
}

// After
public sealed class WorkspaceListCommand(ILogger<WorkspaceListCommand> logger, IMonitorService monitorService) : ...
{
    public override async Task<CommandResponse> ExecuteAsync(...)
    {
        var workspaces = await monitorService.ListWorkspaces(...); // injected at construction
        ...
    }
}

Tests

Unit test constructors updated to pass mocked services directly to commands instead of registering them in a ServiceCollection. Removed unused McpServer field/import from EntityGetHealthCommandTests.

Invoking Livetests

Copilot submitted PRs are not trustworthy by default. Users with write access to the repo need to validate the contents of this PR before leaving a comment with the text /azp run mcp - pullrequest - live. This will trigger the necessary livetest workflows to complete required validation.

Original prompt

---

This section details on the original issue you should resolve

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

Refactor Azure.Mcp.Tools.Monitor 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} = Monitor.

An example PR is: #1815
</issue_description>

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

• Fixes DI Refactor: Azure.Mcp.Tools.Monitor #1862

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.